chore: clean up tests from previous pr

[Lash-L]

Clean up tests from review on #656

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

Files with missing lines	Coverage Δ
tests/devices/test_local_channel.py	95.55% <100.00%> (+0.03%)	⬆️
tests/devices/test_mqtt_channel.py	97.12% <100.00%> (+0.02%)	⬆️
tests/test_protocol.py	100.00% <100.00%> (ø)

... and 15 files with indirect coverage changes

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[Copilot]

Pull request overview

This PR consolidates and cleans up test files from PR #656. The main changes involve moving decoder/padding tests from a device-specific test file to a more general protocol test file, and improving logging test assertions in multiple test files.

• Tests for decoder padding behavior moved from tests/devices/test_local_decoder_padding.py to tests/test_protocol.py
• Improved logging test assertions by using caplog.at_level() context manager and simplifying record filtering
• Consolidated multiple test cases into a parametrized test for cleaner code organization

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File	Description
tests/test_protocol.py	New file consolidating decoder padding tests with parametrization for different garbage byte scenarios
tests/devices/test_mqtt_channel.py	Improved logging test using caplog.at_level() context manager and simplified record filtering
tests/devices/test_local_decoder_padding.py	Deleted - tests moved to tests/test_protocol.py
tests/devices/test_local_channel.py	Added missing logging import and improved logging test pattern consistent with mqtt_channel changes

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

The parametrize decorator has incorrect syntax. The first argument should be either a plain string "garbage" or a tuple with a trailing comma ("garbage",). Currently, ("garbage") without a trailing comma is just a string in parentheses, not a tuple. This should be changed to either:

• "garbage" (recommended for single parameter)
• ("garbage",) (tuple with trailing comma)