test: make sure http pipelining does not emit a warning · nodejs/node@31fe3b2 · GitHub

nodejs / node Public

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

• Code
• Issues 636
• Pull requests 762
• Actions
• Projects
• Security and quality
• Insights

Additional navigation options

Commit 31fe3b2

Browse filesBrowse the repository at this point in the historyBrowse files

mcollina

authored and

lpinca

committed

test: make sure http pipelining does not emit a warning

PR-URL: #37964 Refs: #37937 Reviewed-By: Robert Nagy <ronagy@icloud.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com>

1 parent 2fa7d33 commit 31fe3b2
Copy full SHA for 31fe3b2

1 file changed

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

File tree

Expand file treeCollapse file tree

Open diff view settings

Filter options

• test/parallel
  • test-http-many-ended-pipelines.js

Expand file treeCollapse file tree

Open diff view settings

Collapse file

‎test/parallel/test-http-many-ended-pipelines.js‎

Copy file name to clipboardExpand all lines: test/parallel/test-http-many-ended-pipelines.js

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

Original file line number	Diff line number	Diff line change
@@ -20,7 +20,7 @@
20	20	// USE OR OTHER DEALINGS IN THE SOFTWARE.
21	21
22	22	'use strict';
23		- require('../common');
	23	+ const common = require('../common');
24	24
25	25	// No warnings should happen!
26	26	const trace = console.trace;
@@ -59,3 +59,5 @@ server.listen(0, function() {
59	59	client.end();
60	60	client.pipe(process.stdout);
61	61	});
	62	+
	63	+ process.on('warning', common.mustNotCall());

0 commit comments

Comments

0 (0)