Skip to content

Commit 024abde

Browse files
Merge pull request #22 from pm-azhar-mulla/remove-schain-file
Updated test cases
2 parents 4658bb1 + 9b1ec90 commit 024abde

17 files changed

+114
-621
lines changed

test/spec/modules/BTBidAdapter_spec.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,6 @@ import 'modules/consentManagementUsp.js';
1212
import 'modules/consentManagementGpp.js';
1313
import 'modules/tcfControl.js';
1414
import 'modules/gppControl_usnat.js';
15-
import 'modules/schain.js';
1615

1716
describe('BT Bid Adapter', () => {
1817
const ENDPOINT_URL = 'https://pbs.btloader.com/openrtb2/auction';

test/spec/modules/blastoBidAdapter_spec.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,6 @@ import 'modules/multibid/index.js';
1313
import 'modules/priceFloors.js';
1414
import 'modules/consentManagementTcf.js';
1515
import 'modules/consentManagementUsp.js';
16-
import 'modules/schain.js';
1716

1817
const SIMPLE_BID_REQUEST = {
1918
bidder: 'blasto',

test/spec/modules/bridgeuppBidAdapter_spec.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -269,7 +269,7 @@ describe('bridgeuppBidAdapter_spec', function () {
269269
},
270270
];
271271
const ortbRequest = spec.buildRequests(bidRequests, await addFPDToBidderRequest({...bidderRequest, ortb2})).data;
272-
expect(ortbRequest.source.schain).to.deep.equal(expectedSchain);
272+
expect(ortbRequest.source.ext.schain).to.deep.equal(expectedSchain);
273273
expect(ortbRequest.source.pchain).to.equal('sonarads');
274274
});
275275

test/spec/modules/conversantBidAdapter_spec.js

Lines changed: 13 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,6 @@ import 'modules/userId/index.js'; // handles eids
1010
import 'modules/priceFloors.js';
1111
import 'modules/consentManagementTcf.js';
1212
import 'modules/consentManagementUsp.js';
13-
import 'modules/schain.js'; // handles schain
1413
import {hook} from '../../../src/hook.js'
1514
import {BANNER} from '../../../src/mediaTypes';
1615

@@ -451,9 +450,21 @@ describe('Conversant adapter tests', function() {
451450
it('Verify supply chain data', () => {
452451
const bidderRequest = {refererInfo: {page: 'http://test.com?a=b&c=123'}};
453452
const schain = {complete: 1, ver: '1.0', nodes: [{asi: 'bidderA.com', sid: '00001', hp: 1}]};
453+
454+
// Add schain to bidderRequest
455+
bidderRequest.ortb2 = {
456+
source: {
457+
ext: {schain: schain}
458+
}
459+
};
460+
454461
const bidsWithSchain = bidRequests.map((bid) => {
455462
return Object.assign({
456-
ortb2: {source: {schain: schain}}
463+
ortb2: {
464+
source: {
465+
ext: {schain: schain}
466+
}
467+
}
457468
}, bid);
458469
});
459470
const request = spec.buildRequests(bidsWithSchain, bidderRequest);

test/spec/modules/criteoBidAdapter_spec.js

Lines changed: 19 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ import 'modules/userId/index.js';
1010
import 'modules/consentManagementTcf.js';
1111
import 'modules/consentManagementUsp.js';
1212
import 'modules/consentManagementGpp.js';
13-
import 'modules/schain.js';
13+
1414
import {hook} from '../../../src/hook';
1515

1616
describe('The Criteo bidding adapter', function () {
@@ -1219,14 +1219,18 @@ describe('The Criteo bidding adapter', function () {
12191219
expect(ortbRequest.regs.ext.dsa).to.deep.equal(dsa);
12201220
});
12211221

1222-
it('should properly build a request with schain object', async function () {
1222+
it('should properly build a request with schain object', function () {
12231223
const expectedSchain = {
12241224
someProperty: 'someValue'
12251225
};
12261226
const bidRequests = [
12271227
{
12281228
bidder: 'criteo',
1229-
ortb2: {source: {schain: expectedSchain}},
1229+
ortb2: {
1230+
source: {
1231+
ext: {schain: expectedSchain}
1232+
}
1233+
},
12301234
adUnitCode: 'bid-123',
12311235
mediaTypes: {
12321236
banner: {
@@ -1238,9 +1242,19 @@ describe('The Criteo bidding adapter', function () {
12381242
},
12391243
},
12401244
];
1245+
1246+
// Create a modified bidderRequest with schain
1247+
const modifiedBidderRequest = {
1248+
...bidderRequest,
1249+
ortb2: {
1250+
source: {
1251+
ext: {schain: expectedSchain}
1252+
}
1253+
}
1254+
};
12411255

1242-
const ortbRequest = spec.buildRequests(bidRequests, await addFPDToBidderRequest(bidderRequest)).data;
1243-
expect(ortbRequest.source.ext.schain).to.equal(expectedSchain);
1256+
const ortbRequest = spec.buildRequests(bidRequests, modifiedBidderRequest).data;
1257+
expect(ortbRequest.source.ext.schain).to.deep.equal(expectedSchain);
12441258
});
12451259

12461260
it('should properly build a request with bcat field', async function () {

test/spec/modules/escalaxBidAdapter_spec.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,6 @@ import 'modules/multibid/index.js';
1212
import 'modules/priceFloors.js';
1313
import 'modules/consentManagementTcf.js';
1414
import 'modules/consentManagementUsp.js';
15-
import 'modules/schain.js';
1615

1716
const SIMPLE_BID_REQUEST = {
1817
bidder: 'escalax',

test/spec/modules/improvedigitalBidAdapter_spec.js

Lines changed: 19 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,6 @@ import 'modules/multibid/index.js';
1212
import 'modules/priceFloors.js';
1313
import 'modules/consentManagementTcf.js';
1414
import 'modules/consentManagementUsp.js';
15-
import 'modules/schain.js';
1615
import {decorateAdUnitsWithNativeParams} from '../../../src/native.js';
1716
import {hook} from '../../../src/hook.js';
1817
import {addFPDToBidderRequest} from '../../helpers/fpd.js';
@@ -553,8 +552,25 @@ describe('Improve Digital Adapter Tests', function () {
553552
it('should add schain', function () {
554553
const schain = '{"ver":"1.0","complete":1,"nodes":[{"asi":"headerlift.com","sid":"xyz","hp":1}]}';
555554
const bidRequest = Object.assign({}, simpleBidRequest);
556-
bidRequest.ortb2 = {source: {schain: schain}};
557-
const request = spec.buildRequests([bidRequest], bidderRequestReferrer)[0];
555+
556+
// Add schain to both locations in the bid
557+
bidRequest.ortb2 = {
558+
source: {
559+
ext: {schain: schain}
560+
}
561+
};
562+
563+
// Add schain to bidderRequest as well
564+
const modifiedBidderRequest = {
565+
...bidderRequestReferrer,
566+
ortb2: {
567+
source: {
568+
ext: {schain: schain}
569+
}
570+
}
571+
};
572+
573+
const request = spec.buildRequests([bidRequest], modifiedBidderRequest)[0];
558574
const payload = JSON.parse(request.data);
559575
expect(payload.source.ext.schain).to.equal(schain);
560576
});

test/spec/modules/openxBidAdapter_spec.js

Lines changed: 13 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,6 @@ import 'modules/multibid/index.js';
1212
import 'modules/priceFloors.js';
1313
import 'modules/consentManagementTcf.js';
1414
import 'modules/consentManagementUsp.js';
15-
import 'modules/schain.js';
1615
import 'modules/paapi.js';
1716

1817
import {deepClone} from 'src/utils.js';
@@ -1106,13 +1105,24 @@ describe('OpenxRtbAdapter', function () {
11061105
bidId: 'test-bid-id-1',
11071106
bidderRequestId: 'test-bid-request-1',
11081107
auctionId: 'test-auction-1',
1109-
ortb2: {source: {schain: schainConfig}}
1108+
ortb2: {source: {
1109+
schain: schainConfig,
1110+
ext: {schain: schainConfig}
1111+
}}
11101112
}];
1113+
1114+
// Add schain to mockBidderRequest as well
1115+
mockBidderRequest.ortb2 = {
1116+
source: {
1117+
schain: schainConfig,
1118+
ext: {schain: schainConfig}
1119+
}
1120+
};
11111121
});
11121122

11131123
it('should send a supply chain object', function () {
11141124
const request = spec.buildRequests(bidRequests, mockBidderRequest);
1115-
expect(request[0].data.source.ext.schain).to.equal(schainConfig);
1125+
expect(request[0].data.source.ext.schain).to.deep.equal(schainConfig);
11161126
});
11171127

11181128
it('should send the supply chain object with the right version', function () {

test/spec/modules/pulsepointBidAdapter_spec.js

Lines changed: 27 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,6 @@ import {deepClone} from '../../../src/utils';
66
import 'modules/consentManagementTcf';
77
import 'modules/consentManagementUsp';
88
import 'modules/userId/index';
9-
import 'modules/schain';
109

1110
describe('PulsePoint Adapter Tests', function () {
1211
const slotConfigs = [{
@@ -136,7 +135,7 @@ describe('PulsePoint Adapter Tests', function () {
136135
bidfloor: 1.5,
137136
badv: ['cocacola.com', 'lays.com']
138137
},
139-
ortb2: {source: {schain: {
138+
ortb2: {source: {ext: {schain: {
140139
'ver': '1.0',
141140
'complete': 1,
142141
'nodes': [
@@ -149,7 +148,7 @@ describe('PulsePoint Adapter Tests', function () {
149148
'domain': 'publisher.com'
150149
}
151150
]
152-
}}}
151+
}}}}
153152
}];
154153

155154
const bidderRequest = {
@@ -467,8 +466,31 @@ describe('PulsePoint Adapter Tests', function () {
467466
expect(ortbRequest.imp[0].ext).to.be.undefined;
468467
});
469468

470-
it('Verify schain parameters', async function () {
471-
const request = spec.buildRequests(schainParamsSlotConfig, await addFPDToBidderRequest(bidderRequest));
469+
it('Verify schain parameters', function () {
470+
const modifiedBidderRequest = {
471+
...bidderRequest,
472+
ortb2: {
473+
source: {
474+
ext: {
475+
schain: {
476+
'ver': '1.0',
477+
'complete': 1,
478+
'nodes': [
479+
{
480+
'asi': 'exchange1.com',
481+
'sid': '1234',
482+
'hp': 1,
483+
'rid': 'bid-request-1',
484+
'name': 'publisher',
485+
'domain': 'publisher.com'
486+
}
487+
]
488+
}
489+
}
490+
}
491+
}
492+
};
493+
const request = spec.buildRequests(schainParamsSlotConfig, modifiedBidderRequest);
472494
const ortbRequest = request.data;
473495
expect(ortbRequest).to.not.equal(null);
474496
expect(ortbRequest.source).to.not.equal(null);

test/spec/modules/r2b2BidAdapter_spec.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
import {expect} from 'chai';
22
import {spec, internal as r2b2, internal} from 'modules/r2b2BidAdapter.js';
33
import * as utils from '../../../src/utils';
4-
import 'modules/schain.js';
54
import 'modules/userId/index.js';
65

76
function encodePlacementIds (ids) {

test/spec/modules/rubiconBidAdapter_spec.js

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,6 @@ import {
1212
import {config} from 'src/config.js';
1313
import * as utils from 'src/utils.js';
1414
import {find} from 'src/polyfill.js';
15-
import 'modules/schain.js';
1615
import 'modules/consentManagementTcf.js';
1716
import 'modules/consentManagementUsp.js';
1817
import 'modules/userId/index.js';
@@ -4673,7 +4672,7 @@ describe('the rubicon adapter', function () {
46734672
it('should copy the schain JSON to to bid.source.ext.schain', () => {
46744673
const bidderRequest = createVideoBidderRequest();
46754674
const schain = getSupplyChainConfig();
4676-
bidderRequest.bids[0].ortb2.source.schain = schain;
4675+
bidderRequest.bids[0].ortb2.source.ext = { schain: schain };
46774676
const request = spec.buildRequests(bidderRequest.bids, bidderRequest);
46784677
expect(request[0].data.source.ext.schain).to.deep.equal(schain);
46794678
});

0 commit comments

Comments
 (0)