Skip to content

Commit 3534cba

Browse files
committed
fix: oppty format
1 parent 1bd652f commit 3534cba

File tree

2 files changed

+13
-3
lines changed

2 files changed

+13
-3
lines changed

src/readability/shared/async-mystique.js

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -134,6 +134,8 @@ async function sendOpportunityBatch(
134134
readabilityIssues,
135135
siteId,
136136
jobId,
137+
auditUrl,
138+
site,
137139
context,
138140
) {
139141
const {
@@ -169,11 +171,14 @@ async function sendOpportunityBatch(
169171
// Send single SQS message with S3 key path
170172
const mystiqueMessage = {
171173
type: READABILITY_GUIDANCE_TYPE,
174+
time: new Date().toISOString(),
175+
deliveryType: site.getDeliveryType(),
172176
siteId,
173177
auditId: jobId,
178+
url: auditUrl,
174179
mode: 'opportunity',
175180
data: {
176-
s3ResultsPath: s3Key,
181+
s3BatchPath: s3Key,
177182
},
178183
};
179184

@@ -229,6 +234,8 @@ export async function sendReadabilityToMystique(
229234
readabilityIssues,
230235
siteId,
231236
jobId,
237+
auditUrl,
238+
site,
232239
context,
233240
);
234241
}

test/audits/readability/async-mystique.test.js

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -115,7 +115,8 @@ describe('async-mystique sendReadabilityToMystique', () => {
115115
expect(sentMessage.siteId).to.equal('site-123');
116116
expect(sentMessage.auditId).to.equal('audit-456');
117117
expect(sentMessage.mode).to.equal('opportunity');
118-
expect(sentMessage.data.s3ResultsPath).to.equal('readability/batch-requests/site-123/audit-456.json');
118+
expect(sentMessage.url).to.equal('https://example.com');
119+
expect(sentMessage.data.s3BatchPath).to.equal('readability/batch-requests/site-123/audit-456.json');
119120

120121
// Should NOT interact with AsyncJob
121122
expect(mockContext.dataAccess.AsyncJob.findById).to.not.have.been.called;
@@ -489,7 +490,9 @@ describe('async-mystique sendReadabilityToMystique', () => {
489490
expect(sentMessage.siteId).to.equal('site-abc');
490491
expect(sentMessage.auditId).to.equal('audit-xyz');
491492
expect(sentMessage.mode).to.equal('opportunity');
492-
expect(sentMessage.data.s3ResultsPath).to.equal('readability/batch-requests/site-abc/audit-xyz.json');
493+
expect(sentMessage.url).to.equal('https://example.com');
494+
expect(sentMessage.deliveryType).to.equal('aem_edge');
495+
expect(sentMessage.data.s3BatchPath).to.equal('readability/batch-requests/site-abc/audit-xyz.json');
493496
});
494497

495498
it('should construct correct Mystique message for preflight mode', async () => {

0 commit comments

Comments
 (0)