From 042aea980d61024fab3abac55e9e32b6b9ff6692 Mon Sep 17 00:00:00 2001 From: Mike Bayer Date: Sat, 9 Nov 2019 13:12:13 -0500 Subject: Fix exclusions for multiple fails_on The fails_on decorator was not being interpreted correctly when multiple were present. Remove obsolete fails_on from test_types that no longer take place for MySQL, Oracle. Ensure test_types tests are using __backend__ mark currently failing Oracle interval tests Change-Id: If8db0c02b31a8008fd1673c2380f1f974c3806a6 --- lib/sqlalchemy/testing/exclusions.py | 21 ++++++++++----------- 1 file changed, 10 insertions(+), 11 deletions(-) (limited to 'lib/sqlalchemy') diff --git a/lib/sqlalchemy/testing/exclusions.py b/lib/sqlalchemy/testing/exclusions.py index edb9caa16..8f63fcbf8 100644 --- a/lib/sqlalchemy/testing/exclusions.py +++ b/lib/sqlalchemy/testing/exclusions.py @@ -141,19 +141,18 @@ class compound(object): def _expect_success(self, config, name="block"): if not self.fails: return + for fail in self.fails: - if not fail(config): - break - else: - raise AssertionError( - "Unexpected success for '%s' (%s)" - % ( - name, - " and ".join( - fail._as_string(config) for fail in self.fails - ), + if fail(config): + raise AssertionError( + "Unexpected success for '%s' (%s)" + % ( + name, + " and ".join( + fail._as_string(config) for fail in self.fails + ), + ) ) - ) def requires_tag(tagname): -- cgit v1.2.1