Skip to content

Commit 4609cb7

Browse files
committed
test: use client response for realistic test case
1 parent a0fd4b7 commit 4609cb7

1 file changed

Lines changed: 19 additions & 8 deletions

File tree

test/parallel/test-stream-finished.js

Lines changed: 19 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -603,14 +603,25 @@ testClosed((opts) => new Writable({ write() {}, ...opts }));
603603
}
604604

605605
{
606-
const readable = new Readable({
607-
read() {}
608-
});
609-
readable.req = new EE();
610-
readable.aborted = true;
611-
612-
finished(readable, common.mustCall((err) => {
613-
assert.strictEqual(err, undefined);
606+
let serverRes;
607+
const server = http.createServer(common.mustCall((req, res) => {
608+
serverRes = res;
609+
res.write('hello');
610+
})).listen(0, common.mustCall(function() {
611+
http.get({ port: this.address().port }, common.mustCall((res) => {
612+
res.on('aborted', common.mustCall(() => {
613+
finished(res, common.mustCall((err) => {
614+
assert.strictEqual(err.code, 'ECONNRESET');
615+
assert.strictEqual(err.message, 'aborted');
616+
server.close();
617+
}));
618+
}));
619+
res.on('error', common.expectsError({
620+
code: 'ECONNRESET',
621+
message: 'aborted',
622+
}));
623+
serverRes.destroy();
624+
})).on('error', common.mustNotCall());
614625
}));
615626
}
616627

0 commit comments

Comments
 (0)