Fix .count() in queries with group by defined (#59)

Author: G4brym

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "workers-qb",
-  "version": "1.6.2",
+  "version": "1.6.3",
   "description": "Zero dependencies Query Builder for Cloudflare Workers",
   "main": "./dist/index.js",
   "module": "./dist/index.mjs",

package.json

@@ -111,6 +111,7 @@ export class QueryBuilder<GenericResultWrapper, IsAsync extends boolean = true>
         ...params,
         fields: 'count(*) as total',
         offset: undefined,
+        groupBy: undefined,
         limit: 1,
       }),
       typeof params.where === 'object' && !Array.isArray(params.where) && params.where?.params
@@ -134,6 +135,7 @@ export class QueryBuilder<GenericResultWrapper, IsAsync extends boolean = true>
         ...params,
         fields: 'count(*) as total',
         offset: undefined,
+        groupBy: undefined,
         limit: 1,
       }),
       typeof params.where === 'object' && !Array.isArray(params.where) && params.where?.params

src/builder.ts

@@ -160,6 +160,22 @@ describe('Select Builder', () => {
     }
   })
 
+  test('count should remove group by', async () => {
+    for (const result of [
+      await new QuerybuilderTest()
+        .fetchAll({
+          tableName: 'testTable',
+          offset: 4,
+          groupBy: ['field'],
+        })
+        .count(),
+    ]) {
+      expect((result.results as any).query).toEqual('SELECT count(*) as total FROM testTable LIMIT 1')
+      expect((result.results as any).arguments).toEqual(undefined)
+      expect((result.results as any).fetchType).toEqual('ONE')
+    }
+  })
+
   test('select with simplified where list', async () => {
     for (const result of [
       new QuerybuilderTest().fetchOne({