fix(2074): No quick info on function and other similar tokens (#2078)

Author: a-tarasyuk

internal/checker/checker.go

@@ -30518,7 +30518,7 @@ func (c *Checker) getSymbolAtLocation(node *ast.Node, ignoreErrors bool) *ast.Sy
 		}
 		return nil
 	case ast.KindDefaultKeyword, ast.KindFunctionKeyword, ast.KindEqualsGreaterThanToken, ast.KindClassKeyword:
-		return c.getSymbolOfNode(node)
+		return c.getSymbolOfNode(node.Parent)
 	case ast.KindImportType:
 		if ast.IsLiteralImportTypeNode(node) {
 			return c.getSymbolAtLocation(node.AsImportTypeNode().Argument.AsLiteralTypeNode().Literal, ignoreErrors)

internal/checker/nodebuilderimpl.go

@@ -886,6 +886,9 @@ func (b *NodeBuilderImpl) getNameOfSymbolAsWritten(symbol *ast.Symbol) string {
 	if len(name) > 0 {
 		return name
 	}
+	if symbol.Name == ast.InternalSymbolNameMissing {
+		return "__missing"
+	}
 	return symbol.Name
 }
 

internal/fourslash/tests/quickInfoFunction_test.go

@@ -0,0 +1,17 @@
+package fourslash_test
+
+import (
+	"testing"
+
+	"github.com/microsoft/typescript-go/internal/fourslash"
+	"github.com/microsoft/typescript-go/internal/testutil"
+)
+
+func TestQuickInfoFunction(t *testing.T) {
+	t.Parallel()
+	defer testutil.RecoverAndFail(t, "Panic on fourslash test")
+	const content = `/**/function foo() { return "hi"; }`
+
+	f := fourslash.NewFourslash(t, nil /*capabilities*/, content)
+	f.VerifyQuickInfoAt(t, "", "function foo(): string", "")
+}

testdata/baselines/reference/fourslash/documentHighlights/documentHighlights_40082.baseline.jsonc

@@ -1,6 +1,6 @@
 // === documentHighlights ===
 // === /documentHighlights_40082.ts ===
 // export = (state, messages) => {
-//    export /*HIGHLIGHTS*/default {
+//    export /*HIGHLIGHTS*/[|default|] {
 //    }
 // }

testdata/baselines/reference/fourslash/findAllReferences/findAllRefsExportDefaultClassConstructor.baseline.jsonc

@@ -1,5 +1,5 @@
 // === findAllReferences ===
 // === /findAllRefsExportDefaultClassConstructor.ts ===
 // export default class {
-//     /*FIND ALL REFS*/constructor() {}
+//     /*FIND ALL REFS*/[|constructor|]() {}
 // }

testdata/baselines/reference/fourslash/findAllReferences/findAllRefsForDefaultExport04.baseline.jsonc

@@ -23,18 +23,30 @@
 // === findAllReferences ===
 // === /a.ts ===
 // const a = 0;
-// export /*FIND ALL REFS*/default a;
+// export /*FIND ALL REFS*/[|default|] a;
+
+// === /b.ts ===
+// import [|a|] from "./a";
+// [|a|];
 
 
 
 // === findAllReferences ===
+// === /a.ts ===
+// const a = 0;
+// export [|default|] a;
+
 // === /b.ts ===
 // import /*FIND ALL REFS*/[|a|] from "./a";
 // [|a|];
 
 
 
 // === findAllReferences ===
+// === /a.ts ===
+// const a = 0;
+// export [|default|] a;
+
 // === /b.ts ===
 // import [|a|] from "./a";
 // /*FIND ALL REFS*/[|a|];

testdata/baselines/reference/fourslash/findAllReferences/findAllRefsForDefaultExport_anonymous.baseline.jsonc

@@ -1,3 +1,6 @@
 // === findAllReferences ===
 // === /a.ts ===
-// export /*FIND ALL REFS*/default 1;
+// export /*FIND ALL REFS*/[|default|] 1;
+
+// === /b.ts ===
+// import [|a|] from "./a";

testdata/baselines/reference/fourslash/findAllReferences/findAllRefsForFunctionExpression01.baseline.jsonc

@@ -1,7 +1,7 @@
 // === findAllReferences ===
 // === /file1.ts ===
-// var foo = /*FIND ALL REFS*/function foo(a = foo(), b = () => foo) {
-//     foo(foo, foo);
+// var foo = /*FIND ALL REFS*/function [|foo|](a = [|foo|](), b = () => [|foo|]) {
+//     [|foo|]([|foo|], [|foo|]);
 // }
 
 

testdata/baselines/reference/fourslash/findAllReferences/findAllRefsWithLeadingUnderscoreNames8.baseline.jsonc

@@ -1,7 +1,7 @@
 // === findAllReferences ===
 // === /findAllRefsWithLeadingUnderscoreNames8.ts ===
 // (/*FIND ALL REFS*/function __foo() {
-//     __foo();
+//     [|__foo|]();
 // })
 
 

testdata/baselines/reference/fourslash/findAllReferences/findAllRefsWithLeadingUnderscoreNames9.baseline.jsonc

@@ -1,7 +1,7 @@
 // === findAllReferences ===
 // === /findAllRefsWithLeadingUnderscoreNames9.ts ===
 // (/*FIND ALL REFS*/function ___foo() {
-//     ___foo();
+//     [|___foo|]();
 // })