More multi-variable support

openrewrite · Oct 4, 2024 · 2c40069 · 2c40069
1 parent ee68354
commit 2c40069
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 4 deletions.
diff --git a/openrewrite/src/javascript/parser.ts b/openrewrite/src/javascript/parser.ts
@@ -1255,10 +1255,9 @@ export class JavaScriptParserVisitor {
         );
     }
 
-    visitVariableStatement(node: ts.VariableStatement): J.VariableDeclarations | J.Unknown {
+    visitVariableStatement(node: ts.VariableStatement) {
         if (node.declarationList.declarations.length > 1) {
-            // we can't map this to a `J.VariableDeclarations` because the variables can all declare their own type
-            return this.visitUnknown(node);
+            return this.visitVariableDeclarationList(node.declarationList);
         }
         return new J.VariableDeclarations(
             randomId(),

diff --git a/openrewrite/test/javascript/parser/variableDeclarations.test.ts b/openrewrite/test/javascript/parser/variableDeclarations.test.ts
@@ -26,7 +26,6 @@ describe('variable declaration mapping', () => {
             })
         );
     });
-
     test('const2', () => {
         rewriteRun(
           typeScript(
@@ -48,4 +47,16 @@ describe('variable declaration mapping', () => {
             })
         );
     });
+    test('multi', () => {
+        rewriteRun(
+          //language=typescript
+          typeScript('let a=2, b=2 ')
+        );
+    });
+    test('multi typed', () => {
+        rewriteRun(
+          //language=typescript
+          typeScript('let a: number =2, b: string = "2" ')
+        );
+    });
 });