]> 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:30:12 +0000 (14:30 -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
(cherry picked from commit 046934083bbf0193e29c990bed0527fdac56f9a8)

test/sql/test_defaults.py

index cbed21e26232a4e811868f26d8fe41cd8fa5b178..b7454cf198f9f6f402f5a230cc977296294d569e 100644 (file)
@@ -861,8 +861,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",