refactor(black): remove trailing comma when there were more character…

…s after it

Pipfile

@@ -4,7 +4,7 @@ url = "https://pypi.org/simple"
 verify_ssl = true
 
 [dev-packages]
-black = "==19.10b0"
+black = "==20.8b1"
 pytest = "==6.1.1"
 pytest-asyncio = "==0.14.0"
 pytest-cov = "==2.10.1"

apps/controllerx/cx_core/controller.py

@@ -72,9 +72,7 @@ class Controller(Hass, Mqtt, abc.ABC):
     """
 
     async def initialize(self) -> None:
-        self.log(
-            f"🎮 ControllerX {cx_version.__version__}", ascii_encode=False,
-        )
+        self.log(f"🎮 ControllerX {cx_version.__version__}", ascii_encode=False)
         self.check_ad_version()
 
         # Get arguments
@@ -232,9 +230,7 @@ async def call_service(self, service: str, **attributes) -> None:
         for attribute, value in attributes.items():
             if isinstance(value, float):
                 value = f"{value:.2f}"
-            self.log(
-                f"  - {attribute}: {value}", level="INFO", ascii_encode=False,
-            )
+            self.log(f"  - {attribute}: {value}", level="INFO", ascii_encode=False)
         return await Hass.call_service(self, service, **attributes)
 
     async def handle_action(self, action_key: str) -> None:

apps/controllerx/cx_core/feature_support/cover.py

@@ -23,7 +23,7 @@ class CoverSupport(FeatureSupport):
     SET_TILT_POSITION = 128
 
     def __init__(
-        self, entity: str, controller: TypeController, update_supported_features: bool,
+        self, entity: str, controller: TypeController, update_supported_features: bool
     ) -> None:
         super().__init__(
             entity,

apps/controllerx/cx_core/feature_support/light.py

@@ -12,7 +12,7 @@ class LightSupport(FeatureSupport):
     WHITE_VALUE = 128
 
     def __init__(
-        self, entity: str, controller: TypeController, update_supported_features: bool,
+        self, entity: str, controller: TypeController, update_supported_features: bool
     ) -> None:
         super().__init__(
             entity,

apps/controllerx/cx_core/feature_support/media_player.py

@@ -21,7 +21,7 @@ class MediaPlayerSupport(FeatureSupport):
     SELECT_SOUND_MODE = 65536
 
     def __init__(
-        self, entity: str, controller: TypeController, update_supported_features: bool,
+        self, entity: str, controller: TypeController, update_supported_features: bool
     ) -> None:
         super().__init__(
             entity,

apps/controllerx/cx_core/integration/mqtt.py

@@ -15,7 +15,7 @@ def get_actions_mapping(self) -> Optional[TypeActionsMapping]:
 
     def listen_changes(self, controller_id: str) -> None:
         Mqtt.listen_event(
-            self.controller, self.event_callback, topic=controller_id, namespace="mqtt",
+            self.controller, self.event_callback, topic=controller_id, namespace="mqtt"
         )
 
     async def event_callback(self, event_name: str, data: dict, kwargs: dict) -> None:

apps/controllerx/cx_core/type/light_controller.py

@@ -109,7 +109,7 @@ async def initialize(self) -> None:
     def get_domain(self) -> List[str]:
         return ["light"]
 
-    def get_type_actions_mapping(self,) -> TypeActionsMapping:
+    def get_type_actions_mapping(self) -> TypeActionsMapping:
         return {
             Light.ON: self.on,
             Light.OFF: self.off,

tests/test_utils.py

@@ -46,9 +46,7 @@ def _all_subclasses(cls):
 
 
 def get_classes(file_, package_, class_, instantiate=False):
-    _import_modules(
-        file_, package_,
-    )
+    _import_modules(file_, package_)
     subclasses = _all_subclasses(class_)
     subclasses = [
         cls_() if instantiate else cls_

tests/unit_tests/cx_core/controller_test.py

@@ -163,7 +163,7 @@ async def test_initialize(
 )
 @pytest.mark.asyncio
 async def test_merge_mapping(
-    sut, monkeypatch, mocker, mapping, merge_mapping, actions_output, expected_error,
+    sut, monkeypatch, mocker, mapping, merge_mapping, actions_output, expected_error
 ):
     actions_input = ["action1", "action2", "action3"]
     actions = {action: action for action in actions_input}

tests/unit_tests/cx_core/custom_controller_test.py

@@ -132,7 +132,7 @@ async def test_custom_controllers(
 )
 @pytest.mark.asyncio
 async def test_call_service_controller(
-    hass_mock, monkeypatch, mocker, integration, services, expected_calls,
+    hass_mock, monkeypatch, mocker, integration, services, expected_calls
 ):
     sut = CallServiceController()  # type: ignore
     sut.args = {

tests/unit_tests/cx_core/feature_support/cover_support_test.py

@@ -6,7 +6,7 @@
 @pytest.mark.parametrize(
     "number, expected_supported_features",
     [
-        (1, {CoverSupport.OPEN},),
+        (1, {CoverSupport.OPEN}),
         (
             15,
             {

tests/unit_tests/cx_core/integration/zha_test.py

@@ -31,7 +31,7 @@
 )
 @pytest.mark.asyncio
 async def test_get_integrations(
-    fake_controller, mocker, command, args, expected_called_with,
+    fake_controller, mocker, command, args, expected_called_with
 ):
     data = {"command": command, "args": args}
     handle_action_patch = mocker.patch.object(fake_controller, "handle_action")

tests/unit_tests/cx_core/type/light_controller_test.py

@@ -183,7 +183,7 @@ async def fake_get_entity_state(entity, attribute=None):
             False,
             51,
         ),
-        (0, "xy_color", Stepper.UP, CircularStepper(0, 30, 30), "on", False, False, 0,),
+        (0, "xy_color", Stepper.UP, CircularStepper(0, 30, 30), "on", False, False, 0),
         (
             499,
             "color_temp",
@@ -616,9 +616,7 @@ async def fake_get_attribute(*args, **kwargs):
         super_hold_patch.assert_called_with(attribute_input, expected_direction)
 
 
-@pytest.mark.parametrize(
-    "value_attribute", [10, None],
-)
+@pytest.mark.parametrize("value_attribute", [10, None])
 @pytest.mark.asyncio
 async def test_hold_loop(sut, mocker, value_attribute):
     attribute = "test_attribute"

tests/unit_tests/cx_core/type/media_player_controller_test.py

@@ -144,7 +144,7 @@ async def test_hold_loop(
         )
     else:
         called_service_patch.assert_called_once_with(
-            f"media_player/volume_{direction_input}", entity_id=sut.media_player,
+            f"media_player/volume_{direction_input}", entity_id=sut.media_player
         )
 
 

tests/unit_tests/cx_core/type_controller_test.py

@@ -23,8 +23,8 @@ def sut(hass_mock):
     "entity, domains, entities, error_expected",
     [
         ("light.kitchen", ["light"], [], False),
-        ("light1.kitchen", ["light"], [], True,),
-        ("media_player.kitchen", ["light"], [], True,),
+        ("light1.kitchen", ["light"], [], True),
+        ("media_player.kitchen", ["light"], [], True),
         ("media_player.bedroom", ["media_player"], [], False),
         ("group.all_lights", ["light"], ["light.light1", "light.light2"], False),
         ("group.all_lights", ["light"], ["light1.light1", "light2.light2"], True),

tests/unit_tests/cx_devices/aqara_test.py

@@ -8,7 +8,7 @@
         ({"command": "shake"}, "shake"),
         ({"command": "knock"}, "knock"),
         ({"command": "slide"}, "slide"),
-        ({"command": "flip", "args": {"flip_degrees": 90}}, "flip90",),
+        ({"command": "flip", "args": {"flip_degrees": 90}}, "flip90"),
         ({"command": "flip", "args": {"flip_degrees": 180}}, "flip180"),
         ({"command": "rotate_left"}, "rotate_left"),
         ({"command": "rotate_right"}, "rotate_right"),