Set Matter battery sensors as diagnostic#95794
Conversation
home-assistant
Bot
added
cla-signed
has-tests
integration: matter
new-feature
small-pr
|
Hey there @home-assistant/matter, mind taking a look at this pull request as it has been labeled with an integration ( Code owner commandsCode owners of
|
|
|
||
|
|
||
| @pytest.fixture(name="door_lock") | ||
| async def door_lock_fixture( |
There was a problem hiding this comment.
I moved this fixture as it can be used for more tests than the lock tests.
| def binary_sensor_platform(): | ||
| """Load only the binary sensor platform.""" | ||
| with patch( | ||
| "homeassistant.components.matter.discovery.DISCOVERY_SCHEMAS", |
There was a problem hiding this comment.
I had to patch this to make sure the event callback is for the binary sensor entity as the lock fixture creates more than one entity otherwise.
|
Didn't we have the eve door sensor already in the fixtures ? |
I didn't find it. I'll doublecheck. |
No, you're right. I created a mock dump for the contact-sensor back then so maybe just replace that with the real dump now |
2 participants
Breaking change
Proposed change
Type of change
Additional information
Checklist
black --fast homeassistant tests)If user exposed functionality or configuration variables are added/changed:
If the code communicates with devices, web services, or third-party tools:
Updated and included derived files by running:
python3 -m script.hassfest.requirements_all.txt.Updated by running
python3 -m script.gen_requirements_all..coveragerc.To help with the load of incoming pull requests: