Add more test coverage for unusedParameterInLambda

This commit is contained in:
Andrew Casey 2018-01-11 10:51:55 -08:00
parent e354754b2a
commit 3b5689fa1f
4 changed files with 44 additions and 2 deletions

View file

@ -3,11 +3,12 @@
// @noUnusedLocals: true
// @noUnusedParameters: true
//// function f1() {
//// [|return (x:number) => {}|]
//// [|return /*~a*/(/*~b*/x/*~c*/:/*~d*/number/*~e*/)/*~f*/ => /*~g*/{/*~h*/}/*~i*/|]
//// }
// In a perfect world, /*~f*/ and /*~h*/ would probably be retained.
verify.codeFix({
description: "Remove declaration for: 'x'",
index: 0,
newRangeContent: "return () => {}",
newRangeContent: "return /*~a*/() => /*~g*/ { }/*~i*/",
});

View file

@ -0,0 +1,14 @@
/// <reference path='fourslash.ts' />
// @noUnusedLocals: true
// @noUnusedParameters: true
//// function f1() {
//// [|return /*~a*/x/*~b*/ /*~c*/=>/*~d*/ {/*~e*/}/*~f*/|]
//// }
// In a perfect world, /*~c*/ and /*~e*/ would probably be retained.
verify.codeFix({
description: "Remove declaration for: 'x'",
index: 0,
newRangeContent: "return /*~a*/() => /*~d*/ { }/*~f*/",
});

View file

@ -0,0 +1,14 @@
/// <reference path='fourslash.ts' />
// @noUnusedLocals: true
// @noUnusedParameters: true
//// function f1() {
//// [|return /*~a*/(/*~b*/x/*~c*/,/*~d*/y/*~e*/)/*~f*/ => /*~g*/x/*~h*/|]
//// }
// In a perfect world, /*~c*/ would probably be retained, rather than /*~e*/.
verify.codeFix({
description: "Remove declaration for: 'y'",
index: 0,
newRangeContent: "return /*~a*/(/*~b*/x/*~e*/)/*~f*/ => /*~g*/x/*~h*/",
});

View file

@ -0,0 +1,13 @@
/// <reference path='fourslash.ts' />
// @noUnusedLocals: true
// @noUnusedParameters: true
//// function f1() {
//// [|return /*~a*/(/*~b*/x/*~c*/,/*~d*/y/*~e*/)/*~f*/ => /*~g*/y/*~h*/|]
//// }
verify.codeFix({
description: "Remove declaration for: 'x'",
index: 0,
newRangeContent: "return /*~a*/(/*~d*/y/*~e*/)/*~f*/ => /*~g*/y/*~h*/",
});