Skip to content

Commit eeafcc9

Browse files
authored
chore: enforce curly braces (#1429)
* chore: enforce curly braces * refactor: apply eslint fixes
1 parent e44693f commit eeafcc9

File tree

5 files changed

+22
-7
lines changed

5 files changed

+22
-7
lines changed

.eslintrc.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -84,6 +84,7 @@ module.exports = {
8484
'prefer-rest-params': 'error',
8585
'prefer-const': ['error', { destructuring: 'all' }],
8686
'no-var': 'error',
87+
curly: 'error',
8788
},
8889
overrides: [
8990
{

src/rules/expect-expect.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,9 @@ function matchesAssertFunctionName(
2828
`^${p
2929
.split('.')
3030
.map(x => {
31-
if (x === '**') return '[a-z\\d\\.]*';
31+
if (x === '**') {
32+
return '[a-z\\d\\.]*';
33+
}
3234
3335
return x.replace(/\*/gu, '[a-z\\d]*');
3436
})

src/rules/no-test-return-statement.ts

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,9 @@ export default createRule({
4747
t => t.type === AST_NODE_TYPES.ReturnStatement,
4848
);
4949

50-
if (!returnStmt) return;
50+
if (!returnStmt) {
51+
return;
52+
}
5153

5254
context.report({ messageId: 'noReturnValue', node: returnStmt });
5355
},
@@ -58,13 +60,17 @@ export default createRule({
5860
context,
5961
);
6062

61-
if (testCallExpressions.length === 0) return;
63+
if (testCallExpressions.length === 0) {
64+
return;
65+
}
6266

6367
const returnStmt = node.body.body.find(
6468
t => t.type === AST_NODE_TYPES.ReturnStatement,
6569
);
6670

67-
if (!returnStmt) return;
71+
if (!returnStmt) {
72+
return;
73+
}
6874

6975
context.report({ messageId: 'noReturnValue', node: returnStmt });
7076
},

src/rules/no-untyped-mock-factory.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -70,7 +70,9 @@ export default createRule({
7070
data: { moduleName: moduleName?.raw ?? './module-name' },
7171
node,
7272
fix(fixer) {
73-
if (!moduleName) return [];
73+
if (!moduleName) {
74+
return [];
75+
}
7476

7577
return [
7678
fixer.insertTextAfter(

src/rules/prefer-spy-on.ts

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -85,11 +85,15 @@ export default createRule({
8585
AssignmentExpression(node) {
8686
const { left, right } = node;
8787

88-
if (left.type !== AST_NODE_TYPES.MemberExpression) return;
88+
if (left.type !== AST_NODE_TYPES.MemberExpression) {
89+
return;
90+
}
8991

9092
const jestFnCall = getJestFnCall(right);
9193

92-
if (!jestFnCall) return;
94+
if (!jestFnCall) {
95+
return;
96+
}
9397

9498
context.report({
9599
node,

0 commit comments

Comments
 (0)