This is an automated email from the ASF dual-hosted git repository. rfscholte pushed a commit to branch MJDEPS-26 in repository https://gitbox.apache.org/repos/asf/maven-jdeps-plugin.git
commit 893bfca8528b412ba31a1f9c30f723b6c98288b1 Author: rfscholte <[email protected]> AuthorDate: Fri Jul 2 13:23:01 2021 +0200 [MJDEPS-26] Plugin doesn't recognized latest Java8 output --- .../maven/plugins/jdeps/consumers/JDepsConsumer.java | 2 +- .../maven/plugins/jdeps/consumers/JDepsConsumerTest.java | 15 +++++++++++++++ 2 files changed, 16 insertions(+), 1 deletion(-) diff --git a/src/main/java/org/apache/maven/plugins/jdeps/consumers/JDepsConsumer.java b/src/main/java/org/apache/maven/plugins/jdeps/consumers/JDepsConsumer.java index 7d9df41..3dffdb0 100644 --- a/src/main/java/org/apache/maven/plugins/jdeps/consumers/JDepsConsumer.java +++ b/src/main/java/org/apache/maven/plugins/jdeps/consumers/JDepsConsumer.java @@ -43,7 +43,7 @@ public class JDepsConsumer * JDK8 Linux: JDK internal API (JDK removed internal API) * JDK9: JDK internal API (java.base) */ - private static final Pattern JDKINTERNALAPI = Pattern.compile( ".+->\\s([a-z\\.]+)\\s+(JDK internal API .+)" ); + private static final Pattern JDKINTERNALAPI = Pattern.compile( ".+->\\s([a-z\\.]+)\\s+(JDK (?:removed )?internal API.*)" ); /** * <dl> diff --git a/src/test/java/org/apache/maven/plugins/jdeps/consumers/JDepsConsumerTest.java b/src/test/java/org/apache/maven/plugins/jdeps/consumers/JDepsConsumerTest.java index aafad5c..ed60478 100644 --- a/src/test/java/org/apache/maven/plugins/jdeps/consumers/JDepsConsumerTest.java +++ b/src/test/java/org/apache/maven/plugins/jdeps/consumers/JDepsConsumerTest.java @@ -59,6 +59,21 @@ public class JDepsConsumerTest } @Test + public void testJDKInternalAPI_Java8_291() + { + consumer = new JDepsConsumer(); + consumer.consumeLine( "classes -> JDK removed internal API" ); + consumer.consumeLine( "classes -> java.base" ); + consumer.consumeLine( " <unnamed> -> java.io java.base" ); + consumer.consumeLine( " <unnamed> -> java.lang java.base" ); + consumer.consumeLine( " <unnamed> -> sun.misc JDK removed internal API" ); + + assertEquals( 1, consumer.getOffendingPackages().size() ); + assertEquals( "JDK removed internal API", consumer.getOffendingPackages().get( "sun.misc" ) ); + assertEquals( 0, consumer.getProfiles().size() ); + } + + @Test public void testProfile() { consumer = new JDepsConsumer();
