Refactor tests and improve coverage configuration.

Replaced `TransportInterface` with `SseTransportInterface` in `MCPProtocolTest` for better alignment. Added new test cases to validate `handleMessage` and `ToolRepository` behavior. Updated PHPUnit configuration to exclude `routes.php` from coverage reports.

Author: klapaudius

phpunit.xml.dist

@@ -27,6 +27,9 @@
         <include>
             <directory suffix=".php">./src</directory>
         </include>
+        <exclude>
+            <file>src/Resources/config/routes.php</file>
+        </exclude>
     </source>
     <coverage>
         <report>

tests/Protocol/MCPProtocolTest.php

@@ -5,6 +5,7 @@
 use KLP\KlpMcpServer\Exceptions\ToolParamsValidatorException;
 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;
@@ -13,13 +14,13 @@
 #[Small]
 class MCPProtocolTest extends TestCase
 {
-    private TransportInterface|MockObject $mockTransport;
+    private SseTransportInterface|MockObject $mockTransport;
 
     private MCPProtocol $mcpProtocol;
 
     protected function setUp(): void
     {
-        $this->mockTransport = $this->createMock(TransportInterface::class);
+        $this->mockTransport = $this->createMock(SseTransportInterface::class);
         $this->mcpProtocol = new MCPProtocol($this->mockTransport);
     }
 
@@ -287,6 +288,19 @@ public function test_handle_message_handles_no_method(): void
         $this->mcpProtocol->handleMessage($clientId, $noMethodMessage);
     }
 
+    public function test_handle_message_handles_ping_request(): void
+    {
+        $clientId = 'client_1';
+        // response to a ping request from client
+        $noMethodMessage = ['jsonrpc' => '2.0', 'id' => 1, 'result' => []];
+
+        $this->mockTransport
+            ->expects($this->never())
+            ->method('pushMessage');
+
+        $this->mcpProtocol->handleMessage($clientId, $noMethodMessage);
+    }
+
     public function test_handle_message_handles_invalid_params(): void
     {
         $clientId = 'client_1';

tests/Services/ToolService/ToolRepositoryTest.php

@@ -5,13 +5,15 @@
 use InvalidArgumentException;
 use KLP\KlpMcpServer\Services\ToolService\ToolInterface;
 use KLP\KlpMcpServer\Services\ToolService\ToolRepository;
+use PHPUnit\Framework\Attributes\Small;
 use PHPUnit\Framework\MockObject\MockObject;
 use PHPUnit\Framework\TestCase;
 use stdClass;
 use Symfony\Component\DependencyInjection\ContainerInterface;
 use Symfony\Component\DependencyInjection\Exception\ServiceCircularReferenceException;
 use Symfony\Component\DependencyInjection\Exception\ServiceNotFoundException;
 
+#[Small]
 class ToolRepositoryTest extends TestCase
 {
     private ContainerInterface|MockObject $container;
@@ -47,6 +49,48 @@ public function test_register_many_with_valid_tool_instances(): void
         $this->assertSame($tool2, $tools['tool2']);
     }
 
+    /**
+     * Tests the registration of multiple valid tool instances in the tool repository.
+     *
+     * Validates that the tool repository correctly registers multiple tool instances
+     * and ensures that the tools can be retrieved and matched against their original instances.
+     */
+    public function test_register_many_is_called_on_constructor(): void
+    {
+        $tool1 = $this->createMock(ToolInterface::class);
+        $tool2 = $this->createMock(ToolInterface::class);
+
+        $tool1->method('getName')->willReturn('tool1');
+        $tool2->method('getName')->willReturn('tool2');
+        $invocations = [ 'tool1', 'tool2' ];
+        $this->container
+            ->expects($matcher = $this->exactly(count($invocations)))
+            ->method('get')
+            ->with($this->callback(function ($toolClass) use ($invocations, $matcher) {
+                $this->assertEquals($invocations[$matcher->numberOfInvocations() - 1], $toolClass);
+                return true;
+            }))
+            ->willReturnOnConsecutiveCalls(
+                $tool1,
+                $tool2
+            );
+        $this->container
+            ->expects($this->once())
+            ->method('getParameter')
+            ->with('klp_mcp_server.tools')
+            ->willReturn([ 'tool1', 'tool2' ]);
+
+        $toolRepository = new ToolRepository($this->container);
+
+        $tools = $toolRepository->getTools();
+
+        $this->assertCount(2, $tools);
+        $this->assertSame($tool1, $tools['tool1']);
+        $this->assertSame($tool2, $tools['tool2']);
+        $this->assertSame($tool1, $toolRepository->getTool('tool1'));
+
+    }
+
     /**
      * Tests the registration of multiple tools using valid tool class names.
      *