]> git.ipfire.org Git - thirdparty/sqlalchemy/sqlalchemy.git/commitdiff
Repair CTE a in b tests
authorMike Bayer <mike_mp@zzzcomputing.com>
Fri, 10 Apr 2020 18:28:54 +0000 (14:28 -0400)
committerMike Bayer <mike_mp@zzzcomputing.com>
Fri, 10 Apr 2020 18:28:54 +0000 (14:28 -0400)
In I27cac9bd265c86ff2a3381ff9f844f60ef991cfc we modernized
the default tests and converted the "a in b" CTE tests to combinations,
however apparently the existing tests were not testing all
combinations and had repeats instead.   The combinations
decorator has made this much easier to spot, so use
the correct combinations that were originally intended.

Change-Id: Icd904887bff00c31525497d0b1508fabaf052dc9

test/sql/test_defaults.py

index 52e73b22331c2d3d1eb0cd32000d3c157e75e30d..0a50a6356c39f2622c2c961e212e282b08a6916e 100644 (file)
@@ -858,8 +858,8 @@ class CTEDefaultTest(fixtures.TablesTest):
 
     @testing.combinations(
         ("update", "select", testing.requires.ctes_on_dml),
-        ("update", "select", testing.requires.ctes_on_dml),
-        ("update", "select", testing.requires.ctes_on_dml),
+        ("delete", "select", testing.requires.ctes_on_dml),
+        ("insert", "select", testing.requires.ctes_on_dml),
         ("select", "update"),
         ("select", "insert"),
         argnames="a, b",