Use disconnect in Node

Author: SteveSandersonMS

nodejs/test/e2e/multi-client.test.ts

@@ -15,7 +15,7 @@ describe("Multi-client broadcast", async () => {
 
     // Trigger connection so we can read the port
     const initSession = await client1.createSession({ onPermissionRequest: approveAll });
-    await initSession.destroy();
+    await initSession.disconnect();
 
     const actualPort = (client1 as unknown as { actualPort: number }).actualPort;
     let client2 = new CopilotClient({ cliUrl: `localhost:${actualPort}` });
@@ -79,7 +79,7 @@ describe("Multi-client broadcast", async () => {
         expect(client1ToolCompleted.length).toBeGreaterThan(0);
         expect(client2ToolCompleted.length).toBeGreaterThan(0);
 
-        await session2.destroy();
+        await session2.disconnect();
     });
 
     it("one client approves permission and both see the result", async () => {
@@ -137,7 +137,7 @@ describe("Multi-client broadcast", async () => {
             expect(event.data.result.kind).toBe("approved");
         }
 
-        await session2.destroy();
+        await session2.disconnect();
     });
 
     it("one client rejects permission and both see the result", async () => {
@@ -195,7 +195,7 @@ describe("Multi-client broadcast", async () => {
             expect(event.data.result.kind).toBe("denied-interactively-by-user");
         }
 
-        await session2.destroy();
+        await session2.disconnect();
     });
 
     it(
@@ -238,7 +238,7 @@ describe("Multi-client broadcast", async () => {
             expect(response?.data.content).toContain("CITY_FOR_US");
             expect(response?.data.content).toContain("CURRENCY_FOR_US");
 
-            await session2.destroy();
+            await session2.disconnect();
         }
     );