socketduct 0.1.17 → 0.1.18

package/test/integration/reverse-route-class.test.js

@@ -132,8 +132,8 @@ test("route-class clients fail closed per OPEN while unaffected routes and legac
     hub: {host: "127.0.0.1", port: hub.port, token: "gateway-a-token"},
     tls,
     authorizeTarget: (target) => target.host === targetA.host && target.port === targetA.port,
-    reconnectDelayMs: 10_000,
-    maxReconnectDelayMs: 10_000
+    reconnectDelayMs: 10,
+    maxReconnectDelayMs: 20
   })
   const gatewayB = new OutboundReverseGateway({
     gatewayId: "server-b-starlink",
@@ -141,8 +141,8 @@ test("route-class clients fail closed per OPEN while unaffected routes and legac
     hub: {host: "127.0.0.1", port: hub.port, token: "gateway-b-token"},
     tls,
     authorizeTarget: (target) => target.host === targetB.host && target.port === targetB.port,
-    reconnectDelayMs: 10_000,
-    maxReconnectDelayMs: 10_000
+    reconnectDelayMs: 10,
+    maxReconnectDelayMs: 20
   })
   t.after(() => gatewayA.close())
   t.after(() => gatewayB.close())
@@ -206,18 +206,22 @@ test("route-class clients fail closed per OPEN while unaffected routes and legac
   await Promise.all([onceEvent(streamA, "connect"), onceEvent(streamB, "connect")])
   assert.deepEqual(await Promise.all([exchange(streamA, "one"), exchange(streamB, "two")]), ["a:one", "b:two"])
 
-  const streamAError = nextError(streamA)
+  const resumed = Promise.all([onceEvent(streamA, "ready"), onceEvent(streamB, "ready")])
   await hub.closeActiveGateway("server-a-starlink")
-  assert.equal((await streamAError).code, "GATEWAY_DISCONNECTED")
-  await waitUntil(() => hub.status().healthyGateways === 1, "gateway A disconnect")
-
-  const disconnected = client.createConnection({host: targetA.host, port: targetA.port})
-  assert.equal((await nextError(disconnected)).code, "GATEWAY_UNAVAILABLE")
+  await withTimeout(resumed, "route-class streams resume after gateway transport replacement")
+  await waitUntil(() => hub.status().healthyGateways === 2, "gateway A re-registration")
+  assert.equal(await exchange(streamA, "resumed"), "a:resumed")
   assert.equal(await exchange(streamB, "still-open"), "b:still-open")
+
+  const anotherA = client.createConnection({host: targetA.host, port: targetA.port})
+  await onceEvent(anotherA, "connect")
+  assert.equal(await exchange(anotherA, "new-a"), "a:new-a")
   const anotherB = client.createConnection({host: targetB.host, port: targetB.port})
   await onceEvent(anotherB, "connect")
   assert.equal(await exchange(anotherB, "new"), "b:new")
 
+  streamA.destroy()
+  anotherA.destroy()
   await gatewayA.close()
   const replacementA = new OutboundReverseGateway({
     gatewayId: "server-a-starlink",
@@ -232,7 +236,7 @@ test("route-class clients fail closed per OPEN while unaffected routes and legac
   const rejoinedA = client.createConnection({host: targetA.host, port: targetA.port})
   await onceEvent(rejoinedA, "connect")
   assert.equal(await exchange(rejoinedA, "rejoined"), "a:rejoined")
-  assert.equal(client.status().physicalConnects, 1)
+  assert(client.status().physicalConnects >= 2)
   assert.equal(hub.status().clientPhysicalConnections, 1)
   streamB.destroy()
   anotherB.destroy()
@@ -360,7 +364,7 @@ test("a backpressured gateway does not block DATA or a new OPEN for another rout
     t.after(() => socket.destroy())
     return nextError(socket)
   })
-  assert.equal((await withTimeout(streamAError, "gateway A output limit")).code, "GATEWAY_DISCONNECTED")
+  assert.equal((await withTimeout(streamAError, "gateway A output limit")).code, "OUTPUT_BACKPRESSURE_LIMIT")
   await withTimeout(Promise.all(queuedAErrors), "queued gateway A stream cleanup")
   assert.equal(await exchange(streamB, "after-limit"), "b:after-limit")
   assert.equal(hub.status().healthyGateways, 1)