modelcontextprotocol/php-sdk
Watch
1
0
Fork
You've already forked php-sdk
 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Feat: Improve invalid resource template messages (#245)

Browse source
This commit is contained in:
Sule-Balogun Olanrewaju 2026-02-13 20:46:03 +00:00 • committed by GitHub
commit aafac3ddf5
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 90 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

4
src/Schema/ResourceTemplate.php
View file

@ -59,10 +59,10 @@ class ResourceTemplate implements \JsonSerializable
public readonly ?array $meta = null,
) {
if (!preg_match(self::RESOURCE_NAME_PATTERN, $name)) {
throw new InvalidArgumentException('Invalid resource name: must contain only alphanumeric characters, underscores, and hyphens.');
throw new InvalidArgumentException(\sprintf('Invalid resource name "%s": must contain only alphanumeric characters, underscores, and hyphens.', $name));
}
if (!preg_match(self::URI_TEMPLATE_PATTERN, $uriTemplate)) {
throw new InvalidArgumentException('Invalid URI template: must be a valid URI template with at least one placeholder.');
throw new InvalidArgumentException(\sprintf('Invalid URI template : "%s" must be a valid URI template with at least one placeholder.', $uriTemplate));
}
}

88
tests/Unit/Schema/ResourceTemplateTest.php Normal file
View file

@ -0,0 +1,88 @@
<?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 Mcp\Tests\Unit\Schema;
use Mcp\Exception\InvalidArgumentException;
use Mcp\Schema\ResourceTemplate;
use PHPUnit\Framework\Attributes\DataProvider;
use PHPUnit\Framework\TestCase;
class ResourceTemplateTest extends TestCase
{
private const VALID_URI = 'https://example.com/list-books/{id}';
public function testConstructorValid(): void
{
$uri = self::VALID_URI;
$resource = new ResourceTemplate(
uriTemplate: $uri,
name: 'list-books',
);
$this->assertInstanceOf(ResourceTemplate::class, $resource);
$this->assertSame($uri, $resource->uriTemplate);
}
public function testConstructorInvalid(): void
{
$uri = '/list-books';
$this->expectException(InvalidArgumentException::class);
$this->expectExceptionMessage('Invalid URI template : "/list-books" must be a valid URI template with at least one placeholder.');
$resource = new ResourceTemplate(
uriTemplate: $uri,
name: 'list-books',
);
}
public function testFromArrayValid(): void
{
$resource = ResourceTemplate::fromArray([
'uriTemplate' => self::VALID_URI,
'name' => 'list-books',
]);
$this->assertInstanceOf(ResourceTemplate::class, $resource);
$this->assertSame(self::VALID_URI, $resource->uriTemplate);
$this->assertSame('list-books', $resource->name);
$this->assertNull($resource->description);
$this->assertNull($resource->meta);
}
#[DataProvider('provideInvalidResources')]
public function testFromArrayInvalid(array $input, string $expectedExceptionMessage): void
{
$this->expectException(InvalidArgumentException::class);
$this->expectExceptionMessage($expectedExceptionMessage);
ResourceTemplate::fromArray($input);
}
public static function provideInvalidResources(): iterable
{
yield 'missing uri' => [[], 'Invalid or missing "uriTemplate" in ResourceTemplate data.'];
yield 'missing name' => [
['uriTemplate' => self::VALID_URI],
'Invalid or missing "name" in ResourceTemplate data.',
];
yield 'meta' => [
[
'uriTemplate' => self::VALID_URI,
'name' => 'list-books',
'_meta' => 'foo',
],
'Invalid "_meta" in ResourceTemplate data.',
];
}
}