fix: align migrations with models for bigint ids and required deck_id

Author: lesnik512

app/models.py

@@ -27,4 +27,4 @@ class Card(BigIntAuditBase):
     front: orm.Mapped[str] = orm.mapped_column(sa.String, nullable=False)
     back: orm.Mapped[str | None] = orm.mapped_column(sa.String, nullable=True)
     hint: orm.Mapped[str | None] = orm.mapped_column(sa.String, nullable=True)
-    deck_id: orm.Mapped[int] = orm.mapped_column(sa.Integer, sa.ForeignKey("decks.id"))
+    deck_id: orm.Mapped[int] = orm.mapped_column(sa.BigInteger, sa.ForeignKey("decks.id"))

migrations/versions/2026-10-16_bigint_ids_and_required_deck_id.py

@@ -0,0 +1,33 @@
+"""bigint ids and required deck_id.
+
+Revision ID: 8e4a2c7b1f6d
+Revises: 5d1c0f3a9b2e
+Create Date: 2026-10-16 22:00:00.000000
+
+"""
+
+import sqlalchemy as sa
+from alembic import op
+
+
+# revision identifiers, used by Alembic.
+revision = "8e4a2c7b1f6d"
+down_revision = "5d1c0f3a9b2e"
+branch_labels = None
+depends_on = None
+
+
+def upgrade() -> None:
+    op.alter_column("decks", "id", type_=sa.BigInteger(), existing_type=sa.Integer(), existing_nullable=False)
+    op.execute("ALTER SEQUENCE decks_id_seq AS bigint")
+    op.alter_column("cards", "id", type_=sa.BigInteger(), existing_type=sa.Integer(), existing_nullable=False)
+    op.execute("ALTER SEQUENCE cards_id_seq AS bigint")
+    op.alter_column("cards", "deck_id", type_=sa.BigInteger(), existing_type=sa.Integer(), nullable=False)
+
+
+def downgrade() -> None:
+    op.alter_column("cards", "deck_id", type_=sa.Integer(), existing_type=sa.BigInteger(), nullable=True)
+    op.execute("ALTER SEQUENCE cards_id_seq AS integer")
+    op.alter_column("cards", "id", type_=sa.Integer(), existing_type=sa.BigInteger(), existing_nullable=False)
+    op.execute("ALTER SEQUENCE decks_id_seq AS integer")
+    op.alter_column("decks", "id", type_=sa.Integer(), existing_type=sa.BigInteger(), existing_nullable=False)

tests/test_migrations.py

@@ -0,0 +1,38 @@
+import typing
+
+import sqlalchemy as sa
+from alembic.autogenerate import compare_metadata
+from alembic.runtime.migration import MigrationContext
+
+from app.models import METADATA
+from app.resources.db import create_primary_sa_engine
+
+
+def _compare_with_models(connection: sa.Connection) -> list[typing.Any]:
+    return compare_metadata(MigrationContext.configure(connection), METADATA)
+
+
+async def test_migrations_match_models() -> None:
+    engine: typing.Final = create_primary_sa_engine()
+    try:
+        async with engine.connect() as connection:
+            diff = await connection.run_sync(_compare_with_models)
+    finally:
+        await engine.dispose()
+
+    assert diff == []
+
+
+async def test_id_sequences_are_bigint() -> None:
+    engine: typing.Final = create_primary_sa_engine()
+    try:
+        async with engine.connect() as connection:
+            sequence_types = dict(
+                (await connection.execute(sa.text("SELECT sequence_name, data_type FROM information_schema.sequences")))
+                .tuples()
+                .all()
+            )
+    finally:
+        await engine.dispose()
+
+    assert sequence_types == {"decks_id_seq": "bigint", "cards_id_seq": "bigint"}