Mark config-flow as done in SFR Box IQS (home-assistant#157439)

Author: epenet

homeassistant/components/sfr_box/config_flow.py

@@ -39,14 +39,18 @@ class SFRBoxFlowHandler(ConfigFlow, domain=DOMAIN):
 
     VERSION = 1
     _box: SFRBox
-    _config: dict[str, Any] = {}
+
+    def __init__(self) -> None:
+        """Initialize SFR Box flow."""
+        self._config: dict[str, Any] = {}
 
     async def async_step_user(
         self, user_input: dict[str, str] | None = None
     ) -> ConfigFlowResult:
         """Handle a flow initialized by the user."""
         errors = {}
         if user_input is not None:
+            self._async_abort_entries_match({CONF_HOST: user_input[CONF_HOST]})
             box = SFRBox(
                 ip=user_input[CONF_HOST], client=async_get_clientsession(self.hass)
             )
@@ -60,7 +64,6 @@ async def async_step_user(
                     assert system_info is not None
                 await self.async_set_unique_id(system_info.mac_addr)
                 self._abort_if_unique_id_configured()
-                self._async_abort_entries_match({CONF_HOST: user_input[CONF_HOST]})
                 self._box = box
                 self._config.update(user_input)
                 return await self.async_step_choose_auth()

homeassistant/components/sfr_box/quality_scale.yaml

@@ -1,13 +1,6 @@
 rules:
-  # Special note, doesn't fall in any of the categories below
-  # Why can the coordinator state be None? As in, I understand that it can be None if it doesn't support it, but wouldn't these coordinators not be created at all?
-
   ## Bronze
-  config-flow:
-    status: todo
-    comment: |
-      SFRBoxFlowHandler has a classvar _config. This can be dangerous
-      self._async_abort_entries_match({CONF_HOST: user_input[CONF_HOST]}) can be tested at the first if statement
+  config-flow: done
   test-before-configure: done
   unique-config-entry: done
   config-flow-test-coverage: