You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by po...@apache.org on 2025/12/09 10:20:18 UTC
(airflow) branch main updated: Remove UUID case for sqlalachemy 2 from tests (#59230)
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new 4438c28bee1 Remove UUID case for sqlalachemy 2 from tests (#59230)
4438c28bee1 is described below
commit 4438c28bee19e52b0ad4221bac10ff3856117c43
Author: Jarek Potiuk <ja...@potiuk.com>
AuthorDate: Tue Dec 9 11:20:10 2025 +0100
Remove UUID case for sqlalachemy 2 from tests (#59230)
After #59109 - the team_name is used not id so no UUID suffix needs
to be added for SQLAlchemy 2 in tests when checking for detail
content.
---
airflow-core/tests/unit/api_fastapi/common/test_exceptions.py | 7 ++-----
1 file changed, 2 insertions(+), 5 deletions(-)
diff --git a/airflow-core/tests/unit/api_fastapi/common/test_exceptions.py b/airflow-core/tests/unit/api_fastapi/common/test_exceptions.py
index f77b741bcaa..f39a1f061e3 100644
--- a/airflow-core/tests/unit/api_fastapi/common/test_exceptions.py
+++ b/airflow-core/tests/unit/api_fastapi/common/test_exceptions.py
@@ -104,9 +104,6 @@ def get_unique_constraint_error_prefix():
return ""
-uuid_suffix = "" if SQLALCHEMY_V_1_4 else "::UUID"
-
-
class TestUniqueConstraintErrorHandler:
unique_constraint_error_handler = _UniqueConstraintErrorHandler()
@@ -151,7 +148,7 @@ class TestUniqueConstraintErrorHandler:
status_code=status.HTTP_409_CONFLICT,
detail={
"reason": "Unique constraint violation",
- "statement": f"INSERT INTO slot_pool (pool, slots, description, include_deferred, team_name) VALUES (%(pool)s, %(slots)s, %(description)s, %(include_deferred)s, %(team_name)s{uuid_suffix}) RETURNING slot_pool.id",
+ "statement": "INSERT INTO slot_pool (pool, slots, description, include_deferred, team_name) VALUES (%(pool)s, %(slots)s, %(description)s, %(include_deferred)s, %(team_name)s) RETURNING slot_pool.id",
"orig_error": 'duplicate key value violates unique constraint "slot_pool_pool_uq"\nDETAIL: Key (pool)=(test_pool) already exists.\n',
"message": MESSAGE,
},
@@ -180,7 +177,7 @@ class TestUniqueConstraintErrorHandler:
status_code=status.HTTP_409_CONFLICT,
detail={
"reason": "Unique constraint violation",
- "statement": f"INSERT INTO variable (key, val, description, is_encrypted, team_name) VALUES (%(key)s, %(val)s, %(description)s, %(is_encrypted)s, %(team_name)s{uuid_suffix}) RETURNING variable.id",
+ "statement": "INSERT INTO variable (key, val, description, is_encrypted, team_name) VALUES (%(key)s, %(val)s, %(description)s, %(is_encrypted)s, %(team_name)s) RETURNING variable.id",
"orig_error": 'duplicate key value violates unique constraint "variable_key_uq"\nDETAIL: Key (key)=(test_key) already exists.\n',
"message": MESSAGE,
},