test: remove workaround code in debugger test

Remove code that made a check more lenient to account for a known issue
that is no longer reproducible.

Refs: nodejs/node-inspect#35

PR-URL: #39238
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Jan Krems <[email protected]>

Author: Trott

test/sequential/test-debugger-preserve-breaks.js

@@ -53,19 +53,9 @@ const path = require('path');
     })
     .then(() => cli.command('breakpoints'))
     .then(() => {
-      // 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/);
+      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);
     })
     .then(() => cli.quit())
     .then(null, onFatal);