forkpty: Fix test on BSDs

test/test_pty.rs

@@ -3,13 +3,11 @@ use std::path::Path;
 use std::os::unix::prelude::*;
 use tempfile::tempfile;
 
-use libc::_exit;
 use nix::fcntl::{OFlag, open};
 use nix::pty::*;
 use nix::sys::stat;
 use nix::sys::termios::*;
-use nix::sys::wait::wait;
-use nix::unistd::{write, close};
+use nix::unistd::{write, close, pause};
 
 /// Regression test for Issue #659
 /// This is the correct way to explicitly close a `PtyMaster`
@@ -207,6 +205,8 @@ fn test_openpty_with_termios() {
 #[test]
 fn test_forkpty() {
     use nix::unistd::ForkResult::*;
+    use nix::sys::signal::*;
+    use nix::sys::wait::wait;
     // forkpty calls openpty which uses ptname(3) internally.
     let _m0 = ::PTSNAME_MTX.lock().expect("Mutex got poisoned by another test");
     // forkpty spawns a child process
@@ -218,13 +218,14 @@ fn test_forkpty() {
     match pty.fork_result {
         Child => {
             write(0, string.as_bytes()).unwrap();
-            unsafe { _exit(0) }
+            pause();  // we need the child to stay alive until the parent calls read
         },
         Parent { child } => {
-            wait().unwrap(); // keep other tests using generic wait from getting our child
             let mut buf = [0u8; 10];
             assert!(child.as_raw() > 0);
             ::read_exact(pty.master, &mut buf);
+            kill(child, SIGTERM).unwrap();
+            wait().unwrap(); // keep other tests using generic wait from getting our child
             assert_eq!(&buf, echoed_string.as_bytes());
             close(pty.master).unwrap();
         },