Skip to content

Commit 955c094

Browse files
fixes Protocol version negotiation incorrect #41 (fixed tests)
1 parent 036946d commit 955c094

File tree

1 file changed

+7
-16
lines changed

1 file changed

+7
-16
lines changed

tests/Server/MCPServerTest.php

Lines changed: 7 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -558,21 +558,12 @@ public function test_initialize_throws_exception_for_unsupported_protocol_versio
558558
$unsupportedVersion = '2025-12-01';
559559
$initializeData = new InitializeData('2.0', ['mock-capability' => true], $unsupportedVersion);
560560

561-
// Act & Assert
562-
try {
563-
$server->initialize($initializeData);
564-
$this->fail('Expected JsonRpcErrorException not thrown');
565-
} catch (JsonRpcErrorException $e) {
566-
$this->assertEquals(-32602, $e->getJsonRpcErrorCode());
567-
$this->assertEquals('Unsupported protocol version', $e->getMessage());
568-
569-
$errorData = $e->getErrorData();
570-
$this->assertIsArray($errorData);
571-
$this->assertArrayHasKey('supported', $errorData);
572-
$this->assertArrayHasKey('requested', $errorData);
573-
$this->assertEquals($unsupportedVersion, $errorData['requested']);
574-
$this->assertContains(MCPProtocolInterface::PROTOCOL_VERSION_SSE, $errorData['supported']);
575-
$this->assertContains(MCPProtocolInterface::PROTOCOL_VERSION_STREAMABE_HTTP, $errorData['supported']);
576-
}
561+
// Act
562+
$initializeResource = $server->initialize($initializeData);
563+
564+
// Assert
565+
$this->assertInstanceOf(InitializeResource::class, $initializeResource);
566+
$this->assertEquals(MCPProtocolInterface::PROTOCOL_VERSION_STREAMABE_HTTP, $initializeResource->protocolVersion);
567+
$this->assertEquals($serverInfo, $initializeResource->serverInfo);
577568
}
578569
}

0 commit comments

Comments
 (0)