fix: check for class properties (fixes #210)

gajus · May 26, 2020 · 6015308 · 6015308
1 parent cd19943
commit 6015308
Show file tree

Hide file tree

Showing 2 changed files with 31 additions and 0 deletions.
diff --git a/src/rules/semi.js b/src/rules/semi.js
@@ -57,6 +57,9 @@ const create = (context) => {
   return {
     OpaqueType: checkForSemicolon,
     TypeAlias: checkForSemicolon,
+    TypeAnnotation: (node) => {
+      checkForSemicolon(node.parent);
+    },
   };
 };
 

diff --git a/tests/rules/assertions/semi.js b/tests/rules/assertions/semi.js
@@ -1,5 +1,25 @@
 export default {
   invalid: [
+    {
+      code: 'class Foo { foo: string }',
+      errors: [
+        {
+          message: 'Missing semicolon.',
+        },
+      ],
+      options: ['always'],
+      output: 'class Foo { foo: string; }',
+    },
+    {
+      code: 'class Foo { foo: string; }',
+      errors: [
+        {
+          message: 'Extra semicolon.',
+        },
+      ],
+      options: ['never'],
+      output: 'class Foo { foo: string }',
+    },
     {
       code: 'type FooType = {}',
       errors: [
@@ -80,6 +100,14 @@ export default {
       code: 'type FooType = {};',
       options: ['always'],
     },
+    {
+      code: 'class Foo { foo: string; }',
+      options: ['always'],
+    },
+    {
+      code: 'class Foo { foo: string }',
+      options: ['never'],
+    },
     {
       code: 'type FooType = { a: number;\n b: string;\n };',
       options: ['always'],