Skip to content

Commit cd9c7b5

Browse files
committed
✅ fix tests
1 parent 4594818 commit cd9c7b5

File tree

1 file changed

+13
-3
lines changed

1 file changed

+13
-3
lines changed

test/test.js

Lines changed: 13 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@ var assert = require('assert')
33
var http = require('http')
44
var request = require('supertest')
55
var timeout = require('..')
6+
var isFinished = require('on-finished').isFinished
67

78
describe('timeout()', function () {
89
it('should have a default timeout', function (done) {
@@ -43,7 +44,10 @@ describe('timeout()', function () {
4344
it('should respond with 503 Request timeout', function (done) {
4445
var server = createServer(null, null, function (req, res) {
4546
assert.ok(req.timedout)
46-
res.end('Hello')
47+
if (!isFinished(res)) {
48+
// check response is writable for node 14
49+
res.end('Hello')
50+
}
4751
})
4852

4953
request(server)
@@ -54,7 +58,10 @@ describe('timeout()', function () {
5458
it('should pass the error to next()', function (done) {
5559
var server = createServer(null, null, function (req, res) {
5660
assert.ok(req.timedout)
57-
res.end('Hello')
61+
if (!isFinished(res)) {
62+
// check response is writable for node 14
63+
res.end('Hello')
64+
}
5865
})
5966

6067
request(server)
@@ -170,7 +177,10 @@ describe('timeout()', function () {
170177
function (req, res) {
171178
assert.ok(aborted)
172179
assert.ok(!req.timedout)
173-
done()
180+
181+
server.close(function () {
182+
done()
183+
})
174184
})
175185
var test = request(server).post('/')
176186
test.write('0')

0 commit comments

Comments
 (0)