diff options
| author | Mike Bayer <mike_mp@zzzcomputing.com> | 2018-02-16 18:13:30 -0500 |
|---|---|---|
| committer | Mike Bayer <mike_mp@zzzcomputing.com> | 2018-02-16 18:13:30 -0500 |
| commit | c94e4d888a87b82c062c5ecb9d97fca89bfdde12 (patch) | |
| tree | ab782c81e70f90e50915c6d3678254a5dce279bb /lib/sqlalchemy/testing/plugin | |
| parent | 0cdd822207aedd4a10235146dd2efc81f3a26344 (diff) | |
| download | sqlalchemy-c94e4d888a87b82c062c5ecb9d97fca89bfdde12.tar.gz | |
Remove erroneous skip message
The logic here seems to try to say something different when
__backend__ is True but it produces a nonsensical message,
since __only_on__ doesn't tell us about the implementation
we are actually running.
Change-Id: I14e20cadcba975f8efe8fdefa439c8b8c480b3ed
Diffstat (limited to 'lib/sqlalchemy/testing/plugin')
| -rw-r--r-- | lib/sqlalchemy/testing/plugin/plugin_base.py | 32 |
1 files changed, 14 insertions, 18 deletions
diff --git a/lib/sqlalchemy/testing/plugin/plugin_base.py b/lib/sqlalchemy/testing/plugin/plugin_base.py index 6b08ef49a..1f74321a8 100644 --- a/lib/sqlalchemy/testing/plugin/plugin_base.py +++ b/lib/sqlalchemy/testing/plugin/plugin_base.py @@ -554,24 +554,20 @@ def _do_skips(cls): ) if not all_configs: - if getattr(cls, '__backend__', False): - msg = "'%s' unsupported for implementation '%s'" % ( - cls.__name__, cls.__only_on__) - else: - msg = "'%s' unsupported on any DB implementation %s%s" % ( - cls.__name__, - ", ".join( - "'%s(%s)+%s'" % ( - config_obj.db.name, - ".".join( - str(dig) for dig in - config_obj.db.dialect.server_version_info), - config_obj.db.driver - ) - for config_obj in config.Config.all_configs() - ), - ", ".join(reasons) - ) + msg = "'%s' unsupported on any DB implementation %s%s" % ( + cls.__name__, + ", ".join( + "'%s(%s)+%s'" % ( + config_obj.db.name, + ".".join( + str(dig) for dig in + config_obj.db.dialect.server_version_info), + config_obj.db.driver + ) + for config_obj in config.Config.all_configs() + ), + ", ".join(reasons) + ) config.skip_test(msg) elif hasattr(cls, '__prefer_backends__'): non_preferred = set() |
