Skip to content

Commit a8866b0

Browse files
muskan1012MikaelSmith
muskan1012
authored and
MikaelSmith
committed
CDPD-89903. HADOOP-19243. Upgrade Mockito version to 4.11.0 (apache#6968)
Mockito is now at a JDK-17 compatible version. Contributed by Muskan Mishra (cherry picked from commit f7651e2)
1 parent 61d3f3f commit a8866b0

File tree

84 files changed

+267
-138
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

84 files changed

+267
-138
lines changed

‎hadoop-client-modules/hadoop-client-minicluster/pom.xml‎

Lines changed: 11 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -411,7 +411,7 @@
411411
<!-- Add back in Mockito since the hadoop-hdfs test jar needs it. -->
412412
<dependency>
413413
<groupId>org.mockito</groupId>
414-
<artifactId>mockito-core</artifactId>
414+
<artifactId>mockito-inline</artifactId>
415415
<optional>true</optional>
416416
</dependency>
417417
<!-- Add back in the transitive dependencies excluded from hadoop-common in client TODO remove once we have a filter for "is in these artifacts" -->
@@ -699,7 +699,7 @@
699699

700700
<!-- Mockito tries to include its own unrelocated copy of hamcrest. :( -->
701701
<filter>
702-
<artifact>org.mockito:mockito-core</artifact>
702+
<artifact>org.mockito:mockito-inline</artifact>
703703
<excludes>
704704
<exclude>asm-license.txt</exclude>
705705
<exclude>cglib-license.txt</exclude>
@@ -711,6 +711,15 @@
711711
<exclude>org/objenesis/*.class</exclude>
712712
</excludes>
713713
</filter>
714+
<!-- Additional filters to exclude unexpected contents -->
715+
<filter>
716+
<artifact>*:*</artifact>
717+
<excludes>
718+
<exclude>mockito-extensions/**</exclude>
719+
<exclude>win32-x86/**</exclude>
720+
<exclude>win32-x86-64/**</exclude>
721+
</excludes>
722+
</filter>
714723
<!-- skip grizzly internals we don't need to run. -->
715724
<filter>
716725
<artifact>org.glassfish.grizzly:grizzly-http-servlet</artifact>

‎hadoop-cloud-storage-project/hadoop-huaweicloud/pom.xml‎

Lines changed: 15 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -179,15 +179,27 @@
179179
</dependency>
180180
<dependency>
181181
<groupId>org.powermock</groupId>
182-
<artifactId>powermock-api-mockito</artifactId>
183-
<version>1.7.4</version>
182+
<artifactId>powermock-api-mockito2</artifactId>
183+
<version>2.0.9</version>
184184
<scope>test</scope>
185+
<exclusions>
186+
<exclusion>
187+
<groupId>org.mockito</groupId>
188+
<artifactId>mockito-core</artifactId>
189+
</exclusion>
190+
</exclusions>
185191
</dependency>
186192
<dependency>
187193
<groupId>org.powermock</groupId>
188194
<artifactId>powermock-module-junit4</artifactId>
189-
<version>1.7.4</version>
195+
<version>2.0.9</version>
190196
<scope>test</scope>
197+
<exclusions>
198+
<exclusion>
199+
<groupId>org.mockito</groupId>
200+
<artifactId>mockito-core</artifactId>
201+
</exclusion>
202+
</exclusions>
191203
</dependency>
192204
</dependencies>
193205
</project>

‎hadoop-common-project/hadoop-auth/pom.xml‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -55,6 +55,7 @@
5555
<dependency>
5656
<groupId>org.mockito</groupId>
5757
<artifactId>mockito-core</artifactId>
58+
<version>4.11.0</version>
5859
<scope>test</scope>
5960
</dependency>
6061
<dependency>

‎hadoop-common-project/hadoop-common/pom.xml‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -217,7 +217,7 @@
217217
</dependency>
218218
<dependency>
219219
<groupId>org.mockito</groupId>
220-
<artifactId>mockito-core</artifactId>
220+
<artifactId>mockito-inline</artifactId>
221221
<scope>test</scope>
222222
</dependency>
223223
<dependency>

‎hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/ipc/TestServer.java‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,8 @@
3535
import org.junit.Test;
3636
import org.slf4j.Logger;
3737

38+
import static org.apache.hadoop.test.MockitoUtil.verifyZeroInteractions;
39+
3840
/**
3941
* This is intended to be a set of unit tests for the
4042
* org.apache.hadoop.ipc.Server class.

‎hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/security/http/TestCrossOriginFilter.java‎

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -36,6 +36,8 @@
3636
import org.junit.Test;
3737
import org.mockito.Mockito;
3838

39+
import static org.apache.hadoop.test.MockitoUtil.verifyZeroInteractions;
40+
3941
public class TestCrossOriginFilter {
4042

4143
@Test
@@ -59,7 +61,7 @@ public void testSameOrigin() throws ServletException, IOException {
5961
filter.init(filterConfig);
6062
filter.doFilter(mockReq, mockRes, mockChain);
6163

62-
Mockito.verifyZeroInteractions(mockRes);
64+
verifyZeroInteractions(mockRes);
6365
Mockito.verify(mockChain).doFilter(mockReq, mockRes);
6466
}
6567

@@ -224,7 +226,7 @@ public void testDisallowedOrigin() throws ServletException, IOException {
224226
filter.init(filterConfig);
225227
filter.doFilter(mockReq, mockRes, mockChain);
226228

227-
Mockito.verifyZeroInteractions(mockRes);
229+
verifyZeroInteractions(mockRes);
228230
Mockito.verify(mockChain).doFilter(mockReq, mockRes);
229231
}
230232

@@ -252,7 +254,7 @@ public void testDisallowedMethod() throws ServletException, IOException {
252254
filter.init(filterConfig);
253255
filter.doFilter(mockReq, mockRes, mockChain);
254256

255-
Mockito.verifyZeroInteractions(mockRes);
257+
verifyZeroInteractions(mockRes);
256258
Mockito.verify(mockChain).doFilter(mockReq, mockRes);
257259
}
258260

@@ -283,7 +285,7 @@ public void testDisallowedHeader() throws ServletException, IOException {
283285
filter.init(filterConfig);
284286
filter.doFilter(mockReq, mockRes, mockChain);
285287

286-
Mockito.verifyZeroInteractions(mockRes);
288+
verifyZeroInteractions(mockRes);
287289
Mockito.verify(mockChain).doFilter(mockReq, mockRes);
288290
}
289291

‎hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/security/http/TestRestCsrfPreventionFilter.java‎

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,8 @@
3232
import org.junit.Test;
3333
import org.mockito.Mockito;
3434

35+
import static org.apache.hadoop.test.MockitoUtil.verifyZeroInteractions;
36+
3537
/**
3638
* This class tests the behavior of the RestCsrfPreventionFilter.
3739
*
@@ -75,7 +77,7 @@ public void testNoHeaderDefaultConfigBadRequest()
7577

7678
verify(mockRes, atLeastOnce()).sendError(
7779
HttpServletResponse.SC_BAD_REQUEST, EXPECTED_MESSAGE);
78-
Mockito.verifyZeroInteractions(mockChain);
80+
verifyZeroInteractions(mockChain);
7981
}
8082

8183
@Test
@@ -110,7 +112,7 @@ public void testNoHeaderCustomAgentConfigBadRequest()
110112

111113
verify(mockRes, atLeastOnce()).sendError(
112114
HttpServletResponse.SC_BAD_REQUEST, EXPECTED_MESSAGE);
113-
Mockito.verifyZeroInteractions(mockChain);
115+
verifyZeroInteractions(mockChain);
114116
}
115117

116118
@Test
@@ -228,7 +230,7 @@ public void testMissingHeaderWithCustomHeaderConfigBadRequest()
228230
filter.init(filterConfig);
229231
filter.doFilter(mockReq, mockRes, mockChain);
230232

231-
Mockito.verifyZeroInteractions(mockChain);
233+
verifyZeroInteractions(mockChain);
232234
}
233235

234236
@Test
@@ -260,7 +262,7 @@ public void testMissingHeaderNoMethodsToIgnoreConfigBadRequest()
260262
filter.init(filterConfig);
261263
filter.doFilter(mockReq, mockRes, mockChain);
262264

263-
Mockito.verifyZeroInteractions(mockChain);
265+
verifyZeroInteractions(mockChain);
264266
}
265267

266268
@Test
@@ -356,6 +358,6 @@ public void testMissingHeaderMultipleIgnoreMethodsConfigBadRequest()
356358
filter.init(filterConfig);
357359
filter.doFilter(mockReq, mockRes, mockChain);
358360

359-
Mockito.verifyZeroInteractions(mockChain);
361+
verifyZeroInteractions(mockChain);
360362
}
361363
}

‎hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/test/MockitoUtil.java‎

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -61,4 +61,13 @@ public Object answer(InvocationOnMock invocation) throws Throwable {
6161
}
6262
});
6363
}
64+
65+
/**
66+
* Verifies that there were no interactions with the given mock objects.
67+
*
68+
* @param mocks the mock objects to verify
69+
*/
70+
public static void verifyZeroInteractions(Object... mocks) {
71+
Mockito.verifyNoInteractions(mocks);
72+
}
6473
}

‎hadoop-common-project/hadoop-kms/pom.xml‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@
4545
</dependency>
4646
<dependency>
4747
<groupId>org.mockito</groupId>
48-
<artifactId>mockito-core</artifactId>
48+
<artifactId>mockito-inline</artifactId>
4949
<scope>test</scope>
5050
</dependency>
5151
<dependency>

‎hadoop-common-project/hadoop-nfs/pom.xml‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,7 @@
6666
</dependency>
6767
<dependency>
6868
<groupId>org.mockito</groupId>
69-
<artifactId>mockito-core</artifactId>
69+
<artifactId>mockito-inline</artifactId>
7070
<scope>test</scope>
7171
</dependency>
7272
<dependency>

0 commit comments

Comments
 (0)