Merge pull request #7 from z-dev/node-15-modernization

Node 15 modernization

Author: richardgill

.circleci/config.yml

@@ -21,7 +21,7 @@ ecmaFeatures:
 
 env:
   browser: true
-  mocha: true
+  jest: true
 
 rules:
   arrow-body-style: 0

.eslintrc

@@ -0,0 +1,21 @@
+# This workflow will do a clean install of node dependencies, build the source code and run tests across different versions of node
+# For more information see: https://help.github.com/actions/language-and-framework-guides/using-nodejs-with-github-actions
+
+name: Node.js CI
+
+on: [pull_request, push]
+
+jobs:
+  build:
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        node-version: [15.x]
+    steps:
+      - uses: actions/checkout@v2
+      - name: Use Node.js ${{ matrix.node-version }}
+        uses: actions/setup-node@v1
+        with:
+          node-version: ${{ matrix.node-version }}
+      - run: npm install -g yarn
+      - run: yarn install && yarn ci

.github/workflows/node.js.yml

@@ -38,3 +38,5 @@ dist
 build
 
 .DS_Store
+
+.eslintcache

.gitignore

@@ -6,7 +6,6 @@
   "scripts": {
     "check:node": "check-node-version --package",
     "ci": "yarn run lint:js && yarn run test",
-    "precommit": "lint-staged && yarn run lint",
     "eslint-check": "eslint --print-config .eslintrc | eslint-config-prettier-check",
     "format:js": "prettier --write \"{src,test}/**/*.js\"",
     "postinstall": "yarn run check:node",
@@ -15,46 +14,47 @@
     "lint:js:without:prettier": "yarn run lint:js -- --rule 'prettier/prettier: 0'",
     "lint:watch": "nodemon --watch src --exec \"yarn run lint:without:prettier\"",
     "lint:without:prettier": "yarn run lint:js:without:prettier",
-    "start": "node --es-module-specifier-resolution=node --experimental-loader ./customLoader.mjs src/index.js",
+    "start": "node --es-module-specifier-resolution=node src/index.js",
     "start:built": "node build/index.js",
-    "start:debug": "node inspect --es-module-specifier-resolution=node --experimental-loader ./customLoader.mjs src/index.js",
-    "start:docker": "docker build -t lead-finder-server . && docker run --rm -it lead-finder-server",
-    "test": "NODE_OPTIONS=\"--experimental-specifier-resolution node --experimental-loader ./customLoader.mjs\" node_modules/.bin/mocha test/index.cjs --delay",
+    "start:debug": "node inspect --es-module-specifier-resolution=node src/index.js",
+    "test": "node --experimental-vm-modules node_modules/.bin/jest",
     "test:watch": "nodemon -w src -w test --exec 'yarn run test'"
   },
   "dependencies": {
     "lodash": "^4.17.12"
   },
   "devDependencies": {
     "babel-eslint": "^7.2.3",
-    "chai": "^4.0.1",
-    "chai-moment": "^0.1.0",
     "check-node-version": "^2.1.0",
     "eslint": "^4.18.2",
     "eslint-config-airbnb-base": "^11.2.0",
     "eslint-config-prettier": "^2.1.0",
     "eslint-plugin-import": "2.2.0",
     "eslint-plugin-prettier": "^2.0.1",
-    "husky": "^1.0.1",
-    "lint-staged": "^7.3.0",
-    "mocha": "^5.2.0",
+    "husky": "^4.3.7",
+    "jest": "^26.6.3",
+    "lint-staged": "^10.5.3",
     "nodemon": "^1.19.4",
     "prettier": "^1.2.2",
     "sort-package-json": "^1.15.0"
   },
   "engines": {
-    "node": ">= 13.4 < 14"
+    "node": ">= 15.5 < 16"
   },
   "engine-strict": true,
+  "husky": {
+    "hooks": {
+      "pre-commit": "lint-staged"
+    }
+  },
   "lint-staged": {
     "{src,test}/**/*.js": [
-      "prettier --write",
-      "git add"
+      "prettier --write"
     ],
     "package.json": [
-      "sort-package-json",
-      "git add"
-    ]
+      "sort-package-json"
+    ],
+    "*.js": "eslint --cache --fix"
   },
   "type": "module"
 }

customLoader.mjs

@@ -1,5 +1,5 @@
 import _ from 'lodash'
-import defaults from '~/config/defaults'
+import defaults from './config/defaults'
 
 console.log(defaults)
 

package.json

@@ -1,11 +1,11 @@
-import assert from 'assert'
-import defaults from '~/config/defaults'
+import defaults from '../src/config/defaults'
 
 console.log(defaults)
+
 describe('Array', () => {
   describe('#indexOf()', () => {
     it('should return -1 when the value is not present', () => {
-      assert.equal([1, 2, 3].indexOf(4), -1)
+      expect([1, 2, 3].indexOf(4)).toBe(-1)
     })
   })
 })