Remove asserts no longer needed.

[plomtask] / plomtask / days.py

diff --git a/plomtask/days.py b/plomtask/days.py
index d838039a715677ffb3c9022f82715859d80fd652..ce112006042681ba059bc5129d7c45bb3d8c6d21 100644 (file)
--- a/plomtask/days.py
+++ b/plomtask/days.py
@@ -24,7 +24,7 @@ def todays_date() -> str:
     return datetime.now().strftime(DATE_FORMAT)
 
 
-class Day(BaseModel):
+class Day(BaseModel[str]):
     """Individual days defined by their dates."""
     table_name = 'days'
     to_save = ['comment']
@@ -72,13 +72,11 @@ class Day(BaseModel):
         """Retrieve Day by date if in DB (prefer cache), else return None."""
         day, _ = super()._by_id(db_conn, date)
         if day:
-            assert isinstance(day, Day)
             return day
         if not create:
             raise NotFoundException(f'Day not found for date: {date}')
         day = cls(date)
-        db_conn.cached_days[date] = day
-        assert isinstance(day, Day)
+        day.cache()
         return day
 
     @property