]> git.ipfire.org Git - thirdparty/sqlalchemy/sqlalchemy.git/commitdiff
Remove erroneous skip message
authorMike Bayer <mike_mp@zzzcomputing.com>
Fri, 16 Feb 2018 23:13:30 +0000 (18:13 -0500)
committerMike Bayer <mike_mp@zzzcomputing.com>
Fri, 16 Feb 2018 23:13:30 +0000 (18:13 -0500)
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

lib/sqlalchemy/testing/plugin/plugin_base.py

index 6b08ef49ab82f0472da3f644ce4eb6465c970cec..1f74321a8b39caf67c786f91febbbba2b58148e0 100644 (file)
@@ -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()