Update test dependencies

prebid · Oct 11, 2021 · aece6c0 · aece6c0
1 parent 854f5b0
commit aece6c0
Show file tree

Hide file tree

Showing 44 changed files with 241 additions and 232 deletions.
diff --git a/pom.xml b/pom.xml
@@ -47,14 +47,15 @@
         <maxmind-client.version>2.12.0</maxmind-client.version>
 
         <!-- test dependencies versions -->
-        <junit.version>4.13.1</junit.version>
-        <mockito.version>2.23.4</mockito.version>
-        <assertj.version>3.8.0</assertj.version>
-        <wiremock.version>2.26.3</wiremock.version>
-        <awaitility.version>4.0.1</awaitility.version>
+        <junit.version>4.13.2</junit.version>
+        <mockito.version>3.12.4</mockito.version>
+        <assertj.version>3.21.0</assertj.version>
+        <wiremock.version>2.27.2</wiremock.version>
+        <awaitility.version>4.1.0</awaitility.version>
         <jetty.version>9.4.43.v20210629</jetty.version>
-        <restassured.version>3.0.6</restassured.version>
-        <h2.version>1.4.196</h2.version>
+        <restassured.version>3.3.0</restassured.version>
+        <h2.version>1.4.200</h2.version>
+        <bytebuddy.version>1.11.13</bytebuddy.version>
 
         <!-- plugin versions -->
         <checkstyle-plugin.version>3.1.0</checkstyle-plugin.version>
@@ -400,6 +401,12 @@
             <version>${restassured.version}</version>
             <scope>test</scope>
         </dependency>
+        <dependency>
+            <groupId>net.bytebuddy</groupId>
+            <artifactId>byte-buddy</artifactId>
+            <version>${bytebuddy.version}</version>
+            <scope>test</scope>
+        </dependency>
         <!-- for Mockito to be able to mock Vert.x-specific classes -->
         <dependency>
             <groupId>io.vertx</groupId>

diff --git a/src/test/java/org/prebid/server/analytics/pubstack/PubstackAnalyticsReporterTest.java b/src/test/java/org/prebid/server/analytics/pubstack/PubstackAnalyticsReporterTest.java
@@ -37,7 +37,8 @@
 import static org.mockito.Mockito.mock;
 import static org.mockito.Mockito.times;
 import static org.mockito.Mockito.verify;
-import static org.mockito.Mockito.verifyZeroInteractions;
+import static org.mockito.Mockito.verifyNoInteractions;
+import static org.mockito.Mockito.verifyNoMoreInteractions;
 
 public class PubstackAnalyticsReporterTest extends VertxTest {
 
@@ -118,8 +119,8 @@ public void initializeShouldFailUpdateSendBuffersAndSetTimerWhenEndpointFromRemo
                 .isInstanceOf(PreBidException.class);
         verify(auctionHandler).reportEvents();
         verify(setuidHandler).reportEvents();
-        verifyZeroInteractions(auctionHandler);
-        verifyZeroInteractions(setuidHandler);
+        verifyNoMoreInteractions(auctionHandler);
+        verifyNoMoreInteractions(setuidHandler);
         verify(vertx).setPeriodic(anyLong(), any());
     }
 
@@ -156,8 +157,8 @@ public void initializeShouldNotSendEventsAndUpdateConfigsWhenResponseStatusIsNot
         // then
         verify(vertx).setPeriodic(anyLong(), any());
         verify(httpClient).get(anyString(), anyLong());
-        verifyZeroInteractions(auctionHandler);
-        verifyZeroInteractions(setuidHandler);
+        verifyNoInteractions(auctionHandler);
+        verifyNoInteractions(setuidHandler);
     }
 
     @Test
@@ -172,8 +173,8 @@ public void initializeShouldNotSendEventsAndUpdateConfigsWhenCantParseResponse()
         // then
         verify(vertx).setPeriodic(anyLong(), any());
         verify(httpClient).get(anyString(), anyLong());
-        verifyZeroInteractions(auctionHandler);
-        verifyZeroInteractions(setuidHandler);
+        verifyNoInteractions(auctionHandler);
+        verifyNoInteractions(setuidHandler);
     }
 
     @Test

diff --git a/src/test/java/org/prebid/server/analytics/pubstack/PubstackEventHandlerTest.java b/src/test/java/org/prebid/server/analytics/pubstack/PubstackEventHandlerTest.java
@@ -36,7 +36,7 @@
 import static org.mockito.Mockito.mock;
 import static org.mockito.Mockito.times;
 import static org.mockito.Mockito.verify;
-import static org.mockito.Mockito.verifyZeroInteractions;
+import static org.mockito.Mockito.verifyNoInteractions;
 
 public class PubstackEventHandlerTest extends VertxTest {
 
@@ -88,7 +88,7 @@ public void handleShouldNotAcceptEventsWhenNotEnabled() {
                 (AtomicReference<Set<String>>) ReflectionTestUtils
                         .getField(pubstackEventHandler, "events");
         assertThat(events.get()).isEmpty();
-        verifyZeroInteractions(httpClient);
+        verifyNoInteractions(httpClient);
     }
 
     @Test

diff --git a/src/test/java/org/prebid/server/auction/ExchangeServiceTest.java b/src/test/java/org/prebid/server/auction/ExchangeServiceTest.java
@@ -182,8 +182,8 @@
 import static org.mockito.Mockito.never;
 import static org.mockito.Mockito.times;
 import static org.mockito.Mockito.verify;
+import static org.mockito.Mockito.verifyNoInteractions;
 import static org.mockito.Mockito.verifyNoMoreInteractions;
-import static org.mockito.Mockito.verifyZeroInteractions;
 import static org.prebid.server.assertion.FutureAssertion.assertThat;
 import static org.prebid.server.proto.openrtb.ext.response.BidType.banner;
 import static org.prebid.server.proto.openrtb.ext.response.BidType.video;
@@ -351,8 +351,8 @@ public void shouldTolerateImpWithoutExtension() {
         final BidResponse bidResponse = exchangeService.holdAuction(givenRequestContext(bidRequest)).result();
 
         // then
-        verifyZeroInteractions(bidderCatalog);
-        verifyZeroInteractions(httpBidderRequester);
+        verifyNoInteractions(bidderCatalog);
+        verifyNoInteractions(httpBidderRequester);
         assertThat(bidResponse).isNotNull();
     }
 
@@ -368,7 +368,7 @@ public void shouldTolerateImpWithUnknownBidderInExtension() {
 
         // then
         verify(bidderCatalog).isValidName(eq("invalid"));
-        verifyZeroInteractions(httpBidderRequester);
+        verifyNoInteractions(httpBidderRequester);
         assertThat(bidResponse).isNotNull();
     }
 
@@ -507,7 +507,7 @@ public void shouldSkipBidderWhenRejectedByBidderRequestHooks() {
         exchangeService.holdAuction(givenRequestContext(bidRequest));
 
         // then
-        verifyZeroInteractions(httpBidderRequester);
+        verifyNoInteractions(httpBidderRequester);
     }
 
     @Test
@@ -667,7 +667,7 @@ public void shouldPassRequestWithExtPrebidToDefinedBidder() {
         assertThat(prebid1).isNotNull();
         final JsonNode bidders1 = prebid1.getBidders();
         assertThat(bidders1).isNotNull();
-        assertThat(bidders1.fields()).hasSize(1)
+        assertThat(bidders1.fields()).toIterable().hasSize(1)
                 .containsOnly(entry("bidder", mapper.createObjectNode().put("test1", "test1")));
 
         final ArgumentCaptor<BidderRequest> bidRequest2Captor = ArgumentCaptor.forClass(BidderRequest.class);
@@ -677,7 +677,7 @@ public void shouldPassRequestWithExtPrebidToDefinedBidder() {
         assertThat(prebid2).isNotNull();
         final JsonNode bidders2 = prebid2.getBidders();
         assertThat(bidders2).isNotNull();
-        assertThat(bidders2.fields()).hasSize(1)
+        assertThat(bidders2.fields()).toIterable().hasSize(1)
                 .containsOnly(entry("bidder", mapper.createObjectNode().put("test2", "test2")));
     }
 
@@ -796,7 +796,7 @@ public void shouldNotCreateRequestForBidderRestrictedByPrivacyEnforcement() {
         exchangeService.holdAuction(givenRequestContext(bidRequest));
 
         // then
-        verifyZeroInteractions(httpBidderRequester);
+        verifyNoInteractions(httpBidderRequester);
     }
 
     @Test
@@ -1787,7 +1787,7 @@ public void shouldAddMultiBidInfoAboutRequestedBidderIfDataShouldNotBeSuppressed
         final ExtRequest extRequest = captureBidRequest().getExt();
         assertThat(extRequest)
                 .extracting(ExtRequest::getPrebid)
-                .flatExtracting("multibid")
+                .extracting(ExtRequestPrebid::getMultibid).asList()
                 .containsExactly(ExtRequestPrebidMultiBid.of("someBidder", null, 3, "prefix"));
     }
 
@@ -1807,7 +1807,7 @@ public void shouldAddMultibidInfoOnlyAboutRequestedBidder() {
         final ExtRequest extRequest = captureBidRequest().getExt();
         assertThat(extRequest)
                 .extracting(ExtRequest::getPrebid)
-                .flatExtracting("multibid")
+                .extracting(ExtRequestPrebid::getMultibid).asList()
                 .containsExactly(ExtRequestPrebidMultiBid.of("someBidder", null, 3, null));
     }
 
@@ -2481,10 +2481,10 @@ public void shouldRemoveSiteIfBothSiteAndAppPresent() {
         final BidRequest captureBidRequest = captureBidRequest();
         assertThat(captureBidRequest)
                 .extracting(BidRequest::getSite)
-                .containsNull();
+                .isNull();
         assertThat(captureBidRequest)
                 .extracting(BidRequest::getApp)
-                .doesNotContainNull();
+                .isNotNull();
     }
 
     @Test
@@ -2502,7 +2502,7 @@ public void shouldAddDebugWarningIfBothSiteAndAppPresent() {
         // then
         assertThat(givenContext)
                 .extracting(AuctionContext::getDebugWarnings)
-                .containsExactly(singletonList("BidRequest contains app and site. Removed site object"));
+                .isEqualTo(singletonList("BidRequest contains app and site. Removed site object"));
     }
 
     @Test
@@ -3253,7 +3253,7 @@ public void shouldReturnEmptyBidResponseWhenRequestIsRejected() {
         final Future<BidResponse> result = exchangeService.holdAuction(auctionContext);
 
         // then
-        verifyZeroInteractions(storedResponseProcessor, httpBidderRequester, hookStageExecutor, bidResponseCreator);
+        verifyNoInteractions(storedResponseProcessor, httpBidderRequester, hookStageExecutor, bidResponseCreator);
         assertThat(result).succeededWith(BidResponse.builder()
                 .seatbid(emptyList())
                 .build());

diff --git a/src/test/java/org/prebid/server/auction/PrivacyEnforcementServiceTest.java b/src/test/java/org/prebid/server/auction/PrivacyEnforcementServiceTest.java
@@ -84,7 +84,7 @@
 import static org.mockito.Mockito.mock;
 import static org.mockito.Mockito.verify;
 import static org.mockito.Mockito.verifyNoMoreInteractions;
-import static org.mockito.Mockito.verifyZeroInteractions;
+import static org.mockito.Mockito.verifyNoInteractions;
 
 public class PrivacyEnforcementServiceTest extends VertxTest {
 
@@ -407,7 +407,7 @@ public void shouldMaskForCoppaWhenDeviceLmtIsEnforceAndOneAndRegsCoppaIsOneAndDo
                 .build();
         assertThat(result).isEqualTo(singletonList(expected));
 
-        verifyZeroInteractions(tcfDefinerService);
+        verifyNoInteractions(tcfDefinerService);
     }
 
     @Test

diff --git a/src/test/java/org/prebid/server/auction/StoredRequestProcessorTest.java b/src/test/java/org/prebid/server/auction/StoredRequestProcessorTest.java
@@ -59,7 +59,7 @@
 import static org.mockito.ArgumentMatchers.eq;
 import static org.mockito.BDDMockito.given;
 import static org.mockito.Mockito.verify;
-import static org.mockito.Mockito.verifyZeroInteractions;
+import static org.mockito.Mockito.verifyNoInteractions;
 
 public class StoredRequestProcessorTest extends VertxTest {
 
@@ -543,7 +543,7 @@ public void shouldReturnImpAndBidRequestWithoutChangesIfStoredRequestIsAbsentInP
         final Future<BidRequest> bidRequestFuture = storedRequestProcessor.processStoredRequests(null, bidRequest);
 
         // then
-        verifyZeroInteractions(applicationSettings, metrics);
+        verifyNoInteractions(applicationSettings, metrics);
         assertThat(bidRequestFuture.succeeded()).isTrue();
         assertThat(bidRequestFuture.result().getImp().get(0)).isSameAs(imp);
         assertThat(bidRequestFuture.result()).isSameAs(bidRequest);
@@ -737,7 +737,7 @@ public void processStoredRequestsShouldNotUpdateMetricsIfApplicationSettingsFail
                         ExtRequestPrebid.builder().storedrequest(ExtStoredRequest.of("bidRequest")).build()))));
 
         // then
-        verifyZeroInteractions(metrics);
+        verifyNoInteractions(metrics);
     }
 
     @Test
@@ -750,7 +750,7 @@ public void processAmpRequestShouldNotUpdateMetricsIfApplicationSettingsFailed()
         storedRequestProcessor.processAmpRequest(null, "123", BidRequest.builder().build());
 
         // then
-        verifyZeroInteractions(metrics);
+        verifyNoInteractions(metrics);
     }
 
     @Test

diff --git a/src/test/java/org/prebid/server/auction/StoredResponseProcessorTest.java b/src/test/java/org/prebid/server/auction/StoredResponseProcessorTest.java
@@ -47,7 +47,7 @@
 import static org.assertj.core.api.Assertions.assertThatThrownBy;
 import static org.mockito.ArgumentMatchers.any;
 import static org.mockito.BDDMockito.given;
-import static org.mockito.Mockito.verifyZeroInteractions;
+import static org.mockito.Mockito.verifyNoInteractions;
 
 public class StoredResponseProcessorTest extends VertxTest {
 
@@ -110,7 +110,7 @@ public void getStoredResponseResultShouldNotChangeImpsAndReturnSeatBidsWhenThere
                 singletonList(imp),
                 emptyList(),
                 emptyMap()));
-        verifyZeroInteractions(applicationSettings);
+        verifyNoInteractions(applicationSettings);
     }
 
     @Test
@@ -132,7 +132,7 @@ public void getStoredResponseResultShouldAddImpToRequiredRequestWhenItsStoredAuc
                         .build()),
                 emptyList(),
                 emptyMap()));
-        verifyZeroInteractions(applicationSettings);
+        verifyNoInteractions(applicationSettings);
     }
 
     @Test

diff --git a/src/test/java/org/prebid/server/auction/requestfactory/AmpRequestFactoryTest.java b/src/test/java/org/prebid/server/auction/requestfactory/AmpRequestFactoryTest.java
@@ -85,7 +85,7 @@
 import static org.mockito.Mockito.doAnswer;
 import static org.mockito.Mockito.doReturn;
 import static org.mockito.Mockito.verify;
-import static org.mockito.Mockito.verifyZeroInteractions;
+import static org.mockito.Mockito.verifyNoInteractions;
 import static org.prebid.server.assertion.FutureAssertion.assertThat;
 
 public class AmpRequestFactoryTest extends VertxTest {
@@ -184,7 +184,7 @@ public void shouldReturnFailedFutureIfRequestHasNoTagId() {
         final Future<?> future = target.fromRequest(routingContext, 0L);
 
         // then
-        verifyZeroInteractions(storedRequestProcessor);
+        verifyNoInteractions(storedRequestProcessor);
         assertThat(future.failed()).isTrue();
         assertThat(future.cause()).isInstanceOf(InvalidRequestException.class);
         assertThat(((InvalidRequestException) future.cause()).getMessages())
@@ -1344,7 +1344,7 @@ public void shouldReturnBidRequestWithExtPrebidDataBiddersUpdatedByFpdResolver()
         verify(fpdResolver).resolveBidRequestExt(any(), any());
         assertThat(request)
                 .extracting(BidRequest::getExt)
-                .containsOnly(ExtRequest.of(ExtRequestPrebid.builder()
+                .isEqualTo(ExtRequest.of(ExtRequestPrebid.builder()
                         .data(ExtRequestPrebidData.of(Arrays.asList("appnexus", "rubicon"), null)).build()));
     }
 

diff --git a/...st/java/org/prebid/server/auction/requestfactory/Ortb2ImplicitParametersResolverTest.java b/...st/java/org/prebid/server/auction/requestfactory/Ortb2ImplicitParametersResolverTest.java
@@ -1096,7 +1096,7 @@ public void shouldSetSiteExtIfNoReferer() {
         // then
         assertThat(result.getSite())
                 .extracting(Site::getExt)
-                .containsOnly(ExtSite.of(0, null));
+                .isEqualTo(ExtSite.of(0, null));
     }
 
     @Test
@@ -1628,7 +1628,7 @@ public void shouldSetDefaultIncludeBidderKeysToFalseIfIncludeBidderKeysIsMissedA
         // then
         assertThat(result.getExt())
                 .extracting(extBidRequest -> extBidRequest.getPrebid().getTargeting().getIncludebidderkeys())
-                .containsOnly(false);
+                .isEqualTo(false);
     }
 
     @Test