update

Author: dsherret

runtime/permissions/lib.rs

@@ -842,6 +842,11 @@ impl<
   ) -> PermissionState {
     if self.is_flag_denied(desc) || self.is_prompt_denied(desc) {
       PermissionState::Denied
+    } else if desc
+      .map(|desc| self.flag_ignored_list.iter().any(|v| desc.matches_deny(v)))
+      .unwrap_or(false)
+    {
+      PermissionState::Ignored
     } else if self.is_granted(desc) {
       match allow_partial {
         AllowPartial::TreatAsGranted => PermissionState::Granted,
@@ -860,7 +865,7 @@ impl<
           }
         }
       }
-    } else if self.is_flag_ignored(desc) {
+    } else if self.flag_ignored_global {
       PermissionState::Ignored
     } else if matches!(allow_partial, AllowPartial::TreatAsDenied)
       && self.is_partial_flag_denied(desc)
@@ -957,16 +962,6 @@ impl<
     }
   }
 
-  fn is_flag_ignored(&self, query: Option<&TAllowDesc::QueryDesc<'_>>) -> bool {
-    match query {
-      Some(query) => {
-        self.flag_ignored_global
-          || self.flag_ignored_list.iter().any(|v| query.matches_deny(v))
-      }
-      None => self.flag_ignored_global,
-    }
-  }
-
   fn is_prompt_denied(
     &self,
     query: Option<&TAllowDesc::QueryDesc<'_>>,

tests/specs/permission/ignore_env/__test__.jsonc

@@ -0,0 +1,21 @@
+{
+  "envs": {
+    "VAR1": "1",
+    "VAR2": "2"
+  },
+  "tests": {
+    "all": {
+      "args": "run --ignore-env main.ts",
+      "output": "undefined\nundefined\n[Object: null prototype] {}\n"
+    },
+    "some": {
+      "args": "run --ignore-env=VAR1 main.ts",
+      "output": "undefined\nerror: [WILDCARD]\n",
+      "exitCode": 1
+    },
+    "some_allow_env": {
+      "args": "run --ignore-env=VAR1 --allow-env main.ts",
+      "output": "undefined\n2\n"
+    }
+  }
+}

tests/specs/permission/ignore_env/main.ts

@@ -0,0 +1,3 @@
+console.log(Deno.env.get("VAR1"));
+console.log(Deno.env.get("VAR2"));
+console.log(Deno.env.toObject());