From f61311769823d0431cd90a05c125d88905c0555e Mon Sep 17 00:00:00 2001
From: ardit-baloku <77985953+ardit-baloku@users.noreply.github.com>
Date: Fri, 29 Jan 2021 11:51:51 +0100
Subject: [PATCH] Malltv Bid Adapter : added data object as a param (#6232)

* Updated malltv adapter

* Updated markdown

* Added test for malltvBidAdapter
---
 modules/malltvBidAdapter.js                |  7 ++-
 modules/malltvBidAdapter.md                | 63 +++++++++++++++-------
 test/spec/modules/malltvBidAdapter_spec.js | 30 +++++++++--
 3 files changed, 74 insertions(+), 26 deletions(-)

diff --git a/modules/malltvBidAdapter.js b/modules/malltvBidAdapter.js
index 776e3f6458b..7deffe6c07a 100644
--- a/modules/malltvBidAdapter.js
+++ b/modules/malltvBidAdapter.js
@@ -31,6 +31,7 @@ export const spec = {
     let bidderRequestId = '';
     let url = '';
     let contents = [];
+    let data = {};
 
     let placements = validBidRequests.map(bidRequest => {
       if (!propertyId) { propertyId = bidRequest.params.propertyId; }
@@ -38,7 +39,8 @@ export const spec = {
       if (!storageId && bidRequest.params) { storageId = bidRequest.params.storageId || ''; }
       if (!bidderRequestId) { bidderRequestId = bidRequest.bidderRequestId; }
       if (!url && bidderRequest) { url = bidderRequest.refererInfo.referer; }
-      if (!contents.length && bidRequest.params.contents && bidRequest.params.contents.length) { contents = bidRequest.params.contents }
+      if (!contents.length && bidRequest.params.contents && bidRequest.params.contents.length) { contents = bidRequest.params.contents; }
+      if (Object.keys(data).length === 0 && bidRequest.params.data && Object.keys(bidRequest.params.data).length !== 0) { data = bidRequest.params.data; }
 
       let adUnitId = bidRequest.adUnitCode;
       let placementId = bidRequest.params.placementId;
@@ -61,7 +63,8 @@ export const spec = {
       url: url,
       requestid: bidderRequestId,
       placements: placements,
-      contents: contents
+      contents: contents,
+      data: data
     }
 
     return [{
diff --git a/modules/malltvBidAdapter.md b/modules/malltvBidAdapter.md
index 3d419fa0916..e32eb54f90f 100644
--- a/modules/malltvBidAdapter.md
+++ b/modules/malltvBidAdapter.md
@@ -1,45 +1,68 @@
 # Overview
-Module Name: MallTV Bidder Adapter Module 
-Type: Bidder Adapter 
-Maintainer: drilon@gjirafa.com
+Module Name: MallTV Bidder Adapter Module
+
+Type: Bidder Adapter
+
+Maintainer: arditb@gjirafa.com
 
 # Description
 MallTV Bidder Adapter for Prebid.js.
 
 # Test Parameters
+```js
 var adUnits = [
     {
         code: 'test-div',
         mediaTypes: {
             banner: {
-                sizes: [[300, 250], [300, 300]]
+                sizes: [
+                    [300, 250],
+                    [300, 300]
+                ]
             }
         },
-        bids: [
-            {
-                 bidder: 'malltv',
-                 params: {
-                    propertyId: '105134',
-                    placementId: '846832'
+        bids: [{
+            bidder: 'malltv',
+            params: {
+                propertyId: '105134',  //Required
+                placementId: '846832', //Required
+                data: {                //Optional
+                    catalogs: [{
+                        catalogId: 9,
+                        items: ["193", "4", "1"]
+                    }],
+                    inventory: {
+                        category: ["tech"],
+                        query: ["iphone 12"]
+                    }
                 }
             }
-        ]
+        }]
     },
     {
         code: 'test-div',
         mediaTypes: {
-             video: {
+            video: {
                 context: 'instream'
-             }
+            }
         },
-        bids: [
-            {
-                 bidder: 'malltv',
-                 params: {
-                    propertyId: '105134',
-                    placementId: '846841'
+        bids: [{
+            bidder: 'malltv',
+            params: {
+                propertyId: '105134',  //Required
+                placementId: '846832', //Required
+                data: {                //Optional
+                    catalogs: [{
+                        catalogId: 9,
+                        items: ["193", "4", "1"]
+                    }],
+                    inventory: {
+                        category: ["tech"],
+                        query: ["iphone 12"]
+                    }
                 }
             }
-        ]
+        }]
     }
 ];
+```
diff --git a/test/spec/modules/malltvBidAdapter_spec.js b/test/spec/modules/malltvBidAdapter_spec.js
index e1e9ad867e7..ffe08ad1a5e 100644
--- a/test/spec/modules/malltvBidAdapter_spec.js
+++ b/test/spec/modules/malltvBidAdapter_spec.js
@@ -34,9 +34,7 @@ describe('malltvAdapterTest', () => {
     it('bidRequest without propertyId or placementId', () => {
       expect(spec.isBidRequestValid({
         bidder: 'malltv',
-        params: {
-          propertyId: '{propertyId}',
-        }
+        params: {}
       })).to.equal(false);
     });
   });
@@ -46,7 +44,17 @@ describe('malltvAdapterTest', () => {
       'bidder': 'malltv',
       'params': {
         'propertyId': '{propertyId}',
-        'placementId': '{placementId}'
+        'placementId': '{placementId}',
+        'data': {
+          'catalogs': [{
+            'catalogId': 1,
+            'items': ['1', '2', '3']
+          }],
+          'inventory': {
+            'category': ['category1', 'category2'],
+            'query': ['query']
+          }
+        }
       },
       'adUnitCode': 'hb-leaderboard',
       'transactionId': 'b6b889bb-776c-48fd-bc7b-d11a1cf0425e',
@@ -86,6 +94,20 @@ describe('malltvAdapterTest', () => {
         expect(requestItem.data.placements[0].sizes).to.equal('300x250');
       });
     });
+
+    it('bidRequest data param', () => {
+      const requests = spec.buildRequests(bidRequests);
+      requests.forEach((requestItem) => {
+        expect(requestItem.data.data).to.exist;
+        expect(requestItem.data.data.catalogs).to.exist;
+        expect(requestItem.data.data.inventory).to.exist;
+        expect(requestItem.data.data.catalogs.length).to.equal(1);
+        expect(requestItem.data.data.catalogs[0].items.length).to.equal(3);
+        expect(Object.keys(requestItem.data.data.inventory).length).to.equal(2);
+        expect(requestItem.data.data.inventory.category.length).to.equal(2);
+        expect(requestItem.data.data.inventory.query.length).to.equal(1);
+      });
+    });
   });
 
   describe('interpretResponse', () => {