test: don't fail http2 abort test if 'data' is called multiple times · nodejs/node@793a5bb · GitHub

nodejs / node Public

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

• Code
• Issues 638
• Pull requests 758
• Actions
• Projects
• Security and quality
• Insights

Additional navigation options

Commit 793a5bb

Browse filesBrowse the repository at this point in the historyBrowse files

rubys

authored and

targos

committed

test: don't fail http2 abort test if 'data' is called multiple times

PR-URL: #21925 Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Anatoli Papirovski <apapirovski@mac.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com> Reviewed-By: Matteo Collina <matteo.collina@gmail.com>

1 parent 478dbee commit 793a5bb
Copy full SHA for 793a5bb

1 file changed

+1-1Lines changed: 1 addition & 1 deletion

File tree

Expand file treeCollapse file tree

Open diff view settings

Filter options

• test/parallel
  • test-http2-respond-with-file-connection-abort.js

Expand file treeCollapse file tree

Open diff view settings

Collapse file

‎test/parallel/test-http2-respond-with-file-connection-abort.js‎

Copy file name to clipboardExpand all lines: test/parallel/test-http2-respond-with-file-connection-abort.js

+1-1Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
@@ -24,7 +24,7 @@ server.listen(0, common.mustCall(() => {
24	24	const req = client.request();
25	25
26	26	req.on('response', common.mustCall(() => {}));
27		- req.on('data', common.mustCallAtLeast(() => {
	27	+ req.once('data', common.mustCall(() => {
28	28	net.Socket.prototype.destroy.call(client.socket);
29	29	server.close();
30	30	}));

0 commit comments

Comments

0 (0)