Move to jest

Author: geowarin

package.json

@@ -4,16 +4,17 @@
   "description": "",
   "main": "index.js",
   "scripts": {
-    "test": "ava"
+    "test": "jest"
   },
   "files": [
     "src",
     "index.js"
   ],
   "keywords": [
-    "awesome",
-    "react",
-    "cli"
+    "friendly",
+    "errors",
+    "webpack",
+    "plugin"
   ],
   "author": "Geoffroy Warin",
   "repository": {
@@ -26,8 +27,6 @@
   "homepage": "http://geowarin.github.io/friendly-errors-webpack-plugin",
   "license": "MIT",
   "devDependencies": {
-    "assert-diff": "^1.1.0",
-    "ava": "^0.16.0",
     "babel-core": "^6.13.2",
     "babel-eslint": "^6.1.2",
     "babel-loader": "^6.2.5",
@@ -36,6 +35,8 @@
     "eslint": "^3.3.1",
     "eslint-loader": "^1.5.0",
     "expect": "^1.20.2",
+    "jest": "^18.1.0",
+    "memory-fs": "^0.4.1",
     "webpack": "^2.1.0-beta.21"
   },
   "ava": {
@@ -45,7 +46,6 @@
   },
   "dependencies": {
     "chalk": "^1.1.3",
-    "error-stack-parser": "^1.3.6",
-    "memory-fs": "^0.4.1"
+    "error-stack-parser": "^1.3.6"
   }
 }

test/integration.spec.js

@@ -1,7 +1,5 @@
-const test = require('ava');
 const output = require('../src/output');
 const deasync = require('deasync');
-const assert = require('assert-diff');
 const webpack = require('webpack');
 const FriendlyErrorsWebpackPlugin = require('../src/friendly-errors-plugin');
 const MemoryFileSystem = require('memory-fs');
@@ -17,26 +15,23 @@ const syncWebpackWithPlugin = deasync(function(config, fn) {
   return compiler;
 });
 
-test('integration : success', t => {
+it('integration : success', () => {
 
-  var logs = output.captureLogs(() => {
+  const logs = output.captureLogs(() => {
     syncWebpack(require('./fixtures/success/webpack.config'));
   });
 
-  assert(
-    /DONE  Compiled successfully in (.\d*)ms/.test(logs),
-    'Expected logs to include \'DONE  Compiled successfully in {{number}}ms\''
-  );
+  expect(logs.join('\n')).toMatch(/DONE  Compiled successfully in (.\d*)ms/);
 });
 
 
-test('integration : module-errors', t => {
+it('integration : module-errors', () => {
 
-  var logs = output.captureLogs(() => {
+  const logs = output.captureLogs(() => {
     syncWebpack(require('./fixtures/module-errors/webpack.config.js'));
   });
 
-  assert.deepEqual(logs, [
+  expect(logs).toEqual([
     ' ERROR  Failed to compile with 2 errors',
     '',
     'These dependencies were not found in node_modules:',
@@ -48,16 +43,16 @@ test('integration : module-errors', t => {
   ]);
 });
 
-test('integration : should display eslint warnings', t => {
+it('integration : should display eslint warnings', () => {
 
-  var logs = output.captureLogs(() => {
+  const logs = output.captureLogs(() => {
     syncWebpack(require('./fixtures/eslint-warnings/webpack.config.js'));
   });
 
-  assert.deepEqual(logs, [
+  expect(logs).toEqual([
     ' WARNING  Compiled with 1 warnings',
     '',
-    ' warning  in ./fixtures/eslint-warnings/index.js',
+    ' warning  in ./test/fixtures/eslint-warnings/index.js',
     '',
     `${__dirname}/fixtures/eslint-warnings/index.js
   1:7  warning  'unused' is assigned a value but never used  no-unused-vars
@@ -71,16 +66,16 @@ test('integration : should display eslint warnings', t => {
   ]);
 });
 
-test('integration : babel syntax error', t => {
+it('integration : babel syntax error', () => {
 
-  var logs = output.captureLogs(() => {
+  const logs = output.captureLogs(() => {
     syncWebpack(require('./fixtures/babel-syntax/webpack.config'));
   });
 
-  assert.deepEqual(logs, [
+  expect(logs).toEqual([
     ' ERROR  Failed to compile with 1 errors',
     '',
-    ' error  in ./fixtures/babel-syntax/index.js',
+    ' error  in ./test/fixtures/babel-syntax/index.js',
     '',
     `SyntaxError: Unexpected token (5:11)
 
@@ -94,25 +89,22 @@ test('integration : babel syntax error', t => {
   ]);
 });
 
-test('integration : webpack multi compiler : success', t => {
+it('integration : webpack multi compiler : success', () => {
 
-  var logs = output.captureLogs(() => {
+  const logs = output.captureLogs(() => {
     syncWebpackWithPlugin(require('./fixtures/multi-compiler-success/webpack.config'));
   });
 
-  assert(
-    /DONE  Compiled successfully in (.\d*)ms/.test(logs),
-    'Expected logs to include \'DONE  Compiled successfully in {{number}}ms\''
-  );
+  expect(logs.join('\n')).toMatch(/DONE  Compiled successfully in (.\d*)ms/)
 });
 
-test('integration : webpack multi compiler : module-errors', t => {
+it('integration : webpack multi compiler : module-errors', () => {
 
-  var logs = output.captureLogs(() => {
+  const logs = output.captureLogs(() => {
     syncWebpackWithPlugin(require('./fixtures/multi-compiler-module-errors/webpack.config'));
   });
 
-  assert.deepEqual(logs, [
+  expect(logs).toEqual([
     ' ERROR  Failed to compile with 2 errors',
     '',
     'These dependencies were not found in node_modules:',

test/unit/formatErrors.spec.js

@@ -1,6 +1,5 @@
 const formatErrors = require('../../src/core/formatErrors');
 const expect = require('expect');
-const test = require('ava');
 
 const simple = (errors) => errors
   .filter(({ type }) => !type).map(({ message }) => message);
@@ -13,7 +12,7 @@ const notFound = (errors) => errors
 
 const formatters = [allCaps];
 
-test('formats the error based on the matching formatters', () => {
+it('formats the error based on the matching formatters', () => {
   const errors = [
     { message: 'Error 1', type: undefined },
     { message: 'Error 2', type: 'other' },

test/unit/formatters/defaultError.spec.js

@@ -1,12 +1,11 @@
 const defaultError = require('../../../src/formatters/defaultError');
 const expect = require('expect');
-const test = require('ava');
 const chalk = require('chalk');
 
 const noColor = (arr) => arr.map(chalk.stripColor);
 const error = { message: 'Error message', file: './src/index.js' };
 
-test('Formats errors with no type', () => {
+it('Formats errors with no type', () => {
   expect(noColor(defaultError([error], 'Warning'))).toEqual([
     ' Warning  in ./src/index.js',
     '',
@@ -15,7 +14,7 @@ test('Formats errors with no type', () => {
   ]);
 });
 
-test('Does not format other errors', () => {
-  const otherError = { ...error, type: 'other-error' };
+it('Does not format other errors', () => {
+  const otherError = { type: 'other-error' };
   expect(noColor(defaultError([otherError], 'Error'))).toEqual([]);
 });

test/unit/formatters/moduleNotFound.spec.js

@@ -1,8 +1,7 @@
 const moduleNotFound = require('../../../src/formatters/moduleNotFound');
 const expect = require('expect');
-const test = require('ava');
 
-test('Formats module-not-found errors', () => {
+it('Formats module-not-found errors', () => {
   const error = { type: 'module-not-found', module: 'redux' };
   expect(moduleNotFound([error])).toEqual([
     'This dependency was not found in node_modules:',
@@ -13,7 +12,7 @@ test('Formats module-not-found errors', () => {
   ]);
 });
 
-test('Groups all module-not-found into one', () => {
+it('Groups all module-not-found into one', () => {
   const reduxError = { type: 'module-not-found', module: 'redux' };
   const reactError = { type: 'module-not-found', module: 'react' };
   expect(moduleNotFound([reduxError, reactError])).toEqual([
@@ -26,7 +25,7 @@ test('Groups all module-not-found into one', () => {
   ]);
 });
 
-test('Does not format other errors', () => {
+it('Does not format other errors', () => {
   const otherError = { type: 'other-error', module: 'foo' };
   expect(moduleNotFound([otherError])).toEqual([]);
 });

test/unit/plugin/friendlyErrors.spec.js

@@ -1,52 +1,49 @@
 const expect = require('expect');
-const test = require('ava');
 const EventEmitter = require('events');
 const Stats = require('webpack/lib/Stats');
 const Module = require('webpack/lib/Module');
 EventEmitter.prototype.plugin = EventEmitter.prototype.on;
-var assert = require('assert-diff');
 
 const output = require("../../../src/output");
 const FriendlyErrorsPlugin = require("../../../index");
 
 const notifierPlugin = new FriendlyErrorsPlugin();
-var mockCompiler = new EventEmitter();
+const mockCompiler = new EventEmitter();
 notifierPlugin.apply(mockCompiler);
 
-test('friendlyErrors : capture invalid message', t => {
+it('friendlyErrors : capture invalid message', () => {
 
-  var logs = output.captureLogs(() => {
+  const logs = output.captureLogs(() => {
     mockCompiler.emit('invalid');
   });
 
-  assert.deepEqual(logs, [
+  expect(logs).toEqual([
     ' WAIT  Compiling...',
     ''
-    ]
-  );
+    ]);
 });
 
-test('friendlyErrors : capture compilation without errors', t => {
+it('friendlyErrors : capture compilation without errors', () => {
 
-  var stats = successfulCompilationStats();
-  var logs = output.captureLogs(() => {
+  const stats = successfulCompilationStats();
+  const logs = output.captureLogs(() => {
     mockCompiler.emit('done', stats);
   });
 
-  assert.deepEqual(logs, [
+  expect(logs).toEqual([
     ' DONE  Compiled successfully in 100ms',
     ''
   ]);
 });
 
-test('friendlyErrors : default clearConsole option', t => {
+it('friendlyErrors : default clearConsole option', () => {
   const plugin = new FriendlyErrorsPlugin();
-  assert.strictEqual(plugin.shouldClearConsole, true)
+  expect(plugin.shouldClearConsole).toBeTruthy()
 });
 
-test('friendlyErrors : clearConsole option', t => {
+it('friendlyErrors : clearConsole option', () => {
   const plugin = new FriendlyErrorsPlugin({ clearConsole: false });
-  assert.strictEqual(plugin.shouldClearConsole, false)
+  expect(plugin.shouldClearConsole).toBeFalsy()
 });
 
 function successfulCompilationStats() {

test/unit/transformers/babelSyntax.spec.js

@@ -1,18 +1,17 @@
 const babelSyntax = require('../../../src/transformers/babelSyntax');
 const expect = require('expect');
-const test = require('ava');
 
-test('Sets severity to 1000', () => {
+it('Sets severity to 1000', () => {
   const error = { name: 'ModuleBuildError', message: 'SyntaxError' };
   expect(babelSyntax(error).severity).toEqual(1000);
 });
 
-test('Does not set type (it should be handle by the default formatter)', () => {
+it('Does not set type (it should be handle by the default formatter)', () => {
   const error = { name: 'ModuleBuildError', message: 'SyntaxError' };
   expect(babelSyntax(error).type).toEqual(undefined);
 });
 
-test('Ignores other errors', () => {
+it('Ignores other errors', () => {
   const error = { name: 'OtherError' };
   expect(babelSyntax(error)).toEqual(error);
 });

test/unit/transformers/moduleNotFound.spec.js

@@ -1,6 +1,5 @@
 const moduleNotFound = require('../../../src/transformers/moduleNotFound');
 const expect = require('expect');
-const test = require('ava');
 
 const error = {
   name: 'ModuleNotFoundError',
@@ -10,20 +9,20 @@ const error = {
   },
 };
 
-test('Sets severity to 900', () => {
+it('Sets severity to 900', () => {
   expect(moduleNotFound(error).severity).toEqual(900);
 });
 
-test('Sets module name', () => {
+it('Sets module name', () => {
   expect(moduleNotFound(error).module).toEqual('redux');
 });
 
-test('Sets the appropiate message', () => {
+it('Sets the appropiate message', () => {
   const message = 'Module not found redux';
   expect(moduleNotFound(error).message).toEqual(message);
 });
 
-test('Sets the appropiate message', () => {
+it('Sets the appropiate message', () => {
   const message = 'Module not found redux';
   expect(moduleNotFound({
     name: 'ModuleNotFoundError',
@@ -32,7 +31,7 @@ test('Sets the appropiate message', () => {
   }).type).toEqual('module-not-found');
 });
 
-test('Ignores other errors', () => {
+it('Ignores other errors', () => {
   const error = { name: 'OtherError' };
   expect(moduleNotFound(error)).toEqual(error);
 });

test/unit/utils/utils.spec.js

@@ -1,27 +1,25 @@
 const utils = require('../../../src/utils');
-const test = require('ava');
-const assert = require('assert-diff');
 
 const concat = utils.concat;
 const uniqueBy = utils.uniqueBy;
 
-test('concat should concat removing undefined and null values', () => {
+it('concat should concat removing undefined and null values', () => {
   const result = concat(1, undefined, '', null);
-  assert.deepEqual(result,
+  expect(result).toEqual(
     [1, '']
   );
 });
 
-test('concat should handle arrays', () => {
+it('concat should handle arrays', () => {
   const result = concat(1, [2, 3], null);
-  assert.deepEqual(result,
+  expect(result).toEqual(
     [1, 2, 3]
   );
 });
 
-test('uniqueBy should dedupe based on criterion returned from iteratee function', () => {
+it('uniqueBy should dedupe based on criterion returned from iteratee function', () => {
   const result = uniqueBy([{ id: 1 }, { id: 2 }, { id: 3 }, { id: 3 }], function(val) {
     return val.id;
   });
-  assert.deepEqual(result, [{ id: 1 }, { id: 2 }, { id: 3 }]);
+  expect(result).toEqual([{ id: 1 }, { id: 2 }, { id: 3 }]);
 });