diff --git a/django/test/testcases.py b/django/test/testcases.py index 37529603be..d4d61c8fba 100644 --- a/django/test/testcases.py +++ b/django/test/testcases.py @@ -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", ) diff --git a/tests/test_utils/tests.py b/tests/test_utils/tests.py index fa2bee7870..494a0ea8d3 100644 --- a/tests/test_utils/tests.py +++ b/tests/test_utils/tests.py @@ -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")