Wait for pending ticks started by test (#995)

Author: geek

lib/runner.js

@@ -662,6 +662,11 @@ internals.protect = function (item, state) {
 
  try {
  await item.fn.call(null, flags);
+ // wait for any pending ticks started by the test
+ await new Promise((next) => {
+
+ setImmediate(next);
+ });
  finish();
  }
  catch (ex) {

test/cli.js

@@ -889,4 +889,13 @@ describe('CLI', () => {
  expect(result.output).to.contain('1 of 2 tests failed');
  expect(result.output).to.contain('oops');
  });
+
+ it('awaits any pending ticks started by test before starting next test', async () => {
+
+ const { code, output } = await RunCli(['test/cli_nexttick/test.js']);
+
+ expect(code).to.equal(1);
+ expect(output).to.contain('does not crash lab');
+ expect(output).to.not.contain('has another test');
+ });
 });

test/cli_nexttick/test.js

@@ -0,0 +1,34 @@
+'use strict';
+
+// Load modules
+
+const Hoek = require('@hapi/hoek');
+const _Lab = require('../../test_runner');
+
+
+// Declare internals
+
+const internals = {};
+
+
+// Test shortcuts
+
+const lab = exports.lab = _Lab.script();
+const describe = lab.describe;
+const it = lab.it;
+
+
+describe('Test CLI', () => {
+
+ it('does not crash lab', async () => {
+
+ await Hoek.wait(1);
+
+ process.nextTick(() => {
+
+ throw new Error('fail');
+ });
+ });
+
+ it('has another test', async () => {});
+});