Skip to content

Commit 9eaf6e5

Browse files
osazosbretg
authored andcommitted
AdagioBidAdapter: cast organizationId param as a string (#4721)
* AdagioBidAdapter: cast organizationId param as a string * AdagioBidAdapter: Fix tests
1 parent ae820af commit 9eaf6e5

File tree

2 files changed

+13
-1
lines changed

2 files changed

+13
-1
lines changed

modules/adagioBidAdapter.js

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -394,6 +394,9 @@ export const spec = {
394394

395395
// Regroug ad units by siteId
396396
const groupedAdUnits = adUnits.reduce((groupedAdUnits, adUnit) => {
397+
if (adUnit.params && adUnit.params.organizationId) {
398+
adUnit.params.organizationId = adUnit.params.organizationId.toString();
399+
}
397400
(groupedAdUnits[adUnit.params.organizationId] = groupedAdUnits[adUnit.params.organizationId] || []).push(adUnit);
398401
return groupedAdUnits;
399402
}, {});

test/spec/modules/adagioBidAdapter_spec.js

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -414,7 +414,16 @@ describe('adagioAdapter', () => {
414414
const requests = spec.buildRequests([bidRequests[0]], bidderRequest);
415415
const request = requests[0];
416416
expect(request.data.adUnits[0].features.print_number).to.equal('2');
417-
})
417+
});
418+
419+
it('organizationId param key must be a string', () => {
420+
const requests = spec.buildRequests([Object.assign({}, bidRequests[0], {params: {organizationId: 1010}})], bidderRequest);
421+
const request = requests[0];
422+
expect(request.data.adUnits[0].params).to.exist;
423+
expect(request.data.adUnits[0].params.organizationId).to.deep.equal('1010');
424+
expect(request.data.organizationId).to.exist;
425+
expect(request.data.organizationId).to.deep.equal('1010');
426+
});
418427

419428
it('GDPR consent is applied', () => {
420429
const requests = spec.buildRequests(bidRequests, bidderRequest);

0 commit comments

Comments
 (0)