OpenX: Populate BidderBid.videoInfo for targeting (#3364)

prebid · Aug 9, 2024 · b93c5ce · b93c5ce
1 parent 075fd90
commit b93c5ce
Show file tree

Hide file tree

Showing 2 changed files with 55 additions and 1 deletion.
diff --git a/src/main/java/org/prebid/server/bidder/openx/OpenxBidder.java b/src/main/java/org/prebid/server/bidder/openx/OpenxBidder.java
@@ -29,6 +29,7 @@
 import org.prebid.server.proto.openrtb.ext.request.ExtRequest;
 import org.prebid.server.proto.openrtb.ext.request.openx.ExtImpOpenx;
 import org.prebid.server.proto.openrtb.ext.response.BidType;
+import org.prebid.server.proto.openrtb.ext.response.ExtBidPrebidVideo;
 import org.prebid.server.proto.openrtb.ext.response.FledgeAuctionConfig;
 import org.prebid.server.util.BidderUtil;
 import org.prebid.server.util.HttpUtil;
@@ -253,10 +254,26 @@ private static List<BidderBid> bidsFromResponse(BidRequest bidRequest, OpenxBidR
  .map(SeatBid::getBid)
  .filter(Objects::nonNull)
  .flatMap(Collection::stream)
- .map(bid -> BidderBid.of(bid, getBidType(bid, impIdToBidType), bidCurrency))
+ .map(bid -> toBidderBid(bid, impIdToBidType, bidCurrency))
  .toList();
  }
 
+ private static BidderBid toBidderBid(Bid bid, Map<String, BidType> impIdToBidType, String bidCurrency) {
+ final BidType bidType = getBidType(bid, impIdToBidType);
+ final ExtBidPrebidVideo videoInfo = bidType == BidType.video ? getVideoInfo(bid) : null;
+ return BidderBid.builder()
+ .bid(bid)
+ .type(bidType)
+ .bidCurrency(bidCurrency)
+ .videoInfo(videoInfo)
+ .build();
+ }
+
+ private static ExtBidPrebidVideo getVideoInfo(Bid bid) {
+ final String primaryCategory = CollectionUtils.isEmpty(bid.getCat()) ? null : bid.getCat().getFirst();
+ return ExtBidPrebidVideo.of(bid.getDur(), primaryCategory);
+ }
+
  private static Map<String, BidType> impIdToBidType(BidRequest bidRequest) {
  return bidRequest.getImp().stream()
  .collect(Collectors.toMap(Imp::getId, imp -> imp.getBanner() != null ? BidType.banner : BidType.video));

diff --git a/src/test/java/org/prebid/server/bidder/openx/OpenxBidderTest.java b/src/test/java/org/prebid/server/bidder/openx/OpenxBidderTest.java
@@ -34,6 +34,7 @@
 import org.prebid.server.proto.openrtb.ext.request.ExtUser;
 import org.prebid.server.proto.openrtb.ext.request.openx.ExtImpOpenx;
 import org.prebid.server.proto.openrtb.ext.response.BidType;
+import org.prebid.server.proto.openrtb.ext.response.ExtBidPrebidVideo;
 import org.prebid.server.proto.openrtb.ext.response.FledgeAuctionConfig;
 
 import java.math.BigDecimal;
@@ -522,6 +523,42 @@ public void makeBidsShouldReturnResultWithExpectedFields() throws JsonProcessing
  .build());
  }
 
+ @Test
+ public void makeBidsShouldReturnVideoInfoWhenAvailable() throws JsonProcessingException {
+ // given
+ final BidderCall<BidRequest> httpCall = givenHttpCall(mapper.writeValueAsString(BidResponse.builder()
+ .seatbid(singletonList(SeatBid.builder()
+ .bid(singletonList(Bid.builder()
+ .w(200)
+ .h(150)
+ .price(BigDecimal.ONE)
+ .impid("impId1")
+ .dealid("dealid")
+ .adm("<div>This is an Ad</div>")
+ .dur(30)
+ .cat(singletonList("category1"))
+ .build()))
+ .build()))
+ .build()));
+
+ final BidRequest bidRequest = BidRequest.builder()
+ .id("bidRequestId")
+ .imp(singletonList(Imp.builder()
+ .id("impId1")
+ .video(Video.builder().build())
+ .build()))
+ .build();
+
+ // when
+ final CompositeBidderResponse result = target.makeBidderResponse(httpCall, bidRequest);
+
+ // then
+ assertThat(result.getErrors()).isEmpty();
+ assertThat(result.getBids()).hasSize(1)
+ .extracting(BidderBid::getVideoInfo)
+ .containsExactly(ExtBidPrebidVideo.of(30, "category1"));
+ }
+
  @Test
  public void makeBidsShouldReturnFledgeConfigEvenIfNoBids() throws JsonProcessingException {
  // given