fix: don't pass prerequire (#57)

mocha will check arguments when there is no test files
eggjs · Jun 1, 2017 · e0d03d3 · e0d03d3
1 parent c1520b5
commit e0d03d3
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 0 deletions.
diff --git a/lib/cmd/cov.js b/lib/cmd/cov.js
@@ -55,6 +55,7 @@ class CovCommand extends Command {
     if (argv.prerequire) {
       process.env.EGG_BIN_PREREQUIRE = 'true';
     }
+    delete argv.prerequire;
 
     // istanbul coverage ignore
     if (argv.x) {

diff --git a/test/lib/cmd/cov.test.js b/test/lib/cmd/cov.test.js
@@ -131,6 +131,16 @@ describe('test/lib/cmd/cov.test.js', () => {
           .expect('code', 0)
           .end();
       });
+
+      it('should run cov when no test files', function* () {
+        mm(process.env, 'TESTS', 'noexist.js');
+        const cwd = path.join(__dirname, '../../fixtures/prerequire');
+        yield coffee.fork(eggBin, [ 'cov' ], { cwd })
+          .debug()
+          .coverage(false)
+          .expect('code', 0)
+          .end();
+      });
     });
   } else {
     it('should exec test instead of cov in win32', done => {