From 7a5b07089ea0ce5238ecbc36eff7ee208887b12a Mon Sep 17 00:00:00 2001
From: Kevin Siow <kevin.siow@dailymotion.com>
Date: Fri, 19 Apr 2024 15:15:00 +0200
Subject: [PATCH] Dailymotion Bid Adaptor: only support video adunits in
 context instream

---
 modules/dailymotionBidAdapter.js              | 19 ++++++-
 modules/dailymotionBidAdapter.md              | 23 ++++++--
 .../modules/dailymotionBidAdapter_spec.js     | 55 ++++++++++++++++++-
 3 files changed, 90 insertions(+), 7 deletions(-)

diff --git a/modules/dailymotionBidAdapter.js b/modules/dailymotionBidAdapter.js
index 4196910111a..4985297b287 100644
--- a/modules/dailymotionBidAdapter.js
+++ b/modules/dailymotionBidAdapter.js
@@ -76,7 +76,24 @@ export const spec = {
    * @return boolean True if this is a valid bid, and false otherwise.
    */
   isBidRequestValid: function (bid) {
-    return typeof bid?.params?.apiKey === 'string' && bid.params.apiKey.length > 10;
+    if (bid?.params) {
+      // We only accept video adUnits
+      if (!bid?.mediaTypes?.[VIDEO]) return false;
+
+      // As `context`, `placement` & `plcmt` are optional (although recommended)
+      // values, we check the 3 of them to see if we are in an instream video context
+      const isInstream = bid.mediaTypes[VIDEO].context === 'instream' ||
+        bid.mediaTypes[VIDEO].placement === 1 ||
+        bid.mediaTypes[VIDEO].plcmt === 1;
+
+      // We only accept instream video context
+      if (!isInstream) return false;
+
+      // We need API key
+      return typeof bid.params.apiKey === 'string' && bid.params.apiKey.length > 10;
+    }
+
+    return false;
   },
 
   /**
diff --git a/modules/dailymotionBidAdapter.md b/modules/dailymotionBidAdapter.md
index c145817a118..6162d020c0c 100644
--- a/modules/dailymotionBidAdapter.md
+++ b/modules/dailymotionBidAdapter.md
@@ -9,10 +9,11 @@ Maintainer: ad-leo-engineering@dailymotion.com
 # Description
 
 Dailymotion prebid adapter.
+Supports video ad units in instream context.
 
 # Configuration options
 
-Before calling this adapter, you need to set at least the API key in the bid parameters:
+Before calling this adapter, you need to at least set a video adUnit in an instream context and the API key in the bid parameters:
 
 ```javascript
 const adUnits = [
@@ -21,8 +22,14 @@ const adUnits = [
       bidder: 'dailymotion',
       params: {
         apiKey: 'fake_api_key'
-      }
-    }]
+      },
+    }],
+    code: 'test-ad-unit',
+    mediaTypes: {
+      video: {
+        context: 'instream',
+      },
+    },
   }
 ];
 ```
@@ -40,8 +47,14 @@ const adUnits = [
       bidder: 'dailymotion',
       params: {
         apiKey: 'dailymotion-testing'
-      }
-    }]
+      },
+    }],
+    code: 'test-ad-unit',
+    mediaTypes: {
+      video: {
+        context: 'instream',
+      },
+    },
   }
 ];
 ```
diff --git a/test/spec/modules/dailymotionBidAdapter_spec.js b/test/spec/modules/dailymotionBidAdapter_spec.js
index 1ab93e47099..7d409a0e149 100644
--- a/test/spec/modules/dailymotionBidAdapter_spec.js
+++ b/test/spec/modules/dailymotionBidAdapter_spec.js
@@ -1,7 +1,7 @@
 import { config } from 'src/config.js';
 import { expect } from 'chai';
 import { spec } from 'modules/dailymotionBidAdapter.js';
-import { VIDEO } from '../../../src/mediaTypes';
+import { BANNER, VIDEO } from '../../../src/mediaTypes';
 
 describe('dailymotionBidAdapterTests', () => {
   // Validate that isBidRequestValid only validates requests with apiKey
@@ -12,6 +12,11 @@ describe('dailymotionBidAdapterTests', () => {
       params: {
         apiKey: '',
       },
+      mediaTypes: {
+        [VIDEO]: {
+          context: 'instream',
+        },
+      },
     };
 
     expect(config.runWithBidder('dailymotion', () => spec.isBidRequestValid(bidWithEmptyApi))).to.be.false;
@@ -20,9 +25,57 @@ describe('dailymotionBidAdapterTests', () => {
       params: {
         apiKey: 'test_api_key',
       },
+      mediaTypes: {
+        [VIDEO]: {
+          context: 'instream',
+        },
+      },
     };
 
     expect(config.runWithBidder('dailymotion', () => spec.isBidRequestValid(bidWithApi))).to.be.true;
+
+    const bidWithEmptyMediaTypes = {
+      params: {
+        apiKey: '',
+      },
+    };
+
+    expect(config.runWithBidder('dailymotion', () => spec.isBidRequestValid(bidWithEmptyMediaTypes))).to.be.false;
+
+    const bidWithEmptyVideoAdUnit = {
+      params: {
+        apiKey: '',
+      },
+      mediaTypes: {
+        [VIDEO]: {},
+      },
+    };
+
+    expect(config.runWithBidder('dailymotion', () => spec.isBidRequestValid(bidWithEmptyVideoAdUnit))).to.be.false;
+
+    const bidWithBannerMediaType = {
+      params: {
+        apiKey: 'test_api_key',
+      },
+      mediaTypes: {
+        [BANNER]: {},
+      },
+    };
+
+    expect(config.runWithBidder('dailymotion', () => spec.isBidRequestValid(bidWithBannerMediaType))).to.be.false;
+
+    const bidWithOutstreamContext = {
+      params: {
+        apiKey: 'test_api_key',
+      },
+      mediaTypes: {
+        video: {
+          context: 'outstream',
+        },
+      },
+    };
+
+    expect(config.runWithBidder('dailymotion', () => spec.isBidRequestValid(bidWithOutstreamContext))).to.be.false;
   });
 
   // Validate request generation