Refactor Growatt Server integration tests (home-assistant#156413)

Co-authored-by: Claude <[email protected]>

Author: johanzander

tests/components/growatt_server/conftest.py

@@ -37,7 +37,10 @@ def mock_growatt_v1_api():
     Methods mocked for switch and number operations:
     - min_write_parameter: Called by switch/number entities to change settings
     """
-    with patch("growattServer.OpenApiV1", autospec=True) as mock_v1_api_class:
+    with patch(
+        "homeassistant.components.growatt_server.config_flow.growattServer.OpenApiV1",
+        autospec=True,
+    ) as mock_v1_api_class:
         mock_v1_api = mock_v1_api_class.return_value
 
         # Called during setup to discover devices
@@ -119,7 +122,10 @@ def mock_growatt_classic_api():
     Methods mocked for device-specific tests:
     - tlx_detail: Provides TLX device data (kept for potential future tests)
     """
-    with patch("growattServer.GrowattApi", autospec=True) as mock_classic_api_class:
+    with patch(
+        "homeassistant.components.growatt_server.config_flow.growattServer.GrowattApi",
+        autospec=True,
+    ) as mock_classic_api_class:
         # Use the autospec'd mock instance instead of creating a new Mock()
         mock_classic_api = mock_classic_api_class.return_value
 
@@ -167,10 +173,10 @@ def mock_config_entry() -> MockConfigEntry:
             CONF_TOKEN: "test_token_123",
             CONF_URL: DEFAULT_URL,
             "user_id": "12345",
-            CONF_PLANT_ID: "plant_123",
+            CONF_PLANT_ID: "123456",
             "name": "Test Plant",
         },
-        unique_id="plant_123",
+        unique_id="123456",
     )
 
 
@@ -188,10 +194,10 @@ def mock_config_entry_classic() -> MockConfigEntry:
             CONF_USERNAME: "test_user",
             CONF_PASSWORD: "test_password",
             CONF_URL: DEFAULT_URL,
-            CONF_PLANT_ID: "12345",
+            CONF_PLANT_ID: "123456",
             "name": "Test Plant",
         },
-        unique_id="12345",
+        unique_id="123456",
     )
 
 
@@ -215,3 +221,13 @@ async def init_integration(
     await hass.config_entries.async_setup(mock_config_entry.entry_id)
     await hass.async_block_till_done()
     return mock_config_entry
+
+
+@pytest.fixture
+def mock_setup_entry():
+    """Mock async_setup_entry to prevent actual setup during config flow tests."""
+    with patch(
+        "homeassistant.components.growatt_server.async_setup_entry",
+        return_value=True,
+    ) as mock:
+        yield mock

tests/components/growatt_server/snapshots/test_init.ambr

@@ -61,65 +61,3 @@
     'via_device_id': None,
   })
 # ---
-# name: test_multiple_devices_discovered[device_min123456]
-  DeviceRegistryEntrySnapshot({
-    'area_id': None,
-    'config_entries': <ANY>,
-    'config_entries_subentries': <ANY>,
-    'configuration_url': None,
-    'connections': set({
-    }),
-    'disabled_by': None,
-    'entry_type': None,
-    'hw_version': None,
-    'id': <ANY>,
-    'identifiers': set({
-      tuple(
-        'growatt_server',
-        'MIN123456',
-      ),
-    }),
-    'labels': set({
-    }),
-    'manufacturer': 'Growatt',
-    'model': None,
-    'model_id': None,
-    'name': 'MIN123456',
-    'name_by_user': None,
-    'primary_config_entry': <ANY>,
-    'serial_number': None,
-    'sw_version': None,
-    'via_device_id': None,
-  })
-# ---
-# name: test_multiple_devices_discovered[device_min789012]
-  DeviceRegistryEntrySnapshot({
-    'area_id': None,
-    'config_entries': <ANY>,
-    'config_entries_subentries': <ANY>,
-    'configuration_url': None,
-    'connections': set({
-    }),
-    'disabled_by': None,
-    'entry_type': None,
-    'hw_version': None,
-    'id': <ANY>,
-    'identifiers': set({
-      tuple(
-        'growatt_server',
-        'MIN789012',
-      ),
-    }),
-    'labels': set({
-    }),
-    'manufacturer': 'Growatt',
-    'model': None,
-    'model_id': None,
-    'name': 'MIN789012',
-    'name_by_user': None,
-    'primary_config_entry': <ANY>,
-    'serial_number': None,
-    'sw_version': None,
-    'via_device_id': None,
-  })
-# ---

tests/components/growatt_server/snapshots/test_number.ambr

@@ -1,35 +1,4 @@
 # serializer version: 1
-# name: test_number_device_registry
-  DeviceRegistryEntrySnapshot({
-    'area_id': None,
-    'config_entries': <ANY>,
-    'config_entries_subentries': <ANY>,
-    'configuration_url': None,
-    'connections': set({
-    }),
-    'disabled_by': None,
-    'entry_type': None,
-    'hw_version': None,
-    'id': <ANY>,
-    'identifiers': set({
-      tuple(
-        'growatt_server',
-        'MIN123456',
-      ),
-    }),
-    'labels': set({
-    }),
-    'manufacturer': 'Growatt',
-    'model': None,
-    'model_id': None,
-    'name': 'MIN123456',
-    'name_by_user': None,
-    'primary_config_entry': <ANY>,
-    'serial_number': None,
-    'sw_version': None,
-    'via_device_id': None,
-  })
-# ---
 # name: test_number_entities[number.min123456_battery_charge_power_limit-entry]
   EntityRegistryEntrySnapshot({
     'aliases': set({