Do not cancel already cancelled events

Author: Gehock

CHANGELOG.md

@@ -5,6 +5,10 @@ The project uses semantic versioning (see [SemVer](https://semver.org)).
 
 ## [Unreleased]
 
+### Fixed
+
+- Do not cancel same events multiple times
+
 ## v0.50.0 - 2025-02-01
 
 ### Added

src/operationbot/eventDatabase.py

@@ -227,7 +227,8 @@ def cancel_empty_events(
         now = datetime.now(cfg.TIME_ZONE)
         for event in cls.events.values():
             if (
-                event.date.astimezone(cfg.TIME_ZONE) - now < threshold
+                not event.cancelled
+                and event.date.astimezone(cfg.TIME_ZONE) - now < threshold
                 and event.is_empty()
             ):
                 events.append(event)

tests/test_database.py

@@ -121,7 +121,8 @@ def _init_event(event: Event, signup=False) -> None:
     assert len(db.events) == 4
     assert len(db.eventsArchive) == 0
     events = db.cancel_empty_events(timedelta(hours=2))
-    # print(events)
-    assert events[0].cancelled
     assert len(events) == 1
+    assert events[0].cancelled
     assert len(db.events) == 4
+    events = db.cancel_empty_events(timedelta(hours=2))
+    assert len(events) == 0