feature: @putout/plugin-putout: convert-push-object-to-push-path: multiple fix

Author: coderaiser

packages/plugin-putout/lib/convert-push-object-to-push-path/fixture/multiple-fix.js

@@ -0,0 +1,20 @@
+module.exports.fix = ({path, from, to, toRename}) => {
+    if (isImportSpecifier(path)) {
+        path.node.imported = path.node.local;
+        return;
+    }
+    
+    if (toRename)
+        rename(path, from, to);
+    
+    path.node.shorthand = true;
+};
+
+module.exports.traverse = ({push, options}) => ({
+    ImportSpecifier(path) {
+        if (path.node.imported.value === path.node.local.name)
+            push({
+                path,
+            });
+    },
+});

…-object-to-push-path/fixture/multiple.js …ct-to-push-path/fixture/multiple-push.jspackages/plugin-putout/lib/convert-push-object-to-push-path/fixture/multiple.js renamed to packages/plugin-putout/lib/convert-push-object-to-push-path/fixture/multiple-push.js packages/plugin-putout/lib/convert-push-object-to-push-path/fixture/multiple.js renamed to packages/plugin-putout/lib/convert-push-object-to-push-path/fixture/multiple-push.js

@@ -0,0 +1,9 @@
+export const fix = ({path}) => {
+    removeClassName(path, 'menu-item-selected');
+};
+
+export const traverse = ({push}) => ({
+    JSXElement(path) {
+        push(path);
+    },
+});

packages/plugin-putout/lib/convert-push-object-to-push-path/fixture/no-rename-fix.js

@@ -0,0 +1,12 @@
+export const fix = ({path}) => {
+    removeClassName(path, 'menu-item-selected');
+};
+
+export const traverse = ({push}) => ({
+    JSXElement(path) {
+        push({
+            path,
+        });
+    },
+});
+

packages/plugin-putout/lib/convert-push-object-to-push-path/fixture/no-rename.js

@@ -5,6 +5,18 @@ const {traverse} = operator;
 export const report = () => `Use 'push(__a)' instead of 'push({path: __a})'`;
 
 export const match = () => ({
+    'push({path})': (vars, path) => {
+        let is = false;
+        const programPath = path.scope.getProgramParent().path;
+        
+        traverse(programPath, {
+            'export const fix = ({path}) => __body': () => {
+                is = true;
+            },
+        });
+        
+        return is;
+    },
     'export const fix = ({path}) => __a': (vars, {parentPath}) => {
         let is = false;
 
@@ -20,5 +32,6 @@ export const match = () => ({
 
 export const replace = () => ({
     'push({path: __a})': 'push(__a)',
+    'push({path})': 'push(path)',
     'export const fix = ({path}) => __a': 'export const fix = (path) => __a',
 });

packages/plugin-putout/lib/convert-push-object-to-push-path/index.js

@@ -17,7 +17,17 @@ test('putout: convert-push-object-to-push-path: transform', (t) => {
     t.end();
 });
 
-test('putout: convert-push-object-to-push-path: no report: multiple', (t) => {
-    t.noReport('multiple');
+test('putout: convert-push-object-to-push-path: no report: multiple-push', (t) => {
+    t.noReport('multiple-push');
+    t.end();
+});
+
+test('putout: convert-push-object-to-push-path: no report: multiple-fix', (t) => {
+    t.noReport('multiple-fix');
+    t.end();
+});
+
+test('putout: convert-push-object-to-push-path: transform: no-rename', (t) => {
+    t.transform('no-rename');
     t.end();
 });