Improve SABnzbd config flow tests (#131234)

tests/components/sabnzbd/conftest.py

@@ -22,7 +22,7 @@ def mock_setup_entry() -> Generator[AsyncMock]:
         yield mock_setup_entry
 
 
-@pytest.fixture(name="sabnzbd")
+@pytest.fixture(name="sabnzbd", autouse=True)
 def mock_sabnzbd() -> Generator[AsyncMock]:
     """Mock the Sabnzbd API."""
     with patch(
@@ -35,7 +35,7 @@ def mock_sabnzbd() -> Generator[AsyncMock]:
 
 
 @pytest.fixture(name="config_entry")
-async def mock_config_entry(hass: HomeAssistant, sabnzbd: AsyncMock) -> MockConfigEntry:
+async def mock_config_entry(hass: HomeAssistant) -> MockConfigEntry:
     """Return a MockConfigEntry for testing."""
     config_entry = MockConfigEntry(
         domain=DOMAIN,
@@ -53,7 +53,7 @@ async def mock_config_entry(hass: HomeAssistant, sabnzbd: AsyncMock) -> MockConf
 
 @pytest.fixture(name="setup_integration")
 async def mock_setup_integration(
-    hass: HomeAssistant, config_entry: MockConfigEntry, sabnzbd: AsyncMock
+    hass: HomeAssistant, config_entry: MockConfigEntry
 ) -> None:
     """Fixture for setting up the component."""
     assert await async_setup_component(hass, DOMAIN, {})

tests/components/sabnzbd/test_config_flow.py

@@ -1,6 +1,6 @@
 """Define tests for the Sabnzbd config flow."""
 
-from unittest.mock import AsyncMock, patch
+from unittest.mock import AsyncMock
 
 from pysabnzbd import SabnzbdApiException
 import pytest
@@ -28,31 +28,25 @@ async def test_create_entry(hass: HomeAssistant, mock_setup_entry: AsyncMock) ->
     assert result["type"] is FlowResultType.FORM
     assert result["errors"] == {}
 
-    with patch(
+    result = await hass.config_entries.flow.async_configure(
-        "homeassistant.components.sabnzbd.sab.SabnzbdApi.check_available",
-        return_value=True,
-    ):
-        result2 = await hass.config_entries.flow.async_configure(
         result["flow_id"],
         VALID_CONFIG,
     )
     await hass.async_block_till_done()
 
-        assert result2["type"] is FlowResultType.CREATE_ENTRY
+    assert result["type"] is FlowResultType.CREATE_ENTRY
-        assert result2["title"] == "edc3eee7330e"
+    assert result["title"] == "edc3eee7330e"
-        assert result2["data"] == {
+    assert result["data"] == {
         CONF_API_KEY: "edc3eee7330e4fdda04489e3fbc283d0",
         CONF_URL: "http://localhost:8080",
     }
     assert len(mock_setup_entry.mock_calls) == 1
 
 
-async def test_auth_error(hass: HomeAssistant) -> None:
+async def test_auth_error(hass: HomeAssistant, sabnzbd: AsyncMock) -> None:
-    """Test that the user step fails."""
+    """Test when the user step fails and if we can recover."""
-    with patch(
+    sabnzbd.check_available.side_effect = SabnzbdApiException("Some error")
-        "homeassistant.components.sabnzbd.sab.SabnzbdApi.check_available",
+
-        side_effect=SabnzbdApiException("Some error"),
-    ):
     result = await hass.config_entries.flow.async_init(
         DOMAIN,
         context={"source": SOURCE_USER},
@@ -60,3 +54,20 @@ async def test_auth_error(hass: HomeAssistant) -> None:
     )
 
     assert result["errors"] == {"base": "cannot_connect"}
+
+    # reset side effect and check if we can recover
+    sabnzbd.check_available.side_effect = None
+
+    result = await hass.config_entries.flow.async_configure(
+        result["flow_id"],
+        VALID_CONFIG,
+    )
+    await hass.async_block_till_done()
+
+    assert "errors" not in result
+    assert result["type"] is FlowResultType.CREATE_ENTRY
+    assert result["title"] == "edc3eee7330e"
+    assert result["data"] == {
+        CONF_API_KEY: "edc3eee7330e4fdda04489e3fbc283d0",
+        CONF_URL: "http://localhost:8080",
+    }