[JCR-4179] Test failures with Java 9 when using javax.imageio.spi.ServiceRegistry

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.14.2, 2.15.5
• Fix Version/s: 2.16, 2.14.3, 2.15.6
• Component/s: jackrabbit-jcr-client, jackrabbit-jcr-commons, jackrabbit-spi-commons
• Environment: Java 9 b181
• Assignee: Robert Munteanu
• Reporter: Robert Munteanu
• Created: 05/Sep/17 08:25
• Updated: 18/Jul/18 15:19
• Resolved: 05/Sep/17 08:42

Description

When building with Java 9 there is at least one test failure due to using javax.imageio.spi.ServiceRegistry as a general-purpose service loader, which is not possible anymore with Java 9:

[ERROR] Tests run: 3, Failures: 0, Errors: 1, Skipped: 0, Time elapsed: 0.13 s <<< FAILURE! - in org.apache.jackrabbit.commons.JcrUtilsTest
[ERROR] testGetRepository(org.apache.jackrabbit.commons.JcrUtilsTest)  Time elapsed: 0.128 s  <<< ERROR!
java.lang.IllegalArgumentException: javax.jcr.RepositoryFactory is not an ImageIO SPI class
	at org.apache.jackrabbit.commons.JcrUtilsTest.testGetRepository(JcrUtilsTest.java:50)

We should move away from this ServiceRegistry and use java.util.ServiceLoader instead.

Issue Links

• blocks: SLING-7102 DavEx integration tests fail on Java 9 - javax.jcr.RepositoryFactory is not an ImageIO SPI class
• is duplicated by: JCR-4131 JDK 9 support
• is related to: SLING-7159 Java9: Upgrade Jackrabbit Dependencies to make Commons Testing compatible with Java9