Kubient Bid Adapter: update if bidfloor is zero

modules/kubientBidAdapter.js

@@ -24,22 +24,23 @@ export const spec = {
       return;
     }
     return validBidRequests.map(function (bid) {
-      let floor = 0.0;
+      let adSlot = {
+        bidId: bid.bidId,
+        zoneId: bid.params.zoneid || ''
+      };
+
       if (typeof bid.getFloor === 'function') {
         const mediaType = (Object.keys(bid.mediaTypes).length == 1) ? Object.keys(bid.mediaTypes)[0] : '*';
         const sizes = bid.sizes || '*';
         const floorInfo = bid.getFloor({currency: 'USD', mediaType: mediaType, size: sizes});
-        if (typeof floorInfo === 'object' && floorInfo.currency === 'USD' && !isNaN(parseFloat(floorInfo.floor))) {
-          floor = parseFloat(floorInfo.floor);
+        if (typeof floorInfo === 'object' && floorInfo.currency === 'USD') {
+          let floor = parseFloat(floorInfo.floor)
+          if (!isNaN(floor) && floor > 0) {
+            adSlot.floor = parseFloat(floorInfo.floor);
+          }
         }
       }
 
-      let adSlot = {
-        bidId: bid.bidId,
-        zoneId: bid.params.zoneid || '',
-        floor: floor || 0.0
-      };
-
       if (bid.mediaTypes.banner) {
         adSlot.banner = bid.mediaTypes.banner;
       }

test/spec/modules/kubientBidAdapter_spec.js

@@ -19,7 +19,7 @@ describe('KubientAdapter', function () {
     },
     getFloor: function(params) {
       return {
-        floor: 0.05,
+        floor: 0,
         currency: 'USD'
       };
     },
@@ -127,10 +127,9 @@ describe('KubientAdapter', function () {
         expect(data.uspConsent).to.exist.and.to.equal(uspConsentData);
         for (let j = 0; j < data['adSlots'].length; j++) {
           let adSlot = data['adSlots'][i];
-          expect(adSlot).to.have.all.keys('bidId', 'zoneId', 'floor', 'banner', 'schain');
+          expect(adSlot).to.have.all.keys('bidId', 'zoneId', 'banner', 'schain');
           expect(adSlot.bidId).to.be.a('string').and.to.equal(bidBanner.bidId);
           expect(adSlot.zoneId).to.be.a('string').and.to.equal(bidBanner.params.zoneid);
-          expect(adSlot.floor).to.be.a('number');
           expect(adSlot.schain).to.be.an('object');
           expect(adSlot.banner).to.be.an('object');
         }
@@ -202,10 +201,9 @@ describe('KubientAdapter', function () {
         expect(data.uspConsent).to.exist.and.to.equal(uspConsentData);
         for (let j = 0; j < data['adSlots'].length; j++) {
           let adSlot = data['adSlots'][i];
-          expect(adSlot).to.have.all.keys('bidId', 'zoneId', 'floor', 'banner', 'schain');
+          expect(adSlot).to.have.all.keys('bidId', 'zoneId', 'banner', 'schain');
           expect(adSlot.bidId).to.be.a('string').and.to.equal(bidBanner.bidId);
           expect(adSlot.zoneId).to.be.a('string').and.to.equal(bidBanner.params.zoneid);
-          expect(adSlot.floor).to.be.a('number');
           expect(adSlot.schain).to.be.an('object');
           expect(adSlot.banner).to.be.an('object');
         }