(maven-scm) branch master updated: Make asserts meaningful (#232)

[elharo]

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

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


The following commit(s) were added to refs/heads/master by this push:
     new 14872511e Make asserts meaningful (#232)
14872511e is described below

commit 14872511e5d85deec1ccb67af6291c264af1c7fd
Author: Elliotte Rusty Harold <elh...@users.noreply.github.com>
AuthorDate: Thu Jan 9 20:53:13 2025 +0000

    Make asserts meaningful (#232)
---
 .../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 f3d00cbe2..2760211ca 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());
         }
     }
 }