add failure test & bump version

Signed-off-by: Bugen Zhao <[email protected]>
risinglightdb · Sep 15, 2023 · 2c78e58 · 2c78e58
1 parent 4b2ac70
commit 2c78e58
Show file tree

Hide file tree

Showing 7 changed files with 23 additions and 8 deletions.
diff --git a/Cargo.lock b/Cargo.lock
diff --git a/Cargo.toml b/Cargo.toml
@@ -2,7 +2,7 @@
 members = ["sqllogictest", "sqllogictest-bin", "sqllogictest-engines", "tests"]
 
 [workspace.package]
-version = "0.15.3"
+version = "0.16.0"
 edition = "2021"
 homepage = "https://github.com/risinglightdb/sqllogictest-rs"
 keywords = ["sql", "database", "parser", "cli"]

diff --git a/sqllogictest-bin/Cargo.toml b/sqllogictest-bin/Cargo.toml
@@ -24,8 +24,8 @@ glob = "0.3"
 itertools = "0.11"
 quick-junit = { version = "0.3" }
 rand = "0.8"
-sqllogictest = { path = "../sqllogictest", version = "0.15" }
-sqllogictest-engines = { path = "../sqllogictest-engines", version = "0.15" }
+sqllogictest = { path = "../sqllogictest", version = "0.16" }
+sqllogictest-engines = { path = "../sqllogictest-engines", version = "0.16" }
 tokio = { version = "1", features = [
     "rt",
     "rt-multi-thread",

diff --git a/sqllogictest-engines/Cargo.toml b/sqllogictest-engines/Cargo.toml
@@ -19,7 +19,7 @@ postgres-types = { version = "0.2.5", features = ["derive", "with-chrono-0_4"] }
 rust_decimal = { version = "1.30.0", features = ["tokio-pg"] }
 serde = { version = "1", features = ["derive"] }
 serde_json = "1"
-sqllogictest = { path = "../sqllogictest", version = "0.15" }
+sqllogictest = { path = "../sqllogictest", version = "0.16" }
 thiserror = "1"
 tokio = { version = "1", features = [
     "rt",

diff --git a/sqllogictest/src/runner.rs b/sqllogictest/src/runner.rs
@@ -613,7 +613,7 @@ impl<D: AsyncDB, M: MakeConnection<Conn = D>> Runner<D, M> {
                 let result = D::run_command(cmd).await;
 
                 #[derive(thiserror::Error, Debug)]
-                #[error("external command exited unsuccessfully: {0}")]
+                #[error("process exited unsuccessfully: {0}")] // message from unstable `ExitStatusError`
                 struct SystemError(ExitStatus);
 
                 let error: Option<Arc<dyn std::error::Error + Send + Sync>> = match result {

diff --git a/tests/system_command/system_command.rs b/tests/system_command/system_command.rs
@@ -47,3 +47,16 @@ fn test() {
         .run_file("./system_command/system_command.slt")
         .unwrap();
 }
+
+#[test]
+fn test_fail() {
+    let mut tester = sqllogictest::Runner::new(|| async { Ok(FakeDB) });
+    // enable `__TEST_DIR__` override
+    tester.enable_testdir();
+
+    let err = tester
+        .run_file("./system_command/system_command_fail.slt")
+        .unwrap_err();
+
+    assert!(err.to_string().contains("system command failed"), "{err}");
+}
diff --git a/tests/system_command/system_command_fail.slt b/tests/system_command/system_command_fail.slt
@@ -0,0 +1,2 @@
+system ok
+__TEST_DIR__/something_than_not_exist
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1,2 @@
		system ok
		__TEST_DIR__/something_than_not_exist