Upgrade dev dependencies (#245)

* Upgrade dev dependencies

* Reformat code

* Bump node version to 12

Author: skovhus

.circleci/config.yml

@@ -2,7 +2,7 @@ version: 2
 
 defaults: &defaults
   docker:
-    - image: circleci/node:10
+    - image: cimg/node:12.22.12
 
 _steps:
   save_npm_cache: &save_npm_cache

.eslintrc.js

@@ -6,24 +6,14 @@ module.exports = {
     'plugin:jest/recommended',
     'plugin:@typescript-eslint/recommended',
     'prettier',
-    'prettier/@typescript-eslint',
   ],
   env: {
     node: true,
     es6: true,
   },
   root: true,
   rules: {
-    'prettier/prettier': [
-      'error',
-      {
-        trailingComma: 'es5',
-        singleQuote: true,
-        printWidth: 90,
-        semi: false,
-        tabWidth: 2,
-      },
-    ],
+    'prettier/prettier': 'error',
     '@typescript-eslint/no-unused-vars': [
       'error',
       {
@@ -43,14 +33,16 @@ module.exports = {
     ],
     'prefer-const': 'error',
     'prefer-template': 'error',
-    'simple-import-sort/sort': 'error',
+    'simple-import-sort/imports': 'error',
+    'simple-import-sort/exports': 'error',
 
     'no-underscore-dangle': 'off',
     'no-param-reassign': 'off',
     'no-console': 'off',
     'no-warning-comments': ['warn', { terms: ['fixme'], location: 'start' }],
     'no-unused-vars': 'off', // replaced by @typescript-eslint/no-unused-vars
     '@typescript-eslint/no-empty-function': 'off',
+    'jest/valid-title': 'off',
 
     // FIXME: enable
     '@typescript-eslint/explicit-function-return-type': 'off',

.husky/.gitignore

@@ -0,0 +1 @@
+_

.husky/pre-commit

@@ -0,0 +1,4 @@
+#!/bin/sh
+. "$(dirname "$0")/_/husky.sh"
+
+yarn lint-staged

.prettierrc.js

@@ -0,0 +1,7 @@
+module.exports = {
+  trailingComma: 'es5',
+  singleQuote: true,
+  printWidth: 90,
+  semi: false,
+  tabWidth: 2,
+}

package.json

@@ -11,7 +11,7 @@
   },
   "bin": "bin/jest-codemods.js",
   "engines": {
-    "node": ">=10"
+    "node": ">=12"
   },
   "files": [
     "bin",
@@ -28,12 +28,8 @@
     "test": "jest --runInBand",
     "verify": "yarn lint && yarn build && yarn test:cov",
     "verify:bail": "yarn lint:bail && yarn verify",
-    "ci": "yarn verify:bail && codecov"
-  },
-  "husky": {
-    "hooks": {
-      "pre-commit": "lint-staged"
-    }
+    "ci": "yarn verify:bail && codecov",
+    "prepare": "husky install"
   },
   "lint-staged": {
     "*.js": [
@@ -69,23 +65,23 @@
     "update-notifier": "^4.1.0"
   },
   "devDependencies": {
-    "@types/jest": "^25.2.3",
-    "@types/jscodeshift": "^0.7.1",
-    "@types/update-notifier": "^4.1.0",
-    "@typescript-eslint/eslint-plugin": "^3.1.0",
-    "@typescript-eslint/parser": "^3.1.0",
-    "codecov": "^3.7.0",
-    "eslint": "^7.1.0",
-    "eslint-config-prettier": "^6.11.0",
-    "eslint-plugin-jest": "^23.13.2",
-    "eslint-plugin-prettier": "^3.1.3",
-    "eslint-plugin-simple-import-sort": "^5.0.3",
-    "husky": "^4.2.5",
-    "jest": "^26.0.1",
-    "lint-staged": "^10.2.7",
-    "prettier": "^2.0.5",
-    "ts-jest": "^26.1.0",
-    "typescript": "^3.9.3"
+    "@types/jest": "^27.4.1",
+    "@types/jscodeshift": "^0.11.4",
+    "@types/update-notifier": "^5.1.0",
+    "@typescript-eslint/eslint-plugin": "^5.20.0",
+    "@typescript-eslint/parser": "^5.20.0",
+    "codecov": "^3.8.3",
+    "eslint": "^8.13.0",
+    "eslint-config-prettier": "^8.5.0",
+    "eslint-plugin-jest": "^26.1.4",
+    "eslint-plugin-prettier": "^4.0.0",
+    "eslint-plugin-simple-import-sort": "^7.0.0",
+    "husky": "^7.0.4",
+    "jest": "^27.5.1",
+    "lint-staged": "^12.4.0",
+    "prettier": "^2.6.2",
+    "ts-jest": "^27.1.4",
+    "typescript": "^4.6.3"
   },
   "jest": {
     "preset": "ts-jest",

src/transformers/jasmine-this.ts

@@ -1,7 +1,7 @@
 /**
  * Codemod for transforming Jasmine `this` context into Jest v20+ compatible syntax.
  */
-import { NodePath } from 'ast-types'
+import type { NodePath } from 'ast-types/lib/node-path'
 import * as jscodeshift from 'jscodeshift'
 import { Collection } from 'jscodeshift/src/Collection'
 

src/transformers/tape.test.ts

@@ -228,7 +228,7 @@ test(function() {
   )
 })
 
-test('test options: removes', () => {
+test('options: removes', () => {
   expectTransformation(
     `
 import test from 'tape';
@@ -244,7 +244,7 @@ test('mytest', () => {
   )
 })
 
-test('test options: skip', () => {
+test('options: skip', () => {
   expectTransformation(
     `
 import test from 'tape';

src/utils/chai-chain-utils.ts

@@ -1,23 +1,19 @@
 import { JEST_MATCHER_TO_MAX_ARGS } from './consts'
 
-export const createCallUtil = (j) => (
-  fnName: string,
-  args: any,
-  rest: any,
-  containsNot?: boolean
-) => {
-  const expression = containsNot ? j.memberExpression(rest, j.identifier('not')) : rest
-
-  const numberOfArgs = JEST_MATCHER_TO_MAX_ARGS[fnName]
-  if (typeof numberOfArgs === 'undefined') {
-    throw new Error(`Unknown matcher "${fnName}" (JEST_MATCHER_TO_MAX_ARGS)`)
-  }
+export const createCallUtil =
+  (j) => (fnName: string, args: any, rest: any, containsNot?: boolean) => {
+    const expression = containsNot ? j.memberExpression(rest, j.identifier('not')) : rest
 
-  return j.memberExpression(
-    expression,
-    j.callExpression(j.identifier(fnName), args.slice(0, numberOfArgs))
-  )
-}
+    const numberOfArgs = JEST_MATCHER_TO_MAX_ARGS[fnName]
+    if (typeof numberOfArgs === 'undefined') {
+      throw new Error(`Unknown matcher "${fnName}" (JEST_MATCHER_TO_MAX_ARGS)`)
+    }
+
+    return j.memberExpression(
+      expression,
+      j.callExpression(j.identifier(fnName), args.slice(0, numberOfArgs))
+    )
+  }
 
 export const chainContainsUtil = (j) => (fnName, node, end) => {
   let curr = node