Skip to content

Commit

Permalink
Revert "ci: Java 17 profile to run tests with old Mockito (#1343)" (#…
Browse files Browse the repository at this point in the history
…1349)

This reverts commit 035adde.
  • Loading branch information
tom-andersen committed Jul 5, 2023
1 parent 0b4180e commit 7ff8e0d
Showing 1 changed file with 0 additions and 30 deletions.
30 changes: 0 additions & 30 deletions pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -264,35 +264,5 @@
<sourceFileExclude>com/google/cloud/firestore/v1/package-info.java</sourceFileExclude>
</properties>
</profile>

<profile>
<!--
Due to old Mockito version, naive Java 17 junit test fails due to
'ClassFormatError accessible: module java.base does not "opens java.lang"
to unnamed module'
-->
<id>java17</id>
<activation>
<jdk>[17,)</jdk>
<property>
<!--
In Java 8 unit tests where we run tests in Java 8 after building
bytecode on Java 17, we don't want to add the argLine
-->
<name>!jvm</name>
</property>
</activation>
<build>
<plugins>
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-surefire-plugin</artifactId>
<configuration>
<argLine>--add-opens=java.base/java.lang=ALL-UNNAMED</argLine>
</configuration>
</plugin>
</plugins>
</build>
</profile>
</profiles>
</project>

0 comments on commit 7ff8e0d

Please sign in to comment.