Merge pull request #2924 from specklesystems/fabians/stats-ioc-3

chore(server): stats IoC 3 - getTotalObjectCountFactory

packages/server/modules/stats/graph/resolvers/stats.ts

@@ -8,7 +8,7 @@ import {
   getUserHistory,
   getTotalStreamCountFactory,
   getTotalCommitCountFactory,
-  getTotalObjectCount,
+  getTotalObjectCountFactory,
   getTotalUserCount
 } from '@/modules/stats/repositories/index'
 import { Roles, Scopes } from '@speckle/shared'
@@ -37,7 +37,7 @@ export = {
     },
 
     async totalObjectCount() {
-      return await getTotalObjectCount()
+      return await getTotalObjectCountFactory({ db })()
     },
 
     async totalUserCount() {

packages/server/modules/stats/repositories/index.ts

@@ -13,9 +13,9 @@ export const getTotalCommitCountFactory = (deps: { db: Knex }) => async () => {
   return parseInt(result.rows[0].count)
 }
 
-export async function getTotalObjectCount() {
+export const getTotalObjectCountFactory = (deps: { db: Knex }) => async () => {
   const query = 'SELECT COUNT(*) FROM objects'
-  const result = await knex.raw(query)
+  const result = await deps.db.raw(query)
   return parseInt(result.rows[0].count)
 }
 

packages/server/modules/stats/tests/stats.spec.ts

@@ -16,7 +16,7 @@ import {
   getUserHistory,
   getTotalStreamCountFactory,
   getTotalCommitCountFactory,
-  getTotalObjectCount,
+  getTotalObjectCountFactory,
   getTotalUserCount
 } from '@/modules/stats/repositories/index'
 import { Scopes } from '@speckle/shared'
@@ -49,7 +49,7 @@ describe('Server stats services @stats-services', function () {
   })
 
   it('should return the total number of objects on this server', async () => {
-    const res = await getTotalObjectCount()
+    const res = await getTotalObjectCountFactory({ db })()
     expect(res).to.equal(params.numObjects)
   })