Fixed Copilot comments on unit and integration tests

Author: srbhaakamai

linode_api4/objects/monitor.py

@@ -363,7 +363,7 @@ class EmailChannelContent(JSONObject):
     Represents the content for an email alert channel.
     """
 
-    email_addresses: List[str] = None
+    email_addresses: Optional[List[str]] = None
 
 
 @dataclass
@@ -372,7 +372,7 @@ class ChannelContent(JSONObject):
     Represents the content block for an AlertChannel, which varies by channel type.
     """
 
-    email: EmailChannelContent = None
+    email: Optional[EmailChannelContent] = None
     # Other channel types like 'webhook', 'slack' could be added here as Optional fields.
 
 

test/fixtures/monitor_alert-definitions.json

@@ -4,7 +4,7 @@
       "id": 12345,
       "label": "Test Alert for DBAAS",
       "service_type": "dbaas",
-      "severity": "warning",
+      "severity": 1,
       "type": "user",
       "description": "A test alert for dbaas service",
       "conditions": [],

test/fixtures/monitor_services_dbaas_alert-definitions.json

@@ -4,8 +4,8 @@
       "id": 12345,
       "label": "Test Alert for DBAAS",
       "service_type": "dbaas",
-      "severity": "warning",
-      "type": "cpu",
+      "severity": 1,
+      "type": "user",
       "description": "A test alert for dbaas service",
       "entity_ids": [
         13217

test/unit/groups/monitor_api_test.py

@@ -96,7 +96,7 @@ def test_create_alert_definition(self):
             alert = self.client.monitor.create_alert_definition(
                 service_type=service_type,
                 label="Created Alert",
-                severity="warning",
+                severity=1,
                 channel_ids=[1, 2],
                 rule_criteria={"rules": []},
                 trigger_conditions={"criteria_condition": "ALL"},