This is an automated email from the ASF dual-hosted git repository. leerho pushed a commit to branch fix-test-failures in repository https://gitbox.apache.org/repos/asf/datasketches-java.git
commit 033178eafc8ce7759c42c101f11633a5d5330fbe Author: Lee Rhodes <[email protected]> AuthorDate: Sat Sep 26 14:27:18 2026 -0700 Fix pom so cross-language test groups are excluded by default The composite testng.* group properties were concatenated without commas, so surefire's excludedGroups only matched generate_java_files and check_java_files; C++/Go/Rust/CL-binary/historical groups ran on every 'mvn test'. Also point the check_CL_binary_files and check_cpp_historical_files profiles at the renamed properties. Co-Authored-By: Claude Opus 5.5 <[email protected]> --- pom.xml | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/pom.xml b/pom.xml index bdd818a48..40fa32fe6 100644 --- a/pom.xml +++ b/pom.xml @@ -146,9 +146,9 @@ under the License. <!-- Language Groups --> <testng.check_java_cpp>${testng.generate_java_files},${testng.check_java_files},${testng.check_cpp_files}</testng.check_java_cpp> <testng.check_go_rust>${testng.check_go_files},${testng.check_rust_files}</testng.check_go_rust> - <testng.check_specials>${testng.check_CL_binaries}${testng.check_cpp_historicals}</testng.check_specials> + <testng.check_specials>${testng.check_CL_binaries},${testng.check_cpp_historicals}</testng.check_specials> <!-- All Languages & types --> - <testng.all_groups>${testng.check_java_cpp}${testng.check_go_rust}${testng.check_specials}</testng.all_groups> + <testng.all_groups>${testng.check_java_cpp},${testng.check_go_rust},${testng.check_specials}</testng.all_groups> <test.data.root>${project.basedir}</test.data.root> <print>false</print> <!-- diagnostic printing of file names read/written/not-found in UtilityIO; override with -Dprint=true --> @@ -870,7 +870,7 @@ under the License. <groupId>org.apache.maven.plugins</groupId> <artifactId>maven-surefire-plugin</artifactId> <configuration> - <groups>${testng.check_CL_binary_files}</groups> + <groups>${testng.check_CL_binaries}</groups> <excludedGroups combine.self="override"></excludedGroups> </configuration> </plugin> @@ -886,7 +886,7 @@ under the License. <groupId>org.apache.maven.plugins</groupId> <artifactId>maven-surefire-plugin</artifactId> <configuration> - <groups>${testng.check_cpp_historical_files}</groups> + <groups>${testng.check_cpp_historicals}</groups> <excludedGroups combine.self="override"></excludedGroups> </configuration> </plugin> --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
