Skip to content

Commit 60f5524

Browse files
committed
Fix test for empty registry case
The test case "empty registry returns success" was falling into the default case which expected at least one server. Added specific handling for the empty registry case to properly assert an empty server list. Fixes #426 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]> :house: Remote-Dev: homespace
1 parent 200dafd commit 60f5524

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

internal/api/handlers/v0/servers_test.go

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -330,6 +330,8 @@ func TestServersListEndpoint(t *testing.T) {
330330
case "combined search and updated_since filter":
331331
assert.Len(t, resp.Servers, 1, "Expected one server matching both filters")
332332
assert.Contains(t, resp.Servers[0].Name, "combined", "Server name should contain search term")
333+
case "empty registry returns success":
334+
assert.Empty(t, resp.Servers, "Expected empty server list for empty registry")
333335
case "comprehensive query with all parameters":
334336
// Should return only latest versions of servers matching "filesystem" search term
335337
// Expected: 2 servers (filesystem-server v2.0.0 and filesystem-tools v3.0.0)

0 commit comments

Comments
 (0)