@pbvision/cloud-run-service 0.0.30 → 0.0.31

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pbvision/cloud-run-service",
-  "version": "0.0.30",
+  "version": "0.0.31",
   "description": "fastify-firestore-service Web Framework on Cloud Run",
   "license": "Apache-2.0",
   "type": "module",

package/src/analytics.js

@@ -21,7 +21,6 @@ export class DatabaseAPIWithAnalytics extends DatabaseAPI {
     super(fastify, req, reply)
     this.__analyticsEvents = []
     this.__analyticsUserProfileUpdates = {} // uid to $set/$set_once to changes
-    this.__analyticsSent = false
   }
 
   async postCommit (respData) {
@@ -31,7 +30,6 @@ export class DatabaseAPIWithAnalytics extends DatabaseAPI {
   }
 
   logAnalyticsEvent (mixpanelUserId, eventName, inputProperties, deviceId = null) {
-    assert(this.__analyticsSent === false)
     this.__analyticsEvents.push({
       event: eventName,
       properties: addSenderId(mixpanelUserId, {
@@ -60,14 +58,13 @@ export class DatabaseAPIWithAnalytics extends DatabaseAPI {
   }
 
   async sendAnalyticsEvents () {
-    // istanbul ignore if
-    if (this.__analyticsSent) {
-      return
-    }
-    this.__analyticsSent = true
+    const events = this.__analyticsEvents
+    this.__analyticsEvents = []
+    const userProfileUpdates = this.__analyticsUserProfileUpdates
+    this.__analyticsUserProfileUpdates = {}
 
     const promises = []
-    if (this.__analyticsEvents.length) {
+    if (events.length) {
       const uaData = {}
       const parser = new UAParser(this.req.headers['user-agent'])
       const browser = parser.getBrowser()
@@ -92,7 +89,7 @@ export class DatabaseAPIWithAnalytics extends DatabaseAPI {
         }
       }
 
-      for (const x of this.__analyticsEvents) {
+      for (const x of events) {
         Object.assign(x.properties, uaData)
       }
       // send all the events in one Mixpanel API call
@@ -100,12 +97,12 @@ export class DatabaseAPIWithAnalytics extends DatabaseAPI {
         method: 'POST',
         url: 'https://api.mixpanel.com/track',
         headers: { accept: 'text/plain' },
-        body: this.__analyticsEvents
+        body: events
       }))
     }
     const sent = []
-    for (const type of Object.keys(this.__analyticsUserProfileUpdates)) {
-      const updatesByUser = this.__analyticsUserProfileUpdates[type]
+    for (const type of Object.keys(userProfileUpdates)) {
+      const updatesByUser = userProfileUpdates[type]
       const body = []
       for (const uid of Object.keys(updatesByUser)) {
         const updates = updatesByUser[uid]

package/src/placeholder.js

@@ -54,12 +54,18 @@ export class TestAnalyticsAPI extends DatabaseAPIWithAnalytics {
   async computeResponse () {
     // istanbul ignore next
     assert(isUnitTesting || isLocalhost)
-    const { eventCalls, profileUpdates } = this.req.body
+    const { eventCalls, profileUpdates, moreEventCalls } = this.req.body
     for (const x of (eventCalls ?? [])) {
       this.logAnalyticsEvent(...x)
     }
     for (const x of (profileUpdates ?? [])) {
       this.updateAnalyticsUserProfile(...x)
     }
+    if (moreEventCalls) {
+      await this.sendAnalyticsEvents()
+      for (const x of moreEventCalls) {
+        this.logAnalyticsEvent(...x)
+      }
+    }
   }
 }

package/test/unit-test-analytics.js

@@ -63,6 +63,39 @@ class AnalyticsTest extends AppTest {
     }).expect(expRespCode)
   }
 
+  async testSendInBatches () {
+    await this.app.post('/analytics')
+      .send({
+        eventCalls: [
+          ['xyz', 'some event', { cool: 1, hi: 'world' }]
+        ],
+        moreEventCalls: [
+          ['xyz', 'another event', { x: 2 }]
+        ]
+      }).expect(200)
+    const calls = this.fetchMock.mock.calls
+    expect(calls.length).toBe(2)
+    const actualBody = JSON.parse(calls[0][1].body)
+    expect(actualBody).toEqual([{
+      event: 'some event',
+      properties: expect.objectContaining({
+        cool: 1,
+        hi: 'world',
+        $user_id: 'xyz',
+        token: mixpanelToken
+      })
+    }])
+    const actualBody2 = JSON.parse(calls[1][1].body)
+    expect(actualBody2).toEqual([{
+      event: 'another event',
+      properties: expect.objectContaining({
+        x: 2,
+        $user_id: 'xyz',
+        token: mixpanelToken
+      })
+    }])
+  }
+
   async testDeviceId () {
     await this.app.post('/analytics')
       .send({