You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maka.apache.org by wy...@apache.org on 2026/09/23 21:09:10 UTC

(maka) 05/05: test(cli): await finite activation cleanup before releasing the writer

This is an automated email from the ASF dual-hosted git repository.

M4n5ter pushed a commit to branch feat/runtime-host-rust
in repository https://gitbox.apache.org/repos/asf/maka.git

commit a66604c8d1d2b67d1176e94766063cc284865487
Author: M4n5ter <m4...@gmail.com>
AuthorDate: Thu Sep 24 05:04:25 2026 +0800

    test(cli): await finite activation cleanup before releasing the writer
    
    Verify the worker deadline and idle executor before subsequent deployment operations; observer timeout alone is not evidence of cleanup.
    
    Generated-by: Codex
---
 crates/cli/tests/integration/deployment.rs | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/crates/cli/tests/integration/deployment.rs b/crates/cli/tests/integration/deployment.rs
index adfc1165ca..41bfdef657 100644
--- a/crates/cli/tests/integration/deployment.rs
+++ b/crates/cli/tests/integration/deployment.rs
@@ -99,10 +99,13 @@ fn managed_installation_pins_code_before_migration_and_preserves_live_authority(
         {
             // A live but unreachable writer is not a stale lock. The command
             // must leave on its own deadline without taking over that Root.
+            // Run the finite worker directly: its exit, unlike the observer's
+            // timeout, proves cleanup before we release this writer or mutate.
             let owner = RootOwner::open(&fixture.root, &namespaces).unwrap();
             let started = Instant::now();
             let waiting = Command::new(executable)
                 .args([
+                    "--operation-worker",
                     "host",
                     "activate",
                     "--root-id",
@@ -114,6 +117,12 @@ fn managed_installation_pins_code_before_migration_and_preserves_live_authority(
                 .unwrap();
             assert!(!waiting.status.success());
             assert!(started.elapsed() < Duration::from_secs(3));
+            assert!(
+                String::from_utf8_lossy(&waiting.stderr)
+                    .contains("Host did not become Ready before the activation deadline"),
+                "{waiting:?}"
+            );
+            assert_eq!(query("status")["operation"], "idle");
             assert!(RootOwner::open(&fixture.root, &namespaces).is_err());
             assert_eq!(query("status")["deployment"], installed);
             drop(owner);


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@maka.apache.org
For additional commands, e-mail: commits-help@maka.apache.org