Merge pull request microsoft#16026 from Microsoft/importEqualsMethod

Andy · web-flow · commit 8b15e2b52387 · 2017-05-23T09:57:32.000-07:00
buildSymbolDisplay: Handle alias parentSymbol
diff --git a/src/compiler/checker.ts b/src/compiler/checker.ts
@@ -2033,7 +2033,7 @@ namespace ts {
                                 ts.forEach(symbolFromSymbolTable.declarations, isExternalModuleImportEqualsDeclaration)) {
 
                                 const resolvedImportedSymbol = resolveAlias(symbolFromSymbolTable);
-                                if (isAccessible(symbolFromSymbolTable, resolveAlias(symbolFromSymbolTable))) {
+                                if (isAccessible(symbolFromSymbolTable, resolvedImportedSymbol)) {
                                     return [symbolFromSymbolTable];
                                 }
 
@@ -3093,8 +3093,8 @@ namespace ts {
                         // Write type arguments of instantiated class/interface here
                         if (flags & SymbolFormatFlags.WriteTypeParametersOrArguments) {
                             if (getCheckFlags(symbol) & CheckFlags.Instantiated) {
-                                buildDisplayForTypeArgumentsAndDelimiters(getTypeParametersOfClassOrInterface(parentSymbol),
-                                    (<TransientSymbol>symbol).mapper, writer, enclosingDeclaration);
+                                const params = getTypeParametersOfClassOrInterface(parentSymbol.flags & SymbolFlags.Alias ? resolveAlias(parentSymbol) : parentSymbol);
+                                buildDisplayForTypeArgumentsAndDelimiters(params, (<TransientSymbol>symbol).mapper, writer, enclosingDeclaration);
                             }
                             else {
                                 buildTypeParameterDisplayFromSymbol(parentSymbol, writer, enclosingDeclaration);
diff --git a/tests/cases/fourslash/quickInfoOnMethodOfImportEquals.ts b/tests/cases/fourslash/quickInfoOnMethodOfImportEquals.ts
@@ -0,0 +1,16 @@
+/// <reference path="fourslash.ts" />
+
+// Test for https://github.com/Microsoft/TypeScript/issues/15931
+
+// @Filename: /a.d.ts
+////declare class C<T> {
+////    m(): void;
+////}
+////export = C;
+
+// @Filename: /b.ts
+////import C = require("./a");
+////declare var x: C<number>;
+////x./**/m;
+
+verify.quickInfoAt("", "(method) C<number>.m(): void", undefined);
