Alle Kommandos via mqtt abbilden #8
Open
Conversation
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
STX messages (sensor data) can be interleaved with command responses, similar to MU/MS messages. The controller must skip these messages and continue waiting for the actual command response. The STX message is parsed to ensure sensor data is handled correctly while waiting.
- Fix busy loop in `mock_transport` fixture by adding `asyncio.sleep` to `readline` side effect. - Fix `test_initialize_retry_logic` assertion to account for 'XQ' command sent during initialization. - Fix `test_stx_message_bypasses_command_response` by manually starting controller tasks and updating mock response. - Fix `test_send_command_with_response` by starting the missing `_parser_task`, updating mock to avoid StopIteration, and increasing timeout. - Fix `test_message_callback` mock setup to yield message once and then None. - Fix `test_send_command_fire_and_forget` cleanup logic to remove undefined `reader_task` cancellation.
This commit introduces new functionality to process Signalduino commands received via MQTT, significantly enhancing the integration capabilities of PySignalduino. Key changes include: - Extended the command list in `signalduino/commands.py` to include new MQTT-related commands and specific CC1101 register commands. - Implemented and integrated the MQTT publisher logic within `signalduino/controller.py`. - Introduced and adapted unit tests in `tests/test_mqtt_commands.py` and other test files to ensure full coverage of the new MQTT command processing.
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #8 +/- ##
==========================================
+ Coverage 79.02% 80.57% +1.55%
==========================================
Files 55 56 +1
Lines 5177 5014 -163
Branches 628 590 -38
==========================================
- Hits 4091 4040 -51
+ Misses 863 767 -96
+ Partials 223 207 -16
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.