diff --git a/lib/index.js b/lib/index.js index 12ab57b0..038ab787 100644 --- a/lib/index.js +++ b/lib/index.js @@ -6,6 +6,6 @@ "use strict" module.exports = { - configs: {recommended: require("./recommended.json")}, + configs: { recommended: require("./recommended.json") }, rules: require("./rules.js"), } diff --git a/lib/rules/exports-style.js b/lib/rules/exports-style.js index 0830035f..c4169000 100644 --- a/lib/rules/exports-style.js +++ b/lib/rules/exports-style.js @@ -308,7 +308,7 @@ module.exports = { }, { type: "object", - properties: {allowBatchAssign: {type: "boolean"}}, + properties: { allowBatchAssign: { type: "boolean" } }, additionalProperties: false, }, ], diff --git a/lib/rules/no-deprecated-api.js b/lib/rules/no-deprecated-api.js index 8d8a4637..4d0c4e80 100644 --- a/lib/rules/no-deprecated-api.js +++ b/lib/rules/no-deprecated-api.js @@ -465,7 +465,7 @@ function create(context) { checkVariable( findVariable(node.local, globalScope), path, - Object.assign({}, infoMap, {default: infoMap}) + Object.assign({}, infoMap, { default: infoMap }) ) break @@ -576,19 +576,19 @@ module.exports = { properties: { ignoreModuleItems: { type: "array", - items: {enum: MODULE_ITEMS}, + items: { enum: MODULE_ITEMS }, additionalItems: false, uniqueItems: true, }, ignoreGlobalItems: { type: "array", - items: {enum: GLOBAL_ITEMS}, + items: { enum: GLOBAL_ITEMS }, additionalItems: false, uniqueItems: true, }, // Deprecated since v4.2.0 - ignoreIndirectDependencies: {type: "boolean"}, + ignoreIndirectDependencies: { type: "boolean" }, }, additionalProperties: false, }, diff --git a/lib/rules/no-hide-core-modules.js b/lib/rules/no-hide-core-modules.js index ff81a8b2..c8717be7 100644 --- a/lib/rules/no-hide-core-modules.js +++ b/lib/rules/no-hide-core-modules.js @@ -74,7 +74,7 @@ function create(context) { continue } - const resolved = resolve.sync(name, {basedir: dirPath}) + const resolved = resolve.sync(name, { basedir: dirPath }) const isCore = resolved === name if (isCore) { @@ -117,12 +117,12 @@ module.exports = { properties: { allow: { type: "array", - items: {enum: Array.from(CORE_MODULES)}, + items: { enum: Array.from(CORE_MODULES) }, additionalItems: false, uniqueItems: true, }, - ignoreDirectDependencies: {type: "boolean"}, - ignoreIndirectDependencies: {type: "boolean"}, + ignoreDirectDependencies: { type: "boolean" }, + ignoreIndirectDependencies: { type: "boolean" }, }, additionalProperties: false, }, diff --git a/lib/rules/no-unpublished-bin.js b/lib/rules/no-unpublished-bin.js index dd2e2ace..ec12b6ca 100644 --- a/lib/rules/no-unpublished-bin.js +++ b/lib/rules/no-unpublished-bin.js @@ -83,7 +83,7 @@ function create(context) { message: "npm ignores '{{name}}'. " + "Check 'files' field of 'package.json' or '.npmignore'.", - data: {name: relativePath}, + data: { name: relativePath }, }) }, } diff --git a/lib/rules/no-unsupported-features.js b/lib/rules/no-unsupported-features.js index 481dab40..0ed1493d 100644 --- a/lib/rules/no-unsupported-features.js +++ b/lib/rules/no-unsupported-features.js @@ -31,7 +31,7 @@ const VERSION_MAP = new Map([ ]) const VERSION_SCHEMA = { anyOf: [ - {enum: Array.from(VERSION_MAP.keys())}, + { enum: Array.from(VERSION_MAP.keys()) }, { type: "string", pattern: "^(?:0|[1-9]\\d*)\\.(?:0|[1-9]\\d*)\\.(?:0|[1-9]\\d*)$", @@ -688,7 +688,7 @@ module.exports = { version: VERSION_SCHEMA, ignores: { type: "array", - items: {enum: getIgnoresEnum()}, + items: { enum: getIgnoresEnum() }, uniqueItems: true, }, }, diff --git a/lib/rules/process-exit-as-throw.js b/lib/rules/process-exit-as-throw.js index 66e02f59..4ac2ac28 100644 --- a/lib/rules/process-exit-as-throw.js +++ b/lib/rules/process-exit-as-throw.js @@ -63,7 +63,8 @@ function forwardCurrentToHead(analyzer, node) { analyzer.emitter.emit( "onCodePathSegmentEnd", currentSegment, - node) + node + ) } } } @@ -82,7 +83,8 @@ function forwardCurrentToHead(analyzer, node) { analyzer.emitter.emit( "onCodePathSegmentStart", headSegment, - node) + node + ) } } } diff --git a/lib/util/cache.js b/lib/util/cache.js index 9294292f..9cf05c2a 100644 --- a/lib/util/cache.js +++ b/lib/util/cache.js @@ -62,7 +62,7 @@ module.exports = class Cache { entry.expire = expire } else { - this.map.set(key, {value, expire}) + this.map.set(key, { value, expire }) } } } diff --git a/lib/util/check-publish.js b/lib/util/check-publish.js index 31be7559..2340d176 100644 --- a/lib/util/check-publish.js +++ b/lib/util/check-publish.js @@ -74,7 +74,7 @@ module.exports = function checkForPublish(context, filePath, targets) { node: target.node, loc: target.node.loc, message: "\"{{name}}\" is not published.", - data: {name: target.moduleName || target.name}, + data: { name: target.moduleName || target.name }, }) } } diff --git a/lib/util/get-convert-path.js b/lib/util/get-convert-path.js index bde58cb0..f055951d 100644 --- a/lib/util/get-convert-path.js +++ b/lib/util/get-convert-path.js @@ -163,7 +163,7 @@ module.exports.schema = { patternProperties: { "^.+$": { type: "array", - items: {type: "string"}, + items: { type: "string" }, minItems: 2, maxItems: 2, }, @@ -177,18 +177,18 @@ module.exports.schema = { properties: { include: { type: "array", - items: {type: "string"}, + items: { type: "string" }, minItems: 1, uniqueItems: true, }, exclude: { type: "array", - items: {type: "string"}, + items: { type: "string" }, uniqueItems: true, }, replace: { type: "array", - items: {type: "string"}, + items: { type: "string" }, minItems: 2, maxItems: 2, }, diff --git a/lib/util/get-import-export-targets.js b/lib/util/get-import-export-targets.js index 9fafc99f..8cf7146a 100644 --- a/lib/util/get-import-export-targets.js +++ b/lib/util/get-import-export-targets.js @@ -41,7 +41,7 @@ module.exports = function getImportExportTargets(context, programNode, includeCo const basedir = path.dirname(path.resolve(context.getFilename())) const paths = getResolvePaths(context) const extensions = getTryExtensions(context) - const options = {basedir, paths, extensions} + const options = { basedir, paths, extensions } for (const statement of programNode.body) { // Skip if it's not a module declaration. diff --git a/lib/util/get-npmignore.js b/lib/util/get-npmignore.js index 15da07b5..51ee79dc 100644 --- a/lib/util/get-npmignore.js +++ b/lib/util/get-npmignore.js @@ -150,7 +150,7 @@ function parseNpmignore(basedir, filesFieldExists) { * `match` returns `true` if a given file path should be ignored. */ module.exports = function getNpmignore(startPath) { - const retv = {match: notAncestorFiles} + const retv = { match: notAncestorFiles } const p = getPackageJson(startPath) if (p) { diff --git a/lib/util/get-require-targets.js b/lib/util/get-require-targets.js index 09e41263..4cc6d056 100644 --- a/lib/util/get-require-targets.js +++ b/lib/util/get-require-targets.js @@ -65,7 +65,7 @@ module.exports = function getRequireTargets(context, includeCore) { const paths = getResolvePaths(context) const references = getReferencesOfRequire(context.getScope()) const extensions = getTryExtensions(context) - const options = {basedir, paths, extensions} + const options = { basedir, paths, extensions } for (const reference of references) { const node = reference.identifier diff --git a/lib/util/get-resolve-paths.js b/lib/util/get-resolve-paths.js index 09d871e5..c2d56e05 100644 --- a/lib/util/get-resolve-paths.js +++ b/lib/util/get-resolve-paths.js @@ -48,6 +48,6 @@ module.exports = function getResolvePaths(context) { module.exports.schema = { type: "array", - items: {type: "string"}, + items: { type: "string" }, uniqueItems: true, } diff --git a/package.json b/package.json index d1bcf328..dd62eb60 100644 --- a/package.json +++ b/package.json @@ -32,14 +32,14 @@ "semver": "^5.4.1" }, "devDependencies": { - "codecov": "^2.2.0", - "eslint": "^4.1.0", - "eslint-config-mysticatea": "^11.0.0", - "mocha": "^3.4.2", + "codecov": "^3.0.0", + "eslint": "^4.17.0", + "eslint-config-mysticatea": "^13.0.2", + "mocha": "^5.0.0", "nyc": "^11.0.1", "opener": "^1.4.3", "rimraf": "^2.6.1", - "shelljs": "^0.7.7" + "shelljs": "^0.8.1" }, "repository": { "type": "git", diff --git a/tests/lib/rules/exports-style.js b/tests/lib/rules/exports-style.js index fdc02c0a..c00cfcf6 100644 --- a/tests/lib/rules/exports-style.js +++ b/tests/lib/rules/exports-style.js @@ -20,129 +20,129 @@ new RuleTester().run("exports-style", rule, { valid: [ { code: "module.exports = {foo: 1}", - globals: {module: false, exports: true}, + globals: { module: false, exports: true }, }, { code: "module.exports = {foo: 1}", options: ["module.exports"], - globals: {module: false, exports: true}, + globals: { module: false, exports: true }, }, { code: "exports.foo = 1", options: ["exports"], - globals: {module: false, exports: true}, + globals: { module: false, exports: true }, }, { code: "exports = module.exports = {foo: 1}", - options: ["module.exports", {allowBatchAssign: true}], - globals: {module: false, exports: true}, + options: ["module.exports", { allowBatchAssign: true }], + globals: { module: false, exports: true }, }, { code: "module.exports = exports = {foo: 1}", - options: ["module.exports", {allowBatchAssign: true}], - globals: {module: false, exports: true}, + options: ["module.exports", { allowBatchAssign: true }], + globals: { module: false, exports: true }, }, { code: "exports = module.exports = {foo: 1}", - options: ["exports", {allowBatchAssign: true}], - globals: {module: false, exports: true}, + options: ["exports", { allowBatchAssign: true }], + globals: { module: false, exports: true }, }, { code: "module.exports = exports = {foo: 1}", - options: ["exports", {allowBatchAssign: true}], - globals: {module: false, exports: true}, + options: ["exports", { allowBatchAssign: true }], + globals: { module: false, exports: true }, }, { code: "exports = module.exports = {foo: 1}; exports.bar = 2", - options: ["exports", {allowBatchAssign: true}], - globals: {module: false, exports: true}, + options: ["exports", { allowBatchAssign: true }], + globals: { module: false, exports: true }, }, { code: "module.exports = exports = {foo: 1}; exports.bar = 2", - options: ["exports", {allowBatchAssign: true}], - globals: {module: false, exports: true}, + options: ["exports", { allowBatchAssign: true }], + globals: { module: false, exports: true }, }, // allow accesses of `modules` except `module.exports` { code: "module = {}; module.foo = 1", options: ["exports"], - globals: {module: false, exports: true}, + globals: { module: false, exports: true }, }, // Ignores if it's not defined. - {code: "exports.foo = 1", options: ["module.exports"]}, - {code: "module.exports = {foo: 1}", options: ["exports"]}, + { code: "exports.foo = 1", options: ["module.exports"] }, + { code: "module.exports = {foo: 1}", options: ["exports"] }, ], invalid: [ { code: "exports = {foo: 1}", - globals: {module: false, exports: true}, + globals: { module: false, exports: true }, errors: ["Unexpected access to 'exports'. Use 'module.exports' instead."], }, { code: "exports.foo = 1", - globals: {module: false, exports: true}, + globals: { module: false, exports: true }, errors: ["Unexpected access to 'exports'. Use 'module.exports' instead."], }, { code: "module.exports = exports = {foo: 1}", - globals: {module: false, exports: true}, + globals: { module: false, exports: true }, errors: ["Unexpected access to 'exports'. Use 'module.exports' instead."], }, { code: "exports = module.exports = {foo: 1}", - globals: {module: false, exports: true}, + globals: { module: false, exports: true }, errors: ["Unexpected access to 'exports'. Use 'module.exports' instead."], }, { code: "exports = {foo: 1}", options: ["module.exports"], - globals: {module: false, exports: true}, + globals: { module: false, exports: true }, errors: ["Unexpected access to 'exports'. Use 'module.exports' instead."], }, { code: "exports.foo = 1", options: ["module.exports"], - globals: {module: false, exports: true}, + globals: { module: false, exports: true }, errors: ["Unexpected access to 'exports'. Use 'module.exports' instead."], }, { code: "module.exports = exports = {foo: 1}", options: ["module.exports"], - globals: {module: false, exports: true}, + globals: { module: false, exports: true }, errors: ["Unexpected access to 'exports'. Use 'module.exports' instead."], }, { code: "exports = module.exports = {foo: 1}", options: ["module.exports"], - globals: {module: false, exports: true}, + globals: { module: false, exports: true }, errors: ["Unexpected access to 'exports'. Use 'module.exports' instead."], }, { code: "exports = {foo: 1}", options: ["exports"], - globals: {module: false, exports: true}, + globals: { module: false, exports: true }, errors: ["Unexpected assignment to 'exports'. Don't modify 'exports' itself."], }, { code: "module.exports = {foo: 1}", options: ["exports"], - globals: {module: false, exports: true}, + globals: { module: false, exports: true }, errors: ["Unexpected access to 'module.exports'. Use 'exports' instead."], }, { code: "module.exports.foo = 1", options: ["exports"], - globals: {module: false, exports: true}, + globals: { module: false, exports: true }, errors: ["Unexpected access to 'module.exports'. Use 'exports' instead."], }, { code: "module.exports = exports = {foo: 1}", options: ["exports"], - globals: {module: false, exports: true}, + globals: { module: false, exports: true }, errors: [ "Unexpected access to 'module.exports'. Use 'exports' instead.", "Unexpected assignment to 'exports'. Don't modify 'exports' itself.", @@ -151,7 +151,7 @@ new RuleTester().run("exports-style", rule, { { code: "exports = module.exports = {foo: 1}", options: ["exports"], - globals: {module: false, exports: true}, + globals: { module: false, exports: true }, errors: [ "Unexpected assignment to 'exports'. Don't modify 'exports' itself.", "Unexpected access to 'module.exports'. Use 'exports' instead.", @@ -159,14 +159,14 @@ new RuleTester().run("exports-style", rule, { }, { code: "module.exports = exports = {foo: 1}; exports = obj", - options: ["exports", {allowBatchAssign: true}], - globals: {module: false, exports: true}, + options: ["exports", { allowBatchAssign: true }], + globals: { module: false, exports: true }, errors: ["Unexpected assignment to 'exports'. Don't modify 'exports' itself."], }, { code: "exports = module.exports = {foo: 1}; exports = obj", - options: ["exports", {allowBatchAssign: true}], - globals: {module: false, exports: true}, + options: ["exports", { allowBatchAssign: true }], + globals: { module: false, exports: true }, errors: ["Unexpected assignment to 'exports'. Don't modify 'exports' itself."], }, ], diff --git a/tests/lib/rules/no-deprecated-api.js b/tests/lib/rules/no-deprecated-api.js index c4c45eab..50597446 100644 --- a/tests/lib/rules/no-deprecated-api.js +++ b/tests/lib/rules/no-deprecated-api.js @@ -22,125 +22,125 @@ ruleTester.run("no-deprecated-api", rule, { valid: [ { code: "require('buffer').Buffer", - env: {node: true}, + env: { node: true }, }, { code: "foo(require('buffer').Buffer)", - env: {node: true}, + env: { node: true }, }, { code: "new (require('another-buffer').Buffer)()", - env: {node: true}, + env: { node: true }, }, { code: "var http = require('http'); http.request()", - env: {node: true}, + env: { node: true }, }, { code: "var {request} = require('http'); request()", - env: {node: true, es6: true}, + env: { node: true, es6: true }, }, { code: "(s ? require('https') : require('http')).request()", - env: {node: true}, + env: { node: true }, }, { code: "require(HTTP).createClient", - env: {node: true}, + env: { node: true }, }, { code: "import {Buffer} from 'another-buffer'; new Buffer()", - env: {es6: true}, - parserOptions: {sourceType: "module"}, + env: { es6: true }, + parserOptions: { sourceType: "module" }, }, { code: "import {request} from 'http'; request()", - env: {es6: true}, - parserOptions: {sourceType: "module"}, + env: { es6: true }, + parserOptions: { sourceType: "module" }, }, // On Node v6.8.0, fs.existsSync revived. { code: "require('fs').existsSync;", - env: {node: true}, + env: { node: true }, }, // use third parties. { code: "require('domain/');", - env: {node: true}, + env: { node: true }, }, { code: "import domain from 'domain/';", - parserOptions: {sourceType: "module"}, + parserOptions: { sourceType: "module" }, }, // https://github.com/mysticatea/eslint-plugin-node/issues/55 { code: "undefinedVar = require('fs')", - env: {node: true}, + env: { node: true }, }, // ignore options { code: "new (require('buffer').Buffer)()", - env: {node: true}, + env: { node: true }, options: [{ // ignoreModuleItems: ["new buffer.Buffer()"], }], }, { code: "require('buffer').Buffer()", - env: {node: true}, + env: { node: true }, options: [{ // ignoreModuleItems: ["buffer.Buffer()"], }], }, { code: "require('domain');", - env: {node: true}, + env: { node: true }, options: [{ // ignoreModuleItems: ["domain"], }], }, { code: "require('events').EventEmitter.listenerCount;", - env: {node: true}, + env: { node: true }, options: [{ // ignoreModuleItems: ["events.EventEmitter.listenerCount"], }], }, { code: "require('events').listenerCount;", - env: {node: true}, + env: { node: true }, options: [{ // ignoreModuleItems: ["events.listenerCount"], }], }, { code: "new Buffer;", - env: {node: true}, + env: { node: true }, options: [{ // ignoreGlobalItems: ["new Buffer()"], }], }, { code: "Buffer();", - env: {node: true}, + env: { node: true }, options: [{ // ignoreGlobalItems: ["Buffer()"], }], }, { code: "Intl.v8BreakIterator;", - env: {node: true}, + env: { node: true }, options: [{ // ignoreGlobalItems: ["Intl.v8BreakIterator"], }], }, { code: "let {env: {NODE_REPL_HISTORY_FILE}} = process;", - env: {node: true, es6: true}, + env: { node: true, es6: true }, options: [{ // ignoreGlobalItems: ["process.env.NODE_REPL_HISTORY_FILE"], }], @@ -149,14 +149,14 @@ ruleTester.run("no-deprecated-api", rule, { // https://github.com/mysticatea/eslint-plugin-node/issues/65 { code: "require(\"domain/\")", - env: {node: true}, - options: [{ignoreIndirectDependencies: true}], + env: { node: true }, + options: [{ ignoreIndirectDependencies: true }], }, // https://github.com/mysticatea/eslint-plugin-node/issues/87 { code: "let fs = fs || require(\"fs\")", - env: {node: true, es6: true}, + env: { node: true, es6: true }, }, ], invalid: [ @@ -165,37 +165,37 @@ ruleTester.run("no-deprecated-api", rule, { //---------------------------------------------------------------------- { code: "new (require('buffer').Buffer)()", - env: {node: true}, + env: { node: true }, errors: ["'new buffer.Buffer()' was deprecated since v6. Use 'buffer.Buffer.alloc()' or 'buffer.Buffer.from()' (use 'https://www.npmjs.com/package/safe-buffer' for '<4.5.0') instead."], }, { code: "require('buffer').Buffer()", - env: {node: true}, + env: { node: true }, errors: ["'buffer.Buffer()' was deprecated since v6. Use 'buffer.Buffer.alloc()' or 'buffer.Buffer.from()' (use 'https://www.npmjs.com/package/safe-buffer' for '<4.5.0') instead."], }, { code: "var b = require('buffer'); new b.Buffer()", - env: {node: true}, + env: { node: true }, errors: ["'new buffer.Buffer()' was deprecated since v6. Use 'buffer.Buffer.alloc()' or 'buffer.Buffer.from()' (use 'https://www.npmjs.com/package/safe-buffer' for '<4.5.0') instead."], }, { code: "var b = require('buffer'); new b['Buffer']()", - env: {node: true}, + env: { node: true }, errors: ["'new buffer.Buffer()' was deprecated since v6. Use 'buffer.Buffer.alloc()' or 'buffer.Buffer.from()' (use 'https://www.npmjs.com/package/safe-buffer' for '<4.5.0') instead."], }, { code: "var b = require('buffer'); new b[`Buffer`]()", - env: {node: true, es6: true}, + env: { node: true, es6: true }, errors: ["'new buffer.Buffer()' was deprecated since v6. Use 'buffer.Buffer.alloc()' or 'buffer.Buffer.from()' (use 'https://www.npmjs.com/package/safe-buffer' for '<4.5.0') instead."], }, { code: "var b = require('buffer').Buffer; new b()", - env: {node: true}, + env: { node: true }, errors: ["'new buffer.Buffer()' was deprecated since v6. Use 'buffer.Buffer.alloc()' or 'buffer.Buffer.from()' (use 'https://www.npmjs.com/package/safe-buffer' for '<4.5.0') instead."], }, { code: "var b; new ((b = require('buffer')).Buffer)(); new b.Buffer()", - env: {node: true}, + env: { node: true }, errors: [ "'new buffer.Buffer()' was deprecated since v6. Use 'buffer.Buffer.alloc()' or 'buffer.Buffer.from()' (use 'https://www.npmjs.com/package/safe-buffer' for '<4.5.0') instead.", "'new buffer.Buffer()' was deprecated since v6. Use 'buffer.Buffer.alloc()' or 'buffer.Buffer.from()' (use 'https://www.npmjs.com/package/safe-buffer' for '<4.5.0') instead.", @@ -203,366 +203,366 @@ ruleTester.run("no-deprecated-api", rule, { }, { code: "var {Buffer: b} = require('buffer'); new b()", - env: {node: true, es6: true}, + env: { node: true, es6: true }, errors: ["'new buffer.Buffer()' was deprecated since v6. Use 'buffer.Buffer.alloc()' or 'buffer.Buffer.from()' (use 'https://www.npmjs.com/package/safe-buffer' for '<4.5.0') instead."], }, { code: "var {['Buffer']: b = null} = require('buffer'); new b()", - env: {node: true, es6: true}, + env: { node: true, es6: true }, errors: ["'new buffer.Buffer()' was deprecated since v6. Use 'buffer.Buffer.alloc()' or 'buffer.Buffer.from()' (use 'https://www.npmjs.com/package/safe-buffer' for '<4.5.0') instead."], }, { code: "var {'Buffer': b = null} = require('buffer'); new b()", - env: {node: true, es6: true}, + env: { node: true, es6: true }, errors: ["'new buffer.Buffer()' was deprecated since v6. Use 'buffer.Buffer.alloc()' or 'buffer.Buffer.from()' (use 'https://www.npmjs.com/package/safe-buffer' for '<4.5.0') instead."], }, { code: "var {Buffer: b = require('buffer').Buffer} = {}; new b()", - env: {node: true, es6: true}, + env: { node: true, es6: true }, errors: ["'new buffer.Buffer()' was deprecated since v6. Use 'buffer.Buffer.alloc()' or 'buffer.Buffer.from()' (use 'https://www.npmjs.com/package/safe-buffer' for '<4.5.0') instead."], }, { code: "require('buffer').SlowBuffer", - env: {node: true}, + env: { node: true }, errors: ["'buffer.SlowBuffer' was deprecated since v6. Use 'buffer.Buffer.allocUnsafeSlow()' instead."], }, { code: "var b = require('buffer'); b.SlowBuffer", - env: {node: true}, + env: { node: true }, errors: ["'buffer.SlowBuffer' was deprecated since v6. Use 'buffer.Buffer.allocUnsafeSlow()' instead."], }, { code: "var {SlowBuffer: b} = require('buffer');", - env: {node: true, es6: true}, + env: { node: true, es6: true }, errors: ["'buffer.SlowBuffer' was deprecated since v6. Use 'buffer.Buffer.allocUnsafeSlow()' instead."], }, //---------------------------------------------------------------------- { code: "require('_linklist');", - env: {node: true}, + env: { node: true }, errors: ["'_linklist' module was deprecated since v5."], }, { code: "require('constants');", - env: {node: true}, + env: { node: true }, errors: ["'constants' module was deprecated since v6.3. Use 'constants' property of each module instead."], }, { code: "require('crypto').Credentials;", - env: {node: true}, + env: { node: true }, errors: ["'crypto.Credentials' was deprecated since v0.12. Use 'tls.SecureContext' instead."], }, { code: "require('crypto').createCredentials;", - env: {node: true}, + env: { node: true }, errors: ["'crypto.createCredentials' was deprecated since v0.12. Use 'tls.createSecureContext()' instead."], }, { code: "require('domain');", - env: {node: true}, + env: { node: true }, errors: ["'domain' module was deprecated since v4."], }, { code: "require('events').EventEmitter.listenerCount;", - env: {node: true}, + env: { node: true }, errors: ["'events.EventEmitter.listenerCount' was deprecated since v4. Use 'events.EventEmitter#listenerCount()' instead."], }, { code: "require('events').listenerCount;", - env: {node: true}, + env: { node: true }, errors: ["'events.listenerCount' was deprecated since v4. Use 'events.EventEmitter#listenerCount()' instead."], }, { code: "require('freelist');", - env: {node: true}, + env: { node: true }, errors: ["'freelist' module was deprecated since v4."], }, { code: "require('fs').SyncWriteStream;", - env: {node: true}, + env: { node: true }, errors: ["'fs.SyncWriteStream' was deprecated since v4."], }, { code: "require('fs').exists;", - env: {node: true}, + env: { node: true }, errors: ["'fs.exists' was deprecated since v4. Use 'fs.stat()' or 'fs.access()' instead."], }, { code: "require('fs').lchmod;", - env: {node: true}, + env: { node: true }, errors: ["'fs.lchmod' was deprecated since v0.4."], }, { code: "require('fs').lchmodSync;", - env: {node: true}, + env: { node: true }, errors: ["'fs.lchmodSync' was deprecated since v0.4."], }, { code: "require('fs').lchown;", - env: {node: true}, + env: { node: true }, errors: ["'fs.lchown' was deprecated since v0.4."], }, { code: "require('fs').lchownSync;", - env: {node: true}, + env: { node: true }, errors: ["'fs.lchownSync' was deprecated since v0.4."], }, { code: "require('http').createClient;", - env: {node: true}, + env: { node: true }, errors: ["'http.createClient' was deprecated since v0.10. Use 'http.request()' instead."], }, { code: "require('module').requireRepl;", - env: {node: true}, + env: { node: true }, errors: ["'module.requireRepl' was deprecated since v6. Use 'require(\"repl\")' instead."], }, { code: "require('module').Module.requireRepl;", - env: {node: true}, + env: { node: true }, errors: ["'module.Module.requireRepl' was deprecated since v6. Use 'require(\"repl\")' instead."], }, { code: "require('os').getNetworkInterfaces;", - env: {node: true}, + env: { node: true }, errors: ["'os.getNetworkInterfaces' was deprecated since v0.6. Use 'os.networkInterfaces()' instead."], }, { code: "require('os').tmpDir;", - env: {node: true}, + env: { node: true }, errors: ["'os.tmpDir' was deprecated since v7. Use 'os.tmpdir()' instead."], }, { code: "require('punycode');", - env: {node: true}, + env: { node: true }, errors: ["'punycode' module was deprecated since v7. Use 'https://www.npmjs.com/package/punycode' instead."], }, { code: "require('readline').codePointAt;", - env: {node: true}, + env: { node: true }, errors: ["'readline.codePointAt' was deprecated since v4."], }, { code: "require('readline').getStringWidth;", - env: {node: true}, + env: { node: true }, errors: ["'readline.getStringWidth' was deprecated since v6."], }, { code: "require('readline').isFullWidthCodePoint;", - env: {node: true}, + env: { node: true }, errors: ["'readline.isFullWidthCodePoint' was deprecated since v6."], }, { code: "require('readline').stripVTControlCharacters;", - env: {node: true}, + env: { node: true }, errors: ["'readline.stripVTControlCharacters' was deprecated since v6."], }, { code: "require('sys');", - env: {node: true}, + env: { node: true }, errors: ["'sys' module was deprecated since v0.3. Use 'util' module instead."], }, { code: "require('tls').CleartextStream;", - env: {node: true}, + env: { node: true }, errors: ["'tls.CleartextStream' was deprecated since v0.10."], }, { code: "require('tls').CryptoStream;", - env: {node: true}, + env: { node: true }, errors: ["'tls.CryptoStream' was deprecated since v0.12. Use 'tls.TLSSocket' instead."], }, { code: "require('tls').SecurePair;", - env: {node: true}, + env: { node: true }, errors: ["'tls.SecurePair' was deprecated since v6. Use 'tls.TLSSocket' instead."], }, { code: "require('tls').createSecurePair;", - env: {node: true}, + env: { node: true }, errors: ["'tls.createSecurePair' was deprecated since v6. Use 'tls.TLSSocket' instead."], }, { code: "require('tty').setRawMode;", - env: {node: true}, + env: { node: true }, errors: ["'tty.setRawMode' was deprecated since v0.10. Use 'tty.ReadStream#setRawMode()' (e.g. 'process.stdin.setRawMode()') instead."], }, { code: "require('util').debug;", - env: {node: true}, + env: { node: true }, errors: ["'util.debug' was deprecated since v0.12. Use 'console.error()' instead."], }, { code: "require('util').error;", - env: {node: true}, + env: { node: true }, errors: ["'util.error' was deprecated since v0.12. Use 'console.error()' instead."], }, { code: "require('util').isArray;", - env: {node: true}, + env: { node: true }, errors: ["'util.isArray' was deprecated since v4. Use 'Array.isArray()' instead."], }, { code: "require('util').isBoolean;", - env: {node: true}, + env: { node: true }, errors: ["'util.isBoolean' was deprecated since v4."], }, { code: "require('util').isBuffer;", - env: {node: true}, + env: { node: true }, errors: ["'util.isBuffer' was deprecated since v4. Use 'Buffer.isBuffer()' instead."], }, { code: "require('util').isDate;", - env: {node: true}, + env: { node: true }, errors: ["'util.isDate' was deprecated since v4."], }, { code: "require('util').isError;", - env: {node: true}, + env: { node: true }, errors: ["'util.isError' was deprecated since v4."], }, { code: "require('util').isFunction;", - env: {node: true}, + env: { node: true }, errors: ["'util.isFunction' was deprecated since v4."], }, { code: "require('util').isNull;", - env: {node: true}, + env: { node: true }, errors: ["'util.isNull' was deprecated since v4."], }, { code: "require('util').isNullOrUndefined;", - env: {node: true}, + env: { node: true }, errors: ["'util.isNullOrUndefined' was deprecated since v4."], }, { code: "require('util').isNumber;", - env: {node: true}, + env: { node: true }, errors: ["'util.isNumber' was deprecated since v4."], }, { code: "require('util').isObject;", - env: {node: true}, + env: { node: true }, errors: ["'util.isObject' was deprecated since v4."], }, { code: "require('util').isPrimitive;", - env: {node: true}, + env: { node: true }, errors: ["'util.isPrimitive' was deprecated since v4."], }, { code: "require('util').isRegExp;", - env: {node: true}, + env: { node: true }, errors: ["'util.isRegExp' was deprecated since v4."], }, { code: "require('util').isString;", - env: {node: true}, + env: { node: true }, errors: ["'util.isString' was deprecated since v4."], }, { code: "require('util').isSymbol;", - env: {node: true}, + env: { node: true }, errors: ["'util.isSymbol' was deprecated since v4."], }, { code: "require('util').isUndefined;", - env: {node: true}, + env: { node: true }, errors: ["'util.isUndefined' was deprecated since v4."], }, { code: "require('util').log;", - env: {node: true}, + env: { node: true }, errors: ["'util.log' was deprecated since v6. Use a third party module instead."], }, { code: "require('util').print;", - env: {node: true}, + env: { node: true }, errors: ["'util.print' was deprecated since v0.12. Use 'console.log()' instead."], }, { code: "require('util').pump;", - env: {node: true}, + env: { node: true }, errors: ["'util.pump' was deprecated since v0.10. Use 'stream.Readable#pipe()' instead."], }, { code: "require('util').puts;", - env: {node: true}, + env: { node: true }, errors: ["'util.puts' was deprecated since v0.12. Use 'console.log()' instead."], }, { code: "require('util')._extend;", - env: {node: true}, + env: { node: true }, errors: ["'util._extend' was deprecated since v6. Use 'Object.assign()' instead."], }, { code: "require('vm').runInDebugContext;", - env: {node: true}, + env: { node: true }, errors: ["'vm.runInDebugContext' was deprecated since v8."], }, // ES2015 Modules { code: "import b from 'buffer'; new b.Buffer()", - env: {es6: true}, - parserOptions: {sourceType: "module"}, + env: { es6: true }, + parserOptions: { sourceType: "module" }, errors: ["'new buffer.Buffer()' was deprecated since v6. Use 'buffer.Buffer.alloc()' or 'buffer.Buffer.from()' (use 'https://www.npmjs.com/package/safe-buffer' for '<4.5.0') instead."], }, { code: "import * as b from 'buffer'; new b.Buffer()", - env: {es6: true}, - parserOptions: {sourceType: "module"}, + env: { es6: true }, + parserOptions: { sourceType: "module" }, errors: ["'new buffer.Buffer()' was deprecated since v6. Use 'buffer.Buffer.alloc()' or 'buffer.Buffer.from()' (use 'https://www.npmjs.com/package/safe-buffer' for '<4.5.0') instead."], }, { code: "import * as b from 'buffer'; new b.default.Buffer()", - env: {es6: true}, - parserOptions: {sourceType: "module"}, + env: { es6: true }, + parserOptions: { sourceType: "module" }, errors: ["'new buffer.default.Buffer()' was deprecated since v6. Use 'buffer.Buffer.alloc()' or 'buffer.Buffer.from()' (use 'https://www.npmjs.com/package/safe-buffer' for '<4.5.0') instead."], }, { code: "import {Buffer as b} from 'buffer'; new b()", - env: {es6: true}, - parserOptions: {sourceType: "module"}, + env: { es6: true }, + parserOptions: { sourceType: "module" }, errors: ["'new buffer.Buffer()' was deprecated since v6. Use 'buffer.Buffer.alloc()' or 'buffer.Buffer.from()' (use 'https://www.npmjs.com/package/safe-buffer' for '<4.5.0') instead."], }, { code: "import b from 'buffer'; b.SlowBuffer", - env: {es6: true}, - parserOptions: {sourceType: "module"}, + env: { es6: true }, + parserOptions: { sourceType: "module" }, errors: ["'buffer.SlowBuffer' was deprecated since v6. Use 'buffer.Buffer.allocUnsafeSlow()' instead."], }, { code: "import * as b from 'buffer'; b.SlowBuffer", - env: {es6: true}, - parserOptions: {sourceType: "module"}, + env: { es6: true }, + parserOptions: { sourceType: "module" }, errors: ["'buffer.SlowBuffer' was deprecated since v6. Use 'buffer.Buffer.allocUnsafeSlow()' instead."], }, { code: "import * as b from 'buffer'; b.default.SlowBuffer", - env: {es6: true}, - parserOptions: {sourceType: "module"}, + env: { es6: true }, + parserOptions: { sourceType: "module" }, errors: ["'buffer.default.SlowBuffer' was deprecated since v6. Use 'buffer.Buffer.allocUnsafeSlow()' instead."], }, { code: "import {SlowBuffer as b} from 'buffer';", - env: {es6: true}, - parserOptions: {sourceType: "module"}, + env: { es6: true }, + parserOptions: { sourceType: "module" }, errors: ["'buffer.SlowBuffer' was deprecated since v6. Use 'buffer.Buffer.allocUnsafeSlow()' instead."], }, { code: "import domain from 'domain';", - env: {es6: true}, - parserOptions: {sourceType: "module"}, + env: { es6: true }, + parserOptions: { sourceType: "module" }, errors: ["'domain' module was deprecated since v4."], }, { code: "new (require('buffer').Buffer)()", - env: {node: true}, + env: { node: true }, options: [{ // ignoreModuleItems: ["buffer.Buffer()"], ignoreGlobalItems: ["Buffer()", "new Buffer()"], @@ -571,7 +571,7 @@ ruleTester.run("no-deprecated-api", rule, { }, { code: "require('buffer').Buffer()", - env: {node: true}, + env: { node: true }, options: [{ // ignoreModuleItems: ["new buffer.Buffer()"], ignoreGlobalItems: ["Buffer()", "new Buffer()"], @@ -584,64 +584,64 @@ ruleTester.run("no-deprecated-api", rule, { //---------------------------------------------------------------------- { code: "new Buffer;", - env: {node: true}, + env: { node: true }, errors: ["'new Buffer()' was deprecated since v6. Use 'Buffer.alloc()' or 'Buffer.from()' (use 'https://www.npmjs.com/package/safe-buffer' for '<4.5.0') instead."], }, { code: "Buffer();", - env: {node: true}, + env: { node: true }, errors: ["'Buffer()' was deprecated since v6. Use 'Buffer.alloc()' or 'Buffer.from()' (use 'https://www.npmjs.com/package/safe-buffer' for '<4.5.0') instead."], }, { code: "GLOBAL; /*globals GLOBAL*/", - env: {node: true}, + env: { node: true }, errors: ["'GLOBAL' was deprecated since v6. Use 'global' instead."], }, { code: "Intl.v8BreakIterator;", - env: {node: true}, + env: { node: true }, errors: ["'Intl.v8BreakIterator' was deprecated since v7."], }, { code: "require.extensions;", - env: {node: true}, + env: { node: true }, errors: ["'require.extensions' was deprecated since v0.12. Use compiling them ahead of time instead."], }, { code: "root;", - env: {node: true}, + env: { node: true }, errors: ["'root' was deprecated since v6. Use 'global' instead."], }, { code: "process.EventEmitter;", - env: {node: true}, + env: { node: true }, errors: ["'process.EventEmitter' was deprecated since v0.6. Use 'require(\"events\")' instead."], }, { code: "process.env.NODE_REPL_HISTORY_FILE;", - env: {node: true}, + env: { node: true }, errors: ["'process.env.NODE_REPL_HISTORY_FILE' was deprecated since v4. Use 'NODE_REPL_HISTORY' instead."], }, { code: "let {env: {NODE_REPL_HISTORY_FILE}} = process;", - env: {node: true, es6: true}, + env: { node: true, es6: true }, errors: ["'process.env.NODE_REPL_HISTORY_FILE' was deprecated since v4. Use 'NODE_REPL_HISTORY' instead."], }, { code: "require('domain');", - env: {node: true}, + env: { node: true }, errors: ["'domain' module was deprecated since v4."], }, { code: "import domain from 'domain';", - parserOptions: {sourceType: "module"}, + parserOptions: { sourceType: "module" }, errors: ["'domain' module was deprecated since v4."], }, { code: "new Buffer()", - env: {node: true}, + env: { node: true }, options: [{ // ignoreModuleItems: ["buffer.Buffer()", "new buffer.Buffer()"], ignoreGlobalItems: ["Buffer()"], @@ -650,7 +650,7 @@ ruleTester.run("no-deprecated-api", rule, { }, { code: "Buffer()", - env: {node: true}, + env: { node: true }, options: [{ // ignoreModuleItems: ["buffer.Buffer()", "new buffer.Buffer()"], ignoreGlobalItems: ["new Buffer()"], diff --git a/tests/lib/rules/no-extraneous-import.js b/tests/lib/rules/no-extraneous-import.js index 91e894d3..28f29c55 100644 --- a/tests/lib/rules/no-extraneous-import.js +++ b/tests/lib/rules/no-extraneous-import.js @@ -30,7 +30,7 @@ function fixture(name) { // Tests //------------------------------------------------------------------------------ -const tester = new RuleTester({parserOptions: {sourceType: "module"}}) +const tester = new RuleTester({ parserOptions: { sourceType: "module" } }) tester.run("no-extraneous-import", rule, { valid: [ diff --git a/tests/lib/rules/no-extraneous-require.js b/tests/lib/rules/no-extraneous-require.js index e6f4e185..a60e1868 100644 --- a/tests/lib/rules/no-extraneous-require.js +++ b/tests/lib/rules/no-extraneous-require.js @@ -30,7 +30,7 @@ function fixture(name) { // Tests //------------------------------------------------------------------------------ -const tester = new RuleTester({env: {node: true}}) +const tester = new RuleTester({ env: { node: true } }) tester.run("no-extraneous-require", rule, { valid: [ diff --git a/tests/lib/rules/no-missing-import.js b/tests/lib/rules/no-missing-import.js index 8d2eba95..bafa75ec 100644 --- a/tests/lib/rules/no-missing-import.js +++ b/tests/lib/rules/no-missing-import.js @@ -30,7 +30,7 @@ function fixture(name) { // Test //------------------------------------------------------------------------------ -const ruleTester = new RuleTester({parserOptions: {sourceType: "module"}}) +const ruleTester = new RuleTester({ parserOptions: { sourceType: "module" } }) ruleTester.run("no-missing-import", rule, { valid: [ { @@ -86,12 +86,12 @@ ruleTester.run("no-missing-import", rule, { { filename: fixture("test.js"), code: "import c from './c';", - options: [{tryExtensions: [".coffee"]}], + options: [{ tryExtensions: [".coffee"] }], }, { filename: fixture("test.js"), code: "import c from './c';", - settings: {node: {tryExtensions: [".coffee"]}}, + settings: { node: { tryExtensions: [".coffee"] } }, }, // Ignores it if the filename is unknown. @@ -136,27 +136,27 @@ ruleTester.run("no-missing-import", rule, { { filename: fixture("test.js"), code: "import electron from 'electron';", - options: [{allowModules: ["electron"]}], + options: [{ allowModules: ["electron"] }], }, // resolvePaths { filename: fixture("test.js"), code: "import a from 'fixtures/no-missing/a';", - env: {node: true}, - settings: {node: {resolvePaths: [path.resolve(__dirname, "../../")]}}, + env: { node: true }, + settings: { node: { resolvePaths: [path.resolve(__dirname, "../../")] } }, }, { filename: fixture("test.js"), code: "import a from 'fixtures/no-missing/a';", - env: {node: true}, - options: [{resolvePaths: [path.resolve(__dirname, "../../")]}], + env: { node: true }, + options: [{ resolvePaths: [path.resolve(__dirname, "../../")] }], }, { filename: fixture("test.js"), code: "import a from 'fixtures/no-missing/a';", - env: {node: true}, - options: [{resolvePaths: ["tests"]}], + env: { node: true }, + options: [{ resolvePaths: ["tests"] }], }, ], invalid: [ diff --git a/tests/lib/rules/no-missing-require.js b/tests/lib/rules/no-missing-require.js index d9b40af3..92b6064c 100644 --- a/tests/lib/rules/no-missing-require.js +++ b/tests/lib/rules/no-missing-require.js @@ -36,108 +36,108 @@ ruleTester.run("no-missing-require", rule, { { filename: fixture("test.js"), code: "require('fs');", - env: {node: true}, + env: { node: true }, }, { filename: fixture("test.js"), code: "require('eslint');", - env: {node: true}, + env: { node: true }, }, { filename: fixture("test.js"), code: "require('eslint/lib/ast-utils');", - env: {node: true}, + env: { node: true }, }, { filename: fixture("test.js"), code: "require('./a');", - env: {node: true}, + env: { node: true }, }, { filename: fixture("test.js"), code: "require('./a.js');", - env: {node: true}, + env: { node: true }, }, { filename: fixture("test.js"), code: "require('./a.config');", - env: {node: true}, + env: { node: true }, }, { filename: fixture("test.js"), code: "require('./a.config.js');", - env: {node: true}, + env: { node: true }, }, { filename: fixture("test.js"), code: "require('./b');", - env: {node: true}, + env: { node: true }, }, { filename: fixture("test.js"), code: "require('./b.json');", - env: {node: true}, + env: { node: true }, }, { filename: fixture("test.js"), code: "require('./c.coffee');", - env: {node: true}, + env: { node: true }, }, { filename: fixture("test.js"), code: "require('mocha');", - env: {node: true}, + env: { node: true }, }, { filename: fixture("test.js"), code: "require(`eslint`);", - env: {node: true, es6: true}, + env: { node: true, es6: true }, }, { filename: fixture("test.js"), code: "require('mocha!foo?a=b&c=d');", - env: {node: true}, + env: { node: true }, }, // tryExtensions { filename: fixture("test.js"), code: "require('./c');", - env: {node: true}, - options: [{tryExtensions: [".coffee"]}], + env: { node: true }, + options: [{ tryExtensions: [".coffee"] }], }, { filename: fixture("test.js"), code: "require('./c');", - env: {node: true}, - settings: {node: {tryExtensions: [".coffee"]}}, + env: { node: true }, + settings: { node: { tryExtensions: [".coffee"] } }, }, // resolvePaths { filename: fixture("test.js"), code: "require('fixtures/no-missing/a');", - env: {node: true}, - settings: {node: {resolvePaths: [path.resolve(__dirname, "../../")]}}, + env: { node: true }, + settings: { node: { resolvePaths: [path.resolve(__dirname, "../../")] } }, }, { filename: fixture("test.js"), code: "require('fixtures/no-missing/a');", - env: {node: true}, - options: [{resolvePaths: [path.resolve(__dirname, "../../")]}], + env: { node: true }, + options: [{ resolvePaths: [path.resolve(__dirname, "../../")] }], }, { filename: fixture("test.js"), code: "require('fixtures/no-missing/a');", - env: {node: true}, - options: [{resolvePaths: ["tests"]}], + env: { node: true }, + options: [{ resolvePaths: ["tests"] }], }, // Ignores it if not callee. { filename: fixture("test.js"), code: "require;", - env: {node: true}, + env: { node: true }, }, // Ignores it if the global variable of `require` is not defined. @@ -149,112 +149,112 @@ ruleTester.run("no-missing-require", rule, { // Ignores it if the filename is unknown. { code: "require('no-exist-package-0');", - env: {node: true}, + env: { node: true }, }, { code: "require('./b');", - env: {node: true}, + env: { node: true }, }, // Ignores it if the target is not string. { filename: fixture("test.js"), code: "require();", - env: {node: true}, + env: { node: true }, }, { filename: fixture("test.js"), code: "require(foo);", - env: {node: true}, + env: { node: true }, }, { filename: fixture("test.js"), code: "require(777);", - env: {node: true}, + env: { node: true }, }, { filename: fixture("test.js"), code: "require(`foo${bar}`);", //eslint-disable-line no-template-curly-in-string - env: {node: true, es6: true}, + env: { node: true, es6: true }, }, // Should work fine if the filename is relative. { filename: "tests/fixtures/no-missing/test.js", code: "require('eslint');", - env: {node: true}, + env: { node: true }, }, { filename: "tests/fixtures/no-missing/test.js", code: "require('./a');", - env: {node: true}, + env: { node: true }, }, // Relative paths to a directory should work. { filename: fixture("test.js"), code: "require('.');", - env: {node: true}, + env: { node: true }, }, { filename: fixture("test.js"), code: "require('./');", - env: {node: true}, + env: { node: true }, }, { filename: fixture("test.js"), code: "require('./foo');", - env: {node: true}, + env: { node: true }, }, { filename: fixture("test.js"), code: "require('./foo/');", - env: {node: true}, + env: { node: true }, }, // allow option { filename: fixture("test.js"), code: "require('electron');", - options: [{allowModules: ["electron"]}], - env: {node: true}, + options: [{ allowModules: ["electron"] }], + env: { node: true }, }, { filename: fixture("test.js"), code: "require('jquery.cookie');", - options: [{allowModules: ["jquery.cookie"]}], - env: {node: true}, + options: [{ allowModules: ["jquery.cookie"] }], + env: { node: true }, }, ], invalid: [ { filename: fixture("test.js"), code: "require('no-exist-package-0');", - env: {node: true}, + env: { node: true }, errors: ["\"no-exist-package-0\" is not found."], }, { filename: fixture("test.js"), code: "require('@mysticatea/test');", - env: {node: true}, + env: { node: true }, errors: ["\"@mysticatea/test\" is not found."], }, { filename: fixture("test.js"), code: "require('./c');", - env: {node: true}, + env: { node: true }, errors: ["\"./c\" is not found."], }, { filename: fixture("test.js"), code: "require('./d');", - env: {node: true}, + env: { node: true }, errors: ["\"./d\" is not found."], }, { filename: fixture("test.js"), code: "require('./a.json');", - env: {node: true}, + env: { node: true }, errors: ["\"./a.json\" is not found."], }, @@ -262,13 +262,13 @@ ruleTester.run("no-missing-require", rule, { { filename: "tests/fixtures/no-missing/test.js", code: "require('no-exist-package-0');", - env: {node: true}, + env: { node: true }, errors: ["\"no-exist-package-0\" is not found."], }, { filename: "tests/fixtures/no-missing/test.js", code: "require('./c');", - env: {node: true}, + env: { node: true }, errors: ["\"./c\" is not found."], }, @@ -276,13 +276,13 @@ ruleTester.run("no-missing-require", rule, { { filename: fixture("test.js"), code: "require('./bar');", - env: {node: true}, + env: { node: true }, errors: ["\"./bar\" is not found."], }, { filename: fixture("test.js"), code: "require('./bar/');", - env: {node: true}, + env: { node: true }, errors: ["\"./bar/\" is not found."], }, @@ -290,7 +290,7 @@ ruleTester.run("no-missing-require", rule, { { filename: fixture("test.js"), code: "require('./A');", - env: {node: true}, + env: { node: true }, errors: ["\"./A\" is not found."], }, ], @@ -314,7 +314,7 @@ describe("On specific working directory:", () => { { filename: fixture("test.js"), code: "require('../../lib/rules/no-missing-require');", - env: {node: true}, + env: { node: true }, }, ], invalid: [], diff --git a/tests/lib/rules/no-unpublished-bin.js b/tests/lib/rules/no-unpublished-bin.js index f448b4cb..03a3cb09 100644 --- a/tests/lib/rules/no-unpublished-bin.js +++ b/tests/lib/rules/no-unpublished-bin.js @@ -78,50 +78,50 @@ new RuleTester().run("no-unpublished-bin", rule, { }, // empty name - {code: "'stdin'"}, + { code: "'stdin'" }, // convertPath option { filename: fixture("simple-files/a.js"), code: "'simple-files/a.js'", - options: [{convertPath: {"a.js": ["a.js", "lib/a.js"]}}], + options: [{ convertPath: { "a.js": ["a.js", "lib/a.js"] } }], }, { filename: fixture("multi-files/a.js"), code: "'multi-files/a.js'", - options: [{convertPath: {"a.js": ["a.js", "lib/a.js"]}}], + options: [{ convertPath: { "a.js": ["a.js", "lib/a.js"] } }], }, { filename: fixture("simple-npmignore/a.js"), code: "'simple-npmignore/a.js'", - options: [{convertPath: {"a.js": ["a.js", "lib/a.js"]}}], + options: [{ convertPath: { "a.js": ["a.js", "lib/a.js"] } }], }, { filename: fixture("multi-npmignore/a.js"), code: "'multi-npmignore/a.js'", - options: [{convertPath: {"a.js": ["a.js", "lib/a.js"]}}], + options: [{ convertPath: { "a.js": ["a.js", "lib/a.js"] } }], }, // convertPath shared setting { filename: fixture("simple-files/a.js"), code: "'simple-files/a.js'", - settings: {node: {convertPath: {"a.js": ["a.js", "lib/a.js"]}}}, + settings: { node: { convertPath: { "a.js": ["a.js", "lib/a.js"] } } }, }, { filename: fixture("multi-files/a.js"), code: "'multi-files/a.js'", - settings: {node: {convertPath: {"a.js": ["a.js", "lib/a.js"]}}}, + settings: { node: { convertPath: { "a.js": ["a.js", "lib/a.js"] } } }, }, { filename: fixture("simple-npmignore/a.js"), code: "'simple-npmignore/a.js'", - settings: {node: {convertPath: {"a.js": ["a.js", "lib/a.js"]}}}, + settings: { node: { convertPath: { "a.js": ["a.js", "lib/a.js"] } } }, }, { filename: fixture("multi-npmignore/a.js"), code: "'multi-npmignore/a.js'", - settings: {node: {convertPath: {"a.js": ["a.js", "lib/a.js"]}}}, + settings: { node: { convertPath: { "a.js": ["a.js", "lib/a.js"] } } }, }, ], invalid: [ @@ -158,19 +158,19 @@ new RuleTester().run("no-unpublished-bin", rule, { { filename: fixture("simple-files/x.js"), code: "'simple-files/x.js'", - options: [{convertPath: {"x.js": ["x.js", "a.js"]}}], + options: [{ convertPath: { "x.js": ["x.js", "a.js"] } }], errors: ["npm ignores 'a.js'. Check 'files' field of 'package.json' or '.npmignore'."], }, { filename: fixture("multi-files/x.js"), code: "'multi-files/x.js'", - options: [{convertPath: {"x.js": ["x.js", "a.js"]}}], + options: [{ convertPath: { "x.js": ["x.js", "a.js"] } }], errors: ["npm ignores 'a.js'. Check 'files' field of 'package.json' or '.npmignore'."], }, { filename: fixture("multi-files/x.js"), code: "'multi-files/x.js'", - options: [{convertPath: {"x.js": ["x.js", "b.js"]}}], + options: [{ convertPath: { "x.js": ["x.js", "b.js"] } }], errors: ["npm ignores 'b.js'. Check 'files' field of 'package.json' or '.npmignore'."], }, @@ -178,13 +178,13 @@ new RuleTester().run("no-unpublished-bin", rule, { { filename: fixture("simple-npmignore/x.js"), code: "'simple-npmignore/x.js'", - options: [{convertPath: {"x.js": ["x.js", "a.js"]}}], + options: [{ convertPath: { "x.js": ["x.js", "a.js"] } }], errors: ["npm ignores 'a.js'. Check 'files' field of 'package.json' or '.npmignore'."], }, { filename: fixture("multi-npmignore/x.js"), code: "'multi-npmignore/x.js'", - options: [{convertPath: {"x.js": ["x.js", "a.js"]}}], + options: [{ convertPath: { "x.js": ["x.js", "a.js"] } }], errors: ["npm ignores 'a.js'. Check 'files' field of 'package.json' or '.npmignore'."], }, { @@ -218,19 +218,19 @@ new RuleTester().run("no-unpublished-bin", rule, { { filename: fixture("simple-files/x.js"), code: "'simple-files/x.js'", - settings: {node: {convertPath: {"x.js": ["x.js", "a.js"]}}}, + settings: { node: { convertPath: { "x.js": ["x.js", "a.js"] } } }, errors: ["npm ignores 'a.js'. Check 'files' field of 'package.json' or '.npmignore'."], }, { filename: fixture("multi-files/x.js"), code: "'multi-files/x.js'", - settings: {node: {convertPath: {"x.js": ["x.js", "a.js"]}}}, + settings: { node: { convertPath: { "x.js": ["x.js", "a.js"] } } }, errors: ["npm ignores 'a.js'. Check 'files' field of 'package.json' or '.npmignore'."], }, { filename: fixture("multi-files/x.js"), code: "'multi-files/x.js'", - settings: {node: {convertPath: {"x.js": ["x.js", "b.js"]}}}, + settings: { node: { convertPath: { "x.js": ["x.js", "b.js"] } } }, errors: ["npm ignores 'b.js'. Check 'files' field of 'package.json' or '.npmignore'."], }, @@ -238,13 +238,13 @@ new RuleTester().run("no-unpublished-bin", rule, { { filename: fixture("simple-npmignore/x.js"), code: "'simple-npmignore/x.js'", - settings: {node: {convertPath: {"x.js": ["x.js", "a.js"]}}}, + settings: { node: { convertPath: { "x.js": ["x.js", "a.js"] } } }, errors: ["npm ignores 'a.js'. Check 'files' field of 'package.json' or '.npmignore'."], }, { filename: fixture("multi-npmignore/x.js"), code: "'multi-npmignore/x.js'", - settings: {node: {convertPath: {"x.js": ["x.js", "a.js"]}}}, + settings: { node: { convertPath: { "x.js": ["x.js", "a.js"] } } }, errors: ["npm ignores 'a.js'. Check 'files' field of 'package.json' or '.npmignore'."], }, ], diff --git a/tests/lib/rules/no-unpublished-import.js b/tests/lib/rules/no-unpublished-import.js index ef126a57..62a95bcb 100644 --- a/tests/lib/rules/no-unpublished-import.js +++ b/tests/lib/rules/no-unpublished-import.js @@ -30,7 +30,7 @@ function fixture(name) { // Test //------------------------------------------------------------------------------ -const ruleTester = new RuleTester({parserOptions: {sourceType: "module"}}) +const ruleTester = new RuleTester({ parserOptions: { sourceType: "module" } }) ruleTester.run("no-unpublished-import", rule, { valid: [ { @@ -119,14 +119,14 @@ ruleTester.run("no-unpublished-import", rule, { { filename: fixture("1/test.js"), code: "import electron from 'electron';", - options: [{allowModules: ["electron"]}], + options: [{ allowModules: ["electron"] }], }, // Auto-published files only apply to root package directory { filename: fixture("3/src/readme.js"), code: "import bbb from 'bbb';", - env: {node: true}, + env: { node: true }, }, // Negative patterns in files field. @@ -192,7 +192,7 @@ ruleTester.run("no-unpublished-import", rule, { errors: ["\"../test\" is not published."], settings: { node: { - convertPath: {"src/**/*.jsx": ["src/(.+?)\\.jsx", "pub/$1.js"]}, + convertPath: { "src/**/*.jsx": ["src/(.+?)\\.jsx", "pub/$1.js"] }, tryExtensions: [".js", ".jsx", ".json"], }, }, @@ -202,7 +202,7 @@ ruleTester.run("no-unpublished-import", rule, { code: "import a from '../test';", errors: ["\"../test\" is not published."], options: [{ - convertPath: {"src/**/*.jsx": ["src/(.+?)\\.jsx", "pub/$1.js"]}, + convertPath: { "src/**/*.jsx": ["src/(.+?)\\.jsx", "pub/$1.js"] }, tryExtensions: [".js", ".jsx", ".json"], }], }, @@ -241,7 +241,7 @@ ruleTester.run("no-unpublished-import", rule, { { filename: fixture("1/test.js"), code: "import a from '../2/a.js';", - env: {node: true}, + env: { node: true }, errors: ["\"../2/a.js\" is not published."], }, ], diff --git a/tests/lib/rules/no-unpublished-require.js b/tests/lib/rules/no-unpublished-require.js index 5b68a23a..bd408576 100644 --- a/tests/lib/rules/no-unpublished-require.js +++ b/tests/lib/rules/no-unpublished-require.js @@ -36,97 +36,97 @@ ruleTester.run("no-unpublished-require", rule, { { filename: fixture("1/test.js"), code: "require('fs');", - env: {node: true}, + env: { node: true }, }, { filename: fixture("1/test.js"), code: "require('aaa');", - env: {node: true}, + env: { node: true }, }, { filename: fixture("1/test.js"), code: "require('aaa/a/b/c');", - env: {node: true}, + env: { node: true }, }, { filename: fixture("1/test.js"), code: "require('./a');", - env: {node: true}, + env: { node: true }, }, { filename: fixture("1/test.js"), code: "require('./a.js');", - env: {node: true}, + env: { node: true }, }, { filename: fixture("2/ignore1.js"), code: "require('./test');", - env: {node: true}, + env: { node: true }, }, { filename: fixture("2/ignore1.js"), code: "require('bbb');", - env: {node: true}, + env: { node: true }, }, { filename: fixture("2/ignore1.js"), code: "require('bbb/a/b/c');", - env: {node: true}, + env: { node: true }, }, { filename: fixture("2/ignore1.js"), code: "require('./ignore2');", - env: {node: true}, + env: { node: true }, }, { filename: fixture("3/test.js"), code: "require('./pub/a');", - env: {node: true}, + env: { node: true }, }, { filename: fixture("3/test.js"), code: "require('./test2');", - env: {node: true}, + env: { node: true }, }, { filename: fixture("3/test.js"), code: "require('aaa');", - env: {node: true}, + env: { node: true }, }, { filename: fixture("3/test.js"), code: "require('bbb');", - env: {node: true}, + env: { node: true }, }, { filename: fixture("3/pub/ignore1.js"), code: "require('bbb');", - env: {node: true}, + env: { node: true }, }, { filename: fixture("3/pub/test.js"), code: "require('../package.json');", - env: {node: true}, + env: { node: true }, }, { filename: fixture("3/src/pub/test.js"), code: "require('bbb');", - env: {node: true}, + env: { node: true }, }, { filename: fixture("3/src/pub/test.js"), code: "require('bbb!foo?a=b&c=d');", - env: {node: true}, + env: { node: true }, }, // `convertPath` option. { filename: fixture("3/src/test.jsx"), code: "require('./a');", - env: {node: true}, + env: { node: true }, settings: { node: { - convertPath: {"src/**/*.jsx": ["src/(.+?)\\.jsx", "pub/$1.js"]}, + convertPath: { "src/**/*.jsx": ["src/(.+?)\\.jsx", "pub/$1.js"] }, tryExtensions: [".js", ".jsx", ".json"], }, }, @@ -134,16 +134,16 @@ ruleTester.run("no-unpublished-require", rule, { { filename: fixture("3/src/test.jsx"), code: "require('./a');", - env: {node: true}, + env: { node: true }, options: [{ - convertPath: {"src/**/*.jsx": ["src/(.+?)\\.jsx", "pub/$1.js"]}, + convertPath: { "src/**/*.jsx": ["src/(.+?)\\.jsx", "pub/$1.js"] }, tryExtensions: [".js", ".jsx", ".json"], }], }, { filename: fixture("3/src/test.jsx"), code: "require('../test');", - env: {node: true}, + env: { node: true }, settings: { node: { convertPath: [ @@ -159,7 +159,7 @@ ruleTester.run("no-unpublished-require", rule, { { filename: fixture("3/src/test.jsx"), code: "require('../test');", - env: {node: true}, + env: { node: true }, options: [{ convertPath: [ { @@ -175,7 +175,7 @@ ruleTester.run("no-unpublished-require", rule, { { filename: fixture("1/test.js"), code: "require;", - env: {node: true}, + env: { node: true }, }, // Ignores it if the global variable of `require` is not defined. @@ -187,117 +187,117 @@ ruleTester.run("no-unpublished-require", rule, { // Ignores it if the filename is unknown. { code: "require('no-exist-package-0');", - env: {node: true}, + env: { node: true }, }, { code: "require('./b');", - env: {node: true}, + env: { node: true }, }, // Ignores it if the target is not string. { filename: fixture("1/test.js"), code: "require();", - env: {node: true}, + env: { node: true }, }, { filename: fixture("1/test.js"), code: "require(foo);", - env: {node: true}, + env: { node: true }, }, { filename: fixture("1/test.js"), code: "require(777);", - env: {node: true}, + env: { node: true }, }, { filename: fixture("1/test.js"), code: "require(`foo${bar}`);", //eslint-disable-line no-template-curly-in-string - env: {node: true, es6: true}, + env: { node: true, es6: true }, }, // Should work fine if the filename is relative. { filename: "tests/fixtures/no-unpublished/2/test.js", code: "require('aaa');", - env: {node: true}, + env: { node: true }, }, { filename: "tests/fixtures/no-unpublished/2/test.js", code: "require('./a');", - env: {node: true}, + env: { node: true }, }, // allowModules option { filename: fixture("1/test.js"), code: "require('electron');", - options: [{allowModules: ["electron"]}], - env: {node: true}, + options: [{ allowModules: ["electron"] }], + env: { node: true }, }, // Auto-published files only apply to root package directory { filename: fixture("3/src/readme.js"), code: "require('bbb');", - env: {node: true}, + env: { node: true }, }, // Negative patterns in files field. { filename: fixture("negative-in-files/lib/__test__/index.js"), code: "require('bbb');", - env: {node: true}, + env: { node: true }, }, ], invalid: [ { filename: fixture("2/test.js"), code: "require('./ignore1.js');", - env: {node: true}, + env: { node: true }, errors: ["\"./ignore1.js\" is not published."], }, { filename: fixture("2/test.js"), code: "require('./ignore1');", - env: {node: true}, + env: { node: true }, errors: ["\"./ignore1\" is not published."], }, { filename: fixture("3/pub/test.js"), code: "require('bbb');", - env: {node: true}, + env: { node: true }, errors: ["\"bbb\" is not published."], }, { filename: fixture("3/pub/test.js"), code: "require('./ignore1');", - env: {node: true}, + env: { node: true }, errors: ["\"./ignore1\" is not published."], }, { filename: fixture("3/pub/test.js"), code: "require('./abc');", - env: {node: true}, + env: { node: true }, errors: ["\"./abc\" is not published."], }, { filename: fixture("3/pub/test.js"), code: "require('../test');", - env: {node: true}, + env: { node: true }, errors: ["\"../test\" is not published."], }, { filename: fixture("3/pub/test.js"), code: "require('../src/pub/a.js');", - env: {node: true}, + env: { node: true }, errors: ["\"../src/pub/a.js\" is not published."], }, { filename: fixture("1/test.js"), code: "require('../a.js');", - env: {node: true}, + env: { node: true }, errors: ["\"../a.js\" is not published."], }, @@ -306,21 +306,21 @@ ruleTester.run("no-unpublished-require", rule, { filename: fixture("3/src/test.jsx"), code: "require('../test');", errors: ["\"../test\" is not published."], - env: {node: true}, - settings: {node: {convertPath: {"src/**/*.jsx": ["src/(.+?)\\.jsx", "pub/$1.js"]}}}, + env: { node: true }, + settings: { node: { convertPath: { "src/**/*.jsx": ["src/(.+?)\\.jsx", "pub/$1.js"] } } }, }, { filename: fixture("3/src/test.jsx"), code: "require('../test');", errors: ["\"../test\" is not published."], - env: {node: true}, - options: [{convertPath: {"src/**/*.jsx": ["src/(.+?)\\.jsx", "pub/$1.js"]}}], + env: { node: true }, + options: [{ convertPath: { "src/**/*.jsx": ["src/(.+?)\\.jsx", "pub/$1.js"] } }], }, { filename: fixture("3/src/test.jsx"), code: "require('../test');", errors: ["\"../test\" is not published."], - env: {node: true}, + env: { node: true }, settings: { node: { convertPath: [ @@ -336,7 +336,7 @@ ruleTester.run("no-unpublished-require", rule, { filename: fixture("3/src/test.jsx"), code: "require('../test');", errors: ["\"../test\" is not published."], - env: {node: true}, + env: { node: true }, options: [{ convertPath: [ { @@ -352,14 +352,14 @@ ruleTester.run("no-unpublished-require", rule, { filename: "tests/fixtures/no-unpublished/2/test.js", code: "require('./ignore1');", errors: ["\"./ignore1\" is not published."], - env: {node: true}, + env: { node: true }, }, // outside of the package. { filename: fixture("1/test.js"), code: "require('../2/a.js');", - env: {node: true}, + env: { node: true }, errors: ["\"../2/a.js\" is not published."], }, ], diff --git a/tests/lib/rules/no-unsupported-features.js b/tests/lib/rules/no-unsupported-features.js index 732f6d84..a9435ede 100644 --- a/tests/lib/rules/no-unsupported-features.js +++ b/tests/lib/rules/no-unsupported-features.js @@ -45,7 +45,7 @@ function convertPattern(retv, pattern) { ) { convertPattern( retv, - Object.create(pattern, {modules: {value: true}}) + Object.create(pattern, { modules: { value: true } }) ) } @@ -71,8 +71,8 @@ function convertPattern(retv, pattern) { // If this is supported, add to a valid pattern. retv.valid.push({ code: `/*${pattern.name}: ${versionText}*/ ${pattern.code}`, - env: {es6: true}, - globals: {SharedArrayBuffer: false, Atomics: false}, + env: { es6: true }, + globals: { SharedArrayBuffer: false, Atomics: false }, options: [version], parserOptions: { ecmaVersion: 8, @@ -84,9 +84,9 @@ function convertPattern(retv, pattern) { // If this is not supported, add to a valid pattern with a "ignores" option. [].push.apply(retv.valid, pattern.keys.map(key => ({ code: `/*${pattern.name}: ${versionText}, ignores: ["${key}"]*/ ${pattern.code}`, - env: {es6: true}, - globals: {SharedArrayBuffer: false, Atomics: false}, - options: [{version, ignores: [key]}], + env: { es6: true }, + globals: { SharedArrayBuffer: false, Atomics: false }, + options: [{ version, ignores: [key] }], parserOptions: { ecmaVersion: 8, sourceType: pattern.modules ? "module" : "script", @@ -96,8 +96,8 @@ function convertPattern(retv, pattern) { // If this is not supported, add to a invalid pattern. retv.invalid.push({ code: `/*${pattern.name}: ${versionText}*/ ${pattern.code}`, - env: {es6: true}, - globals: {SharedArrayBuffer: false, Atomics: false}, + env: { es6: true }, + globals: { SharedArrayBuffer: false, Atomics: false }, options: [version], parserOptions: { ecmaVersion: 8, @@ -1133,51 +1133,51 @@ ruleTester.run("no-unsupported-features", rule, [ { filename: fixture("gte-4.0.0/a.js"), code: "var a = () => 1", - env: {es6: true}, + env: { es6: true }, }, { filename: fixture("gte-4.4.0-lt-5.0.0/a.js"), code: "var a = () => 1", - env: {es6: true}, + env: { es6: true }, }, { filename: fixture("hat-4.1.2/a.js"), code: "var a = () => 1", - env: {es6: true}, + env: { es6: true }, }, { code: "'\\\\u{0123}'", - env: {es6: true}, + env: { es6: true }, }, { filename: fixture("gte-4.0.0/a.js"), code: "var a = async () => 1", - parserOptions: {ecmaVersion: 2017}, - options: [{ignores: ["asyncAwait"]}], + parserOptions: { ecmaVersion: 2017 }, + options: [{ ignores: ["asyncAwait"] }], }, { filename: fixture("gte-7.6.0/a.js"), code: "var a = async () => 1", - parserOptions: {ecmaVersion: 2017}, + parserOptions: { ecmaVersion: 2017 }, }, { filename: fixture("gte-7.10.0/a.js"), code: "var a = async () => 1", - parserOptions: {ecmaVersion: 2017}, + parserOptions: { ecmaVersion: 2017 }, }, { filename: fixture("invalid/a.js"), code: "var a = () => 1", - env: {es6: true}, + env: { es6: true }, }, { filename: fixture("nothing/a.js"), code: "var a = () => 1", - env: {es6: true}, + env: { es6: true }, }, { code: "var a = async () => 1", - parserOptions: {ecmaVersion: 2017}, + parserOptions: { ecmaVersion: 2017 }, options: ["7.10.0"], }, ], @@ -1185,39 +1185,39 @@ ruleTester.run("no-unsupported-features", rule, [ { filename: fixture("gte-0.12.8/a.js"), code: "var a = () => 1", - env: {es6: true}, + env: { es6: true }, errors: ["Arrow functions are not supported yet on Node >=0.12.8."], }, { filename: fixture("invalid/a.js"), code: "var a = (b,) => 1", - parserOptions: {ecmaVersion: 2017}, - env: {es6: true}, + parserOptions: { ecmaVersion: 2017 }, + env: { es6: true }, errors: ["Trailing commas in functions are not supported yet on Node 4.0.0."], }, { filename: fixture("lt-6.0.0/a.js"), code: "var a = () => 1", - parserOptions: {ecmaVersion: 2017}, - env: {es6: true}, + parserOptions: { ecmaVersion: 2017 }, + env: { es6: true }, errors: ["Arrow functions are not supported yet on Node <6.0.0."], }, { filename: fixture("nothing/a.js"), code: "var a = (b,) => 1", - parserOptions: {ecmaVersion: 2017}, - env: {es6: true}, + parserOptions: { ecmaVersion: 2017 }, + env: { es6: true }, errors: ["Trailing commas in functions are not supported yet on Node 4.0.0."], }, { filename: fixture("gte-7.5.0/a.js"), code: "var a = async () => 1", - parserOptions: {ecmaVersion: 2017}, + parserOptions: { ecmaVersion: 2017 }, errors: ["Async functions are not supported yet on Node >=7.5.0."], }, { code: "var a = async () => 1", - parserOptions: {ecmaVersion: 2017}, + parserOptions: { ecmaVersion: 2017 }, options: ["7.1.0"], errors: ["Async functions are not supported yet on Node 7.1.0."], }, diff --git a/tests/lib/rules/shebang.js b/tests/lib/rules/shebang.js index 8f2c2790..b918cb27 100644 --- a/tests/lib/rules/shebang.js +++ b/tests/lib/rules/shebang.js @@ -64,32 +64,32 @@ ruleTester.run("shebang", rule, { { filename: fixture("string-bin/src/bin/test.js"), code: "#!/usr/bin/env node\nhello();", - options: [{convertPath: {"src/**": ["^src/(.+)$", "$1"]}}], + options: [{ convertPath: { "src/**": ["^src/(.+)$", "$1"] } }], }, { filename: fixture("string-bin/src/lib/test.js"), code: "hello();", - options: [{convertPath: {"src/**": ["^src/(.+)$", "$1"]}}], + options: [{ convertPath: { "src/**": ["^src/(.+)$", "$1"] } }], }, { filename: fixture("object-bin/src/bin/a.js"), code: "#!/usr/bin/env node\nhello();", - options: [{convertPath: {"src/**": ["^src/(.+)$", "$1"]}}], + options: [{ convertPath: { "src/**": ["^src/(.+)$", "$1"] } }], }, { filename: fixture("object-bin/src/bin/b.js"), code: "#!/usr/bin/env node\nhello();", - options: [{convertPath: {"src/**": ["^src/(.+)$", "$1"]}}], + options: [{ convertPath: { "src/**": ["^src/(.+)$", "$1"] } }], }, { filename: fixture("object-bin/src/bin/c.js"), code: "hello();", - options: [{convertPath: {"src/**": ["^src/(.+)$", "$1"]}}], + options: [{ convertPath: { "src/**": ["^src/(.+)$", "$1"] } }], }, { filename: fixture("no-bin-field/src/lib/test.js"), code: "hello();", - options: [{convertPath: {"src/**": ["^src/(.+)$", "$1"]}}], + options: [{ convertPath: { "src/**": ["^src/(.+)$", "$1"] } }], }, // Should work fine if the filename is relative. @@ -181,56 +181,56 @@ ruleTester.run("shebang", rule, { filename: fixture("string-bin/src/bin/test.js"), code: "hello();", output: "#!/usr/bin/env node\nhello();", - options: [{convertPath: {"src/**": ["^src/(.+)$", "$1"]}}], + options: [{ convertPath: { "src/**": ["^src/(.+)$", "$1"] } }], errors: ["This file needs shebang \"#!/usr/bin/env node\"."], }, { filename: fixture("string-bin/src/bin/test.js"), code: "hello();", output: "#!/usr/bin/env node\nhello();", - settings: {node: {convertPath: {"src/**": ["^src/(.+)$", "$1"]}}}, + settings: { node: { convertPath: { "src/**": ["^src/(.+)$", "$1"] } } }, errors: ["This file needs shebang \"#!/usr/bin/env node\"."], }, { filename: fixture("string-bin/src/bin/test.js"), code: "#!/usr/bin/node\nhello();", output: "#!/usr/bin/env node\nhello();", - options: [{convertPath: {"src/**": ["^src/(.+)$", "$1"]}}], + options: [{ convertPath: { "src/**": ["^src/(.+)$", "$1"] } }], errors: ["This file needs shebang \"#!/usr/bin/env node\"."], }, { filename: fixture("string-bin/src/lib/test.js"), code: "#!/usr/bin/env node\nhello();", output: "hello();", - options: [{convertPath: {"src/**": ["^src/(.+)$", "$1"]}}], + options: [{ convertPath: { "src/**": ["^src/(.+)$", "$1"] } }], errors: ["This file needs no shebang."], }, { filename: fixture("object-bin/src/bin/a.js"), code: "hello();", output: "#!/usr/bin/env node\nhello();", - options: [{convertPath: {"src/**": ["^src/(.+)$", "$1"]}}], + options: [{ convertPath: { "src/**": ["^src/(.+)$", "$1"] } }], errors: ["This file needs shebang \"#!/usr/bin/env node\"."], }, { filename: fixture("object-bin/src/bin/b.js"), code: "#!/usr/bin/node\nhello();", output: "#!/usr/bin/env node\nhello();", - options: [{convertPath: {"src/**": ["^src/(.+)$", "$1"]}}], + options: [{ convertPath: { "src/**": ["^src/(.+)$", "$1"] } }], errors: ["This file needs shebang \"#!/usr/bin/env node\"."], }, { filename: fixture("object-bin/src/bin/c.js"), code: "#!/usr/bin/env node\nhello();", output: "hello();", - options: [{convertPath: {"src/**": ["^src/(.+)$", "$1"]}}], + options: [{ convertPath: { "src/**": ["^src/(.+)$", "$1"] } }], errors: ["This file needs no shebang."], }, { filename: fixture("no-bin-field/src/lib/test.js"), code: "#!/usr/bin/env node\nhello();", output: "hello();", - options: [{convertPath: {"src/**": ["^src/(.+)$", "$1"]}}], + options: [{ convertPath: { "src/**": ["^src/(.+)$", "$1"] } }], errors: ["This file needs no shebang."], },