fixed formatting errors using python black

Author: srbhaakamai

linode_api4/groups/monitor.py

@@ -268,7 +268,7 @@ def create_alert_definition(
         if entity_ids is not None:
             params["entity_ids"] = entity_ids
 
-        #API will handle check for service_type return error if missing
+        # API will handle check for service_type return error if missing
         result = self.client.post(
             f"/monitor/services/{service_type}/alert-definitions", data=params
         )
@@ -279,4 +279,4 @@ def create_alert_definition(
                 json=result,
             )
 
-        return AlertDefinition(self.client, result["id"],service_type, result)
+        return AlertDefinition(self.client, result["id"], service_type, result)

linode_api4/objects/monitor.py

@@ -69,6 +69,7 @@ class MetricType(StrEnum):
     histogram = "histogram"
     summary = "summary"
 
+
 class CriteriaCondition(StrEnum):
     """
     Enum for supported CriteriaCondition
@@ -220,6 +221,7 @@ class TriggerConditions(JSONObject):
       - polling_interval_seconds: how often metrics are sampled (seconds)
       - trigger_occurrences: how many consecutive evaluation periods must match to trigger
     """
+
     criteria_condition: CriteriaCondition = CriteriaCondition.all
     evaluation_period_seconds: int = 0
     polling_interval_seconds: int = 0
@@ -239,6 +241,7 @@ class DimensionFilter(JSONObject):
         "value": "primary"
       }
     """
+
     dimension_label: str = ""
     label: str = ""
     operator: str = ""
@@ -260,6 +263,7 @@ class Rule(JSONObject):
         "unit": "percent"
       }
     """
+
     aggregate_function: Union[AggregateFunction, str] = ""
     dimension_filters: Optional[List[DimensionFilter]] = None
     label: str = ""
@@ -275,6 +279,7 @@ class RuleCriteria(JSONObject):
     Container for a list of Rule objects, matching the JSON shape:
       "rule_criteria": { "rules": [ { ... }, ... ] }
     """
+
     rules: List[Rule] = field(default_factory=list)
 
 
@@ -295,11 +300,13 @@ class AlertChannelEnvelope(JSONObject):
       - type: str - Channel type (e.g. 'webhook', 'email', 'pagerduty').
       - url: str - Destination URL or address associated with the channel.
     """
+
     id: int = 0
     label: str = ""
     type: str = ""
     url: str = ""
 
+
 class AlertType(StrEnum):
     """
     Enumeration of alert origin types used by alert definitions.
@@ -312,6 +319,7 @@ class AlertType(StrEnum):
     This enum can be used to compare or validate the `type` value when
     processing alert definitions.
     """
+
     system = "system"
     user = "user"
 
@@ -354,6 +362,7 @@ class EmailChannelContent(JSONObject):
     """
     Represents the content for an email alert channel.
     """
+
     email_addresses: List[str] = None
 
 
@@ -362,9 +371,11 @@ class ChannelContent(JSONObject):
     """
     Represents the content block for an AlertChannel, which varies by channel type.
     """
+
     email: EmailChannelContent = None
     # Other channel types like 'webhook', 'slack' could be added here as Optional fields.
-    
+
+
 class AlertChannel(Base):
     """
     Represents an alert channel used to deliver notifications when alerts

test/integration/conftest.py

@@ -678,4 +678,4 @@ def test_monitor_client(get_monitor_token_for_db_entities):
         ca_path=api_ca_file,
     )
 
-    return client, entity_ids
+    return client, entity_ids

test/integration/models/monitor/test_monitor.py

@@ -132,12 +132,12 @@ def test_integration_create_get_update_delete_alert_definition(
                         "operator": "eq",
                         "value": "primary",
                     }
-                    ],
-                    "label": "Memory Usage",
-                    "metric": "memory_usage",
-                    "operator": "gt",
-                    "threshold": 90,
-                    "unit": "percent",
+                ],
+                "label": "Memory Usage",
+                "metric": "memory_usage",
+                "operator": "gt",
+                "threshold": 90,
+                "unit": "percent",
             }  # <-- Close the rule dictionary here
         ]
     }
@@ -155,7 +155,9 @@ def test_integration_create_get_update_delete_alert_definition(
     # Pick an existing alert channel to attach to the definition; skip if none
     channels = list(client.monitor.alert_channels())
     if not channels:
-        pytest.skip("No alert channels available on account for creating alert definitions")
+        pytest.skip(
+            "No alert channels available on account for creating alert definitions"
+        )
 
     created = None
     try:
@@ -177,23 +179,29 @@ def test_integration_create_get_update_delete_alert_definition(
         timeout = 120
         interval = 10
         start = time.time()
-        while getattr(created, "status", None) == "in progress" and (time.time() - start) < timeout:
+        while (
+            getattr(created, "status", None) == "in progress"
+            and (time.time() - start) < timeout
+        ):
             time.sleep(interval)
             try:
-                created = client.load(AlertDefinition,created.id,service_type)
+                created = client.load(AlertDefinition, created.id, service_type)
             except Exception:
                 # transient errors while polling; continue until timeout
                 pass
 
-        update_alert= client.load(AlertDefinition, created.id, service_type)
+        update_alert = client.load(AlertDefinition, created.id, service_type)
         update_alert.label = f"{label}-updated"
         update_alert.save()
 
         updated = client.load(AlertDefinition, update_alert.id, service_type)
-        while getattr(updated, "status", None) == "in progress" and (time.time() - start) < timeout:
+        while (
+            getattr(updated, "status", None) == "in progress"
+            and (time.time() - start) < timeout
+        ):
             time.sleep(interval)
             try:
-                updated = client.load(AlertDefinition,updated.id,service_type)
+                updated = client.load(AlertDefinition, updated.id, service_type)
             except Exception:
                 # transient errors while polling; continue until timeout
                 pass
@@ -204,21 +212,23 @@ def test_integration_create_get_update_delete_alert_definition(
     finally:
         if created:
             # Best-effort cleanup; allow transient errors.
-             # max time alert should take to update
+            # max time alert should take to update
             try:
-                delete_alert = client.load(AlertDefinition, created.id, service_type)
+                delete_alert = client.load(
+                    AlertDefinition, created.id, service_type
+                )
                 delete_alert.delete()
             except Exception:
                 pass
 
             # confirm it's gone (if API returns 404 or raises)
             try:
-                client.load(AlertDefinition,created.id,service_type)
+                client.load(AlertDefinition, created.id, service_type)
                 # If no exception, fail explicitly
                 assert False, "Alert definition still retrievable after delete"
             except ApiError:
                 # Expected: alert definition is deleted and API returns 404 or similar error
                 pass
             except Exception:
                 # Any other exception is acceptable here, as the resource should be gone
-                pass
+                pass