Skip to content

Add connection manager tests#5

Open
quiloos39 wants to merge 1 commit intomainfrom
add-connection-manager-tests
Open

Add connection manager tests#5
quiloos39 wants to merge 1 commit intomainfrom
add-connection-manager-tests

Conversation

@quiloos39
Copy link
Owner

@quiloos39 quiloos39 commented Feb 16, 2026

Summary

  • Add 12 vitest tests for the global connection map in src/connections.ts
  • Covers store/retrieve, delete, ready state tracking, multiple connections, same-ID replacement, iteration, cleanupAllConnections(), MAX_OUTPUT_BYTES constant, and StoredConnection interface shape

Test plan

  • Run npx vitest run tests/connection-manager.test.ts and verify all 12 tests pass
  • Confirm MAX_OUTPUT_BYTES matches the value in src/connections.ts

@quiloos39 quiloos39 changed the title Add connection manager tests\n\nAdds tests for the global connection map including storing, retrieving, removing connections, ready state tracking, cleanup, and the MAX_OUTPUT_BYTES constant. Add connection manager tests Feb 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant