From 6e8d6f599419f6cea2c9b5531743fbcbb8582d23 Mon Sep 17 00:00:00 2001 From: "Mr. Bubbles" Date: Thu, 6 Jun 2024 12:15:13 +0200 Subject: [PATCH] Load fixture with decorator to avoid variable not accessed error (#118954) Use fixture decorator --- tests/components/ista_ecotrend/test_config_flow.py | 5 ++--- tests/components/ista_ecotrend/test_init.py | 5 +++-- tests/components/ista_ecotrend/test_sensor.py | 5 +---- 3 files changed, 6 insertions(+), 9 deletions(-) diff --git a/tests/components/ista_ecotrend/test_config_flow.py b/tests/components/ista_ecotrend/test_config_flow.py index 6dfa692841a..e465d85e517 100644 --- a/tests/components/ista_ecotrend/test_config_flow.py +++ b/tests/components/ista_ecotrend/test_config_flow.py @@ -12,9 +12,8 @@ from homeassistant.core import HomeAssistant from homeassistant.data_entry_flow import FlowResultType -async def test_form( - hass: HomeAssistant, mock_setup_entry: AsyncMock, mock_ista: MagicMock -) -> None: +@pytest.mark.usefixtures("mock_ista") +async def test_form(hass: HomeAssistant, mock_setup_entry: AsyncMock) -> None: """Test we get the form.""" result = await hass.config_entries.flow.async_init( DOMAIN, context={"source": SOURCE_USER} diff --git a/tests/components/ista_ecotrend/test_init.py b/tests/components/ista_ecotrend/test_init.py index c7faa2171d6..13b17333bbe 100644 --- a/tests/components/ista_ecotrend/test_init.py +++ b/tests/components/ista_ecotrend/test_init.py @@ -19,8 +19,9 @@ from homeassistant.helpers import device_registry as dr from tests.common import MockConfigEntry +@pytest.mark.usefixtures("mock_ista") async def test_entry_setup_unload( - hass: HomeAssistant, ista_config_entry: MockConfigEntry, mock_ista: MagicMock + hass: HomeAssistant, ista_config_entry: MockConfigEntry ) -> None: """Test integration setup and unload.""" @@ -79,10 +80,10 @@ async def test_config_entry_error( assert ista_config_entry.state is ConfigEntryState.SETUP_ERROR +@pytest.mark.usefixtures("mock_ista") async def test_device_registry( hass: HomeAssistant, ista_config_entry: MockConfigEntry, - mock_ista: MagicMock, device_registry: dr.DeviceRegistry, snapshot: SnapshotAssertion, ) -> None: diff --git a/tests/components/ista_ecotrend/test_sensor.py b/tests/components/ista_ecotrend/test_sensor.py index ca109455885..82a15872b59 100644 --- a/tests/components/ista_ecotrend/test_sensor.py +++ b/tests/components/ista_ecotrend/test_sensor.py @@ -1,7 +1,5 @@ """Tests for the ista EcoTrend Sensors.""" -from unittest.mock import MagicMock - import pytest from syrupy.assertion import SnapshotAssertion @@ -12,11 +10,10 @@ from homeassistant.helpers import entity_registry as er from tests.common import MockConfigEntry, snapshot_platform -@pytest.mark.usefixtures("entity_registry_enabled_by_default") +@pytest.mark.usefixtures("mock_ista", "entity_registry_enabled_by_default") async def test_setup( hass: HomeAssistant, ista_config_entry: MockConfigEntry, - mock_ista: MagicMock, entity_registry: er.EntityRegistry, snapshot: SnapshotAssertion, ) -> None: