Skip to content

fix: update test expectations to match actual error messages #592

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
lpcox merged 1 commit into from
Feb 2, 2026
+3 −3
Merged

fix: update test expectations to match actual error messages #592

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions internal/server/register_tools_from_backend_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -239,8 +239,8 @@ func TestRegisterToolsFromBackend_BackendError(t *testing.T) {
// Attempt to register tools should fail with backend error
err = us.registerToolsFromBackend("error-backend")
require.Error(err, "Should fail when backend returns error")
require.Contains(err.Error(), "backend error listing tools", "Error should mention backend error")
require.Contains(err.Error(), "code=-32603", "Error should include error code")
require.Contains(err.Error(), "failed to list tools", "Error should mention failed to list tools")
require.Contains(err.Error(), "unable to list tools", "Error should include backend error message")
}

// TestRegisterToolsFromBackend_InvalidJSON tests that registration
Expand Down Expand Up @@ -310,7 +310,7 @@ func TestRegisterToolsFromBackend_InvalidJSON(t *testing.T) {
// Attempt to register tools should fail with JSON parsing error
err = us.registerToolsFromBackend("invalid-json-backend")
require.Error(err, "Should fail to parse invalid JSON")
require.Contains(err.Error(), "failed to parse tools", "Error should mention parsing failure")
require.Contains(err.Error(), "cannot unmarshal", "Error should mention JSON unmarshal failure")
}

// TestRegisterToolsFromBackend_EmptyToolList tests that registration
Expand Down