Skip to content

Commit b8ab280

Browse files
committed
Adding unit tests
1 parent 38d8ac1 commit b8ab280

File tree

2 files changed

+14
-3
lines changed

2 files changed

+14
-3
lines changed

tests/mcp/helpers.py

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -123,26 +123,28 @@ async def get_prompt(
123123
return GetPromptResult(description=f"Fake prompt: {name}", messages=[message])
124124

125125
async def list_resources(self, run_context=None, agent=None) -> ListResourcesResult:
126-
"""Return empty list of resources for fake server"""
126+
"""Return empty list of resources for the fake server"""
127127
return ListResourcesResult(resources=self.resources)
128128

129129
async def list_resource_templates(self, run_context=None, agent=None) \
130130
-> ListResourceTemplatesResult:
131-
"""Return empty list of resources templates for fake server"""
131+
"""Return empty list of resources templates for the fake server"""
132132
return ListResourceTemplatesResult(resourceTemplates=self.resources_templates)
133133

134134
async def read_resource(self, uri: AnyUrl) -> ReadResourceResult:
135-
"""Return a fake resource read for fake server"""
135+
"""Return a fake resource read for the fake server"""
136136
for resource in self.resources:
137137
if resource.uri == uri:
138138
return ReadResourceResult(**resource.model_dump(), contents=[])
139139

140140
raise KeyError
141141

142142
def add_resource(self, uri: AnyUrl, name: str, description: str | None = None):
143+
"""Add a resource to the fake server"""
143144
self.resources.append(Resource(uri=uri, description=description, name=name))
144145

145146
def add_resource_template(self, uri: str, name: str, description: str | None = None):
147+
"""Add a resource template to the fake server"""
146148
self.resources_templates.append(
147149
ResourceTemplate(uriTemplate=uri, description=description, name=name)
148150
)

tests/mcp/test_resources_server.py

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -34,3 +34,12 @@ async def test_read_resource():
3434
server.add_resource(AnyUrl("docs://api/reference"), name="Docs Search")
3535

3636
await server.read_resource(AnyUrl("docs://api/reference"))
37+
38+
@pytest.mark.asyncio
39+
async def test_read_resource_not_found():
40+
"""Test getting a resource that doesn't exist"""
41+
server = FakeMCPServer()
42+
43+
uri = AnyUrl("docs://api/reference")
44+
with pytest.raises(ValueError, match=f"Resource {uri} not found"):
45+
await server.read_resource(uri)

0 commit comments

Comments
 (0)