refactor: move jest around and add code coverage

Author: jsjoeio

.gitignore

@@ -14,3 +14,4 @@ node-*
 /plugins
 /lib/coder-cloud-agent
 .home
+coverage

ci/dev/test.sh

@@ -3,12 +3,17 @@ set -euo pipefail
 
 main() {
   cd "$(dirname "$0")/../.."
-
-  cd test/test-plugin
-  make -s out/index.js
-  cd "$OLDPWD/test"
+  cd test
+  # We need to make sure the dependencies are installed
+  # because we run jest from node_modules
   yarn
-  yarn test "$@"
+  cd test-plugin
+  make -s out/index.js
+  # We must keep jest in a sub-directory. See ../../test/package.json for more
+  # information. We must also run it from the root otherwise coverage will not
+  # include our source files.
+  cd "$OLDPWD"
+  ./test/node_modules/.bin/jest "$@"
 }
 
 main "$@"

package.json

@@ -37,7 +37,6 @@
     "@types/http-proxy": "^1.17.4",
     "@types/js-yaml": "^3.12.3",
     "@types/node": "^12.12.7",
-    "@types/node-fetch": "^2.5.7",
     "@types/parcel-bundler": "^1.12.1",
     "@types/pem": "^1.9.5",
     "@types/proxy-from-env": "^1.0.1",
@@ -107,5 +106,28 @@
   ],
   "engines": {
     "node": ">= 12"
+  },
+  "jest": {
+    "transform": {
+      "^.+\\.ts$": "<rootDir>/test/node_modules/ts-jest"
+    },
+    "testEnvironment": "node",
+    "testPathIgnorePatterns": [
+      "/node_modules/",
+      "<rootDir>/lib/vscode",
+      "<rootDir>/out"
+    ],
+    "collectCoverage": true,
+    "collectCoverageFrom": [
+      "<rootDir>/src/**/*.ts"
+    ],
+    "coverageDirectory": "<rootDir>/coverage",
+    "coverageReporters": [
+      "json",
+      "text"
+    ],
+    "coveragePathIgnorePatterns": [
+      "out"
+    ]
   }
 }

test/jest.config.js

@@ -1,22 +1,13 @@
 {
-  "name": "test",
-  "version": "1.0.0",
-  "description": "",
-  "main": "index.js",
-  "scripts": {
-    "test": "jest"
-  },
-  "keywords": [],
-  "author": "",
-  "license": "ISC",
+  "#": "We must put jest in a sub-directory otherwise VS Code somehow picks up",
+  "#": "the types and generates conflicts with mocha.",
   "devDependencies": {
     "@types/jest": "^26.0.20",
     "@types/node-fetch": "^2.5.8",
     "@types/supertest": "^2.0.10",
     "jest": "^26.6.3",
     "node-fetch": "^2.6.1",
     "supertest": "^6.1.1",
-    "ts-jest": "^26.4.4",
-    "typescript": "^4.1.3"
+    "ts-jest": "^26.4.4"
   }
 }

test/package.json

@@ -1,10 +1,4 @@
 {
   "extends": "../tsconfig.json",
-  "compilerOptions": {
-    "typeRoots": ["node_modules/@types", "../typings"],
-    "composite": true
-  },
-  "include": ["./**/*.ts"],
-  "exclude": ["node_modules/**"],
-  "types": ["jest"]
+  "include": ["./**/*.ts"]
 }

test/tsconfig.json

@@ -3523,11 +3523,6 @@ typedarray-to-buffer@^3.1.5:
   dependencies:
     is-typedarray "^1.0.0"
 
-typescript@^4.1.3:
-  version "4.1.3"
-  resolved "https://registry.yarnpkg.com/typescript/-/typescript-4.1.3.tgz#519d582bd94cba0cf8934c7d8e8467e473f53bb7"
-  integrity sha512-B3ZIOf1IKeH2ixgHhj6la6xdwR9QrLC5d1VKeCSY4tvkqhF2eqd9O7txNlS0PO3GrBAFIdr3L1ndNwteUbZLYg==
-
 union-value@^1.0.0:
   version "1.0.1"
   resolved "https://registry.yarnpkg.com/union-value/-/union-value-1.0.1.tgz#0b6fe7b835aecda61c6ea4d4f02c14221e109847"

test/yarn.lock

@@ -15,7 +15,7 @@
     "sourceMap": true,
     "tsBuildInfoFile": "./.cache/tsbuildinfo",
     "incremental": true,
-    "typeRoots": ["./node_modules/@types", "./typings"],
+    "typeRoots": ["./node_modules/@types", "./typings", "./test/node_modules/@types"],
     "downlevelIteration": true
   },
   "include": ["./src/**/*.ts"],