From 75f20f9b65b6880aa400882725e7bb3f6162836c Mon Sep 17 00:00:00 2001 From: Vyacheslav Pukhanov Date: Thu, 31 May 2018 00:51:08 +0300 Subject: [PATCH] fixUnusedIdentifier: fix "Remove variable statement" codefix (#24497) * fixUnusedIdentifier: fix "Remove variable statement" codefix * Add test for a single "Remove variable statement" codefix in a file --- src/services/codefixes/fixUnusedIdentifier.ts | 2 +- ...eFixUnusedIdentifier_removeVariableStatement.ts | 14 ++++++++++++++ 2 files changed, 15 insertions(+), 1 deletion(-) create mode 100644 tests/cases/fourslash/codeFixUnusedIdentifier_removeVariableStatement.ts diff --git a/src/services/codefixes/fixUnusedIdentifier.ts b/src/services/codefixes/fixUnusedIdentifier.ts index abff1a5a02f..c97902e7713 100644 --- a/src/services/codefixes/fixUnusedIdentifier.ts +++ b/src/services/codefixes/fixUnusedIdentifier.ts @@ -29,7 +29,7 @@ namespace ts.codefix { } const delVar = textChanges.ChangeTracker.with(context, t => tryDeleteFullVariableStatement(t, sourceFile, startToken, /*deleted*/ undefined)); if (delVar.length) { - return [createCodeFixAction(fixName, delDestructure, Diagnostics.Remove_variable_statement, fixIdDelete, Diagnostics.Delete_all_unused_declarations)]; + return [createCodeFixAction(fixName, delVar, Diagnostics.Remove_variable_statement, fixIdDelete, Diagnostics.Delete_all_unused_declarations)]; } const token = getToken(sourceFile, textSpanEnd(context.span)); diff --git a/tests/cases/fourslash/codeFixUnusedIdentifier_removeVariableStatement.ts b/tests/cases/fourslash/codeFixUnusedIdentifier_removeVariableStatement.ts new file mode 100644 index 00000000000..f57ee074097 --- /dev/null +++ b/tests/cases/fourslash/codeFixUnusedIdentifier_removeVariableStatement.ts @@ -0,0 +1,14 @@ +/// + +// @noUnusedLocals: true + +////function f() { +//// let a = 1, b = 2, c = 3; +////} + +verify.codeFix({ + description: "Remove variable statement", + newFileContent: +`function f() { +}`, +});