fix:LintErrors

Author: RSam-NI

examples/alarm/alarm.py

@@ -38,7 +38,8 @@
 id = client.create_or_update_alarm(create_request)
 
 # Get the alarm by its instance ID (the unique occurrence identifier)
-alarm = client.get_alarm(id)
+if id:
+    alarm = client.get_alarm(id)
 print(f"Retrieved alarm: {alarm.alarm_id}, Condition: {alarm.condition}")
 
 # Update the alarm with a higher severity (same alarm_id, updates the same instance)
@@ -70,10 +71,11 @@
 for alarm in query_response.alarms:
     print(f"  Alarm ID: {alarm.alarm_id}, Transitions: {len(alarm.transitions)}")
     for transition in alarm.transitions:
-        print(f"    - {transition.transition_type}: {transition.condition}")
+        print(f"- {transition.transition_type}: {transition.condition}")
 
-# Acknowledge the alarm
-client.acknowledge_alarms(ids=[id])
+if id:
+    # Acknowledge the alarm
+    client.acknowledge_alarms(ids=[id])
 
 # Clear the alarm with 409 conflict handling - Method 1: Manual exception handling
 # A 409 Conflict response indicates that the requested transition would not change the alarm's state.
@@ -105,5 +107,6 @@
 else:
     print(f"Alarm cleared successfully: {result}")
 
-# Delete the alarm by its instance ID
-client.delete_alarm(id)
+if id:
+    # Delete the alarm by its instance ID
+    client.delete_alarm(id)

nisystemlink/clients/alarm/_alarm_client.py

@@ -60,7 +60,10 @@ def acknowledge_alarms(
         ...
 
     def create_or_update_alarm(
-        self, request: models.CreateOrUpdateAlarmRequest, *, ignore_conflict: bool = False
+        self,
+        request: models.CreateOrUpdateAlarmRequest,
+        *,
+        ignore_conflict: bool = False,
     ) -> str | None:
         """Creates or updates an instance, or occurrence, of an alarm.
 

tests/integration/alarm/test_alarm_client.py

@@ -40,7 +40,7 @@ def _unique_identifier() -> str:
 @pytest.fixture
 def create_alarms(
     client: AlarmClient,
-) -> Generator[Callable[[str, int, str], str], None, None]:
+) -> Generator[Callable[[str, int, str], str | None], None, None]:
     """Fixture to return a factory that creates alarms.
 
     Returns instance_id (referred to as 'id' in tests) for each created alarm.
@@ -51,7 +51,7 @@ def _create_alarms(
         alarm_id: str,
         severity_level: int = 3,
         condition: str = "Test Condition",
-    ) -> str:
+    ) -> str | None:
         """Create an alarm and return its instance_id."""
         request = CreateOrUpdateAlarmRequest(
             alarm_id=alarm_id,
@@ -62,7 +62,8 @@ def _create_alarms(
             ),
         )
         id = client.create_or_update_alarm(request)
-        created_ids.append(id)
+        if id:
+            created_ids.append(id)
         return id
 
     yield _create_alarms
@@ -320,6 +321,7 @@ def test__delete_alarm__returns_none(
         )
         id = client.create_or_update_alarm(request)
 
+        assert id is not None
         # Delete returns None on success
         result = client.delete_alarm(id)
         assert result is None