JDK-8235453 : tools/jpackage/junit/junit.java failed due to "module not found: jdk.incubator.jpackage"
  • Type: Bug
  • Component: tools
  • Sub-Component: jpackage
  • Affected Version: 14
  • Priority: P3
  • Status: Resolved
  • Resolution: Fixed
  • OS: solaris
  • CPU: sparc_64
  • Submitted: 2019-12-05
  • Updated: 2019-12-17
  • Resolved: 2019-12-09
The Version table provides details related to the release that this issue/RFE will be addressed.

Unresolved : Release in which this issue/RFE will be addressed.
Resolved: Release in which this issue/RFE has been resolved.
Fixed : Release in which this issue/RFE has been fixed. The release containing this fix may be available for download as an Early Access Release or a General Availability Release.

To download the current JDK release, click here.
JDK 14
14 b27Fixed
Related Reports
Relates :  
JDK-8212780 - Packaging Tool Implementation
Description
The following test fails in the JDK14 CI:

tools/jpackage/junit/junit.java

Here's a snippet from the log file:

#section:shell
----------messages:(3/113)----------
command: shell run_junit.sh
reason: User specified action: run shell run_junit.sh 
elapsed time (seconds): 7.802
----------System.out:(0/0)----------
----------System.err:(58/7866)----------
+ set -e
+ [ -z '' ]
+ /bin/bash /scratch/opt/mach5/mesos/work_dir/jib-master/install/jdk-14+27-1254/src.full/open/test/jdk/tools/jpackage/junit/run_junit.sh
+ set -e
+ '[' -z /bin/bash ']'
+ sources=()
+ classes=()
++ grep -v junit.java
++ find /scratch/opt/mach5/mesos/work_dir/jib-master/install/jdk-14+27-1254/src.full/open/test/jdk/tools/jpackage/junit -name '*.java'
+ for s in $(find "${TESTSRC}" -name  "*.java" | grep -v junit.java)
+ sources+=("$s")
+ classes+=($(echo "$s" | sed -e "s|${TESTSRC}/||" -e 's|/|.|g' -e 's/.java$//'))
++ echo /scratch/opt/mach5/mesos/work_dir/jib-master/install/jdk-14+27-1254/src.full/open/test/jdk/tools/jpackage/junit/jdk/incubator/jpackage/internal/DottedVersionTest.java
++ sed -e 's|/scratch/opt/mach5/mesos/work_dir/jib-master/install/jdk-14+27-1254/src.full/open/test/jdk/tools/jpackage/junit/||' -e 's|/|.|g' -e 's/.java$//'
+ for s in $(find "${TESTSRC}" -name  "*.java" | grep -v junit.java)
+ sources+=("$s")
+ classes+=($(echo "$s" | sed -e "s|${TESTSRC}/||" -e 's|/|.|g' -e 's/.java$//'))
++ echo /scratch/opt/mach5/mesos/work_dir/jib-master/install/jdk-14+27-1254/src.full/open/test/jdk/tools/jpackage/junit/jdk/incubator/jpackage/internal/OverridableResourceTest.java
++ sed -e 's|/scratch/opt/mach5/mesos/work_dir/jib-master/install/jdk-14+27-1254/src.full/open/test/jdk/tools/jpackage/junit/||' -e 's|/|.|g' -e 's/.java$//'
+ for s in $(find "${TESTSRC}" -name  "*.java" | grep -v junit.java)
+ sources+=("$s")
+ classes+=($(echo "$s" | sed -e "s|${TESTSRC}/||" -e 's|/|.|g' -e 's/.java$//'))
++ echo /scratch/opt/mach5/mesos/work_dir/jib-master/install/jdk-14+27-1254/src.full/open/test/jdk/tools/jpackage/junit/jdk/incubator/jpackage/internal/PathGroupTest.java
++ sed -e 's|/scratch/opt/mach5/mesos/work_dir/jib-master/install/jdk-14+27-1254/src.full/open/test/jdk/tools/jpackage/junit/||' -e 's|/|.|g' -e 's/.java$//'
+ for s in $(find "${TESTSRC}" -name  "*.java" | grep -v junit.java)
+ sources+=("$s")
+ classes+=($(echo "$s" | sed -e "s|${TESTSRC}/||" -e 's|/|.|g' -e 's/.java$//'))
++ echo /scratch/opt/mach5/mesos/work_dir/jib-master/install/jdk-14+27-1254/src.full/open/test/jdk/tools/jpackage/junit/jdk/incubator/jpackage/internal/ApplicationLayoutTest.java
++ sed -e 's|/scratch/opt/mach5/mesos/work_dir/jib-master/install/jdk-14+27-1254/src.full/open/test/jdk/tools/jpackage/junit/||' -e 's|/|.|g' -e 's/.java$//'
+ for s in $(find "${TESTSRC}" -name  "*.java" | grep -v junit.java)
+ sources+=("$s")
+ classes+=($(echo "$s" | sed -e "s|${TESTSRC}/||" -e 's|/|.|g' -e 's/.java$//'))
++ echo /scratch/opt/mach5/mesos/work_dir/jib-master/install/jdk-14+27-1254/src.full/open/test/jdk/tools/jpackage/junit/jdk/incubator/jpackage/internal/DeployParamsTest.java
++ sed -e 's|/scratch/opt/mach5/mesos/work_dir/jib-master/install/jdk-14+27-1254/src.full/open/test/jdk/tools/jpackage/junit/||' -e 's|/|.|g' -e 's/.java$//'
+ for s in $(find "${TESTSRC}" -name  "*.java" | grep -v junit.java)
+ sources+=("$s")
+ classes+=($(echo "$s" | sed -e "s|${TESTSRC}/||" -e 's|/|.|g' -e 's/.java$//'))
++ echo /scratch/opt/mach5/mesos/work_dir/jib-master/install/jdk-14+27-1254/src.full/open/test/jdk/tools/jpackage/junit/jdk/incubator/jpackage/internal/InvalidDottedVersionTest.java
++ sed -e 's|/scratch/opt/mach5/mesos/work_dir/jib-master/install/jdk-14+27-1254/src.full/open/test/jdk/tools/jpackage/junit/||' -e 's|/|.|g' -e 's/.java$//'
+ for s in $(find "${TESTSRC}" -name  "*.java" | grep -v junit.java)
+ sources+=("$s")
+ classes+=($(echo "$s" | sed -e "s|${TESTSRC}/||" -e 's|/|.|g' -e 's/.java$//'))
++ echo /scratch/opt/mach5/mesos/work_dir/jib-master/install/jdk-14+27-1254/src.full/open/test/jdk/tools/jpackage/junit/jdk/incubator/jpackage/internal/AppImageFileTest.java
++ sed -e 's|/scratch/opt/mach5/mesos/work_dir/jib-master/install/jdk-14+27-1254/src.full/open/test/jdk/tools/jpackage/junit/||' -e 's|/|.|g' -e 's/.java$//'
+ for s in $(find "${TESTSRC}" -name  "*.java" | grep -v junit.java)
+ sources+=("$s")
+ classes+=($(echo "$s" | sed -e "s|${TESTSRC}/||" -e 's|/|.|g' -e 's/.java$//'))
++ echo /scratch/opt/mach5/mesos/work_dir/jib-master/install/jdk-14+27-1254/src.full/open/test/jdk/tools/jpackage/junit/jdk/incubator/jpackage/internal/CompareDottedVersionTest.java
++ sed -e 's|/scratch/opt/mach5/mesos/work_dir/jib-master/install/jdk-14+27-1254/src.full/open/test/jdk/tools/jpackage/junit/||' -e 's|/|.|g' -e 's/.java$//'
+ for s in $(find "${TESTSRC}" -name  "*.java" | grep -v junit.java)
+ sources+=("$s")
+ classes+=($(echo "$s" | sed -e "s|${TESTSRC}/||" -e 's|/|.|g' -e 's/.java$//'))
++ echo /scratch/opt/mach5/mesos/work_dir/jib-master/install/jdk-14+27-1254/src.full/open/test/jdk/tools/jpackage/junit/jdk/incubator/jpackage/internal/ToolValidatorTest.java
++ sed -e 's|/scratch/opt/mach5/mesos/work_dir/jib-master/install/jdk-14+27-1254/src.full/open/test/jdk/tools/jpackage/junit/||' -e 's|/|.|g' -e 's/.java$//'
+ common_args=(--add-modules jdk.incubator.jpackage --patch-module jdk.incubator.jpackage="${TESTSRC}${PS}${TESTCLASSES}" --add-reads jdk.incubator.jpackage=ALL-UNNAMED --add-exports jdk.incubator.jpackage/jdk.incubator.jpackage.internal=ALL-UNNAMED -classpath "${TESTCLASSPATH}")
+ /scratch/opt/mach5/mesos/work_dir/jib-master/install/jdk-14+27-1254/solaris-sparcv9.jdk/jdk-14/bin/javac -J-Xmx512m -J-XX:MaxRAMPercentage=2 -J-ea -J-esa --add-modules jdk.incubator.jpackage --patch-module jdk.incubator.jpackage=/scratch/opt/mach5/mesos/work_dir/jib-master/install/jdk-14+27-1254/src.full/open/test/jdk/tools/jpackage/junit:/scratch/opt/mach5/mesos/work_dir/47707759-80b9-4734-85d0-f9d401f920f8/testoutput/test-support/jtreg_open_test_jdk_tier2/classes/9/tools/jpackage/junit/junit.d --add-reads jdk.incubator.jpackage=ALL-UNNAMED --add-exports jdk.incubator.jpackage/jdk.incubator.jpackage.internal=ALL-UNNAMED -classpath /scratch/opt/mach5/mesos/work_dir/47707759-80b9-4734-85d0-f9d401f920f8/testoutput/test-support/jtreg_open_test_jdk_tier2/classes/9/tools/jpackage/junit/junit.d:/scratch/opt/mach5/mesos/work_dir/jib-master/install/java/re/jtreg/4.2/promoted/all/b14/bundles/jtreg_bin-4.2.zip/jtreg/lib/junit.jar -d /scratch/opt/mach5/mesos/work_dir/47707759-80b9-4734-85d0-f9d401f920f8/testoutput/test-support/jtreg_open_test_jdk_tier2/classes/9/tools/jpackage/junit/junit.d /scratch/opt/mach5/mesos/work_dir/jib-master/install/jdk-14+27-1254/src.full/open/test/jdk/tools/jpackage/junit/jdk/incubator/jpackage/internal/DottedVersionTest.java /scratch/opt/mach5/mesos/work_dir/jib-master/install/jdk-14+27-1254/src.full/open/test/jdk/tools/jpackage/junit/jdk/incubator/jpackage/internal/OverridableResourceTest.java /scratch/opt/mach5/mesos/work_dir/jib-master/install/jdk-14+27-1254/src.full/open/test/jdk/tools/jpackage/junit/jdk/incubator/jpackage/internal/PathGroupTest.java /scratch/opt/mach5/mesos/work_dir/jib-master/install/jdk-14+27-1254/src.full/open/test/jdk/tools/jpackage/junit/jdk/incubator/jpackage/internal/ApplicationLayoutTest.java /scratch/opt/mach5/mesos/work_dir/jib-master/install/jdk-14+27-1254/src.full/open/test/jdk/tools/jpackage/junit/jdk/incubator/jpackage/internal/DeployParamsTest.java /scratch/opt/mach5/mesos/work_dir/jib-master/install/jdk-14+27-1254/src.full/open/test/jdk/tools/jpackage/junit/jdk/incubator/jpackage/internal/InvalidDottedVersionTest.java /scratch/opt/mach5/mesos/work_dir/jib-master/install/jdk-14+27-1254/src.full/open/test/jdk/tools/jpackage/junit/jdk/incubator/jpackage/internal/AppImageFileTest.java /scratch/opt/mach5/mesos/work_dir/jib-master/install/jdk-14+27-1254/src.full/open/test/jdk/tools/jpackage/junit/jdk/incubator/jpackage/internal/CompareDottedVersionTest.java /scratch/opt/mach5/mesos/work_dir/jib-master/install/jdk-14+27-1254/src.full/open/test/jdk/tools/jpackage/junit/jdk/incubator/jpackage/internal/ToolValidatorTest.java
error: module not found: jdk.incubator.jpackage
1 error
----------rerun:(33/2634)*----------

This error message:

> error: module not found: jdk.incubator.jpackage

makes me wonder why that module is missing.
Do we have a build race?
Comments
URL: https://hg.openjdk.java.net/jdk/jdk/rev/93a167720c90 User: herrick Date: 2019-12-09 18:02:32 +0000
09-12-2019
only junit,java was failing because all other tests have some form of @module jdk.incubator.jpackage annotation. simply adding that to junit.java is enough to fix. webrev: http://cr.openjdk.java.net/~herrick/8235453/webrev.02/
09-12-2019
webrev: http://cr.openjdk.java.net/~herrick/8235453/webrev.01/index.html running test on solaris now to confirm if this covers all the failing tests.
06-12-2019
This test failure started with the push for JDK-8212780.
06-12-2019