From fee6ffbeb7401ff1b1d3e73e9aee430d30484a9f Mon Sep 17 00:00:00 2001
From: Moshe Atlow <moshe@atlow.co.il>
Date: Tue, 23 May 2023 22:19:42 +0300
Subject: [PATCH] test: fix flaky test-watch-mode

---
 test/fixtures/watch-mode/dependant.js         |   2 -
 test/fixtures/watch-mode/dependant.mjs        |   2 -
 test/fixtures/watch-mode/dependency.js        |   1 -
 test/fixtures/watch-mode/dependency.mjs       |   1 -
 .../fixtures/watch-mode/event_loop_blocked.js |   4 -
 test/fixtures/watch-mode/failing.js           |   1 -
 test/fixtures/watch-mode/parse_args.js        |   4 -
 test/fixtures/watch-mode/process_exit.js      |   1 -
 test/fixtures/watch-mode/subdir/file.js       |   1 -
 test/sequential/test-watch-mode.mjs           | 390 +++++++++---------
 10 files changed, 199 insertions(+), 208 deletions(-)
 delete mode 100644 test/fixtures/watch-mode/dependant.js
 delete mode 100644 test/fixtures/watch-mode/dependant.mjs
 delete mode 100644 test/fixtures/watch-mode/dependency.js
 delete mode 100644 test/fixtures/watch-mode/dependency.mjs
 delete mode 100644 test/fixtures/watch-mode/event_loop_blocked.js
 delete mode 100644 test/fixtures/watch-mode/failing.js
 delete mode 100644 test/fixtures/watch-mode/parse_args.js
 delete mode 100644 test/fixtures/watch-mode/process_exit.js
 delete mode 100644 test/fixtures/watch-mode/subdir/file.js

diff --git a/test/fixtures/watch-mode/dependant.js b/test/fixtures/watch-mode/dependant.js
deleted file mode 100644
index 25a0ec095d7cf2..00000000000000
--- a/test/fixtures/watch-mode/dependant.js
+++ /dev/null
@@ -1,2 +0,0 @@
-const dependency = require('./dependency');
-console.log(dependency);
diff --git a/test/fixtures/watch-mode/dependant.mjs b/test/fixtures/watch-mode/dependant.mjs
deleted file mode 100644
index dff99bb2714728..00000000000000
--- a/test/fixtures/watch-mode/dependant.mjs
+++ /dev/null
@@ -1,2 +0,0 @@
-import dependency from './dependency.mjs';
-console.log(dependency);
diff --git a/test/fixtures/watch-mode/dependency.js b/test/fixtures/watch-mode/dependency.js
deleted file mode 100644
index f053ebf7976e37..00000000000000
--- a/test/fixtures/watch-mode/dependency.js
+++ /dev/null
@@ -1 +0,0 @@
-module.exports = {};
diff --git a/test/fixtures/watch-mode/dependency.mjs b/test/fixtures/watch-mode/dependency.mjs
deleted file mode 100644
index ff8b4c56321a33..00000000000000
--- a/test/fixtures/watch-mode/dependency.mjs
+++ /dev/null
@@ -1 +0,0 @@
-export default {};
diff --git a/test/fixtures/watch-mode/event_loop_blocked.js b/test/fixtures/watch-mode/event_loop_blocked.js
deleted file mode 100644
index e74b7fd389339c..00000000000000
--- a/test/fixtures/watch-mode/event_loop_blocked.js
+++ /dev/null
@@ -1,4 +0,0 @@
-console.log('running');
-Atomics.wait(new Int32Array(new SharedArrayBuffer(4)), 0, 0);
-console.log('don\'t show me');
-
diff --git a/test/fixtures/watch-mode/failing.js b/test/fixtures/watch-mode/failing.js
deleted file mode 100644
index d1e87944d9f33c..00000000000000
--- a/test/fixtures/watch-mode/failing.js
+++ /dev/null
@@ -1 +0,0 @@
-throw new Error('fails');
diff --git a/test/fixtures/watch-mode/parse_args.js b/test/fixtures/watch-mode/parse_args.js
deleted file mode 100644
index 06c7227cee5933..00000000000000
--- a/test/fixtures/watch-mode/parse_args.js
+++ /dev/null
@@ -1,4 +0,0 @@
-const { parseArgs } = require('node:util');
-
-const { values } = parseArgs({ options: { random: { type: 'string' } } });
-console.log(values.random);
diff --git a/test/fixtures/watch-mode/process_exit.js b/test/fixtures/watch-mode/process_exit.js
deleted file mode 100644
index cbe6cdd84cc073..00000000000000
--- a/test/fixtures/watch-mode/process_exit.js
+++ /dev/null
@@ -1 +0,0 @@
-setImmediate(() => process.exit(0));
diff --git a/test/fixtures/watch-mode/subdir/file.js b/test/fixtures/watch-mode/subdir/file.js
deleted file mode 100644
index 8b137891791fe9..00000000000000
--- a/test/fixtures/watch-mode/subdir/file.js
+++ /dev/null
@@ -1 +0,0 @@
-
diff --git a/test/sequential/test-watch-mode.mjs b/test/sequential/test-watch-mode.mjs
index e0b23107f703f6..2874f3e1bb13f5 100644
--- a/test/sequential/test-watch-mode.mjs
+++ b/test/sequential/test-watch-mode.mjs
@@ -1,96 +1,70 @@
 import * as common from '../common/index.mjs';
-import * as fixtures from '../common/fixtures.mjs';
 import tmpdir from '../common/tmpdir.js';
 import assert from 'node:assert';
 import path from 'node:path';
 import { execPath } from 'node:process';
 import { describe, it } from 'node:test';
 import { spawn } from 'node:child_process';
-import { writeFileSync, readFileSync } from 'node:fs';
+import { writeFileSync, readFileSync, mkdirSync } from 'node:fs';
 import { inspect } from 'node:util';
-import { once } from 'node:events';
 import { createInterface } from 'node:readline/promises';
 
 if (common.isIBMi)
   common.skip('IBMi does not support `fs.watch()`');
 
 const supportsRecursive = common.isOSX || common.isWindows;
-let disableRestart = false;
 
-function restart(file) {
+function restart(file, content = readFileSync(file)) {
   // To avoid flakiness, we save the file repeatedly until test is done
-  writeFileSync(file, readFileSync(file));
-  const timer = setInterval(() => {
-    if (!disableRestart) {
-      writeFileSync(file, readFileSync(file));
-    }
-  }, common.platformTimeout(1000));
+  writeFileSync(file, content);
+  const timer = setInterval(() => writeFileSync(file, content), common.platformTimeout(2500));
   return () => clearInterval(timer);
 }
 
-async function spawnWithRestarts({
-  args,
-  file,
-  watchedFile = file,
-  restarts = 1,
-  isReady,
-}) {
-  args ??= [file];
-  const printedArgs = inspect(args.slice(args.indexOf(file)).join(' '));
-  isReady ??= (data) => Boolean(data.match(new RegExp(`(Failed|Completed) running ${printedArgs.replace(/\\/g, '\\\\')}`, 'g'))?.length);
+let tmpFiles = 0;
+function createTmpFile(content = 'console.log("running");', ext = '.js', basename = tmpdir.path) {
+  const file = path.join(basename, `${tmpFiles++}${ext}`);
+  writeFileSync(file, content);
+  return file;
+}
 
+async function runWriteSucceed({
+  file, watchedFile, args = [file], env = {}, completed = 'Completed running', restarts = 2
+}) {
+  const child = spawn(execPath, ['--watch', '--no-warnings', ...args], { encoding: 'utf8', stdio: 'pipe', env });
+  let completes = 0;
+  let cancelRestarts = () => {};
   let stderr = '';
-  let stdout = '';
-  let cancelRestarts;
+  const stdout = [];
 
-  disableRestart = true;
-  const child = spawn(execPath, ['--watch', '--no-warnings', ...args], { encoding: 'utf8' });
   child.stderr.on('data', (data) => {
     stderr += data;
   });
-  child.stdout.on('data', async (data) => {
-    if (data.toString().includes('Restarting')) {
-      disableRestart = true;
-    }
-    stdout += data;
-    const restartsCount = stdout.match(new RegExp(`Restarting ${printedArgs.replace(/\\/g, '\\\\')}`, 'g'))?.length ?? 0;
-    if (restarts === 0 || !isReady(data.toString())) {
-      return;
-    }
-    if (restartsCount >= restarts) {
-      cancelRestarts?.();
-      child.kill();
-      return;
-    }
-    cancelRestarts ??= restart(watchedFile);
-    if (isReady(data.toString())) {
-      disableRestart = false;
-    }
-  });
-
-  await once(child, 'exit');
-  cancelRestarts?.();
-  return { stderr, stdout };
-}
 
-let tmpFiles = 0;
-function createTmpFile(content = 'console.log("running");', ext = '.js') {
-  const file = path.join(tmpdir.path, `${tmpFiles++}${ext}`);
-  writeFileSync(file, content);
-  return file;
-}
-
-function assertRestartedCorrectly({ stdout, messages: { inner, completed, restarted } }) {
-  const lines = stdout.split(/\r?\n/).filter(Boolean);
-
-  const start = [inner, completed, restarted].filter(Boolean);
-  const end = [restarted, inner, completed].filter(Boolean);
-  assert.deepStrictEqual(lines.slice(0, start.length), start);
-  assert.deepStrictEqual(lines.slice(-end.length), end);
+  try {
+    // Break the chunks into lines
+    for await (const data of createInterface({ input: child.stdout })) {
+      stdout.push(data);
+      if (data.startsWith(completed)) {
+        completes++;
+        if (completes === restarts) {
+          break;
+        }
+        if (completes === 1) {
+          cancelRestarts = restart(watchedFile);
+        }
+      }
+    }
+  } finally {
+    child.kill();
+    cancelRestarts();
+  }
+  return { stdout, stderr };
 }
 
 async function failWriteSucceed({ file, watchedFile }) {
-  const child = spawn(execPath, ['--watch', '--no-warnings', file], { encoding: 'utf8' });
+  const child = spawn(execPath, ['--watch', '--no-warnings', file], { encoding: 'utf8', stdio: 'pipe' });
+  let cancelRestarts = () => {};
 
   try {
     // Break the chunks into lines
@@ -99,189 +73,219 @@ async function failWriteSucceed({ file, watchedFile }) {
         break;
       }
       if (data.startsWith('Failed running')) {
-        writeFileSync(watchedFile, 'console.log("test has ran");');
+        cancelRestarts = restart(watchedFile, 'console.log("test has ran");');
       }
     }
   } finally {
     child.kill();
+    cancelRestarts();
   }
 }
 
 tmpdir.refresh();
 
-// Warning: this suite cannot run safely with concurrency: true
-// because of the disableRestart flag used for controlling restarts
-describe('watch mode', { concurrency: false, timeout: 60_000 }, () => {
+describe('watch mode', { concurrency: true, timeout: 60_000 }, () => {
   it('should watch changes to a file - event loop ended', async () => {
     const file = createTmpFile();
-    const { stderr, stdout } = await spawnWithRestarts({ file });
+    const { stderr, stdout } = await runWriteSucceed({ file, watchedFile: file });
 
     assert.strictEqual(stderr, '');
-    assertRestartedCorrectly({
-      stdout,
-      messages: { inner: 'running', completed: `Completed running ${inspect(file)}`, restarted: `Restarting ${inspect(file)}` },
-    });
+    assert.deepStrictEqual(stdout, [
+      'running',
+      `Completed running ${inspect(file)}`,
+      `Restarting ${inspect(file)}`,
+      'running',
+      `Completed running ${inspect(file)}`,
+    ]);
   });
 
   it('should watch changes to a failing file', async () => {
-    const file = fixtures.path('watch-mode/failing.js');
-    const { stderr, stdout } = await spawnWithRestarts({ file });
-
-    // Use match first to pretty print diff on failure
-    assert.match(stderr, /Error: fails\r?\n/);
-    // Test that failures happen once per restart
-    assert(stderr.match(/Error: fails\r?\n/g).length >= 2);
-    assertRestartedCorrectly({
-      stdout,
-      messages: { completed: `Failed running ${inspect(file)}`, restarted: `Restarting ${inspect(file)}` },
-    });
+    const file = createTmpFile('throw new Error("fails");');
+    const { stderr, stdout } = await runWriteSucceed({ file, watchedFile: file, completed: 'Failed running' });
+
+    assert.strictEqual(stderr.match(/Error: fails\r?\n/g).length, 2);
+    assert.deepStrictEqual(stdout, [
+      `Failed running ${inspect(file)}`,
+      `Restarting ${inspect(file)}`,
+      `Failed running ${inspect(file)}`,
+    ]);
   });
 
   it('should watch changes to a file with watch-path', {
     skip: !supportsRecursive,
   }, async () => {
+    const dir = path.join(tmpdir.path, 'subdir1');
+    mkdirSync(dir);
     const file = createTmpFile();
-    const watchedFile = fixtures.path('watch-mode/subdir/file.js');
-    const { stderr, stdout } = await spawnWithRestarts({
-      file,
-      watchedFile,
-      args: ['--watch-path', fixtures.path('./watch-mode/subdir'), file],
-    });
+    const watchedFile = createTmpFile('', '.js', dir);
+    const args = ['--watch-path', dir, file];
+    const { stderr, stdout } = await runWriteSucceed({ file, watchedFile, args });
+
+    assert.strictEqual(stderr, '');
+    assert.deepStrictEqual(stdout, [
+      'running',
+      `Completed running ${inspect(file)}`,
+      `Restarting ${inspect(file)}`,
+      'running',
+      `Completed running ${inspect(file)}`,
+    ]);
     assert.strictEqual(stderr, '');
-    assertRestartedCorrectly({
-      stdout,
-      messages: { inner: 'running', completed: `Completed running ${inspect(file)}`, restarted: `Restarting ${inspect(file)}` },
-    });
   });
 
   it('should watch when running an non-existing file - when specified under --watch-path', {
     skip: !supportsRecursive
   }, async () => {
-    const file = fixtures.path('watch-mode/subdir/non-existing.js');
-    const watchedFile = fixtures.path('watch-mode/subdir/file.js');
-    const { stderr, stdout } = await spawnWithRestarts({
-      file,
-      watchedFile,
-      args: ['--watch-path', fixtures.path('./watch-mode/subdir/'), file],
-    });
-
-    assert.match(stderr, /Error: Cannot find module/);
-    assert(stderr.match(/Error: Cannot find module/g).length >= 2);
-
-    assertRestartedCorrectly({
-      stdout,
-      messages: { completed: `Failed running ${inspect(file)}`, restarted: `Restarting ${inspect(file)}` },
-    });
+    const dir = path.join(tmpdir.path, 'subdir2');
+    mkdirSync(dir);
+    const file = path.join(dir, 'non-existing.js');
+    const watchedFile = createTmpFile('', '.js', dir);
+    const args = ['--watch-path', dir, file];
+    const { stderr, stdout } = await runWriteSucceed({ file, watchedFile, args, completed: 'Failed running' });
+
+    assert.strictEqual(stderr.match(/Error: Cannot find module/g).length, 2);
+    assert.deepStrictEqual(stdout, [
+      `Failed running ${inspect(file)}`,
+      `Restarting ${inspect(file)}`,
+      `Failed running ${inspect(file)}`,
+    ]);
   });
 
   it('should watch when running an non-existing file - when specified under --watch-path with equals', {
     skip: !supportsRecursive
   }, async () => {
-    const file = fixtures.path('watch-mode/subdir/non-existing.js');
-    const watchedFile = fixtures.path('watch-mode/subdir/file.js');
-    const { stderr, stdout } = await spawnWithRestarts({
-      file,
-      watchedFile,
-      args: [`--watch-path=${fixtures.path('./watch-mode/subdir/')}`, file],
-    });
-
-    assert.match(stderr, /Error: Cannot find module/);
-    assert(stderr.match(/Error: Cannot find module/g).length >= 2);
-    assertRestartedCorrectly({
-      stdout,
-      messages: { completed: `Failed running ${inspect(file)}`, restarted: `Restarting ${inspect(file)}` },
-    });
+    const dir = path.join(tmpdir.path, 'subdir3');
+    mkdirSync(dir);
+    const file = path.join(dir, 'non-existing.js');
+    const watchedFile = createTmpFile('', '.js', dir);
+    const args = [`--watch-path=${dir}`, file];
+    const { stderr, stdout } = await runWriteSucceed({ file, watchedFile, args, completed: 'Failed running' });
+
+    assert.strictEqual(stderr.match(/Error: Cannot find module/g).length, 2);
+    assert.deepStrictEqual(stdout, [
+      `Failed running ${inspect(file)}`,
+      `Restarting ${inspect(file)}`,
+      `Failed running ${inspect(file)}`,
+    ]);
   });
 
-  it('should watch changes to a file - event loop blocked', async () => {
-    const file = fixtures.path('watch-mode/event_loop_blocked.js');
-    const { stderr, stdout } = await spawnWithRestarts({
-      file,
-      isReady: (data) => data.startsWith('running'),
-    });
+  it('should watch changes to a file - event loop blocked', { timeout: 5000 }, async () => {
+    const file = createTmpFile(`
+console.log("running");
+Atomics.wait(new Int32Array(new SharedArrayBuffer(4)), 0, 0);
+console.log("don't show me");`);
+    const { stderr, stdout } = await runWriteSucceed({ file, watchedFile: file, completed: 'running' });
 
     assert.strictEqual(stderr, '');
-    assertRestartedCorrectly({
-      stdout,
-      messages: { inner: 'running', restarted: `Restarting ${inspect(file)}` },
-    });
+    assert.deepStrictEqual(stdout, [
+      'running',
+      `Restarting ${inspect(file)}`,
+      'running',
+    ]);
   });
 
   it('should watch changes to dependencies - cjs', async () => {
-    const file = fixtures.path('watch-mode/dependant.js');
-    const dependency = fixtures.path('watch-mode/dependency.js');
-    const { stderr, stdout } = await spawnWithRestarts({
-      file,
-      watchedFile: dependency,
-    });
+    const dependency = createTmpFile('module.exports = {};');
+    const file = createTmpFile(`
+const dependency = require('${dependency.replace(/\\/g, '/')}');
+console.log(dependency);
+`);
+    const { stderr, stdout } = await runWriteSucceed({ file, watchedFile: dependency });
 
     assert.strictEqual(stderr, '');
-    assertRestartedCorrectly({
-      stdout,
-      messages: { inner: '{}', restarted: `Restarting ${inspect(file)}`, completed: `Completed running ${inspect(file)}` },
-    });
+    assert.deepStrictEqual(stdout, [
+      '{}',
+      `Completed running ${inspect(file)}`,
+      `Restarting ${inspect(file)}`,
+      '{}',
+      `Completed running ${inspect(file)}`,
+    ]);
   });
 
   it('should watch changes to dependencies - esm', async () => {
-    const file = fixtures.path('watch-mode/dependant.mjs');
-    const dependency = fixtures.path('watch-mode/dependency.mjs');
-    const { stderr, stdout } = await spawnWithRestarts({
-      file,
-      watchedFile: dependency,
-    });
+    const dependency = createTmpFile('module.exports = {};');
+    const file = createTmpFile(`
+import dependency from 'file://${dependency.replace(/\\/g, '/')}';
+console.log(dependency);
+`, '.mjs');
+    const { stderr, stdout } = await runWriteSucceed({ file, watchedFile: dependency });
 
     assert.strictEqual(stderr, '');
-    assertRestartedCorrectly({
-      stdout,
-      messages: { inner: '{}', restarted: `Restarting ${inspect(file)}`, completed: `Completed running ${inspect(file)}` },
-    });
+    assert.deepStrictEqual(stdout, [
+      '{}',
+      `Completed running ${inspect(file)}`,
+      `Restarting ${inspect(file)}`,
+      '{}',
+      `Completed running ${inspect(file)}`,
+    ]);
   });
 
   it('should restart multiple times', async () => {
     const file = createTmpFile();
-    const { stderr, stdout } = await spawnWithRestarts({ file, restarts: 3 });
+    const { stderr, stdout } = await runWriteSucceed({ file, watchedFile: file, restarts: 3 });
 
     assert.strictEqual(stderr, '');
-    assert.strictEqual(stdout.match(new RegExp(`Restarting ${inspect(file).replace(/\\/g, '\\\\')}`, 'g')).length, 3);
+    assert.deepStrictEqual(stdout, [
+      'running',
+      `Completed running ${inspect(file)}`,
+      `Restarting ${inspect(file)}`,
+      'running',
+      `Completed running ${inspect(file)}`,
+      `Restarting ${inspect(file)}`,
+      'running',
+      `Completed running ${inspect(file)}`,
+    ]);
   });
 
   it('should pass arguments to file', async () => {
-    const file = fixtures.path('watch-mode/parse_args.js');
+    const file = createTmpFile(`
+const { parseArgs } = require('node:util');
+const { values } = parseArgs({ options: { random: { type: 'string' } } });
+console.log(values.random);
+    `);
     const random = Date.now().toString();
     const args = [file, '--random', random];
-    const { stderr, stdout } = await spawnWithRestarts({ file, args });
+    const { stderr, stdout } = await runWriteSucceed({ file, watchedFile: file, args });
 
     assert.strictEqual(stderr, '');
-    assertRestartedCorrectly({
-      stdout,
-      messages: { inner: random, restarted: `Restarting ${inspect(args.join(' '))}`, completed: `Completed running ${inspect(args.join(' '))}` },
-    });
+    assert.deepStrictEqual(stdout, [
+      random,
+      `Completed running ${inspect(`${file} --random ${random}`)}`,
+      `Restarting ${inspect(`${file} --random ${random}`)}`,
+      random,
+      `Completed running ${inspect(`${file} --random ${random}`)}`,
+    ]);
   });
 
   it('should not load --require modules in main process', async () => {
-    const file = createTmpFile('');
-    const required = fixtures.path('watch-mode/process_exit.js');
+    const file = createTmpFile();
+    const required = createTmpFile('setImmediate(() => process.exit(0));');
     const args = ['--require', required, file];
-    const { stderr, stdout } = await spawnWithRestarts({ file, args });
+    const { stderr, stdout } = await runWriteSucceed({ file, watchedFile: file, args });
 
     assert.strictEqual(stderr, '');
-    assertRestartedCorrectly({
-      stdout,
-      messages: { restarted: `Restarting ${inspect(file)}`, completed: `Completed running ${inspect(file)}` },
-    });
+    assert.deepStrictEqual(stdout, [
+      'running',
+      `Completed running ${inspect(file)}`,
+      `Restarting ${inspect(file)}`,
+      'running',
+      `Completed running ${inspect(file)}`,
+    ]);
   });
 
   it('should not load --import modules in main process', async () => {
-    const file = createTmpFile('');
-    const imported = fixtures.fileURL('watch-mode/process_exit.js');
+    const file = createTmpFile();
+    const imported = `file://${createTmpFile('setImmediate(() => process.exit(0));')}`;
     const args = ['--import', imported, file];
-    const { stderr, stdout } = await spawnWithRestarts({ file, args });
+    const { stderr, stdout } = await runWriteSucceed({ file, watchedFile: file, args });
 
     assert.strictEqual(stderr, '');
-    assertRestartedCorrectly({
-      stdout,
-      messages: { restarted: `Restarting ${inspect(file)}`, completed: `Completed running ${inspect(file)}` },
-    });
+    assert.deepStrictEqual(stdout, [
+      'running',
+      `Completed running ${inspect(file)}`,
+      `Restarting ${inspect(file)}`,
+      'running',
+      `Completed running ${inspect(file)}`,
+    ]);
   });
 
   // TODO: Remove skip after https://github.com/nodejs/node/pull/45271 lands
@@ -319,26 +323,30 @@ describe('watch mode', { concurrency: false, timeout: 60_000 }, () => {
     await failWriteSucceed({ file: dependant, watchedFile: dependency });
   });
 
+  it('should clear output between runs', async () => {
+    const file = createTmpFile();
+    const { stderr, stdout } = await runWriteSucceed({ file, watchedFile: file, env: { FORCE_COLOR: '1' } });
+
+    assert.strictEqual(stderr, '');
+    assert.deepStrictEqual(stdout, [
+      'running',
+      `Completed running ${inspect(file)}`,
+      `Restarting ${inspect(file)}`,
+      'running',
+      `Completed running ${inspect(file)}`,
+    ]);
+  });
+
   it('should preserve output when --watch-preserve-output flag is passed', async () => {
     const file = createTmpFile();
-    const { stderr, stdout } = await spawnWithRestarts({
-      file,
-      args: ['--watch-preserve-output', file],
-    });
+    const args = ['--watch-preserve-output', file];
+    const { stderr, stdout } = await runWriteSucceed({ file, watchedFile: file, args, env: { FORCE_COLOR: '1' } });
 
     assert.strictEqual(stderr, '');
-    // Checks if the existing output is preserved
-    assertRestartedCorrectly({
-      stdout,
-      messages: {
-        inner: 'running',
-        restarted: `Restarting ${inspect(file)}`,
-        completed: `Completed running ${inspect(file)}`,
-      },
-    });
-    // Remove the first 3 lines from stdout
-    const lines = stdout.split(/\r?\n/).filter(Boolean).slice(3);
-    assert.deepStrictEqual(lines, [
+    assert.deepStrictEqual(stdout, [
+      'running',
+      `Completed running ${inspect(file)}`,
+      `Restarting ${inspect(file)}`,
       'running',
       `Completed running ${inspect(file)}`,
     ]);