minor pylint fixes to config_flow

Author: craigbarratt

custom_components/pyscript/config_flow.py

@@ -45,9 +45,9 @@ async def async_step_import(self, import_config: Dict[str, Any] = None) -> Dict[
 
             # Update values for all keys, excluding `allow_all_imports` for entries
             # set up through the UI.
-            for k, v in import_config.items():
-                if entry.source == SOURCE_IMPORT or k != CONF_ALLOW_ALL_IMPORTS:
-                    updated_data[k] = v
+            for key, val in import_config.items():
+                if entry.source == SOURCE_IMPORT or key != CONF_ALLOW_ALL_IMPORTS:
+                    updated_data[key] = val
 
             # Remove values for all keys in entry.data that are not in the imported config,
             # excluding `allow_all_imports` for entries set up through the UI.

tests/test_config_flow.py

@@ -152,7 +152,7 @@ async def test_import_flow_update_user(hass):
     assert result["type"] == data_entry_flow.RESULT_TYPE_ABORT
     assert result["reason"] == "updated_entry"
 
-    hass.config_entries.async_entries(DOMAIN)[0].data == {
+    assert hass.config_entries.async_entries(DOMAIN)[0].data == {
         CONF_ALLOW_ALL_IMPORTS: True,
         "apps": {"test_app": {"param": 1}},
     }
@@ -173,4 +173,4 @@ async def test_import_flow_update_import(hass):
     assert result["type"] == data_entry_flow.RESULT_TYPE_ABORT
     assert result["reason"] == "updated_entry"
 
-    hass.config_entries.async_entries(DOMAIN)[0].data == {"apps": {"test_app": {"param": 1}}}
+    assert hass.config_entries.async_entries(DOMAIN)[0].data == {"apps": {"test_app": {"param": 1}}}