fix: rename destroyTimeout option

Since the connections are being closed and not destroyed after the
timeout, I thought the name was misleading.

index.js

@@ -5,7 +5,7 @@ const { Pool } = require('undici')
 
 const {
   kHostsMap,
-  kDestroyTimeout,
+  kCloseTimeout,
   kMaxHosts,
   kDefaultConnectionOptions,
   kSetupConnection,
@@ -26,20 +26,20 @@ function onTimeout (key, pool, client) {
 
 class Agent11 {
   constructor ({
-    destroyTimeout = 6e4,
+    closeTimeout = 6e4,
     maxHosts = Infinity,
     connectionOptions = {}
   } = {}) {
-    if (typeof destroyTimeout !== 'number' || destroyTimeout <= 0) {
+    if (typeof closeTimeout !== 'number' || closeTimeout <= 0) {
       throw new Error(
-        'destroyTimeout must be a number > 0'
+        'closeTimeout must be a number > 0'
       )
     }
     if (typeof maxHosts !== 'number' || maxHosts <= 0) {
       throw new Error('maxHosts must be a number > 0')
     }
     this[kHostsMap] = new Map()
-    this[kDestroyTimeout] = destroyTimeout
+    this[kCloseTimeout] = closeTimeout
     this[kMaxHosts] = maxHosts
     this[kDefaultConnectionOptions] = connectionOptions
     this[kTimersMap] = new Map()
@@ -63,7 +63,7 @@ class Agent11 {
     this[kHostsMap].set(key, pool)
     this[kTimersMap].set(
       pool,
-      setTimeout(onTimeout, this[kDestroyTimeout], key, pool, this)
+      setTimeout(onTimeout, this[kCloseTimeout], key, pool, this)
     )
   }
 

symbols.js

@@ -2,7 +2,7 @@
 
 module.exports = {
   kHostsMap: Symbol('kHostsMap'),
-  kDestroyTimeout: Symbol('kDestroyTimeout'),
+  kCloseTimeout: Symbol('kDestroyTimeout'),
   kMaxHosts: Symbol('kMaxHosts'),
   kDefaultConnectionOptions: Symbol('kDefaultConnectionOptions'),
   kSetupConnection: Symbol('kSetupConnection'),

test.js

@@ -11,15 +11,15 @@ test('invalid options', t => {
   const list = [
     {
       value: {
-        destroyTimeout: 'string'
+        closeTimeout: 'string'
       },
-      message: 'destroyTimeout must be a number > 0'
+      message: 'closeTimeout must be a number > 0'
     },
     {
       value: {
-        destroyTimeout: 0
+        closeTimeout: 0
       },
-      message: 'destroyTimeout must be a number > 0'
+      message: 'closeTimeout must be a number > 0'
     },
     {
       value: {
@@ -148,7 +148,7 @@ test('getConnection should error if the url is invalid', t => {
 })
 
 test('should terminate inactive connections', async (t) => {
-  const agent = new Agent11({ destroyTimeout: 200 })
+  const agent = new Agent11({ closeTimeout: 200 })
   t.teardown(() => agent.close())
   agent.getConnection(new URL('http://xyz.xyz'))
   t.is(1, agent.size)
@@ -157,7 +157,7 @@ test('should terminate inactive connections', async (t) => {
 })
 
 test('shuld keep alive used connections', async (t) => {
-  const agent = new Agent11({ destroyTimeout: 200 })
+  const agent = new Agent11({ closeTimeout: 200 })
   t.teardown(() => agent.close())
   agent.getConnection(new URL('http://xyz.xyz'))
   t.is(1, agent.size)