test: fix macos tests

ecyrbe · Mar 1, 2020 · e537177 · e537177
1 parent 947653f
commit e537177
Show file tree

Hide file tree

Showing 2 changed files with 36 additions and 7 deletions.
diff --git a/cli/priority.rs b/cli/priority.rs
@@ -149,13 +149,20 @@ mod tests {
     assert_eq!(priority, PRIORITY_BELOW_NORMAL);
   }
 
+  #[cfg(not(target_os = "macos"))]
   #[test]
   fn test_set_current_process_priority_to_normal() {
     set_priority(0, PRIORITY_NORMAL).expect("Should set priority");
     let priority = get_priority(0).expect("Should get priority");
     assert_eq!(priority, PRIORITY_NORMAL);
   }
 
+  #[cfg(target_os = "macos")]
+  #[test]
+  fn test_set_current_process_normal_priority_should_fail() {
+    assert!(set_priority(0, PRIORITY_NORMAL).is_err());
+  }
+
   #[cfg(unix)]
   #[test]
   fn test_set_current_process_high_priority_should_fail() {

diff --git a/std/node/os_test.ts b/std/node/os_test.ts
@@ -60,13 +60,35 @@ test({
   }
 });
 
-test({
-  name: "setPriority(): should set current process priority to normal",
-  fn() {
-    os.setPriority(os.constants.priority.PRIORITY_NORMAL);
-    assertEquals(os.getPriority(), os.constants.priority.PRIORITY_NORMAL);
-  }
-});
+if (os.platform() !== "darwin")
+  test({
+    name: "setPriority(): should set current process priority to normal",
+    fn() {
+      os.setPriority(os.constants.priority.PRIORITY_NORMAL);
+      assertEquals(os.getPriority(), os.constants.priority.PRIORITY_NORMAL);
+    }
+  });
+
+if (os.platform() === "win32")
+  test({
+    name: "setPriority(): should set current process priority to above normal",
+    fn() {
+      os.setPriority(os.constants.priority.PRIORITY_ABOVE_NORMAL);
+      assertEquals(
+        os.getPriority(),
+        os.constants.priority.PRIORITY_ABOVE_NORMAL
+      );
+    }
+  });
+
+if (os.platform() === "win32")
+  test({
+    name: "setPriority(): should set current process priority to high",
+    fn() {
+      os.setPriority(os.constants.priority.PRIORITY_HIGH);
+      assertEquals(os.getPriority(), os.constants.priority.PRIORITY_HIGH);
+    }
+  });
 
 test({
   name: "getPriority(): PID must be a 32 bit integer",