Fixes for snapshot tests

- Fixed the unit tests for dash and bash subprocesses
- Fixed the order of the flushing of files so that processes are notified of exit only after the open file handles are flushed

lib/wasi/src/bin_factory/exec.rs

@@ -177,11 +177,12 @@ pub fn spawn_exec_module(
                 } else {
                     Errno::Success.into()
                 };
-                debug!("wasi[{pid}]::main() has exited with {code}");
-                thread.thread.set_status_finished(ret.map(|a| a.into()));
 
                 // Cleanup the environment
                 wasi_env.data(&store).blocking_cleanup(Some(code));
+
+                debug!("wasi[{pid}]::main() has exited with {code}");
+                thread.thread.set_status_finished(ret.map(|a| a.into()));
             }
         };
 

lib/wasi/tests/runners.rs

@@ -51,7 +51,7 @@ mod wasi {
             WasiError::Exit(code) => *code,
             other => unreachable!("Something else went wrong: {:?}", other),
         };
-        assert_eq!(exit_code, 1);
+        assert_eq!(exit_code.is_success(), true);
     }
 
     #[tokio::test]
@@ -77,7 +77,7 @@ mod wasi {
             WasiError::Exit(code) => *code,
             other => unreachable!("Something else went wrong: {:?}", other),
         };
-        assert_eq!(exit_code, 42);
+        assert_eq!(exit_code.raw(), 42);
     }
 }
 

tests/integration/cli/tests/snapshot.rs

@@ -546,7 +546,7 @@ fn test_snapshot_dash_dash() {
     let snapshot = TestBuilder::new()
         .with_name(function!())
         .use_dash()
-        .stdin_str("/bin/dash\necho hi\nexit\necho hi2\n")
+        .stdin_str("/bin/dash\necho hi\nexit\nexit\n")
         .run_wasm(include_bytes!("./wasm/dash.wasm"));
     assert_json_snapshot!(snapshot);
 }
@@ -556,7 +556,7 @@ fn test_snapshot_dash_bash() {
     let snapshot = TestBuilder::new()
         .with_name(function!())
         .use_bash()
-        .stdin_str("/bin/bash\necho hi\nexit\necho hi2\n")
+        .stdin_str("/bin/bash\necho hi\nexit\nexit\n")
         .run_wasm(include_bytes!("./wasm/dash.wasm"));
     assert_json_snapshot!(snapshot);
 }
@@ -617,7 +617,7 @@ fn test_snapshot_bash_dash() {
     let snapshot = TestBuilder::new()
         .with_name(function!())
         .use_dash()
-        .stdin_str("/bin/dash\necho hi\nexit\necho hi2\n")
+        .stdin_str("/bin/dash\necho hi\nexit\nexit\n")
         .run_wasm(include_bytes!("./wasm/bash.wasm"));
     assert_json_snapshot!(snapshot);
 }

tests/integration/cli/tests/snapshots/snapshot__snapshot_bash_bash.snap

@@ -0,0 +1,57 @@
+---
+source: tests/integration/cli/tests/snapshot.rs
+assertion_line: 594
+expression: snapshot
+---
+{
+  "spec": {
+    "name": "snapshot::test_snapshot_bash_bash",
+    "use_packages": [
+      "sharrattj/bash"
+    ],
+    "cli_args": [],
+    "stdin": [
+      47,
+      98,
+      105,
+      110,
+      47,
+      98,
+      97,
+      115,
+      104,
+      10,
+      101,
+      99,
+      104,
+      111,
+      32,
+      104,
+      105,
+      10,
+      101,
+      120,
+      105,
+      116,
+      10,
+      101,
+      99,
+      104,
+      111,
+      32,
+      104,
+      105,
+      50,
+      10
+    ],
+    "debug_output": false,
+    "enable_threads": true
+  },
+  "result": {
+    "Success": {
+      "stdout": "hi\nhi2\n",
+      "stderr": "test.wasm-5.1# bash-5.1# bash-5.1# exit\ntest.wasm-5.1# test.wasm-5.1# exit\n",
+      "exit_code": 0
+    }
+  }
+}

tests/integration/cli/tests/snapshots/snapshot__snapshot_bash_dash.snap

@@ -0,0 +1,53 @@
+---
+source: tests/integration/cli/tests/snapshot.rs
+assertion_line: 622
+expression: snapshot
+---
+{
+  "spec": {
+    "name": "snapshot::test_snapshot_bash_dash",
+    "use_packages": [
+      "sharrattj/dash"
+    ],
+    "cli_args": [],
+    "stdin": [
+      47,
+      98,
+      105,
+      110,
+      47,
+      100,
+      97,
+      115,
+      104,
+      10,
+      101,
+      99,
+      104,
+      111,
+      32,
+      104,
+      105,
+      10,
+      101,
+      120,
+      105,
+      116,
+      10,
+      101,
+      120,
+      105,
+      116,
+      10
+    ],
+    "debug_output": false,
+    "enable_threads": true
+  },
+  "result": {
+    "Success": {
+      "stdout": "hi\n",
+      "stderr": "test.wasm-5.1# # # test.wasm-5.1# exit\n",
+      "exit_code": 78
+    }
+  }
+}

tests/integration/cli/tests/snapshots/snapshot__snapshot_dash_bash.snap

@@ -0,0 +1,53 @@
+---
+source: tests/integration/cli/tests/snapshot.rs
+assertion_line: 561
+expression: snapshot
+---
+{
+  "spec": {
+    "name": "snapshot::test_snapshot_dash_bash",
+    "use_packages": [
+      "sharrattj/bash"
+    ],
+    "cli_args": [],
+    "stdin": [
+      47,
+      98,
+      105,
+      110,
+      47,
+      98,
+      97,
+      115,
+      104,
+      10,
+      101,
+      99,
+      104,
+      111,
+      32,
+      104,
+      105,
+      10,
+      101,
+      120,
+      105,
+      116,
+      10,
+      101,
+      120,
+      105,
+      116,
+      10
+    ],
+    "debug_output": false,
+    "enable_threads": true
+  },
+  "result": {
+    "Success": {
+      "stdout": "hi\n",
+      "stderr": "# bash-5.1# exit\n# # ",
+      "exit_code": 78
+    }
+  }
+}

tests/integration/cli/tests/snapshots/snapshot__snapshot_dash_dash.snap

@@ -0,0 +1,53 @@
+---
+source: tests/integration/cli/tests/snapshot.rs
+assertion_line: 551
+expression: snapshot
+---
+{
+  "spec": {
+    "name": "snapshot::test_snapshot_dash_dash",
+    "use_packages": [
+      "sharrattj/dash"
+    ],
+    "cli_args": [],
+    "stdin": [
+      47,
+      98,
+      105,
+      110,
+      47,
+      100,
+      97,
+      115,
+      104,
+      10,
+      101,
+      99,
+      104,
+      111,
+      32,
+      104,
+      105,
+      10,
+      101,
+      120,
+      105,
+      116,
+      10,
+      101,
+      120,
+      105,
+      116,
+      10
+    ],
+    "debug_output": false,
+    "enable_threads": true
+  },
+  "result": {
+    "Success": {
+      "stdout": "hi\n",
+      "stderr": "# # \n# # ",
+      "exit_code": 78
+    }
+  }
+}