fix(15230): change the diagnostic message about a non-module file in …

…an --isolatedModules project
microsoft · Aug 13, 2020 · 2838326 · 2838326
1 parent 610fa28
commit 2838326
Show file tree

Hide file tree

Showing 11 changed files with 21 additions and 20 deletions.
diff --git a/src/compiler/diagnosticMessages.json b/src/compiler/diagnosticMessages.json
@@ -659,7 +659,7 @@
         "category": "Error",
         "code": 1207
     },
-    "All files must be modules when the '--isolatedModules' flag is provided.": {
+    "'{0}' cannot be compiled using '--isolatedModules' because it is global. Ensure that it contains an import, export, or an empty 'export {}' statement.": {
         "category": "Error",
         "code": 1208
     },

diff --git a/src/compiler/program.ts b/src/compiler/program.ts
@@ -3110,7 +3110,8 @@ namespace ts {
                 const firstNonExternalModuleSourceFile = find(files, f => !isExternalModule(f) && !isSourceFileJS(f) && !f.isDeclarationFile && f.scriptKind !== ScriptKind.JSON);
                 if (firstNonExternalModuleSourceFile) {
                     const span = getErrorSpanForNode(firstNonExternalModuleSourceFile, firstNonExternalModuleSourceFile);
-                    programDiagnostics.add(createFileDiagnostic(firstNonExternalModuleSourceFile, span.start, span.length, Diagnostics.All_files_must_be_modules_when_the_isolatedModules_flag_is_provided));
+                    programDiagnostics.add(createFileDiagnostic(firstNonExternalModuleSourceFile, span.start, span.length,
+                        Diagnostics._0_cannot_be_compiled_using_isolatedModules_because_it_is_global_Ensure_that_it_contains_an_import_export_or_an_empty_export_statement, getBaseFileName(firstNonExternalModuleSourceFile.fileName)));
                 }
             }
             else if (firstNonAmbientExternalModuleSourceFile && languageVersion < ScriptTarget.ES2015 && options.module === ModuleKind.None) {

diff --git a/tests/baselines/reference/importHelpersInIsolatedModules.errors.txt b/tests/baselines/reference/importHelpersInIsolatedModules.errors.txt
@@ -1,4 +1,4 @@
-tests/cases/compiler/script.ts(1,1): error TS1208: All files must be modules when the '--isolatedModules' flag is provided.
+tests/cases/compiler/script.ts(1,1): error TS1208: 'script.ts' cannot be compiled using '--isolatedModules' because it is global. Ensure that it contains an import, export, or an empty 'export {}' statement.
 
 
 ==== tests/cases/compiler/external.ts (0 errors) ====
@@ -16,7 +16,7 @@ tests/cases/compiler/script.ts(1,1): error TS1208: All files must be modules whe
 ==== tests/cases/compiler/script.ts (1 errors) ====
     class A { }
     ~~~~~
-!!! error TS1208: All files must be modules when the '--isolatedModules' flag is provided.
+!!! error TS1208: 'script.ts' cannot be compiled using '--isolatedModules' because it is global. Ensure that it contains an import, export, or an empty 'export {}' statement.
     class B extends A { }
 
     declare var dec: any;

diff --git a/tests/baselines/reference/isolatedModulesNoExternalModule.errors.txt b/tests/baselines/reference/isolatedModulesNoExternalModule.errors.txt
@@ -1,7 +1,7 @@
-tests/cases/compiler/file1.ts(1,1): error TS1208: All files must be modules when the '--isolatedModules' flag is provided.
+tests/cases/compiler/file1.ts(1,1): error TS1208: 'file1.ts' cannot be compiled using '--isolatedModules' because it is global. Ensure that it contains an import, export, or an empty 'export {}' statement.
 
 
 ==== tests/cases/compiler/file1.ts (1 errors) ====
     var x;
     ~~~
-!!! error TS1208: All files must be modules when the '--isolatedModules' flag is provided.
+!!! error TS1208: 'file1.ts' cannot be compiled using '--isolatedModules' because it is global. Ensure that it contains an import, export, or an empty 'export {}' statement.
diff --git a/tests/baselines/reference/isolatedModulesOut.errors.txt b/tests/baselines/reference/isolatedModulesOut.errors.txt
@@ -1,6 +1,6 @@
 error TS5053: Option 'out' cannot be specified with option 'isolatedModules'.
 tests/cases/compiler/file1.ts(1,1): error TS6131: Cannot compile modules using option 'out' unless the '--module' flag is 'amd' or 'system'.
-tests/cases/compiler/file2.ts(1,1): error TS1208: All files must be modules when the '--isolatedModules' flag is provided.
+tests/cases/compiler/file2.ts(1,1): error TS1208: 'file2.ts' cannot be compiled using '--isolatedModules' because it is global. Ensure that it contains an import, export, or an empty 'export {}' statement.
 
 
 !!! error TS5053: Option 'out' cannot be specified with option 'isolatedModules'.
@@ -11,4 +11,4 @@ tests/cases/compiler/file2.ts(1,1): error TS1208: All files must be modules when
 ==== tests/cases/compiler/file2.ts (1 errors) ====
     var y;
     ~~~
-!!! error TS1208: All files must be modules when the '--isolatedModules' flag is provided.
+!!! error TS1208: 'file2.ts' cannot be compiled using '--isolatedModules' because it is global. Ensure that it contains an import, export, or an empty 'export {}' statement.
diff --git a/tests/baselines/reference/isolatedModulesPlainFile-AMD.errors.txt b/tests/baselines/reference/isolatedModulesPlainFile-AMD.errors.txt
@@ -1,9 +1,9 @@
-tests/cases/compiler/isolatedModulesPlainFile-AMD.ts(1,1): error TS1208: All files must be modules when the '--isolatedModules' flag is provided.
+tests/cases/compiler/isolatedModulesPlainFile-AMD.ts(1,1): error TS1208: 'isolatedModulesPlainFile-AMD.ts' cannot be compiled using '--isolatedModules' because it is global. Ensure that it contains an import, export, or an empty 'export {}' statement.
 
 
 ==== tests/cases/compiler/isolatedModulesPlainFile-AMD.ts (1 errors) ====
     declare function run(a: number): void;
     ~~~~~~~
-!!! error TS1208: All files must be modules when the '--isolatedModules' flag is provided.
+!!! error TS1208: 'isolatedModulesPlainFile-AMD.ts' cannot be compiled using '--isolatedModules' because it is global. Ensure that it contains an import, export, or an empty 'export {}' statement.
     run(1);
 
diff --git a/tests/baselines/reference/isolatedModulesPlainFile-CommonJS.errors.txt b/tests/baselines/reference/isolatedModulesPlainFile-CommonJS.errors.txt
@@ -1,9 +1,9 @@
-tests/cases/compiler/isolatedModulesPlainFile-CommonJS.ts(1,1): error TS1208: All files must be modules when the '--isolatedModules' flag is provided.
+tests/cases/compiler/isolatedModulesPlainFile-CommonJS.ts(1,1): error TS1208: 'isolatedModulesPlainFile-CommonJS.ts' cannot be compiled using '--isolatedModules' because it is global. Ensure that it contains an import, export, or an empty 'export {}' statement.
 
 
 ==== tests/cases/compiler/isolatedModulesPlainFile-CommonJS.ts (1 errors) ====
     declare function run(a: number): void;
     ~~~~~~~
-!!! error TS1208: All files must be modules when the '--isolatedModules' flag is provided.
+!!! error TS1208: 'isolatedModulesPlainFile-CommonJS.ts' cannot be compiled using '--isolatedModules' because it is global. Ensure that it contains an import, export, or an empty 'export {}' statement.
     run(1);
 
diff --git a/tests/baselines/reference/isolatedModulesPlainFile-ES6.errors.txt b/tests/baselines/reference/isolatedModulesPlainFile-ES6.errors.txt
@@ -1,9 +1,9 @@
-tests/cases/compiler/isolatedModulesPlainFile-ES6.ts(1,1): error TS1208: All files must be modules when the '--isolatedModules' flag is provided.
+tests/cases/compiler/isolatedModulesPlainFile-ES6.ts(1,1): error TS1208: 'isolatedModulesPlainFile-ES6.ts' cannot be compiled using '--isolatedModules' because it is global. Ensure that it contains an import, export, or an empty 'export {}' statement.
 
 
 ==== tests/cases/compiler/isolatedModulesPlainFile-ES6.ts (1 errors) ====
     declare function run(a: number): void;
     ~~~~~~~
-!!! error TS1208: All files must be modules when the '--isolatedModules' flag is provided.
+!!! error TS1208: 'isolatedModulesPlainFile-ES6.ts' cannot be compiled using '--isolatedModules' because it is global. Ensure that it contains an import, export, or an empty 'export {}' statement.
     run(1);
 
diff --git a/tests/baselines/reference/isolatedModulesPlainFile-System.errors.txt b/tests/baselines/reference/isolatedModulesPlainFile-System.errors.txt
@@ -1,9 +1,9 @@
-tests/cases/compiler/isolatedModulesPlainFile-System.ts(1,1): error TS1208: All files must be modules when the '--isolatedModules' flag is provided.
+tests/cases/compiler/isolatedModulesPlainFile-System.ts(1,1): error TS1208: 'isolatedModulesPlainFile-System.ts' cannot be compiled using '--isolatedModules' because it is global. Ensure that it contains an import, export, or an empty 'export {}' statement.
 
 
 ==== tests/cases/compiler/isolatedModulesPlainFile-System.ts (1 errors) ====
     declare function run(a: number): void;
     ~~~~~~~
-!!! error TS1208: All files must be modules when the '--isolatedModules' flag is provided.
+!!! error TS1208: 'isolatedModulesPlainFile-System.ts' cannot be compiled using '--isolatedModules' because it is global. Ensure that it contains an import, export, or an empty 'export {}' statement.
     run(1);
 
diff --git a/tests/baselines/reference/isolatedModulesPlainFile-UMD.errors.txt b/tests/baselines/reference/isolatedModulesPlainFile-UMD.errors.txt
@@ -1,9 +1,9 @@
-tests/cases/compiler/isolatedModulesPlainFile-UMD.ts(1,1): error TS1208: All files must be modules when the '--isolatedModules' flag is provided.
+tests/cases/compiler/isolatedModulesPlainFile-UMD.ts(1,1): error TS1208: 'isolatedModulesPlainFile-UMD.ts' cannot be compiled using '--isolatedModules' because it is global. Ensure that it contains an import, export, or an empty 'export {}' statement.
 
 
 ==== tests/cases/compiler/isolatedModulesPlainFile-UMD.ts (1 errors) ====
     declare function run(a: number): void;
     ~~~~~~~
-!!! error TS1208: All files must be modules when the '--isolatedModules' flag is provided.
+!!! error TS1208: 'isolatedModulesPlainFile-UMD.ts' cannot be compiled using '--isolatedModules' because it is global. Ensure that it contains an import, export, or an empty 'export {}' statement.
     run(1);
 
diff --git a/...ured-projects/should-always-return-the-file-itself-if-'--isolatedModules'-is-specified.js b/...ured-projects/should-always-return-the-file-itself-if-'--isolatedModules'-is-specified.js
@@ -37,7 +37,7 @@ Output::
 [[90m12:00:23 AM[0m] Starting compilation in watch mode...
 
 
-[96ma/b/globalFile3.ts[0m:[93m1[0m:[93m1[0m - [91merror[0m[90m TS1208: [0mAll files must be modules when the '--isolatedModules' flag is provided.
+[96ma/b/globalFile3.ts[0m:[93m1[0m:[93m1[0m - [91merror[0m[90m TS1208: [0m'globalFile3.ts' cannot be compiled using '--isolatedModules' because it is global. Ensure that it contains an import, export, or an empty 'export {}' statement.
 
 [7m1[0m interface GlobalFoo { age: number }
 [7m [0m [91m~~~~~~~~~[0m
@@ -136,7 +136,7 @@ Output::
 [[90m12:00:38 AM[0m] File change detected. Starting incremental compilation...
 
 
-[96ma/b/globalFile3.ts[0m:[93m1[0m:[93m1[0m - [91merror[0m[90m TS1208: [0mAll files must be modules when the '--isolatedModules' flag is provided.
+[96ma/b/globalFile3.ts[0m:[93m1[0m:[93m1[0m - [91merror[0m[90m TS1208: [0m'globalFile3.ts' cannot be compiled using '--isolatedModules' because it is global. Ensure that it contains an import, export, or an empty 'export {}' statement.
 
 [7m1[0m interface GlobalFoo { age: number }
 [7m [0m [91m~~~~~~~~~[0m