Skip to content

✅ Integration tests & polishing of the existing codebase#48

Merged
roma-glushko merged 13 commits intomainfrom
fix-order-params
Feb 15, 2026
Merged

✅ Integration tests & polishing of the existing codebase#48
roma-glushko merged 13 commits intomainfrom
fix-order-params

Conversation

@roma-glushko
Copy link
Owner

@roma-glushko roma-glushko commented Feb 14, 2026

Changes

  • Covered functionality by integration tests
  • Exposed a new param event_buffer_size to control size of the internal Notifier buffer (it's important as we will start missing events on buffer overflow).
  • Cleaned NotifierMock.__init__() params
  • Handle properly Mutex.lock exceptions. Added more debug statements in event of potential event misses, so misses are not silent
  • Stop leaking tokio tasks on event draining
  • Fixed rust compilation warnings about dead code and non_local_definitions
  • Run tests on PRs via Github Actions

@roma-glushko roma-glushko self-assigned this Feb 14, 2026
@roma-glushko roma-glushko changed the title Fix order params Integration tests & polishing of the existing codebase Feb 14, 2026
@roma-glushko roma-glushko added the enhancement New feature or request label Feb 14, 2026
@roma-glushko roma-glushko changed the title Integration tests & polishing of the existing codebase ✅ Integration tests & polishing of the existing codebase Feb 15, 2026
@roma-glushko roma-glushko merged commit ca08ab2 into main Feb 15, 2026
16 checks passed
@roma-glushko roma-glushko deleted the fix-order-params branch February 15, 2026 00:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant