adapt-authoring-tags 1.3.1 → 1.3.3

package/.github/workflows/standardjs.yml

@@ -1,4 +1,4 @@
-name: Standard.js formatting check
+name: Lint
 on: push
 jobs:
   default:
@@ -9,4 +9,4 @@ jobs:
         with:
           node-version: 'lts/*'
       - run: npm install
-      - run: npx standard
+      - run: npx standard

package/lib/TagsModule.js

@@ -102,11 +102,11 @@ class TagsModule extends AbstractApiModule {
   }
 
   /** @override */
-  async insert (...args) {
+  async insert (data, options = {}, mongoOptions = {}) {
     // allow attempts to create tags that already exist
-    const [tag] = await this.find(...args)
+    const tag = await this.findOne(data, { ...options, strict: false }, mongoOptions)
     if (tag) return tag
-    return super.insert(...args)
+    return super.insert(data, options, mongoOptions)
   }
 
   /** @override */

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "adapt-authoring-tags",
-  "version": "1.3.1",
+  "version": "1.3.3",
   "description": "Module for managing tags",
   "homepage": "https://github.com/adapt-security/adapt-authoring-tags",
   "license": "GPL-3.0",

package/tests/TagsModule.spec.js

@@ -478,16 +478,16 @@ describe('TagsModule', () => {
       assert.equal(parentInsert.mock.calls.length, 1)
     })
 
-    it('should pass through all arguments to find', async () => {
+    it('should pass through all arguments to findOne', async () => {
       const { instance } = createInstance()
-      instance.find = mock.fn(async () => [{ _id: 'tag1', title: 'test' }])
+      instance.findOne = mock.fn(async () => ({ _id: 'tag1', title: 'test' }))
       const data = { title: 'test' }
       const options = { validate: false }
       const mongoOpts = { limit: 1 }
       await instance.insert(data, options, mongoOpts)
-      const findArgs = instance.find.mock.calls[0].arguments
+      const findArgs = instance.findOne.mock.calls[0].arguments
       assert.equal(findArgs[0], data)
-      assert.equal(findArgs[1], options)
+      assert.deepEqual(findArgs[1], { validate: false, strict: false })
       assert.equal(findArgs[2], mongoOpts)
     })
   })