refactor(std): Uncomment disabled tests, use skip option (#4378)

1 files changed, 44 insertions, 42 deletions

diff --git a/std/node/_fs/_fs_chown_test.ts b/std/node/_fs/_fs_chown_test.ts
index 51a463d88..f7b4a8cec 100644
--- a/std/node/_fs/_fs_chown_test.ts
+++ b/std/node/_fs/_fs_chown_test.ts
@@ -3,48 +3,50 @@ const { test } = Deno;
 import { fail, assertEquals } from "../../testing/asserts.ts";
 import { chown, chownSync } from "./_fs_chown.ts";
 
-if (Deno.build.os !== "win") {
-  //chown is difficult to test.  Best we can do is set the existing user id/group id again
-  test({
-    name: "ASYNC: setting existing uid/gid works as expected (non-Windows)",
-    async fn() {
-      const tempFile: string = await Deno.makeTempFile();
-      const originalUserId: number | null = (await Deno.lstat(tempFile)).uid;
-      const originalGroupId: number | null = (await Deno.lstat(tempFile)).gid;
-      await new Promise((resolve, reject) => {
-        chown(tempFile, originalUserId!, originalGroupId!, err => {
-          if (err) reject(err);
-          else resolve();
-        });
+//chown is difficult to test.  Best we can do is set the existing user id/group id again
+const skip = Deno.build.os == "win";
+
+test({
+  skip,
+  name: "ASYNC: setting existing uid/gid works as expected (non-Windows)",
+  async fn() {
+    const tempFile: string = await Deno.makeTempFile();
+    const originalUserId: number | null = (await Deno.lstat(tempFile)).uid;
+    const originalGroupId: number | null = (await Deno.lstat(tempFile)).gid;
+    await new Promise((resolve, reject) => {
+      chown(tempFile, originalUserId!, originalGroupId!, err => {
+        if (err) reject(err);
+        else resolve();
+      });
+    })
+      .then(() => {
+        const newUserId: number | null = Deno.lstatSync(tempFile).uid;
+        const newGroupId: number | null = Deno.lstatSync(tempFile).gid;
+        assertEquals(newUserId, originalUserId);
+        assertEquals(newGroupId, originalGroupId);
+      })
+      .catch(() => {
+        fail();
       })
-        .then(() => {
-          const newUserId: number | null = Deno.lstatSync(tempFile).uid;
-          const newGroupId: number | null = Deno.lstatSync(tempFile).gid;
-          assertEquals(newUserId, originalUserId);
-          assertEquals(newGroupId, originalGroupId);
-        })
-        .catch(() => {
-          fail();
-        })
-        .finally(() => {
-          Deno.removeSync(tempFile);
-        });
-    }
-  });
+      .finally(() => {
+        Deno.removeSync(tempFile);
+      });
+  }
+});
 
-  test({
-    name: "SYNC: setting existing uid/gid works as expected (non-Windows)",
-    fn() {
-      const tempFile: string = Deno.makeTempFileSync();
-      const originalUserId: number | null = Deno.lstatSync(tempFile).uid;
-      const originalGroupId: number | null = Deno.lstatSync(tempFile).gid;
-      chownSync(tempFile, originalUserId!, originalGroupId!);
+test({
+  skip,
+  name: "SYNC: setting existing uid/gid works as expected (non-Windows)",
+  fn() {
+    const tempFile: string = Deno.makeTempFileSync();
+    const originalUserId: number | null = Deno.lstatSync(tempFile).uid;
+    const originalGroupId: number | null = Deno.lstatSync(tempFile).gid;
+    chownSync(tempFile, originalUserId!, originalGroupId!);
 
-      const newUserId: number | null = Deno.lstatSync(tempFile).uid;
-      const newGroupId: number | null = Deno.lstatSync(tempFile).gid;
-      assertEquals(newUserId, originalUserId);
-      assertEquals(newGroupId, originalGroupId);
-      Deno.removeSync(tempFile);
-    }
-  });
-}
+    const newUserId: number | null = Deno.lstatSync(tempFile).uid;
+    const newGroupId: number | null = Deno.lstatSync(tempFile).gid;
+    assertEquals(newUserId, originalUserId);
+    assertEquals(newGroupId, originalGroupId);
+    Deno.removeSync(tempFile);
+  }
+});