[flink] branch master updated: [FLINK-28379][tests] Prevent mocked UGI bleeding into other tests

[ch...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

chesnay pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/flink.git


The following commit(s) were added to refs/heads/master by this push:
     new 1ef0e7c34f5 [FLINK-28379][tests] Prevent mocked UGI bleeding into other tests
1ef0e7c34f5 is described below

commit 1ef0e7c34f594d9fa34f34477dfdb691d2836841
Author: Chesnay Schepler <ch...@apache.org>
AuthorDate: Mon Jul 11 16:11:08 2022 +0200

    [FLINK-28379][tests] Prevent mocked UGI bleeding into other tests
---
 ...osLoginProviderTest.java => KerberosLoginProviderITCase.java} | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/flink-runtime/src/test/java/org/apache/flink/runtime/security/token/KerberosLoginProviderTest.java b/flink-runtime/src/test/java/org/apache/flink/runtime/security/token/KerberosLoginProviderITCase.java
similarity index 97%
rename from flink-runtime/src/test/java/org/apache/flink/runtime/security/token/KerberosLoginProviderTest.java
rename to flink-runtime/src/test/java/org/apache/flink/runtime/security/token/KerberosLoginProviderITCase.java
index debe531c403..f8406399224 100644
--- a/flink-runtime/src/test/java/org/apache/flink/runtime/security/token/KerberosLoginProviderTest.java
+++ b/flink-runtime/src/test/java/org/apache/flink/runtime/security/token/KerberosLoginProviderITCase.java
@@ -40,8 +40,13 @@ import static org.mockito.Mockito.mock;
 import static org.mockito.Mockito.mockStatic;
 import static org.mockito.Mockito.when;
 
-/** Test for {@link KerberosLoginProvider}. */
-public class KerberosLoginProviderTest {
+/**
+ * Test for {@link KerberosLoginProvider}.
+ *
+ * <p>This class is an ITCase because the mocking breaks the {@link UserGroupInformation} class for
+ * other tests.
+ */
+public class KerberosLoginProviderITCase {
 
     @Test
     public void isLoginPossibleMustReturnFalseByDefault() throws IOException {