Uploaded image for project: 'Maven Surefire'
  1. Maven Surefire
  2. SUREFIRE-1531

Option to switch-off Java 9 modules

    XMLWordPrintableJSON

    Details

    • Type: New Feature
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 2.22.0
    • Fix Version/s: 3.0.0-M2
    • Component/s: Maven Surefire Plugin
    • Labels:
      None

      Description

      I am working on a library and I am adding support for Java 9 modules. Surefire 2.21.0 by default executes tests with Java 9 modules switched-on if it detects module-info.java While it may make sense in some cases, in my case I'd like the switch it off.

      The reason is simple. I am using Mockito to mock an interface that extends a Spring interface. Mockito has to create an implementation of this interface (proxy or subclass) and for this it needs to have access to the Spring interfaces. If Java 9 modules are enabled for the tests I have to manually add each such dependencies to Surefire configuration, which does not make much sense. To makes things worse, the interface actually extends two Spring interfaces form two different Spring modules so the configuration is almost impossible to get right.

      So far I am at ( and I am still getting IllegalAccessErrors)

      --add-exports spring.context/org.springframework.scheduling=org.mockito
      --add-exports spring.beans/org.springframework.beans.factory=org.mockito
      

      I would prefer to switch-off the Java 9 modules for the test module altogether (same behavior as pre 2.21.0)

       

      The test is here https://github.com/lukas-krecan/ShedLock/blob/java9/shedlock-spring/src/test/java/net/javacrumbs/shedlock/spring/CleanupTest.java

       

      If you want, I can send a pull request.

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                tibordigana Tibor Digana
                Reporter:
                lukas_krecan Lukáš Křečan
              • Votes:
                0 Vote for this issue
                Watchers:
                6 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: