test: rm or fix flaky tests

Author: ava

@commitlint/core/package.json

@@ -63,7 +63,6 @@
   "license": "MIT",
   "devDependencies": {
     "@commitlint/utils": "^3.1.1",
-    "ansi-styles": "3.1.0",
     "ava": "0.22.0",
     "babel-cli": "^6.26.0",
     "babel-preset-commitlint": "^3.2.0",
@@ -73,7 +72,6 @@
     "dependency-check": "2.7.0",
     "execa": "0.6.3",
     "globby": "6.1.0",
-    "has-ansi": "3.0.0",
     "import-from": "2.1.0",
     "nyc": "10.3.2",
     "path-exists": "3.0.0",

@commitlint/core/src/format.test.js

@@ -1,7 +1,5 @@
 import test from 'ava';
-import hasAnsi from 'has-ansi';
 import chalk from 'chalk';
-import {yellow, red, magenta, blue} from 'ansi-styles';
 import {includes} from 'lodash';
 import format from './format';
 
@@ -49,19 +47,6 @@ test('returns a correct of empty .errors and .warnings', t => {
 	t.true(includes(msg, '1 problems, 1 warnings'));
 });
 
-test('colors messages by default', t => {
-	const [msg] = format({
-		errors: [],
-		warnings: []
-	});
-	t.true(hasAnsi(msg));
-});
-
-test('does not color messages if configured', t => {
-	const [msg] = format({}, {color: false});
-	t.false(hasAnsi(msg));
-});
-
 test('uses appropriate signs by default', t => {
 	const [err, warn] = format({
 		errors: [
@@ -110,54 +95,3 @@ test('uses signs as configured', t => {
 	t.true(includes(err, 'ERR'));
 	t.true(includes(warn, 'WRN'));
 });
-
-test('uses appropriate colors by default', t => {
-	const [err, warn] = format({
-		errors: [
-			{
-				level: 2,
-				name: 'error-name',
-				message: 'There was an error'
-			}
-		],
-		warnings: [
-			{
-				level: 1,
-				name: 'warning-name',
-				message: 'There was a problem'
-			}
-		]
-	});
-
-	t.true(includes(err, red.open));
-	t.true(includes(warn, yellow.open));
-});
-
-if (process.platform !== 'win32') {
-	test('uses colors as configured', t => {
-		const [err, warn] = format(
-			{
-				errors: [
-					{
-						level: 2,
-						name: 'error-name',
-						message: 'There was an error'
-					}
-				],
-				warnings: [
-					{
-						level: 1,
-						name: 'warning-name',
-						message: 'There was a problem'
-					}
-				]
-			},
-			{
-				colors: ['white', 'magenta', 'blue']
-			}
-		);
-
-		t.true(includes(err, blue.open));
-		t.true(includes(warn, magenta.open));
-	});
-}

@commitlint/core/src/load.test.js

@@ -34,7 +34,7 @@ test('uses seed with parserPreset', async t => {
 
 test('invalid extend should throw', async t => {
 	const cwd = await bootstrap('fixtures/extends-invalid');
-	t.throws(load({}, {cwd}));
+	await t.throws(load({}, {cwd}));
 });
 
 test('empty file should have no rules', async t => {