This is an automated email from the ASF dual-hosted git repository.
rexxiong pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/celeborn.git
The following commit(s) were added to refs/heads/main by this push:
new e5ccc9b62 [CELEBORN-81][FOLLOWUP] Correct scala test plugin args
e5ccc9b62 is described below
commit e5ccc9b623cbd3c52b4f09f960d5f8f28e129066
Author: gaoyajun02 <[email protected]>
AuthorDate: Wed Apr 9 10:23:33 2025 +0800
[CELEBORN-81][FOLLOWUP] Correct scala test plugin args
### What changes were proposed in this pull request?
Modifies the maven configuration to properly pass Jacoco's argLine to
ScalaTest plugin, enabling code coverage measurement for Scala tests.
### Why are the changes needed?
Previously Scala tests were not properly included in code coverage reports
### Does this PR introduce _any_ user-facing change?
No
### How was this patch tested?
CI
Closes #3205 from gaoyajun02/ci.
Authored-by: gaoyajun02 <[email protected]>
Signed-off-by: Shuang <[email protected]>
---
pom.xml | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/pom.xml b/pom.xml
index 959aeb245..e523c2eae 100644
--- a/pom.xml
+++ b/pom.xml
@@ -915,7 +915,7 @@
<version>${maven.plugin.scalatest.version}</version>
<configuration>
<reportsDirectory>${project.build.directory}/surefire-reports</reportsDirectory>
- <argLine>-ea -Xmx4g -Xss4m -XX:MaxMetaspaceSize=2g
-XX:ReservedCodeCacheSize=128m ${extraJavaTestArgs}
-Dio.netty.tryReflectionSetAccessible=true</argLine>
+ <argLine>${argLine} -ea -Xmx4g -Xss4m -XX:MaxMetaspaceSize=2g
-XX:ReservedCodeCacheSize=128m ${extraJavaTestArgs}
-Dio.netty.tryReflectionSetAccessible=true</argLine>
<filereports>TestSuite.txt</filereports>
<!--
the plugin does not support always now, we must be careful w/
singletons.
