From 58def02cd12ff9248d74bb84b9b07c4628a341ed Mon Sep 17 00:00:00 2001 From: Rob Bygrave Date: Mon, 26 Feb 2018 14:41:56 +1300 Subject: [PATCH] #1280 - Update test for - Remove "order by" clause of RawSql query when executing findCount() The code fix associated with this is in CQueryBuilder line 276 --- src/test/java/org/tests/rawsql/TestRawSqlOrmQuery.java | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/src/test/java/org/tests/rawsql/TestRawSqlOrmQuery.java b/src/test/java/org/tests/rawsql/TestRawSqlOrmQuery.java index 5a9d4dd33..ffa8b83c8 100644 --- a/src/test/java/org/tests/rawsql/TestRawSqlOrmQuery.java +++ b/src/test/java/org/tests/rawsql/TestRawSqlOrmQuery.java @@ -228,10 +228,7 @@ public class TestRawSqlOrmQuery extends BaseTestCase { query.order("id desc"); PagedList pagedList = query.findPagedList(); pagedList.getList(); - if (!isSqlServer()) { - // sql server doesn't support order by in the count query, I wonder if we can remove it? - pagedList.getTotalCount(); - } + pagedList.getTotalCount(); if (isSqlServer()) { assertThat(sqlOf(query)).contains("select top 100 ");