Skip to content

Commit ad3c1b1

Browse files
committed
Fix test for empty resources response
Update test_resources_returns_empty_array_when_no_resources to use correct mock response format with empty resources array instead of empty result object. Also reorder test to group with other resources tests. Follow up: modelcontextprotocol#160
1 parent c510600 commit ad3c1b1

File tree

1 file changed

+12
-12
lines changed

1 file changed

+12
-12
lines changed

test/mcp/client_test.rb

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -84,6 +84,18 @@ def test_resources_sends_request_to_transport_and_returns_resources_array
8484
assert_equal("file:///path/to/resource2", resources.last["uri"])
8585
end
8686

87+
def test_resources_returns_empty_array_when_no_resources
88+
transport = mock
89+
mock_response = { "result" => { "resources" => [] } }
90+
91+
transport.expects(:send_request).returns(mock_response).once
92+
93+
client = Client.new(transport: transport)
94+
resources = client.resources
95+
96+
assert_empty(resources)
97+
end
98+
8799
def test_read_resource_sends_request_to_transport_and_returns_contents
88100
transport = mock
89101
uri = "file:///path/to/resource.txt"
@@ -129,17 +141,5 @@ def test_read_resource_returns_empty_array_when_no_contents
129141

130142
assert_empty(contents)
131143
end
132-
133-
def test_resources_returns_empty_array_when_no_resources
134-
transport = mock
135-
mock_response = { "result" => {} }
136-
137-
transport.expects(:send_request).returns(mock_response).once
138-
139-
client = Client.new(transport: transport)
140-
resources = client.resources
141-
142-
assert_empty(resources)
143-
end
144144
end
145145
end

0 commit comments

Comments
 (0)