[#93048] server: tests: Pass test_model_definitions_match_ddl

sa.String() was turning into VARCHAR, contrary to the DDL definitions
which were denoting these fields as TEXT

Author: Kacper Zienkiewicz

server/alembic/versions/5_add_tags.py

@@ -21,7 +21,7 @@
 def upgrade() -> None:
     op.create_table('devices_tags',
     sa.Column('device_id', sa.Integer(), nullable=False),
-    sa.Column('tag', sa.String(), nullable=False),
+    sa.Column('tag', sa.Text(), nullable=False),
     sa.ForeignKeyConstraint(['device_id'], ['devices.id'], ondelete='RESTRICT'),
     sa.PrimaryKeyConstraint('device_id', 'tag')
     )

server/alembic/versions/6_add_action_logs.py

@@ -21,10 +21,10 @@
 def upgrade() -> None:
     op.create_table('action_logs',
     sa.Column('id', sa.String(), nullable=False),
-    sa.Column('action_id', sa.String(), nullable=False),
-    sa.Column('mac_address', sa.String(), nullable=False),
+    sa.Column('action_id', sa.Text(), nullable=False),
+    sa.Column('mac_address', sa.Text(), nullable=False),
     sa.Column('created', sa.DateTime(), nullable=False),
-    sa.Column('status', sa.String(), nullable=False),
+    sa.Column('status', sa.Text(), nullable=False),
     sa.PrimaryKeyConstraint('id'),
     )
 

server/alembic/versions/7_add_device_updates.py

@@ -21,9 +21,9 @@
 def upgrade() -> None:
     op.create_table('device_updates',
     sa.Column('id', sa.Integer(), autoincrement=True, nullable=False),
-    sa.Column('mac_address', sa.String(), nullable=False),
+    sa.Column('mac_address', sa.Text(), nullable=False),
     sa.Column('created', sa.DateTime(), nullable=False),
-    sa.Column('version', sa.String(), nullable=False),
+    sa.Column('version', sa.Text(), nullable=False),
     sa.Column('progress', sa.Integer(), nullable=False),
     sa.PrimaryKeyConstraint('id'),
     )

server/alembic/versions/9_action_log_url.py

@@ -19,7 +19,7 @@
 
 
 def upgrade() -> None:
-    op.add_column('action_logs', sa.Column('download_url', sa.String(), nullable=True))
+    op.add_column('action_logs', sa.Column('download_url', sa.Text(), nullable=True))
 
 
 def downgrade() -> None:

server/src/models/models.py

@@ -6,3 +6,4 @@
 from models.log import Log
 from models.permission import Permission
 from models.action_log import ActionLog
+from models.device_update import DeviceUpdate

server/tests/test-alembic-migrations.py

@@ -4,6 +4,7 @@
 # This tests the whole migration chain down to up.
 # It is parametrized to run on PostgreSQL and SQLite.
 from pytest_alembic.tests import test_upgrade
+from pytest_alembic.tests import test_model_definitions_match_ddl
 
 def test_migration_8(alembic_engine, alembic_runner):
     """