@adobe/spacecat-shared-data-access 1.2.5 → 1.2.7

package/CHANGELOG.md

@@ -1,3 +1,17 @@
+# [@adobe/spacecat-shared-data-access-v1.2.7](https://github.com/adobe-rnd/spacecat-shared/compare/@adobe/spacecat-shared-data-access-v1.2.6...@adobe/spacecat-shared-data-access-v1.2.7) (2023-12-11)
+
+
+### Bug Fixes
+
+* sort order sites with latest audits ([#46](https://github.com/adobe-rnd/spacecat-shared/issues/46)) ([8da2bf1](https://github.com/adobe-rnd/spacecat-shared/commit/8da2bf16b904ced6787d8c56ba2b4b72678687a0))
+
+# [@adobe/spacecat-shared-data-access-v1.2.6](https://github.com/adobe-rnd/spacecat-shared/compare/@adobe/spacecat-shared-data-access-v1.2.5...@adobe/spacecat-shared-data-access-v1.2.6) (2023-12-11)
+
+
+### Bug Fixes
+
+* include sites without latest audits ([#45](https://github.com/adobe-rnd/spacecat-shared/issues/45)) ([b843190](https://github.com/adobe-rnd/spacecat-shared/commit/b843190da09916832a743d6fc4bf83e804b61912))
+
 # [@adobe/spacecat-shared-data-access-v1.2.5](https://github.com/adobe-rnd/spacecat-shared/compare/@adobe/spacecat-shared-data-access-v1.2.4...@adobe/spacecat-shared-data-access-v1.2.5) (2023-12-10)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@adobe/spacecat-shared-data-access",
-  "version": "1.2.5",
+  "version": "1.2.7",
   "description": "Shared modules of the Spacecat Services - Data Access",
   "type": "module",
   "main": "src/index.js",

package/src/service/sites/accessPatterns.js

@@ -55,16 +55,18 @@ export const getSitesToAudit = async (dynamoClient, config) => {
 };
 
 /**
- * Retrieves sites with their latest audit of a specified type.
- *
+ * Retrieves all sites with their latest audit. Sites without a latest audit will be included
+ * in the result, but will have an empty audits array. The sites are sorted by their latest
+ * audit scores in ascending order by default. The sortAuditsAscending parameter can be used
+ * to change the sort order. If a site has no latest audit, it will be sorted at the end of
+ * the list.
  * @param {DynamoDbClient} dynamoClient - The DynamoDB client.
  * @param {DataAccessConfig} config - The data access config.
  * @param {Logger} log - The logger.
- * @param {string} auditType - The type of the latest audits to retrieve for each site.
- * @param {boolean} [sortAuditsAscending] - Optional. Determines if the audits
- * should be sorted ascending or descending by scores.
- * @returns {Promise<Readonly<Site>[]>} A promise that resolves to an array of site objects,
- * each with its latest audit of the specified type.
+ * @param {string} auditType - The type of audits to retrieve for the sites.
+ * @param {boolean} [sortAuditsAscending=true] - Determines if the audits should be sorted in
+ * @return {Promise<Readonly<Site>[]>} A promise that resolves to an array of sites with their
+ * latest audit.
  */
 export const getSitesWithLatestAudit = async (
   dynamoClient,
@@ -79,15 +81,25 @@ export const getSitesWithLatestAudit = async (
   ]);
 
   const sitesMap = new Map(sites.map((site) => [site.getId(), site]));
+  const orderedSites = [];
 
-  return latestAudits.reduce((result, audit) => {
+  // First, append sites with a latest audit in the sorted order
+  latestAudits.forEach((audit) => {
     const site = sitesMap.get(audit.getSiteId());
     if (site) {
       site.setAudits([audit]);
-      result.push(site);
+      orderedSites.push(site);
+      sitesMap.delete(site.getId()); // Remove the site from the map to avoid adding it again
     }
-    return result;
-  }, []);
+  });
+
+  // Then, append the remaining sites (without a latest audit)
+  sitesMap.forEach((site) => {
+    site.setAudits([]);
+    orderedSites.push(site);
+  });
+
+  return orderedSites;
 };
 
 /**