Removed @skipIfDBFeature silencing of nonexistent features.
Some checks failed
Linters / isort (push) Has been cancelled
Linters / black (push) Has been cancelled
Tests / Windows, SQLite, Python 3.13 (push) Has been cancelled
Tests / JavaScript tests (push) Has been cancelled
Linters / flake8 (push) Has been cancelled

Follow up to f5df7ed7e6.
This commit is contained in:
Tim Graham 2025-06-21 08:51:34 -04:00 committed by GitHub
parent f5df7ed7e6
commit 54402a7529
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 3 additions and 5 deletions

View file

@ -1582,9 +1582,7 @@ def _deferredSkip(condition, reason, name):
def skipIfDBFeature(*features):
"""Skip a test if a database has at least one of the named features."""
return _deferredSkip(
lambda: any(
getattr(connection.features, feature, False) for feature in features
),
lambda: any(getattr(connection.features, feature) for feature in features),
"Database has feature(s) %s" % ", ".join(features),
"skipIfDBFeature",
)

View file

@ -133,10 +133,10 @@ class SkippingTestCase(SimpleTestCase):
raise ValueError
self._assert_skipping(test_func, unittest.SkipTest)
self._assert_skipping(test_func2, ValueError)
self._assert_skipping(test_func2, AttributeError)
self._assert_skipping(test_func3, unittest.SkipTest)
self._assert_skipping(test_func4, unittest.SkipTest)
self._assert_skipping(test_func5, ValueError)
self._assert_skipping(test_func5, AttributeError)
class SkipTestCase(SimpleTestCase):
@skipIfDBFeature("missing")