Skip to content

Rename rubicon leftovers to magnite #4580

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 2 commits into from
Jul 30, 2026
Merged

Rename rubicon leftovers to magnite #4580

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
fd5adb2
Rename `rubicon` to `magnite`
CTMBNara Jul 29, 2026
1cf4d49
Resolve conversation
CTMBNara Jul 30, 2026
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
6 changes: 3 additions & 3 deletions ...server/hooks/modules/greenbids/real/time/data/core/GreenbidsInferenceDataServiceTest.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -90,7 +90,7 @@ public void extractThrottlingMessagesFromBidRequestShouldReturnValidThrottlingMe
assertThat(throttlingMessages).isNotEmpty();
assertThat(throttlingMessages)
.extracting(ThrottlingMessage::getBidder)
.containsExactly("rubicon", "appnexus", "pubmatic");
.containsExactly("magnite", "appnexus", "pubmatic");

throttlingMessages.forEach(message -> {
assertThat(message.getAdUnitCode()).isEqualTo("adunitcodevalue");
Expand Down Expand Up @@ -128,7 +128,7 @@ public void extractThrottlingMessagesFromBidRequestShouldReturnValidThrottlingMe
assertThat(throttlingMessages).isNotEmpty();
assertThat(throttlingMessages)
.extracting(ThrottlingMessage::getBidder)
.containsExactly("rubicon", "appnexus", "pubmatic");
.containsExactly("magnite", "appnexus", "pubmatic");

throttlingMessages.forEach(message -> {
assertThat(message.getAdUnitCode()).isEqualTo("adunitcodevalue");
Expand Down Expand Up @@ -162,7 +162,7 @@ public void extractThrottlingMessagesFromBidRequestShouldHandleMissingIp() {
assertThat(throttlingMessages).isNotEmpty();
assertThat(throttlingMessages)
.extracting(ThrottlingMessage::getBidder)
.containsExactly("rubicon", "appnexus", "pubmatic");
.containsExactly("magnite", "appnexus", "pubmatic");

throttlingMessages.forEach(message -> {
assertThat(message.getAdUnitCode()).isEqualTo("adunitcodevalue");
Expand Down
10 changes: 5 additions & 5 deletions ...ver/hooks/modules/greenbids/real/time/data/core/GreenbidsInvocationResultCreatorTest.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ public void createGreenbidsInvocationResultWhenNotExploration() {
assertThat(ortb2ImpExtResult).isNotNull();
assertThat(ortb2ImpExtResult.getGreenbids().getIsExploration()).isFalse();
assertThat(ortb2ImpExtResult.getGreenbids().getFingerprint()).isNotNull();
assertThat(keptInAuction.get("rubicon")).isTrue();
assertThat(keptInAuction.get("magnite")).isTrue();
assertThat(keptInAuction.get("appnexus")).isFalse();
assertThat(keptInAuction.get("pubmatic")).isFalse();
}
Expand Down Expand Up @@ -78,7 +78,7 @@ public void createShouldReturnNoActionWhenExploration() {
assertThat(ortb2ImpExtResult).isNotNull();
assertThat(ortb2ImpExtResult.getGreenbids().getIsExploration()).isTrue();
assertThat(ortb2ImpExtResult.getGreenbids().getFingerprint()).isNotNull();
assertThat(keptInAuction.get("rubicon")).isFalse();
assertThat(keptInAuction.get("magnite")).isFalse();
assertThat(keptInAuction.get("appnexus")).isFalse();
assertThat(keptInAuction.get("pubmatic")).isFalse();
}
Expand Down Expand Up @@ -107,14 +107,14 @@ public void createShouldReturnRejectWhenAllImpsAreFilteredOutAndNoExploration()
assertThat(ortb2ImpExtResult).isNotNull();
assertThat(ortb2ImpExtResult.getGreenbids().getIsExploration()).isFalse();
assertThat(ortb2ImpExtResult.getGreenbids().getFingerprint()).isNotNull();
assertThat(keptInAuction.get("rubicon")).isFalse();
assertThat(keptInAuction.get("magnite")).isFalse();
assertThat(keptInAuction.get("appnexus")).isFalse();
assertThat(keptInAuction.get("pubmatic")).isFalse();
}

private Map<String, Map<String, Boolean>> givenImpsBiddersFilterMap() {
final Map<String, Boolean> biddersFitlerMap = Map.of(
"rubicon", true,
"magnite", true,
"appnexus", false,
"pubmatic", false);

Expand All @@ -123,7 +123,7 @@ private Map<String, Map<String, Boolean>> givenImpsBiddersFilterMap() {

private Map<String, Map<String, Boolean>> givenFilterMapWithAllFilteredImps() {
final Map<String, Boolean> biddersFitlerMap = Map.of(
"rubicon", false,
"magnite", false,
"appnexus", false,
"pubmatic", false);

Expand Down
12 changes: 6 additions & 6 deletions ...rebid/server/hooks/modules/greenbids/real/time/data/core/GreenbidsPayloadUpdaterTest.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@
import static org.assertj.core.api.Assertions.assertThat;
import static org.prebid.server.hooks.modules.greenbids.real.time.data.util.TestBidRequestProvider.getAppnexusNode;
import static org.prebid.server.hooks.modules.greenbids.real.time.data.util.TestBidRequestProvider.getPubmaticNode;
import static org.prebid.server.hooks.modules.greenbids.real.time.data.util.TestBidRequestProvider.getRubiconNode;
import static org.prebid.server.hooks.modules.greenbids.real.time.data.util.TestBidRequestProvider.getMagniteNode;
import static org.prebid.server.hooks.modules.greenbids.real.time.data.util.TestBidRequestProvider.givenBidRequest;
import static org.prebid.server.hooks.modules.greenbids.real.time.data.util.TestBidRequestProvider.givenImpExt;

Expand All @@ -22,18 +22,18 @@ public void updateShouldReturnUpdatedBidRequest() {
// given
final Imp givenImp = Imp.builder()
.id("adunitcodevalue")
.ext(givenImpExt(getRubiconNode(), getAppnexusNode(), getPubmaticNode()))
.ext(givenImpExt(getMagniteNode(), getAppnexusNode(), getPubmaticNode()))
.build();

// when
final BidRequest result = GreenbidsPayloadUpdater.update(
givenBidRequest(identity(), List.of(givenImp)),
Map.of("adunitcodevalue", Map.of("rubicon", true, "appnexus", false, "pubmatic", false)));
Map.of("adunitcodevalue", Map.of("magnite", true, "appnexus", false, "pubmatic", false)));

// then
final Imp expectedImp = Imp.builder()
.id("adunitcodevalue")
.ext(givenImpExt(getRubiconNode(), null, null))
.ext(givenImpExt(getMagniteNode(), null, null))
.build();

assertThat(result.getImp()).containsOnly(expectedImp);
Expand All @@ -44,13 +44,13 @@ public void updateShouldRemoveImpFromUpdateBidRequestWhenAllBiddersFiltered() {
// given
final Imp givenImp = Imp.builder()
.id("adunitcodevalue")
.ext(givenImpExt(getRubiconNode(), null, null))
.ext(givenImpExt(getMagniteNode(), null, null))
.build();

// when
final BidRequest result = GreenbidsPayloadUpdater.update(
givenBidRequest(identity(), List.of(givenImp)),
Map.of("adunitcodevalue", Map.of("rubicon", false, "appnexus", false, "pubmatic", false)));
Map.of("adunitcodevalue", Map.of("magnite", false, "appnexus", false, "pubmatic", false)));

// then
assertThat(result.getImp()).isEmpty();
Expand Down
2 changes: 1 addition & 1 deletion ...va/org/prebid/server/hooks/modules/greenbids/real/time/data/core/OnnxModelRunnerTest.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ public void setUp() throws OrtException, IOException {
public void runModelShouldReturnProbabilitiesWhenValidThrottlingInferenceRow() throws OrtException {
// given
final String[][] throttlingInferenceRow = {{
"Chrome 59", "rubicon", "adunitcodevalue", "US", "www.leparisien.fr", "PC", "10", "1"}};
"Chrome 59", "magnite", "adunitcodevalue", "US", "www.leparisien.fr", "PC", "10", "1"}};

// when
final OrtSession.Result actualResult = target.runModel(throttlingInferenceRow);
Expand Down
10 changes: 5 additions & 5 deletions ...org/prebid/server/hooks/modules/greenbids/real/time/data/util/TestBidRequestProvider.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -37,14 +37,14 @@ public static Site givenSite() {
}

public static ObjectNode givenImpExt() {
return givenImpExt(getRubiconNode(), getAppnexusNode(), getPubmaticNode());
return givenImpExt(getMagniteNode(), getAppnexusNode(), getPubmaticNode());
}

public static ObjectNode givenImpExt(ObjectNode rubiconNode, ObjectNode appnexusNode, ObjectNode pubmaticNode) {
public static ObjectNode givenImpExt(ObjectNode magniteNode, ObjectNode appnexusNode, ObjectNode pubmaticNode) {
final ObjectNode bidderNode = MAPPER.createObjectNode();

if (rubiconNode != null) {
bidderNode.set("rubicon", rubiconNode);
if (magniteNode != null) {
bidderNode.set("magnite", magniteNode);
}

if (appnexusNode != null) {
Expand All @@ -70,7 +70,7 @@ public static ObjectNode getAppnexusNode() {
return MAPPER.createObjectNode().put("placementId", 123456);
}

public static ObjectNode getRubiconNode() {
public static ObjectNode getMagniteNode() {
return MAPPER.createObjectNode()
.put("accountId", 1001)
.put("siteId", 267318)
Expand Down
14 changes: 7 additions & 7 deletions ...les/greenbids/real/time/data/v1/GreenbidsRealTimeDataProcessedAuctionRequestHookTest.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@
import static org.mockito.Mockito.mock;
import static org.prebid.server.auction.model.BidRejectionReason.REQUEST_BLOCKED_OPTIMIZED;
import static org.prebid.server.hooks.modules.greenbids.real.time.data.util.TestBidRequestProvider.MAPPER;
import static org.prebid.server.hooks.modules.greenbids.real.time.data.util.TestBidRequestProvider.getRubiconNode;
import static org.prebid.server.hooks.modules.greenbids.real.time.data.util.TestBidRequestProvider.getMagniteNode;
import static org.prebid.server.hooks.modules.greenbids.real.time.data.util.TestBidRequestProvider.givenBidRequest;
import static org.prebid.server.hooks.modules.greenbids.real.time.data.util.TestBidRequestProvider.givenImpExt;

Expand Down Expand Up @@ -84,7 +84,7 @@ public void callShouldReturnAnalyticTagsWithoutFilteringOutBiddersWhenExploratio
final AuctionInvocationContext invocationContext = givenAuctionInvocationContext(explorationRate);

given(filterService.filterBidders(any(), any(), any())).willReturn(Map.of("adunitcodevalue",
Map.of("rubicon", false, "appnexus", false, "pubmatic", false)));
Map.of("magnite", false, "appnexus", false, "pubmatic", false)));

// when
final Future<InvocationResult<AuctionRequestPayload>> future = target
Expand All @@ -100,11 +100,11 @@ public void callShouldReturnAnalyticTagsWithoutFilteringOutBiddersWhenExploratio
final ResultImpl actualResult = (ResultImpl) actualActivity.results().getFirst();
final AppliedTo acctualAppliedTo = actualResult.appliedTo();

assertThat(acctualAppliedTo.bidders()).containsOnly("appnexus", "pubmatic", "rubicon");
assertThat(acctualAppliedTo.bidders()).containsOnly("appnexus", "pubmatic", "magnite");
assertThat(acctualAppliedTo.impIds()).containsOnly("adunitcodevalue");
assertThat(actualResult.values().get("adunitcodevalue").get("greenbids").get("keptInAuction"))
.isEqualTo(MAPPER.createObjectNode()
.put("rubicon", false)
.put("magnite", false)
.put("appnexus", false)
.put("pubmatic", false));
assertThat(actualResult.values().get("adunitcodevalue").get("greenbids").get("fingerprint").asText())
Expand All @@ -127,7 +127,7 @@ public void callShouldFilterBiddersBasedOnModelResultsWhenExplorationIsFalse() {
final AuctionInvocationContext invocationContext = givenAuctionInvocationContext(explorationRate);

given(filterService.filterBidders(any(), any(), any())).willReturn(Map.of("adunitcodevalue",
Map.of("rubicon", true, "appnexus", false, "pubmatic", false)));
Map.of("magnite", true, "appnexus", false, "pubmatic", false)));

// when
final Future<InvocationResult<AuctionRequestPayload>> future = target
Expand All @@ -145,7 +145,7 @@ public void callShouldFilterBiddersBasedOnModelResultsWhenExplorationIsFalse() {

final Imp expectedImp = Imp.builder()
.id("adunitcodevalue")
.ext(givenImpExt(getRubiconNode(), null, null))
.ext(givenImpExt(getMagniteNode(), null, null))
.build();
assertThat(resultBidRequest).isEqualTo(givenBidRequest(identity(), List.of(expectedImp)));

Expand All @@ -157,7 +157,7 @@ public void callShouldFilterBiddersBasedOnModelResultsWhenExplorationIsFalse() {
assertThat(acctualAppliedTo.impIds()).containsOnly("adunitcodevalue");
assertThat(actualResult.values().get("adunitcodevalue").get("greenbids").get("keptInAuction"))
.isEqualTo(MAPPER.createObjectNode()
.put("rubicon", true)
.put("magnite", true)
.put("appnexus", false)
.put("pubmatic", false));
assertThat(actualResult.values().get("adunitcodevalue").get("greenbids").get("fingerprint").asText())
Expand Down
10 changes: 5 additions & 5 deletions src/test/groovy/org/prebid/server/functional/tests/BidderFieldDisplayBehaviorSpec.groovy
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@ import static org.prebid.server.functional.model.Currency.USD
import static org.prebid.server.functional.model.bidder.BidderName.ALIAS
import static org.prebid.server.functional.model.bidder.BidderName.GENERIC
import static org.prebid.server.functional.model.bidder.BidderName.OPENX
import static org.prebid.server.functional.model.bidder.BidderName.RUBICON
import static org.prebid.server.functional.model.bidder.BidderName.MAGNITE
import static org.prebid.server.functional.model.mock.services.currencyconversion.CurrencyConversionRatesResponse.defaultConversionRates
import static org.prebid.server.functional.model.request.auction.AdjustmentType.MULTIPLIER
import static org.prebid.server.functional.model.request.auction.BidAdjustmentMediaType.BANNER
Expand Down Expand Up @@ -342,7 +342,7 @@ class BidderFieldDisplayBehaviorSpec extends BaseSpec {
ext.prebid.alternateBidderCodes = new AlternateBidderCodes().tap {
it.enabled = true
it.bidders = [(GENERIC): new CodesBidderConfig(enabled: true, allowedBidderCodes: [GENERIC]),
(RUBICON): new CodesBidderConfig(enabled: true, allowedBidderCodes: [RUBICON])]
(MAGNITE): new CodesBidderConfig(enabled: true, allowedBidderCodes: [MAGNITE])]
}
}

Expand Down Expand Up @@ -497,11 +497,11 @@ class BidderFieldDisplayBehaviorSpec extends BaseSpec {
given: "Default basic BidRequest with generic bidder with includeBidderKeys = false"
def bidRequest = BidRequest.defaultBidRequest

and: "Set maxbids = 2 for generic and rubicon bidder"
and: "Set maxbids = 2 for generic and magnite bidder"
def maxBids = 2
def genericMultiBid = new MultiBid(bidder: GENERIC, maxBids: maxBids, targetBidderCodePrefix: PBSUtils.randomString)
def rubiconMultiBid = new MultiBid(bidder: RUBICON, maxBids: maxBids, targetBidderCodePrefix: PBSUtils.randomString)
bidRequest.ext.prebid.multibid = [genericMultiBid, rubiconMultiBid]
def magniteMultiBid = new MultiBid(bidder: MAGNITE, maxBids: maxBids, targetBidderCodePrefix: PBSUtils.randomString)
bidRequest.ext.prebid.multibid = [genericMultiBid, magniteMultiBid]

and: "Default basic bid"
def bidResponse = BidResponse.getDefaultBidResponse(bidRequest)
Expand Down
Loading
Loading