fix

Author: MikailBag

ci/windows.ps1

@@ -4,7 +4,7 @@ Write-Output "::group::Info"
 Write-Output "Target: $env:CI_TARGET"
 Write-Output "::group::Running tests"
 
-$env:RUST_BACKTRACE = full
+$env:RUST_BACKTRACE = "full"
 ./tests/$env:CI_TARGET/minion-tests.exe --trace
 if ($LASTEXITCODE -ne 0) {
     throw "tests failed"

src/windows/child.rs

@@ -35,9 +35,11 @@ impl WindowsChildProcess {
                 cwd: options.pwd.into(),
             },
         )?;
+        let child = OwnedHandle::new(info.hProcess);
+        options.sandbox.job.add_process(&child)?;
 
         Ok(WindowsChildProcess {
-            child: OwnedHandle::new(info.hProcess),
+            child,
             main_thread: OwnedHandle::new(info.hThread),
             stdin: child_stdin,
             stdout: child_stdout,

src/windows/constrain.rs

@@ -68,11 +68,11 @@ impl Job {
     pub(crate) fn kill(&self) -> Result<(), Error> {
         unsafe { Cvt::nonzero(TerminateJobObject(self.handle.as_raw(), 0xDEADBEEF)).map(|_| ()) }
     }
-    pub(crate) fn add_process(&self, process_handle: HANDLE) -> Result<(), Error> {
+    pub(crate) fn add_process(&self, process_handle: &OwnedHandle) -> Result<(), Error> {
         unsafe {
             Cvt::nonzero(AssignProcessToJobObject(
                 self.handle.as_raw(),
-                process_handle,
+                process_handle.as_raw(),
             ))
             .map(|_| ())
         }