use std::env::temp_dir() to retrieve the temp directory in test_mkstemp

Author: ndusart

test/test_unistd.rs

@@ -4,7 +4,7 @@ use nix::unistd::*;
 use nix::unistd::ForkResult::*;
 use nix::sys::wait::*;
 use nix::sys::stat;
-use std::iter;
+use std::{env, iter};
 use std::ffi::CString;
 use std::fs::File;
 use std::io::Write;
@@ -64,27 +64,23 @@ fn test_wait() {
 
 #[test]
 fn test_mkstemp() {
-    #[cfg(target_os = "android")]
-    let tmp = "/data/local/tmp/";
-    #[cfg(not(target_os = "android"))]
-    let tmp = "/tmp/";
+    let mut path = env::temp_dir();
+    path.push("nix_tempfile.XXXXXX");
 
-    let result = mkstemp((tmp.to_owned() + "nix_tempfile.XXXXXX").as_str());
+    let result = mkstemp(&path);
     match result {
         Ok((fd, path)) => {
             close(fd).unwrap();
             unlink(path.as_path()).unwrap();
         },
         Err(e) => panic!("mkstemp failed: {}", e)
     }
+}
 
-    let result = mkstemp(tmp);
-    match result {
-        Ok(_) => {
-            panic!("mkstemp succeeded even though it should fail (provided a directory)");
-        },
-        Err(_) => {}
-    }
+#[test]
+fn test_mkstemp_directory() {
+    // mkstemp should fail if a directory is given
+    assert!(mkstemp(&env::temp_dir()).is_err());
 }
 
 #[test]