Fix getter of local symbol for export= when it is json module

sheetalkamat · sheetalkamat · commit a0d4e4f059bf · 2018-05-22T17:10:56.000-07:00
Fixes microsoft#24341
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");

Original file line number	Diff line number	Diff line change
`@@ -579,6 +579,9 @@ namespace ts.FindAllReferences {`
`579`	`579`	else if (isBinaryExpression(decl)) { // `module.exports = class {}`
`580`	`580`	`return Debug.assertDefined(decl.right.symbol);`
`581`	`581`	`}`
	`582`	`+ else if (isSourceFile(decl)) { // json module`
	`583`	`+ return Debug.assertDefined(decl.symbol);`
	`584`	`+ }`
`582`	`585`	`return Debug.fail();`
`583`	`586`	`}`
`584`	`587`