chore: add test for importing json file in npm package (#30072)

cli/lib/loader.rs

@@ -14,16 +14,15 @@ pub fn module_type_from_media_and_requested_type(
   requested_module_type: &RequestedModuleType,
 ) -> ModuleType {
   match requested_module_type {
-    RequestedModuleType::Json => ModuleType::Json,
     RequestedModuleType::Text => ModuleType::Text,
     RequestedModuleType::Bytes => ModuleType::Bytes,
-    RequestedModuleType::None | RequestedModuleType::Other(_) => {
-      match media_type {
+    RequestedModuleType::None
+    | RequestedModuleType::Other(_)
+    | RequestedModuleType::Json => match media_type {
       MediaType::Json => ModuleType::Json,
       MediaType::Wasm => ModuleType::Wasm,
       _ => ModuleType::JavaScript,
-      }
-    }
+    },
   }
 }
 

tests/specs/npm/json_import/__test__.jsonc

@@ -0,0 +1,14 @@
+{
+  "tempDir": true,
+  "tests": {
+    "has_attribute": {
+      "args": "run has_attribute.ts",
+      "output": "[WILDCARD]"
+    },
+    "missing_attribute": {
+      "args": "run missing_attribute.ts",
+      "output": "missing.out",
+      "exitCode": 1
+    }
+  }
+}

tests/specs/npm/json_import/has_attribute.ts

@@ -0,0 +1,2 @@
+import data from "package/package.json" with { type: "json" };
+console.log(data);

tests/specs/npm/json_import/missing.out

@@ -0,0 +1 @@
+error: Attempted to load JSON module without specifying "type": "json" attribute in the import statement.

tests/specs/npm/json_import/missing_attribute.ts

@@ -0,0 +1,2 @@
+import data from "package/package.json";
+console.log(data);

tests/specs/npm/json_import/node_modules/package/package.json

@@ -0,0 +1,3 @@
+{
+  "name": "package"
+}

tests/specs/npm/json_import/package.json

@@ -0,0 +1,2 @@
+{
+}