Ignore PlexusRequirementTest on Java17 unless testing with guice5

Author: mcculls

org.eclipse.sisu.plexus/pom.xml

@@ -285,6 +285,9 @@
                 <additionalClasspathElement>${project.build.directory}/guice3/guava.jar</additionalClasspathElement>
                 <additionalClasspathElement>${project.build.directory}/guice3/guice.jar</additionalClasspathElement>
               </additionalClasspathElements>
+              <systemPropertyVariables>
+                <guiceVersion>guice3</guiceVersion>
+              </systemPropertyVariables>
             </configuration>
           </execution>
           <execution>
@@ -299,6 +302,9 @@
                 <additionalClasspathElement>${project.build.directory}/guice4/guava.jar</additionalClasspathElement>
                 <additionalClasspathElement>${project.build.directory}/guice4/guice.jar</additionalClasspathElement>
               </additionalClasspathElements>
+              <systemPropertyVariables>
+                <guiceVersion>guice4</guiceVersion>
+              </systemPropertyVariables>
             </configuration>
           </execution>
           <execution>
@@ -314,6 +320,9 @@
                 <additionalClasspathElement>${project.build.directory}/guice423/guava.jar</additionalClasspathElement>
                 <additionalClasspathElement>${project.build.directory}/guice423/guice.jar</additionalClasspathElement>
               </additionalClasspathElements>
+              <systemPropertyVariables>
+                <guiceVersion>guice423</guiceVersion>
+              </systemPropertyVariables>
             </configuration>
           </execution>
           <execution>
@@ -329,6 +338,9 @@
                 <additionalClasspathElement>${project.build.directory}/guice5/guava.jar</additionalClasspathElement>
                 <additionalClasspathElement>${project.build.directory}/guice5/guice.jar</additionalClasspathElement>
               </additionalClasspathElements>
+              <systemPropertyVariables>
+                <guiceVersion>guice5</guiceVersion>
+              </systemPropertyVariables>
             </configuration>
           </execution>
         </executions>

org.eclipse.sisu.plexus/src/test/java/org/eclipse/sisu/plexus/PlexusRequirementTest.java

@@ -594,6 +594,12 @@ public void testBadDeferredRole()
     public void testPlexus121Compatibility()
         throws Exception
     {
+        if ( "17".equals( System.getProperty( "java.specification.version", "undefined" ) )
+            && !"guice5".equals( System.getProperty( "guiceVersion", "undefined" ) ) )
+        {
+            return; // skip test on Java17 + guice3/4, is not working
+        }
+
         final List<URL> urls = new ArrayList<URL>();
         urls.add( new File( "target/dependency/plexus-component-annotations-1.2.1.jar" ).toURI().toURL() );
         Collections.addAll( urls, new URLClassSpace( getClass().getClassLoader() ).getURLs() );