fix: moved dependency to depends_on

* we have created reverse dependency from invenio_accounts,
  so this one should be as dependency and not down revision
  as well.
* added missing alembic migration
* added alembic tests

Author: mesemus

invenio_communities/alembic/02cd82910727_update_role_id_type_upgrade.py

@@ -20,11 +20,14 @@
 # revision identifiers, used by Alembic.
 revision = "02cd82910727"
 down_revision = (
-    "f9843093f686",
+    # our own 37b21951084c_update_role_id_type_downgrade.py
     "37b21951084c",
-)  # Depends on invenio-access revision id (f9843093f686)
+)
 branch_labels = ()
-depends_on = None
+depends_on = [
+    # invenio_accounts/alembic/f2522cdd5fcd_change_accountsrole_primary_key_to_string.py
+    "f2522cdd5fcd",
+]
 
 
 def upgrade():

invenio_communities/alembic/1777209602_removed_index_by_file_id_added_indices_.py

@@ -0,0 +1,54 @@
+#
+# This file is part of Invenio.
+# Copyright (C) 2016-2018 CERN.
+# Copyright (C) 2026 CESNET z.s.p.o.
+#
+# Invenio is free software; you can redistribute it and/or modify it
+# under the terms of the MIT License; see LICENSE file for more details.
+
+"""Removed index by file id, added indices by community and community + file key."""
+
+from alembic import op
+
+# revision identifiers, used by Alembic.
+revision = "1777209602"
+down_revision = "dfbc96c5211f"
+branch_labels = ()
+depends_on = None
+
+
+def upgrade():
+    """Upgrade database."""
+    # ### commands auto generated by Alembic - please adjust! ###
+    op.drop_index(op.f("uidx_communities_files_id_key"), table_name="communities_files")
+    op.create_index(
+        op.f("ix_communities_files_record_id"),
+        "communities_files",
+        ["record_id"],
+        unique=False,
+    )
+    op.create_index(
+        "uidx_communities_files_record_id_key",
+        "communities_files",
+        ["record_id", "key"],
+        unique=True,
+    )
+    # ### end Alembic commands ###
+
+
+def downgrade():
+    """Downgrade database."""
+    # ### commands auto generated by Alembic - please adjust! ###
+    op.drop_index(
+        "uidx_communities_files_record_id_key", table_name="communities_files"
+    )
+    op.drop_index(
+        op.f("ix_communities_files_record_id"), table_name="communities_files"
+    )
+    op.create_index(
+        op.f("uidx_communities_files_id_key"),
+        "communities_files",
+        ["id", "key"],
+        unique=True,
+    )
+    # ### end Alembic commands ###

tests/test_alembic.py

@@ -0,0 +1,47 @@
+# -*- coding: utf-8 -*-
+#
+# Copyright (C) 2023 CERN.
+# Copyright (C) 2024 Graz University of Technology.
+# Copyright (C) 2026 CESNET z.s.p.o.
+#
+# Invenio-communities is free software; you can redistribute it and/or modify
+# it under the terms of the MIT License; see LICENSE file for more details.
+"""Test invenio-communities alembic."""
+
+import pytest
+from invenio_db.utils import alembic_test_context, drop_alembic_version_table
+
+
+def test_alembic(base_app, database, extra_entry_points):
+    """Test alembic recipes."""
+    db = database
+    ext = base_app.extensions["invenio-db"]
+
+    if db.engine.name == "sqlite":
+        raise pytest.skip("Upgrades are not supported on SQLite.")
+
+    base_app.config["ALEMBIC_CONTEXT"] = alembic_test_context()
+
+    # Check that this package's SQLAlchemy models have been properly registered
+    tables = [x for x in db.metadata.tables]
+    assert "communities_metadata" in tables
+    assert "communities_members" in tables
+
+    # Check that Alembic agrees that there's no further tables to create.
+    assert len(ext.alembic.compare_metadata()) == 0
+
+    # Drop everything and recreate tables all with Alembic
+    db.drop_all()
+    drop_alembic_version_table()
+    ext.alembic.upgrade()
+
+    # expected non-covered migrations (from previous tests that use mock_module):
+    # ('add_table', Table('mock_metadata'
+    # ('add_table', Table('mock_community'
+    for m in ext.alembic.compare_metadata():
+        if not (
+            m[0] == "add_table" and m[1].name in ["mock_metadata", "mock_community"]
+        ):
+            raise RuntimeError(f"Unexpected migration: {m}")
+
+    drop_alembic_version_table()