Skip to content

Commit dd85298

Browse files
dfa1claude
andcommitted
refactor(integration): rename *IT to *IntegrationTest
Surefire picks up *Test.java by default, so *IntegrationTest.java was running without the required JVM flags (--add-opens, --enable-native-access). Fix: exclude *IntegrationTest from surefire; failsafe includes them with the correct argLine. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent a63980f commit dd85298

3 files changed

Lines changed: 14 additions & 2 deletions

File tree

integration/pom.xml

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -58,11 +58,23 @@
5858

5959
<build>
6060
<plugins>
61+
<plugin>
62+
<groupId>org.apache.maven.plugins</groupId>
63+
<artifactId>maven-surefire-plugin</artifactId>
64+
<configuration>
65+
<excludes>
66+
<exclude>**/*IntegrationTest.java</exclude>
67+
</excludes>
68+
</configuration>
69+
</plugin>
6170
<plugin>
6271
<groupId>org.apache.maven.plugins</groupId>
6372
<artifactId>maven-failsafe-plugin</artifactId>
6473
<version>3.5.2</version>
6574
<configuration>
75+
<includes>
76+
<include>**/*IntegrationTest.java</include>
77+
</includes>
6678
<argLine>
6779
--add-opens java.base/java.nio=ALL-UNNAMED
6880
--enable-native-access=ALL-UNNAMED

integration/src/test/java/io/github/dfa1/vortex/integration/JavaWritesRustReadsIT.java renamed to integration/src/test/java/io/github/dfa1/vortex/integration/JavaWritesRustReadsIntegrationTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@
3131
import static org.assertj.core.api.Assertions.assertThat;
3232

3333
/// Cross-compatibility: Java writer → Rust (JNI) reader.
34-
class JavaWritesRustReadsIT {
34+
class JavaWritesRustReadsIntegrationTest {
3535

3636
static {
3737
NativeLoader.loadJni();

integration/src/test/java/io/github/dfa1/vortex/integration/RustWritesJavaReadsIT.java renamed to integration/src/test/java/io/github/dfa1/vortex/integration/RustWritesJavaReadsIntegrationTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@
3232
import static org.assertj.core.api.Assertions.assertThat;
3333

3434
/// Cross-compatibility: Rust (JNI) writer → Java reader.
35-
class RustWritesJavaReadsIT {
35+
class RustWritesJavaReadsIntegrationTest {
3636

3737
static {
3838
NativeLoader.loadJni();

0 commit comments

Comments
 (0)