Move pipe related test cases into one

to avoid setting pipefail flags concurrently

Author: tao-guo

tests/test_macros.rs

@@ -150,6 +150,14 @@ fn test_pipe() {
 
     let wc_cmd = "wc";
     assert!(run_cmd!(ls | $wc_cmd).is_ok());
+
+    // test pipefail
+    assert!(run_cmd!(false | true).is_err());
+    assert!(run_fun!(false | true).is_err());
+    assert!(run_fun!(ignore false | true).is_ok());
+    set_pipefail(false);
+    assert!(run_fun!(ignore false | true).is_ok());
+    set_pipefail(true);
 }
 
 #[test]
@@ -255,13 +263,3 @@ fn test_empty_arg() {
     let opt = "";
     assert!(run_cmd!(ls $opt).is_ok());
 }
-
-#[test]
-fn test_pipefail_in_fun() {
-    assert!(run_cmd!(false | true).is_err());
-    assert!(run_fun!(false | true).is_err());
-    assert!(run_fun!(ignore false | true).is_ok());
-    set_pipefail(false);
-    assert!(run_fun!(ignore false | true).is_ok());
-    set_pipefail(true);
-}