refactor(tests): convert to TypeScript

Closes #293.
Fixes #509.

.ecrc

@@ -6,6 +6,7 @@
     "^es/",
     "^helpers/",
     "^modules/",
+    "^test/",
     "^h.d.ts",
     "^h.js",
     "^h.js.map",

.gitignore

@@ -33,6 +33,7 @@
 /es/modules/style.d.ts
 /es/modules/style.js
 /es/modules/style.js.map
+/es/test/
 /es/h.d.ts
 /es/h.js
 /es/h.js.map
@@ -92,6 +93,7 @@
 /modules/style.d.ts
 /modules/style.js
 /modules/style.js.map
+/test/
 /h.d.ts
 /h.js
 /h.js.map

karma.conf.js

@@ -19,7 +19,7 @@ module.exports = function(config) {
     basePath: '.',
     frameworks: ['mocha', 'karma-typescript'],
     // list of files / patterns to load in the browser
-    files: [{pattern: 'src/**/*.ts'}, {pattern: 'test/**/*'}],
+    files: [{pattern: 'src/**/*.ts'}],
     plugins: [
       'karma-mocha',
       'karma-chrome-launcher',
@@ -29,8 +29,7 @@ module.exports = function(config) {
     ],
     hostname: ci ? ip : 'localhost',
     preprocessors: {
-      'src/**/*.ts': ['karma-typescript'],
-      'test/**/*.{js,ts,tsx}': ['karma-typescript'],
+      'src/**/*.{ts,tsx}': ['karma-typescript']
     },
     browserStack: {
       name: 'Snabbdom',
@@ -43,16 +42,12 @@ module.exports = function(config) {
     customLaunchers: browserstack,
     karmaTypescriptConfig: {
       coverageOptions: {
-        exclude: /test\//,
-      },
-      compilerOptions: {
-        allowJs: true,
-        declaration: false
+        exclude: /^src{\/|\\}test{\/|\\}/,
       },
       tsconfig: './tsconfig.json',
       include: {
         mode: 'merge',
-        values: ['test/**/*'],
+        values: ['src/test/**/*'],
       },
     },
     reporters: ['dots', 'karma-typescript', 'BrowserStack'],

package.json

@@ -11,6 +11,7 @@
   },
   "devDependencies": {
     "@types/assert": "^1.4.3",
+    "@types/lodash.shuffle": "^4.2.6",
     "@types/mocha": "^5.2.7",
     "@typescript-eslint/eslint-plugin": "^2.7.0",
     "benchmark": "^2.1.4",
@@ -22,7 +23,6 @@
     "eslint-plugin-node": "^10.0.0",
     "eslint-plugin-promise": "^4.2.1",
     "eslint-plugin-standard": "^4.0.1",
-    "fake-raf": "1.0.1",
     "gulp": "^3.9.1",
     "gulp-clean": "^0.3.2",
     "gulp-rename": "^1.2.2",
@@ -35,7 +35,7 @@
     "karma-firefox-launcher": "^1.2.0",
     "karma-mocha": "^1.3.0",
     "karma-typescript": "^3.0.13",
-    "knuth-shuffle": "^1.0.1",
+    "lodash.shuffle": "^4.2.0",
     "mocha": "^5.2.0",
     "npm-run-all": "^4.1.5",
     "remark-cli": "^7.0.1",

test/attachto.js → src/test/attachto.ts

@@ -1,12 +1,13 @@
-var assert = require('assert');
-var snabbdom = require('../snabbdom');
+import assert from 'assert'
+import { init } from '../snabbdom'
+import { RemoveHook } from '../hooks';
 
-var patch = snabbdom.init([]);
-var attachTo = require('../helpers/attachto').default;
-var h = require('../h').default;
+var patch = init([]);
+import attachTo from '../helpers/attachto'
+import h from '../h'
 
 describe('attachTo', function() {
-  var elm, vnode0;
+  var elm: any, vnode0: any;
   beforeEach(function() {
     elm = document.createElement('div');
     vnode0 = elm;
@@ -76,9 +77,10 @@ describe('attachTo', function() {
     assert.equal(elm.children.length, 1);
   });
   it('remove hook receives real element', function() {
-    function rm(vnode, cb) {
-      assert.equal(vnode.elm.tagName, 'DIV');
-      assert.equal(vnode.elm.innerHTML, 'First text');
+    const rm: RemoveHook = (vnode, cb) => {
+      const elm = vnode.elm as HTMLDivElement;
+      assert.equal(elm.tagName, 'DIV');
+      assert.equal(elm.innerHTML, 'First text');
       cb();
     }
     var vnode1 = h('div', [

test/attributes.js → src/test/attributes.ts

@@ -1,13 +1,13 @@
-var assert = require('assert');
-
-var snabbdom = require('../snabbdom');
-var patch = snabbdom.init([
-  require('../modules/attributes').default,
+import assert from 'assert'
+import { init } from '../snabbdom'
+import attributesModule from '../modules/attributes'
+var patch = init([
+  attributesModule
 ]);
-var h = require('../h').default;
+import h from '../h'
 
 describe('attributes', function() {
-  var elm, vnode0;
+  var elm: any, vnode0: any;
   beforeEach(function() {
     elm = document.createElement('div');
     vnode0 = elm;
@@ -35,7 +35,7 @@ describe('attributes', function() {
     assert.strictEqual(elm.getAttribute('selected'), '');
   });
   it('are not omitted when falsy values are provided', function() {
-    var vnode1 = h('div', {attrs: {href: null, minlength: 0, value: '', title:'undefined'}});
+    var vnode1 = h('div', {attrs: {href: null as any, minlength: 0, value: '', title:'undefined'}});
     elm = patch(vnode0, vnode1).elm;
     assert.strictEqual(elm.getAttribute('href'), 'null');
     assert.strictEqual(elm.getAttribute('minlength'), '0');
@@ -77,7 +77,7 @@ describe('attributes', function() {
       assert.strictEqual(elm.getAttribute('required'), null);
     });
     it('is not omitted if the value is falsy but casted to string', function() {
-      var vnode1 = h('div', {attrs: {readonly: 0, noresize: null}});
+      var vnode1 = h('div', {attrs: {readonly: 0, noresize: null as any}});
       elm = patch(vnode0, vnode1).elm;
       assert.strictEqual(elm.getAttribute('readonly'), '0');
       assert.strictEqual(elm.getAttribute('noresize'), 'null');