diff --git a/src/services/goToDefinition.ts b/src/services/goToDefinition.ts
index 946924bcf87..d0901de9a47 100644
--- a/src/services/goToDefinition.ts
+++ b/src/services/goToDefinition.ts
@@ -51,7 +51,10 @@ namespace ts.GoToDefinition {
// import {A, B} from "mod";
// to jump to the implementation directly.
if (symbol.flags & SymbolFlags.Alias && shouldSkipAlias(node, symbol.declarations[0])) {
- symbol = typeChecker.getAliasedSymbol(symbol);
+ const aliased = typeChecker.getAliasedSymbol(symbol);
+ if (aliased.declarations) {
+ symbol = aliased;
+ }
}
// Because name in short-hand property assignment has two different meanings: property name and property value,
diff --git a/tests/cases/fourslash/ambientShorthandGotoDefinition.ts b/tests/cases/fourslash/ambientShorthandGotoDefinition.ts
index 410fc932cec..74e348beb74 100644
--- a/tests/cases/fourslash/ambientShorthandGotoDefinition.ts
+++ b/tests/cases/fourslash/ambientShorthandGotoDefinition.ts
@@ -12,7 +12,7 @@
verify.quickInfoAt("useFoo", "import foo");
verify.goToDefinition({
- useFoo: "importFoo",
+ useFoo: "module",
importFoo: "module"
});
@@ -27,6 +27,6 @@ verify.goToDefinition({
verify.quickInfoAt("useBang", "import bang = require(\"jquery\")");
verify.goToDefinition({
- useBang: "importBang",
+ useBang: "module",
importBang: "module"
});
diff --git a/tests/cases/fourslash/goToDefinition_untypedModule.ts b/tests/cases/fourslash/goToDefinition_untypedModule.ts
index b4571438434..fe29f7104cf 100644
--- a/tests/cases/fourslash/goToDefinition_untypedModule.ts
+++ b/tests/cases/fourslash/goToDefinition_untypedModule.ts
@@ -4,7 +4,7 @@
////not read
// @Filename: /a.ts
-////import { f } from "foo";
-/////**/f();
+////import { /*def*/f } from "foo";
+/////*use*/f();
-verify.goToDefinition("", []);
+verify.goToDefinition("use", "def");
diff --git a/tests/cases/fourslash/quickInfoMeaning.ts b/tests/cases/fourslash/quickInfoMeaning.ts
index a3b9f988e1d..2c7aa5a0ccd 100644
--- a/tests/cases/fourslash/quickInfoMeaning.ts
+++ b/tests/cases/fourslash/quickInfoMeaning.ts
@@ -8,13 +8,13 @@
// @Filename: foo.d.ts
////declare const /*foo_value_declaration*/foo: number;
////declare module "foo_module" {
-//// interface I { x: number; y: number }
+//// interface /*foo_type_declaration*/I { x: number; y: number }
//// export = I;
////}
// @Filename: foo_user.ts
///////
-////import /*foo_type_declaration*/foo = require("foo_module");
+////import foo = require("foo_module");
////const x = foo/*foo_value*/;
////const i: foo/*foo_type*/ = { x: 1, y: 2 };
@@ -39,13 +39,13 @@ verify.goToDefinitionIs("foo_type_declaration");
// @Filename: bar.d.ts
////declare interface /*bar_type_declaration*/bar { x: number; y: number }
////declare module "bar_module" {
-//// const x: number;
+//// const /*bar_value_declaration*/x: number;
//// export = x;
////}
// @Filename: bar_user.ts
///////
-////import /*bar_value_declaration*/bar = require("bar_module");
+////import bar = require("bar_module");
////const x = bar/*bar_value*/;
////const i: bar/*bar_type*/ = { x: 1, y: 2 };
diff --git a/tests/cases/fourslash/tsxGoToDefinitionClassInDifferentFile.ts b/tests/cases/fourslash/tsxGoToDefinitionClassInDifferentFile.ts
index f217e4096fc..7aa1c11fdc3 100644
--- a/tests/cases/fourslash/tsxGoToDefinitionClassInDifferentFile.ts
+++ b/tests/cases/fourslash/tsxGoToDefinitionClassInDifferentFile.ts
@@ -3,10 +3,10 @@
// @jsx: preserve
// @Filename: C.tsx
-////export default class C {}
+////export default class /*def*/C {}
// @Filename: a.tsx
-////import /*def*/C from "./C";
+////import C from "./C";
////const foo = *use*/C />;
verify.noErrors();
diff --git a/tests/cases/fourslash/untypedModuleImport.ts b/tests/cases/fourslash/untypedModuleImport.ts
index 4dac6ac76bb..5501cec4792 100644
--- a/tests/cases/fourslash/untypedModuleImport.ts
+++ b/tests/cases/fourslash/untypedModuleImport.ts
@@ -17,6 +17,6 @@ const [r0, r1, r2] = test.ranges();
verify.singleReferenceGroup('"foo"', [r1]);
goTo.marker("foo");
-verify.goToDefinitionIs([]);
+verify.goToDefinitionIs("foo");
verify.quickInfoIs("import foo");
verify.singleReferenceGroup("import foo", [r0, r2]);