fix edge case.

eslint · Nov 14, 2017 · da28611 · da28611
1 parent 7d97579
commit da28611
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 0 deletions.
diff --git a/lib/rules/one-var.js b/lib/rules/one-var.js
@@ -247,6 +247,7 @@ module.exports = {
             const declarationCounts = countDeclarations(declarations);
             const currentOptions = options[statementType] || {};
             const currentScope = getCurrentScope(statementType);
+            const hasRequires = declarations.some(isRequire);
 
             if (currentOptions.uninitialized === MODE_ALWAYS && currentOptions.initialized === MODE_ALWAYS) {
                 if (currentScope.uninitialized || currentScope.initialized) {
@@ -264,6 +265,9 @@ module.exports = {
                     return false;
                 }
             }
+            if (currentScope.required && hasRequires) {
+                return false;
+            }
             recordTypes(statementType, declarations, currentScope);
             return true;
         }

diff --git a/tests/lib/rules/one-var.js b/tests/lib/rules/one-var.js
@@ -597,6 +597,17 @@ ruleTester.run("one-var", rule, {
                 line: 1,
                 column: 1
             }]
+        },
+        {
+            code: "const foo = require('foo'); const bar = require('bar');",
+            options: [{ separateRequires: true, const: "always" }],
+            parserOptions: { env: { node: true } },
+            errors: [{
+                message: "Combine this with the previous 'const' statement.",
+                type: "VariableDeclaration",
+                line: 1,
+                column: 29
+            }]
         }
     ]
 });