From: Mike Bayer <mike_mp@zzzcomputing.com>
Date: Sat, 1 Nov 2025 03:39:37 +0000 (-0400)
Subject: update the fk_names req
X-Git-Tag: rel_1_17_2~5
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=20c108720d7127ca;p=thirdparty%2Fsqlalchemy%2Falembic.git

update the fk_names req

This is not a limitation this is just a bug in sqlalchemy,
being fixed by #12954.    we dont have a system in place
to cross-check gerrits right now so just merge this and then
merge the SQLAlchemy fix

Change-Id: I9c074c0edb9f55d2400e59a9b2b02603458e8f0b
---

diff --git a/alembic/testing/suite/test_autogen_fks.py b/alembic/testing/suite/test_autogen_fks.py
index 0240b98d..355e244c 100644
--- a/alembic/testing/suite/test_autogen_fks.py
+++ b/alembic/testing/suite/test_autogen_fks.py
@@ -199,6 +199,7 @@ class AutogenerateForeignKeysTest(AutogenFixtureTest, TestBase):
 
         eq_(diffs, [])
 
+    @config.requirements.fk_names
     def test_casing_convention_changed_so_put_drops_first(self):
         m1 = MetaData()
         m2 = MetaData()
@@ -247,7 +248,7 @@ class AutogenerateForeignKeysTest(AutogenFixtureTest, TestBase):
             ["test2"],
             "some_table",
             ["test"],
-            name="MyFK" if config.requirements.fk_names.enabled else None,
+            name="MyFK",
         )
 
         self._assert_fk_diff(
diff --git a/tests/requirements.py b/tests/requirements.py
index 4db5579e..b8c8a680 100644
--- a/tests/requirements.py
+++ b/tests/requirements.py
@@ -62,8 +62,13 @@ class DefaultRequirements(SuiteRequirements):
 
     @property
     def fk_names(self):
-        """foreign key constraints always have names in the DB"""
-        return exclusions.fails_on("sqlite")
+        """backend can reflect foreign key names"""
+
+        # issue here was fixed in SQLAlchemy #12954 for sqlite, 2.0
+        # release
+        return exclusions.fails_if(
+            lambda config: not sqla_compat.sqla_2 and config.against("sqlite")
+        )
 
     @property
     def reflects_fk_options(self):