Revert "Fixes offset limit duplication for SqlServer (#490)"

This reverts commit f4a680b.
sebastienros · Oct 11, 2023 · af1bb31 · af1bb31
1 parent 4bb7978
commit af1bb31
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 6 deletions.
diff --git a/src/YesSql.Provider.SqlServer/SqlServerDialect.cs b/src/YesSql.Provider.SqlServer/SqlServerDialect.cs
@@ -171,8 +171,6 @@ public override string GetTypeName(DbType dbType, int? length, byte? precision,
 
         public override void Page(ISqlBuilder sqlBuilder, string offset, string limit)
         {
-            sqlBuilder.ClearTrail();
-
             if (offset != null)
             {
                 sqlBuilder.Trail(" OFFSET ");

diff --git a/test/YesSql.Tests/CoreTests.cs b/test/YesSql.Tests/CoreTests.cs
@@ -45,7 +45,7 @@ public async Task InitializeAsync()
             if (_configuration == null)
             {
                 _configuration = CreateConfiguration();
-
+                
                 CleanDatabase(_configuration, false);
 
                 _store = await StoreFactory.CreateAndInitializeAsync(_configuration);
@@ -2914,9 +2914,6 @@ public async Task ShouldPageResults()
                 {
                     Assert.Equal(1, String.Compare(ordered[i].SomeName, ordered[i - 1].SomeName));
                 }
-
-                //query should discard previous skip/take
-                Assert.Equal(5, (await query.Skip(10).Take(5).ListAsync()).Count());
             }
         }