Report errors for module generation other than "commonjs" with --reso…

…lveJsonModule

Fixes #25517

src/compiler/diagnosticMessages.json

@@ -2864,6 +2864,10 @@
         "category": "Error",
         "code": 5070
     },
+    "Option '--resolveJsonModule' can only be specified when module code generation is 'commonjs'.": {
+        "category": "Error",
+        "code": 5071
+    },
 
     "Generates a sourcemap for each corresponding '.d.ts' file.": {
         "category": "Message",

src/compiler/program.ts

@@ -2548,6 +2548,10 @@ namespace ts {
                 if (getEmitModuleResolutionKind(options) !== ModuleResolutionKind.NodeJs) {
                     createDiagnosticForOptionName(Diagnostics.Option_resolveJsonModule_cannot_be_specified_without_node_module_resolution_strategy, "resolveJsonModule");
                 }
+                // Any emit other than common js is error
+                else if (options.module !== undefined && options.module !== ModuleKind.CommonJS) {
+                    createDiagnosticForOptionName(Diagnostics.Option_resolveJsonModule_can_only_be_specified_when_module_code_generation_is_commonjs, "resolveJsonModule", "module");
+                }
             }
 
             // there has to be common source directory if user specified --outdir || --sourceRoot

tests/baselines/reference/requireOfJsonFileWithModuleEmitNone.errors.txt

@@ -0,0 +1,18 @@
+error TS5070: Option '--resolveJsonModule' cannot be specified without 'node' module resolution strategy.
+tests/cases/compiler/file1.ts(1,1): error TS1148: Cannot use imports, exports, or module augmentations when '--module' is 'none'.
+tests/cases/compiler/file1.ts(1,20): error TS2307: Cannot find module './b.json'.
+
+
+!!! error TS5070: Option '--resolveJsonModule' cannot be specified without 'node' module resolution strategy.
+==== tests/cases/compiler/file1.ts (2 errors) ====
+    import * as b from './b.json';
+    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+!!! error TS1148: Cannot use imports, exports, or module augmentations when '--module' is 'none'.
+                       ~~~~~~~~~~
+!!! error TS2307: Cannot find module './b.json'.
+
+==== tests/cases/compiler/b.json (0 errors) ====
+    {
+        "a": true,
+        "b": "hello"
+    }

tests/baselines/reference/requireOfJsonFileWithModuleEmitNone.js

@@ -0,0 +1,14 @@
+//// [tests/cases/compiler/requireOfJsonFileWithModuleEmitNone.ts] ////
+
+//// [file1.ts]
+import * as b from './b.json';
+
+//// [b.json]
+{
+    "a": true,
+    "b": "hello"
+}
+
+//// [out/file1.js]
+"use strict";
+exports.__esModule = true;

tests/baselines/reference/requireOfJsonFileWithModuleEmitNone.symbols

@@ -0,0 +1,4 @@
+=== tests/cases/compiler/file1.ts ===
+import * as b from './b.json';
+>b : Symbol(b, Decl(file1.ts, 0, 6))
+

tests/baselines/reference/requireOfJsonFileWithModuleEmitNone.types

@@ -0,0 +1,4 @@
+=== tests/cases/compiler/file1.ts ===
+import * as b from './b.json';
+>b : any
+

tests/baselines/reference/requireOfJsonFileWithModuleEmitUndefined.js

@@ -0,0 +1,19 @@
+//// [tests/cases/compiler/requireOfJsonFileWithModuleEmitUndefined.ts] ////
+
+//// [file1.ts]
+import * as b from './b.json';
+
+//// [b.json]
+{
+    "a": true,
+    "b": "hello"
+}
+
+//// [out/b.json]
+{
+    "a": true,
+    "b": "hello"
+}
+//// [out/file1.js]
+"use strict";
+exports.__esModule = true;

tests/baselines/reference/requireOfJsonFileWithModuleEmitUndefined.symbols

@@ -0,0 +1,12 @@
+=== tests/cases/compiler/file1.ts ===
+import * as b from './b.json';
+>b : Symbol(b, Decl(file1.ts, 0, 6))
+
+=== tests/cases/compiler/b.json ===
+{
+    "a": true,
+>"a" : Symbol("a", Decl(b.json, 0, 1))
+
+    "b": "hello"
+>"b" : Symbol("b", Decl(b.json, 1, 14))
+}

tests/baselines/reference/requireOfJsonFileWithModuleEmitUndefined.types

@@ -0,0 +1,16 @@
+=== tests/cases/compiler/file1.ts ===
+import * as b from './b.json';
+>b : { "a": boolean; "b": string; }
+
+=== tests/cases/compiler/b.json ===
+{
+>{    "a": true,    "b": "hello"} : { "a": boolean; "b": string; }
+
+    "a": true,
+>"a" : boolean
+>true : true
+
+    "b": "hello"
+>"b" : string
+>"hello" : "hello"
+}

tests/baselines/reference/requireOfJsonFileWithModuleNodeResolutionEmitAmd.errors.txt

@@ -0,0 +1,12 @@
+error TS5071: Option '--resolveJsonModule' can only be specified when module code generation is 'commonjs'.
+
+
+!!! error TS5071: Option '--resolveJsonModule' can only be specified when module code generation is 'commonjs'.
+==== tests/cases/compiler/file1.ts (0 errors) ====
+    import * as b from './b.json';
+
+==== tests/cases/compiler/b.json (0 errors) ====
+    {
+        "a": true,
+        "b": "hello"
+    }

tests/baselines/reference/requireOfJsonFileWithModuleNodeResolutionEmitAmd.js

@@ -0,0 +1,21 @@
+//// [tests/cases/compiler/requireOfJsonFileWithModuleNodeResolutionEmitAmd.ts] ////
+
+//// [file1.ts]
+import * as b from './b.json';
+
+//// [b.json]
+{
+    "a": true,
+    "b": "hello"
+}
+
+//// [out/b.json]
+{
+    "a": true,
+    "b": "hello"
+}
+//// [out/file1.js]
+define(["require", "exports"], function (require, exports) {
+    "use strict";
+    exports.__esModule = true;
+});

tests/baselines/reference/requireOfJsonFileWithModuleNodeResolutionEmitAmd.symbols

@@ -0,0 +1,12 @@
+=== tests/cases/compiler/file1.ts ===
+import * as b from './b.json';
+>b : Symbol(b, Decl(file1.ts, 0, 6))
+
+=== tests/cases/compiler/b.json ===
+{
+    "a": true,
+>"a" : Symbol("a", Decl(b.json, 0, 1))
+
+    "b": "hello"
+>"b" : Symbol("b", Decl(b.json, 1, 14))
+}

tests/baselines/reference/requireOfJsonFileWithModuleNodeResolutionEmitAmd.types

@@ -0,0 +1,16 @@
+=== tests/cases/compiler/file1.ts ===
+import * as b from './b.json';
+>b : { "a": boolean; "b": string; }
+
+=== tests/cases/compiler/b.json ===
+{
+>{    "a": true,    "b": "hello"} : { "a": boolean; "b": string; }
+
+    "a": true,
+>"a" : boolean
+>true : true
+
+    "b": "hello"
+>"b" : string
+>"hello" : "hello"
+}

tests/baselines/reference/requireOfJsonFileWithModuleNodeResolutionEmitEs2015.errors.txt

@@ -0,0 +1,12 @@
+error TS5071: Option '--resolveJsonModule' can only be specified when module code generation is 'commonjs'.
+
+
+!!! error TS5071: Option '--resolveJsonModule' can only be specified when module code generation is 'commonjs'.
+==== tests/cases/compiler/file1.ts (0 errors) ====
+    import * as b from './b.json';
+
+==== tests/cases/compiler/b.json (0 errors) ====
+    {
+        "a": true,
+        "b": "hello"
+    }

tests/baselines/reference/requireOfJsonFileWithModuleNodeResolutionEmitEs2015.js

@@ -0,0 +1,17 @@
+//// [tests/cases/compiler/requireOfJsonFileWithModuleNodeResolutionEmitEs2015.ts] ////
+
+//// [file1.ts]
+import * as b from './b.json';
+
+//// [b.json]
+{
+    "a": true,
+    "b": "hello"
+}
+
+//// [out/b.json]
+{
+    "a": true,
+    "b": "hello"
+}
+//// [out/file1.js]

tests/baselines/reference/requireOfJsonFileWithModuleNodeResolutionEmitEs2015.symbols

@@ -0,0 +1,12 @@
+=== tests/cases/compiler/file1.ts ===
+import * as b from './b.json';
+>b : Symbol(b, Decl(file1.ts, 0, 6))
+
+=== tests/cases/compiler/b.json ===
+{
+    "a": true,
+>"a" : Symbol("a", Decl(b.json, 0, 1))
+
+    "b": "hello"
+>"b" : Symbol("b", Decl(b.json, 1, 14))
+}

tests/baselines/reference/requireOfJsonFileWithModuleNodeResolutionEmitEs2015.types

@@ -0,0 +1,16 @@
+=== tests/cases/compiler/file1.ts ===
+import * as b from './b.json';
+>b : { "a": boolean; "b": string; }
+
+=== tests/cases/compiler/b.json ===
+{
+>{    "a": true,    "b": "hello"} : { "a": boolean; "b": string; }
+
+    "a": true,
+>"a" : boolean
+>true : true
+
+    "b": "hello"
+>"b" : string
+>"hello" : "hello"
+}

tests/baselines/reference/requireOfJsonFileWithModuleNodeResolutionEmitEsNext.errors.txt

@@ -0,0 +1,12 @@
+error TS5071: Option '--resolveJsonModule' can only be specified when module code generation is 'commonjs'.
+
+
+!!! error TS5071: Option '--resolveJsonModule' can only be specified when module code generation is 'commonjs'.
+==== tests/cases/compiler/file1.ts (0 errors) ====
+    import * as b  from './b.json';
+
+==== tests/cases/compiler/b.json (0 errors) ====
+    {
+        "a": true,
+        "b": "hello"
+    }

tests/baselines/reference/requireOfJsonFileWithModuleNodeResolutionEmitEsNext.js

@@ -0,0 +1,17 @@
+//// [tests/cases/compiler/requireOfJsonFileWithModuleNodeResolutionEmitEsNext.ts] ////
+
+//// [file1.ts]
+import * as b  from './b.json';
+
+//// [b.json]
+{
+    "a": true,
+    "b": "hello"
+}
+
+//// [out/b.json]
+{
+    "a": true,
+    "b": "hello"
+}
+//// [out/file1.js]

tests/baselines/reference/requireOfJsonFileWithModuleNodeResolutionEmitEsNext.symbols

@@ -0,0 +1,12 @@
+=== tests/cases/compiler/file1.ts ===
+import * as b  from './b.json';
+>b : Symbol(b, Decl(file1.ts, 0, 6))
+
+=== tests/cases/compiler/b.json ===
+{
+    "a": true,
+>"a" : Symbol("a", Decl(b.json, 0, 1))
+
+    "b": "hello"
+>"b" : Symbol("b", Decl(b.json, 1, 14))
+}

tests/baselines/reference/requireOfJsonFileWithModuleNodeResolutionEmitEsNext.types

@@ -0,0 +1,16 @@
+=== tests/cases/compiler/file1.ts ===
+import * as b  from './b.json';
+>b : { "a": boolean; "b": string; }
+
+=== tests/cases/compiler/b.json ===
+{
+>{    "a": true,    "b": "hello"} : { "a": boolean; "b": string; }
+
+    "a": true,
+>"a" : boolean
+>true : true
+
+    "b": "hello"
+>"b" : string
+>"hello" : "hello"
+}

tests/baselines/reference/requireOfJsonFileWithModuleNodeResolutionEmitNone.errors.txt

@@ -0,0 +1,15 @@
+error TS5071: Option '--resolveJsonModule' can only be specified when module code generation is 'commonjs'.
+tests/cases/compiler/file1.ts(1,1): error TS1148: Cannot use imports, exports, or module augmentations when '--module' is 'none'.
+
+
+!!! error TS5071: Option '--resolveJsonModule' can only be specified when module code generation is 'commonjs'.
+==== tests/cases/compiler/file1.ts (1 errors) ====
+    import * as b from './b.json';
+    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+!!! error TS1148: Cannot use imports, exports, or module augmentations when '--module' is 'none'.
+
+==== tests/cases/compiler/b.json (0 errors) ====
+    {
+        "a": true,
+        "b": "hello"
+    }

tests/baselines/reference/requireOfJsonFileWithModuleNodeResolutionEmitNone.js

@@ -0,0 +1,19 @@
+//// [tests/cases/compiler/requireOfJsonFileWithModuleNodeResolutionEmitNone.ts] ////
+
+//// [file1.ts]
+import * as b from './b.json';
+
+//// [b.json]
+{
+    "a": true,
+    "b": "hello"
+}
+
+//// [out/b.json]
+{
+    "a": true,
+    "b": "hello"
+}
+//// [out/file1.js]
+"use strict";
+exports.__esModule = true;

tests/baselines/reference/requireOfJsonFileWithModuleNodeResolutionEmitNone.symbols

@@ -0,0 +1,12 @@
+=== tests/cases/compiler/file1.ts ===
+import * as b from './b.json';
+>b : Symbol(b, Decl(file1.ts, 0, 6))
+
+=== tests/cases/compiler/b.json ===
+{
+    "a": true,
+>"a" : Symbol("a", Decl(b.json, 0, 1))
+
+    "b": "hello"
+>"b" : Symbol("b", Decl(b.json, 1, 14))
+}

tests/baselines/reference/requireOfJsonFileWithModuleNodeResolutionEmitNone.types

@@ -0,0 +1,16 @@
+=== tests/cases/compiler/file1.ts ===
+import * as b from './b.json';
+>b : { "a": boolean; "b": string; }
+
+=== tests/cases/compiler/b.json ===
+{
+>{    "a": true,    "b": "hello"} : { "a": boolean; "b": string; }
+
+    "a": true,
+>"a" : boolean
+>true : true
+
+    "b": "hello"
+>"b" : string
+>"hello" : "hello"
+}

tests/baselines/reference/requireOfJsonFileWithModuleNodeResolutionEmitSystem.errors.txt

@@ -0,0 +1,12 @@
+error TS5071: Option '--resolveJsonModule' can only be specified when module code generation is 'commonjs'.
+
+
+!!! error TS5071: Option '--resolveJsonModule' can only be specified when module code generation is 'commonjs'.
+==== tests/cases/compiler/file1.ts (0 errors) ====
+    import * as b from './b.json';
+
+==== tests/cases/compiler/b.json (0 errors) ====
+    {
+        "a": true,
+        "b": "hello"
+    }