eslint-config-uphold 0.2.0 → 2.1.0

package/LICENSE

@@ -0,0 +1,21 @@
+MIT License
+
+Copyright (c) 2016 Uphold
+
+Permission is hereby granted, free of charge, to any person obtaining a copy
+of this software and associated documentation files (the "Software"), to deal
+in the Software without restriction, including without limitation the rights
+to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+copies of the Software, and to permit persons to whom the Software is
+furnished to do so, subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included in all
+copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+SOFTWARE.

package/README.md

@@ -1,22 +1,27 @@
 # eslint-config-uphold
 
-Uphold-flavored ESLint config.
+Uphold-flavored [ESLint](https://eslint.org/) config.
+
+The [rules defined here](https://github.com/uphold/eslint-config-uphold/blob/master/src/index.js)
+extend the [eslint-recommended](https://github.com/eslint/eslint/blob/master/conf/eslint-recommended.js) preset,
+as well as the [overrides](https://github.com/prettier/eslint-config-prettier/blob/master/index.js)
+that allow the [Prettier](https://prettier.io) pretty-printer to reformat the code without conflicts.
 
 ## Installation
 
 ```sh
-$ npm install eslint eslint-config-uphold --save-dev
+$ npm install eslint eslint-config-uphold prettier --save-dev
 ```
 
 ## Usage
 
-Create an `.eslintrc.yml` file with the following:
+Create an `.eslintrc.yml` file with the following content:
 
 ```yaml
 extends: uphold
 ```
 
-Add the following `script` to your `package.json`:
+Add a `lint` command to the `scripts` section of your `package.json`, like so:
 
 ```json
 {
@@ -31,3 +36,17 @@ and run the linter with:
 ```sh
 $ npm run lint
 ```
+
+To automatically fix all lint issues, use the `--fix` option:
+
+```sh
+$ npm run lint --fix
+```
+
+## Upgrading eslint
+
+See the [eslint repo](https://github.com/eslint/eslint#semantic-versioning-policy) for eslint's guidelines on semantic versioning.
+
+A [tilde range](https://docs.npmjs.com/cli/v6/using-npm/semver#tilde-ranges-123-12-1) is set for the eslint dependency to pick up any patch changes by default.
+
+For any minor/major upgrades to `eslint` it is recommended to update both eslint and `eslint-config-uphold` and keep them in parallel. This is down to the fact that no guarantee is made that minor upgrades do not cause conflicts or issues with existing or new rules. The downside here is a package update is required for any security or other bug fixes. The benefit however is the included rules are always guaranteed to be stable.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-config-uphold",
-  "version": "0.2.0",
+  "version": "2.1.0",
   "description": "Uphold-flavored ESLint config",
   "keywords": [
     "config",
@@ -19,31 +19,42 @@
     "type": "git",
     "url": "git+ssh://git@github.com/uphold/eslint-config-uphold.git"
   },
+  "files": [
+    "src"
+  ],
   "scripts": {
-    "changelog": "github-changelog-generator --future-release=v$npm_config_future_release > CHANGELOG.md",
-    "lint": "eslint --config src/index.js src/index.js test/index.js",
+    "changelog": "echo \"$(github-changelog-generator --owner uphold --repo eslint-config-uphold --future-release=$npm_package_version)\n$(tail -n +2 CHANGELOG.md)\" > CHANGELOG.md",
+    "lint": "eslint src test/index.js",
     "test": "mocha $npm_package_options_mocha",
     "version": "npm run changelog --future-release=$npm_package_version && git add -A CHANGELOG.md"
   },
   "dependencies": {
-    "babel-eslint": "^7.1.1",
-    "eslint-plugin-mocha": "^5.1.0",
-    "eslint-plugin-sort-class-members": "^1.0.1",
+    "@babel/core": "^7.16.12",
+    "@babel/eslint-parser": "^7.16.5",
+    "eslint-config-prettier": "^7.2.0",
+    "eslint-plugin-mocha": "^5.3.0",
+    "eslint-plugin-prettier": "^3.3.1",
+    "eslint-plugin-rulesdir": "^0.1.0",
     "eslint-plugin-sort-imports-es6": "^0.0.3",
     "eslint-plugin-sql-template": "^2.0.0"
   },
   "devDependencies": {
-    "@uphold/github-changelog-generator": "^0.7.0",
-    "eslint": "^4.19.0",
-    "mocha": "^3.1.1",
+    "@uphold/github-changelog-generator": "^3.0.0",
+    "eslint": "~8.7.0",
+    "mocha": "^9.2.0",
     "pre-commit": "^1.2.2",
+    "prettier": "^2.2.1",
     "should": "^9.0.2"
   },
+  "peerDependencies": {
+    "eslint": "~8.7.0",
+    "prettier": "^2.2.1"
+  },
   "pre-commit": [
     "lint"
   ],
   "engines": {
-    "node": ">=6"
+    "node": ">=10.23.2"
   },
   "options": {
     "mocha": "-t 10000 --require should test"

package/src/index.js

@@ -1,3 +1,14 @@
+/**
+ * Module dependencies.
+ */
+
+const rulesDir = require('eslint-plugin-rulesdir');
+
+/**
+ * Configure the rulesdir plugin.
+ */
+
+rulesDir.RULES_DIR = `${__dirname}/rules`;
 
 /**
  * Export `uphold` shared configuration preset.
@@ -11,97 +22,56 @@ module.exports = {
     mocha: true,
     node: true
   },
-  extends: ['eslint:recommended'],
-  parser: 'babel-eslint',
-  plugins: ['mocha', 'sort-class-members', 'sort-imports-es6', 'sql-template'],
+  extends: ['eslint:recommended', 'plugin:prettier/recommended'],
+  parser: '@babel/eslint-parser',
+  parserOptions: {
+    requireConfigFile: false
+  },
+  plugins: ['mocha', 'rulesdir', 'sort-imports-es6', 'sql-template'],
   root: true,
   rules: {
     'accessor-pairs': 'error',
-    'array-bracket-spacing': 'error',
-    'arrow-parens': ['error', 'as-needed'],
-    'arrow-spacing': 'error',
     'block-scoped-var': 'error',
-    'block-spacing': 'off',
-    'brace-style': ['error', '1tbs', {
-      allowSingleLine: true
-    }],
-    camelcase: 'off',
-    'comma-dangle': 'error',
-    'comma-spacing': 'error',
-    'comma-style': 'error',
-    complexity: 'off',
-    'computed-property-spacing': 'error',
-    'consistent-return': 'off',
     'consistent-this': ['error', 'self'],
     curly: 'error',
     'default-case': 'error',
-    'dot-location': ['error', 'property'],
     'dot-notation': 'error',
-    'eol-last': 'error',
     eqeqeq: ['error', 'smart'],
-    'func-names': 'off',
-    'func-style': ['error', 'declaration', {
-      allowArrowFunctions: true
-    }],
-    'generator-star-spacing': ['error', 'before'],
-    'id-length': ['error', {
-      exceptions: ['_', 'e', 'i']
-    }],
-    'id-match': ['error', '^_$|^[a-zA-Z][a-zA-Z0-9]*$|^[A-Z][_A-Z0-9]+[A-Z0-9]$', {
-      onlyDeclarations: true,
-      properties: true
-    }],
-    indent: ['error', 2, {
-      SwitchCase: 1
-    }],
-    'key-spacing': 'error',
-    'keyword-spacing': 'error',
-    'linebreak-style': 'error',
-    'lines-around-comment': 'off',
+    'func-style': ['error', 'declaration', { allowArrowFunctions: true }],
+    'id-length': ['error', { exceptions: ['_', 'e', 'i'] }],
+    'id-match': [
+      'error',
+      '^_$|^[$_a-zA-Z]*[_a-zA-Z0-9]*[a-zA-Z0-9]*$|^[A-Z][_A-Z0-9]+[A-Z0-9]$',
+      { onlyDeclarations: true, properties: true }
+    ],
     'max-depth': 'error',
-    'max-nested-callbacks': 'off',
     'max-params': ['error', 4],
     'mocha/no-exclusive-tests': 'error',
     'mocha/no-identical-title': 'error',
     'mocha/no-nested-tests': 'error',
     'mocha/no-sibling-hooks': 'error',
     'new-cap': 'error',
-    'new-parens': 'error',
-    'newline-after-var': 'error',
-    'newline-before-return': 'error',
     'no-alert': 'error',
     'no-array-constructor': 'error',
     'no-bitwise': 'error',
     'no-caller': 'error',
-    'no-catch-shadow': 'off',
     'no-cond-assign': ['error', 'always'],
-    'no-confusing-arrow': 'error',
     'no-div-regex': 'error',
     'no-duplicate-imports': 'error',
     'no-else-return': 'error',
-    'no-empty': 'error',
-    'no-empty-label': 'off',
     'no-eq-null': 'error',
     'no-eval': 'error',
     'no-extend-native': 'error',
     'no-extra-bind': 'error',
-    'no-extra-parens': 'error',
-    'no-floating-decimal': 'error',
     'no-implied-eval': 'error',
     'no-inline-comments': 'error',
     'no-iterator': 'error',
-    'no-label-var': 'off',
     'no-labels': 'error',
     'no-lone-blocks': 'error',
     'no-lonely-if': 'error',
     'no-loop-func': 'error',
     'no-mixed-requires': 'error',
-    'no-multi-spaces': 'error',
     'no-multi-str': 'error',
-    'no-multiple-empty-lines': ['error', {
-      max: 1
-    }],
-    'no-native-reassign': 'error',
     'no-nested-ternary': 'error',
     'no-new': 'error',
     'no-new-func': 'error',
@@ -118,16 +88,11 @@ module.exports = {
     'no-script-url': 'error',
     'no-self-compare': 'error',
     'no-sequences': 'error',
-    'no-shadow': 'off',
-    'no-shadow-restricted-names': 'error',
-    'no-spaced-func': 'error',
     'no-sync': 'error',
+    'no-tabs': ['error', { allowIndentationTabs: true }],
     'no-throw-literal': 'error',
-    'no-trailing-spaces': 'error',
     'no-undef-init': 'error',
-    'no-undefined': 'off',
     'no-underscore-dangle': 'error',
-    'no-unexpected-multiline': 'error',
     'no-unneeded-ternary': 'error',
     'no-unused-expressions': 'error',
     'no-use-before-define': 'error',
@@ -135,60 +100,45 @@ module.exports = {
     'no-useless-concat': 'error',
     'no-var': 'error',
     'no-void': 'error',
-    'no-warning-comments': 'off',
-    'no-with': 'error',
-    'object-curly-spacing': ['error', 'always'],
     'object-shorthand': 'error',
-    'one-var': ['error', 'never'],
-    'one-var-declaration-per-line': ['error', 'always'],
     'operator-assignment': 'error',
-    'operator-linebreak': ['error', 'none'],
-    'padded-blocks': ['error', 'never'],
-    'prefer-arrow-callback': 'error',
+    'padding-line-between-statements': [
+      'error',
+      { blankLine: 'always', next: 'return', prev: '*' },
+      { blankLine: 'always', next: '*', prev: ['const', 'let', 'var'] },
+      { blankLine: 'any', next: ['const', 'let', 'var'], prev: ['const', 'let', 'var'] }
+    ],
     'prefer-const': 'error',
-    'prefer-destructuring': ['error', {
-      AssignmentExpression: {
-        array: false,
-        object: false
+    'prefer-destructuring': [
+      'error',
+      {
+        AssignmentExpression: { array: false, object: false },
+        VariableDeclarator: { array: true, object: true }
       },
-      VariableDeclarator: {
-        array: true,
-        object: true
+      {
+        enforceForRenamedProperties: false
       }
-    }, {
-      enforceForRenamedProperties: false
-    }],
+    ],
     'prefer-spread': 'error',
     'prefer-template': 'error',
-    'quote-props': ['error', 'as-needed'],
-    quotes: ['error', 'single', {
-      allowTemplateLiterals: true
-    }],
+    'prettier/prettier': ['error', { arrowParens: 'avoid', printWidth: 120, singleQuote: true, trailingComma: 'none' }],
     radix: 'error',
+    'require-atomic-updates': 'off',
     'require-await': 'error',
-    'require-yield': 'error',
-    semi: 'error',
-    'semi-spacing': 'error',
-    'sort-imports-es6/sort-imports-es6': ['error', {
-      ignoreCase: false,
-      ignoreMemberSort: false,
-      memberSyntaxSortOrder: ['none', 'all', 'multiple', 'single']
-    }],
-    'sort-keys': ['error', 'asc', {
-      natural: true
-    }],
-    'space-before-blocks': 'error',
-    'space-before-function-paren': ['error', { anonymous: 'never', named: 'never' }],
-    'space-in-parens': 'error',
-    'space-infix-ops': 'error',
-    'space-unary-ops': 'error',
+    'rulesdir/explicit-sinon-use-fake-timers': 'error',
+    'sort-imports-es6/sort-imports-es6': [
+      'error',
+      {
+        ignoreCase: false,
+        ignoreMemberSort: false,
+        memberSyntaxSortOrder: ['none', 'all', 'multiple', 'single']
+      }
+    ],
+    'sort-keys': ['error', 'asc', { natural: true }],
     'spaced-comment': 'error',
     'sql-template/no-unsafe-query': 'error',
-    strict: 'off',
-    'template-curly-spacing': 'error',
     'valid-jsdoc': 'error',
     'vars-on-top': 'error',
-    'wrap-iife': ['error', 'inside'],
     yoda: 'error'
   }
 };

package/src/rules/explicit-sinon-use-fake-timers.js

@@ -0,0 +1,50 @@
+'use strict';
+
+// ------------------------------------------------------------------------------
+// Validates that `sinon.useFakeTimers()` is always called with an explicit `toFake` property.
+// ------------------------------------------------------------------------------
+
+module.exports = {
+  meta: {
+    messages: {
+      avoidName: 'Calls to `sinon.useFakeTimers()` must provide a `toFake` configuration'
+    }
+  },
+  // eslint-disable-next-line sort-keys
+  create(context) {
+    return {
+      CallExpression(node) {
+        if (!node.callee || !node.callee.object || !node.callee.property) {
+          return;
+        }
+
+        if (node.callee.object.name === 'sinon' && node.callee.property.name === 'useFakeTimers') {
+          if (!node.arguments.length) {
+            context.report({
+              message: 'Must pass an object with `toFake` configuration',
+              node
+            });
+          }
+
+          for (const argument of node.arguments) {
+            if (argument.type === 'ObjectExpression') {
+              if (!argument.properties.find(({ key: { name } }) => name === 'toFake')) {
+                context.report({
+                  message: 'Object must contain `toFake` configuration',
+                  node
+                });
+              }
+
+              continue;
+            }
+
+            context.report({
+              message: 'Not an object',
+              node
+            });
+          }
+        }
+      }
+    };
+  }
+};

package/.travis.yml

@@ -1,12 +0,0 @@
-language: node_js
-
-cache: yarn
-
-node_js:
-  - 6
-  - 8
-  - 10
-
-script:
-  - yarn lint
-  - yarn test

package/CHANGELOG.md

@@ -1,21 +0,0 @@
-# Changelog
-
-## [v0.2.0](https://github.com/uphold/eslint-config-uphold/releases/tag/v0.2.0) (2018-07-16)
-- Add @uphold/github-changelog-generator [\#13](https://github.com/uphold/eslint-config-uphold/pull/13) ([Americas](https://github.com/Americas))
-- Replace drone with travis [\#10](https://github.com/uphold/eslint-config-uphold/pull/10) ([Americas](https://github.com/Americas))
-- Add pre-commit lint hook [\#12](https://github.com/uphold/eslint-config-uphold/pull/12) ([Americas](https://github.com/Americas))
-- Raise minimum node engine requirement to 6 [\#11](https://github.com/uphold/eslint-config-uphold/pull/11) ([Americas](https://github.com/Americas))
-- Fix linting errors [\#9](https://github.com/uphold/eslint-config-uphold/pull/9) ([Americas](https://github.com/Americas))
-- Add new mocha rules [\#8](https://github.com/uphold/eslint-config-uphold/pull/8) ([rplopes](https://github.com/rplopes))
-- Update eslint-plugin-mocha@^5.1.0 [\#7](https://github.com/uphold/eslint-config-uphold/pull/7) ([Americas](https://github.com/Americas))
-
-## [v0.1.1](https://github.com/uphold/eslint-config-uphold/releases/tag/v0.1.1) (2018-03-20)
-- Allow computed properties in prefer destructuring [\#5](https://github.com/uphold/eslint-config-uphold/pull/5) ([nunofgs](https://github.com/nunofgs))
-
-## [v0.1.0](https://github.com/uphold/eslint-config-uphold/releases/tag/v0.1.0) (2017-12-26)
-- Add `jasmine` and `jest` to eslint environments [\#4](https://github.com/uphold/eslint-config-uphold/pull/4) ([nunofgs](https://github.com/nunofgs))
-
-## [v0.0.1](https://github.com/uphold/eslint-config-uphold/releases/tag/v0.0.1) (2017-09-07)
-- Add prefer-destructuring rule [\#2](https://github.com/uphold/eslint-config-uphold/pull/2) ([nunofgs](https://github.com/nunofgs))
-- Replace travis with drone [\#3](https://github.com/uphold/eslint-config-uphold/pull/3) ([franciscocardoso](https://github.com/franciscocardoso))
-- Update eslint@4.4.1 [\#1](https://github.com/uphold/eslint-config-uphold/pull/1) ([nunofgs](https://github.com/nunofgs))