fix: close all connections in pool.end() (#633)

* fix: close all connections in pool.end()

* chore: add changeset

.changeset/giant-chicken-help.md

@@ -0,0 +1,5 @@
+---
+"slonik": major
+---
+
+fix: close all connections in pool.end(). Previously a subset idleConnections would hang until idleTimeout was reached

packages/slonik/src/factories/createConnectionPool.ts

@@ -108,7 +108,11 @@ export const createConnectionPool = ({
     // e.g. "waits for all connections to be established before attempting to terminate the pool" test
     await delay(0);
 
-    await Promise.all(connections.map((connection) => connection.destroy()));
+    // Make a copy of `connections` array as items are removed from it during the map iteration.
+    // If `connections` array is used directly, the loop will skip some items.
+    await Promise.all(
+      [...connections].map((connection) => connection.destroy()),
+    );
   };
 
   const acquire = async () => {

packages/slonik/src/helpers.test/createIntegrationTests.ts

@@ -926,7 +926,7 @@ export const createIntegrationTests = (
 
     const pool = await createPool(t.context.dsn, {
       driverFactory,
-      idleTimeout: 500,
+      idleTimeout: 'DISABLE_TIMEOUT',
       maximumPoolSize: 5,
     });
 
@@ -968,8 +968,6 @@ export const createIntegrationTests = (
 
     await pool.end();
 
-    await delay(600);
-
     t.deepEqual(pool.state(), {
       acquiredConnections: 0,
       idleConnections: 0,