From: Christian Heller Date: Thu, 25 Apr 2024 04:07:25 +0000 (+0200) Subject: Remove asserts no longer needed. X-Git-Url: https://plomlompom.com/repos/?a=commitdiff_plain;h=23c7ff7f9833ff5b0e547c19a4ad85325a5d3400;p=plomtask Remove asserts no longer needed. --- diff --git a/plomtask/conditions.py b/plomtask/conditions.py index 4b01249..b2ecda1 100644 --- a/plomtask/conditions.py +++ b/plomtask/conditions.py @@ -24,7 +24,6 @@ class Condition(BaseModel[int]): row: Row | list[Any]) -> Condition: """Build condition from row, including VersionedAttributes.""" condition = super().from_table_row(db_conn, row) - assert isinstance(condition, Condition) for name in ('title', 'description'): table_name = f'condition_{name}s' for row_ in db_conn.row_where(table_name, 'parent', row[0]): @@ -57,7 +56,6 @@ class Condition(BaseModel[int]): raise NotFoundException(f'Condition not found of id: {id_}') condition = cls(id_, False) condition.save(db_conn) - assert isinstance(condition, Condition) return condition def save(self, db_conn: DatabaseConnection) -> None: @@ -65,7 +63,6 @@ class Condition(BaseModel[int]): self.save_core(db_conn) self.title.save(db_conn) self.description.save(db_conn) - assert isinstance(self.id_, int) class ConditionsRelations: diff --git a/plomtask/days.py b/plomtask/days.py index 5fe984b..ce11200 100644 --- a/plomtask/days.py +++ b/plomtask/days.py @@ -72,13 +72,11 @@ class Day(BaseModel[str]): """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) day.cache() - assert isinstance(day, Day) return day @property diff --git a/plomtask/db.py b/plomtask/db.py index e8a542e..bf2b79f 100644 --- a/plomtask/db.py +++ b/plomtask/db.py @@ -145,7 +145,6 @@ class BaseModel(Generic[X]): else: for row in db_conn.row_where(cls.table_name, 'id', id_): obj = cls.from_table_row(db_conn, row) - assert isinstance(obj, cls) obj.cache() break return obj, from_cache diff --git a/plomtask/http.py b/plomtask/http.py index d7be135..c848600 100644 --- a/plomtask/http.py +++ b/plomtask/http.py @@ -245,7 +245,6 @@ class TaskHandler(BaseHTTPRequestHandler): process.set_enables(self.conn, self.form_data.get_all_int('enables')) process.set_disables(self.conn, self.form_data.get_all_int('disables')) process.save_core(self.conn) - assert process.id_ is not None # for mypy process.explicit_steps = [] steps: list[tuple[int | None, int, int | None]] = [] for step_id in self.form_data.get_all_int('steps'): diff --git a/plomtask/processes.py b/plomtask/processes.py index 590c5bc..654e5fc 100644 --- a/plomtask/processes.py +++ b/plomtask/processes.py @@ -208,7 +208,6 @@ class ProcessStep(BaseModel[int]): """Retrieve ProcessStep by id_, or throw NotFoundException.""" step, _ = super()._by_id(db_conn, id_) if step: - assert isinstance(step, ProcessStep) return step raise NotFoundException(f'found no ProcessStep of ID {id_}') diff --git a/plomtask/todos.py b/plomtask/todos.py index 0b42d47..5901571 100644 --- a/plomtask/todos.py +++ b/plomtask/todos.py @@ -76,7 +76,6 @@ class Todo(BaseModel[int], ConditionsRelations): 'todo', todo.id_): target = getattr(todo, name) target += [Condition.by_id(db_conn, cond_id)] - assert isinstance(todo, Todo) return todo @classmethod @@ -135,7 +134,6 @@ class Todo(BaseModel[int], ConditionsRelations): if s.parent_step_id is None] for child_process_id in [c.process.id_ for c in self.children]: if child_process_id in unsatisfied: - assert isinstance(child_process_id, int) unsatisfied.remove(child_process_id) return unsatisfied @@ -194,7 +192,6 @@ class Todo(BaseModel[int], ConditionsRelations): and (c not in potentially_enabled)]: children += [make_node(condition)] else: - assert isinstance(step, Condition) seen = step.id_ in seen_conditions seen_conditions.add(step.id_) return TodoStepsNode(step, is_todo, children, seen)