Fix bug: Get merged module symbol in forEachExternalModule (#24295)
This commit is contained in:
parent
c09cc70ebe
commit
440291e316
|
@ -498,7 +498,7 @@ namespace ts.codefix {
|
||||||
}
|
}
|
||||||
for (const sourceFile of allSourceFiles) {
|
for (const sourceFile of allSourceFiles) {
|
||||||
if (isExternalOrCommonJsModule(sourceFile)) {
|
if (isExternalOrCommonJsModule(sourceFile)) {
|
||||||
cb(sourceFile.symbol, sourceFile);
|
cb(checker.getMergedSymbol(sourceFile.symbol), sourceFile);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
36
tests/cases/fourslash/completionsImport_augmentation.ts
Normal file
36
tests/cases/fourslash/completionsImport_augmentation.ts
Normal file
|
@ -0,0 +1,36 @@
|
||||||
|
/// <reference path="fourslash.ts" />
|
||||||
|
|
||||||
|
// @Filename: /a.ts
|
||||||
|
////export const foo = 0;
|
||||||
|
|
||||||
|
// @Filename: /bar.ts
|
||||||
|
////export {};
|
||||||
|
////declare module "./a" {
|
||||||
|
//// export const bar = 0;
|
||||||
|
////}
|
||||||
|
|
||||||
|
// @Filename: /user.ts
|
||||||
|
/////**/
|
||||||
|
|
||||||
|
verify.completions({
|
||||||
|
marker: "",
|
||||||
|
includes: [
|
||||||
|
{
|
||||||
|
name: "foo",
|
||||||
|
text: "const foo: 0",
|
||||||
|
source: "/a",
|
||||||
|
sourceDisplay: "./a",
|
||||||
|
hasAction: true,
|
||||||
|
},
|
||||||
|
{
|
||||||
|
name: "bar",
|
||||||
|
text: "const bar: 0",
|
||||||
|
source: "/a",
|
||||||
|
sourceDisplay: "./a",
|
||||||
|
hasAction: true,
|
||||||
|
},
|
||||||
|
],
|
||||||
|
preferences: {
|
||||||
|
includeCompletionsForModuleExports: true,
|
||||||
|
},
|
||||||
|
});
|
Loading…
Reference in a new issue