prebid.js 8.1.0 → 8.3.0

package/test/spec/modules/consentManagementGpp_spec.js

@@ -1,4 +1,11 @@
-import { setConsentConfig, requestBidsHook, resetConsentData, userCMP, consentTimeout, staticConsentData } from 'modules/consentManagementGpp.js';
+import {
+  setConsentConfig,
+  requestBidsHook,
+  resetConsentData,
+  userCMP,
+  consentTimeout,
+  storeConsentData, lookupIabConsent
+} from 'modules/consentManagementGpp.js';
 import { gppDataHandler } from 'src/adapterManager.js';
 import * as utils from 'src/utils.js';
 import { config } from 'src/config.js';
@@ -94,16 +101,79 @@ describe('consentManagementGpp', function () {
       });
     });
 
+    describe('lookupIABConsent', () => {
+      let mockCmp, mockCmpEvent, gppData, sectionData
+      beforeEach(() => {
+        gppData = {
+          gppString: 'mockString',
+          applicableSections: [],
+          pingData: {}
+        };
+        sectionData = {};
+        mockCmp = sinon.stub().callsFake(({command, callback, parameter}) => {
+          let res;
+          switch (command) {
+            case 'addEventListener':
+              mockCmpEvent = callback;
+              break;
+            case 'getGPPData':
+              res = gppData;
+              break;
+            case 'getSection':
+              res = sectionData[parameter];
+              break;
+          }
+          return Promise.resolve(res);
+        });
+      })
+
+      function runLookup() {
+        return new Promise((resolve, reject) => lookupIabConsent({onSuccess: resolve, onError: reject}, () => mockCmp));
+      }
+
+      function oneShotLookup() {
+        const pm = runLookup();
+        mockCmpEvent({eventName: 'sectionChange'});
+        return pm;
+      }
+
+      it('fetches all sections', () => {
+        gppData.pingData.supportedAPIs = ['usnat', 'usca']
+        sectionData = {
+          usnat: {mock: 'usnat'},
+          usca: {mock: 'usca'}
+        };
+        return oneShotLookup().then((res) => {
+          expect(res.sectionData).to.eql(sectionData);
+        });
+      });
+
+      it('does not choke if some section data is not available', () => {
+        gppData.pingData.supportedAPIs = ['usnat', 'usca']
+        sectionData = {
+          usca: {mock: 'data'}
+        };
+        return oneShotLookup().then((res) => {
+          expect(res.sectionData).to.eql(sectionData);
+        })
+      });
+    })
+
     describe('static consent string setConsentConfig value', () => {
       afterEach(() => {
         config.resetConfig();
       });
 
-      it('results in user settings overriding system defaults for v2 spec', () => {
+      it('results in user settings overriding system defaults', () => {
         let staticConfig = {
           gpp: {
             cmpApi: 'static',
             timeout: 7500,
+            sectionData: {
+              usnat: {
+                MockUsnatParsedFlag: true
+              }
+            },
             consentData: {
               applicableSections: [7],
               gppString: 'ABCDEFG1234',
@@ -116,11 +186,10 @@ describe('consentManagementGpp', function () {
 
         setConsentConfig(staticConfig);
         expect(userCMP).to.be.equal('static');
-        expect(consentTimeout).to.be.equal(0); // should always return without a timeout when config is used
         const consent = gppDataHandler.getConsentData();
         expect(consent.gppString).to.eql(staticConfig.gpp.consentData.gppString);
-        expect(consent.fullGppData).to.eql(staticConfig.gpp.consentData);
-        expect(staticConsentData).to.be.equal(staticConfig.gpp.consentData);
+        expect(consent.gppData).to.eql(staticConfig.gpp.consentData);
+        expect(consent.sectionData).to.eql(staticConfig.gpp.sectionData);
       });
     });
   });
@@ -334,6 +403,14 @@ describe('consentManagementGpp', function () {
                     success: true
                   }
                 }
+              } else if (data[`${prefix}Call`].command === 'getSection') {
+                response = {
+                  [`${prefix}Return`]: {
+                    callId,
+                    returnValue: {},
+                    success: true
+                  }
+                }
               }
               event.source.postMessage(stringifyResponse ? JSON.stringify(response) : response, '*');
             }
@@ -367,7 +444,7 @@ describe('consentManagementGpp', function () {
         resetConsentData();
       });
 
-      describe('v2 CMP workflow for iframe pages:', function () {
+      describe('workflow for iframe pages:', function () {
         stringifyResponse = false;
         let ifr2 = null;
 
@@ -410,7 +487,7 @@ describe('consentManagementGpp', function () {
         resetConsentData();
       });
 
-      describe('v2 CMP workflow for normal pages:', function () {
+      describe('CMP workflow for normal pages:', function () {
         beforeEach(function () {
           window.__gpp = function () {};
         });

package/test/spec/modules/consentManagement_spec.js

@@ -1,18 +1,17 @@
 import {
-  setConsentConfig,
-  requestBidsHook,
-  resetConsentData,
-  userCMP,
-  consentTimeout,
   actionTimeout,
-  staticConsentData,
+  consentTimeout,
   gdprScope,
   loadConsentData,
-  setActionTimeout
+  requestBidsHook,
+  resetConsentData,
+  setConsentConfig,
+  staticConsentData,
+  userCMP
 } from 'modules/consentManagement.js';
-import { gdprDataHandler } from 'src/adapterManager.js';
+import {gdprDataHandler} from 'src/adapterManager.js';
 import * as utils from 'src/utils.js';
-import { config } from 'src/config.js';
+import {config} from 'src/config.js';
 import 'src/prebid.js';
 
 let expect = require('chai').expect;
@@ -487,15 +486,6 @@ describe('consentManagement', function () {
 
         testIFramedPage('with/JSON response', false, 'abc12345234', 2);
         testIFramedPage('with/String response', true, 'abc12345234', 2);
-
-        it('should contain correct v2 CMP definition', (done) => {
-          setConsentConfig(goodConfig);
-          requestBidsHook(() => {
-            const nbArguments = window.__tcfapi.toString().split('\n')[0].split(', ').length;
-            expect(nbArguments).to.equal(4);
-            done();
-          }, {});
-        });
       });
     });
 

package/test/spec/modules/datablocksBidAdapter_spec.js

@@ -2,6 +2,7 @@ import { expect } from 'chai';
 import { spec } from '../../../modules/datablocksBidAdapter.js';
 import { BotClientTests } from '../../../modules/datablocksBidAdapter.js';
 import { getStorageManager } from '../../../src/storageManager.js';
+import {deepClone} from '../../../src/utils.js';
 
 const bid = {
   bidId: '2dd581a2b6281d',
@@ -409,7 +410,7 @@ describe('DatablocksAdapter', function() {
       expect(spec.isBidRequestValid(bid)).to.be.true;
     });
     it('Should return false when host/source_id is not set', function() {
-      let moddedBid = Object.assign({}, bid);
+      let moddedBid = deepClone(bid);
       delete moddedBid.params.source_id;
       expect(spec.isBidRequestValid(moddedBid)).to.be.false;
     });
@@ -442,9 +443,12 @@ describe('DatablocksAdapter', function() {
   });
 
   describe('buildRequests', function() {
-    let request = spec.buildRequests([bid, bid2, nativeBid], bidderRequest);
+    let request;
+    before(() => {
+      request = spec.buildRequests([bid, bid2, nativeBid], bidderRequest);
+      expect(request).to.exist;
+    })
 
-    expect(request).to.exist;
     it('Returns POST method', function() {
       expect(request.method).to.exist;
       expect(request.method).to.equal('POST');

package/test/spec/modules/eids_spec.js

@@ -187,6 +187,24 @@ describe('eids array generation for known sub-modules', function() {
     });
   });
 
+  it('bidswitch with ext', function() {
+    const userId = {
+      bidswitch: {'id': 'sample_id', 'ext': {'provider': 'some.provider.com'}}
+    };
+    const newEids = createEidsArray(userId);
+    expect(newEids.length).to.equal(1);
+    expect(newEids[0]).to.deep.equal({
+      source: 'bidswitch.net',
+      uids: [{
+        id: 'sample_id',
+        atype: 3,
+        ext: {
+          provider: 'some.provider.com'
+        }
+      }]
+    });
+  });
+
   it('medianet', function() {
     const userId = {
       medianet: {'id': 'sample_id'}
@@ -202,6 +220,57 @@ describe('eids array generation for known sub-modules', function() {
     });
   });
 
+  it('medianet with ext', function() {
+    const userId = {
+      medianet: {'id': 'sample_id', 'ext': {'provider': 'some.provider.com'}}
+    };
+    const newEids = createEidsArray(userId);
+    expect(newEids.length).to.equal(1);
+    expect(newEids[0]).to.deep.equal({
+      source: 'media.net',
+      uids: [{
+        id: 'sample_id',
+        atype: 3,
+        ext: {
+          provider: 'some.provider.com'
+        }
+      }]
+    });
+  });
+
+  it('magnite', function() {
+    const userId = {
+      magnite: {'id': 'sample_id'}
+    };
+    const newEids = createEidsArray(userId);
+    expect(newEids.length).to.equal(1);
+    expect(newEids[0]).to.deep.equal({
+      source: 'rubiconproject.com',
+      uids: [{
+        id: 'sample_id',
+        atype: 3
+      }]
+    });
+  });
+
+  it('magnite with ext', function() {
+    const userId = {
+      magnite: {'id': 'sample_id', 'ext': {'provider': 'some.provider.com'}}
+    };
+    const newEids = createEidsArray(userId);
+    expect(newEids.length).to.equal(1);
+    expect(newEids[0]).to.deep.equal({
+      source: 'rubiconproject.com',
+      uids: [{
+        id: 'sample_id',
+        atype: 3,
+        ext: {
+          provider: 'some.provider.com'
+        }
+      }]
+    });
+  });
+
   it('liveIntentId; getValue call and NO ext', function() {
     const userId = {
       lipb: {
@@ -348,6 +417,24 @@ describe('eids array generation for known sub-modules', function() {
     });
   });
 
+  it('uid2 with ext', function() {
+    const userId = {
+      uid2: {'id': 'Sample_AD_Token', 'ext': {'provider': 'some.provider.com'}}
+    };
+    const newEids = createEidsArray(userId);
+    expect(newEids.length).to.equal(1);
+    expect(newEids[0]).to.deep.equal({
+      source: 'uidapi.com',
+      uids: [{
+        id: 'Sample_AD_Token',
+        atype: 3,
+        ext: {
+          provider: 'some.provider.com'
+        }
+      }]
+    });
+  });
+
   it('euid', function() {
     const userId = {
       euid: {'id': 'Sample_AD_Token'}

package/test/spec/modules/holidBidAdapter_spec.js

@@ -2,11 +2,14 @@ import { expect } from 'chai'
 import { spec } from 'modules/holidBidAdapter.js'
 
 describe('holidBidAdapterTests', () => {
+  const bidderRequest = {
+    bidderRequestId: 'test-id'
+  }
+
   const bidRequestData = {
     bidder: 'holid',
     adUnitCode: 'test-div',
     bidId: 'bid-id',
-    auctionId: 'test-id',
     params: { adUnitID: '12345' },
     mediaTypes: { banner: {} },
     sizes: [[300, 250]],
@@ -48,18 +51,26 @@ describe('holidBidAdapterTests', () => {
 
   describe('buildRequests', () => {
     const bid = JSON.parse(JSON.stringify(bidRequestData))
-    const request = spec.buildRequests([bid], bid)
+    const request = spec.buildRequests([bid], bidderRequest)
     const payload = JSON.parse(request[0].data)
 
+    it('should include id in request', () => {
+      expect(payload.id).to.equal('test-id')
+    })
+
     it('should include ext in imp', () => {
-      expect(payload.imp[0].ext).to.exist
       expect(payload.imp[0].ext).to.deep.equal({
         prebid: { storedrequest: { id: '12345' } },
       })
     })
 
+    it('should include ext in request', () => {
+      expect(payload.ext).to.deep.equal({
+        prebid: { storedrequest: { id: '12345' } },
+      })
+    })
+
     it('should include banner format in imp', () => {
-      expect(payload.imp[0].banner).to.exist
       expect(payload.imp[0].banner).to.deep.equal({
         format: [{ w: 300, h: 250 }],
       })
@@ -146,6 +157,10 @@ describe('holidBidAdapterTests', () => {
       }
       const uspConsent = 'mkjvbiniwot4827obfoy8sdg8203gb'
       const expectedUserSyncs = [
+        {
+          type: 'image',
+          url: 'https://track.adform.net/Serving/TrackPoint/?pm=2992097&lid=132720821',
+        },
         {
           type: 'iframe',
           url: 'https://null.holid.io/sync.html?bidders=%5B%22test%20seat%201%22%2C%22test%20seat%202%22%5D&gdpr=1&gdpr_consent=dkj49Sjmfjuj34as:12jaf90123hufabidfy9u23brfpoig&usp_consent=mkjvbiniwot4827obfoy8sdg8203gb&type=iframe',
@@ -162,7 +177,7 @@ describe('holidBidAdapterTests', () => {
       expect(userSyncs).to.deep.equal(expectedUserSyncs)
     })
 
-    it('should return empty user syncs when responsetimemillis is not defined', () => {
+    it('should return base user syncs when responsetimemillis is not defined', () => {
       const optionsType = {
         iframeEnabled: true,
         pixelEnabled: true,
@@ -179,7 +194,12 @@ describe('holidBidAdapterTests', () => {
         consentString: 'dkj49Sjmfjuj34as:12jaf90123hufabidfy9u23brfpoig',
       }
       const uspConsent = 'mkjvbiniwot4827obfoy8sdg8203gb'
-      const expectedUserSyncs = []
+      const expectedUserSyncs = [
+        {
+          type: 'image',
+          url: 'https://track.adform.net/Serving/TrackPoint/?pm=2992097&lid=132720821',
+        }
+      ]
 
       const userSyncs = spec.getUserSyncs(
         optionsType,

package/test/spec/modules/insticatorBidAdapter_spec.js

@@ -185,6 +185,7 @@ describe('InsticatorBidAdapter', function () {
     let getDataFromLocalStorageStub, localStorageIsEnabledStub;
     let getCookieStub, cookiesAreEnabledStub;
     let sandbox;
+    let serverRequests, serverRequest;
 
     beforeEach(() => {
       $$PREBID_GLOBAL$$.bidderSettings = {
@@ -210,12 +211,15 @@ describe('InsticatorBidAdapter', function () {
       $$PREBID_GLOBAL$$.bidderSettings = {};
     });
 
-    const serverRequests = spec.buildRequests([bidRequest], bidderRequest);
+    before(() => {
+      serverRequests = spec.buildRequests([bidRequest], bidderRequest);
+      serverRequest = serverRequests[0];
+    })
+
     it('should create a request', function () {
       expect(serverRequests).to.have.length(1);
     });
 
-    const serverRequest = serverRequests[0];
     it('should create a request object with method, URL, options and data', function () {
       expect(serverRequest).to.exist;
       expect(serverRequest.method).to.exist;

package/test/spec/modules/intentIqIdSystem_spec.js

@@ -1,15 +1,17 @@
 import { expect } from 'chai';
-import { intentIqIdSubmodule, readData, FIRST_PARTY_KEY } from 'modules/intentIqIdSystem.js';
+import { intentIqIdSubmodule, storage } from 'modules/intentIqIdSystem.js';
 import * as utils from 'src/utils.js';
 import { server } from 'test/mocks/xhr.js';
 
 const partner = 10;
 const pai = '11';
 const pcid = '12';
+const enableCookieStorage = true;
 const defaultConfigParams = { params: { partner: partner } };
 const paiConfigParams = { params: { partner: partner, pai: pai } };
 const pcidConfigParams = { params: { partner: partner, pcid: pcid } };
-const allConfigParams = { params: { partner: partner, pai: pai, pcid: pcid } };
+const enableCookieConfigParams = { params: { partner: partner, enableCookieStorage: enableCookieStorage } };
+const allConfigParams = { params: { partner: partner, pai: pai, pcid: pcid, enableCookieStorage: enableCookieStorage } };
 const responseHeader = { 'Content-Type': 'application/json' }
 
 describe('IntentIQ tests', function () {
@@ -62,6 +64,38 @@ describe('IntentIQ tests', function () {
     expect(submodule).to.be.undefined;
   });
 
+  it('should not save data in cookie if enableCookieStorage configParam not set', function () {
+    let callBackSpy = sinon.spy();
+    let submoduleCallback = intentIqIdSubmodule.getId(defaultConfigParams).callback;
+    submoduleCallback(callBackSpy);
+    let request = server.requests[0];
+    expect(request.url).to.contain('https://api.intentiq.com/profiles_engine/ProfilesEngineServlet?at=39&mi=10&dpi=10&pt=17&dpn=1&iiqidtype=2&iiqpcid=');
+    request.respond(
+      200,
+      responseHeader,
+      JSON.stringify({ pid: 'test_pid', data: 'test_personid', ls: true })
+    );
+    expect(callBackSpy.calledOnce).to.be.true;
+    expect(storage.getCookie('_iiq_fdata_' + partner)).to.equal(null);
+  });
+
+  it('should save data in cookie if enableCookieStorage configParam set to true', function () {
+    let callBackSpy = sinon.spy();
+    let submoduleCallback = intentIqIdSubmodule.getId(allConfigParams).callback;
+    submoduleCallback(callBackSpy);
+    let request = server.requests[0];
+    expect(request.url).to.contain('https://api.intentiq.com/profiles_engine/ProfilesEngineServlet?at=39&mi=10&dpi=10&pt=17&dpn=1&pcid=12&pai=11&iiqidtype=2&iiqpcid=');
+    request.respond(
+      200,
+      responseHeader,
+      JSON.stringify({ pid: 'test_pid', data: 'test_personid', ls: true })
+    );
+    expect(callBackSpy.calledOnce).to.be.true;
+    const cookieValue = storage.getCookie('_iiq_fdata_' + partner)
+    expect(cookieValue).to.not.equal(null)
+    expect(JSON.parse(cookieValue).data).to.be.equal('test_personid');
+  });
+
   it('should call the IntentIQ endpoint with only partner', function () {
     let callBackSpy = sinon.spy();
     let submoduleCallback = intentIqIdSubmodule.getId(defaultConfigParams).callback;

package/test/spec/modules/ixBidAdapter_spec.js

@@ -759,7 +759,8 @@ describe('IndexexchangeAdapter', function () {
     uid2: { id: 'testuid2' }, // UID 2.0
     // similar to uid2, but id5's getValue takes .uid
     id5id: { uid: 'testid5id' }, // ID5
-    imuid: 'testimuid'
+    imuid: 'testimuid',
+    '33acrossId': { envelope: 'v1.5fs.1000.fjdiosmclds' }
   };
 
   const DEFAULT_USERIDASEIDS_DATA = createEidsArray(DEFAULT_USERID_DATA);
@@ -813,7 +814,12 @@ describe('IndexexchangeAdapter', function () {
     }, {
       source: 'intimatemerger.com',
       uids: [{
-        id: DEFAULT_USERID_DATA.imuid,
+        id: DEFAULT_USERID_DATA.imuid
+      }]
+    }, {
+      source: '33across.com',
+      uids: [{
+        id: DEFAULT_USERID_DATA['33acrossId'].envelope
       }]
     }
   ];
@@ -832,6 +838,18 @@ describe('IndexexchangeAdapter', function () {
   });
 
   describe('getUserSync tests', function () {
+    before(() => {
+      // TODO (dgirardi): the assertions in this block rely on
+      // global state (consent and siteId) set up in the SOT
+      // this is an outsider's attempt to recreate that state more explicitly after shuffling around setup code
+      // in this test suite
+
+      // please make your preconditions explicit;
+      // also, please, avoid global state if possible.
+
+      spec.buildRequests(DEFAULT_BANNER_VALID_BID, DEFAULT_OPTION);
+    })
+
     it('UserSync test : check type = iframe, check usermatch URL', function () {
       const syncOptions = {
         'iframeEnabled': true
@@ -1207,7 +1225,7 @@ describe('IndexexchangeAdapter', function () {
         const payload = extractPayload(request[0]);
         expect(request).to.be.an('array');
         expect(request).to.have.lengthOf.above(0); // should be 1 or more
-        expect(payload.user.eids).to.have.lengthOf(7);
+        expect(payload.user.eids).to.have.lengthOf(8);
         expect(payload.user.eids).to.deep.include(DEFAULT_USERID_PAYLOAD[0]);
       });
     });
@@ -1395,7 +1413,7 @@ describe('IndexexchangeAdapter', function () {
       cloneValidBid[0].userIdAsEids = utils.deepClone(DEFAULT_USERIDASEIDS_DATA);
       const request = spec.buildRequests(cloneValidBid, DEFAULT_OPTION)[0];
       const payload = extractPayload(request);
-      expect(payload.user.eids).to.have.lengthOf(7);
+      expect(payload.user.eids).to.have.lengthOf(8);
       expect(payload.user.eids).to.have.deep.members(DEFAULT_USERID_PAYLOAD);
     });
 
@@ -1528,7 +1546,7 @@ describe('IndexexchangeAdapter', function () {
       })
 
       expect(payload.user).to.exist;
-      expect(payload.user.eids).to.have.lengthOf(9);
+      expect(payload.user.eids).to.have.lengthOf(10);
 
       expect(payload.user.eids).to.have.deep.members(validUserIdPayload);
     });
@@ -1570,7 +1588,7 @@ describe('IndexexchangeAdapter', function () {
       });
 
       const payload = extractPayload(request);
-      expect(payload.user.eids).to.have.lengthOf(8);
+      expect(payload.user.eids).to.have.lengthOf(9);
       expect(payload.user.eids).to.have.deep.members(validUserIdPayload);
     });
   });
@@ -1771,16 +1789,19 @@ describe('IndexexchangeAdapter', function () {
   });
 
   describe('buildRequests', function () {
-    let request = spec.buildRequests(DEFAULT_BANNER_VALID_BID, DEFAULT_OPTION)[0];
-    const requestUrl = request.url;
-    const requestMethod = request.method;
-    const payloadData = request.data;
-
     const bidWithoutSchain = utils.deepClone(DEFAULT_BANNER_VALID_BID);
     delete bidWithoutSchain[0].schain;
-    const requestWithoutSchain = spec.buildRequests(bidWithoutSchain, DEFAULT_OPTION)[0];
-    const payloadWithoutSchain = extractPayload(requestWithoutSchain);
     const GPID = '/19968336/some-adunit-path';
+    let request, requestUrl, requestMethod, payloadData, requestWithoutSchain, payloadWithoutSchain;
+
+    before(() => {
+      request = spec.buildRequests(DEFAULT_BANNER_VALID_BID, DEFAULT_OPTION)[0];
+      requestUrl = request.url;
+      requestMethod = request.method;
+      payloadData = request.data;
+      requestWithoutSchain = spec.buildRequests(bidWithoutSchain, DEFAULT_OPTION)[0];
+      payloadWithoutSchain = extractPayload(requestWithoutSchain);
+    });
 
     it('request should be made to IX endpoint with POST method and siteId in query param', function () {
       expect(requestMethod).to.equal('POST');
@@ -2302,7 +2323,11 @@ describe('IndexexchangeAdapter', function () {
     });
 
     describe('request should contain both banner and video requests', function () {
-      const request = spec.buildRequests([DEFAULT_BANNER_VALID_BID[0], DEFAULT_VIDEO_VALID_BID[0]], {});
+      let request;
+      before(() => {
+        request = spec.buildRequests([DEFAULT_BANNER_VALID_BID[0], DEFAULT_VIDEO_VALID_BID[0]], {});
+      });
+
       it('should have banner request', () => {
         const bannerImpression = extractPayload(request[0]).imp[0];
         const sidValue = DEFAULT_BANNER_VALID_BID[0].params.id;
@@ -2333,7 +2358,10 @@ describe('IndexexchangeAdapter', function () {
     });
 
     describe('request should contain both banner and native requests', function () {
-      const request = spec.buildRequests([DEFAULT_BANNER_VALID_BID[0], DEFAULT_NATIVE_VALID_BID[0]]);
+      let request;
+      before(() => {
+        request = spec.buildRequests([DEFAULT_BANNER_VALID_BID[0], DEFAULT_NATIVE_VALID_BID[0]]);
+      })
 
       it('should have banner request', () => {
         const bannerImpression = extractPayload(request[0]).imp[0];
@@ -2541,8 +2569,11 @@ describe('IndexexchangeAdapter', function () {
   });
 
   describe('buildRequestVideo', function () {
-    const request = spec.buildRequests(DEFAULT_VIDEO_VALID_BID, DEFAULT_OPTION);
-    const payloadData = request[0].data;
+    let request, payloadData;
+    before(() => {
+      request = spec.buildRequests(DEFAULT_VIDEO_VALID_BID, DEFAULT_OPTION);
+      payloadData = request[0].data;
+    })
 
     it('auction type should be set correctly', function () {
       const at = payloadData.at;
@@ -2922,7 +2953,10 @@ describe('IndexexchangeAdapter', function () {
 
     describe('both banner and video bidder params set', function () {
       const bids = [DEFAULT_MULTIFORMAT_BANNER_VALID_BID[0], DEFAULT_MULTIFORMAT_VIDEO_VALID_BID[0]];
-      const request = spec.buildRequests(bids, {});
+      let request;
+      before(() => {
+        request = spec.buildRequests(bids, {});
+      })
 
       it('should return valid banner requests', function () {
         const impressions = extractPayload(request[0]).imp;

package/test/spec/modules/lassoBidAdapter_spec.js

@@ -79,10 +79,12 @@ describe('lassoBidAdapter', function () {
   });
 
   describe('buildRequests', function () {
-    const validBidRequests = spec.buildRequests([bid], bidderRequest);
-    expect(validBidRequests).to.be.an('array').that.is.not.empty;
-
-    const bidRequest = validBidRequests[0];
+    let validBidRequests, bidRequest;
+    before(() => {
+      validBidRequests = spec.buildRequests([bid], bidderRequest);
+      expect(validBidRequests).to.be.an('array').that.is.not.empty;
+      bidRequest = validBidRequests[0];
+    })
 
     it('Returns valid bidRequest', function () {
       expect(bidRequest).to.exist;