Skip to content

Commit 7998058

Browse files
apapirovskiMylesBorins
authored andcommittedJan 9, 2018
test: fix flaky http-writable-true-after-close
PR-URL: #17952 Fixes: #16321 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Matteo Collina <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]>
1 parent 591dd4e commit 7998058

File tree

1 file changed

+8
-8
lines changed

1 file changed

+8
-8
lines changed
 

‎test/sequential/test-http-writable-true-after-close.js ‎test/parallel/test-http-writable-true-after-close.js

+8-8
Original file line numberDiff line numberDiff line change
@@ -12,16 +12,16 @@ let external;
1212

1313
// Proxy server
1414
const server = createServer(common.mustCall((req, res) => {
15-
get(`http://127.0.0.1:${internal.address().port}`, common.mustCall((inner) => {
16-
const listener = common.mustCall(() => {
17-
assert.strictEqual(res.writable, true);
18-
});
15+
const listener = common.mustCall(() => {
16+
assert.strictEqual(res.writable, true);
17+
});
1918

20-
// on CentOS 5, 'finish' is emitted
21-
res.on('finish', listener);
22-
// everywhere else, 'close' is emitted
23-
res.on('close', listener);
19+
// on CentOS 5, 'finish' is emitted
20+
res.on('finish', listener);
21+
// everywhere else, 'close' is emitted
22+
res.on('close', listener);
2423

24+
get(`http://127.0.0.1:${internal.address().port}`, common.mustCall((inner) => {
2525
inner.pipe(res);
2626
}));
2727
})).listen(0, () => {

0 commit comments

Comments
 (0)
Please sign in to comment.