Merge pull request #13 from bluecurrent/development

fixed lint issues
bluecurrent · May 5, 2023 · c4bf561 · c4bf561
2 parents 8ae73f8 + 49edc6e
commit c4bf561
Show file tree

Hide file tree

Showing 3 changed files with 35 additions and 26 deletions.
diff --git a/README.md b/README.md
@@ -1,4 +1,4 @@
-# Integration Blueprint
+# Blue Current Home Assistant integration
 
 [![hacs_badge](https://img.shields.io/badge/HACS-Custom-41BDF5.svg?style=for-the-badge)](https://github.com/hacs/integration)
 

diff --git a/custom_components/blue_current/manifest.json b/custom_components/blue_current/manifest.json
@@ -5,6 +5,7 @@
   "config_flow": true,
   "documentation": "https://www.home-assistant.io/integrations/blue_current",
   "iot_class": "cloud_push",
+  "issue_tracker": "https://github.com/bluecurrent/ha-bluecurrent/issues",
   "requirements": ["bluecurrent-api==1.0.3"],
-  "version": 1
+  "version": "1.0.0"
 }
diff --git a/tests/test_config_flow.py b/tests/test_config_flow.py
@@ -9,11 +9,13 @@
 from pytest_homeassistant_custom_component.common import MockConfigEntry
 
 from custom_components.blue_current import DOMAIN
-from custom_components.blue_current.config_flow import (AlreadyConnected,
-                                                        InvalidApiToken,
-                                                        NoCardsFound,
-                                                        RequestLimitReached,
-                                                        WebsocketException)
+from custom_components.blue_current.config_flow import (
+    AlreadyConnected,
+    InvalidApiToken,
+    NoCardsFound,
+    RequestLimitReached,
+    WebsocketException,
+)
 
 
 async def test_form(hass: HomeAssistant) -> None:
@@ -57,9 +59,10 @@ async def test_user_card(hass: HomeAssistant) -> None:
     )
     assert result["errors"] == {}
 
-    with patch("bluecurrent_api.Client.validate_api_token", return_value=True,), patch(
-        "bluecurrent_api.Client.get_email", return_value="test@email.com"
-    ), patch(
+    with patch(
+        "bluecurrent_api.Client.validate_api_token",
+        return_value=True,
+    ), patch("bluecurrent_api.Client.get_email", return_value="test@email.com"), patch(
         "bluecurrent_api.Client.get_charge_cards",
         return_value=[{"name": "card 1", "uid": 1}, {"name": "card 2", "uid": 2}],
     ), patch(
@@ -163,9 +166,10 @@ async def test_form_cannot_connect(hass: HomeAssistant) -> None:
 async def test_form_no_cards_found(hass: HomeAssistant) -> None:
     """Test if a no cards error is handled."""
 
-    with patch("bluecurrent_api.Client.validate_api_token", return_value=True,), patch(
-        "bluecurrent_api.Client.get_email", return_value="test@email.com"
-    ), patch(
+    with patch(
+        "bluecurrent_api.Client.validate_api_token",
+        return_value=True,
+    ), patch("bluecurrent_api.Client.get_email", return_value="test@email.com"), patch(
         "bluecurrent_api.Client.get_charge_cards",
         side_effect=NoCardsFound,
     ):
@@ -181,9 +185,10 @@ async def test_form_no_cards_found(hass: HomeAssistant) -> None:
 async def test_form_cannot_connect_card(hass: HomeAssistant) -> None:
     """Test if a connection error on get_charge_cards is handled."""
 
-    with patch("bluecurrent_api.Client.validate_api_token", return_value=True,), patch(
-        "bluecurrent_api.Client.get_email", return_value="test@email.com"
-    ), patch(
+    with patch(
+        "bluecurrent_api.Client.validate_api_token",
+        return_value=True,
+    ), patch("bluecurrent_api.Client.get_email", return_value="test@email.com"), patch(
         "bluecurrent_api.Client.get_charge_cards",
         side_effect=WebsocketException,
     ):
@@ -198,9 +203,10 @@ async def test_form_cannot_connect_card(hass: HomeAssistant) -> None:
 
 async def test_form_limit_reached_card(hass: HomeAssistant) -> None:
     """Test if an limit reached error is handled."""
-    with patch("bluecurrent_api.Client.validate_api_token", return_value=True,), patch(
-        "bluecurrent_api.Client.get_email", return_value="test@email.com"
-    ), patch(
+    with patch(
+        "bluecurrent_api.Client.validate_api_token",
+        return_value=True,
+    ), patch("bluecurrent_api.Client.get_email", return_value="test@email.com"), patch(
         "bluecurrent_api.Client.get_charge_cards",
         side_effect=RequestLimitReached,
     ):
@@ -214,9 +220,10 @@ async def test_form_limit_reached_card(hass: HomeAssistant) -> None:
 
 async def test_form_already_connected_card(hass: HomeAssistant) -> None:
     """Test if an already connected error is handled."""
-    with patch("bluecurrent_api.Client.validate_api_token", return_value=True,), patch(
-        "bluecurrent_api.Client.get_email", return_value="test@email.com"
-    ), patch(
+    with patch(
+        "bluecurrent_api.Client.validate_api_token",
+        return_value=True,
+    ), patch("bluecurrent_api.Client.get_email", return_value="test@email.com"), patch(
         "bluecurrent_api.Client.get_charge_cards",
         side_effect=AlreadyConnected,
     ):
@@ -230,9 +237,10 @@ async def test_form_already_connected_card(hass: HomeAssistant) -> None:
 
 async def test_form_exception_card(hass: HomeAssistant) -> None:
     """Test if an exception is handled."""
-    with patch("bluecurrent_api.Client.validate_api_token", return_value=True,), patch(
-        "bluecurrent_api.Client.get_email", return_value="test@email.com"
-    ), patch(
+    with patch(
+        "bluecurrent_api.Client.validate_api_token",
+        return_value=True,
+    ), patch("bluecurrent_api.Client.get_email", return_value="test@email.com"), patch(
         "bluecurrent_api.Client.get_charge_cards",
         side_effect=Exception,
     ):
@@ -279,4 +287,4 @@ async def test_flow_reauth(hass: HomeAssistant) -> None:
         assert entry.data.copy() == {"api_token": "1234567890"}
 
         assert await entry.async_unload(hass)
-        await hass.async_block_till_done()
+        await hass.async_block_till_done()