From 96dffd0ae9d19050f1d6000c35c4347bcb2dc6fe Mon Sep 17 00:00:00 2001 From: Mike Bayer Date: Thu, 25 Mar 2021 10:37:12 -0400 Subject: [PATCH] don't rely on pk-only group_by Fixed the test test_aliased_stmt_includes_unnamed_fn recently added in c1c999c01d to include the additional column in the GROUP BY statement, allowing this statement to succeed on SQL Server which seems to not support "group by pk only" syntax. Change-Id: I42184d77c9017ebdb76ac5a9ae1f8128561243ec --- test/orm/test_eager_relations.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/orm/test_eager_relations.py b/test/orm/test_eager_relations.py index b42c704310..4cb6932cbc 100644 --- a/test/orm/test_eager_relations.py +++ b/test/orm/test_eager_relations.py @@ -513,7 +513,7 @@ class EagerTest(_fixtures.FixtureTest, testing.AssertsCompiledSQL): q = ( s.query(User, func.count(User.id)) .order_by(User.id) - .group_by(User.id) + .group_by(User.id, User.name) .limit(1) ) -- 2.47.2