Add test for requestMessage invoking transport processMessage

Removed unused import and redundant exception handling in MCPProtocol. Added a test to verify that requestMessage correctly invokes the transport's processMessage method. This improves test coverage and simplifies the error-handling logic.

Author: klapaudius

src/Protocol/MCPProtocol.php

@@ -149,8 +149,6 @@ public function handleMessage(string $clientId, array $message): void
             throw new JsonRpcErrorException(message: 'Invalid Request: Message format not recognized', code: JsonRpcErrorCode::INVALID_REQUEST, data: $message);
         } catch (JsonRpcErrorException $e) {
             $this->pushMessage(clientId: $clientId, message: new JsonRpcErrorResource(exception: $e, id: $messageId));
-        } catch (Exception $e) {
-            throw $e;
         }
     }
 

tests/Protocol/MCPProtocolTest.php

@@ -6,7 +6,6 @@
 use KLP\KlpMcpServer\Protocol\Handlers\NotificationHandler;
 use KLP\KlpMcpServer\Protocol\MCPProtocol;
 use KLP\KlpMcpServer\Transports\SseTransportInterface;
-use KLP\KlpMcpServer\Transports\TransportInterface;
 use PHPUnit\Framework\Attributes\Small;
 use PHPUnit\Framework\MockObject\MockObject;
 use PHPUnit\Framework\TestCase;
@@ -386,4 +385,20 @@ public function test_handle_message_handles_notification_handler_throw_exception
 
         $this->mcpProtocol->handleMessage($clientId, $validNotificationMessage);
     }
+
+    /**
+     * Test that requestMessage invokes processMessage on the transport
+     */
+    public function test_request_message_invokes_transport_process_message(): void
+    {
+        $clientId = 'test_client';
+        $message = ['key' => 'value'];
+
+        $this->mockTransport
+            ->expects($this->once())
+            ->method('processMessage')
+            ->with($this->equalTo($clientId), $this->equalTo($message));
+
+        $this->mcpProtocol->requestMessage($clientId, $message);
+    }
 }