fix(tests): update tests for inside CI

also make "codeframe" the default CI reporter Signed-off-by: Charlike Mike Reagent <olsten.larck@gmail.com>
tunnckoCore · Jul 19, 2018 · 46e02d4 · 46e02d4
1 parent f1a8e14
commit 46e02d4
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/src/utils.js b/src/utils.js
@@ -30,7 +30,7 @@ function getRelativePath(fp) {
 
 function getReporter(argv = {}) {
   if (isCI) {
-    argv.reporter = 'tap'; // eslint-disable-line no-param-reassign
+    return reporters.codeframe;
   }
 
   if (typeof argv.reporter === 'string') {

diff --git a/test/utils.test.js b/test/utils.test.js
@@ -1,5 +1,6 @@
 'use strict';
 
+const isCI = require('is-ci');
 const test = require('../src');
 
 const {
@@ -46,7 +47,9 @@ test('getParsedArgv gets correct when no env given', (t) => {
   t.strictEqual(argv.foobar, 'hohoho');
 });
 
-test('getReporter returns default "mini" reporter when no argv', (t) => {
+const testFn = isCI ? test.skip : test;
+
+testFn('getReporter returns default "mini" reporter when no argv', (t) => {
   const reporter = createReporter();
 
   t.strictEqual(typeof getReporter(), 'function');