bump net.sourceforge.pmd:pmd-java from 7.1.0 to 7.11.0 (via #2903)

Author: dependabot[bot]

allure-generator/src/main/java/io/qameta/allure/history/HistoryPlugin.java

@@ -33,9 +33,8 @@
 import java.nio.file.Files;
 import java.nio.file.Path;
 import java.util.ArrayList;
-import java.util.Arrays;
+import java.util.EnumSet;
 import java.util.HashMap;
-import java.util.HashSet;
 import java.util.List;
 import java.util.Map;
 import java.util.Objects;
@@ -52,9 +51,7 @@
  */
 public class HistoryPlugin extends CommonJsonAggregator2 implements Reader {
 
-    private static final Set<Status> MARK_STATUSES = new HashSet<>(Arrays.asList(
-            Status.FAILED, Status.BROKEN, Status.PASSED
-    ));
+    private static final Set<Status> MARK_STATUSES = EnumSet.of(Status.FAILED, Status.BROKEN, Status.PASSED);
 
     private static final String HISTORY_BLOCK_NAME = "history";
     private static final String HISTORY_FILE_NAME = "history.json";

build.gradle.kts

@@ -94,7 +94,7 @@ subprojects {
             dependency("commons-beanutils:commons-beanutils:1.10.0")
             dependency("commons-io:commons-io:2.18.0")
             dependency("javax.xml.bind:jaxb-api:2.3.1")
-            dependency("net.sourceforge.pmd:pmd-java:7.1.0")
+            dependency("net.sourceforge.pmd:pmd-java:7.11.0")
             dependency("org.allurefw:allure1-model:1.0")
             dependency("org.apache.commons:commons-collections4:4.4")
             dependency("org.apache.commons:commons-lang3:3.17.0")

gradle/quality-configs/pmd/pmd.xml

@@ -15,12 +15,8 @@
     <exclude name="AccessorMethodGeneration"/>
     <exclude name="ArrayIsStoredDirectly"/>
     <exclude name="GuardLogStatement"/>
-    <exclude name="JUnitAssertionsShouldIncludeMessage"/>
-    <exclude name="JUnit4SuitesShouldUseSuiteAnnotation"/>
-    <exclude name="JUnitTestsShouldIncludeAssert"/>
-    <exclude name="JUnit4TestShouldUseBeforeAnnotation"/>
-    <exclude name="JUnit4TestShouldUseAfterAnnotation"/>
-    <exclude name="JUnit4TestShouldUseTestAnnotation"/>
+    <!--  Doesn't work with Java 8 switch clause  -->
+    <exclude name="ExhaustiveSwitchHasDefault"/>
   </rule>
 
 

plugins/behaviors-plugin/src/main/java/io/qameta/allure/behaviors/BehaviorsPlugin.java

@@ -33,7 +33,7 @@
 import java.util.Arrays;
 import java.util.Collection;
 import java.util.Comparator;
-import java.util.HashMap;
+import java.util.EnumMap;
 import java.util.List;
 import java.util.Map;
 import java.util.Objects;
@@ -121,7 +121,7 @@ private static class CsvExportAggregator extends CommonCsvExportAggregator2<CsvE
         protected List<CsvExportBehavior> getData(final List<LaunchResults> launchesResults) {
             final List<CsvExportBehavior> exportBehaviors = new ArrayList<>();
             launchesResults.stream().flatMap(launch -> launch.getResults().stream()).forEach(result -> {
-                final Map<LabelName, List<String>> epicFeatureStoryMap = new HashMap<>();
+                final Map<LabelName, List<String>> epicFeatureStoryMap = new EnumMap<>(LabelName.class);
                 Arrays.asList(LABEL_NAMES).forEach(
                         label -> epicFeatureStoryMap.put(label, result.findAllLabels(label))
                 );

plugins/xray-plugin/src/main/java/io/qameta/allure/xray/XrayTestRunExportPlugin.java

@@ -32,6 +32,7 @@
 import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.Collection;
+import java.util.EnumMap;
 import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
@@ -191,7 +192,7 @@ private void updateTestRunStatus(final JiraService jiraService,
     }
 
     private static Map<Status, String> getEnvStatusesMap() {
-        final Map<Status, String> statues = new HashMap<>();
+        final Map<Status, String> statues = new EnumMap<>(Status.class);
         getProperty(ALLURE_XRAY_STATUS_PASSED).ifPresent(value -> statues.put(Status.PASSED, value));
         getProperty(ALLURE_XRAY_STATUS_FAILED).ifPresent(value -> statues.put(Status.FAILED, value));
         getProperty(ALLURE_XRAY_STATUS_BROKEN).ifPresent(value -> statues.put(Status.BROKEN, value));
@@ -201,7 +202,7 @@ private static Map<Status, String> getEnvStatusesMap() {
     }
 
     private static Map<Status, String> getDefaultStatusesMap() {
-        final Map<Status, String> statues = new HashMap<>();
+        final Map<Status, String> statues = new EnumMap<>(Status.class);
         statues.put(Status.PASSED, XRAY_STATUS_PASS);
         statues.put(Status.FAILED, XRAY_STATUS_FAIL);
         statues.put(Status.BROKEN, XRAY_STATUS_FAIL);