From 280e9577f24c370979c2712c5657ae8fda145e04 Mon Sep 17 00:00:00 2001
From: Amanda Dillon <41923726+agdillon@users.noreply.github.com>
Date: Mon, 28 Sep 2020 04:32:09 -0600
Subject: [PATCH] SpotX bid adapter: add page parameter (#5784)

---
 modules/spotxBidAdapter.js                | 23 ++++++++---
 test/spec/modules/spotxBidAdapter_spec.js | 47 +++++++++++++++++++++++
 2 files changed, 64 insertions(+), 6 deletions(-)

diff --git a/modules/spotxBidAdapter.js b/modules/spotxBidAdapter.js
index 7c4e2e3ef63..6104fce1d97 100644
--- a/modules/spotxBidAdapter.js
+++ b/modules/spotxBidAdapter.js
@@ -1,4 +1,5 @@
 import * as utils from '../src/utils.js';
+import { config } from '../src/config.js';
 import { Renderer } from '../src/Renderer.js';
 import { registerBidder } from '../src/adapters/bidderFactory.js';
 import { VIDEO } from '../src/mediaTypes.js';
@@ -19,7 +20,7 @@ export const spec = {
    * From Prebid.js: isBidRequestValid - Verify the the AdUnits.bids, respond with true (valid) or false (invalid).
    *
    * @param {object} bid The bid to validate.
-   * @return boolean True if this is a valid bid, and false otherwise.
+   * @return {boolean} True if this is a valid bid, and false otherwise.
    */
   isBidRequestValid: function(bid) {
     if (bid && typeof bid.params !== 'object') {
@@ -64,14 +65,24 @@ export const spec = {
    * from Prebid.js: buildRequests - Takes an array of valid bid requests, all of which are guaranteed to have passed the isBidRequestValid() test.
    *
    * @param {BidRequest[]} bidRequests A non-empty list of bid requests which should be sent to the Server.
-   * @return ServerRequest Info describing the request to the server.
+   * @param {object} bidderRequest - The master bidRequest object.
+   * @return {ServerRequest} Info describing the request to the server.
    */
   buildRequests: function(bidRequests, bidderRequest) {
-    const page = bidderRequest.refererInfo.referer;
-    const isPageSecure = !!page.match(/^https:/)
+    const referer = bidderRequest.refererInfo.referer;
+    const isPageSecure = !!referer.match(/^https:/);
 
     const siteId = '';
     const spotxRequests = bidRequests.map(function(bid) {
+      let page;
+      if (utils.getBidIdParameter('page', bid.params)) {
+        page = utils.getBidIdParameter('page', bid.params);
+      } else if (config.getConfig('pageUrl')) {
+        page = config.getConfig('pageUrl');
+      } else {
+        page = referer;
+      }
+
       const channelId = utils.getBidIdParameter('channel_id', bid.params);
       let pubcid = null;
 
@@ -436,11 +447,11 @@ function createOutstreamScript(bid) {
 
   const customOverride = utils.getBidIdParameter('custom_override', bid.renderer.config.outstream_options);
   if (customOverride && utils.isPlainObject(customOverride)) {
-    utils.logMessage('[SPOTX][renderer] Custom beahavior.');
+    utils.logMessage('[SPOTX][renderer] Custom behavior.');
     for (let name in customOverride) {
       if (customOverride.hasOwnProperty(name)) {
         if (name === 'channel_id' || name === 'vast_url' || name === 'content_page_url' || name === 'ad_unit') {
-          utils.logWarn('[SPOTX][renderer] Custom beahavior: following option cannot be overrided: ' + name);
+          utils.logWarn('[SPOTX][renderer] Custom behavior: following option cannot be overridden: ' + name);
         } else {
           dataSpotXParams['data-spotx_' + name] = customOverride[name];
         }
diff --git a/test/spec/modules/spotxBidAdapter_spec.js b/test/spec/modules/spotxBidAdapter_spec.js
index cf651670518..798fb3eec10 100644
--- a/test/spec/modules/spotxBidAdapter_spec.js
+++ b/test/spec/modules/spotxBidAdapter_spec.js
@@ -1,4 +1,5 @@
 import {expect} from 'chai';
+import {config} from 'src/config.js';
 import {spec, GOOGLE_CONSENT} from 'modules/spotxBidAdapter.js';
 
 describe('the spotx adapter', function () {
@@ -89,6 +90,7 @@ describe('the spotx adapter', function () {
       expect(spec.isBidRequestValid(bid)).to.equal(false);
     });
   });
+
   describe('buildRequests', function() {
     var bid, bidRequestObj;
 
@@ -125,6 +127,7 @@ describe('the spotx adapter', function () {
         page: 'prebid.js'
       });
     });
+
     it('should change request parameters based on options sent', function() {
       var request = spec.buildRequests([bid], bidRequestObj)[0];
       expect(request.data.imp.video.ext).to.deep.equal({
@@ -332,6 +335,50 @@ describe('the spotx adapter', function () {
       expect(request.data.imp.video.ext.placement).to.equal(2);
       expect(request.data.imp.video.ext.pos).to.equal(5);
     });
+
+    it('should pass page param and override refererInfo.referer', function() {
+      var request;
+
+      bid.params.page = 'https://example.com';
+
+      var origGetConfig = config.getConfig;
+      sinon.stub(config, 'getConfig').callsFake(function (key) {
+        if (key === 'pageUrl') {
+          return 'https://www.spotx.tv';
+        }
+        return origGetConfig.apply(config, arguments);
+      });
+
+      request = spec.buildRequests([bid], bidRequestObj)[0];
+
+      expect(request.data.site.page).to.equal('https://example.com');
+      config.getConfig.restore();
+    });
+
+    it('should use pageUrl from config if page param is not passed', function() {
+      var request;
+
+      var origGetConfig = config.getConfig;
+      sinon.stub(config, 'getConfig').callsFake(function (key) {
+        if (key === 'pageUrl') {
+          return 'https://www.spotx.tv';
+        }
+        return origGetConfig.apply(config, arguments);
+      });
+
+      request = spec.buildRequests([bid], bidRequestObj)[0];
+
+      expect(request.data.site.page).to.equal('https://www.spotx.tv');
+      config.getConfig.restore();
+    });
+
+    it('should use refererInfo.referer if no page or pageUrl are passed', function() {
+      var request;
+
+      request = spec.buildRequests([bid], bidRequestObj)[0];
+
+      expect(request.data.site.page).to.equal('prebid.js');
+    });
   });
 
   describe('interpretResponse', function() {