monastery 3.0.6 → 3.0.7

package/changelog.md

@@ -2,6 +2,8 @@
 
 All notable changes to this project will be documented in this file. See [standard-version](https://github.com/conventional-changelog/standard-version) for commit guidelines.
 
+### [3.0.7](https://github.com/boycce/monastery/compare/3.0.5...3.0.7) (2024-04-29)
+
 ### [3.0.6](https://github.com/boycce/monastery/compare/3.0.5...3.0.6) (2024-04-29)
 
 ### [3.0.5](https://github.com/boycce/monastery/compare/3.0.4...3.0.5) (2024-04-29)

package/docs/readme.md

@@ -92,6 +92,7 @@ You can view MongoDB's [compatibility table here](https://www.mongodb.com/docs/d
   - db._client moved to db.client
   - db._db moved to db.db
   - db.catch/then() moved to db.onError/db.onOpen()
+  - next() is now redundant when returning promises from hooks, e.g. `afterFind: [async (data) => {...}]`
 
 ## Roadmap
 

package/lib/util.js

@@ -328,8 +328,9 @@ module.exports = {
           console.error(`Monastery ${info} error: you cannot return a promise AND call next()`)
           return
         }
+        current++
         results.push(result)
-        if (!err && ++current < tasks.length) callTask(current)
+        if (!err && current < tasks.length) callTask(current)
         else done(err)
       }
       function done(err) {
@@ -345,7 +346,7 @@ module.exports = {
         const next2 = next.bind(null, i)
         const res = tasks[i](next2)
         if (res instanceof Promise) {
-          res.then((result) => next2(null, result)).catch(next2)
+          res.then((result) => next2(null, result)).catch((e) => next2(e))
         }
       }
       

package/package.json

@@ -2,7 +2,7 @@
   "name": "monastery",
   "description": "⛪ A simple, straightforward MongoDB ODM",
   "author": "Ricky Boyce",
-  "version": "3.0.6",
+  "version": "3.0.7",
   "license": "MIT",
   "repository": "github:boycce/monastery",
   "homepage": "https://boycce.github.io/monastery/",

package/test/crud.js

@@ -902,6 +902,7 @@ test('hooks > async and next conflict', async () => {
     fields: { age: { type: 'number'} },
     afterFind: [
       async (data, next) => {
+        data.age = data.age + 1
         next() // should console an error for empty promise
       },
       async (data, next) => {
@@ -909,7 +910,7 @@ test('hooks > async and next conflict', async () => {
       },
     ],
   })
-  let user4 = db.model('user3', {
+  let user4 = db.model('user4', {
     fields: { age: { type: 'number'} },
     afterFind: [
       async (data, next) => {
@@ -926,10 +927,27 @@ test('hooks > async and next conflict', async () => {
       },
     ],
   })
+
+  let user5 = db.model('user5', {
+    fields: { age: { type: 'number'} },
+    afterFind: [
+      async (data, next) => {
+        const promise = Promise.reject(new Error('An async error occurred with Martin3'))
+        next(new Error('An async error occurred with Martin3'))
+        return promise
+      },
+      async (data, next) => {
+        data.age = data.age + 1 // shouldn't be reached
+      },
+    ],
+  })
+
+
   let user1Doc = await user1.insert({ data: { age: 0 } })
   let user2Doc = await user2.insert({ data: { age: 0 } })
   let user3Doc = await user3.insert({ data: { age: 0 } })
   let user4Doc = await user4.insert({ data: { age: 0 } })
+  let user5Doc = await user5.insert({ data: { age: 0 } })
 
   const logSpy = jest.spyOn(console, 'error').mockImplementation(() => {})
 
@@ -946,4 +964,7 @@ test('hooks > async and next conflict', async () => {
   await expect(user4.find({ query: user4Doc._id })).resolves.toEqual({ _id: expect.any(Object), age: 2 })
   expect(logSpy).toHaveBeenCalledWith('Monastery afterFind error: you cannot return a promise AND call next()')
 
+  await expect(user5.find({ query: user5Doc._id })).rejects.toThrow('An async error occurred with Martin3')
+  expect(logSpy).toHaveBeenCalledWith('Monastery afterFind error: you cannot return a promise AND call next()')
+
 })