Commit 23fa84e

authored

Verify that Ecovacs integration is setup without any errors in the tests (home-assistant#152447)

1 parent 7f13141 commit 23fa84eCopy full SHA for 23fa84e

File tree

+10

-1

lines changed

+10

-1

lines changed

Lines changed: 9 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,7 @@`
`1`	`1`	`"""Common fixtures for the Ecovacs tests."""`
`2`	`2`
`3`	`3`	`from collections.abc import AsyncGenerator, Generator`
	`4`	`+import logging`
`4`	`5`	`from typing import Any`
`5`	`6`	`from unittest.mock import AsyncMock, Mock, patch`
`6`	`7`
`@@ -134,6 +135,7 @@ def mock_vacbot(device_fixture: str) -> Generator[Mock]:`
`134`	`135`	`vacbot.lifespanEvents = EventEmitter()`
`135`	`136`	`vacbot.errorEvents = EventEmitter()`
`136`	`137`	`vacbot.battery_status = None`
	`138`	`+ vacbot.charge_status = None`
`137`	`139`	`vacbot.fan_speed = None`
`138`	`140`	`vacbot.components = {}`
`139`	`141`	`yield vacbot`
`@@ -159,6 +161,7 @@ def platforms() -> Platform \| list[Platform]:`
`159`	`161`	`@pytest.fixture`
`160`	`162`	`async def init_integration(`
`161`	`163`	`hass: HomeAssistant,`
	`164`	`+ caplog: pytest.LogCaptureFixture,`
`162`	`165`	`mock_config_entry: MockConfigEntry,`
`163`	`166`	`mock_authenticator: Mock,`
`164`	`167`	`mock_mqtt_client: Mock,`
`@@ -177,6 +180,12 @@ async def init_integration(`
`177`	`180`
`178`	`181`	`await hass.config_entries.async_setup(mock_config_entry.entry_id)`
`179`	`182`	`await hass.async_block_till_done(wait_background_tasks=True)`
	`183`	`+`
	`184`	`+ # No errors should be logged during setup`
	`185`	`+ assert not [t for t in caplog.record_tuples if t[1] >= logging.ERROR], (`
	`186`	`+ "Errors during integration setup"`
	`187`	`+ )`
	`188`	`+`
`180`	`189`	`yield mock_config_entry`
`181`	`190`
`182`	`191`

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -107,7 +107,7 @@ async def test_devices_in_dr(`
`107`	`107`	`[`
`108`	`108`	`("yna5x1", 26),`
`109`	`109`	`("5xu9h3", 25),`
`110`		`- ("123", 2),`
	`110`	`+ ("123", 3),`
`111`	`111`	`],`
`112`	`112`	`)`
`113`	`113`	`async def test_all_entities_loaded(`

Comments

(0)