test: improve test-debug-usage

test-debug-usage fails if run with `--trace-warnings` because the regular expression checking does not allow for the resulting stack trace from the deprecation warning. The test also only tests two parts of the three-part usage message. Improve the test so that it passes with `--trace-warnings` and verifies all three parts of the usage message. Signed-off-by: Rich Trott <[email protected]> PR-URL: #32141 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]>
nodejs · Mar 10, 2020 · 66e32ff · 66e32ff
1 parent 192cb72
commit 66e32ff
Showing 1 changed file with 3 additions and 5 deletions.
diff --git a/test/parallel/test-debug-usage.js b/test/parallel/test-debug-usage.js
@@ -11,8 +11,7 @@ child.stderr.setEncoding('utf8');
 
 const expectedLines = [
   /^\(node:\d+\) \[DEP0068\] DeprecationWarning:/,
-  /^Usage: .*node.* debug script\.js$/,
-  /^       .*node.* debug <host>:<port>$/
+  /Usage: .*node.* debug script\.js\r?\n       .*node.* debug <host>:<port>\r?\n       .*node.* debug -p <pid>\r?\n$/,
 ];
 
 let actualUsageMessage = '';
@@ -21,11 +20,10 @@ child.stderr.on('data', function(data) {
 });
 
 child.on('exit', common.mustCall(function(code) {
-  const outputLines = actualUsageMessage.split('\n');
   assert.strictEqual(code, 1);
   for (let i = 0; i < expectedLines.length; i++)
     assert.ok(
-      expectedLines[i].test(outputLines[i]),
-      `${outputLines[i]} did not match ${expectedLines[i]}`
+      expectedLines[i].test(actualUsageMessage),
+      `${actualUsageMessage} did not match ${expectedLines[i]}`
     );
 }));