Fix test hangs on macos. (#1170)

In the `waitpid` tests, explicitly terminate the child processe and
wait for them to exit, as dropping a `Command` otherwise leaves the
process running. This fixes test hangs on macos.

Author: sunfishcode

tests/process/wait.rs

@@ -22,6 +22,15 @@ fn test_waitpid_none() {
         .expect("failed to wait")
         .unwrap();
     assert!(status.stopped());
+
+    // Clean up the child process.
+    unsafe { kill(child.id() as _, SIGKILL) };
+
+    let (pid, status) = process::waitpid(None, process::WaitOptions::UNTRACED)
+        .expect("failed to wait")
+        .unwrap();
+    assert_eq!(pid, process::Pid::from_child(&child));
+    assert!(status.signaled());
 }
 
 #[test]
@@ -39,6 +48,15 @@ fn test_waitpid_some() {
         .expect("failed to wait")
         .unwrap();
     assert!(status.stopped());
+
+    // Clean up the child process.
+    unsafe { kill(child.id() as _, SIGKILL) };
+
+    let (rpid, status) = process::waitpid(Some(pid), process::WaitOptions::UNTRACED)
+        .expect("failed to wait")
+        .unwrap();
+    assert_eq!(rpid, pid);
+    assert!(status.signaled());
 }
 
 #[test]
@@ -56,6 +74,15 @@ fn test_waitpgid() {
         .expect("failed to wait")
         .unwrap();
     assert!(status.stopped());
+
+    // Clean up the child process.
+    unsafe { kill(child.id() as _, SIGKILL) };
+
+    let (pid, status) = process::waitpgid(pgid, process::WaitOptions::UNTRACED)
+        .expect("failed to wait")
+        .unwrap();
+    assert_eq!(pid, process::Pid::from_child(&child));
+    assert!(status.signaled());
 }
 
 #[cfg(not(any(