- union() and union_all() will not whack

any order_by() that has been applied to the
select()s inside.  If you union() a
select() with order_by() (presumably to support
LIMIT/OFFSET), you should also call self_group()
on it to apply parenthesis.

1 files changed, 10 insertions, 0 deletions

diff --git a/test/sql/select.py b/test/sql/select.py
index 91585e37e..352eed42c 100644
--- a/test/sql/select.py
+++ b/test/sql/select.py
@@ -982,6 +982,16 @@ UNION SELECT mytable.myid FROM mytable"
         s = union(s, s)
         self.assert_compile(s, "SELECT foo, bar UNION SELECT foo, bar UNION (SELECT foo, bar UNION SELECT foo, bar)")
         
+        s = select([column('foo'), column('bar')])
+        # ORDER BY's even though not supported by all DB's, are rendered if requested
+        self.assert_compile(union(s.order_by("foo"), s.order_by("bar")), 
+            "SELECT foo, bar ORDER BY foo UNION SELECT foo, bar ORDER BY bar"
+        )
+        # self_group() is honored
+        self.assert_compile(union(s.order_by("foo").self_group(), s.order_by("bar").limit(10).self_group()), 
+            "(SELECT foo, bar ORDER BY foo) UNION (SELECT foo, bar ORDER BY bar  LIMIT 10)"
+        )
+        
 
     @testing.uses_deprecated()
     def test_binds(self):