refactor: rename check_migrations → is_migrated
test / unit (pull_request) Successful in 58s
test / integration (pull_request) Successful in 18s
test / coverage (pull_request) Successful in 1m6s
lint / lint (push) Successful in 1m55s
prd-number / assign-numbers (push) Successful in 16s
test / unit (push) Successful in 54s
test / integration (push) Successful in 19s
test / coverage (push) Successful in 1m6s
Update Quality Badges / update-badges (push) Successful in 1m2s

Boolean-returning methods should read as predicates.
Renames DbStore.check_migrations, StoreManager.check_migrations,
and the test patch accordingly.
This commit was merged in pull request #320.
This commit is contained in:
2026-07-06 19:19:49 +00:00
parent 041d9bbbf5
commit 142974a4b8
4 changed files with 7 additions and 7 deletions
+4 -4
View File
@@ -15,7 +15,7 @@ _instance: StoreManager | None = None
class StoreManager:
"""Owns db_path and delegates migrate/check_migrations across all stores.
"""Owns db_path and delegates migrate/is_migrated across all stores.
Use instance() for normal access. Call reset(db_path) in tests to swap
the singleton to a temp path, then reset() with no args to restore the
@@ -46,10 +46,10 @@ class StoreManager:
global _instance
_instance = cls(db_path)
def check_migrations(self) -> bool:
def is_migrated(self) -> bool:
return (
QueueStore("", self.db_path).check_migrations()
and AuditStore(self.db_path).check_migrations()
QueueStore("", self.db_path).is_migrated()
and AuditStore(self.db_path).is_migrated()
)
def migrate(self) -> None: