Lint cleanup

Mattchewone · Sep 30, 2020 · bf7d746 · bf7d746
1 parent 4c89570
commit bf7d746
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 8 deletions.
diff --git a/lib/shallow-populate.js b/lib/shallow-populate.js
@@ -114,7 +114,7 @@ module.exports = function (options) {
 
       const providedParams = (isFunction(i.params)) ? i.params(params, context) : i.params
 
-      if (providedParams && params != providedParams) params = merge(params, providedParams)
+      if (providedParams && params !== providedParams) params = merge(params, providedParams)
 
       return params
     })

diff --git a/test/shallow-populate.test.js b/test/shallow-populate.test.js
@@ -251,8 +251,8 @@ describe('shallowPopulate hook', function () {
         service () {
           return {
             find (params = {}) {
-              assert.deepEqual(params.query.id.$in, [], 'we have the params from shallow-populate')
-              assert.deepEqual(params.query.$select, ['id'], 'we have a merged query')
+              assert.deepStrictEqual(params.query.id.$in, [], 'we have the params from shallow-populate')
+              assert.deepStrictEqual(params.query.$select, ['id'], 'we have a merged query')
               return []
             }
           }
@@ -286,7 +286,7 @@ describe('shallowPopulate hook', function () {
         keyHere: 'postsId',
         keyThere: 'id',
         params: (params, context) => {
-          params.query.$select = ["id"]
+          params.query.$select = ['id']
         }
       }
     }
@@ -298,8 +298,8 @@ describe('shallowPopulate hook', function () {
         service () {
           return {
             find (params = {}) {
-              assert.deepEqual(params.query.id.$in, [], 'we have the params from shallow-populate')
-              assert.deepEqual(params.query.$select, ['id'], 'we have a merged query')
+              assert.deepStrictEqual(params.query.id.$in, [], 'we have the params from shallow-populate')
+              assert.deepStrictEqual(params.query.$select, ['id'], 'we have a merged query')
               return []
             }
           }
@@ -334,7 +334,7 @@ describe('shallowPopulate hook', function () {
         keyThere: 'id',
         params: (params, context) => {
           assert(context.method === 'create', 'we can pass the context to include')
-          params.method = context.method;
+          params.method = context.method
         }
       }
     }
@@ -371,7 +371,6 @@ describe('shallowPopulate hook', function () {
       .catch(done)
   })
 
-
   it('does nothing if we have no data', function (done) {
     const options = {
       include: {