Kargo: Normalize placementId naming (#2785)

adapters/kargo/params_test.go

@@ -34,14 +34,20 @@ func TestInvalidParams(t *testing.T) {
 }
 
 var validParams = []string{
+	`{"placementId": ""}`,
+	`{"placementId": "11523"}`,
 	`{"adSlotID": ""}`,
 	`{"adSlotID": "11523"}`,
 }
 
 var invalidParams = []string{
+	`{"placementId": 42}`,
+	`{"placementId": }`,
+	`{"placementID": "32321"}`,
 	`{"adSlotID": 42}`,
 	`{"adSlotID": }`,
 	`{"adSlotId": "32321"}`,
 	`{"id": }`,
 	`{}`,
+	`{"placementId": "11523", "adSlotID": "12345"}`, // Can't include both
 }

openrtb_ext/imp_kargo.go

@@ -1,5 +1,6 @@
 package openrtb_ext
 
 type ImpExtKargo struct {
-	AdSlotID string `json:"adSlotID"`
+	PlacementId string `json:"placementId"`
+	AdSlotID    string `json:"adSlotID"` // Deprecated - Use `placementId`
 }

static/bidder-params/kargo.json

@@ -3,13 +3,18 @@
     "title": "Kargo Adapter Params",
     "description": "A schema which validates params accepted by the Kargo adapter",
     "type": "object",
-
     "properties": {
-        "adSlotID": {
-          "type": "string",
-          "description": "An ID which identifies the adslot placement. Equivalent to the id of target inventory, ad unit code, or placement id"
-        }
+      "placementId": {
+        "type": "string",
+        "description": "An ID which identifies the adslot placement. Equivalent to the id of target inventory, ad unit code, or placement id"
       },
-
-    "required": ["adSlotID"]
+      "adSlotID": {
+        "type": "string",
+        "description": "[Deprecated: Use `placementId`] An ID which identifies the adslot placement. Equivalent to the id of target inventory, ad unit code, or placement id"
+      }
+    },
+    "oneOf": [
+      { "required": ["placementId"] },
+      { "required": ["adSlotID"] }
+    ]
   }