(maven-scm) 01/01: Make asserts meaningful

[elharo]

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

elharo pushed a commit to branch true
in repository https://gitbox.apache.org/repos/asf/maven-scm.git

commit 9853f5e598c80cf06ba374e8490adc06838fbc87
Author: Elliotte Rusty Harold <elh...@ibiblio.org>
AuthorDate: Thu Dec 12 08:26:10 2024 -0500

    Make asserts meaningful
---
 .../src/test/java/org/apache/maven/scm/plugin/ChangeLogMojoTest.java  | 4 ++--
 .../src/test/java/org/apache/maven/scm/plugin/CheckoutMojoTest.java   | 2 +-
 .../src/test/java/org/apache/maven/scm/plugin/UntagMojoTest.java      | 2 +-
 .../src/test/java/org/apache/maven/scm/plugin/ValidateMojoTest.java   | 2 +-
 4 files changed, 5 insertions(+), 5 deletions(-)

diff --git 
a/maven-scm-plugin/src/test/java/org/apache/maven/scm/plugin/ChangeLogMojoTest.java
 
b/maven-scm-plugin/src/test/java/org/apache/maven/scm/plugin/ChangeLogMojoTest.java
index fa66f0284..e1e2e38f2 100644
--- 
a/maven-scm-plugin/src/test/java/org/apache/maven/scm/plugin/ChangeLogMojoTest.java
+++ 
b/maven-scm-plugin/src/test/java/org/apache/maven/scm/plugin/ChangeLogMojoTest.java
@@ -103,7 +103,7 @@ public class ChangeLogMojoTest extends 
AbstractJUnit4MojoTestCase {
 
             fail("mojo execution must fail.");
         } catch (MojoExecutionException e) {
-            assertTrue(true);
+            assertNotNull(e.getMessage());
         }
     }
 
@@ -126,7 +126,7 @@ public class ChangeLogMojoTest extends 
AbstractJUnit4MojoTestCase {
 
             fail("mojo execution must fail.");
         } catch (MojoExecutionException e) {
-            assertTrue(true);
+            assertNotNull(e.getMessage());
         }
     }
 }
diff --git 
a/maven-scm-plugin/src/test/java/org/apache/maven/scm/plugin/CheckoutMojoTest.java
 
b/maven-scm-plugin/src/test/java/org/apache/maven/scm/plugin/CheckoutMojoTest.java
index 354d6eeed..870b7c251 100644
--- 
a/maven-scm-plugin/src/test/java/org/apache/maven/scm/plugin/CheckoutMojoTest.java
+++ 
b/maven-scm-plugin/src/test/java/org/apache/maven/scm/plugin/CheckoutMojoTest.java
@@ -107,7 +107,7 @@ public class CheckoutMojoTest extends 
AbstractJUnit4MojoTestCase {
 
             fail("mojo execution must fail.");
         } catch (MojoExecutionException e) {
-            assertTrue(true);
+            assertNotNull(e.getMessage());
         }
     }
 
diff --git 
a/maven-scm-plugin/src/test/java/org/apache/maven/scm/plugin/UntagMojoTest.java 
b/maven-scm-plugin/src/test/java/org/apache/maven/scm/plugin/UntagMojoTest.java
index ddae0c366..c47ab595d 100644
--- 
a/maven-scm-plugin/src/test/java/org/apache/maven/scm/plugin/UntagMojoTest.java
+++ 
b/maven-scm-plugin/src/test/java/org/apache/maven/scm/plugin/UntagMojoTest.java
@@ -102,7 +102,7 @@ public class UntagMojoTest extends 
AbstractJUnit4MojoTestCase {
 
             fail("mojo execution must fail.");
         } catch (MojoExecutionException e) {
-            assertTrue(true);
+            assertNotNull(e.getMessage());
         }
     }
 
diff --git 
a/maven-scm-plugin/src/test/java/org/apache/maven/scm/plugin/ValidateMojoTest.java
 
b/maven-scm-plugin/src/test/java/org/apache/maven/scm/plugin/ValidateMojoTest.java
index 53cc347ad..5289b7cb8 100644
--- 
a/maven-scm-plugin/src/test/java/org/apache/maven/scm/plugin/ValidateMojoTest.java
+++ 
b/maven-scm-plugin/src/test/java/org/apache/maven/scm/plugin/ValidateMojoTest.java
@@ -52,7 +52,7 @@ public class ValidateMojoTest extends 
AbstractJUnit4MojoTestCase {
 
             fail("mojo execution must fail.");
         } catch (MojoExecutionException e) {
-            assertTrue(true);
+            assertNotNull(e.getMessage());
         }
     }
 }