prebid.js 7.53.0 → 8.0.0

package/src/adapterManager.js

@@ -12,7 +12,7 @@ import {
   getUniqueIdentifierStr,
   getUserConfiguredParams,
   groupBy,
-  isArray,
+  isArray, isPlainObject,
   isValidMediaTypes,
   logError,
   logInfo,
@@ -22,7 +22,6 @@ import {
   shuffle,
   timestamp,
 } from './utils.js';
-import {processAdUnitsForLabels} from './sizeMapping.js';
 import {decorateAdUnitsWithNativeParams, nativeAdapters} from './native.js';
 import {newBidder} from './adapters/bidderFactory.js';
 import {ajaxBuilder} from './ajax.js';
@@ -42,7 +41,7 @@ import {ACTIVITY_FETCH_BIDS, ACTIVITY_REPORT_ANALYTICS} from './activities/activ
 import {ACTIVITY_PARAM_ANL_CONFIG, ACTIVITY_PARAM_S2S_NAME, activityParamsBuilder} from './activities/params.js';
 import {redactor} from './activities/redactor.js';
 
-const PBS_ADAPTER_NAME = 'pbsBidAdapter';
+export const PBS_ADAPTER_NAME = 'pbsBidAdapter';
 export const PARTITIONS = {
   CLIENT: 'client',
   SERVER: 'server'
@@ -69,6 +68,12 @@ var _analyticsRegistry = {};
 
 const activityParams = activityParamsBuilder((alias) => adapterManager.resolveAlias(alias));
 
+export function s2sActivityParams(s2sConfig) {
+  return activityParams(MODULE_TYPE_PREBID, PBS_ADAPTER_NAME, {
+    [ACTIVITY_PARAM_S2S_NAME]: s2sConfig.configName
+  });
+}
+
 /**
  * @typedef {object} LabelDescriptor
  * @property {boolean} labelAll describes whether or not this object expects all labels to match, or any label to match
@@ -205,7 +210,7 @@ export let coppaDataHandler = {
  * they should be exposed under `adUnit.bids[].mediaTypes`.
  */
 export const setupAdUnitMediaTypes = hook('sync', (adUnits, labels) => {
-  return processAdUnitsForLabels(adUnits, labels);
+  return adUnits;
 }, 'setupAdUnitMediaTypes')
 
 /**
@@ -250,8 +255,13 @@ adapterManager.makeBidRequests = hook('sync', function (adUnits, auctionStart, a
     decorateAdUnitsWithNativeParams(adUnits);
   }
 
-  // filter out bidders that cannot participate in the auction
-  adUnits.forEach(au => au.bids = au.bids.filter((bid) => !bid.bidder || dep.isAllowed(ACTIVITY_FETCH_BIDS, activityParams(MODULE_TYPE_BIDDER, bid.bidder))))
+  adUnits.forEach(au => {
+    if (!isPlainObject(au.mediaTypes)) {
+      au.mediaTypes = {};
+    }
+    // filter out bidders that cannot participate in the auction
+    au.bids = au.bids.filter((bid) => !bid.bidder || dep.isAllowed(ACTIVITY_FETCH_BIDS, activityParams(MODULE_TYPE_BIDDER, bid.bidder)))
+  });
 
   adUnits = setupAdUnitMediaTypes(adUnits, labels);
 
@@ -267,9 +277,13 @@ adapterManager.makeBidRequests = hook('sync', function (adUnits, auctionStart, a
   const ortb2 = ortb2Fragments.global || {};
   const bidderOrtb2 = ortb2Fragments.bidder || {};
 
-  function addOrtb2(bidderRequest) {
-    const redact = dep.redact(activityParams(MODULE_TYPE_BIDDER, bidderRequest.bidderCode));
-    const fpd = Object.freeze(redact.ortb2(mergeDeep({}, ortb2, bidderOrtb2[bidderRequest.bidderCode])));
+  function addOrtb2(bidderRequest, s2sActivityParams) {
+    const redact = dep.redact(
+      s2sActivityParams != null
+        ? s2sActivityParams
+        : activityParams(MODULE_TYPE_BIDDER, bidderRequest.bidderCode)
+    );
+    const fpd = Object.freeze(redact.ortb2(mergeDeep({source: {tid: auctionId}}, ortb2, bidderOrtb2[bidderRequest.bidderCode])));
     bidderRequest.ortb2 = fpd;
     bidderRequest.bids = bidderRequest.bids.map((bid) => {
       bid.ortb2 = fpd;
@@ -278,14 +292,9 @@ adapterManager.makeBidRequests = hook('sync', function (adUnits, auctionStart, a
     return bidderRequest;
   }
 
-  function isS2SAllowed(s2sConfig) {
-    return dep.isAllowed(ACTIVITY_FETCH_BIDS, activityParams(MODULE_TYPE_PREBID, PBS_ADAPTER_NAME, {
-      [ACTIVITY_PARAM_S2S_NAME]: s2sConfig.configName
-    }));
-  }
-
   _s2sConfigs.forEach(s2sConfig => {
-    if (s2sConfig && s2sConfig.enabled && isS2SAllowed(s2sConfig)) {
+    const s2sParams = s2sActivityParams(s2sConfig);
+    if (s2sConfig && s2sConfig.enabled && dep.isAllowed(ACTIVITY_FETCH_BIDS, s2sParams)) {
       let {adUnits: adUnitsS2SCopy, hasModuleBids} = getAdUnitCopyForPrebidServer(adUnits, s2sConfig);
 
       // uniquePbsTid is so we know which server to send which bids to during the callBids function
@@ -305,7 +314,7 @@ adapterManager.makeBidRequests = hook('sync', function (adUnits, auctionStart, a
           src: CONSTANTS.S2S.SRC,
           refererInfo,
           metrics,
-        });
+        }, s2sParams);
         if (bidderRequest.bids.length !== 0) {
           bidRequests.push(bidderRequest);
         }
@@ -622,9 +631,11 @@ function invokeBidderMethod(bidder, method, spec, fn, ...params) {
 }
 
 function tryCallBidderMethod(bidder, method, param) {
-  const target = getBidderMethod(bidder, method);
-  if (target != null) {
-    invokeBidderMethod(bidder, method, ...target, param);
+  if (param?.src !== CONSTANTS.S2S.SRC) {
+    const target = getBidderMethod(bidder, method);
+    if (target != null) {
+      invokeBidderMethod(bidder, method, ...target, param);
+    }
   }
 }
 

package/src/adapters/bidderFactory.js

@@ -8,28 +8,24 @@ import {isValidVideoBid} from '../video.js';
 import CONSTANTS from '../constants.json';
 import * as events from '../events.js';
 import {includes} from '../polyfill.js';
-import {ajax} from '../ajax.js';
 import {
-  deepAccess,
   delayExecution,
-  flatten,
   isArray,
   isPlainObject,
   logError,
-  logWarn,
+  logWarn, memoize,
   parseQueryStringParameters,
   parseSizesInput,
-  timestamp,
   uniques
 } from '../utils.js';
-import {ADPOD} from '../mediaTypes.js';
-import {getHook, hook} from '../hook.js';
-import {getCoreStorageManager} from '../storageManager.js';
+import {hook} from '../hook.js';
 import {auctionManager} from '../auctionManager.js';
 import {bidderSettings} from '../bidderSettings.js';
 import {useMetrics} from '../utils/perfMetrics.js';
-
-export const storage = getCoreStorageManager('bidderFactory');
+import {isActivityAllowed} from '../activities/rules.js';
+import {activityParams} from '../activities/activityParams.js';
+import {MODULE_TYPE_BIDDER} from '../activities/modules.js';
+import {ACTIVITY_TRANSMIT_TID} from '../activities/activities.js';
 
 /**
  * This file aims to support Adapters during the Prebid 0.x -> 1.x transition.
@@ -155,8 +151,6 @@ export const storage = getCoreStorageManager('bidderFactory');
 // common params for all mediaTypes
 const COMMON_BID_RESPONSE_KEYS = ['cpm', 'ttl', 'creativeId', 'netRevenue', 'currency'];
 
-const DEFAULT_REFRESHIN_DAYS = 1;
-
 /**
  * Register a bidder with prebid, using the given spec.
  *
@@ -190,6 +184,38 @@ export function registerBidder(spec) {
   }
 }
 
+function guardTids(bidderCode) {
+  if (isActivityAllowed(ACTIVITY_TRANSMIT_TID, activityParams(MODULE_TYPE_BIDDER, bidderCode))) {
+    return {
+      bidRequest: (br) => br,
+      bidderRequest: (br) => br
+    };
+  }
+  function get(target, prop, receiver) {
+    if (['transactionId', 'auctionId'].includes(prop)) {
+      return null;
+    }
+    return Reflect.get(target, prop, receiver);
+  }
+  const bidRequest = memoize((br) => new Proxy(br, {get}), (arg) => arg.bidId)
+  /**
+   * Return a view on bidd(er) requests where auctionId/transactionId are nulled if the bidder is not allowed `transmitTid`.
+   *
+   * Because both auctionId and transactionId are used for Prebid's own internal bookkeeping, we cannot simply erase them
+   * from request objects; and because request objects are quite complex and not easily cloneable, we hide the IDs
+   * with a proxy instead. This should be used only around the adapter logic.
+   */
+  return {
+    bidRequest,
+    bidderRequest: (br) => new Proxy(br, {
+      get(target, prop, receiver) {
+        if (prop === 'bids') return br.bids.map(bidRequest);
+        return get(target, prop, receiver);
+      }
+    })
+  }
+}
+
 /**
  * Make a new bidder from the given spec. This is exported mainly for testing.
  * Adapters will probably find it more convenient to use registerBidder instead.
@@ -206,6 +232,7 @@ export function newBidder(spec) {
       if (!Array.isArray(bidderRequest.bids)) {
         return;
       }
+      const tidGuard = guardTids(bidderRequest.bidderCode);
 
       const adUnitCodesHandled = {};
       function addBidWithCode(adUnitCode, bid) {
@@ -231,7 +258,7 @@ export function newBidder(spec) {
       }
 
       const validBidRequests = adapterMetrics(bidderRequest)
-        .measureTime('validate', () => bidderRequest.bids.filter(filterAndWarn));
+        .measureTime('validate', () => bidderRequest.bids.filter((br) => filterAndWarn(tidGuard.bidRequest(br))));
 
       if (validBidRequests.length === 0) {
         afterAllResponses();
@@ -246,7 +273,7 @@ export function newBidder(spec) {
         }
       });
 
-      processBidderRequests(spec, validBidRequests, bidderRequest, ajax, configEnabledCallback, {
+      processBidderRequests(spec, validBidRequests.map(tidGuard.bidRequest), tidGuard.bidderRequest(bidderRequest), ajax, configEnabledCallback, {
         onRequest: requestObject => events.emit(CONSTANTS.EVENTS.BEFORE_BIDDER_HTTP, bidderRequest, requestObject),
         onResponse: (resp) => {
           onTimelyResponse(spec.code);
@@ -486,83 +513,6 @@ export const registerSyncInner = hook('async', function(spec, responses, gdprCon
 export const addComponentAuction = hook('sync', (adUnitCode, fledgeAuctionConfig) => {
 }, 'addComponentAuction');
 
-export function preloadBidderMappingFile(fn, adUnits) {
-  if (FEATURES.VIDEO) {
-    if (!config.getConfig('adpod.brandCategoryExclusion')) {
-      return fn.call(this, adUnits);
-    }
-
-    let adPodBidders = adUnits
-      .filter((adUnit) => deepAccess(adUnit, 'mediaTypes.video.context') === ADPOD)
-      .map((adUnit) => adUnit.bids.map((bid) => bid.bidder))
-      .reduce(flatten, [])
-      .filter(uniques);
-
-    adPodBidders.forEach(bidder => {
-      let bidderSpec = adapterManager.getBidAdapter(bidder);
-      if (bidderSpec.getSpec().getMappingFileInfo) {
-        let info = bidderSpec.getSpec().getMappingFileInfo();
-        let refreshInDays = (info.refreshInDays) ? info.refreshInDays : DEFAULT_REFRESHIN_DAYS;
-        let key = (info.localStorageKey) ? info.localStorageKey : bidderSpec.getSpec().code;
-        let mappingData = storage.getDataFromLocalStorage(key);
-        try {
-          mappingData = mappingData ? JSON.parse(mappingData) : undefined;
-          if (!mappingData || timestamp() > mappingData.lastUpdated + refreshInDays * 24 * 60 * 60 * 1000) {
-            ajax(info.url,
-              {
-                success: (response) => {
-                  try {
-                    response = JSON.parse(response);
-                    let mapping = {
-                      lastUpdated: timestamp(),
-                      mapping: response.mapping
-                    }
-                    storage.setDataInLocalStorage(key, JSON.stringify(mapping));
-                  } catch (error) {
-                    logError(`Failed to parse ${bidder} bidder translation mapping file`);
-                  }
-                },
-                error: () => {
-                  logError(`Failed to load ${bidder} bidder translation file`)
-                }
-              },
-            );
-          }
-        } catch (error) {
-          logError(`Failed to parse ${bidder} bidder translation mapping file`);
-        }
-      }
-    });
-    fn.call(this, adUnits);
-  } else {
-    return fn.call(this, adUnits)
-  }
-}
-
-getHook('checkAdUnitSetup').before(preloadBidderMappingFile);
-
-/**
- * Reads the data stored in localstorage and returns iab subcategory
- * @param {string} bidderCode bidderCode
- * @param {string} category bidders category
- */
-export function getIabSubCategory(bidderCode, category) {
-  let bidderSpec = adapterManager.getBidAdapter(bidderCode);
-  if (bidderSpec.getSpec().getMappingFileInfo) {
-    let info = bidderSpec.getSpec().getMappingFileInfo();
-    let key = (info.localStorageKey) ? info.localStorageKey : bidderSpec.getBidderCode();
-    let data = storage.getDataFromLocalStorage(key);
-    if (data) {
-      try {
-        data = JSON.parse(data);
-      } catch (error) {
-        logError(`Failed to parse ${bidderCode} mapping data stored in local storage`);
-      }
-      return (data.mapping[category]) ? data.mapping[category] : null;
-    }
-  }
-}
-
 // check that the bid has a width and height set
 function validBidSize(adUnitCode, bid, {index = auctionManager.index} = {}) {
   if ((bid.width || parseInt(bid.width, 10) === 0) && (bid.height || parseInt(bid.height, 10) === 0)) {

package/src/auction.js

@@ -92,7 +92,6 @@ import adapterManager from './adapterManager.js';
 import CONSTANTS from './constants.json';
 import {GreedyPromise} from './utils/promise.js';
 import {useMetrics} from './utils/perfMetrics.js';
-import {createBid} from './bidfactory.js';
 import {adjustCpm} from './utils/cpm.js';
 import {getGlobal} from './prebidGlobal.js';
 
@@ -494,26 +493,11 @@ export function auctionCallbacks(auctionDone, auctionInstance, {index = auctionM
 
   function rejectBidResponse(adUnitCode, bid, reason) {
     return handleBidResponse(adUnitCode, bid, (done) => {
-      // return a "NO_BID" replacement that the caller can decide to continue with
-      // TODO: remove this in v8; see https://github.com/prebid/Prebid.js/issues/8956
-      const noBid = createBid(CONSTANTS.STATUS.NO_BID, bid.getIdentifiers?.());
-      Object.assign(noBid, Object.fromEntries(Object.entries(bid).filter(([k]) => !noBid.hasOwnProperty(k) && ![
-        'ad',
-        'adUrl',
-        'vastXml',
-        'vastUrl',
-        'native',
-      ].includes(k))));
-      noBid.status = CONSTANTS.BID_STATUS.BID_REJECTED;
-      noBid.cpm = 0;
-
       bid.rejectionReason = reason;
       logWarn(`Bid from ${bid.bidder || 'unknown bidder'} was rejected: ${reason}`, bid)
       events.emit(CONSTANTS.EVENTS.BID_REJECTED, bid);
       auctionInstance.addBidRejected(bid);
       done();
-
-      return noBid;
     })
   }
 
@@ -552,12 +536,12 @@ export function auctionCallbacks(auctionDone, auctionInstance, {index = auctionM
         waitFor((bidderRequest && bidderRequest.bidderRequestId) || '', addBidResponse.call({
           dispatch: acceptBidResponse,
         }, adUnitCode, bid, (() => {
-          let rejection;
+          let rejected = false;
           return (reason) => {
-            if (rejection == null) {
-              rejection = rejectBidResponse(adUnitCode, bid, reason);
+            if (!rejected) {
+              rejectBidResponse(adUnitCode, bid, reason);
+              rejected = true;
             }
-            return rejection;
           }
         })()));
       }
@@ -581,8 +565,8 @@ export function addBidToAuction(auctionInstance, bidResponse) {
   setupBidTargeting(bidResponse);
 
   useMetrics(bidResponse.metrics).timeSince('addBidResponse', 'addBidResponse.total');
-  events.emit(CONSTANTS.EVENTS.BID_RESPONSE, bidResponse);
   auctionInstance.addBidReceived(bidResponse);
+  events.emit(CONSTANTS.EVENTS.BID_RESPONSE, bidResponse);
 
   doCallbacksIfTimedout(auctionInstance, bidResponse);
 }

package/src/constants.json

@@ -11,8 +11,7 @@
   },
   "DEBUG_MODE": "pbjs_debug",
   "STATUS": {
-    "GOOD": 1,
-    "NO_BID": 2
+    "GOOD": 1
   },
   "CB": {
     "TYPE": {

package/src/utils.js

@@ -1,9 +1,10 @@
-import { config } from './config.js';
+import {config} from './config.js';
 import clone from 'just-clone';
 import {find, includes} from './polyfill.js';
 import CONSTANTS from './constants.json';
 import {GreedyPromise} from './utils/promise.js';
 import {getGlobal} from './prebidGlobal.js';
+
 export { default as deepAccess } from 'dlv/index.js';
 export { dset as deepSetValue } from 'dset';
 
@@ -1067,39 +1068,6 @@ export function pick(obj, properties) {
   }, {});
 }
 
-/**
- * Converts an object of arrays (either strings or numbers) into an array of objects containing key and value properties
- * normally read from bidder params
- * eg { foo: ['bar', 'baz'], fizz: ['buzz'] }
- * becomes [{ key: 'foo', value: ['bar', 'baz']}, {key: 'fizz', value: ['buzz']}]
- * @param {Object} keywords object of arrays representing keyvalue pairs
- * @param {string} paramName name of parent object (eg 'keywords') containing keyword data, used in error handling
- */
-export function transformBidderParamKeywords(keywords, paramName = 'keywords') {
-  let arrs = [];
-
-  _each(keywords, (v, k) => {
-    if (isArray(v)) {
-      let values = [];
-      _each(v, (val) => {
-        val = getValueString(paramName + '.' + k, val);
-        if (val || val === '') { values.push(val); }
-      });
-      v = values;
-    } else {
-      v = getValueString(paramName + '.' + k, v);
-      if (isStr(v)) {
-        v = [v];
-      } else {
-        return;
-      } // unsuported types - don't send a key
-    }
-    arrs.push({key: k, value: v});
-  });
-
-  return arrs;
-}
-
 /**
  * Try to convert a value to a type.
  * If it can't be done, the value will be returned.

package/test/fixtures/fixtures.js

@@ -797,13 +797,6 @@ export function getAdUnits() {
             'aId': 3080
           }
         },
-        {
-          'bidder': 'aol',
-          'params': {
-            'network': '112345.45',
-            'placement': 12345
-          }
-        },
         {
           'bidder': 'sovrn',
           'params': {

package/test/pages/banner.html

@@ -57,6 +57,7 @@
 		});
 
 		pbjs.que.push(function () {
+			pbjs.setConfig({enableTIDs: true});
 			pbjs.addAdUnits(adUnits);
 			pbjs.requestBids({ bidsBackHandler: sendAdServerRequest });
 		});

package/test/pages/bidderSettings.html

@@ -67,6 +67,7 @@
     });
 
     pbjs.que.push(function () {
+      pbjs.setConfig({enableTIDs: true});
       pbjs.addAdUnits(adUnits);
       pbjs.bidderSettings = {
         appnexus: {

package/test/pages/consent_mgt_gdpr.html

@@ -146,6 +146,7 @@
     pbjs.que.push(function () {
       pbjs.addAdUnits(adUnits);
       pbjs.setConfig({
+        enableTIDs: true,
         consentManagement: {
           gdpr: {
             cmpApi: 'static',

package/test/pages/currency.html

@@ -77,6 +77,7 @@
 
         pbjs.que.push(function() {
             pbjs.setConfig({
+                enableTIDs: true,
                 "currency": {
                     "adServerCurrency": "GBP",
                     "granularityMultiplier": 1,

package/test/pages/instream.html

@@ -60,6 +60,7 @@
 
         pbjs.addAdUnits(videoAdUnit);
         pbjs.setConfig({
+                    enableTIDs: true,
                     debug: true,
                     cache: {
                         url: 'https://prebid.adnxs.com/pbc/v1/cache'

package/test/pages/multiple_bidders.html

@@ -72,7 +72,7 @@
           }
         }]
       }];
-
+      pbjs.setConfig({enableTIDs: true});
       pbjs.addAdUnits(adUnits);
       pbjs.requestBids({
         timeout: PREBID_TIMEOUT,

package/test/pages/native.html

@@ -83,6 +83,7 @@
 		});
 
 		pbjs.que.push(function () {
+			pbjs.setConfig({enableTIDs: true});
 			pbjs.addAdUnits(adUnits);
 			pbjs.requestBids({ bidsBackHandler: sendAdServerRequest });
 		});

package/test/pages/outstream.html

@@ -61,7 +61,10 @@
 		googletag.cmd = googletag.cmd || [];
 
 		pbjs.que.push(function () {
-			pbjs.setConfig({ debug: true });
+			pbjs.setConfig({
+				enableTIDs: true,
+				debug: true
+			});
       pbjs.addAdUnits(outstreamVideoAdUnit);
       pbjs.bidderSettings = {
         appnexus: {

package/test/spec/activities/redactor_spec.js

@@ -7,7 +7,7 @@ import {
 import {ACTIVITY_PARAM_COMPONENT_NAME, ACTIVITY_PARAM_COMPONENT_TYPE} from '../../../src/activities/params.js';
 import {
   ACTIVITY_TRANSMIT_EIDS,
-  ACTIVITY_TRANSMIT_PRECISE_GEO,
+  ACTIVITY_TRANSMIT_PRECISE_GEO, ACTIVITY_TRANSMIT_TID,
   ACTIVITY_TRANSMIT_UFPD
 } from '../../../src/activities/activities.js';
 import {deepAccess, deepSetValue} from '../../../src/utils.js';
@@ -271,6 +271,10 @@ describe('redactor', () => {
     testAllowDeny(ACTIVITY_TRANSMIT_EIDS, (allowed) => {
       testPropertiesAreRemoved(() => redactor.bidRequest, ['userId', 'userIdAsEids'], allowed);
     });
+
+    testAllowDeny(ACTIVITY_TRANSMIT_TID, (allowed) => {
+      testPropertiesAreRemoved(() => redactor.bidRequest, ['ortb2Imp.ext.tid'], allowed);
+    })
   });
 
   describe('.ortb2', () => {
@@ -282,6 +286,10 @@ describe('redactor', () => {
       testPropertiesAreRemoved(() => redactor.ortb2, ORTB_UFPD_PATHS, allowed)
     });
 
+    testAllowDeny(ACTIVITY_TRANSMIT_TID, (allowed) => {
+      testPropertiesAreRemoved(() => redactor.ortb2, ['source.tid'], allowed);
+    });
+
     testAllowDeny(ACTIVITY_TRANSMIT_PRECISE_GEO, (allowed) => {
       ORTB_GEO_PATHS.forEach(path => {
         it(`should ${allowed ? 'NOT ' : ''} round down ${path}`, () => {
@@ -291,6 +299,6 @@ describe('redactor', () => {
           expect(deepAccess(ortb2, path)).to.eql(allowed ? 1.2345 : 1.23);
         })
       })
-    })
+    });
   });
 })

package/test/spec/appnexusKeywords_spec.js

@@ -0,0 +1,70 @@
+import {transformBidderParamKeywords} from '../../libraries/appnexusKeywords/anKeywords.js';
+import {expect} from 'chai/index.js';
+import * as utils from '../../src/utils.js';
+
+describe('transformBidderParamKeywords', function () {
+  it('returns an array of objects when keyvalue is an array', function () {
+    let keywords = {
+      genre: ['rock', 'pop']
+    };
+    let result = transformBidderParamKeywords(keywords);
+    expect(result).to.deep.equal([{
+      key: 'genre',
+      value: ['rock', 'pop']
+    }]);
+  });
+
+  it('returns an array of objects when keyvalue is a string', function () {
+    let keywords = {
+      genre: 'opera'
+    };
+    let result = transformBidderParamKeywords(keywords);
+    expect(result).to.deep.equal([{
+      key: 'genre',
+      value: ['opera']
+    }]);
+  });
+
+  it('returns an array of objects when keyvalue is a number', function () {
+    let keywords = {
+      age: 15
+    };
+    let result = transformBidderParamKeywords(keywords);
+    expect(result).to.deep.equal([{
+      key: 'age',
+      value: ['15']
+    }]);
+  });
+
+  it('returns an array of objects when using multiple keys with values of differing types', function () {
+    let keywords = {
+      genre: 'classical',
+      mix: ['1', 2, '3', 4],
+      age: 10
+    };
+    let result = transformBidderParamKeywords(keywords);
+    expect(result).to.deep.equal([{
+      key: 'genre',
+      value: ['classical']
+    }, {
+      key: 'mix',
+      value: ['1', '2', '3', '4']
+    }, {
+      key: 'age',
+      value: ['10']
+    }]);
+  });
+
+  it('returns an array of objects when the keyvalue uses an empty string', function () {
+    let keywords = {
+      test: [''],
+      test2: ''
+    };
+    let result = transformBidderParamKeywords(keywords);
+    expect(result).to.deep.equal([{
+      key: 'test',
+    }, {
+      key: 'test2',
+    }]);
+  });
+});