Move tests to typescript (#191)

Author: bryanmikaelian

.circleci/config.yml

@@ -33,7 +33,7 @@ jobs:
           paths:
             - node_modules
       - run: yarn run tsc --declaration
-      - run: yarn run karma start karma.conf.coverage.js
+      - run: yarn run karma start karma.conf.coverage.ts
       - store_test_results:
           path: junit-reports
       - run: yarn run codecov

Makefile

@@ -9,8 +9,8 @@ KARMA := node_modules/.bin/karma
 ##
 
 LIBS = $(shell find lib -type f -name "*.js")
-TESTS = $(shell find test -type f -name "*.test.js")
-SUPPORT = $(wildcard karma.conf*.js)
+TESTS = $(shell find test -type f -name "*.test.ts")
+SUPPORT = $(wildcard karma.conf*.ts)
 ALL_FILES = $(LIBS) $(TESTS) $(SUPPORT)
 
 ##
@@ -29,9 +29,9 @@ KARMA_FLAGS += --browsers $(BROWSERS)
 endif
 
 ifdef CI
-KARMA_CONF ?= karma.conf.ci.js
+KARMA_CONF ?= karma.conf.ci.ts
 else
-KARMA_CONF ?= karma.conf.js
+KARMA_CONF ?= karma.conf.ts
 endif
 
 # Mocha flags.

karma.conf.ci.js renamed to karma.conf.ci.ts

@@ -5,27 +5,22 @@
 var TEST_TIMEOUT = 10 * 60 * 1000;
 
 module.exports = function(config) {
-  require('ts-node').register({
-    compilerOptions: {
-      module: 'commonjs'
-    }
-  });
   config.set({
     files: [
       { pattern: 'test/support/*.html', included: false },
-      'test/support/global.js', // NOTE: This must run before all tests
-      'test/**/*.test.js'
+      'test/support/global.ts', // NOTE: This must run before all tests
+      'test/**/*.test.ts'
     ],
     browsers: ['ChromeHeadless'],
 
     singleRun: true,
 
-    frameworks: ['browserify', 'mocha'],
+    frameworks: ['mocha', 'karma-typescript'],
 
     reporters: ['spec'],
 
     preprocessors: {
-      'test/**/*.js': 'browserify'
+      'test/**/*.ts': 'karma-typescript'
     },
 
     browserNoActivityTimeout: TEST_TIMEOUT,
@@ -39,6 +34,19 @@ module.exports = function(config) {
 
     browserify: {
       debug: true
+    },
+
+    karmaTypescriptConfig: {
+      bundlerOptions: {
+        sourceMap: true,
+      },
+      compilerOptions: {
+        module: "commonjs",
+        target: "ES5",
+        allowJs: false,
+      },
+      include: ['test'],
+      exclude: ['node_modules', 'lib', 'test-e2e/*.ts']
     }
   });
 };

karma.conf.coverage.js renamed to karma.conf.coverage.ts

@@ -71,8 +71,10 @@
     "@types/mocha": "^7.0.2",
     "@types/node": "^14.0.6",
     "@types/node-fetch": "^2.5.7",
+    "assert": "1.5.0",
     "browserify": "13.0.0",
     "browserify-istanbul": "^3.0.1",
+    "buffer": "^4.9.2",
     "codeceptjs": "^2.6.5",
     "codecov": "^3.7.2",
     "compat-trigger-event": "^1.0.0",
@@ -96,13 +98,14 @@
     "karma-sauce-launcher": "^1.2.0",
     "karma-spec-reporter": "0.0.32",
     "karma-summary-reporter": "^1.8.0",
+    "karma-typescript": "^5.1.0",
     "lint-staged": "^7.2.0",
     "lodash": "^4.17.15",
     "mocha": "^2.2.5",
     "node-fetch": "^2.6.0",
     "np": "^3.0.4",
     "prettier-eslint-cli": "^4.7.1",
-    "proclaim": "^3.4.1",
+    "proclaim": "^3.5.1",
     "puppeteer": "^3.3.0",
     "sinon": "^1.7.3",
     "snyk": "^1.83.0",
@@ -123,5 +126,9 @@
     "transform": [
       "package-json-versionify"
     ]
+  },
+  "resolutions": {
+    "buffer": "^4.9.2",
+    "assert": "1.5.0"
   }
 }

karma.conf.js renamed to karma.conf.ts

@@ -35,7 +35,7 @@ describe('clone', function() {
       assert.notStrictEqual(cloned.a, input.a);
       assert.notStrictEqual(cloned.a.b, input.a.b);
       assert.notStrictEqual(cloned.a.b[2], input.a.b[2]);
-      assert.strictEqual(cloned.a.b[2].getTime(), input.a.b[2].getTime());
+      assert.strictEqual(cloned.a.b[2].getTime(), (input.a.b[2] as Date).getTime());
       assert.deepEqual(cloned.a.b[3], input.a.b[3]);
       assert.notStrictEqual(cloned.a.b[3], input.a.b[3]);
     });