test: improve assertion error message in test-debug-usage · nodejs/node@d75a3f6 · GitHub

nodejs / node Public

• Notifications You must be signed in to change notification settings
• Fork 36.6k
• Star 120k

• Code
• Issues 635
• Pull requests 755
• Actions
• Projects
• Security and quality
• Insights

Additional navigation options

Commit d75a3f6

Browse filesBrowse the repository at this point in the historyBrowse files

Trott

authored and

BethGriggs

committed

test: improve assertion error message in test-debug-usage

PR-URL: #30913 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Richard Lau <riclau@uk.ibm.com> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>

1 parent 9bc5c9f commit d75a3f6
Copy full SHA for d75a3f6

1 file changed

+4-1Lines changed: 4 additions & 1 deletion

File tree

Expand file treeCollapse file tree

Open diff view settings

Filter options

• test/parallel
  • test-debug-usage.js

Expand file treeCollapse file tree

Open diff view settings

Collapse file

‎test/parallel/test-debug-usage.js‎

Copy file name to clipboardExpand all lines: test/parallel/test-debug-usage.js

+4-1Lines changed: 4 additions & 1 deletion

Original file line number	Diff line number	Diff line change
@@ -24,5 +24,8 @@ child.on('exit', common.mustCall(function(code) {
24	24	const outputLines = actualUsageMessage.split('\n');
25	25	assert.strictEqual(code, 1);
26	26	for (let i = 0; i < expectedLines.length; i++)
27		- assert(expectedLines[i].test(outputLines[i]));
	27	+ assert.ok(
	28	+ expectedLines[i].test(outputLines[i]),
	29	+ `${outputLines[i]} did not match ${expectedLines[i]}`
	30	+ );
28	31	}));

0 commit comments

Comments

0 (0)