Skip to content

Commit 4cc7cef

Browse files
authored
Core: Stop removing AuctionRequest.video.plcmt during OpenRTB downgrade (prebid#3092)
* Stop removing AuctionRequest.video.plcmt during OpenRTB downgrade * Add functional tests * Fix remark
1 parent e8b2400 commit 4cc7cef

File tree

3 files changed

+10
-9
lines changed

3 files changed

+10
-9
lines changed

src/main/java/org/prebid/server/auction/versionconverter/down/BidRequestOrtb26To25Converter.java

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -155,8 +155,7 @@ private static Video modifyVideo(Video video) {
155155
video.getPodseq(),
156156
video.getRqddurs(),
157157
video.getSlotinpod(),
158-
video.getMincpmpersec(),
159-
video.getPlcmt())
158+
video.getMincpmpersec())
160159

161160
? video.toBuilder()
162161
.maxseq(null)
@@ -166,7 +165,6 @@ private static Video modifyVideo(Video video) {
166165
.rqddurs(null)
167166
.slotinpod(null)
168167
.mincpmpersec(null)
169-
.plcmt(null)
170168
.build()
171169

172170
: null;

src/test/groovy/org/prebid/server/functional/tests/OrtbConverterSpec.groovy

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -586,8 +586,9 @@ class OrtbConverterSpec extends BaseSpec {
586586
}
587587
}
588588

589-
def "PBS should remove imp[0].video.* when we don't support ortb 2.6"() {
589+
def "PBS should remove imp[0].video.* and keep imp[0].video.plcmt when we don't support ortb 2.6"() {
590590
given: "Default bid request with imp[0].video.*"
591+
def placement = PBSUtils.getRandomEnum(VideoPlcmtSubtype)
591592
def bidRequest = BidRequest.defaultBidRequest.tap {
592593
imp[0].video = Video.defaultVideo.tap {
593594
rqddurs = [PBSUtils.randomNumber]
@@ -597,24 +598,27 @@ class OrtbConverterSpec extends BaseSpec {
597598
podseq = PBSUtils.randomNumber
598599
mincpmpersec = PBSUtils.randomDecimal
599600
slotinpod = PBSUtils.randomNumber
600-
plcmt = PBSUtils.getRandomEnum(VideoPlcmtSubtype)
601+
plcmt = placement
601602
}
602603
}
603604

604605
when: "Requesting PBS auction with ortb 2.5"
605606
prebidServerServiceWithElderOrtb.sendAuctionRequest(bidRequest)
606607

607-
then: "BidResponse shouldn't contain the imp[0].video.* as on request"
608-
verifyAll(bidder.getBidderRequest(bidRequest.id)) {
608+
then: "Bidder request shouldn't contain the imp[0].video.* as on request"
609+
def bidderRequest = bidder.getBidderRequest(bidRequest.id)
610+
verifyAll(bidderRequest) {
609611
!imp[0].video.rqddurs
610612
!imp[0].video.maxseq
611613
!imp[0].video.poddur
612614
!imp[0].video.podid
613615
!imp[0].video.podseq
614616
!imp[0].video.mincpmpersec
615617
!imp[0].video.slotinpod
616-
!imp[0].video.plcmt
617618
}
619+
620+
and: "Bidder request should contain the imp[0].video.* as on request"
621+
bidderRequest.imp[0].video.plcmt == placement
618622
}
619623

620624
def "PBS shouldn't remove imp[0].video.* when we support ortb 2.6"() {

src/test/java/org/prebid/server/auction/versionconverter/down/BidRequestOrtb26To25ConverterTest.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -209,7 +209,6 @@ public void convertShouldRemoveFieldsThatAreNotInOrtb25() {
209209
.rqddurs(singletonList(1))
210210
.slotinpod(1)
211211
.mincpmpersec(BigDecimal.ONE)
212-
.plcmt(1)
213212
.build())
214213
.audio(Audio.builder()
215214
.poddur(1)

0 commit comments

Comments
 (0)