Skip to content
This repository was archived by the owner on Sep 13, 2024. It is now read-only.

Commit b1ddb81

Browse files
committed
Update test setup so tests will work on Java 17+
1 parent d642297 commit b1ddb81

File tree

3 files changed

+16
-11
lines changed

3 files changed

+16
-11
lines changed

org.eclipse.sisu.plexus/pom.xml

+15-5
Original file line numberDiff line numberDiff line change
@@ -142,6 +142,7 @@
142142
<artifactItem>
143143
<groupId>com.google.inject</groupId>
144144
<artifactId>guice</artifactId>
145+
<classifier>no_aop</classifier>
145146
<version>3.0</version>
146147
</artifactItem>
147148
</artifactItems>
@@ -164,6 +165,7 @@
164165
<artifactItem>
165166
<groupId>com.google.inject</groupId>
166167
<artifactId>guice</artifactId>
168+
<classifier>no_aop</classifier>
167169
<version>4.0</version>
168170
</artifactItem>
169171
</artifactItems>
@@ -191,6 +193,7 @@
191193
<artifactItem>
192194
<groupId>com.google.inject</groupId>
193195
<artifactId>guice</artifactId>
196+
<classifier>no_aop</classifier>
194197
<version>4.2.3</version>
195198
</artifactItem>
196199
</artifactItems>
@@ -215,9 +218,15 @@
215218
<artifactId>guava</artifactId>
216219
<version>30.1-jre</version>
217220
</artifactItem>
221+
<artifactItem>
222+
<groupId>org.ow2.asm</groupId>
223+
<artifactId>asm</artifactId>
224+
<version>9.5</version>
225+
</artifactItem>
218226
<artifactItem>
219227
<groupId>com.google.inject</groupId>
220228
<artifactId>guice</artifactId>
229+
<classifier>classes</classifier>
221230
<version>5.1.0</version>
222231
</artifactItem>
223232
</artifactItems>
@@ -269,7 +278,7 @@
269278
<configuration>
270279
<groups>org.eclipse.sisu.BaseTests</groups>
271280
<additionalClasspathElements>
272-
<additionalClasspathElement>${project.build.directory}/guice3/guice.jar</additionalClasspathElement>
281+
<additionalClasspathElement>${project.build.directory}/guice3/guice-no_aop.jar</additionalClasspathElement>
273282
</additionalClasspathElements>
274283
</configuration>
275284
</execution>
@@ -283,7 +292,7 @@
283292
<excludedGroups>org.eclipse.sisu.BaseTests</excludedGroups>
284293
<additionalClasspathElements>
285294
<additionalClasspathElement>${project.build.directory}/guice3/guava.jar</additionalClasspathElement>
286-
<additionalClasspathElement>${project.build.directory}/guice3/guice.jar</additionalClasspathElement>
295+
<additionalClasspathElement>${project.build.directory}/guice3/guice-no_aop.jar</additionalClasspathElement>
287296
</additionalClasspathElements>
288297
<systemPropertyVariables>
289298
<guiceVersion>guice3</guiceVersion>
@@ -300,7 +309,7 @@
300309
<excludedGroups>org.eclipse.sisu.BaseTests</excludedGroups>
301310
<additionalClasspathElements>
302311
<additionalClasspathElement>${project.build.directory}/guice4/guava.jar</additionalClasspathElement>
303-
<additionalClasspathElement>${project.build.directory}/guice4/guice.jar</additionalClasspathElement>
312+
<additionalClasspathElement>${project.build.directory}/guice4/guice-no_aop.jar</additionalClasspathElement>
304313
</additionalClasspathElements>
305314
<systemPropertyVariables>
306315
<guiceVersion>guice4</guiceVersion>
@@ -318,7 +327,7 @@
318327
<additionalClasspathElements>
319328
<additionalClasspathElement>${project.build.directory}/guice423/failureaccess.jar</additionalClasspathElement>
320329
<additionalClasspathElement>${project.build.directory}/guice423/guava.jar</additionalClasspathElement>
321-
<additionalClasspathElement>${project.build.directory}/guice423/guice.jar</additionalClasspathElement>
330+
<additionalClasspathElement>${project.build.directory}/guice423/guice-no_aop.jar</additionalClasspathElement>
322331
</additionalClasspathElements>
323332
<systemPropertyVariables>
324333
<guiceVersion>guice423</guiceVersion>
@@ -336,7 +345,8 @@
336345
<additionalClasspathElements>
337346
<additionalClasspathElement>${project.build.directory}/guice5/failureaccess.jar</additionalClasspathElement>
338347
<additionalClasspathElement>${project.build.directory}/guice5/guava.jar</additionalClasspathElement>
339-
<additionalClasspathElement>${project.build.directory}/guice5/guice.jar</additionalClasspathElement>
348+
<additionalClasspathElement>${project.build.directory}/guice5/asm.jar</additionalClasspathElement>
349+
<additionalClasspathElement>${project.build.directory}/guice5/guice-classes.jar</additionalClasspathElement>
340350
</additionalClasspathElements>
341351
<systemPropertyVariables>
342352
<guiceVersion>guice5</guiceVersion>

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

+1
Original file line numberDiff line numberDiff line change
@@ -104,6 +104,7 @@ private static void checkBehaviour( final String name )
104104
String origToString = orig.toString().replace( "\"", "" ).replace( ".class", "" );
105105
String cloneToString = clone.toString().replace( '[', '{' ).replace( ']', '}' );
106106
cloneToString = cloneToString.replace( "class ", "" ).replace( "interface ", "" );
107+
cloneToString = cloneToString.replace( "ComponentAnnotationTest$A", "ComponentAnnotationTest.A" );
107108

108109
assertEquals( new HashSet<String>( Arrays.asList( origToString.split( "[(, )]" ) ) ),
109110
new HashSet<String>( Arrays.asList( cloneToString.split( "[(, )]" ) ) ) );

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

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

0 commit comments

Comments
 (0)