Merge pull request #278 from kinode-dao/hf/run-tests-improvements

`run-tests`: improvements
kinode-dao · Dec 28, 2024 · d15ab57 · d15ab57
2 parents 47cdf82 + 643fde5
commit d15ab57
Show file tree

Hide file tree

Showing 15 changed files with 17 additions and 4 deletions.
diff --git a/Cargo.lock b/Cargo.lock
diff --git a/Cargo.toml b/Cargo.toml
@@ -1,6 +1,6 @@
 [package]
 name = "kit"
-version = "0.8.5"
+version = "0.9.0"
 edition = "2021"
 
 [build-dependencies]

diff --git a/src/new/templates/javascript/no-ui/chat/test/tests.toml b/src/new/templates/javascript/no-ui/chat/test/tests.toml
@@ -2,6 +2,7 @@ runtime = { FetchVersion = "latest" }
 # runtime = { RepoPath = "~/git/kinode" }
 persist_home = false
 runtime_build_release = false
+always_print_node_output = false
 
 
 [[tests]]

diff --git a/src/new/templates/javascript/no-ui/echo/test/tests.toml b/src/new/templates/javascript/no-ui/echo/test/tests.toml
@@ -2,7 +2,7 @@ runtime = { FetchVersion = "latest" }
 # runtime = { RepoPath = "~/git/kinode" }
 persist_home = false
 runtime_build_release = false
-
+always_print_node_output = false
 
 [[tests]]
 dependency_package_paths = []

diff --git a/src/new/templates/javascript/no-ui/fibonacci/test/tests.toml b/src/new/templates/javascript/no-ui/fibonacci/test/tests.toml
@@ -2,6 +2,7 @@ runtime = { FetchVersion = "latest" }
 # runtime = { RepoPath = "~/git/kinode" }
 persist_home = false
 runtime_build_release = false
+always_print_node_output = false
 
 
 [[tests]]

diff --git a/src/new/templates/python/no-ui/chat/test/tests.toml b/src/new/templates/python/no-ui/chat/test/tests.toml
@@ -2,6 +2,7 @@ runtime = { FetchVersion = "latest" }
 # runtime = { RepoPath = "~/git/kinode" }
 persist_home = false
 runtime_build_release = false
+always_print_node_output = false
 
 
 [[tests]]

diff --git a/src/new/templates/python/no-ui/echo/test/tests.toml b/src/new/templates/python/no-ui/echo/test/tests.toml
@@ -2,6 +2,7 @@ runtime = { FetchVersion = "latest" }
 # runtime = { RepoPath = "~/git/kinode" }
 persist_home = false
 runtime_build_release = false
+always_print_node_output = false
 
 
 [[tests]]

diff --git a/src/new/templates/python/no-ui/fibonacci/test/tests.toml b/src/new/templates/python/no-ui/fibonacci/test/tests.toml
@@ -2,6 +2,7 @@ runtime = { FetchVersion = "latest" }
 # runtime = { RepoPath = "~/git/kinode" }
 persist_home = false
 runtime_build_release = false
+always_print_node_output = false
 
 
 [[tests]]

diff --git a/src/new/templates/rust/no-ui/chat/test/tests.toml b/src/new/templates/rust/no-ui/chat/test/tests.toml
@@ -2,6 +2,7 @@ runtime = { FetchVersion = "latest" }
 # runtime = { RepoPath = "~/git/kinode" }
 persist_home = false
 runtime_build_release = false
+always_print_node_output = false
 
 
 [[tests]]

diff --git a/src/new/templates/rust/no-ui/echo/test/tests.toml b/src/new/templates/rust/no-ui/echo/test/tests.toml
@@ -2,6 +2,7 @@ runtime = { FetchVersion = "latest" }
 # runtime = { RepoPath = "~/git/kinode" }
 persist_home = false
 runtime_build_release = false
+always_print_node_output = false
 
 
 [[tests]]

diff --git a/src/new/templates/rust/no-ui/fibonacci/test/tests.toml b/src/new/templates/rust/no-ui/fibonacci/test/tests.toml
@@ -2,6 +2,7 @@ runtime = { FetchVersion = "latest" }
 # runtime = { RepoPath = "~/git/kinode" }
 persist_home = false
 runtime_build_release = false
+always_print_node_output = false
 
 
 [[tests]]

diff --git a/src/new/templates/rust/no-ui/file-transfer/test/tests.toml b/src/new/templates/rust/no-ui/file-transfer/test/tests.toml
@@ -2,6 +2,7 @@ runtime = { FetchVersion = "latest" }
 # runtime = { RepoPath = "~/git/kinode" }
 persist_home = false
 runtime_build_release = false
+always_print_node_output = false
 
 
 [[tests]]

diff --git a/src/new/templates/rust/ui/chat/test/tests.toml b/src/new/templates/rust/ui/chat/test/tests.toml
@@ -2,6 +2,7 @@ runtime = { FetchVersion = "latest" }
 # runtime = { RepoPath = "~/git/kinode" }
 persist_home = false
 runtime_build_release = false
+always_print_node_output = false
 
 
 [[tests]]

diff --git a/src/run_tests/mod.rs b/src/run_tests/mod.rs
@@ -694,6 +694,7 @@ async fn handle_test(
     test: Test,
     test_dir_path: &Path,
     persist_home: bool,
+    always_print_node_output: bool,
 ) -> Result<()> {
     let (setup_packages, test_package_paths) = build_packages(
         &test,
@@ -801,7 +802,7 @@ async fn handle_test(
         info!("PASS");
     }
 
-    let _ = send_to_cleanup.send(tests_result.is_err());
+    let _ = send_to_cleanup.send(always_print_node_output || tests_result.is_err());
     for handle in task_handles {
         handle.await.unwrap();
     }
@@ -872,6 +873,7 @@ pub async fn execute(config_path: PathBuf) -> Result<()> {
             test,
             &test_dir_path,
             config.persist_home,
+            config.always_print_node_output,
         )
         .await?;
     }

diff --git a/src/run_tests/types.rs b/src/run_tests/types.rs
@@ -12,6 +12,7 @@ pub struct Config {
     pub runtime: Runtime,
     pub runtime_build_release: bool,
     pub persist_home: bool,
+    pub always_print_node_output: bool,
     pub tests: Vec<Test>,
 }