Handle failure to properly resolve type reference directives

uniqueiniquity · uniqueiniquity · commit 30da23fa7770 · 2018-01-19T16:23:22.000-08:00
diff --git a/src/compiler/checker.ts b/src/compiler/checker.ts
@@ -25000,10 +25000,11 @@ namespace ts {
                 // populate reverse mapping: file path -> type reference directive that was resolved to this file
                 fileToDirective = createMap<string>();
                 resolvedTypeReferenceDirectives.forEach((resolvedDirective, key) => {
-                    if (!resolvedDirective) {
+                    if (!resolvedDirective || !resolvedDirective.resolvedFileName) {
                         return;
                     }
                     const file = host.getSourceFile(resolvedDirective.resolvedFileName);
+                    Debug.assert(!!file, `Resolved filename ${resolvedDirective.resolvedFileName} did not map to existing source file. Consider enabling --preserveSymlinks if appropriate`); // tslint:disable-line
                     fileToDirective.set(file.path, key);
                 });
             }

Original file line number	Diff line number	Diff line change
`@@ -25000,10 +25000,11 @@ namespace ts {`
`25000`	`25000`	`// populate reverse mapping: file path -> type reference directive that was resolved to this file`
`25001`	`25001`	`fileToDirective = createMap<string>();`
`25002`	`25002`	`resolvedTypeReferenceDirectives.forEach((resolvedDirective, key) => {`
`25003`		`- if (!resolvedDirective) {`
	`25003`	`+ if (!resolvedDirective \|\| !resolvedDirective.resolvedFileName) {`
`25004`	`25004`	`return;`
`25005`	`25005`	`}`
`25006`	`25006`	`const file = host.getSourceFile(resolvedDirective.resolvedFileName);`
	`25007`	+ Debug.assert(!!file, `Resolved filename ${resolvedDirective.resolvedFileName} did not map to existing source file. Consider enabling --preserveSymlinks if appropriate`); // tslint:disable-line
`25007`	`25008`	`fileToDirective.set(file.path, key);`
`25008`	`25009`	`});`
`25009`	`25010`	`}`