Fix incorrect noImplicitAny error on contextual union function signature

andrewbranch · andrewbranch · commit 252840ad40f3 · 2019-06-24T17:37:14.000-07:00
diff --git a/src/compiler/checker.ts b/src/compiler/checker.ts
@@ -19095,7 +19095,9 @@ namespace ts {
                 }
             }
             // Result is union of signatures collected (return type is union of return types of this signature set)
-            return signatureList && createUnionSignature(signatureList[0], signatureList);
+            if (signatureList) {
+                return signatureList.length === 1 ? signatureList[0] : createUnionSignature(signatureList[0], signatureList);
+            }
         }
 
         function checkSpreadExpression(node: SpreadElement, checkMode?: CheckMode): Type {
diff --git a/tests/baselines/reference/functionExpressionContextualTyping3.js b/tests/baselines/reference/functionExpressionContextualTyping3.js
@@ -0,0 +1,8 @@
+//// [functionExpressionContextualTyping3.ts]
+// #31114
+declare function f<T>(value: T | number): void;
+f((a: any) => "")
+
+
+//// [functionExpressionContextualTyping3.js]
+f(function (a) { return ""; });
diff --git a/tests/baselines/reference/functionExpressionContextualTyping3.symbols b/tests/baselines/reference/functionExpressionContextualTyping3.symbols
@@ -0,0 +1,12 @@
+=== tests/cases/conformance/expressions/contextualTyping/functionExpressionContextualTyping3.ts ===
+// #31114
+declare function f<T>(value: T | number): void;
+>f : Symbol(f, Decl(functionExpressionContextualTyping3.ts, 0, 0))
+>T : Symbol(T, Decl(functionExpressionContextualTyping3.ts, 1, 19))
+>value : Symbol(value, Decl(functionExpressionContextualTyping3.ts, 1, 22))
+>T : Symbol(T, Decl(functionExpressionContextualTyping3.ts, 1, 19))
+
+f((a: any) => "")
+>f : Symbol(f, Decl(functionExpressionContextualTyping3.ts, 0, 0))
+>a : Symbol(a, Decl(functionExpressionContextualTyping3.ts, 2, 3))
+
diff --git a/tests/baselines/reference/functionExpressionContextualTyping3.types b/tests/baselines/reference/functionExpressionContextualTyping3.types
@@ -0,0 +1,13 @@
+=== tests/cases/conformance/expressions/contextualTyping/functionExpressionContextualTyping3.ts ===
+// #31114
+declare function f<T>(value: T | number): void;
+>f : <T>(value: number | T) => void
+>value : number | T
+
+f((a: any) => "")
+>f((a: any) => "") : void
+>f : <T>(value: number | T) => void
+>(a: any) => "" : (a: any) => ""
+>a : any
+>"" : ""
+
diff --git a/tests/cases/conformance/expressions/contextualTyping/functionExpressionContextualTyping3.ts b/tests/cases/conformance/expressions/contextualTyping/functionExpressionContextualTyping3.ts
@@ -0,0 +1,5 @@
+// @noImplicitAny: true
+
+// #31114
+declare function f<T>(value: T | number): void;
+f((a: any) => "")

Original file line number	Diff line number	Diff line change
`@@ -19095,7 +19095,9 @@ namespace ts {`
`19095`	`19095`	`}`
`19096`	`19096`	`}`
`19097`	`19097`	`// Result is union of signatures collected (return type is union of return types of this signature set)`
`19098`		`- return signatureList && createUnionSignature(signatureList[0], signatureList);`
	`19098`	`+ if (signatureList) {`
	`19099`	`+ return signatureList.length === 1 ? signatureList[0] : createUnionSignature(signatureList[0], signatureList);`
	`19100`	`+ }`
`19099`	`19101`	`}`
`19100`	`19102`
`19101`	`19103`	`function checkSpreadExpression(node: SpreadElement, checkMode?: CheckMode): Type {`