fix: allByIndexKeys query

adobe · Jan 6, 2025 · 13e221e · 13e221e
1 parent 9af0966
commit 13e221e
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 2 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/packages/spacecat-shared-data-access/src/v2/models/base/base.collection.js b/packages/spacecat-shared-data-access/src/v2/models/base/base.collection.js
@@ -284,7 +284,8 @@ class BaseCollection {
    * is not found.
    * @async
    */
-  async allByIndexKeys(keys, options = {}) {
+  async allByIndexKeys(indexKeys, options = {}) {
+    const keys = { pk: entityNameToAllPKValue(this.entityName), ...indexKeys };
     return this.#queryByIndexKeys(keys, options);
   }
 

diff --git a/packages/spacecat-shared-data-access/test/it/latest-audit/latest-audit.test.js b/packages/spacecat-shared-data-access/test/it/latest-audit/latest-audit.test.js
@@ -98,6 +98,17 @@ describe('LatestAudit IT', async () => {
     });
   });
 
+  it('gets all latest audits of a type', async () => {
+    const audits = await LatestAudit.allByAuditType('cwv');
+
+    expect(audits).to.be.an('array');
+    expect(audits.length).to.equal(9);
+    audits.forEach((audit) => {
+      expect(audit.getAuditType()).to.equal('cwv');
+      checkAudit(audit);
+    });
+  });
+
   it('gets latest audits of type for a site', async () => {
     const auditType = 'lhs-mobile';
     const site = sampleData.sites[1];