[pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

Author: pre-commit-ci[bot]

JciHitachi/api.py

@@ -93,9 +93,9 @@ def from_device_names(
                 if device_type in cls.supported_device_type:
                     peripherals[device_name] = cls(result)
 
-        assert device_names is None or len(device_names) == len(
-            peripherals
-        ), "Some of device_names are not available from the API."
+        assert device_names is None or len(device_names) == len(peripherals), (
+            "Some of device_names are not available from the API."
+        )
 
         return peripherals
 
@@ -688,9 +688,9 @@ def from_device_names(
                 if device_type in cls.supported_device_type:
                     things[device_name] = cls(thing)
 
-        assert device_names is None or len(device_names) == len(
-            things
-        ), "Some of device_names are not available from the API."
+        assert device_names is None or len(device_names) == len(things), (
+            "Some of device_names are not available from the API."
+        )
 
         return things
 

JciHitachi/utility.py

@@ -111,7 +111,9 @@ def extract_bytes(v, start, end):  # pragma: no cover
         Extracted value.
     """
 
-    assert start > end and end >= 0, "Starting byte must be greater than ending byte, \
+    assert start > end and end >= 0, (
+        "Starting byte must be greater than ending byte, \
          and ending byte must be greater than zero : \
          {}, {}".format(start, end)
+    )
     return cast_bytes(v >> end * 8, start - end)

tests/test_api.py

@@ -131,17 +131,21 @@ def fixture_aws_identity():
 class TestAWSAPI:
     def test_login(self, fixture_aws_mock_api, fixture_aws_identity):
         api = fixture_aws_mock_api
-        with patch(
-            "JciHitachi.aws_connection.GetUser.get_data"
-        ) as mock_get_data_get_user, patch(
-            "JciHitachi.aws_connection.GetUser.login"
-        ) as mock_login_get_user, patch(
-            "JciHitachi.aws_connection.ListSubUser.get_data"
-        ) as mock_get_data_list_subuser, patch(
-            "JciHitachi.aws_connection.GetAllDevice.get_data"
-        ) as mock_get_data_get_all_device, patch(
-            "JciHitachi.aws_connection.JciHitachiAWSMqttConnection.connect"
-        ) as mock_connect_mqtt:
+        with (
+            patch(
+                "JciHitachi.aws_connection.GetUser.get_data"
+            ) as mock_get_data_get_user,
+            patch("JciHitachi.aws_connection.GetUser.login") as mock_login_get_user,
+            patch(
+                "JciHitachi.aws_connection.ListSubUser.get_data"
+            ) as mock_get_data_list_subuser,
+            patch(
+                "JciHitachi.aws_connection.GetAllDevice.get_data"
+            ) as mock_get_data_get_all_device,
+            patch(
+                "JciHitachi.aws_connection.JciHitachiAWSMqttConnection.connect"
+            ) as mock_connect_mqtt,
+        ):
             # initialization
             aws_tokens = api._aws_tokens
             api._aws_tokens = None
@@ -182,11 +186,14 @@ def test_login(self, fixture_aws_mock_api, fixture_aws_identity):
 
     def test_change_password(self, fixture_aws_mock_api):
         api = fixture_aws_mock_api
-        with patch(
-            "JciHitachi.aws_connection.ChangePassword.get_data"
-        ) as mock_get_data_1, patch(
-            "JciHitachi.connection.UpdateUserCredential.get_data"
-        ) as mock_get_data_2:
+        with (
+            patch(
+                "JciHitachi.aws_connection.ChangePassword.get_data"
+            ) as mock_get_data_1,
+            patch(
+                "JciHitachi.connection.UpdateUserCredential.get_data"
+            ) as mock_get_data_2,
+        ):
             mock_get_data_1.return_value = ("OK", "")
             mock_get_data_2.return_value = ("OK", "")
             api.change_password("new_password")

tests/test_model.py

@@ -15,9 +15,9 @@ def test_sanity(self):
         for device_type in self.device_types:
             for status_name, status_attributes in STATUS_DICT[device_type].items():
                 # Test if attribute keys are in status attributes.
-                assert (
-                    len(attribute_keys - set(status_attributes)) == 0
-                ), f"Lack of attribute keys: {status_name} {set(status_attributes)}"
+                assert len(attribute_keys - set(status_attributes)) == 0, (
+                    f"Lack of attribute keys: {status_name} {set(status_attributes)}"
+                )
                 # Test is_numeric and id2str mutual exclusiveness
                 assert status_attributes["is_numeric"] ^ (
                     "id2str" in status_attributes