"Convert parameters to destructured object": enable for functions wit…

…h just one parameter (#46945) * "Convert parameters to destructured object": enable for functions with just one parameter Fixes #41753 * Add test
microsoft · Feb 3, 2022 · b7d0117 · b7d0117
1 parent 9b0f01a
commit b7d0117
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 1 deletion.
diff --git a/src/services/refactors/convertParamsToDestructuredObject.ts b/src/services/refactors/convertParamsToDestructuredObject.ts
@@ -1,7 +1,7 @@
 /* @internal */
 namespace ts.refactor.convertParamsToDestructuredObject {
     const refactorName = "Convert parameters to destructured object";
-    const minimumParameterLength = 2;
+    const minimumParameterLength = 1;
     const refactorDescription = getLocaleSpecificMessage(Diagnostics.Convert_parameters_to_destructured_object);
 
     const toDestructuredAction = {

diff --git a/...s/fourslash/refactorConvertParamsToDestructuredObject_arrowFunctionWithSingleParameter.ts b/...s/fourslash/refactorConvertParamsToDestructuredObject_arrowFunctionWithSingleParameter.ts
@@ -0,0 +1,13 @@
+/// <reference path='fourslash.ts' />
+
+////const foo = /*a*/(a: number)/*b*/ => { };
+////foo(1);
+
+goTo.select("a", "b");
+edit.applyRefactor({
+    refactorName: "Convert parameters to destructured object",
+    actionName: "Convert parameters to destructured object",
+    actionDescription: "Convert parameters to destructured object",
+    newContent: `const foo = ({ a }: { a: number; }) => { };
+foo({ a: 1 });`,
+});