Integration tests for database migrations #693

Closed
opened 2025-12-29 02:22:09 +01:00 by adam · 1 comment
Owner

Originally created by @kradalby on GitHub (Apr 17, 2024).

between 0.23.0-beta5 and beta6, there was a bad migration that was not detected due to all the tests starting with a fresh database.

We should make a version of some of the base tests (pingall, up down) run with an old database so it has to run through migrations, but we need to find a clever way to inject "old" data so that it is also migrated, not just empty tables.

Originally created by @kradalby on GitHub (Apr 17, 2024). between 0.23.0-beta5 and beta6, there was a bad migration that was not detected due to all the tests starting with a fresh database. We should make a version of some of the base tests (pingall, up down) run with an old database so it has to run through migrations, but we need to find a clever way to inject "old" data so that it is also migrated, not just empty tables.
adam added the enhancement label 2025-12-29 02:22:09 +01:00
adam closed this issue 2025-12-29 02:22:09 +01:00
Author
Owner

@kradalby commented on GitHub (Sep 12, 2024):

Tests for this was implemented in #2076

@kradalby commented on GitHub (Sep 12, 2024): Tests for this was implemented in #2076
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/headscale#693