Remove debug logging

Author: kevin-dp

packages/db/src/query/live/collection-config-builder.ts

@@ -113,7 +113,6 @@ export class CollectionConfigBuilder<
       // no nested runs of the graph
       // which is possible if the `callback`
       // would call `maybeRunGraph` e.g. after it has loaded some more data
-      console.log(`IGNORING MAYBERUNGRAPH`)
       return
     }
 
@@ -128,11 +127,8 @@ export class CollectionConfigBuilder<
         syncState.subscribedToAllCollections
       ) {
         while (syncState.graph.pendingWork()) {
-          console.log(`RUNNING GRAPH`)
           syncState.graph.run()
-          console.log(`Calling callback`)
           callback?.()
-          console.log(`returned from callback`)
         }
 
         // On the initial run, we may need to do an empty commit to ensure that

packages/db/tests/query/distinct.test.ts

@@ -707,11 +707,11 @@ function createDistinctTests(autoIndex: `off` | `eager`): void {
               .distinct(),
         })
 
-        const results = Array.from(distinctJoinedData.values())
-
         // There are 3 distinct departments that have active users
         expect(distinctJoinedData.size).toBe(3)
 
+        const results = Array.from(distinctJoinedData.values())
+
         // Should have distinct combinations of department
         const combinations = results.map((r) => `${r.department}`)
         const uniqueCombinations = [...new Set(combinations)]

packages/db/tests/query/order-by.test.ts

@@ -257,10 +257,8 @@ function createOrderByTests(autoIndex: `off` | `eager`): void {
     let departmentsCollection: ReturnType<typeof createDepartmentsCollection>
 
     beforeEach(() => {
-      console.log(`before each`)
       employeesCollection = createEmployeesCollection(autoIndex)
       departmentsCollection = createDepartmentsCollection(autoIndex)
-      console.log(`end of before each`)
     })
 
     describe(`Basic OrderBy`, () => {
@@ -407,7 +405,6 @@ function createOrderByTests(autoIndex: `off` | `eager`): void {
       })
 
       it(`applies offset correctly with ordering`, async () => {
-        console.log(`in applies offset correctly with ordering`)
         const collection = createLiveQueryCollection((q) =>
           q
             .from({ employees: employeesCollection })
@@ -428,7 +425,6 @@ function createOrderByTests(autoIndex: `off` | `eager`): void {
       })
 
       it(`applies both limit and offset with ordering`, async () => {
-        console.log(`in applies both limit and offset with ordering`)
         const collection = createLiveQueryCollection((q) =>
           q
             .from({ employees: employeesCollection })
@@ -450,7 +446,6 @@ function createOrderByTests(autoIndex: `off` | `eager`): void {
       })
 
       it(`throws error when limit/offset used without orderBy`, () => {
-        console.log(`in throws error when limit/offset used without orderBy`)
         expect(() => {
           createLiveQueryCollection((q) =>
             q
@@ -467,9 +462,6 @@ function createOrderByTests(autoIndex: `off` | `eager`): void {
       })
 
       it(`applies incremental insert of a new row before the topK correctly`, async () => {
-        console.log(
-          `in applies incremental insert of a new row before the topK correctly`
-        )
         const collection = createLiveQueryCollection((q) =>
           q
             .from({ employees: employeesCollection })
@@ -517,9 +509,6 @@ function createOrderByTests(autoIndex: `off` | `eager`): void {
       })
 
       it(`applies incremental insert of a new row inside the topK correctly`, async () => {
-        console.log(
-          `in applies incremental insert of a new row inside the topK correctly`
-        )
         const collection = createLiveQueryCollection((q) =>
           q
             .from({ employees: employeesCollection })
@@ -564,13 +553,9 @@ function createOrderByTests(autoIndex: `off` | `eager`): void {
           [6, 62_000],
           [2, 60_000],
         ])
-        console.log(`end`)
       })
 
       it(`applies incremental insert of a new row inside the topK but after max sent value correctly`, async () => {
-        console.log(
-          `in applies incremental insert of a new row inside the topK but after max sent value correctly`
-        )
         const collection = createLiveQueryCollection((q) =>
           q
             .from({ employees: employeesCollection })
@@ -622,9 +607,6 @@ function createOrderByTests(autoIndex: `off` | `eager`): void {
       })
 
       it(`applies incremental insert of a new row after the topK correctly`, async () => {
-        console.log(
-          `in applies incremental insert of a new row after the topK correctly`
-        )
         const collection = createLiveQueryCollection((q) =>
           q
             .from({ employees: employeesCollection })
@@ -668,9 +650,6 @@ function createOrderByTests(autoIndex: `off` | `eager`): void {
       })
 
       it(`applies incremental update of a row inside the topK correctly`, async () => {
-        console.log(
-          `in applies incremental update of a row inside the topK correctly`
-        )
         const collection = createLiveQueryCollection((q) =>
           q
             .from({ employees: employeesCollection })
@@ -714,9 +693,6 @@ function createOrderByTests(autoIndex: `off` | `eager`): void {
       })
 
       it(`applies incremental delete of a row in the topK correctly`, async () => {
-        console.log(
-          `in applies incremental delete of a row in the topK correctly`
-        )
         const collection = createLiveQueryCollection((q) =>
           q
             .from({ employees: employeesCollection })
@@ -1871,6 +1847,6 @@ function createOrderByTests(autoIndex: `off` | `eager`): void {
 }
 
 describe(`Query2 OrderBy Compiler`, () => {
-  // createOrderByTests(`off`)
+  createOrderByTests(`off`)
   createOrderByTests(`eager`)
 })