[activemq-artemis-native] 01/02: fail the build when the tests fail, ensure e.g CI jobs actually signal issues

[ro...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

robbie pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/activemq-artemis-native.git

commit 3f85794cb5bf487bd57b4a7d9a2448196ca303c8
Author: Robbie Gemmell <ro...@apache.org>
AuthorDate: Tue Jul 12 12:18:36 2022 +0100

    fail the build when the tests fail, ensure e.g CI jobs actually signal issues
---
 pom.xml | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/pom.xml b/pom.xml
index 5e971d9..f21a078 100644
--- a/pom.xml
+++ b/pom.xml
@@ -56,10 +56,7 @@
         </activemq-surefire-argline>
         <activemq.basedir>${project.basedir}</activemq.basedir>
 
-        <!-- Ignore failed tests by default because there are "known" failures in the full test-suite.
-             This will be set to false for the "fast-tests" profile as none of those tests should fail. -->
-        <testFailureIgnore>true</testFailureIgnore>
-
+        <maven.test.failure.ignore>false</maven.test.failure.ignore>
         <maven.test.redirectTestOutputToFile>false</maven.test.redirectTestOutputToFile>
     </properties>
 
@@ -277,7 +274,7 @@
                     <artifactId>maven-surefire-plugin</artifactId>
                     <configuration>
                         <forkMode>once</forkMode>
-                        <testFailureIgnore>${testFailureIgnore}</testFailureIgnore>
+                        <testFailureIgnore>${maven.test.failure.ignore}</testFailureIgnore>
                         <runOrder>alphabetical</runOrder>
                         <redirectTestOutputToFile>${maven.test.redirectTestOutputToFile}</redirectTestOutputToFile>
                         <argLine>${activemq-surefire-argline}</argLine>