Merged
Conversation
…RTC manager - Implement TestLoginConfirmPasswordGetter to verify confirm password retrieval. - Add TestAddAndRemoveVoiceMember to ensure proper member management in voice daemon. - Introduce TestPTTControllerRun to validate PTT controller behavior. - Create voice IPC tests including connection handling and message sending. - Add audio playback tests for buffer management and safety on nil instances. - Implement IPC tests for message encoding/decoding and connection lifecycle. - Add WebRTC manager tests for peer validation, offer/answer flow, and ICE candidate handling.
…close - Introduced unit tests for audio capture and playback functionalities, ensuring proper handling of nil and zero values during close operations. - Enhanced the Capture struct with a sync.Once to ensure safe closure of audio devices. - Added tests for hotkey parsing and server invite repository, covering various scenarios including validation and success cases. - Implemented mock database interactions for user, device, broadcast, channel, and server invite repositories to validate SQL operations without a real database.
- Enhance hotkey parsing tests with additional cases for capslock aliases and error scenarios. - Introduce tests for PTT controller in portal and hotkey modes, including validation error handling. - Add tests for voice stats log loop to ensure counters are flushed correctly on tick. - Implement tests for WebSocket client connection handling and read loop cancellation. - Expand channel service tests to verify default service initialization. - Add error handling tests for user and device repositories in PostgreSQL. - Introduce tests for voice members management in the voice daemon, ensuring proper member handling and broadcasting. - Create tests for CPU usage logging to validate context cancellation behavior.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.