Fix ignorePackages to produce errors

import-js · Oct 28, 2019 · 2bfd9d0 · 2bfd9d0
1 parent 112a0bf
commit 2bfd9d0
Show file tree

Hide file tree

Showing 2 changed files with 28 additions and 1 deletion.
diff --git a/src/rules/extensions.js b/src/rules/extensions.js
@@ -50,6 +50,11 @@ function buildProperties(context) {
       }
     })
 
+    if (result.defaultConfig === 'ignorePackages') {
+      result.defaultConfig = 'always'
+      result.ignorePackages = true
+    }
+
     return result
 }
 

diff --git a/tests/src/rules/extensions.js b/tests/src/rules/extensions.js
@@ -63,7 +63,7 @@ ruleTester.run('extensions', rule, {
       code: `
         import foo from './foo.js'
         import bar from './bar.json'
-        import Component from './Component'
+        import Component from './Component.jsx'
         import express from 'express'
       `,
       options: [ 'ignorePackages' ],
@@ -309,6 +309,28 @@ ruleTester.run('extensions', rule, {
       ],
     }),
 
+    test({
+      code: `
+        import foo from './foo.js'
+        import bar from './bar.json'
+        import Component from './Component'
+        import baz from 'foo/baz'
+        import express from 'express'
+      `,
+      options: [ 'ignorePackages' ],
+      errors: [
+        {
+          message: 'Missing file extension for "./Component"',
+          line: 4,
+          column: 31,
+        }, {
+          message: 'Missing file extension for "foo/baz"',
+          line: 5,
+          column: 25,
+        },
+      ],
+    }),
+
     test({
       code: `
         import foo from './foo.js'