Refactor WebSocket handling and improve test robustness #498
Annotations
1 error
|
Check Commit Messages
You have commit messages with errors
⧗ input: debug(nostr-java-client): add logging to diagnose timeout configuration
Added debug and error logging to StandardWebSocketClient to help diagnose
timeout configuration issues:
- Log actual timeout values when StandardWebSocketClient is created
- Log timeout values being used when waiting for relay response
- Include configured timeout values in error message when timeout occurs
This helps verify that:
1. The constructor is being called with correct timeout values
2. The timeout values are being used in await() calls
3. The actual timeout duration matches the configured value
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
✖ type must be one of [build, chore, ci, docs, feat, fix, perf, refactor, revert, style, test] [type-enum]
✖ found 1 problems, 0 warnings
ⓘ Get help: https://github.com/conventional-changelog/commitlint/#what-is-commitlint
⧗ input: refactor(test): switch to strfry relay and improve integration test robustness
Replaced nostr-rs-relay with dockurr/strfry for integration tests, addressing timeout and crash issues. Enhanced WebSocket client retry logic, added configurable relay port, and included additional wait for indexing. Updated test setup for dynamic relay configuration and removed unused response comparison logic.
✖ body's lines must not be longer than 100 characters [body-max-line-length]
✖ found 1 problems, 0 warnings
ⓘ Get help: https://github.com/conventional-changelog/commitlint/#what-is-commitlint
⧗ input: docs: update integration test documentation with resolved status and strfry fixes
Marked integration tests as resolved with all 24 passing using strfry relay. Documented issues with nostr-rs-relay and provided solutions, including custom strfry configuration, indexing delay, and relay-agnostic assertions. Enhanced troubleshooting section with detailed resolutions for common test issues.
✖ body's lines must not be longer than 100 characters [body-max-line-length]
✖ found 1 problems, 0 warnings
ⓘ Get help: https://github.com/conventional-changelog/commitlint/#what-is-commitlint
|
Loading