archive-gh-me/node-postgres
1
0
Fork 0
mirror of https://github.com/brianc/node-postgres.git synced 2025-12-08 20:16:25 +00:00
Code Issues Projects Releases Wiki Activity

Fix test flake on old node

Browse Source
This commit is contained in:
Brian M. Carlson 2021-07-27 12:16:39 -05:00
parent da49303522
commit 31ca683024
1 changed files with 6 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
packages/pg/test/integration/client/connection-timeout-tests.js
View File

@ -13,7 +13,7 @@ const options = {
database: 'existing',
}
const serverWithConnectionTimeout = (timeout, callback) => {
const serverWithConnectionTimeout = (port, timeout, callback) => {
const sockets = new Set()
const server = net.createServer((socket) => {
@ -47,11 +47,11 @@ const serverWithConnectionTimeout = (timeout, callback) => {
}
}
server.listen(options.port, options.host, () => callback(closeServer))
server.listen(port, options.host, () => callback(closeServer))
}
suite.test('successful connection', (done) => {
serverWithConnectionTimeout(0, (closeServer) => {
serverWithConnectionTimeout(options.port, 0, (closeServer) => {
const timeoutId = setTimeout(() => {
throw new Error('Client should have connected successfully but it did not.')
}, 3000)
@ -67,12 +67,13 @@ suite.test('successful connection', (done) => {
})
suite.test('expired connection timeout', (done) => {
serverWithConnectionTimeout(options.connectionTimeoutMillis * 2, (closeServer) => {
const opts = { ...options, port: 54322 }
serverWithConnectionTimeout(opts.port, opts.connectionTimeoutMillis * 2, (closeServer) => {
const timeoutId = setTimeout(() => {
throw new Error('Client should have emitted an error but it did not.')
}, 3000)
const client = new helper.Client(options)
const client = new helper.Client(opts)
client
.connect()
.then(() => client.end())