prebid.js 5.18.0 → 6.1.0

package/test/spec/unit/secureCreatives_spec.js

@@ -116,7 +116,7 @@ describe('secureCreatives', () => {
     beforeEach(function() {
       spyAddWinningBid = sinon.spy(auctionManager, 'addWinningBid');
       spyLogWarn = sinon.spy(utils, 'logWarn');
-      stubFireNativeTrackers = sinon.stub(native, 'fireNativeTrackers');
+      stubFireNativeTrackers = sinon.stub(native, 'fireNativeTrackers').callsFake(message => { return message.action; });
       stubGetAllAssetsMessage = sinon.stub(native, 'getAllAssetsMessage');
       stubEmit = sinon.stub(events, 'emit');
     });
@@ -263,10 +263,9 @@ describe('secureCreatives', () => {
         sinon.assert.calledOnce(stubGetAllAssetsMessage);
         sinon.assert.calledWith(stubGetAllAssetsMessage, data, adResponse);
         sinon.assert.calledOnce(ev.source.postMessage);
-        sinon.assert.calledOnce(stubFireNativeTrackers);
-        sinon.assert.calledWith(stubFireNativeTrackers, data, adResponse);
-        sinon.assert.calledOnce(spyAddWinningBid);
-        sinon.assert.calledWith(stubEmit, CONSTANTS.EVENTS.BID_WON, adResponse);
+        sinon.assert.notCalled(stubFireNativeTrackers);
+        sinon.assert.neverCalledWith(stubEmit, CONSTANTS.EVENTS.BID_WON);
+        sinon.assert.notCalled(spyAddWinningBid);
         sinon.assert.neverCalledWith(stubEmit, CONSTANTS.EVENTS.STALE_RENDER);
       });
 
@@ -293,14 +292,11 @@ describe('secureCreatives', () => {
         sinon.assert.calledOnce(stubGetAllAssetsMessage);
         sinon.assert.calledWith(stubGetAllAssetsMessage, data, adResponse);
         sinon.assert.calledOnce(ev.source.postMessage);
-        sinon.assert.calledOnce(stubFireNativeTrackers);
-        sinon.assert.calledWith(stubFireNativeTrackers, data, adResponse);
-        sinon.assert.calledOnce(spyAddWinningBid);
-        sinon.assert.calledWith(stubEmit, CONSTANTS.EVENTS.BID_WON, adResponse);
+        sinon.assert.notCalled(stubFireNativeTrackers);
+        sinon.assert.neverCalledWith(stubEmit, CONSTANTS.EVENTS.BID_WON);
+        sinon.assert.notCalled(spyAddWinningBid);
         sinon.assert.neverCalledWith(stubEmit, CONSTANTS.EVENTS.STALE_RENDER);
 
-        expect(adResponse).to.have.property('status', CONSTANTS.BID_STATUS.RENDERED);
-
         resetHistories(ev.source.postMessage);
 
         receiveMessage(ev);
@@ -309,10 +305,9 @@ describe('secureCreatives', () => {
         sinon.assert.calledOnce(stubGetAllAssetsMessage);
         sinon.assert.calledWith(stubGetAllAssetsMessage, data, adResponse);
         sinon.assert.calledOnce(ev.source.postMessage);
-        sinon.assert.calledOnce(stubFireNativeTrackers);
-        sinon.assert.calledWith(stubFireNativeTrackers, data, adResponse);
-        sinon.assert.calledOnce(spyAddWinningBid);
-        sinon.assert.calledWith(stubEmit, CONSTANTS.EVENTS.BID_WON, adResponse);
+        sinon.assert.notCalled(stubFireNativeTrackers);
+        sinon.assert.neverCalledWith(stubEmit, CONSTANTS.EVENTS.BID_WON);
+        sinon.assert.notCalled(spyAddWinningBid);
         sinon.assert.neverCalledWith(stubEmit, CONSTANTS.EVENTS.STALE_RENDER);
       });
 
@@ -341,14 +336,11 @@ describe('secureCreatives', () => {
         sinon.assert.calledOnce(stubGetAllAssetsMessage);
         sinon.assert.calledWith(stubGetAllAssetsMessage, data, adResponse);
         sinon.assert.calledOnce(ev.source.postMessage);
-        sinon.assert.calledOnce(stubFireNativeTrackers);
-        sinon.assert.calledWith(stubFireNativeTrackers, data, adResponse);
-        sinon.assert.calledOnce(spyAddWinningBid);
-        sinon.assert.calledWith(stubEmit, CONSTANTS.EVENTS.BID_WON, adResponse);
+        sinon.assert.notCalled(stubFireNativeTrackers);
+        sinon.assert.neverCalledWith(stubEmit, CONSTANTS.EVENTS.BID_WON);
+        sinon.assert.notCalled(spyAddWinningBid);
         sinon.assert.neverCalledWith(stubEmit, CONSTANTS.EVENTS.STALE_RENDER);
 
-        expect(adResponse).to.have.property('status', CONSTANTS.BID_STATUS.RENDERED);
-
         resetHistories(ev.source.postMessage);
 
         receiveMessage(ev);
@@ -357,14 +349,51 @@ describe('secureCreatives', () => {
         sinon.assert.calledOnce(stubGetAllAssetsMessage);
         sinon.assert.calledWith(stubGetAllAssetsMessage, data, adResponse);
         sinon.assert.calledOnce(ev.source.postMessage);
-        sinon.assert.calledOnce(stubFireNativeTrackers);
-        sinon.assert.calledWith(stubFireNativeTrackers, data, adResponse);
-        sinon.assert.calledOnce(spyAddWinningBid);
-        sinon.assert.calledWith(stubEmit, CONSTANTS.EVENTS.BID_WON, adResponse);
+        sinon.assert.notCalled(stubFireNativeTrackers);
+        sinon.assert.neverCalledWith(stubEmit, CONSTANTS.EVENTS.BID_WON);
+        sinon.assert.notCalled(spyAddWinningBid);
         sinon.assert.neverCalledWith(stubEmit, CONSTANTS.EVENTS.STALE_RENDER);
 
         configObj.setConfig({'auctionOptions': {}});
       });
+
+      it('Prebid native should fire trackers', function () {
+        pushBidResponseToAuction({});
+
+        const data = {
+          adId: bidId,
+          message: 'Prebid Native',
+          action: 'click',
+        };
+
+        const ev = {
+          data: JSON.stringify(data),
+          source: {
+            postMessage: sinon.stub()
+          },
+          origin: 'any origin'
+        };
+
+        receiveMessage(ev);
+
+        sinon.assert.neverCalledWith(spyLogWarn, warning);
+        sinon.assert.calledOnce(stubFireNativeTrackers);
+        sinon.assert.neverCalledWith(stubEmit, CONSTANTS.EVENTS.BID_WON);
+        sinon.assert.notCalled(spyAddWinningBid);
+
+        resetHistories(ev.source.postMessage);
+
+        delete data.action;
+        ev.data = JSON.stringify(data);
+        receiveMessage(ev);
+
+        sinon.assert.neverCalledWith(spyLogWarn, warning);
+        sinon.assert.calledOnce(stubFireNativeTrackers);
+        sinon.assert.calledWith(stubEmit, CONSTANTS.EVENTS.BID_WON, adResponse);
+        sinon.assert.calledOnce(spyAddWinningBid);
+
+        expect(adResponse).to.have.property('status', CONSTANTS.BID_STATUS.RENDERED);
+      });
     });
   });
 });

package/wdio.conf.js

@@ -9,7 +9,7 @@ function getCapabilities() {
     return platformMap[os];
   }
 
-  // only IE 11, Chrome 80 & Firefox 73 run as part of functional tests
+  // only Chrome 80 & Firefox 73 run as part of functional tests
   // rest of the browsers are discarded.
   delete browsers['bs_chrome_79_windows_10'];
   delete browsers['bs_firefox_72_windows_10'];