Skip to content

Commit

Permalink
feat(audit-configs): ess audits disabled by default
Browse files Browse the repository at this point in the history
  • Loading branch information
ekremney committed Jun 20, 2024
1 parent c84df02 commit e55af1c
Show file tree
Hide file tree
Showing 4 changed files with 24 additions and 2 deletions.
2 changes: 2 additions & 0 deletions packages/spacecat-shared-data-access/src/models/audit.js
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,8 @@ export const AUDIT_TYPE_ORGANIC_TRAFFIC = 'organic-traffic';
export const AUDIT_TYPE_CWV = 'cwv';
export const AUDIT_TYPE_LHS_DESKTOP = 'lhs-desktop';
export const AUDIT_TYPE_LHS_MOBILE = 'lhs-mobile';
export const AUDIT_TYPE_EXPERIMENTATION_ESS_MONTHLY = 'experimentation-ess-monthly';
export const AUDIT_TYPE_EXPERIMENTATION_ESS_DAILY = 'experimentation-ess-daily';

const EXPIRES_IN_DAYS = 30;

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -13,12 +13,16 @@
import AuditConfigType from './audit-config-type.js';
import {
AUDIT_TYPE_BROKEN_BACKLINKS,
AUDIT_TYPE_EXPERIMENTATION_ESS_DAILY,
AUDIT_TYPE_EXPERIMENTATION_ESS_MONTHLY,
AUDIT_TYPE_ORGANIC_KEYWORDS,
AUDIT_TYPE_ORGANIC_TRAFFIC,
} from '../audit.js';

const AUDIT_TYPE_DISABLED_DEFAULTS = {
[AUDIT_TYPE_BROKEN_BACKLINKS]: true,
[AUDIT_TYPE_EXPERIMENTATION_ESS_DAILY]: true,
[AUDIT_TYPE_EXPERIMENTATION_ESS_MONTHLY]: true,
[AUDIT_TYPE_ORGANIC_KEYWORDS]: true,
[AUDIT_TYPE_ORGANIC_TRAFFIC]: true,
};
Expand All @@ -27,6 +31,8 @@ function getAuditTypeConfigs(auditTypeConfigs, auditsDisabled) {
if (!auditTypeConfigs || Object.keys(auditTypeConfigs).length === 0) {
return {
[AUDIT_TYPE_BROKEN_BACKLINKS]: AuditConfigType({ disabled: true }),
[AUDIT_TYPE_EXPERIMENTATION_ESS_DAILY]: AuditConfigType({ disabled: true }),
[AUDIT_TYPE_EXPERIMENTATION_ESS_MONTHLY]: AuditConfigType({ disabled: true }),
[AUDIT_TYPE_ORGANIC_KEYWORDS]: AuditConfigType({ disabled: true }),
[AUDIT_TYPE_ORGANIC_TRAFFIC]: AuditConfigType({ disabled: true }),
};
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,11 @@ import { expect } from 'chai';

import AuditConfig from '../../../../src/models/site/audit-config.js';
import {
AUDIT_TYPE_BROKEN_BACKLINKS, AUDIT_TYPE_ORGANIC_KEYWORDS, AUDIT_TYPE_ORGANIC_TRAFFIC,
AUDIT_TYPE_BROKEN_BACKLINKS,
AUDIT_TYPE_EXPERIMENTATION_ESS_DAILY,
AUDIT_TYPE_EXPERIMENTATION_ESS_MONTHLY,
AUDIT_TYPE_ORGANIC_KEYWORDS,
AUDIT_TYPE_ORGANIC_TRAFFIC,
} from '../../../../src/models/audit.js';

describe('AuditConfig Tests', () => {
Expand All @@ -27,6 +31,10 @@ describe('AuditConfig Tests', () => {
const auditTypeConfigs = auditConfig.getAuditTypeConfigs();
expect(auditTypeConfigs[AUDIT_TYPE_BROKEN_BACKLINKS]).to.be.an('object');
expect(auditTypeConfigs[AUDIT_TYPE_BROKEN_BACKLINKS].disabled()).to.be.true;
expect(auditTypeConfigs[AUDIT_TYPE_EXPERIMENTATION_ESS_DAILY]).to.be.an('object');
expect(auditTypeConfigs[AUDIT_TYPE_EXPERIMENTATION_ESS_DAILY].disabled()).to.be.true;
expect(auditTypeConfigs[AUDIT_TYPE_EXPERIMENTATION_ESS_MONTHLY]).to.be.an('object');
expect(auditTypeConfigs[AUDIT_TYPE_EXPERIMENTATION_ESS_MONTHLY].disabled()).to.be.true;
expect(auditTypeConfigs[AUDIT_TYPE_ORGANIC_KEYWORDS]).to.be.an('object');
expect(auditTypeConfigs[AUDIT_TYPE_ORGANIC_KEYWORDS].disabled()).to.be.true;
expect(auditTypeConfigs[AUDIT_TYPE_ORGANIC_TRAFFIC]).to.be.an('object');
Expand Down
8 changes: 7 additions & 1 deletion packages/spacecat-shared-utils/test/helpers.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,11 @@
import { expect } from 'chai';
import { createSite } from '@adobe/spacecat-shared-data-access/src/models/site.js';
import { createOrganization } from '@adobe/spacecat-shared-data-access/src/models/organization.js';
import { AUDIT_TYPE_BROKEN_BACKLINKS } from '@adobe/spacecat-shared-data-access/src/models/audit.js';
import {
AUDIT_TYPE_BROKEN_BACKLINKS,
AUDIT_TYPE_EXPERIMENTATION_ESS_DAILY,
AUDIT_TYPE_EXPERIMENTATION_ESS_MONTHLY,
} from '@adobe/spacecat-shared-data-access/src/models/audit.js';

import {
generateCSVFile,
Expand Down Expand Up @@ -234,6 +238,8 @@ describe('isAuditsDisabled', () => {
});

expect(isAuditsDisabled(site, org, AUDIT_TYPE_BROKEN_BACKLINKS)).to.be.true;
expect(isAuditsDisabled(site, org, AUDIT_TYPE_EXPERIMENTATION_ESS_DAILY)).to.be.true;
expect(isAuditsDisabled(site, org, AUDIT_TYPE_EXPERIMENTATION_ESS_MONTHLY)).to.be.true;
});
});

Expand Down

0 comments on commit e55af1c

Please sign in to comment.