Skip to content

Commit 9ac31e3

Browse files
committed
Fix fallen tests after rebase
1 parent d9db6d0 commit 9ac31e3

File tree

6 files changed

+8
-8
lines changed

6 files changed

+8
-8
lines changed

src/main/java/org/prebid/server/auction/requestfactory/Ortb2RequestFactory.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -192,7 +192,8 @@ public Future<ActivityInfrastructure> activityInfrastructureFrom(AuctionContext
192192
auctionContext.getDebugContext().getTraceLevel()));
193193
}
194194

195-
public Future<BidRequest> validateRequest(Account account, BidRequest bidRequest,
195+
public Future<BidRequest> validateRequest(Account account,
196+
BidRequest bidRequest,
196197
HttpRequestContext httpRequestContext,
197198
DebugContext debugContext,
198199
List<String> warnings) {

src/main/java/org/prebid/server/auction/requestfactory/VideoRequestFactory.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,6 @@
3333
import org.prebid.server.model.HttpRequestContext;
3434
import org.prebid.server.proto.openrtb.ext.request.ExtPublisher;
3535
import org.prebid.server.proto.openrtb.ext.request.ExtPublisherPrebid;
36-
import org.prebid.server.settings.model.Account;
3736
import org.prebid.server.util.HttpUtil;
3837
import org.prebid.server.util.ObjectUtil;
3938

src/test/java/org/prebid/server/auction/requestfactory/AmpRequestFactoryTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1754,7 +1754,7 @@ private void givenBidRequest(
17541754
given(ortb2ImplicitParametersResolver.resolve(any(), any(), any(), anyBoolean())).willAnswer(
17551755
answerWithFirstArgument());
17561756
given(ortb2RequestFactory.validateRequest(any(), any(), any(), any(), any()))
1757-
.willAnswer(invocation -> Future.succeededFuture((BidRequest) invocation.getArgument(0)));
1757+
.willAnswer(invocation -> Future.succeededFuture((BidRequest) invocation.getArgument(1)));
17581758

17591759
given(ortb2RequestFactory.enrichBidRequestWithAccountAndPrivacyData(any()))
17601760
.willAnswer(invocation -> Future.succeededFuture(((AuctionContext) invocation.getArgument(0))

src/test/java/org/prebid/server/auction/requestfactory/AuctionRequestFactoryTest.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -166,7 +166,7 @@ public void setUp() {
166166
.willAnswer(invocation -> Future.succeededFuture(
167167
((AuctionContext) invocation.getArgument(0)).getBidRequest()));
168168
given(ortb2RequestFactory.validateRequest(any(), any(), any(), any(), any()))
169-
.willAnswer(invocationOnMock -> Future.succeededFuture((BidRequest) invocationOnMock.getArgument(0)));
169+
.willAnswer(invocationOnMock -> Future.succeededFuture((BidRequest) invocationOnMock.getArgument(1)));
170170
given(ortb2RequestFactory.removeEmptyEids(any(), any()))
171171
.willAnswer(invocationOnMock -> invocationOnMock.getArgument(0));
172172
given(ortb2RequestFactory.updateTimeout(any())).willAnswer(invocation -> invocation.getArgument(0));
@@ -662,7 +662,7 @@ public void shouldReturnFailedFutureIfRequestValidationFailed() {
662662
// given
663663
givenValidBidRequest();
664664

665-
given(ortb2RequestFactory.validateRequest(any(), any(), any()))
665+
given(ortb2RequestFactory.validateRequest(any(), any(), any(), any(), any()))
666666
.willReturn(Future.failedFuture(new InvalidRequestException("errors")));
667667

668668
// when

src/test/java/org/prebid/server/auction/requestfactory/VideoRequestFactoryTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -405,7 +405,7 @@ private void givenBidRequest(BidRequest bidRequest, List<PodError> podErrors) {
405405
given(ortb2RequestFactory.fetchAccountWithoutStoredRequestLookup(any())).willReturn(Future.succeededFuture());
406406

407407
given(ortb2RequestFactory.validateRequest(any(), any(), any(), any(), any()))
408-
.willAnswer(invocation -> Future.succeededFuture((BidRequest) invocation.getArgument(0)));
408+
.willAnswer(invocation -> Future.succeededFuture((BidRequest) invocation.getArgument(1)));
409409

410410
given(paramsResolver.resolve(any(), any(), any(), anyBoolean()))
411411
.willAnswer(answerWithFirstArgument());

src/test/java/org/prebid/server/validation/RequestValidatorTest.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1195,7 +1195,7 @@ public void validateShouldReturnValidationMessageWhenAliasPointOnNotValidBidderN
11951195
final ValidationResult result = target.validate(Account.empty(ACCOUNT_ID), bidRequest, null, null);
11961196

11971197
// then
1198-
assertThat(result.getErrors()).hasSize(1)
1198+
assertThat(result.getWarnings()).hasSize(1)
11991199
.containsOnly("request.ext.prebid.aliases.alias refers to unknown bidder: fake");
12001200
}
12011201

@@ -1213,7 +1213,7 @@ public void validateShouldReturnValidationMessageWhenAliasPointOnDisabledBidder(
12131213
final ValidationResult result = target.validate(Account.empty(ACCOUNT_ID), bidRequest, null, null);
12141214

12151215
// then
1216-
assertThat(result.getErrors()).hasSize(1)
1216+
assertThat(result.getWarnings()).hasSize(1)
12171217
.containsOnly("request.ext.prebid.aliases.alias refers to disabled bidder: appnexus");
12181218
}
12191219

0 commit comments

Comments
 (0)