This is an automated email from the ASF dual-hosted git repository.
rexxiong pushed a commit to branch branch-0.5
in repository https://gitbox.apache.org/repos/asf/celeborn.git
The following commit(s) were added to refs/heads/branch-0.5 by this push:
new 3f875d4b6 [CELEBORN-81][FOLLOWUP] Correct scala test plugin args
3f875d4b6 is described below
commit 3f875d4b67c54b6c35c4061185a17bd4a6cca38c
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]>
(cherry picked from commit e5ccc9b623cbd3c52b4f09f960d5f8f28e129066)
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 c54294fa2..8e0c4afe3 100644
--- a/pom.xml
+++ b/pom.xml
@@ -837,7 +837,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.
