RFR: 8350051: [JMH] Several tests fails NPE [v2]

SendaoYan syan at openjdk.org
Wed Feb 19 03:21:37 UTC 2025


> Hi all,
> 
> Several JMH tests fails 'Cannot invoke "java.io.InputStream.available()" because "is" is null', because the file 'build/linux-x86_64-server-release/images/test/micro/benchmarks.jar' missing the required xml input file defined by test/micro/org/openjdk/bench/javax/xml/AbstractXMLMicro.java. This PR copy the required xml file to benchmarks.jar, and remove two unexist xml input file.
> 
> After this PR, below JMH tests will run passes.
> 
> 
> org.openjdk.bench.javax.xml.DOM.testBuild
> org.openjdk.bench.javax.xml.DOM.testModify
> org.openjdk.bench.javax.xml.DOM.testWalk
> org.openjdk.bench.javax.xml.SAXUsingJDK.testParse
> org.openjdk.bench.javax.xml.STAX.testParse
> 
> 
> Test command:
> 
> 
> rm -rf build/jmh-result/ ; mkdir -p build/jmh-result/ ; time for test in `cat list.txt` ; do time make test TEST="micro:$test" MICRO="FORK=1;WARMUP_ITER=2" CONF=release &> build/jmh-result/$test.log ; done
> 
> 
> Change has been verified locally, no risk.

SendaoYan has updated the pull request incrementally with one additional commit since the last revision:

  Use SetupCopyFiles macro to copy the dependency files

-------------

Changes:
  - all: https://git.openjdk.org/jdk/pull/23625/files
  - new: https://git.openjdk.org/jdk/pull/23625/files/ba555448..f553bf77

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=23625&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=23625&range=00-01

  Stats: 17 lines in 1 file changed: 11 ins; 5 del; 1 mod
  Patch: https://git.openjdk.org/jdk/pull/23625.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/23625/head:pull/23625

PR: https://git.openjdk.org/jdk/pull/23625


More information about the build-dev mailing list