Skip to content

Commit 788edc3

Browse files
committed
feature: @putout/plugin-remove-useless-arguments: arguments: crawl
1 parent f3467a7 commit 788edc3

File tree

9 files changed

+70
-2
lines changed

9 files changed

+70
-2
lines changed
Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,8 @@
1+
async function call(data, panelParam, options) {
2+
const callback = () => {};
3+
return await createFileTable(data, panelParam, options);
4+
}
5+
6+
async function createFileTable(a, b, c) {
7+
return a + b + c;
8+
}
Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,9 @@
1+
async function call(data, panelParam, options) {
2+
const callback = () => {};
3+
return await createFileTable(data, panelParam, options, callback);
4+
}
5+
6+
async function createFileTable(a, b, c) {
7+
return a + b + c;
8+
}
9+
Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
async function call() {
2+
return await createFileTable(data, panelParam, options);
3+
}
4+
5+
async function createFileTable(data, panelParam, options) {
6+
return data + panelParam + options;
7+
}
Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,8 @@
1+
async function call() {
2+
return await createFileTable(data, panelParam, options, callback);
3+
}
4+
5+
async function createFileTable(data, panelParam, options) {
6+
return data + panelParam + options;
7+
}
8+

packages/plugin-remove-useless-arguments/lib/arguments/index.js

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -71,8 +71,6 @@ function isArguments(path) {
7171

7272
if (name === 'arguments')
7373
is = true;
74-
75-
path.stop();
7674
},
7775
});
7876

packages/plugin-remove-useless-arguments/lib/arguments/index.spec.js

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
import {createTest} from '@putout/test';
22
import * as plugin from './index.js';
3+
import * as unused from '../unused/index.js';
34

45
const test = createTest(import.meta.url, {
56
plugins: [
@@ -42,6 +43,18 @@ test('plugin-remove-useless-arguments: arguments: no transform: arguments', (t)
4243
t.end();
4344
});
4445

46+
test('plugin-remove-useless-arguments: arguments: transform: used', (t) => {
47+
t.transform('used');
48+
t.end();
49+
});
50+
51+
test('plugin-remove-useless-arguments: arguments: transform: unused', (t) => {
52+
t.transform('unused', {
53+
unused,
54+
});
55+
t.end();
56+
});
57+
4558
test('plugin-remove-useless-arguments: arguments: report: fn', (t) => {
4659
t.report('fn', 'Avoid useless argument');
4760
t.end();
Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,8 @@
1+
async function call(data, panelParam, options) {
2+
const callback = () => {};
3+
return await createFileTable(data, panelParam, options);
4+
}
5+
6+
async function createFileTable(a, b, c) {
7+
return a + b + c;
8+
}
Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,9 @@
1+
async function call(data, panelParam, options) {
2+
const callback = () => {};
3+
return await createFileTable(data, panelParam, options, callback);
4+
}
5+
6+
async function createFileTable(a, b, c) {
7+
return a + b + c;
8+
}
9+

packages/plugin-remove-useless-arguments/lib/unused/index.spec.js

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
import {createTest} from '@putout/test';
22
import * as plugin from './index.js';
3+
import * as argumentsPlugin from '../arguments/index.js';
34

45
const test = createTest(import.meta.url, {
56
plugins: [
@@ -52,6 +53,13 @@ test('remove-useless-arguments: unused: no report: export', (t) => {
5253
t.end();
5354
});
5455

56+
test('remove-useless-arguments: unused: transform: used', (t) => {
57+
t.transform('used', {
58+
argumentsPlugin,
59+
});
60+
t.end();
61+
});
62+
5563
test('remove-useless-arguments: unused: transform: anonymous', (t) => {
5664
t.transform('anonymous');
5765
t.end();

0 commit comments

Comments
 (0)