Skip to content

Commit cec6c3a

Browse files
simonihmigsimonihmig
committed
Fix prettier
1 parent b247da2 commit cec6c3a

43 files changed

Lines changed: 736 additions & 577 deletions

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

‎bin/cli.js‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,6 @@
33

44
require('codemod-cli').runTransform(
55
__dirname,
6-
process.argv[2] /* transform name */,
6+
process.argv[2] /* transform name */,
77
process.argv.slice(3) /* paths or globs */
8-
)
8+
);

‎transforms/-preset.js‎

Lines changed: 16 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -3,30 +3,30 @@
33
const path = require('path');
44
const fs = require('fs');
55

6-
module.exports = function(type) {
6+
module.exports = function (type) {
77
let transformPath = path.join(__dirname, type, 'transforms');
88

9-
return function(file, api, options) {
9+
return function (file, api, options) {
1010
let src = file.source;
1111
let error;
1212

13-
fs.readdirSync(transformPath)
14-
.forEach(fileName => {
15-
let fix = require(path.join(transformPath, fileName));
16-
if (typeof(src) === 'undefined') {
17-
return;
18-
}
19-
try {
20-
src = fix(Object.assign({},file, { source: src }), api, options);
21-
} catch(e) {
22-
error = new Error(`Transformation ${fileName} errored on file ${file.path}. Reason ${e}. Please report this in https://github.com/simonihmig/ember-test-helpers-codemod/issues\n\nStack trace:\n${e.stack}\n\nSource:\n${src}`);
23-
}
24-
25-
});
13+
fs.readdirSync(transformPath).forEach((fileName) => {
14+
let fix = require(path.join(transformPath, fileName));
15+
if (typeof src === 'undefined') {
16+
return;
17+
}
18+
try {
19+
src = fix(Object.assign({}, file, { source: src }), api, options);
20+
} catch (e) {
21+
error = new Error(
22+
`Transformation ${fileName} errored on file ${file.path}. Reason ${e}. Please report this in https://github.com/simonihmig/ember-test-helpers-codemod/issues\n\nStack trace:\n${e.stack}\n\nSource:\n${src}`
23+
);
24+
}
25+
});
2626

2727
if (error) {
2828
throw error;
2929
}
3030
return src;
3131
};
32-
}
32+
};

‎transforms/acceptance/index.js‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,4 +2,4 @@
22

33
const preset = require('../-preset');
44

5-
module.exports = preset('acceptance');
5+
module.exports = preset('acceptance');

‎transforms/acceptance/test.js‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,4 +5,4 @@ const { runTransformTest } = require('codemod-cli');
55
runTransformTest({
66
type: 'jscodeshift',
77
name: 'acceptance',
8-
});
8+
});

‎transforms/acceptance/transforms/attr.js‎

Lines changed: 23 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,13 @@
11
'use strict';
22

33
const { getParser } = require('codemod-cli').jscodeshift;
4-
const { createFindExpression, createPropExpression, isFindExpression, addImportStatement, writeImportStatements } = require('../../utils');
4+
const {
5+
createFindExpression,
6+
createPropExpression,
7+
isFindExpression,
8+
addImportStatement,
9+
writeImportStatements,
10+
} = require('../../utils');
511

612
/**
713
* Creates a `find(selector).getAttribute(attr)` expression
@@ -13,10 +19,8 @@ const { createFindExpression, createPropExpression, isFindExpression, addImportS
1319
*/
1420
function createAttributeExpression(j, findArgs, attr) {
1521
return j.callExpression(
16-
j.memberExpression(
17-
createFindExpression(j, findArgs),
18-
j.identifier('getAttribute')
19-
), [attr]
22+
j.memberExpression(createFindExpression(j, findArgs), j.identifier('getAttribute')),
23+
[attr]
2024
);
2125
}
2226

@@ -28,12 +32,14 @@ function createAttributeExpression(j, findArgs, attr) {
2832
* @returns {*|boolean}
2933
*/
3034
function isJQueryExpression(j, node) {
31-
return j.CallExpression.check(node)
32-
&& j.MemberExpression.check(node.callee)
33-
&& isFindExpression(j, node.callee.object)
34-
&& j.Identifier.check(node.callee.property)
35-
&& node.callee.property.name === 'attr'
36-
&& node.arguments.length === 1;
35+
return (
36+
j.CallExpression.check(node) &&
37+
j.MemberExpression.check(node.callee) &&
38+
isFindExpression(j, node.callee.object) &&
39+
j.Identifier.check(node.callee.property) &&
40+
node.callee.property.name === 'attr' &&
41+
node.arguments.length === 1
42+
);
3743
}
3844
/**
3945
* Should we translate this to a `.getAttribute('foo')` or a `.foo` expression?
@@ -64,13 +70,17 @@ function transform(file, api) {
6470
.find(j.CallExpression)
6571
.filter(({ node }) => isJQueryExpression(j, node))
6672
.filter(({ node }) => isPropertyAccessPreferred(j, node))
67-
.replaceWith(({ node }) => createPropExpression(j, node.callee.object.arguments, node.arguments[0].value));
73+
.replaceWith(({ node }) =>
74+
createPropExpression(j, node.callee.object.arguments, node.arguments[0].value)
75+
);
6876

6977
let attrReplacements = root
7078
.find(j.CallExpression)
7179
.filter(({ node }) => isJQueryExpression(j, node))
7280
.filter(({ node }) => !isPropertyAccessPreferred(j, node))
73-
.replaceWith(({ node }) => createAttributeExpression(j, node.callee.object.arguments, node.arguments[0]));
81+
.replaceWith(({ node }) =>
82+
createAttributeExpression(j, node.callee.object.arguments, node.arguments[0])
83+
);
7484

7585
if (propReplacements.length > 0 || attrReplacements.length > 0) {
7686
addImportStatement(['find']);

‎transforms/acceptance/transforms/click.js‎

Lines changed: 10 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,12 @@
11
'use strict';
22

33
const { getParser } = require('codemod-cli').jscodeshift;
4-
const { makeAwait, dropAndThen, addImportStatement, writeImportStatements } = require('../../utils');
4+
const {
5+
makeAwait,
6+
dropAndThen,
7+
addImportStatement,
8+
writeImportStatements,
9+
} = require('../../utils');
510

611
/**
712
* Check if `node` is a `click(selector)` expression
@@ -11,9 +16,9 @@ const { makeAwait, dropAndThen, addImportStatement, writeImportStatements } = re
1116
* @returns {*|boolean}
1217
*/
1318
function isGlobalHelperExpression(j, node) {
14-
return j.CallExpression.check(node)
15-
&& j.Identifier.check(node.callee)
16-
&& node.callee.name === 'click';
19+
return (
20+
j.CallExpression.check(node) && j.Identifier.check(node.callee) && node.callee.name === 'click'
21+
);
1722
}
1823

1924
/**
@@ -31,9 +36,7 @@ function transform(file, api) {
3136

3237
let replacements = root
3338
.find(j.CallExpression)
34-
.filter(({ node }) => isGlobalHelperExpression(j, node))
35-
;
36-
39+
.filter(({ node }) => isGlobalHelperExpression(j, node));
3740
if (replacements.length > 0) {
3841
makeAwait(j, replacements);
3942
dropAndThen(j, root);

‎transforms/acceptance/transforms/each.js‎

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ const {
66
isFindExpression,
77
addImportStatement,
88
writeImportStatements,
9-
transformEachsCallbackArgs
9+
transformEachsCallbackArgs,
1010
} = require('../../utils');
1111

1212
/**
@@ -20,10 +20,8 @@ const {
2020
function createExpression(j, findArgs, eachCallback) {
2121
const transformedCallback = transformEachsCallbackArgs(eachCallback);
2222
return j.callExpression(
23-
j.memberExpression(
24-
createFindAllExpression(j, findArgs),
25-
j.identifier('forEach')
26-
), transformedCallback
23+
j.memberExpression(createFindAllExpression(j, findArgs), j.identifier('forEach')),
24+
transformedCallback
2725
);
2826
}
2927

@@ -35,11 +33,13 @@ function createExpression(j, findArgs, eachCallback) {
3533
* @returns {*|boolean}
3634
*/
3735
function isJQueryExpression(j, node) {
38-
return j.CallExpression.check(node)
39-
&& j.MemberExpression.check(node.callee)
40-
&& isFindExpression(j, node.callee.object)
41-
&& j.Identifier.check(node.callee.property)
42-
&& node.callee.property.name === 'each'
36+
return (
37+
j.CallExpression.check(node) &&
38+
j.MemberExpression.check(node.callee) &&
39+
isFindExpression(j, node.callee.object) &&
40+
j.Identifier.check(node.callee.property) &&
41+
node.callee.property.name === 'each'
42+
);
4343
}
4444

4545
/**

‎transforms/acceptance/transforms/fill-in.js‎

Lines changed: 11 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,12 @@
11
'use strict';
22

33
const { getParser } = require('codemod-cli').jscodeshift;
4-
const { makeAwait, dropAndThen, addImportStatement, writeImportStatements } = require('../../utils');
4+
const {
5+
makeAwait,
6+
dropAndThen,
7+
addImportStatement,
8+
writeImportStatements,
9+
} = require('../../utils');
510

611
/**
712
* Check if `node` is a `fillIn(selector)` expression
@@ -11,9 +16,9 @@ const { makeAwait, dropAndThen, addImportStatement, writeImportStatements } = re
1116
* @returns {*|boolean}
1217
*/
1318
function isGlobalHelperExpression(j, node) {
14-
return j.CallExpression.check(node)
15-
&& j.Identifier.check(node.callee)
16-
&& node.callee.name === 'fillIn';
19+
return (
20+
j.CallExpression.check(node) && j.Identifier.check(node.callee) && node.callee.name === 'fillIn'
21+
);
1722
}
1823

1924
/**
@@ -27,14 +32,11 @@ function transform(file, api) {
2732
let source = file.source;
2833
let j = getParser(api);
2934

30-
3135
let root = j(source);
3236

3337
let replacements = root
34-
.find(j.CallExpression)
35-
.filter(({ node }) => isGlobalHelperExpression(j, node))
36-
;
37-
38+
.find(j.CallExpression)
39+
.filter(({ node }) => isGlobalHelperExpression(j, node));
3840
if (replacements.length > 0) {
3941
makeAwait(j, replacements);
4042
dropAndThen(j, root);

‎transforms/acceptance/transforms/get-value.js‎

Lines changed: 16 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,12 @@
11
'use strict';
22

33
const { getParser } = require('codemod-cli').jscodeshift;
4-
const { createFindExpression, isFindExpression, addImportStatement, writeImportStatements } = require('../../utils');
4+
const {
5+
createFindExpression,
6+
isFindExpression,
7+
addImportStatement,
8+
writeImportStatements,
9+
} = require('../../utils');
510

611
/**
712
* Creates a `find(selector).value` expression
@@ -11,10 +16,7 @@ const { createFindExpression, isFindExpression, addImportStatement, writeImportS
1116
* @returns {*}
1217
*/
1318
function createExpression(j, findArgs) {
14-
return j.memberExpression(
15-
createFindExpression(j, findArgs),
16-
j.identifier('value')
17-
);
19+
return j.memberExpression(createFindExpression(j, findArgs), j.identifier('value'));
1820
}
1921

2022
/**
@@ -25,12 +27,14 @@ function createExpression(j, findArgs) {
2527
* @returns {*|boolean}
2628
*/
2729
function isJQueryExpression(j, node) {
28-
return j.CallExpression.check(node)
29-
&& j.MemberExpression.check(node.callee)
30-
&& isFindExpression(j, node.callee.object)
31-
&& j.Identifier.check(node.callee.property)
32-
&& node.callee.property.name === 'val'
33-
&& node.arguments.length === 0;
30+
return (
31+
j.CallExpression.check(node) &&
32+
j.MemberExpression.check(node.callee) &&
33+
isFindExpression(j, node.callee.object) &&
34+
j.Identifier.check(node.callee.property) &&
35+
node.callee.property.name === 'val' &&
36+
node.arguments.length === 0
37+
);
3438
}
3539

3640
/**
@@ -56,7 +60,7 @@ function transform(file, api) {
5660
}
5761

5862
writeImportStatements(j, root);
59-
return root.toSource({quote: 'single'});
63+
return root.toSource({ quote: 'single' });
6064
}
6165

6266
module.exports = transform;

‎transforms/acceptance/transforms/get.js‎

Lines changed: 9 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -17,11 +17,8 @@ const {
1717
* @returns {*}
1818
*/
1919
function createExpression(j, findArgs, indexArg) {
20-
const index = indexArg[0] && indexArg[0].value || 0;
21-
return j.memberExpression(
22-
createFindAllExpression(j, findArgs),
23-
j.literal(index)
24-
);
20+
const index = (indexArg[0] && indexArg[0].value) || 0;
21+
return j.memberExpression(createFindAllExpression(j, findArgs), j.literal(index));
2522
}
2623

2724
/**
@@ -32,11 +29,13 @@ function createExpression(j, findArgs, indexArg) {
3229
* @returns {*|boolean}
3330
*/
3431
function isJQueryExpression(j, node) {
35-
return j.CallExpression.check(node)
36-
&& j.MemberExpression.check(node.callee)
37-
&& isFindExpression(j, node.callee.object)
38-
&& j.Identifier.check(node.callee.property)
39-
&& node.callee.property.name === 'get';
32+
return (
33+
j.CallExpression.check(node) &&
34+
j.MemberExpression.check(node.callee) &&
35+
isFindExpression(j, node.callee.object) &&
36+
j.Identifier.check(node.callee.property) &&
37+
node.callee.property.name === 'get'
38+
);
4039
}
4140

4241
/**

0 commit comments

Comments
 (0)