fix: single rel is undefined instead of {} when keyHere is null

Author: Andreas Reiterer

lib/shallow-populate.js

@@ -280,7 +280,7 @@ function getRelatedItems (ids, relatedItems, include, params) {
   const limit = _get(params, 'query.$limit', Math.max)
   ids = [].concat(ids || [])
   let skipped = 0
-  let itemOrItems = (asArray) ? [] : {}
+  let itemOrItems = (asArray) ? [] : undefined
 
   let isDone = false
   for (let i = 0, n = relatedItems.length; i < n; i++) {

test/shallow-populate.test.js

@@ -304,6 +304,43 @@ describe('populating thing', () => {
         }
       })
 
+      it('does nothing if keyHere of related item is null', async () => {
+        for (const { type, dataResult } of beforeAfter) {
+          const options = {
+            include: {
+              // from: 'users',
+              service: 'posts',
+              nameAs: 'posts',
+              keyHere: 'postsId',
+              keyThere: 'id',
+              asArray: false
+            }
+          }
+          const context = {
+            app: {
+              service (path) {
+                return services[path]
+              }
+            },
+            method: 'create',
+            type,
+            params: {},
+            [dataResult]: {
+              id: '11',
+              name: 'Dumb Stuff',
+              postsId: null
+            }
+          }
+
+          const shallowPopulate = makePopulate(options)
+
+          const response = await shallowPopulate(context)
+          const result = response[dataResult]
+
+          assert(!result.posts, `${type}: post should have not been populated`)
+        }
+      })
+
       it('populates from local keys dot notation', async () => {
         for (const { type, dataResult } of beforeAfter) {
           const options = {