(maven-dist-tool) branch master updated: JUnit Jupiter best practices (#138)

[slachiewicz]

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

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


The following commit(s) were added to refs/heads/master by this push:
     new 6206c52  JUnit Jupiter best practices (#138)
6206c52 is described below

commit 6206c5202da951aaab5b4f631dbcb427cfe80fa3
Author: Sylwester Lachiewicz <slachiew...@apache.org>
AuthorDate: Tue Nov 4 03:03:30 2025 +0100

    JUnit Jupiter best practices (#138)
    
    
    
    Co-authored-by: Moderne <t...@moderne.io>
---
 src/test/java/org/apache/maven/dist/tools/CheckReportResultIT.java    | 3 +--
 .../maven/dist/tools/committers/MavenCommittersRepositoryTest.java    | 2 +-
 src/test/java/org/apache/maven/dist/tools/site/SiteReportTest.java    | 4 ++--
 src/test/java/org/apache/maven/dist/tools/source/PatternTest.java     | 2 +-
 4 files changed, 5 insertions(+), 6 deletions(-)

diff --git a/src/test/java/org/apache/maven/dist/tools/CheckReportResultIT.java 
b/src/test/java/org/apache/maven/dist/tools/CheckReportResultIT.java
index 752fb7b..3cd498a 100644
--- a/src/test/java/org/apache/maven/dist/tools/CheckReportResultIT.java
+++ b/src/test/java/org/apache/maven/dist/tools/CheckReportResultIT.java
@@ -19,7 +19,6 @@
 package org.apache.maven.dist.tools;
 
 import java.io.File;
-import java.io.IOException;
 import java.nio.file.Files;
 import java.nio.file.Path;
 import java.nio.file.Paths;
@@ -40,7 +39,7 @@ class CheckReportResultIT {
 
     @ParameterizedTest
     
@FieldSource("org.apache.maven.dist.tools.DistCheckErrorsReport#FAILURES_FILENAMES")
-    void shouldBeNoErrors(Path logFile) throws IOException {
+    void shouldBeNoErrors(Path logFile) throws Exception {
         Path logPath = ERROR_LOGS_PATH.resolve(logFile);
         if (Files.isRegularFile(logPath)) {
             List<String> lines = Files.readAllLines(logPath);
diff --git 
a/src/test/java/org/apache/maven/dist/tools/committers/MavenCommittersRepositoryTest.java
 
b/src/test/java/org/apache/maven/dist/tools/committers/MavenCommittersRepositoryTest.java
index 9b9625d..ae82798 100644
--- 
a/src/test/java/org/apache/maven/dist/tools/committers/MavenCommittersRepositoryTest.java
+++ 
b/src/test/java/org/apache/maven/dist/tools/committers/MavenCommittersRepositoryTest.java
@@ -66,7 +66,7 @@ class MavenCommittersRepositoryTest {
             """;
 
     @Test
-    void testDataLoad(WireMockRuntimeInfo wireMockRuntimeInfo) {
+    void dataLoad(WireMockRuntimeInfo wireMockRuntimeInfo) {
 
         stubFor(get("/json/foundation/groups.json")
                 .willReturn(aResponse().withStatus(200).withBody(GROUP)));
diff --git a/src/test/java/org/apache/maven/dist/tools/site/SiteReportTest.java 
b/src/test/java/org/apache/maven/dist/tools/site/SiteReportTest.java
index 778899c..39b0621 100644
--- a/src/test/java/org/apache/maven/dist/tools/site/SiteReportTest.java
+++ b/src/test/java/org/apache/maven/dist/tools/site/SiteReportTest.java
@@ -45,7 +45,7 @@ class SiteReportTest {
      * @throws IOException if the resource cannot be read
      */
     @Test
-    public void testFluidoSkin() throws IOException {
+    void fluidoSkin() throws Exception {
         String comment = readDocument("fluido.html");
         assertTrue(comment.contains("Fluido"));
     }
@@ -56,7 +56,7 @@ class SiteReportTest {
      * @throws IOException if the resource cannot be read
      */
     @Test
-    public void testStylusRightSkin() throws IOException {
+    void stylusRightSkin() throws Exception {
         String comment = readDocument("stylus-right.html");
         assertTrue(comment.contains("Stylus"));
     }
diff --git a/src/test/java/org/apache/maven/dist/tools/source/PatternTest.java 
b/src/test/java/org/apache/maven/dist/tools/source/PatternTest.java
index f5112d5..96ed626 100644
--- a/src/test/java/org/apache/maven/dist/tools/source/PatternTest.java
+++ b/src/test/java/org/apache/maven/dist/tools/source/PatternTest.java
@@ -34,7 +34,7 @@ class PatternTest {
      * Test of getGroupId method, of class ConfigurationLineInfo.
      */
     @Test
-    void testGetGroupId() {
+    void getGroupId() {
         String q = 
DistCheckSourceReleaseReport.getSourceReleasePattern("doxia");
 
         assertTrue("doxia-1.4-source-release.zip.asc".matches(q));