adapt-authoring-spoortracking 1.1.0 → 1.1.1

package/lib/SpoorTrackingModule.js

@@ -34,7 +34,7 @@ class SpoorTrackingModule extends AbstractModule {
       return
     }
     const content = await this.app.waitForModule('content')
-    const [{ _trackingId }] = await content.find({ _courseId: data._courseId }, {}, { limit: 1, sort: [['_trackingId', -1]] })
+    const [{ _trackingId } = {}] = await content.find({ _courseId: data._courseId }, {}, { limit: 1, sort: [['_trackingId', -1]] })
     data._trackingId = (_trackingId ?? 0) + 1
   }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "adapt-authoring-spoortracking",
-  "version": "1.1.0",
+  "version": "1.1.1",
   "description": "Module for making course content compatible with spoor",
   "homepage": "https://github.com/adapt-security/adapt-authoring-spoortracking",
   "license": "GPL-3.0",

package/tests/SpoorTrackingModule.spec.js

@@ -127,17 +127,11 @@ describe('SpoorTrackingModule', () => {
       assert.equal(data._trackingId, 1)
     })
 
-    // TODO: BUG - insertTrackingId crashes when content.find returns an empty array
-    // Destructuring `[{ _trackingId }]` from an empty array gives `undefined`,
-    // causing a TypeError. This happens when a block is the first content item
-    // inserted for a course.
     it('should handle empty find result gracefully', async () => {
       instance._contentMock.find.mock.mockImplementation(async () => [])
       const data = { _type: 'block', _courseId: 'emptyCourse' }
-      await assert.rejects(
-        () => instance.insertTrackingId(data),
-        TypeError
-      )
+      await instance.insertTrackingId(data)
+      assert.equal(data._trackingId, 1)
     })
   })