Enabling more quickInfo tests

sheetalkamat · sheetalkamat · commit 010b9b663e17 · 2014-09-30T17:10:27.000-07:00
diff --git a/tests/cases/fourslash/qualifiedName_import-declaration-with-variable-entity-names.ts b/tests/cases/fourslash/qualifiedName_import-declaration-with-variable-entity-names.ts
@@ -11,7 +11,7 @@
 ////var x = Alpha.[|{| "name" : "mem" |}x|]
 
 goTo.marker('import');
-verify.completionListContains('x', 'number');
+verify.completionListContains('x', '(var) Alpha.x: number');
 
 var def: FourSlashInterface.Range = test.ranges().filter(range => range.marker.data.name === "def")[0];
 var imp: FourSlashInterface.Range = test.ranges().filter(range => range.marker.data.name === "import")[0];
diff --git a/tests/cases/fourslash/quickInfoExportAssignmentOfGenericInterface.ts b/tests/cases/fourslash/quickInfoExportAssignmentOfGenericInterface.ts
@@ -13,4 +13,4 @@
 
 goTo.file("quickInfoExportAssignmentOfGenericInterface_1.ts");
 goTo.marker('1');
-verify.quickInfoIs("a<a<string>>", undefined, "x", "var");
+verify.quickInfoIs("(var) x: a<a<string>>", undefined);
diff --git a/tests/cases/fourslash/quickInfoForAliasedGeneric.ts b/tests/cases/fourslash/quickInfoForAliasedGeneric.ts
@@ -7,11 +7,11 @@
 ////    }
 ////}
 ////import d = M.N;
-////var aa/*1*/: d.C<number>;
-////var bb/*2*/: d.D;
+////var /*1*/aa: d.C<number>;
+////var /*2*/bb: d.D;
 
 goTo.marker('1');
-verify.quickInfoIs('M.N.C<number>');
+verify.quickInfoIs('(var) aa: d.C<number>');
 
 goTo.marker('2');
-verify.quickInfoIs('M.N.D');
+verify.quickInfoIs('(var) bb: d.D');
diff --git a/tests/cases/fourslash/quickInfoForContextuallyTypedFunctionInReturnStatement.ts b/tests/cases/fourslash/quickInfoForContextuallyTypedFunctionInReturnStatement.ts
@@ -17,4 +17,4 @@
 
 
 goTo.marker();
-verify.quickInfoIs("number");
+verify.quickInfoIs("(parameter) value: number");
diff --git a/tests/cases/fourslash/quickInfoForDerivedGenericTypeWithConstructor.ts b/tests/cases/fourslash/quickInfoForDerivedGenericTypeWithConstructor.ts
@@ -10,12 +10,11 @@
 ////class B2<T> extends A<T> {
 ////    bar() { }
 ////}
-
-////var b/*1*/: B<number>;
-////var b2/*2*/: B<number>;
+////var /*1*/b: B<number>;
+////var /*2*/b2: B<number>;
 
 goTo.marker('1');
-verify.quickInfoIs('B<number>');
+verify.quickInfoIs('(var) b: B<number>');
 
 goTo.marker('2');
-verify.quickInfoIs('B<number>');
+verify.quickInfoIs('(var) b2: B<number>');
diff --git a/tests/cases/fourslash/quickInfoForFunctionDeclaration.ts b/tests/cases/fourslash/quickInfoForFunctionDeclaration.ts
@@ -4,17 +4,15 @@
 ////
 ////function ma/*makeA*/keA<T>(t: T): A<T> { return null; }
 ////
-////function f/*f*/<T>(t: T) {
+////function /*f*/f<T>(t: T) {
 ////    return makeA(t);
 ////}
 ////
 ////var x = f(0);
 ////var y = makeA(0);
 
-
-
 goTo.marker("makeA");
-verify.quickInfoIs("<T>(t: T): A<T>", undefined, "makeA", "function");
+verify.quickInfoIs("(function) makeA<T>(t: T): A<T>", undefined);
 
 goTo.marker("f");
-verify.quickInfoIs("<T>(t: T): A<T>", undefined, "f", "function");
+verify.quickInfoIs("(function) f<T>(t: T): A<T>", undefined);
diff --git a/tests/cases/fourslash/quickInfoForGenericConstraints1.ts b/tests/cases/fourslash/quickInfoForGenericConstraints1.ts
@@ -4,4 +4,6 @@
 ////function foo4<T extends Date>(test: any): any { return null; }
 
 goTo.marker();
-verify.quickInfoIs('T extends Date');
+// TODO: formatting type parameter with extends info
+//verify.quickInfoIs('parameter) test: T extends Date');
+verify.quickInfoIs('(parameter) test: T');
diff --git a/tests/cases/fourslash/quickInfoForIndexerResultWithConstraint.ts b/tests/cases/fourslash/quickInfoForIndexerResultWithConstraint.ts
@@ -6,8 +6,8 @@
 
 ////function other2<T extends Date>(arg: T) {
 ////    var b: { [x: string]: T };
-////    var r2/*1*/ = foo(b); // just shows T
+////    var /*1*/r2 = foo(b); // just shows T
 ////}
 
 goTo.marker('1');
-verify.quickInfoIs('{ [x: string]: T; }');
+verify.quickInfoIs('(local var) r2: {\n    [x: string]: T;\n}');
diff --git a/tests/cases/fourslash/quickInfoForOverloadOnConst1.ts b/tests/cases/fourslash/quickInfoForOverloadOnConst1.ts
@@ -20,22 +20,22 @@
 ////c.x1(1, (x/*10*/x) => { return 1; } );
 
 goTo.marker('1');
-verify.quickInfoIs("(a: number, callback: (x: 'hi') => number): any");
+verify.quickInfoIs("(method) I.x1(a: number, callback: (x: 'hi') => number): any");
 goTo.marker('2');
-verify.quickInfoIs("(a: number, callback: (x: 'hi') => number): any (+ 0 overload(s))");
+verify.quickInfoIs("(method) C.x1(a: number, callback: (x: 'hi') => number): any");
 goTo.marker('3');
-verify.quickInfoIs("(x: 'hi') => number");
+verify.quickInfoIs("(parameter) callback: (x: 'hi') => number");
 goTo.marker('4');
-verify.quickInfoIs("(a: number, callback: (x: 'hi') => number): any (+ 0 overload(s))");
+verify.quickInfoIs("(method) C.x1(a: number, callback: (x: 'hi') => number): any");
 goTo.marker('5');
-verify.quickInfoIs('(x: string) => number');
+verify.quickInfoIs('(parameter) callback: (x: string) => number');
 goTo.marker('6');
-verify.quickInfoIs('(x: string): number');
+verify.quickInfoIs('(local function) callback(x: string): number');
 goTo.marker('7');
-verify.quickInfoIs("(a: number, callback: (x: 'hi') => number): any (+ 0 overload(s))");
+verify.quickInfoIs("(method) C.x1(a: number, callback: (x: 'hi') => number): any");
 goTo.marker('8');
-verify.quickInfoIs("'hi'");
+verify.quickInfoIs("(parameter) xx: 'hi'");
 goTo.marker('9');
-verify.quickInfoIs("'bye'");
+verify.quickInfoIs("(parameter) xx: 'bye'");
 goTo.marker('10');
-verify.quickInfoIs("'hi'");
+verify.quickInfoIs("(parameter) xx: 'hi'");
diff --git a/tests/cases/fourslash/quickInfoForTypeofParameter.ts b/tests/cases/fourslash/quickInfoForTypeofParameter.ts
@@ -6,10 +6,10 @@
 ////}
 
 goTo.marker('ref2');
-verify.quickInfoIs("string", undefined, "y1", "local var");
+verify.quickInfoIs("(local var) y1: string", undefined);
 
 goTo.marker('ref1');
-verify.quickInfoIs("string", undefined, "y1", "local var");
+verify.quickInfoIs("(local var) y1: string", undefined);
 
 goTo.marker('ref2');
-verify.quickInfoIs("string", undefined, "y1", "local var");
+verify.quickInfoIs("(local var) y1: string", undefined);
diff --git a/tests/cases/fourslash/quickInfoInFunctionTypeReference.ts b/tests/cases/fourslash/quickInfoInFunctionTypeReference.ts
@@ -8,7 +8,7 @@
 edit.insert('');
 
 goTo.marker("1");
-verify.quickInfoIs("string", undefined, "variable1", "parameter");
+verify.quickInfoIs("(parameter) variable1: string", undefined);
 
 goTo.marker("2");
-verify.quickInfoIs("string", undefined, "variable2", "parameter");
+verify.quickInfoIs("(parameter) variable2: string", undefined);
diff --git a/tests/cases/fourslash/quickInfoInFunctionTypeReference2.ts b/tests/cases/fourslash/quickInfoInFunctionTypeReference2.ts
@@ -1,18 +1,18 @@
 /// <reference path='fourslash.ts'/>
 
 ////class C<T> {
-////    map(fn: (k/*1*/: string, value/*2*/: T, context: any) => void, context: any) {
+////    map(fn: (/*1*/k: string, /*2*/value: T, context: any) => void, context: any) {
 ////    }
 ////}
 
 ////var c: C<number>;
 ////c.map(/*3*/
 
-//goTo.marker('1');
-//verify.quickInfoIs('string');
+goTo.marker('1');
+verify.quickInfoIs('(parameter) k: string');
 
-//goTo.marker('2');
-//verify.quickInfoIs('T');
+goTo.marker('2');
+verify.quickInfoIs('(parameter) value: T');
 
 goTo.marker('3');
 verify.currentSignatureHelpIs('map(fn: (k: string, value: number, context: any) => void, context: any): void');
diff --git a/tests/cases/fourslash/quickInfoInInvalidIndexSignature.ts b/tests/cases/fourslash/quickInfoInInvalidIndexSignature.ts
@@ -0,0 +1,6 @@
+/// <reference path="fourslash.ts" />
+
+//// function method() { var /**/dictionary = <{ [index]: string; }>{}; }
+
+goTo.marker();
+verify.quickInfoIs('(local var) dictionary: {}');
diff --git a/tests/cases/fourslash/quickInfoInObjectLiteral.ts b/tests/cases/fourslash/quickInfoInObjectLiteral.ts
@@ -19,7 +19,7 @@
 ////}
 
 goTo.marker("1");
-verify.quickInfoIs("() => string", undefined, "y1", "property");
+verify.quickInfoIs("(property) y1: () => string", undefined);
 
 goTo.marker("2");
-verify.quickInfoIs("number");
+verify.quickInfoIs("(var) value: number");
diff --git a/tests/cases/fourslash/quickInfoOfGenericTypeAssertions1.ts b/tests/cases/fourslash/quickInfoOfGenericTypeAssertions1.ts
@@ -0,0 +1,17 @@
+/// <reference path='fourslash.ts'/>
+
+////function f<T>(x: T): T { return null; }
+////var /*1*/r = <T>(x: T) => x;
+////var /*2*/r2 = < <T>(x: T) => T>f;
+
+////var a;
+////var /*3*/r3 = < <T>(x: <A>(y: A) => A) => T>a;
+
+goTo.marker('1');
+verify.quickInfoIs('(var) r: <T>(x: T) => T');
+
+goTo.marker('2');
+verify.quickInfoIs('(var) r2: <T>(x: T) => T');
+
+goTo.marker('3');
+verify.quickInfoIs('(var) r3: <T>(x: <A>(y: A) => A) => T');
diff --git a/tests/cases/fourslash/quickInfoOfStringPropertyNames1.ts b/tests/cases/fourslash/quickInfoOfStringPropertyNames1.ts
@@ -5,7 +5,7 @@
 ////}
 
 ////var f: foo;
-////var r/*1*/ = f['foo bar'];
+////var /*1*/r = f['foo bar'];
 
 ////class bar {
 ////    'hello world': number;
@@ -16,18 +16,18 @@
 ////}
 
 ////var b: bar;
-////var r2/*2*/ = b["hello world"];
-////var r4/*3*/ = b['1'];
-////var r5/*4*/ = b[1];
+////var /*2*/r2 = b["hello world"];
+////var /*3*/r4 = b['1'];
+////var /*4*/r5 = b[1];
 
 goTo.marker('1');
-verify.quickInfoIs('string');
+verify.quickInfoIs('(var) r: string');
 
 goTo.marker('2');
-verify.quickInfoIs('number');
+verify.quickInfoIs('(var) r2: number');
 
 goTo.marker('3');
-verify.quickInfoIs('string');
+verify.quickInfoIs('(var) r4: string');
 
 goTo.marker('4');
-verify.quickInfoIs('string');
+verify.quickInfoIs('(var) r5: string');
diff --git a/tests/cases/fourslash/quickInfoOnCatchVariable.ts b/tests/cases/fourslash/quickInfoOnCatchVariable.ts
@@ -6,3 +6,4 @@
 
 goTo.marker();
 verify.quickInfoExists();
+verify.quickInfoIs("(var) e: any");
diff --git a/tests/cases/fourslash/quickInfoOnClassMergedWithFunction.ts b/tests/cases/fourslash/quickInfoOnClassMergedWithFunction.ts
@@ -14,4 +14,4 @@
 ////}
 
 goTo.marker();
-verify.quickInfoIs("string", undefined, "myProp", "property");
+verify.quickInfoIs("(property) myProp: string", undefined);
diff --git a/tests/cases/fourslash/quickInfoOnConstructorWithGenericParameter.ts b/tests/cases/fourslash/quickInfoOnConstructorWithGenericParameter.ts
@@ -25,5 +25,5 @@ edit.insert("null,");
 verify.currentSignatureHelpIs("B(a: Foo<I>, b: number): B");
 edit.insert("10);");
 
-//goTo.marker("2");
-//verify.quickInfoIs("(a: Foo<I>, b: number): B", undefined, "B", "constructor");
+goTo.marker("2");
+verify.quickInfoIs("(constructor) B(a: Foo<I>, b: number): B", undefined);
diff --git a/tests/cases/fourslash/quickInfoOnMergedModule.ts b/tests/cases/fourslash/quickInfoOnMergedModule.ts
@@ -18,5 +18,5 @@
 diagnostics.setEditValidation(IncrementalEditValidation.None);
 
 goTo.marker('1');
-verify.quickInfoIs("string", undefined, "M2.A.foo", "property");
+verify.quickInfoIs("(property) M2.A.foo: string", undefined);
 verify.numberOfErrorsInCurrentFile(0);
diff --git a/tests/cases/fourslash/quickInfoOnObjectLiteralWithAccessors.ts b/tests/cases/fourslash/quickInfoOnObjectLiteralWithAccessors.ts
@@ -0,0 +1,26 @@
+/// <reference path='fourslash.ts' />
+
+////function /*1*/makePoint(x: number) { 
+////    return {
+////        b: 10,
+////        get x() { return x; },
+////        set x(a: number) { this.b = a; }
+////    };
+////};
+////var /*4*/point = makePoint(2);
+////var /*2*/x = point.x;
+////point./*3*/x = 30;
+
+goTo.marker('1');
+verify.quickInfoIs("(function) makePoint(x: number): {\n    b: number;\n    x: number;\n}", undefined);
+
+goTo.marker('2');
+verify.quickInfoIs("(var) x: number", undefined);
+
+goTo.marker('3');
+verify.memberListContains("x", "(property) x: number", undefined);
+verify.memberListContains("b", "(property) b: number", undefined);
+verify.quickInfoIs("(property) x: number", undefined);
+
+goTo.marker('4');
+verify.quickInfoIs("(var) point: {\n    b: number;\n    x: number;\n}", undefined);
diff --git a/tests/cases/fourslash/quickInfoOnObjectLiteralWithOnlyGetter.ts b/tests/cases/fourslash/quickInfoOnObjectLiteralWithOnlyGetter.ts
@@ -0,0 +1,21 @@
+/// <reference path='fourslash.ts' />
+
+////function /*1*/makePoint(x: number) { 
+////    return {
+////        get x() { return x; },
+////    };
+////};
+////var /*4*/point = makePoint(2);
+////var /*2*/x = point./*3*/x;
+
+goTo.marker('1');
+verify.quickInfoIs("(function) makePoint(x: number): {\n    x: number;\n}", undefined);
+
+goTo.marker('2');
+verify.quickInfoIs("(var) x: number", undefined);
+
+goTo.marker('3');
+verify.memberListContains("x", "(property) x: number", undefined);
+
+goTo.marker('4');
+verify.quickInfoIs("(var) point: {\n    x: number;\n}", undefined);
diff --git a/tests/cases/fourslash/quickInfoOnObjectLiteralWithOnlySetter.ts b/tests/cases/fourslash/quickInfoOnObjectLiteralWithOnlySetter.ts
@@ -0,0 +1,21 @@
+/// <reference path='fourslash.ts' />
+
+////function /*1*/makePoint(x: number) { 
+////    return {
+////        b: 10,
+////        set x(a: number) { this.b = a; }
+////    };
+////};
+////var /*3*/point = makePoint(2);
+////point./*2*/x = 30;
+
+goTo.marker('1');
+verify.quickInfoIs("(function) makePoint(x: number): {\n    b: number;\n    x: number;\n}", undefined);
+
+goTo.marker('2');
+verify.memberListContains("x", "(property) x: number", undefined);
+verify.memberListContains("b", "(property) b: number", undefined);
+verify.quickInfoIs("(property) x: number", undefined);
+
+goTo.marker('3');
+verify.quickInfoIs("(var) point: {\n    b: number;\n    x: number;\n}", undefined);
diff --git a/tests/cases/fourslash/quickInfoShowsGenericSpecialization.ts b/tests/cases/fourslash/quickInfoShowsGenericSpecialization.ts
@@ -0,0 +1,7 @@
+/// <reference path='fourslash.ts'/>
+
+////class A<T> { }
+////var /**/foo = new A<number>();
+
+goTo.marker();
+verify.quickInfoIs('(var) foo: A<number>');
diff --git a/tests/cases/fourslash_old/qualifyModuleTypeNames.ts b/tests/cases/fourslash_old/qualifyModuleTypeNames.ts
diff --git a/tests/cases/fourslash_old/quickInfoInFunctionTypeReference2.ts b/tests/cases/fourslash_old/quickInfoInFunctionTypeReference2.ts
diff --git a/tests/cases/fourslash_old/quickInfoInInvalidIndexSignature.ts b/tests/cases/fourslash_old/quickInfoInInvalidIndexSignature.ts
diff --git a/tests/cases/fourslash_old/quickInfoOfGenericTypeAssertions1.ts b/tests/cases/fourslash_old/quickInfoOfGenericTypeAssertions1.ts
diff --git a/tests/cases/fourslash_old/quickInfoOnConstructorWithGenericParameter.ts b/tests/cases/fourslash_old/quickInfoOnConstructorWithGenericParameter.ts
diff --git a/tests/cases/fourslash_old/quickInfoOnObjectLiteralWithAccessors.ts b/tests/cases/fourslash_old/quickInfoOnObjectLiteralWithAccessors.ts
diff --git a/tests/cases/fourslash_old/quickInfoOnObjectLiteralWithOnlyGetter.ts b/tests/cases/fourslash_old/quickInfoOnObjectLiteralWithOnlyGetter.ts
diff --git a/tests/cases/fourslash_old/quickInfoOnObjectLiteralWithOnlySetter.ts b/tests/cases/fourslash_old/quickInfoOnObjectLiteralWithOnlySetter.ts
diff --git a/tests/cases/fourslash_old/quickInfoShowsGenericSpecialization.ts b/tests/cases/fourslash_old/quickInfoShowsGenericSpecialization.ts

Original file line number	Diff line number	Diff line change
`@@ -17,4 +17,4 @@`
`17`	`17`
`18`	`18`
`19`	`19`	`goTo.marker();`
`20`		`-verify.quickInfoIs("number");`
	`20`	`+verify.quickInfoIs("(parameter) value: number");`