Make sure the notifier component does not crash when starting up. Fixes

#1830.
ICTU · Jan 29, 2021 · b332d41 · b332d41
1 parent ea3cc18
commit b332d41
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/components/notifier/src/quality_time_notifier.py b/components/notifier/src/quality_time_notifier.py
@@ -24,8 +24,8 @@ async def notify(log_level: int = None) -> NoReturn:
         f"{os.environ.get('SERVER_PORT', '5001')}/api/{api_version}/reports"
     )
     data_model = await retrieve_data_model(api_version)
-    # needs to be aware for comparison in long_unchanged_status
-    most_recent_measurement_seen = datetime.max.isoformat() + "+00:00"
+    # needs to be timezone aware for comparison in long_unchanged_status
+    most_recent_measurement_seen = datetime.max.replace(microsecond=0).isoformat() + "+00:00"
     outbox = Outbox()
     notification_finder = NotificationFinder(data_model)
     while True:

diff --git a/components/notifier/tests/strategies/test_notification_strategy.py b/components/notifier/tests/strategies/test_notification_strategy.py
@@ -278,8 +278,8 @@ class LongUnchangedTestCase(Base):
 
     def setUp(self):
         """Override to set up a metric fixture."""
-        self.old_timestamp = "2019-02-01T00:00:00"
-        self.new_timestamp = "2020-02-01T00:00:00"
+        self.old_timestamp = "2019-02-01T00:00:00+00:00"
+        self.new_timestamp = "2020-02-01T00:00:00+00:00"
         self.notification_finder = NotificationFinder(self.data_model)
         self.red_metric = self.metric(
             status="target_not_met",

diff --git a/docs/CHANGELOG.md b/docs/CHANGELOG.md
@@ -13,7 +13,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
 
 ### Fixed
 
-- Make sure the notifier component does not crash when .... Fixes [#1830](https://github.com/ICTU/quality-time/issues/1830).
+- Make sure the notifier component does not crash when starting up. Fixes [#1830](https://github.com/ICTU/quality-time/issues/1830).
 
 ## [3.17.1] - [2021-01-24]