Merge pull request microsoft#24348 from Microsoft/findRefInJson

Fix getter of local symbol for export= when it is json module
foreverI68 · May 23, 2018 · 16d7f4c · 16d7f4c
2 parents 44da834 + a0d4e4f
commit 16d7f4c
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 0 deletions.
diff --git a/src/services/importTracker.ts b/src/services/importTracker.ts
@@ -579,6 +579,9 @@ namespace ts.FindAllReferences {
         else if (isBinaryExpression(decl)) { // `module.exports = class {}`
             return Debug.assertDefined(decl.right.symbol);
         }
+        else if (isSourceFile(decl)) { // json module
+            return Debug.assertDefined(decl.symbol);
+        }
         return Debug.fail();
     }
 

diff --git a/tests/cases/fourslash/findAllReferencesOfJsonModule.ts b/tests/cases/fourslash/findAllReferencesOfJsonModule.ts
@@ -0,0 +1,14 @@
+/// <reference path='fourslash.ts' />
+
+// @resolveJsonModule: true
+// @module: commonjs
+// @esModuleInterop: true
+
+// @Filename: /foo.ts
+////import [|{| "isWriteAccess": true, "isDefinition": true |}settings|] from "./settings.json";
+////[|settings|];
+
+// @Filename: /settings.json
+//// {}
+
+verify.singleReferenceGroup("import settings");