Split testsuite for unit and inspector based tests (#82)

Author: chr-hertel

.github/workflows/pipeline.yaml

@@ -6,7 +6,7 @@ permissions:
   pull-requests: write
 
 jobs:
-  tests:
+  unit:
     runs-on: ubuntu-latest
     strategy:
       matrix:
@@ -22,16 +22,36 @@ jobs:
           php-version: ${{ matrix.php }}
           coverage: "none"
 
-      - name: Composer Validation
-        run: composer validate --strict
-
       - name: Install Composer
         uses: "ramsey/composer-install@v3"
         with:
           dependency-versions: "${{ matrix.dependencies }}"
 
       - name: Tests
-        run: vendor/bin/phpunit --exclude-group inspector
+        run: vendor/bin/phpunit --testsuite=unit
+
+  inspector:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v5
+
+      - name: Setup PHP
+        uses: shivammathur/setup-php@v2
+        with:
+          php-version: '8.4'
+          coverage: "none"
+
+      - name: Setup Node
+        uses: actions/setup-node@v5
+        with:
+          node-version: '22'
+
+      - name: Install Composer
+        uses: "ramsey/composer-install@v3"
+
+      - name: Tests
+        run: vendor/bin/phpunit --testsuite=inspector
 
   qa:
     runs-on: ubuntu-latest

.gitignore

@@ -1,6 +1,7 @@
 .phpunit.cache
 .php-cs-fixer.cache
 composer.lock
+coverage
 vendor
 examples/**/dev.log
 examples/**/sessions

Makefile

@@ -15,8 +15,14 @@ phpstan:
 tests:
 	vendor/bin/phpunit
 
+unit-tests:
+	vendor/bin/phpunit --testsuite=unit
+
+inspector-tests:
+	vendor/bin/phpunit --testsuite=inspector
+
 coverage:
-	XDEBUG_MODE=coverage vendor/bin/phpunit --coverage-html=coverage
+	XDEBUG_MODE=coverage vendor/bin/phpunit --testsuite=unit --coverage-html=coverage
 
 ci: ci-stable
 

phpstan.dist.neon

@@ -9,8 +9,8 @@ parameters:
         - tests/
     excludePaths:
         - examples/cli/vendor/* (?)
-        - tests/Capability/Discovery/Fixtures/
-        - tests/Capability/Discovery/SchemaGeneratorFixture.php
+        - tests/Unit/Capability/Discovery/Fixtures/
+        - tests/Unit/Capability/Discovery/SchemaGeneratorFixture.php
     treatPhpDocTypesAsCertain: false
     ignoreErrors:
         -

phpunit.xml.dist

@@ -9,8 +9,11 @@
          failOnWarning="true">
 
     <testsuites>
-        <testsuite name="default">
-            <directory>tests</directory>
+        <testsuite name="unit">
+            <directory>tests/Unit</directory>
+        </testsuite>
+        <testsuite name="inspector">
+            <directory>tests/Inspector</directory>
         </testsuite>
     </testsuites>
 

tests/Example/InspectorSnapshotTestCase.php renamed to tests/Inspector/InspectorSnapshotTestCase.php

@@ -9,7 +9,7 @@
  * file that was distributed with this source code.
  */
 
-namespace Mcp\Tests\Example;
+namespace Mcp\Tests\Inspector;
 
 use PHPUnit\Framework\Attributes\DataProvider;
 use PHPUnit\Framework\TestCase;

tests/Inspector/ManualStdioExampleTest.php

@@ -0,0 +1,29 @@
+<?php
+
+/*
+ * This file is part of the official PHP MCP SDK.
+ *
+ * A collaboration between Symfony and the PHP Foundation.
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace Inspector;
+
+use Mcp\Tests\Inspector\InspectorSnapshotTestCase;
+
+final class ManualStdioExampleTest extends InspectorSnapshotTestCase
+{
+    public static function provideMethods(): array
+    {
+        return [
+            ...parent::provideListMethods(),
+        ];
+    }
+
+    protected function getServerScript(): string
+    {
+        return \dirname(__DIR__, 2).'/examples/03-manual-registration-stdio/server.php';
+    }
+}

tests/Example/StdioCalculatorExampleTest.php renamed to tests/Inspector/StdioCalculatorExampleTest.php

@@ -9,11 +9,8 @@
  * file that was distributed with this source code.
  */
 
-namespace Mcp\Tests\Example;
+namespace Mcp\Tests\Inspector;
 
-use PHPUnit\Framework\Attributes\Group;
-
-#[Group('inspector')]
 final class StdioCalculatorExampleTest extends InspectorSnapshotTestCase
 {
     public static function provideMethods(): array

tests/Inspector/snapshots/ManualStdioExampleTest-prompts_list.json

@@ -0,0 +1,15 @@
+{
+  "prompts": [
+    {
+      "name": "personalized_greeting",
+      "description": "A manually registered prompt template.",
+      "arguments": [
+        {
+          "name": "userName",
+          "description": "the name of the user",
+          "required": true
+        }
+      ]
+    }
+  ]
+}

tests/Inspector/snapshots/ManualStdioExampleTest-resources_list.json

@@ -0,0 +1,10 @@
+{
+  "resources": [
+    {
+      "name": "application_version",
+      "uri": "app://version",
+      "description": "A manually registered resource providing app version.",
+      "mimeType": "text/plain"
+    }
+  ]
+}