Skip to content

Commit 4ef1f3e

Browse files
ryzokukentargos
authored andcommitted
test: rename test-regress-GH-784.js
Rename the test appropriately alongside mentioning the subsystem Also, make a few basic changes to make sure the test conforms to the standard test structure Refs: #19105 Refs: https://github.com/nodejs/node/blob/master/doc/guides/writing-tests.md#test-structure PR-URL: #19161 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]> Reviewed-By: Joyee Cheung <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Gireesh Punathil <[email protected]>
1 parent 0f828bc commit 4ef1f3e

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

test/sequential/test-regress-GH-784.js test/sequential/test-http-econnrefused.js

+5-5
Original file line numberDiff line numberDiff line change
@@ -20,15 +20,15 @@
2020
// USE OR OTHER DEALINGS IN THE SOFTWARE.
2121

2222
'use strict';
23-
// Regression test for GH-784
24-
// https://github.com/joyent/node/issues/784
25-
//
23+
const common = require('../common');
24+
2625
// The test works by making a total of 8 requests to the server. The first
2726
// two are made with the server off - they should come back as ECONNREFUSED.
2827
// The next two are made with server on - they should come back successful.
2928
// The next two are made with the server off - and so on. Without the fix
30-
// we were experiencing parse errors and instead of ECONNREFUSED.
31-
const common = require('../common');
29+
// we were experiencing parse errors instead of ECONNREFUSED.
30+
// https://github.com/nodejs/node-v0.x-archive/issues/784
31+
3232
const http = require('http');
3333
const assert = require('assert');
3434

0 commit comments

Comments
 (0)