Karm / mandrel-integration-tests

Integration tests for GraalVM and its Mandrel distribution. Runs Quarkus, Helidon and Micronaut applications and small targeted reproducers. The focus is solely on native-image utility and compilation of Java applications into native executables.
Apache License 2.0
5 stars 3 forks source link

Fix orsoncharts allowlist #123

Closed jerboaa closed 1 year ago

jerboaa commented 1 year ago

On GraalVM 23.x the warning includes [...] resolve class[...] over [...] resolve com.orsoncharts[...] in the message which the test didn't expect. Example warning on 23.x:

Warning: Could not resolve class com.orsoncharts.Chart3DHints for reflection configuration. Reason: java.lang.ClassNotFoundException: com.orsoncharts.Chart3DHints.
jerboaa commented 1 year ago

Fixes a CI failure we see on graal master that looks like this:

Error:  Failures: 
Error:    AppReproducersTest.imageioAWTTest:429->imageioAWT:551 build-and-run.log log should not contain error or warning lines that are not whitelisted. See /home/runner/work/mandrel/mandrel/mandrel-integration-tests/testsuite/target/archived-logs/org.graalvm.tests.integration.AppReproducersTest/imageioAWTTest/build-and-run.log and check these offending lines: 
Warning: Could not resolve class com.orsoncharts.Chart3DHints for reflection configuration. Reason: java.lang.ClassNotFoundException: com.orsoncharts.Chart3DHints. ==> expected: <true> but was: <false>
[INFO] 
Error:  Tests run: 18, Failures: 1, Errors: 0, Skipped: 5

See: https://github.com/Karm/mandrel-integration-tests/issues/102#issuecomment-1275915184 and https://github.com/Karm/mandrel-integration-tests/issues/102#issuecomment-1275948579

jerboaa commented 1 year ago

FTR The upstream change introducing the regression is https://github.com/oracle/graal/pull/5133/files#diff-3595e7915df93ba62b607b2dbbd626afa20f1cfc9f002b5c4c66461f9b757079R89

Thanks for the detective work!