Add test suits and tests

Author: Kingwl

src/harness/fourslashImpl.ts

@@ -824,6 +824,18 @@ namespace FourSlash {
             });
         }
 
+        public verifyInlineHints(expected: readonly FourSlashInterface.VerifyInlineHintsOptions[], span: ts.TextSpan = { start: 0, length: this.activeFile.content.length }, preference?: ts.InlineHintsOptions) {
+            const hints = this.languageService.provideInlineHints(this.activeFile.fileName, span, preference);
+            assert.equal(hints.length, expected.length, "Number of hints");
+            const sortHints = (a: ts.InlineHint, b: ts.InlineHint) => a.position - b.position;
+            ts.zipWith(hints.sort(sortHints), [...expected].sort(sortHints), (actual, expected) => {
+                assert.equal(actual.text, expected.text, "Text");
+                assert.equal(actual.position, expected.position, "Position");
+                assert.equal(actual.whitespaceBefore, expected.whitespaceBefore, "whitespaceBefore");
+                assert.equal(actual.whitespaceAfter, expected.whitespaceAfter, "whitespaceAfter");
+            });
+        }
+
         public verifyCompletions(options: FourSlashInterface.VerifyCompletionsOptions) {
             if (options.marker === undefined) {
                 this.verifyCompletionsWorker(options);

src/harness/fourslashInterfaceImpl.ts

@@ -243,6 +243,10 @@ namespace FourSlashInterface {
             }
         }
 
+        public getInlineHints(expected: readonly VerifyInlineHintsOptions[], span: ts.TextSpan, preference?: ts.InlineHintsOptions) {
+            this.state.verifyInlineHints(expected, span, preference);
+        }
+
         public quickInfoIs(expectedText: string, expectedDocumentation?: string) {
             this.state.verifyQuickInfoString(expectedText, expectedDocumentation);
         }
@@ -1645,6 +1649,13 @@ namespace FourSlashInterface {
         readonly containerKind?: ts.ScriptElementKind;
     }
 
+    export interface VerifyInlineHintsOptions {
+        text: string;
+        position: number
+        whitespaceBefore?: boolean;
+        whitespaceAfter?: boolean;
+    }
+
     export type ArrayOrSingle<T> = T | readonly T[];
 
     export interface VerifyCompletionListContainsOptions extends ts.UserPreferences {

src/services/inlineHints.ts

@@ -7,7 +7,7 @@ namespace ts.InlineHints {
         whitespaceAfter?: boolean;
     }
 
-    const maxHintsLength = 20;
+    const maxHintsLength = 30;
 
     export function provideInlineHints(context: InlineHintsContext): HintInfo[] {
         const { file, program, span, cancellationToken, preferences } = context;

tests/cases/fourslash/fourslash.ts

@@ -381,6 +381,7 @@ declare namespace FourSlashInterface {
             start: number;
             length: number;
         }, displayParts: ts.SymbolDisplayPart[], documentation: ts.SymbolDisplayPart[], tags: { name: string, text?: string }[] | undefined): void;
+        getInlineHints(expected: readonly VerifyInlineHintsOptions[], span?: TextSpan, preference?: InlineHintsOptions);
         getSyntacticDiagnostics(expected: ReadonlyArray<Diagnostic>): void;
         getSemanticDiagnostics(expected: ReadonlyArray<Diagnostic>): void;
         getSuggestionDiagnostics(expected: ReadonlyArray<Diagnostic>): void;
@@ -614,6 +615,11 @@ declare namespace FourSlashInterface {
         readonly importModuleSpecifierPreference?: "shortest" | "project-relative" | "relative" | "non-relative";
         readonly importModuleSpecifierEnding?: "minimal" | "index" | "js";
     }
+    interface InlineHintsOptions extends UserPreferences {
+        readonly includeInlineParameterName?: boolean;
+        readonly includeInlineFunctionParameterType?: boolean;
+        readonly includeInlineVariableType?: boolean;
+    }
     interface CompletionsOptions {
         readonly marker?: ArrayOrSingle<string | Marker>;
         readonly isNewIdentifierLocation?: boolean;
@@ -711,6 +717,14 @@ declare namespace FourSlashInterface {
         readonly commands?: ReadonlyArray<{}>;
     }
 
+    export interface VerifyInlineHintsOptions {
+        text: string;
+        position: number
+        whitespaceBefore?: boolean;
+        whitespaceAfter?: boolean;
+    }
+
+
     interface VerifyNavigateToOptions {
         readonly pattern: string;
         readonly fileName?: string;

tests/cases/fourslash/inlineHintsShouldWork1.ts

@@ -0,0 +1,20 @@
+/// <reference path="fourslash.ts" />
+
+//// function foo (a: number, b: number) {}
+//// foo(/*a*/1, /*b*/2);
+
+const markers = test.markers();
+verify.getInlineHints([
+    {
+        text: 'a:',
+        position: markers[0].position,
+        whitespaceAfter: true
+    },
+    {
+        text: 'b:',
+        position: markers[1].position,
+        whitespaceAfter: true
+    }
+], undefined, {
+    includeInlineParameterName: true
+});

tests/cases/fourslash/inlineHintsShouldWork10.ts

@@ -0,0 +1,9 @@
+/// <reference path="fourslash.ts" />
+
+//// declare const unknownCall: any;
+//// unknownCall();
+
+const markers = test.markers();
+verify.getInlineHints([], undefined, {
+    includeInlineParameterName: true
+});

tests/cases/fourslash/inlineHintsShouldWork11.ts

@@ -0,0 +1,24 @@
+/// <reference path="fourslash.ts" />
+
+//// function foo(a: number) {
+////     return (b: number) => {
+////         return a + b
+////     }
+//// }
+//// foo(/*a*/1)(/*b*/2);
+
+const markers = test.markers();
+verify.getInlineHints([
+    {
+        text: 'a:',
+        position: markers[0].position,
+        whitespaceAfter: true
+    },
+    {
+        text: 'b:',
+        position: markers[1].position,
+        whitespaceAfter: true
+    },
+], undefined, {
+    includeInlineParameterName: true
+});

tests/cases/fourslash/inlineHintsShouldWork12.ts

@@ -0,0 +1,23 @@
+/// <reference path="fourslash.ts" />
+
+//// function foo(a: (b: number) => number) {
+////     return a(/*a*/1) + 2
+//// }
+
+//// foo(/*b*/(c: number) => c + 1);
+
+const markers = test.markers();
+verify.getInlineHints([
+    {
+        text: 'b:',
+        position: markers[0].position,
+        whitespaceAfter: true
+    },
+    {
+        text: 'a:',
+        position: markers[1].position,
+        whitespaceAfter: true
+    },
+], undefined, {
+    includeInlineParameterName: true
+});

tests/cases/fourslash/inlineHintsShouldWork13.ts

@@ -0,0 +1,16 @@
+/// <reference path="fourslash.ts" />
+
+//// function foo (a: number, b: number) {}
+//// declare const a: 1;
+//// foo(a, /*b*/2);
+
+const markers = test.markers();
+verify.getInlineHints([
+    {
+        text: 'b:',
+        position: markers[0].position,
+        whitespaceAfter: true
+    },
+], undefined, {
+    includeInlineParameterName: true
+});

tests/cases/fourslash/inlineHintsShouldWork14.ts

@@ -0,0 +1,8 @@
+/// <reference path="fourslash.ts" />
+
+//// function foo (a: number, b: number) {}
+//// foo(1, 2);
+
+verify.getInlineHints([], undefined, {
+    includeInlineParameterName: false
+});