update coverage checker to ignore non-public classes

unimined · Mar 19, 2024 · 23773a5 · 23773a5
1 parent af7863a
commit 23773a5
Show file tree

Hide file tree

Showing 4 changed files with 60 additions and 48 deletions.
diff --git a/.gitignore b/.gitignore
@@ -5,6 +5,7 @@ build/
 !**/src/test/**/build/
 
 ### IntelliJ IDEA ###
+.idea/
 .idea/modules.xml
 .idea/jarRepositories.xml
 .idea/compiler.xml

diff --git a/.idea/gradle.xml b/.idea/gradle.xml
diff --git a/.idea/workspace.xml b/.idea/workspace.xml
diff --git a/java-api/src/coverage/java/xyz/wagyourtail/jvmdg/coverage/CoverageChecker.java b/java-api/src/coverage/java/xyz/wagyourtail/jvmdg/coverage/CoverageChecker.java
@@ -168,6 +168,7 @@ public static void compare(List<Path> moduleHolders, Map<String, Pair<String, Cl
                         if (cn.name.startsWith("sun/")) {
                             if (!cn.name.equals("sun/misc/Unsafe")) return;
                         }
+                        if ((cn.access & (Opcodes.ACC_PUBLIC | Opcodes.ACC_PROTECTED)) == 0) return;
                         newClasses.put(cn.name, new Pair<>(modName, cn));
                         if (currentVersion.containsKey(cn.name)) {
                             // check to see what was "removed"