guettli/sharedinbox
1
0
Fork 0
Code Issues 21 Pull Requests 6 Actions Packages Projects Releases Wiki Activity

test(T4): extend migration tests to cover all schema versions up to v24 #32

Merged
guettlibot merged 1 commits from task/t4-migration-tests into main 2026-05-14 03:09:16 +00:00
Conversation 0 Commits 1 Files Changed 1
+2206 -34499
guettlibot commented 2026-05-14 03:01:22 +00:00 (Migrated from codeberg.org)
Copy Link
Copy Source

Summary

  • Updates test/unit/migration_test.dart from 2 tests (v1→v22, fresh install) to 4 tests
  • schemaVersion assertion: hard-codes the expected value (24) so any future bump without a test update fails immediately
  • v1 → latest: existing test expanded to also assert columns added in v23 (list_unsubscribe_header) and v24 (imap_server_id), plus all new tables from v4–v21
  • v22 → latest: new test that builds a minimal v22 schema with raw SQL and asserts the v23/v24 migration steps execute correctly
  • fresh install: existing test extended to verify all 12 tables and the two new columns exist after schema creation

Test plan

  • task check-fast passes (dart format + analyze + 273 tests green)
  • All 4 migration tests pass in isolation
## Summary - Updates `test/unit/migration_test.dart` from 2 tests (v1→v22, fresh install) to 4 tests - **schemaVersion assertion**: hard-codes the expected value (24) so any future bump without a test update fails immediately - **v1 → latest**: existing test expanded to also assert columns added in v23 (`list_unsubscribe_header`) and v24 (`imap_server_id`), plus all new tables from v4–v21 - **v22 → latest**: new test that builds a minimal v22 schema with raw SQL and asserts the v23/v24 migration steps execute correctly - **fresh install**: existing test extended to verify all 12 tables and the two new columns exist after schema creation ## Test plan - [x] `task check-fast` passes (dart format + analyze + 273 tests green) - [x] All 4 migration tests pass in isolation
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
NeedSupervisor
Issue escalated to a human supervisor; agentloop will skip it until cleared.
 State/InProgress
State/Later
State/Planned
automerge
Eligible for automatic merge by CI
 ci-failure
Issue opened by agentloop to track a failing CI workflow; used for deduplication.
 do-not-merge
Plan PR — review only, do not merge.
 loop/code
Add to run the built-in "code" prompt; override at prompts/code.md.
 loop/code-ci-pending
Prompt "code" finished; waiting for the PR's CI to pass before advancing.
 loop/code-done
Prompt "code" finished successfully.
 loop/code-in-process
Agent for the "code" prompt is currently running on this issue.
 loop/merge
Managed by agentloop
 loop/merge-done
Managed by agentloop
 loop/merge-in-process
Managed by agentloop
 loop/plan
Add to run the built-in "plan" prompt; override at prompts/plan.md.
 loop/plan-done
Prompt "plan" finished successfully.
 loop/plan-in-process
Agent for the "plan" prompt is currently running on this issue.
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
guettlibot guettli
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: guettli/sharedinbox#32