Skip to content
This repository was archived by the owner on Mar 23, 2024. It is now read-only.

Commit afacf5c

Browse files
committed
requireSemicolons: check for ClassProperty
Closes gh-2057
1 parent 11ba6fb commit afacf5c

File tree

2 files changed

+34
-1
lines changed

2 files changed

+34
-1
lines changed

lib/rules/require-semicolons.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,8 @@ module.exports.prototype = {
6262
'ThrowStatement',
6363
'BreakStatement',
6464
'ContinueStatement',
65-
'DebuggerStatement'
65+
'DebuggerStatement',
66+
'ClassProperty'
6667
], function(node) {
6768
// ignore variable declaration inside for and for-in
6869
if (node.type === 'VariableDeclaration') {

test/specs/rules/require-semicolons.js

Lines changed: 32 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -393,6 +393,38 @@ describe('rules/require-semicolons', function() {
393393
});
394394
});
395395

396+
describe('class property', function() {
397+
valid([
398+
'class A { asdf; }',
399+
'export class A { asdf; }',
400+
'export default class A { asdf; }',
401+
'export default class A { asdf = 1; }'
402+
]);
403+
404+
invalid([
405+
'class A { asdf }',
406+
'export class A { asdf }',
407+
'export default class A { asdf }',
408+
'export default class A { asdf = 1 }'
409+
]);
410+
411+
reportAndFix({
412+
name: 'class A { asdf }',
413+
rules: config,
414+
errors: 1,
415+
input: 'class A { asdf }',
416+
output: 'class A { asdf; }'
417+
});
418+
419+
reportAndFix({
420+
name: 'class A { asdf = 1 }',
421+
rules: config,
422+
errors: 1,
423+
input: 'class A { asdf = 1 }',
424+
output: 'class A { asdf = 1; }'
425+
});
426+
});
427+
396428
reportAndFix({
397429
name: 'expression call with comments',
398430
rules: config,

0 commit comments

Comments
 (0)