style: format test case

tests/unit/actions/test_client.py

@@ -30,9 +30,17 @@ def test_wait_until_finished(
         running_action,
         successfully_action,
     ):
-        request_mock.side_effect = [running_action, successfully_action]
+        request_mock.side_effect = [
+            running_action,
+            successfully_action,
+        ]
+
         bound_running_action.wait_until_finished()
-        request_mock.assert_called_with(url="/actions/2", method="GET")
+
+        request_mock.assert_called_with(
+            method="GET",
+            url="/actions/2",
+        )
 
         assert bound_running_action.status == "success"
         assert request_mock.call_count == 2
@@ -83,8 +91,13 @@ def test_get_by_id(
         generic_action,
     ):
         request_mock.return_value = generic_action
+
         action = actions_client.get_by_id(1)
-        request_mock.assert_called_with(url="/resource/actions/1", method="GET")
+
+        request_mock.assert_called_with(
+            method="GET",
+            url="/resource/actions/1",
+        )
         assert action._client == actions_client._parent.actions
         assert action.id == 1
         assert action.command == "stop_server"
@@ -101,9 +114,13 @@ def test_get_list(
         params,
     ):
         request_mock.return_value = generic_action_list
+
         result = actions_client.get_list(**params)
+
         request_mock.assert_called_with(
-            url="/resource/actions", method="GET", params=params
+            method="GET",
+            url="/resource/actions",
+            params=params,
         )
 
         assert result.meta is not None
@@ -131,12 +148,15 @@ def test_get_all(
         params,
     ):
         request_mock.return_value = generic_action_list
+
         actions = actions_client.get_all(**params)
 
         params.update({"page": 1, "per_page": 50})
 
         request_mock.assert_called_with(
-            url="/resource/actions", method="GET", params=params
+            method="GET",
+            url="/resource/actions",
+            params=params,
         )
 
         assert len(actions) == 2
@@ -165,8 +185,13 @@ def test_get_by_id(
         generic_action,
     ):
         request_mock.return_value = generic_action
+
         action = actions_client.get_by_id(1)
-        request_mock.assert_called_with(url="/actions/1", method="GET")
+
+        request_mock.assert_called_with(
+            method="GET",
+            url="/actions/1",
+        )
         assert action._client == actions_client._parent.actions
         assert action.id == 1
         assert action.command == "stop_server"
@@ -183,9 +208,15 @@ def test_get_list(
         params,
     ):
         request_mock.return_value = generic_action_list
+
         with pytest.deprecated_call():
             result = actions_client.get_list(**params)
-        request_mock.assert_called_with(url="/actions", method="GET", params=params)
+
+        request_mock.assert_called_with(
+            method="GET",
+            url="/actions",
+            params=params,
+        )
 
         assert result.meta is not None
 
@@ -212,12 +243,17 @@ def test_get_all(
         params,
     ):
         request_mock.return_value = generic_action_list
+
         with pytest.deprecated_call():
             actions = actions_client.get_all(**params)
 
         params.update({"page": 1, "per_page": 50})
 
-        request_mock.assert_called_with(url="/actions", method="GET", params=params)
+        request_mock.assert_called_with(
+            method="GET",
+            url="/actions",
+            params=params,
+        )
 
         assert len(actions) == 2
 

tests/unit/certificates/test_client.py

@@ -29,9 +29,13 @@ def test_get_actions_list(
         params,
     ):
         request_mock.return_value = response_get_actions
+
         result = bound_certificate.get_actions_list(**params)
+
         request_mock.assert_called_with(
-            url="/certificates/14/actions", method="GET", params=params
+            method="GET",
+            url="/certificates/14/actions",
+            params=params,
         )
 
         actions = result.actions
@@ -51,12 +55,15 @@ def test_get_actions(
         response_get_actions,
     ):
         request_mock.return_value = response_get_actions
+
         actions = bound_certificate.get_actions()
 
         params = {"page": 1, "per_page": 50}
 
         request_mock.assert_called_with(
-            url="/certificates/14/actions", method="GET", params=params
+            method="GET",
+            url="/certificates/14/actions",
+            params=params,
         )
 
         assert len(actions) == 1
@@ -95,9 +102,13 @@ def test_update(
         response_update_certificate,
     ):
         request_mock.return_value = response_update_certificate
+
         certificate = bound_certificate.update(name="New name")
+
         request_mock.assert_called_with(
-            url="/certificates/14", method="PUT", json={"name": "New name"}
+            method="PUT",
+            url="/certificates/14",
+            json={"name": "New name"},
         )
 
         assert certificate.id == 2323
@@ -110,8 +121,13 @@ def test_delete(
         generic_action,
     ):
         request_mock.return_value = generic_action
+
         delete_success = bound_certificate.delete()
-        request_mock.assert_called_with(url="/certificates/14", method="DELETE")
+
+        request_mock.assert_called_with(
+            method="DELETE",
+            url="/certificates/14",
+        )
 
         assert delete_success is True
 
@@ -122,9 +138,12 @@ def test_retry_issuance(
         response_retry_issuance_action,
     ):
         request_mock.return_value = response_retry_issuance_action
+
         action = bound_certificate.retry_issuance()
+
         request_mock.assert_called_with(
-            url="/certificates/14/actions/retry", method="POST"
+            method="POST",
+            url="/certificates/14/actions/retry",
         )
 
         assert action.id == 14
@@ -143,8 +162,13 @@ def test_get_by_id(
         certificate_response,
     ):
         request_mock.return_value = certificate_response
+
         certificate = certificates_client.get_by_id(1)
-        request_mock.assert_called_with(url="/certificates/1", method="GET")
+
+        request_mock.assert_called_with(
+            method="GET",
+            url="/certificates/1",
+        )
         assert certificate._client is certificates_client
         assert certificate.id == 2323
         assert certificate.name == "My Certificate"
@@ -170,9 +194,13 @@ def test_get_list(
         params,
     ):
         request_mock.return_value = two_certificates_response
+
         result = certificates_client.get_list(**params)
+
         request_mock.assert_called_with(
-            url="/certificates", method="GET", params=params
+            method="GET",
+            url="/certificates",
+            params=params,
         )
 
         certificates = result.certificates
@@ -200,11 +228,15 @@ def test_get_all(
         params,
     ):
         request_mock.return_value = two_certificates_response
+
         certificates = certificates_client.get_all(**params)
 
         params.update({"page": 1, "per_page": 50})
+
         request_mock.assert_called_with(
-            url="/certificates", method="GET", params=params
+            method="GET",
+            url="/certificates",
+            params=params,
         )
 
         assert len(certificates) == 2
@@ -227,11 +259,15 @@ def test_get_by_name(
         one_certificates_response,
     ):
         request_mock.return_value = one_certificates_response
+
         certificates = certificates_client.get_by_name("My Certificate")
 
         params = {"name": "My Certificate"}
+
         request_mock.assert_called_with(
-            url="/certificates", method="GET", params=params
+            method="GET",
+            url="/certificates",
+            params=params,
         )
 
         assert certificates._client is certificates_client
@@ -245,14 +281,16 @@ def test_create(
         certificate_response,
     ):
         request_mock.return_value = certificate_response
+
         certificate = certificates_client.create(
             name="My Certificate",
             certificate="-----BEGIN CERTIFICATE-----\n...",
             private_key="-----BEGIN PRIVATE KEY-----\n...",
         )
+
         request_mock.assert_called_with(
-            url="/certificates",
             method="POST",
+            url="/certificates",
             json={
                 "name": "My Certificate",
                 "certificate": "-----BEGIN CERTIFICATE-----\n...",
@@ -271,12 +309,14 @@ def test_create_managed(
         create_managed_certificate_response,
     ):
         request_mock.return_value = create_managed_certificate_response
+
         create_managed_certificate_rsp = certificates_client.create_managed(
             name="My Certificate", domain_names=["example.com", "*.example.org"]
         )
+
         request_mock.assert_called_with(
-            url="/certificates",
             method="POST",
+            url="/certificates",
             json={
                 "name": "My Certificate",
                 "domain_names": ["example.com", "*.example.org"],
@@ -301,9 +341,13 @@ def test_update(
         response_update_certificate,
     ):
         request_mock.return_value = response_update_certificate
+
         certificate = certificates_client.update(certificate, name="New name")
+
         request_mock.assert_called_with(
-            url="/certificates/1", method="PUT", json={"name": "New name"}
+            method="PUT",
+            url="/certificates/1",
+            json={"name": "New name"},
         )
 
         assert certificate.id == 2323
@@ -321,8 +365,13 @@ def test_delete(
         generic_action,
     ):
         request_mock.return_value = generic_action
+
         delete_success = certificates_client.delete(certificate)
-        request_mock.assert_called_with(url="/certificates/1", method="DELETE")
+
+        request_mock.assert_called_with(
+            method="DELETE",
+            url="/certificates/1",
+        )
 
         assert delete_success is True
 
@@ -338,9 +387,12 @@ def test_retry_issuance(
         response_retry_issuance_action,
     ):
         request_mock.return_value = response_retry_issuance_action
+
         action = certificates_client.retry_issuance(certificate)
+
         request_mock.assert_called_with(
-            url="/certificates/1/actions/retry", method="POST"
+            method="POST",
+            url="/certificates/1/actions/retry",
         )
 
         assert action.id == 14
@@ -355,7 +407,10 @@ def test_actions_get_by_id(
         request_mock.return_value = {"action": response_get_actions["actions"][0]}
         action = certificates_client.actions.get_by_id(13)
 
-        request_mock.assert_called_with(url="/certificates/actions/13", method="GET")
+        request_mock.assert_called_with(
+            method="GET",
+            url="/certificates/actions/13",
+        )
 
         assert isinstance(action, BoundAction)
         assert action._client == certificates_client._parent.actions
@@ -369,11 +424,12 @@ def test_actions_get_list(
         response_get_actions,
     ):
         request_mock.return_value = response_get_actions
+
         result = certificates_client.actions.get_list()
 
         request_mock.assert_called_with(
-            url="/certificates/actions",
             method="GET",
+            url="/certificates/actions",
             params={},
         )
 
@@ -393,11 +449,12 @@ def test_actions_get_all(
         response_get_actions,
     ):
         request_mock.return_value = response_get_actions
+
         actions = certificates_client.actions.get_all()
 
         request_mock.assert_called_with(
-            url="/certificates/actions",
             method="GET",
+            url="/certificates/actions",
             params={"page": 1, "per_page": 50},
         )