test: remove string literal as message in strictEqual() in stream test · nodejs/node@103d01e · GitHub

nodejs / node Public

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

• Code
• Issues 633
• Pull requests 759
• Actions
• Projects
• Security and quality
• Insights

Additional navigation options

Commit 103d01e

Browse filesBrowse the repository at this point in the historyBrowse files

Trott

authored and

addaleax

committed

test: remove string literal as message in strictEqual() in stream test

This reveals the values that cause the assertion error, should it happen. PR-URL: #30561 Reviewed-By: Denys Otrishko <shishugi@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net>

1 parent ebba322 commit 103d01e
Copy full SHA for 103d01e

1 file changed

+2-3Lines changed: 2 additions & 3 deletions

File tree

Expand file treeCollapse file tree

Open diff view settings

Filter options

• test/sequential
  • test-stream-writable-clear-buffer.js

Expand file treeCollapse file tree

Open diff view settings

Collapse file

‎test/sequential/test-stream-writable-clear-buffer.js‎

Copy file name to clipboardExpand all lines: test/sequential/test-stream-writable-clear-buffer.js

+2-3Lines changed: 2 additions & 3 deletions

Original file line number	Diff line number	Diff line change
@@ -26,9 +26,8 @@ for (let i = 1; i <= 5; i++) {
26	26	testStream.write(i, () => {
27	27	assert.strictEqual(
28	28	testStream._writableState.bufferedRequestCount,
29		- testStream._writableState.getBuffer().length,
30		- 'bufferedRequestCount variable is different from the actual length of' +
31		- ' the buffer');
	29	+ testStream._writableState.getBuffer().length
	30	+ );
32	31	});
33	32	}
34	33

0 commit comments

Comments

0 (0)