test: align client test protocol names and assertions with downstream enum rename

Author: anubhav756

packages/toolbox-core/src/toolbox_core/mcp/v20260618/mcp.ts

@@ -55,7 +55,8 @@ export class McpHttpTransportV20260618 extends McpHttpTransportBase {
       const serverSupported = ((
         (errVal as Record<string, unknown>).data as Record<string, unknown>
       )?.supported || []) as string[];
-      const clientSupported = this.supportedProtocols || getSupportedMcpVersions();
+      const clientSupported =
+        this.supportedProtocols || getSupportedMcpVersions();
       const mutuallySupported = clientSupported.filter(v =>
         serverSupported.includes(v),
       );
@@ -88,12 +89,15 @@ export class McpHttpTransportV20260618 extends McpHttpTransportBase {
 
     if (isLegacyError) {
       // Cascading Fallback
-      const clientSupported = this.supportedProtocols || getSupportedMcpVersions();
+      const clientSupported =
+        this.supportedProtocols || getSupportedMcpVersions();
       const currentIdx = clientSupported.indexOf(
         this._protocolVersion as Protocol,
       );
       if (currentIdx !== -1 && currentIdx + 1 < clientSupported.length) {
-        throw new ProtocolNegotiationError(clientSupported[currentIdx + 1] as Protocol);
+        throw new ProtocolNegotiationError(
+          clientSupported[currentIdx + 1] as Protocol,
+        );
       } else {
         throw new Error(
           "Server threw 'invalid protocol version' but no fallback versions remain in the user's supported protocols array.",
@@ -184,7 +188,6 @@ export class McpHttpTransportV20260618 extends McpHttpTransportBase {
           const err = errResult.data.error;
           message = `MCP request failed with code ${err.code}: ${err.message}`;
           code = String(err.code);
-
         }
 
         throw new AxiosError(

packages/toolbox-core/test/mcp/test.v20260618.ts

@@ -563,7 +563,6 @@ describe('McpHttpTransportV20260618', () => {
       await expect(transport.toolsList()).rejects.toThrow(
         ProtocolNegotiationError,
       );
->>>>>>> df7ce2f (test: resolve unit test and lint compilation issues across downstream protocol branches)
       errorSpy.mockRestore();
     });
 

packages/toolbox-core/test/test.client.ts

@@ -32,6 +32,7 @@ import {ProtocolNegotiationError} from '../src/toolbox_core/errorUtils.js';
 class MockTransport implements ITransport {
   readonly baseUrl: string;
   protocolVersion = Protocol.MCP;
+  supportedProtocols?: Protocol[];
   toolGet: jest.MockedFunction<ITransport['toolGet']>;
   toolsList: jest.MockedFunction<ITransport['toolsList']>;
   toolInvoke: jest.MockedFunction<ITransport['toolInvoke']>;
@@ -249,11 +250,7 @@ describe('ToolboxClient', () => {
         '2025-11-25',
         'DRAFT-2026-v1',
       ]);
-      // The array is sorted newer to older by getSupportedMcpVersions order
-      // so DRAFT-2026-v1 is first.
-      const mockInstance = (McpHttpTransportV20260618 as jest.Mock).mock
-        .instances[0] as unknown as Record<string, unknown>;
-      expect(mockInstance.supportedProtocols).toEqual([
+      expect(mockTransport.supportedProtocols).toEqual([
         'DRAFT-2026-v1',
         '2025-11-25',
       ]);
@@ -677,7 +674,7 @@ describe('ToolboxClient', () => {
         testBaseUrl,
         undefined,
         undefined,
-        Protocol.MCP_DRAFT_2026_v1,
+        Protocol.MCP_DRAFT,
       );
 
       const tool = await client.loadTool('testTool');
@@ -720,7 +717,7 @@ describe('ToolboxClient', () => {
         testBaseUrl,
         undefined,
         undefined,
-        Protocol.MCP_DRAFT_2026_v1,
+        Protocol.MCP_DRAFT,
       );
 
       const tools = await client.loadToolset('set');