Skip to content

GumGum: Collect the ad unit name for reporting #3912

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
CTMBNara merged 9 commits into from
Jul 1, 2025
Merged

GumGum: Collect the ad unit name for reporting #3912

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
6513ab2
initial changes
Mar 12, 2025
7741385
Setting adUnitId to imp.tagId instead of in the imp.ext
Mar 18, 2025
6f3c348
initial test cases
Mar 25, 2025
7d1891d
working on test cases
Apr 4, 2025
e380cea
Pushing test cases and final changes
Apr 15, 2025
c1e2260
changes
Apr 15, 2025
5e84055
added changes as per PR comments
May 12, 2025
48ee12a
changes
Jun 2, 2025
0da7ec3
changes after comments
Jun 10, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
30 changes: 22 additions & 8 deletions src/main/java/org/prebid/server/bidder/gumgum/GumgumBidder.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,7 @@
import org.prebid.server.json.DecodeException;
import org.prebid.server.json.JacksonMapper;
import org.prebid.server.proto.openrtb.ext.ExtPrebid;
import org.prebid.server.proto.openrtb.ext.request.ExtImpPrebid;
import org.prebid.server.proto.openrtb.ext.request.gumgum.ExtImpGumgum;
import org.prebid.server.proto.openrtb.ext.request.gumgum.ExtImpGumgumBanner;
import org.prebid.server.proto.openrtb.ext.request.gumgum.ExtImpGumgumVideo;
Expand All @@ -42,12 +43,13 @@
import java.util.Comparator;
import java.util.List;
import java.util.Objects;
import java.util.Optional;

public class GumgumBidder implements Bidder<BidRequest> {

private static final String REQUEST_EXT_PRODUCT = "product";

private static final TypeReference<ExtPrebid<?, ExtImpGumgum>> GUMGUM_EXT_TYPE_REFERENCE =
private static final TypeReference<ExtPrebid<ExtImpPrebid, ExtImpGumgum>> GUMGUM_EXT_TYPE_REFERENCE =
new TypeReference<>() {
};

Expand Down Expand Up @@ -82,17 +84,24 @@ private BidRequest createBidRequest(BidRequest bidRequest, List<BidderError> err

for (Imp imp : bidRequest.getImp()) {
try {
final ExtImpGumgum extImp = parseImpExt(imp);
modifiedImps.add(modifyImp(imp, extImp));
final ExtPrebid<ExtImpPrebid, ExtImpGumgum> extImp = parseImpExt(imp);
final ExtImpGumgum extImpGumgum = extImp.getBidder();
final String adUnitCode = Optional.ofNullable(extImp.getPrebid())
.map(ExtImpPrebid::getAdUnitCode)
.orElse(null);

final String extZone = extImp.getZone();
modifiedImps.add(modifyImp(imp, extImpGumgum, adUnitCode));

final String extZone = extImpGumgum.getZone();
if (StringUtils.isNotEmpty(extZone)) {
zone = extZone;
}
final BigInteger extPubId = extImp.getPubId();

final BigInteger extPubId = extImpGumgum.getPubId();
if (extPubId != null && !extPubId.equals(BigInteger.ZERO)) {
pubId = extPubId;
}

} catch (PreBidException e) {
errors.add(BidderError.badInput(e.getMessage()));
}
Expand All @@ -108,15 +117,15 @@ private BidRequest createBidRequest(BidRequest bidRequest, List<BidderError> err
.build();
}

private ExtImpGumgum parseImpExt(Imp imp) {
private ExtPrebid<ExtImpPrebid, ExtImpGumgum> parseImpExt(Imp imp) {
try {
return mapper.mapper().convertValue(imp.getExt(), GUMGUM_EXT_TYPE_REFERENCE).getBidder();
return mapper.mapper().convertValue(imp.getExt(), GUMGUM_EXT_TYPE_REFERENCE);
} catch (IllegalArgumentException e) {
throw new PreBidException(e.getMessage());
}
}

private Imp modifyImp(Imp imp, ExtImpGumgum extImp) {
private Imp modifyImp(Imp imp, ExtImpGumgum extImp, String adUnitCode) {
final Imp.ImpBuilder impBuilder = imp.toBuilder();

final String product = extImp.getProduct();
Expand All @@ -125,6 +134,10 @@ private Imp modifyImp(Imp imp, ExtImpGumgum extImp) {
impBuilder.ext(productExt);
}

if (StringUtils.isNotEmpty(adUnitCode)) {
impBuilder.tagid(adUnitCode);
}

final Banner banner = imp.getBanner();
if (banner != null) {
final Banner resolvedBanner = resolveBanner(banner, extImp);
Expand All @@ -141,6 +154,7 @@ private Imp modifyImp(Imp imp, ExtImpGumgum extImp) {
impBuilder.video(resolvedVideo);
}
}

return impBuilder.build();
}

Expand Down
126 changes: 126 additions & 0 deletions src/test/java/org/prebid/server/bidder/gumgum/GumgumBidderTest.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -21,12 +21,15 @@
import org.prebid.server.bidder.model.HttpResponse;
import org.prebid.server.bidder.model.Result;
import org.prebid.server.proto.openrtb.ext.ExtPrebid;
import org.prebid.server.proto.openrtb.ext.request.ExtImpPrebid;
import org.prebid.server.proto.openrtb.ext.request.gumgum.ExtImpGumgum;
import org.prebid.server.proto.openrtb.ext.request.gumgum.ExtImpGumgumBanner;
import org.prebid.server.proto.openrtb.ext.request.gumgum.ExtImpGumgumVideo;

import java.io.IOException;
import java.math.BigDecimal;
import java.math.BigInteger;
import java.util.Collections;
import java.util.List;
import java.util.function.Function;

Expand All @@ -36,6 +39,10 @@
import static org.assertj.core.api.Assertions.assertThat;
import static org.assertj.core.api.Assertions.assertThatIllegalArgumentException;
import static org.assertj.core.api.Assertions.tuple;
import static org.junit.Assert.assertEquals;
import static org.junit.Assert.assertFalse;
import static org.junit.Assert.assertNotNull;
import static org.junit.Assert.assertNull;
import static org.prebid.server.proto.openrtb.ext.response.BidType.banner;
import static org.prebid.server.proto.openrtb.ext.response.BidType.video;

Expand All @@ -45,6 +52,124 @@ public class GumgumBidderTest extends VertxTest {

private final GumgumBidder target = new GumgumBidder(ENDPOINT_URL, jacksonMapper);

@Test
public void makeHttpRequestsShouldReturnErrorIfImpExtCouldNotBeParsed() {
// given
final BidRequest bidRequest = givenBidRequest(impBuilder ->
impBuilder.ext(mapper.valueToTree(ExtPrebid.of(null, mapper.createArrayNode()))));

// when
final Result<List<HttpRequest<BidRequest>>> result = target.makeHttpRequests(bidRequest);

// then
assertThat(result.getErrors()).hasSize(2)
.anySatisfy(error -> {
assertThat(error.getType()).isEqualTo(BidderError.Type.bad_input);
assertThat(error.getMessage()).startsWith("Cannot deserialize value");
});
}

@Test
public void makeHttpRequestsShouldModifyImpressionsWhenValidInput() {
// given
final ObjectNode extImp = mapper.valueToTree(ExtPrebid.of(
ExtImpPrebid.builder().adUnitCode("adUnit123").build(),
ExtImpGumgum.of("zone", BigInteger.TEN, "irisId", null, "product")));

final BidRequest bidRequest = givenBidRequest(impBuilder ->
impBuilder.ext(extImp));

// when
final Result<List<HttpRequest<BidRequest>>> result = target.makeHttpRequests(bidRequest);

// then
assertThat(result.getValue())
.extracting(HttpRequest::getPayload)
.flatExtracting(BidRequest::getImp)
.extracting(Imp::getTagid)
.containsExactly("adUnit123");
}

@Test
public void testMakeHttpRequestsShouldNotSetTagIdFromZoneWhenAdUnitIdIsMissing() throws IOException {
// given
final ObjectNode extImp = mapper.valueToTree(ExtPrebid.of(null,
ExtImpGumgum.of("zone123", BigInteger.TEN, "productA", null, "zone123")));

final Imp imp = Imp.builder()
.id("imp1")
.banner(Banner.builder().w(300).h(250).build())
.ext(extImp)
.build();

final BidRequest bidRequest = BidRequest.builder()
.id("test-bid-request")
.imp(Collections.singletonList(imp))
.site(Site.builder().id("test-site").build())
.build();

// when
final Result<List<HttpRequest<BidRequest>>> result = target.makeHttpRequests(bidRequest);

// then
assertNotNull(result);
assertFalse(result.getValue().isEmpty());

final byte[] requestBody = result.getValue().get(0).getBody();
final BidRequest modifiedRequest = mapper.readValue(requestBody, BidRequest.class);

assertFalse(modifiedRequest.getImp().isEmpty());

final Imp modifiedImp = modifiedRequest.getImp().get(0);

assertNull(modifiedImp.getTagid());
assertEquals("test-site", modifiedRequest.getSite().getId(), "zone123");
}

@Test
public void makeHttpRequestsShouldReturnModifiedBidRequestWhenValidInput() {
// given
final ObjectNode extImp = mapper.valueToTree(ExtPrebid.of(
ExtImpPrebid.builder().adUnitCode("adUnit123").build(),
ExtImpGumgum.of("zone", BigInteger.TEN, "irisId", null, "product")));

final BidRequest bidRequest = givenBidRequest(impBuilder -> impBuilder.ext(extImp));

// when
final Result<List<HttpRequest<BidRequest>>> result = target.makeHttpRequests(bidRequest);

// then
assertThat(result.getErrors()).isEmpty();

assertThat(result.getValue())
.extracting(HttpRequest::getPayload)
.flatExtracting(BidRequest::getImp)
.extracting(Imp::getTagid)
.containsExactly("adUnit123");
}

@Test
public void makeHttpRequestsShouldExtractAdUnitIdWhenPresent() {
// given
final ObjectNode extImp = mapper.valueToTree(ExtPrebid.of(
ExtImpPrebid.builder().adUnitCode("adUnit123").build(),
ExtImpGumgum.of("zone", BigInteger.TEN, "irisId", null, "product")));

final BidRequest bidRequest = givenBidRequest(impBuilder -> impBuilder.ext(extImp));

// when
final Result<List<HttpRequest<BidRequest>>> result = target.makeHttpRequests(bidRequest);

// then
assertThat(result.getErrors()).isEmpty();

assertThat(result.getValue())
.extracting(HttpRequest::getPayload)
.flatExtracting(BidRequest::getImp)
.extracting(Imp::getTagid)
.containsExactly("adUnit123");
}

@Test
public void creationShouldFailOnInvalidEndpointUrl() {
assertThatIllegalArgumentException().isThrownBy(() -> new GumgumBidder("invalid_url", jacksonMapper));
Expand Down Expand Up @@ -422,4 +547,5 @@ private static BidderCall<BidRequest> givenHttpCall(BidRequest bidRequest, Strin
HttpResponse.of(200, null, body),
null);
}

}
Loading