fix(runtime): ignored tests should not cause permission changes (#11278)

cli/tests/integration/test_tests.rs

@@ -31,6 +31,12 @@ itest!(ignore {
   output: "test/ignore.out",
 });
 
+itest!(ignore_permissions {
+  args: "test --unstable test/ignore_permissions.ts",
+  exit_code: 0,
+  output: "test/ignore_permissions.out",
+});
+
 itest!(fail {
   args: "test test/fail.ts",
   exit_code: 1,

cli/tests/test/ignore_permissions.out

@@ -0,0 +1,6 @@
+Check [WILDCARD]/test/ignore_permissions.ts
+running 1 test from [WILDCARD]/test/ignore_permissions.ts
+test ignore ... ignored ([WILDCARD])
+
+test result: ok. 0 passed; 0 failed; 1 ignored; 0 measured; 0 filtered out ([WILDCARD])
+

cli/tests/test/ignore_permissions.ts

@@ -0,0 +1,16 @@
+Deno.test({
+  name: "ignore",
+  permissions: {
+    read: true,
+    write: true,
+    net: true,
+    env: true,
+    run: true,
+    plugin: true,
+    hrtime: true,
+  },
+  ignore: true,
+  fn() {
+    throw new Error("unreachable");
+  },
+});

runtime/js/40_testing.js

@@ -192,17 +192,16 @@ finishing test case.`;
   }
 
   async function runTest({ ignore, fn, permissions }) {
+    if (ignore) {
+      return "ignored";
+    }
+
     let token = null;
 
     try {
       if (permissions) {
         token = pledgeTestPermissions(permissions);
       }
-
-      if (ignore) {
-        return "ignored";
-      }
-
       await fn();
 
       return "ok";