Adagio bid adapter 2.1.0 (#4704)

* adagioBidAdapter: avoid preflight requests * adagioBidAdapter: always return computable features
prebid · Jan 9, 2020 · 60a3317 · 60a3317
1 parent e365a48
commit 60a3317
Show file tree

Hide file tree

Showing 2 changed files with 25 additions and 23 deletions.
diff --git a/modules/adagioBidAdapter.js b/modules/adagioBidAdapter.js
@@ -6,7 +6,7 @@ import JSEncrypt from 'jsencrypt/bin/jsencrypt';
 import sha256 from 'crypto-js/sha256';
 
 const BIDDER_CODE = 'adagio';
-const VERSION = '2.0.0';
+const VERSION = '2.1.0';
 const FEATURES_VERSION = '1';
 const ENDPOINT = 'https://mp.4dex.io/prebid';
 const SUPPORTED_MEDIA_TYPES = ['banner'];
@@ -93,6 +93,9 @@ const _features = {
     const viewportDims = _features.getViewPortDimensions().split('x');
     const w = utils.getWindowTop();
     const body = w.document.body;
+    if (!body) {
+      return ''
+    }
     const html = w.document.documentElement;
     const pageHeight = Math.max(body.scrollHeight, body.offsetHeight, html.clientHeight, html.scrollHeight, html.offsetHeight);
 
@@ -129,6 +132,8 @@ const _features = {
   },
 
   getSlotPosition: function(element) {
+    if (!element) return '';
+
     const w = utils.getWindowTop();
     const d = w.document;
     const el = element;
@@ -288,22 +293,19 @@ function _getFeatures(bidRequest) {
     element = w.document.getElementById(adUnitElementId);
   }
 
-  let features = {};
-  if (element) {
-    features = Object.assign({}, {
-      print_number: _features.getPrintNumber(bidRequest.adUnitCode).toString(),
-      page_dimensions: _features.getPageDimensions().toString(),
-      viewport_dimensions: _features.getViewPortDimensions().toString(),
-      dom_loading: _features.isDomLoading().toString(),
-      // layout: features.getLayout().toString(),
-      adunit_position: _features.getSlotPosition(element).toString(),
-      user_timestamp: _features.getTimestamp().toString(),
-      device: _features.getDevice().toString(),
-      url: w.location.origin + w.location.pathname,
-      browser: _features.getBrowser(),
-      os: _features.getOS()
-    })
-  }
+  const features = {
+    print_number: _features.getPrintNumber(bidRequest.adUnitCode).toString(),
+    page_dimensions: _features.getPageDimensions().toString(),
+    viewport_dimensions: _features.getViewPortDimensions().toString(),
+    dom_loading: _features.isDomLoading().toString(),
+    // layout: features.getLayout().toString(),
+    adunit_position: _features.getSlotPosition(element).toString(),
+    user_timestamp: _features.getTimestamp().toString(),
+    device: _features.getDevice().toString(),
+    url: w.location.origin + w.location.pathname,
+    browser: _features.getBrowser(),
+    os: _features.getOS()
+  };
 
   const adUnitFeature = {};
   adUnitFeature[adUnitElementId] = {
@@ -351,7 +353,7 @@ export const spec = {
       w.ADAGIO = w.ADAGIO || {};
       w.ADAGIO.adUnits = w.ADAGIO.adUnits || {};
       w.ADAGIO.pbjsAdUnits = w.ADAGIO.pbjsAdUnits || [];
-      isValid = !!(organizationId && site && placement && adUnitElementId && document.getElementById(adUnitElementId) !== null);
+      isValid = !!(organizationId && site && placement && adUnitElementId);
       const tempAdUnits = w.ADAGIO.pbjsAdUnits.filter((adUnit) => adUnit.code !== adUnitCode);
       tempAdUnits.push({
         code: adUnitCode,
@@ -415,7 +417,7 @@ export const spec = {
           featuresVersion: FEATURES_VERSION
         },
         options: {
-          contentType: 'application/json'
+          contentType: 'text/plain'
         }
       }
     });

diff --git a/test/spec/modules/adagioBidAdapter_spec.js b/test/spec/modules/adagioBidAdapter_spec.js
@@ -7,7 +7,7 @@ describe('adagioAdapter', () => {
   let utilsMock;
   const adapter = newBidder(spec);
   const ENDPOINT = 'https://mp.4dex.io/prebid';
-  const VERSION = '2.0.0';
+  const VERSION = '2.1.0';
 
   beforeEach(function() {
     localStorage.removeItem('adagioScript');
@@ -330,11 +330,11 @@ describe('adagioAdapter', () => {
       expect(request.data.prebidVersion).to.equal('$prebid.version$');
     });
 
-    it('features params must be empty if param adUnitElementId is not found', () => {
+    it('features params "adunit_position" must be empty if adUnitElement is not found in the DOM', () => {
       const requests = spec.buildRequests([Object.assign({}, bidRequests[0], {params: {adUnitElementId: 'does-not-exist'}})], bidderRequest);
       const request = requests[0];
-      const expected = {}
-      expect(request.data.adUnits[0].features).to.deep.equal(expected);
+      expect(request.data.adUnits[0].features).to.exist;
+      expect(request.data.adUnits[0].features.adunit_position).to.deep.equal('');
     });
 
     it('features params "adunit_position" should be computed even if DOM element is display:none', () => {