chore(specs): add jsr and npm env vars to tests by default (#23171)

tests/specs/bench/filter_group_header/main.out

@@ -7,7 +7,7 @@ benchmark      time (avg)        iter/s             (min … max)       p75
 --------------------------------------------------------------- -----------------------------
 
 group G1
-G1-B          [WILDCARD]
+G1-B         [WILDCARD]
 
 group G2
-G2-B          [WILDCARD]
+G2-B         [WILDCARD]

tests/specs/check/cjs_default_export/__test__.jsonc

@@ -1,5 +1,4 @@
 {
-  "base": "npm",
   "args": "check main.ts",
   "output": "main.out",
   "exitCode": 1

tests/specs/jsr/deps/__test__.jsonc

@@ -1,5 +1,4 @@
 {
-  "base": "jsr",
   "steps": [{
     "args": "run main.ts",
     "output": "main.out"

tests/specs/jsr/import_https_url/__test__.jsonc

@@ -1,5 +1,4 @@
 {
-  "base": "jsr",
   "steps": [{
     "args": "run -A analyzable.ts",
     "output": "analyzable.out",

tests/specs/jsr/jsx_with_no_pragmas/__test__.jsonc

@@ -1,5 +1,4 @@
 {
-  "base": "jsr",
   "args": "run main.ts",
   "output": "main.out",
   "exitCode": 1

tests/specs/jsr/jsx_with_pragmas/__test__.jsonc

@@ -1,5 +1,4 @@
 {
-  "base": "jsr",
   "args": "run main.ts",
   "output": "main.out"
 }

tests/specs/jsr/module_graph/__test__.jsonc

@@ -1,5 +1,4 @@
 {
-  "base": "jsr",
   "steps": [{
     "args": "run main.ts",
     "output": "main.out"

tests/specs/jsr/no_module_graph/__test__.jsonc

@@ -1,5 +1,4 @@
 {
-  "base": "jsr",
   "steps": [{
     "args": "run main.ts",
     "output": "main.out"

tests/specs/jsr/no_unused_params/__test__.jsonc

@@ -1,5 +1,4 @@
 {
-  "base": "jsr",
   "steps": [{
     "args": "check --all main.ts",
     "output": "main.out"

tests/specs/jsr/subset_type_graph/__test__.jsonc

@@ -1,5 +1,4 @@
 {
-  "base": "jsr",
   "args": "check --all main.ts",
   "output": "main.check.out",
   "exitCode": 1

tests/specs/jsr/version_not_found/__test__.jsonc

@@ -1,5 +1,4 @@
 {
-  "base": "jsr",
   "args": "run -A main.ts",
   "output": "main.out",
   "exitCode": 1

tests/specs/mod.rs

@@ -101,15 +101,12 @@ fn run_test(test: &Test, diagnostic_logger: Rc<RefCell<Vec<u8>>>) {
     builder = builder.cwd(cwd.to_string_lossy());
   }
 
-  if let Some(base) = &metadata.base {
-    match base.as_str() {
-      "npm" => {
-        builder = builder.add_npm_env_vars();
-      }
-      "jsr" => {
-        builder = builder.add_jsr_env_vars().add_npm_env_vars();
-      }
-      _ => panic!("Unknown test base: {}", base),
+  match &metadata.base {
+    // todo(dsherret): add bases in the future as needed
+    Some(base) => panic!("Unknown test base: {}", base),
+    None => {
+      // by default add npm and jsr env vars
+      builder = builder.add_jsr_env_vars().add_npm_env_vars();
     }
   }
 

tests/specs/npm/check_pkg_json_import/__test__.jsonc

@@ -1,5 +1,4 @@
 {
-  "base": "npm",
   "args": "check --all main.ts",
   "output": "main.out"
 }

tests/specs/npm/conditional_exports/__test__.jsonc

@@ -1,5 +1,4 @@
 {
-  "base": "npm",
   "args": "run --allow-read main.js",
   "output": "main.out"
 }

tests/specs/npm/conditional_exports_node_modules_dir/__test__.jsonc

@@ -1,5 +1,4 @@
 {
-  "base": "npm",
   "tempDir": true,
   "args": "run --allow-read --node-modules-dir main.js",
   "output": "main.out"

tests/specs/npm/es_module/__test__.jsonc

@@ -1,5 +1,4 @@
 {
-  "base": "npm",
   "steps": [{
     "args": "run --allow-read --allow-env main.js",
     "output": "main.out"