fix(core): retrieve only ObjectIds when populate array is empty

.changeset/modern-squids-change.md

@@ -0,0 +1,5 @@
+---
+"@aeriajs/core": patch
+---
+
+Retrieve only ObjectIds when `populate` array is empty

packages/core/src/collection/reference.ts

@@ -67,6 +67,10 @@ export const getReferences = async (properties: FixedObjectProperty['properties'
       const description = throwIfError(await getCollectionAsset(refProperty.$ref, 'description'))
 
       if( refProperty.populate ) {
+        if( refProperty.populate.length === 0 ) {
+          continue
+        }
+
         const deepReferences = await getReferences(description.properties, {
           depth: depth + 1,
           maxDepth: refProperty.populateDepth || maxDepth,

packages/core/tests/fixtures/aeriaMain.js

@@ -83,6 +83,10 @@ exports.default = init({
         $id: 'project',
         required: [],
         properties: {
+          user_id: {
+            $ref: 'user',
+            populate: [],
+          },
           created_by: {
             $ref: 'person',
             populate: [

packages/core/tests/fixtures/documents.ts

@@ -65,6 +65,7 @@ export const documents = (async () => {
 
     const project = throwIfError(await insert({
       what: {
+        user_id: user1,
         created_by: person1,
         stakeholders: {
           owner: person1,

packages/core/tests/reference.spec.ts

@@ -11,9 +11,18 @@ test('populates top level references', async () => {
   } = await documents
 
   expect(person1.equals(project.created_by._id)).toBe(true)
+  expect(user1.equals(project.user_id)).toBe(true)
   expect(user1.equals(project.created_by.user._id)).toBe(true)
   expect(file1.equals(project.created_by.user.picture_file._id)).toBe(true)
+})
+
+test('respects the "populate" property', async () => {
+  const {
+    user1,
+    project,
+  } = await documents
 
+  expect(user1.equals(project.user_id)).toBe(true)
 })
 
 test('populates deep-nested references', async () => {