chore: migrate to mojotest - PmdReportTest

Signed-off-by: Sandra Parsick <sandra@parsick.dev>

Author: sparsick

src/test/java/org/apache/maven/plugins/pmd/CpdReportTest.java

@@ -18,6 +18,19 @@
  */
 package org.apache.maven.plugins.pmd;
 
+import javax.inject.Inject;
+import javax.xml.parsers.DocumentBuilder;
+import javax.xml.parsers.DocumentBuilderFactory;
+
+import java.io.File;
+import java.io.IOException;
+import java.nio.charset.StandardCharsets;
+import java.nio.file.Files;
+import java.nio.file.Path;
+import java.nio.file.Paths;
+import java.util.Collections;
+import java.util.Locale;
+
 import org.apache.commons.lang3.StringUtils;
 import org.apache.maven.api.plugin.testing.Basedir;
 import org.apache.maven.api.plugin.testing.InjectMojo;
@@ -41,18 +54,6 @@
 import org.mockito.Mockito;
 import org.w3c.dom.Document;
 
-import javax.inject.Inject;
-import javax.xml.parsers.DocumentBuilder;
-import javax.xml.parsers.DocumentBuilderFactory;
-import java.io.File;
-import java.io.IOException;
-import java.nio.charset.StandardCharsets;
-import java.nio.file.Files;
-import java.nio.file.Path;
-import java.nio.file.Paths;
-import java.util.Collections;
-import java.util.Locale;
-
 import static org.apache.maven.api.plugin.testing.MojoExtension.getBasedir;
 import static org.junit.jupiter.api.Assertions.assertEquals;
 import static org.junit.jupiter.api.Assertions.assertFalse;
@@ -79,7 +80,6 @@ public class CpdReportTest {
     @Inject
     private MojoExecution mojoExecution;
 
-
     /**
      * Checks whether the string <code>contained</code> is contained in
      * the given <code>text</code>, ignoring case.
@@ -96,7 +96,7 @@ public static boolean lowerCaseContains(String text, String contains) {
      * {@inheritDoc}
      */
     @BeforeEach
-    public void setUp() throws Exception {
+    public void setUp() {
         ArtifactRepository localRepo = Mockito.mock(ArtifactRepository.class);
         Mockito.when(localRepo.getBasedir())
                 .thenReturn(new File(PlexusExtension.getBasedir(), "target/local-repo").getAbsolutePath());
@@ -208,8 +208,9 @@ public void testInvalidFormat(AbstractPmdReport mojo) {
         try {
             mojo.execute();
 
-            fail("MojoExecutionException must be thrown");
-        } catch (MojoExecutionException e) {
+            // TODO should have more specific exception
+            fail("RuntimeException must be thrown");
+        } catch (RuntimeException e) {
             assertMavenReportException("Can't find CPD custom format xhtml", e);
         }
     }
@@ -396,12 +397,10 @@ private static void assertReportContains(String expectedMessage) throws IOExcept
                 report.contains(expectedMessage), "Expected '" + expectedMessage + "' in cpd.xml, but was:\n" + report);
     }
 
-
     /**
      * Read the contents of the specified file into a string.
      */
     protected String readFile(File file) throws IOException {
         return new String(Files.readAllBytes(file.toPath()));
     }
-
 }