Fix zio#2709: SQL Server top statement doesn't use parenthesis. (zio#…

…2710) Co-authored-by: Juliano Alves <von.juliano@gmail.com>
joelsonoda · Jun 27, 2023 · 29aabcf · 29aabcf
1 parent cc7ecb1
commit 29aabcf
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/quill-engine/src/main/scala/io/getquill/SQLServerDialect.scala b/quill-engine/src/main/scala/io/getquill/SQLServerDialect.scala
@@ -41,7 +41,7 @@ trait SQLServerDialect
     query: Statement
   )(implicit astTokenizer: Tokenizer[Ast], strategy: NamingStrategy) =
     Tokenizer[(Option[Ast], Option[Ast])] {
-      case (Some(limit), None)         => stmt"TOP ${limit.token} $query"
+      case (Some(limit), None)         => stmt"TOP (${limit.token}) $query"
       case (Some(limit), Some(offset)) => stmt"$query OFFSET ${offset.token} ROWS FETCH FIRST ${limit.token} ROWS ONLY"
       case (None, Some(offset))        => stmt"$query OFFSET ${offset.token} ROWS"
       case other                       => super.limitOffsetToken(query).token(other)

diff --git a/quill-sql/src/test/scala/io/getquill/context/sql/idiom/SQLServerDialectSpec.scala b/quill-sql/src/test/scala/io/getquill/context/sql/idiom/SQLServerDialectSpec.scala
@@ -26,7 +26,7 @@ class SQLServerDialectSpec extends Spec {
       qr1.take(15).map(t => t.i)
     }
     ctx.run(q).string mustEqual
-      "SELECT TOP 15 t.i FROM TestEntity t"
+      "SELECT TOP (15) t.i FROM TestEntity t"
   }
 
   "literal booleans" - {