Skip to content

Commit

Permalink
Chore: fix separateRequires tests for one-var rule (#12709)
Browse files Browse the repository at this point in the history
  • Loading branch information
mdjermanovic authored and kaicataldo committed Jan 4, 2020
1 parent e4df7df commit b77b858
Showing 1 changed file with 2 additions and 14 deletions.
16 changes: 2 additions & 14 deletions tests/lib/rules/one-var.js
Expand Up @@ -201,13 +201,11 @@ ruleTester.run("one-var", rule, {
},
{
code: "var foo = require('foo'), bar;",
options: [{ separateRequires: false, var: "always" }],
parserOptions: { env: { node: true } }
options: [{ separateRequires: false, var: "always" }]
},
{
code: "var foo = require('foo'), bar = require('bar');",
options: [{ separateRequires: true, var: "always" }],
parserOptions: { env: { node: true } }
options: [{ separateRequires: true, var: "always" }]
},
{
code: "var bar = 'bar'; var foo = require('foo');",
Expand All @@ -217,11 +215,6 @@ ruleTester.run("one-var", rule, {
code: "var foo = require('foo'); var bar = 'bar';",
options: [{ separateRequires: true, var: "always" }]
},
{
code: "var foo = require('foo'); var bar = 'bar';",
options: [{ separateRequires: true, var: "always" }],
parserOptions: { env: { node: true } }
},

// https://github.com/eslint/eslint/issues/4680
{
Expand Down Expand Up @@ -968,7 +961,6 @@ ruleTester.run("one-var", rule, {
code: "var foo = require('foo'), bar;",
output: null,
options: [{ separateRequires: true, var: "always" }],
parserOptions: { env: { node: true } },
errors: [{
message: "Split requires to be separated into a single block.",
type: "VariableDeclaration",
Expand All @@ -980,7 +972,6 @@ ruleTester.run("one-var", rule, {
code: "var foo, bar = require('bar');",
output: null,
options: [{ separateRequires: true, var: "always" }],
parserOptions: { env: { node: true } },
errors: [{
message: "Split requires to be separated into a single block.",
type: "VariableDeclaration",
Expand All @@ -992,7 +983,6 @@ ruleTester.run("one-var", rule, {
code: "let foo, bar = require('bar');",
output: null,
options: [{ separateRequires: true, let: "always" }],
parserOptions: { env: { node: true } },
errors: [{
message: "Split requires to be separated into a single block.",
type: "VariableDeclaration",
Expand All @@ -1004,7 +994,6 @@ ruleTester.run("one-var", rule, {
code: "const foo = 0, bar = require('bar');",
output: null,
options: [{ separateRequires: true, const: "always" }],
parserOptions: { env: { node: true } },
errors: [{
message: "Split requires to be separated into a single block.",
type: "VariableDeclaration",
Expand All @@ -1016,7 +1005,6 @@ ruleTester.run("one-var", rule, {
code: "const foo = require('foo'); const bar = require('bar');",
output: "const foo = require('foo'), bar = require('bar');",
options: [{ separateRequires: true, const: "always" }],
parserOptions: { env: { node: true } },
errors: [{
message: "Combine this with the previous 'const' statement.",
type: "VariableDeclaration",
Expand Down

0 comments on commit b77b858

Please sign in to comment.