You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@flink.apache.org by ch...@apache.org on 2021/11/26 08:56:10 UTC

[flink] 03/03: [hotfix][tests] Allow @VisibleForTesting calls from static inner classes

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

commit c6af21f2036d26012424e50e902b0a9a74c8ccf3
Author: Chesnay Schepler <ch...@apache.org>
AuthorDate: Thu Nov 25 15:58:26 2021 +0100

    [hotfix][tests] Allow @VisibleForTesting calls from static inner classes
---
 .../org/apache/flink/architecture/rules/ApiAnnotationRules.java  | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)

diff --git a/flink-architecture-tests/src/test/java/org/apache/flink/architecture/rules/ApiAnnotationRules.java b/flink-architecture-tests/src/test/java/org/apache/flink/architecture/rules/ApiAnnotationRules.java
index 1402612..e96e3e3 100644
--- a/flink-architecture-tests/src/test/java/org/apache/flink/architecture/rules/ApiAnnotationRules.java
+++ b/flink-architecture-tests/src/test/java/org/apache/flink/architecture/rules/ApiAnnotationRules.java
@@ -141,11 +141,10 @@ public class ApiAnnotationRules {
                                             if (originOwner.equals(targetOwner)) {
                                                 return false;
                                             }
-                                            if (originOwner.isInnerClass()
-                                                    && originOwner
-                                                            .getEnclosingClass()
-                                                            .map(targetOwner::equals)
-                                                            .orElse(false)) {
+                                            if (originOwner
+                                                    .getEnclosingClass()
+                                                    .map(targetOwner::equals)
+                                                    .orElse(false)) {
                                                 return false;
                                             }
                                             if (targetOwner