diff --git a/tests/components/device_sun_light_trigger/test_init.py b/tests/components/device_sun_light_trigger/test_init.py index 65afd5743f59ab2f01d4a964b0a6ec06be68e370..f3821eb5af91aa758f373a1c88b71295e6c20258 100644 --- a/tests/components/device_sun_light_trigger/test_init.py +++ b/tests/components/device_sun_light_trigger/test_init.py @@ -77,11 +77,10 @@ async def scanner( ) await hass.async_block_till_done() - return scanner - +@pytest.mark.usefixtures("scanner") async def test_lights_on_when_sun_sets( - hass: HomeAssistant, freezer: FrozenDateTimeFactory, scanner + hass: HomeAssistant, freezer: FrozenDateTimeFactory ) -> None: """Test lights go on when there is someone home and the sun sets.""" test_time = datetime(2017, 4, 5, 1, 2, 3, tzinfo=dt_util.UTC) @@ -136,8 +135,9 @@ async def test_lights_turn_off_when_everyone_leaves(hass: HomeAssistant) -> None ) +@pytest.mark.usefixtures("scanner") async def test_lights_turn_on_when_coming_home_after_sun_set( - hass: HomeAssistant, freezer: FrozenDateTimeFactory, scanner + hass: HomeAssistant, freezer: FrozenDateTimeFactory ) -> None: """Test lights turn on when coming home after sun set.""" test_time = datetime(2017, 4, 5, 3, 2, 3, tzinfo=dt_util.UTC) @@ -172,8 +172,9 @@ async def test_lights_turn_on_when_coming_home_after_sun_set( ) +@pytest.mark.usefixtures("scanner") async def test_lights_turn_on_when_coming_home_after_sun_set_person( - hass: HomeAssistant, freezer: FrozenDateTimeFactory, scanner + hass: HomeAssistant, freezer: FrozenDateTimeFactory ) -> None: """Test lights turn on when coming home after sun set.""" device_1 = f"{DOMAIN}.device_1" diff --git a/tests/components/doorbird/conftest.py b/tests/components/doorbird/conftest.py index dbd72a0bac999ea4be73412b0da778faa1a655c6..98dbec18b4ce5feac415cbb8bcd5df3bbc70ba74 100644 --- a/tests/components/doorbird/conftest.py +++ b/tests/components/doorbird/conftest.py @@ -1,6 +1,6 @@ """Test configuration for DoorBird tests.""" -from collections.abc import Generator +from collections.abc import Callable, Coroutine, Generator from contextlib import contextmanager from dataclasses import dataclass from typing import Any @@ -16,6 +16,8 @@ from . import VALID_CONFIG, get_mock_doorbird_api from tests.common import MockConfigEntry, load_json_value_fixture +type DoorbirdMockerType = Callable[[], Coroutine[Any, Any, MockDoorbirdEntry]] + @dataclass class MockDoorbirdEntry: @@ -70,7 +72,7 @@ async def doorbird_mocker( hass: HomeAssistant, doorbird_info: dict[str, Any], doorbird_schedule: dict[str, Any], -) -> MockDoorbirdEntry: +) -> DoorbirdMockerType: """Create a MockDoorbirdEntry.""" async def _async_mock( @@ -79,7 +81,7 @@ async def doorbird_mocker( info: dict[str, Any] | None = None, info_side_effect: Exception | None = None, schedule: list[DoorBirdScheduleEntry] | None = None, - ) -> None: + ) -> MockDoorbirdEntry: """Create a MockDoorbirdEntry from defaults or specific values.""" entry = entry or MockConfigEntry( domain=DOMAIN, diff --git a/tests/components/doorbird/test_button.py b/tests/components/doorbird/test_button.py index 7d1f987017ba8b83cc84c35710015afcc4b45173..fc10362a0773605de02241d5b6a7727911a4ee4a 100644 --- a/tests/components/doorbird/test_button.py +++ b/tests/components/doorbird/test_button.py @@ -1,18 +1,15 @@ """Test DoorBird buttons.""" -from collections.abc import Callable, Coroutine -from typing import Any - from homeassistant.components.button import DOMAIN, SERVICE_PRESS from homeassistant.const import ATTR_ENTITY_ID, STATE_UNKNOWN from homeassistant.core import HomeAssistant -from .conftest import MockDoorbirdEntry +from .conftest import DoorbirdMockerType async def test_relay_button( hass: HomeAssistant, - doorbird_mocker: Callable[[], Coroutine[Any, Any, MockDoorbirdEntry]], + doorbird_mocker: DoorbirdMockerType, ) -> None: """Test pressing a relay button.""" doorbird_entry = await doorbird_mocker() @@ -27,7 +24,7 @@ async def test_relay_button( async def test_ir_button( hass: HomeAssistant, - doorbird_mocker: Callable[[], Coroutine[Any, Any, MockDoorbirdEntry]], + doorbird_mocker: DoorbirdMockerType, ) -> None: """Test pressing the IR button.""" doorbird_entry = await doorbird_mocker() @@ -42,7 +39,7 @@ async def test_ir_button( async def test_reset_favorites_button( hass: HomeAssistant, - doorbird_mocker: Callable[[], Coroutine[Any, Any, MockDoorbirdEntry]], + doorbird_mocker: DoorbirdMockerType, ) -> None: """Test pressing the reset favorites button.""" doorbird_entry = await doorbird_mocker() diff --git a/tests/components/doorbird/test_init.py b/tests/components/doorbird/test_init.py index 5263a1db3897d8b4a108e2e2d3de439a099a802d..6bbf694dd7c53c4ef80c1468f76a503d23d44fd2 100644 --- a/tests/components/doorbird/test_init.py +++ b/tests/components/doorbird/test_init.py @@ -1,18 +1,15 @@ """Test DoorBird init.""" -from collections.abc import Callable, Coroutine -from typing import Any - from homeassistant.components.doorbird.const import DOMAIN from homeassistant.config_entries import ConfigEntryState from homeassistant.core import HomeAssistant from . import mock_unauthorized_exception -from .conftest import MockDoorbirdEntry +from .conftest import DoorbirdMockerType async def test_basic_setup( - doorbird_mocker: Callable[[], Coroutine[Any, Any, MockDoorbirdEntry]], + doorbird_mocker: DoorbirdMockerType, ) -> None: """Test basic setup.""" doorbird_entry = await doorbird_mocker() @@ -22,7 +19,7 @@ async def test_basic_setup( async def test_auth_fails( hass: HomeAssistant, - doorbird_mocker: Callable[[], Coroutine[Any, Any, MockDoorbirdEntry]], + doorbird_mocker: DoorbirdMockerType, ) -> None: """Test basic setup with an auth failure.""" doorbird_entry = await doorbird_mocker( diff --git a/tests/components/homeassistant_hardware/conftest.py b/tests/components/homeassistant_hardware/conftest.py index d3dcd443a07bf8141a7b0a51e3f0f3ef7584348b..b62ccaf855b03338026bf84352673a515e79d2c7 100644 --- a/tests/components/homeassistant_hardware/conftest.py +++ b/tests/components/homeassistant_hardware/conftest.py @@ -11,7 +11,7 @@ import pytest def mock_zha_config_flow_setup() -> Generator[None]: """Mock the radio connection and probing of the ZHA config flow.""" - def mock_probe(config: dict[str, Any]) -> None: + def mock_probe(config: dict[str, Any]) -> dict[str, Any]: # The radio probing will return the correct baudrate return {**config, "baudrate": 115200} diff --git a/tests/components/homeassistant_yellow/conftest.py b/tests/components/homeassistant_yellow/conftest.py index 2e6340bf54a3346276f8e60a09876d43ed6f9ed2..0077fb27058a20ef32ef6389662cdfc1463a3b13 100644 --- a/tests/components/homeassistant_yellow/conftest.py +++ b/tests/components/homeassistant_yellow/conftest.py @@ -11,7 +11,7 @@ import pytest def mock_zha_config_flow_setup() -> Generator[None]: """Mock the radio connection and probing of the ZHA config flow.""" - def mock_probe(config: dict[str, Any]) -> None: + def mock_probe(config: dict[str, Any]) -> dict[str, Any]: # The radio probing will return the correct baudrate return {**config, "baudrate": 115200} diff --git a/tests/components/improv_ble/test_config_flow.py b/tests/components/improv_ble/test_config_flow.py index 53da1f28425e70abea20255cf92d95422adc66a1..640a931bee54649747d975fcc9d00da80731a5de 100644 --- a/tests/components/improv_ble/test_config_flow.py +++ b/tests/components/improv_ble/test_config_flow.py @@ -543,7 +543,7 @@ async def test_authorize_fails(hass: HomeAssistant, exc, error) -> None: assert result["reason"] == error -async def _test_provision_error(hass: HomeAssistant, exc) -> None: +async def _test_provision_error(hass: HomeAssistant, exc) -> str: """Test bluetooth flow with error.""" result = await hass.config_entries.flow.async_init( DOMAIN, diff --git a/tests/components/screenlogic/__init__.py b/tests/components/screenlogic/__init__.py index 9c8a21b1ba48a6002866092eb28102db587d59cb..169c1f28900682b904f08fe00cc112a98d4148f2 100644 --- a/tests/components/screenlogic/__init__.py +++ b/tests/components/screenlogic/__init__.py @@ -20,7 +20,7 @@ GATEWAY_IMPORT_PATH = "homeassistant.components.screenlogic.ScreenLogicGateway" GATEWAY_DISCOVERY_IMPORT_PATH = "homeassistant.components.screenlogic.coordinator.async_discover_gateways_by_unique_id" -def num_key_string_to_int(data: dict) -> None: +def num_key_string_to_int(data: dict) -> dict: """Convert all string number dict keys to integer. This needed for screenlogicpy's data dict format. diff --git a/tests/test_core.py b/tests/test_core.py index 14bde940c12240527066eab232c57a4cb10af5f4..671b9fe400bd95dde72fb342edfefee6daa029ea 100644 --- a/tests/test_core.py +++ b/tests/test_core.py @@ -3231,7 +3231,7 @@ async def test_async_add_import_executor_job(hass: HomeAssistant) -> None: evt = threading.Event() loop = asyncio.get_running_loop() - def executor_func() -> None: + def executor_func() -> threading.Event: evt.set() return evt