add findMany

Author: wojtek-krysiak

spec/resource.spec.js

@@ -1,4 +1,4 @@
-const { ValidationError, Filter } = require('admin-bro')
+const { ValidationError, Filter, BaseRecord } = require('admin-bro')
 const Resource = require('../src/resource')
 const Property = require('../src/property')
 const config = require('../config/config')[process.env.NODE_ENV]
@@ -10,12 +10,12 @@ describe('Resource', function () {
     this.resource = new Resource(this.SequelizeModel)
   })
 
-  after(function () {
-    db.sequelize.close()
+  after(async function () {
+    await db.sequelize.close()
   })
 
-  afterEach(function () {
-    this.SequelizeModel.destroy({ where: {} })
+  afterEach(async function () {
+    await this.SequelizeModel.destroy({ where: {} })
   })
 
   describe('.isAdapterFor', function () {
@@ -61,6 +61,17 @@ describe('Resource', function () {
     })
   })
 
+  describe('#findMany', function () {
+    it('returns array of BaseRecords', async function () {
+      const params = await this.resource.create(this.params)
+
+      const records = await this.resource.findMany([params.id])
+
+      expect(records).to.have.lengthOf(1)
+      expect(records[0]).to.be.instanceOf(BaseRecord)
+    })
+  })
+
   describe('#count', function () {
     it('returns 0 when there are none elements', async function () {
       const count = await this.resource.count(new Filter({}))

src/resource.js

@@ -1,10 +1,7 @@
 /* eslint-disable no-param-reassign */
 
-const {
-  BaseResource,
-  BaseRecord,
-  ValidationError,
-} = require('admin-bro')
+const { BaseResource, BaseRecord, ValidationError } = require('admin-bro')
+const { Op } = require('sequelize')
 
 const Property = require('./property')
 const convertFilter = require('./utils/convert-filter')
@@ -99,6 +96,15 @@ class Resource extends BaseResource {
     return new BaseRecord(sequelizeObject.toJSON(), this)
   }
 
+  async findMany(ids) {
+    const sequelizeObjects = await this.SequelizeModel.findAll({
+      where: {
+        id: { [Op.in]: ids },
+      },
+    })
+    return sequelizeObjects.map(sequelizeObject => new BaseRecord(sequelizeObject.toJSON(), this))
+  }
+
   async findById(id) {
     // versions of Sequelize before 5 had findById method - after that there was findByPk
     const method = this.SequelizeModel.findByPk ? 'findByPk' : 'findById'