fixup! fixup! fixup! test,debugger: migrate node-inspect tests to core

Author: Trott

test/inspector-cli/test-inspector-cli-preserve-breaks.js

@@ -53,18 +53,19 @@ const path = require('path');
     })
     .then(() => cli.command('breakpoints'))
     .then(() => {
-      if (process.platform === 'aix') {
-        // TODO: There is a known issue on AIX where the breakpoints aren't
-        // properly resolved yet when we reach this point.
-        // Eventually that should be figured out but for now we don't want
-        // to fail builds because of it.
-        assert.match(cli.output, /#0 [^\n]+three-lines\.js\$?:2/);
-        assert.match(cli.output, /#1 [^\n]+three-lines\.js\$?:3/);
-      } else {
-        const msg = `SCRIPT: ${script}, OUTPUT: ${cli.output}`;
-        assert.ok(cli.output.includes(`#0 ${script}:2`), msg);
-        assert.ok(cli.output.includes(`#1 ${script}:3`), msg);
-      }
+      // TODO: There is a known issue on AIX and some other operating systems
+      // where the breakpoints aren't properly resolved yet when we reach this
+      // point. Eventually that should be figured out but for now we don't
+      // want to fail builds because of it.
+      // What it should be:
+      //
+      // const msg = `SCRIPT: ${script}, OUTPUT: ${cli.output}`;
+      // assert.ok(cli.output.includes(`#0 ${script}:2`), msg);
+      // assert.ok(cli.output.includes(`#1 ${script}:3`), msg);
+      //
+      // What we're doing for now instead:
+      assert.match(cli.output, /#0 [^\n]+three-lines\.js\$?:2/);
+      assert.match(cli.output, /#1 [^\n]+three-lines\.js\$?:3/);
     })
     .then(() => cli.quit())
     .then(null, onFatal);