From 3a8e6bb23cd5803c93a453efacdad0dbfed5ca12 Mon Sep 17 00:00:00 2001
From: Daniel Rosenwasser <drosen@microsoft.com>
Date: Fri, 14 Aug 2020 23:48:28 +0000
Subject: [PATCH] Cherry-pick PR #40061 into release-4.0

Component commits:
54f988cd9f Add '(approximate)' to the beginning of quick info requests.
---
 src/services/services.ts                         | 13 +++++++++++--
 .../unittests/tsserver/partialSemanticServer.ts  | 16 ++++++++++++++++
 2 files changed, 27 insertions(+), 2 deletions(-)

diff --git a/src/services/services.ts b/src/services/services.ts
index b8300aba0c34c..bb567064e8e6c 100644
--- a/src/services/services.ts
+++ b/src/services/services.ts
@@ -1587,7 +1587,9 @@ namespace ts {
                     kind: ScriptElementKind.unknown,
                     kindModifiers: ScriptElementKindModifier.none,
                     textSpan: createTextSpanFromNode(nodeForQuickInfo, sourceFile),
-                    displayParts: typeChecker.runWithCancellationToken(cancellationToken, typeChecker => typeToDisplayParts(typeChecker, type, getContainerNode(nodeForQuickInfo))),
+                    displayParts: prefixWithApproximate(
+                        typeChecker.runWithCancellationToken(cancellationToken, typeChecker => typeToDisplayParts(typeChecker, type, getContainerNode(nodeForQuickInfo)))
+                    ),
                     documentation: type.symbol ? type.symbol.getDocumentationComment(typeChecker) : undefined,
                     tags: type.symbol ? type.symbol.getJsDocTags() : undefined
                 };
@@ -1600,7 +1602,7 @@ namespace ts {
                 kind: symbolKind,
                 kindModifiers: SymbolDisplay.getSymbolModifiers(symbol),
                 textSpan: createTextSpanFromNode(nodeForQuickInfo, sourceFile),
-                displayParts,
+                displayParts: prefixWithApproximate(displayParts),
                 documentation,
                 tags,
             };
@@ -1630,6 +1632,13 @@ namespace ts {
             }
         }
 
+        function prefixWithApproximate(displayParts: SymbolDisplayPart[]): SymbolDisplayPart[] {
+            if (languageServiceMode === LanguageServiceMode.Semantic) {
+                return displayParts;
+            }
+            return [textPart("(approximate)"), lineBreakPart(), ...displayParts];
+        }
+
         /// Goto definition
         function getDefinitionAtPosition(fileName: string, position: number): readonly DefinitionInfo[] | undefined {
             synchronizeHostData();
diff --git a/src/testRunner/unittests/tsserver/partialSemanticServer.ts b/src/testRunner/unittests/tsserver/partialSemanticServer.ts
index d821ccc1a36bb..a8eca8b8bf555 100644
--- a/src/testRunner/unittests/tsserver/partialSemanticServer.ts
+++ b/src/testRunner/unittests/tsserver/partialSemanticServer.ts
@@ -30,6 +30,22 @@ import { something } from "something";
             return { host, session, file1, file2, file3, something, configFile };
         }
 
+        it("adds '(approximate)' to the description of quick info", () => {
+            const file: File = {
+                path: `${tscWatch.projectRoot}/foo.ts`,
+                content: "export const foo = 100;"
+            };
+            const host = createServerHost([file]);
+            const session = createSession(host, { serverMode: LanguageServiceMode.PartialSemantic, useSingleInferredProject: true });
+            openFilesForSession([file], session);
+            const response = session.executeCommandSeq<protocol.QuickInfoRequest>({
+                command: ts.server.protocol.CommandTypes.Quickinfo,
+                arguments: protocolFileLocationFromSubstring(file, "foo"),
+            }).response as protocol.QuickInfoResponseBody;
+
+            assert(stringContainsAt(response.displayString, "(approximate)", 0));
+        });
+
         it("open files are added to inferred project even if config file is present and semantic operations succeed", () => {
             const { host, session, file1, file2 } = setup();
             const service = session.getProjectService();