Skip www tests for db isolation mode (#41131)

* Remove incremental upper-bind limit The twisted/incremental#106 issue has been addressed in 24.7.2 so we are removing the limit - just in case we also exclude the buggy versions, even if they are yanked. * Skip DB isolation tests for www package Related: #41067 GitOrigin-RevId: 63e7c6354a8e59ac625c88f61cf8730f60d4a6b3
GoogleCloudPlatform · Nov 9, 2024 · 2da98c8 · 2da98c8
1 parent 963171a
commit 2da98c8
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 3 deletions.
diff --git a/hatch_build.py b/hatch_build.py
@@ -201,9 +201,8 @@
         "click>=8.0",
         "gitpython>=3.1.40",
         "hatch>=1.9.1",
-        # Incremental 24.7.0 has broken `python -m virtualenv` command when run in /opt/airflow directory
-        # This limit should be removed after fixing https://github.com/twisted/incremental/issues/106
-        "incremental<=22.10.0",
+        # Incremental 24.7.0, 24.7.1 has broken `python -m virtualenv` command when run in /opt/airflow directory
+        "incremental!=24.7.0,!=24.7.1,>=22.10.0",
         "pipdeptree>=2.13.1",
         "pygithub>=2.1.1",
         "restructuredtext-lint>=1.4.0",

diff --git a/tests/www/test_auth.py b/tests/www/test_auth.py
@@ -29,6 +29,8 @@
 
 mock_call = Mock()
 
+pytestmark = pytest.mark.skip_if_database_isolation_mode
+
 
 class TestHasAccessDecorator:
     def test_has_access_decorator_raises_deprecation_warning(self):

diff --git a/tests/www/test_security_manager.py b/tests/www/test_security_manager.py
@@ -32,6 +32,8 @@
 )
 from airflow.www import app as application
 
+pytestmark = pytest.mark.skip_if_database_isolation_mode
+
 
 @pytest.fixture
 def app():