no-duplicates: Disable autofix for ESLint <= 3

import-js · Mar 31, 2019 · 71a64c6 · 71a64c6
1 parent 9ac41f3
commit 71a64c6
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 1 deletion.
diff --git a/src/rules/no-duplicates.js b/src/rules/no-duplicates.js
@@ -26,6 +26,11 @@ function checkImports(imported, context) {
 }
 
 function getFix(first, rest, sourceCode) {
+  // Sorry ESLint <= 3 users, no autofix for you.
+  if (typeof sourceCode.getCommentsBefore !== 'function') {
+    return undefined
+  }
+
   const defaultImportNames = new Set(
     [first, ...rest].map(getDefaultImportName).filter(Boolean)
   )

diff --git a/tests/src/rules/no-duplicates.js b/tests/src/rules/no-duplicates.js
@@ -1,11 +1,15 @@
 import * as path from 'path'
-import { test } from '../utils'
+import { test as testUtil } from '../utils'
 
 import { RuleTester } from 'eslint'
 
 const ruleTester = new RuleTester()
     , rule = require('rules/no-duplicates')
 
+const test = process.env.ESLINT_VERSION === '3' || process.env.ESLINT_VERSION === '2'
+  ? t => testUtil(Object.assign({}, t, {output: t.code}))
+  : testUtil
+
 ruleTester.run('no-duplicates', rule, {
   valid: [
     test({ code: 'import "./malformed.js"' }),