@adobe/spacecat-shared-data-access 1.3.2 → 1.4.1

package/CHANGELOG.md

@@ -1,3 +1,17 @@
+# [@adobe/spacecat-shared-data-access-v1.4.1](https://github.com/adobe-rnd/spacecat-shared/compare/@adobe/spacecat-shared-data-access-v1.4.0...@adobe/spacecat-shared-data-access-v1.4.1) (2023-12-18)
+
+
+### Bug Fixes
+
+* update dynamo client ([#65](https://github.com/adobe-rnd/spacecat-shared/issues/65)) ([cc09041](https://github.com/adobe-rnd/spacecat-shared/commit/cc09041200dfe23c0a83c384fc9695ac972564e9))
+
+# [@adobe/spacecat-shared-data-access-v1.4.0](https://github.com/adobe-rnd/spacecat-shared/compare/@adobe/spacecat-shared-data-access-v1.3.2...@adobe/spacecat-shared-data-access-v1.4.0) (2023-12-16)
+
+
+### Features
+
+* update audit config ([#63](https://github.com/adobe-rnd/spacecat-shared/issues/63)) ([00a468e](https://github.com/adobe-rnd/spacecat-shared/commit/00a468e9404676212ec9441c8a7fccf1a93c636f))
+
 # [@adobe/spacecat-shared-data-access-v1.3.2](https://github.com/adobe-rnd/spacecat-shared/compare/@adobe/spacecat-shared-data-access-v1.3.1...@adobe/spacecat-shared-data-access-v1.3.2) (2023-12-16)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@adobe/spacecat-shared-data-access",
-  "version": "1.3.2",
+  "version": "1.4.1",
   "description": "Shared modules of the Spacecat Services - Data Access",
   "type": "module",
   "main": "src/index.js",
@@ -29,7 +29,7 @@
     "access": "public"
   },
   "dependencies": {
-    "@adobe/spacecat-shared-dynamo": "1.2.4",
+    "@adobe/spacecat-shared-dynamo": "1.2.5",
     "@adobe/spacecat-shared-utils": "1.2.0",
     "@aws-sdk/client-dynamodb": "3.465.0",
     "@aws-sdk/lib-dynamodb": "3.465.0",

package/src/index.d.ts

@@ -12,66 +12,185 @@
 
 // TODO: introduce AuditType interface or Scores interface
 
+/**
+ * Represents an individual audit of a site.
+ */
 export interface Audit {
+  /**
+   * Retrieves the site ID associated with this audit.
+   * @returns {string} The site ID.
+   */
   getSiteId: () => string;
+
+  /**
+   * Retrieves the timestamp when the audit was performed.
+   * @returns {string} The audit timestamp.
+   */
   getAuditedAt: () => string;
+
+  /**
+   * Retrieves the result of the audit.
+   * @returns {object} The audit result.
+   */
   getAuditResult: () => object;
+
+  /**
+   * Retrieves the type of the audit.
+   * @returns {object} The audit type.
+   */
   getAuditType: () => object;
+
+  /**
+   * Retrieves the expiration date of the audit.
+   * @returns {Date} The expiration date.
+   */
   getExpiresAt: () => Date;
+
+  /**
+   * Retrieves a reference to the full audit.
+   * @returns {string} The full audit reference.
+   */
   getFullAuditRef: () => string;
+
+  /**
+   * Indicates whether the audit is live.
+   * @returns {boolean} True if the audit is live, false otherwise.
+   */
   isLive: () => boolean;
+
+  /**
+   * Indicates whether there was an error in the audit.
+   * @returns {boolean} True if there was an error, false otherwise.
+   */
   isError: () => boolean;
+
+  /**
+   * Retrieves the scores from the audit.
+   * @returns {object} The audit scores.
+   */
   getScores: () => object;
 }
 
 /**
- * AuditConfigType defines the structure for specific audit type configurations
+ * AuditConfigType defines the structure for specific audit type configurations.
  */
 export interface AuditConfigType {
   /**
    * Returns true if the audit type is disabled for the site. If an audit type is disabled, no
    * audits of that type will be scheduled for the site.
-   * @returns True if the audit type is disabled for the site
+   * @returns {boolean} True if the audit type is disabled for the site.
    */
-  disabled(): boolean;
+  disabled: () => boolean;
 }
 
 /**
- * AuditConfig defines the structure for the overall audit configuration of a site
+ * AuditConfig defines the structure for the overall audit configuration of a site.
  */
 export interface AuditConfig {
   /**
    * Returns true if audits are disabled for the site. If audits are disabled, no audits will be
    * scheduled for the site. Overrides any audit type specific configurations.
-   * @returns True if audits are disabled for the site
+   * @returns {boolean} True if audits are disabled for the site.
    */
   auditsDisabled: () => boolean;
+
   /**
    * Returns the audit config for a specific audit type. The audit type is the key.
-   * @param auditType The audit type to get the config for
-   * @returns The audit config for the audit type
+   * @param {string} auditType The audit type to get the config for.
+   * @returns {AuditConfigType} The audit config for the audit type.
    */
   getAuditTypeConfig: (auditType: string) => AuditConfigType;
+
   /**
    * Returns the audit configs for all audit types. The keys are the audit types.
-   * @returns The audit configs for all audit types
+   * @returns {object} The audit configs for all audit types.
    */
   getAuditTypeConfigs: () => object;
 }
 
+/**
+ * Represents a site with associated audit and configuration data.
+ */
 export interface Site {
+  /**
+   * Retrieves the ID of the site.
+   * @returns {string} The site ID.
+   */
   getId: () => string;
+
+  /**
+   * Retrieves the base URL of the site.
+   * @returns {string} The base URL.
+   */
   getBaseURL: () => string;
+
+  /**
+   * Retrieves the GitHub URL associated with the site.
+   * @returns {string} The GitHub URL.
+   */
   getGitHubURL: () => string;
+
+  /**
+   * Retrieves the IMS Organization ID associated with the site.
+   * @returns {string} The IMS Org ID.
+   */
   getImsOrgId: () => string;
+
+  /**
+   * Retrieves the creation timestamp of the site.
+   * @returns {string} The creation timestamp.
+   */
   getCreatedAt: () => string;
+
+  /**
+   * Retrieves the last update timestamp of the site.
+   * @returns {string} The last update timestamp.
+   */
   getUpdatedAt: () => string;
+
+  /**
+   * Retrieves the current audit configuration for the site.
+   * @returns {AuditConfig} The current audit configuration.
+   */
   getAuditConfig: () => AuditConfig;
+
+  /**
+   * Retrieves the audits associated with the site.
+   * @returns {Audit[]} The list of audits.
+   */
   getAudits: () => Audit[];
+
+  /**
+   * Indicates whether the site is live.
+   * @returns {boolean} True if the site is live, false otherwise.
+   */
   isLive: () => boolean;
+
+  /**
+   * Updates the list of audits for the site.
+   * @param {Audit[]} audits The new list of audits.
+   * @returns {Site} The updated site instance.
+   */
   setAudits: (audits: Audit[]) => Site;
+
+  /**
+   * Toggles the live status of the site.
+   * @returns {Site} The updated site instance with the toggled live status.
+   */
   toggleLive: () => Site;
+
+  /**
+   * Updates the GitHub URL of the site.
+   * @param {string} gitHubURL The new GitHub URL.
+   * @returns {Site} The updated site instance.
+   */
   updateGitHubURL: (gitHubURL: string) => Site;
+
+  /**
+   * Updates the IMS Org ID of the site.
+   * @param {string} imsOrgId The new IMS Org ID.
+   * @returns {Site} The updated site instance.
+   */
   updateImsOrgId: (imsOrgId: string) => Site;
 }
 

package/src/models/site/audit-config-type.js

@@ -10,9 +10,21 @@
  * governing permissions and limitations under the License.
  */
 
-const AuditConfigType = (data = {}, allAuditsDisabled = false) => ({
-  disabled: () => allAuditsDisabled || data.disabled || false,
-});
+const AuditConfigType = (data = {}) => {
+  const state = {
+    disabled: data.disabled || false,
+  };
+
+  const self = {
+    disabled: () => state.disabled,
+
+    updateDisabled: (newValue) => {
+      state.disabled = newValue;
+    },
+  };
+
+  return Object.freeze(self);
+};
 
 AuditConfigType.fromDynamoItem = (dynamoItem) => {
   const auditConfigTypeData = {

package/src/models/site/audit-config.js

@@ -9,14 +9,9 @@
  * OF ANY KIND, either express or implied. See the License for the specific language
  * governing permissions and limitations under the License.
  */
+
 import AuditConfigType from './audit-config-type.js';
 
-/**
- * Initializes the audit type configs. If auditsDisabled is true, all audit types will be disabled.
- * @param {object} auditTypeConfigs - Object containing audit type configs.
- * @param {boolean} auditsDisabled - Flag indicating if all audits are disabled.
- * @return {object} Object containing audit type configs.
- */
 function getAuditTypeConfigs(auditTypeConfigs, auditsDisabled) {
   return Object.entries(auditTypeConfigs || {}).reduce((acc, [key, value]) => {
     acc[key] = AuditConfigType(value, auditsDisabled);
@@ -25,12 +20,24 @@ function getAuditTypeConfigs(auditTypeConfigs, auditsDisabled) {
 }
 
 const AuditConfig = (data = {}) => {
-  const auditTypeConfigs = getAuditTypeConfigs(data.auditTypeConfigs, data.auditsDisabled);
-  return {
-    auditsDisabled: () => data.auditsDisabled || false,
-    getAuditTypeConfigs: () => auditTypeConfigs,
-    getAuditTypeConfig: (type) => auditTypeConfigs[type],
+  const state = {
+    auditsDisabled: data.auditsDisabled || false,
+    auditTypeConfigs: getAuditTypeConfigs(data.auditTypeConfigs, data.auditsDisabled),
   };
+
+  const self = {
+    auditsDisabled: () => state.auditsDisabled,
+    getAuditTypeConfigs: () => state.auditTypeConfigs,
+    getAuditTypeConfig: (type) => state.auditTypeConfigs[type],
+    updateAuditsDisabled: (newValue) => {
+      state.auditsDisabled = newValue;
+    },
+    updateAuditTypeConfig: (type, config) => {
+      state.auditTypeConfigs[type] = AuditConfigType(config);
+    },
+  };
+
+  return Object.freeze(self);
 };
 
 AuditConfig.fromDynamoItem = (dynamoItem) => AuditConfig(dynamoItem);

package/src/models/site.js

@@ -68,6 +68,18 @@ const Site = (data = {}) => {
     return self;
   }; */
 
+  self.setAllAuditsDisabled = (disabled) => {
+    self.state.auditConfig.updateAuditsDisabled(disabled);
+    self.touch();
+    return self;
+  };
+
+  self.updateAuditTypeConfig = (type, config) => {
+    self.state.auditConfig.updateAuditTypeConfig(type, config);
+    self.touch();
+    return self;
+  };
+
   /**
    * Updates the GitHub URL belonging to the site.
    * @param {string} gitHubURL - The GitHub URL.