Merge branch 'master' into doc

Author: selutin99

.github/workflows/codecov.yml

@@ -16,7 +16,7 @@ jobs:
       - uses: actions/setup-java@v4
         with:
           distribution: 'temurin'
-          java-version: 11
+          java-version: 21
       - uses: actions/cache@v4
         with:
           path: ~/.m2/repository

README.md

@@ -266,8 +266,9 @@
           <plugin>
             <groupId>com.qulice</groupId>
             <artifactId>qulice-maven-plugin</artifactId>
-            <version>0.22.0</version>
+            <version>0.24.0</version>
             <configuration>
+              <license>file:${basedir}/LICENSE.txt</license>
               <excludes combine.children="append">
                 <exclude>checkstyle:/src/site/resources/.*</exclude>
                 <exclude>checkstyle:/src/test/resources/.*</exclude>

pom.xml

@@ -105,10 +105,12 @@ public enum Metrics {
      * Indicates whether parameters should be included for this metric.
      */
     private final boolean parameters;
+
     /**
      * Mean value for the metric, if applicable.
      */
     private final Double mean;
+
     /**
      * Standard deviation for the metric, if applicable.
      */

src/main/java/org/jpeek/Metrics.java

@@ -21,6 +21,7 @@
 final class ReportWithStatisticsTest {
 
     @Test
+    @SuppressWarnings("PMD.CloseResource")
     void createsXml() {
         final XML xml = new ReportWithStatistics(
             new XMLDocument("<metric/>")
@@ -47,5 +48,4 @@ void createsXml() {
             XhtmlMatchers.hasXPaths("/metric/statistics[total='0']")
         ).affirm();
     }
-
 }

src/test/java/org/jpeek/ReportWithStatisticsTest.java

@@ -66,19 +66,19 @@ void buildsMethodsAsNodes() {
             new AllOf<Iterable<Node>>(
                 new ListOf<>(
                     new HasValuesMatching<>(
-                        node -> node.name().equals(XmlGraphTest.METHOD_ONE)
+                        node -> XmlGraphTest.METHOD_ONE.equals(node.name())
                     ),
                     new HasValuesMatching<>(
-                        node -> node.name().equals(XmlGraphTest.METHOD_TWO)
+                        node -> XmlGraphTest.METHOD_TWO.equals(node.name())
                     ),
                     new HasValuesMatching<>(
-                        node -> node.name().equals(XmlGraphTest.METHOD_THREE)
+                        node -> XmlGraphTest.METHOD_THREE.equals(node.name())
                     ),
                     new HasValuesMatching<>(
-                        node -> node.name().equals(XmlGraphTest.METHOD_FOUR)
+                        node -> XmlGraphTest.METHOD_FOUR.equals(node.name())
                     ),
                     new HasValuesMatching<>(
-                        node -> node.name().equals(XmlGraphTest.METHOD_FIVE)
+                        node -> XmlGraphTest.METHOD_FIVE.equals(node.name())
                     )
                 )
             )

src/test/java/org/jpeek/graph/XmlGraphTest.java

@@ -69,11 +69,11 @@ void oneComponentInClassTest() throws Exception {
 
     /**
      * Check ccm metric for mixed usage: attribute usage, methods calls.
+     * @throws Exception
      * @todo #522:30min there is a 4th step for incorrect calculation: nc
      *  in case of calling one method from another because of
      *  `xsl:if test="$method/ops/op/text()[. = $other/ops/op/text()]"`
      *  method name is not used for creating edge.
-     * @throws Exception
      */
     @Test
     void mixedCallsInClassTest() throws Exception {

src/test/java/org/jpeek/metrics/CcmTest.java

@@ -27,6 +27,7 @@
 final class AsyncReportsTest {
 
     @Test
+    @SuppressWarnings("PMD.CloseResource")
     void rendersOneReport() throws Exception {
         final ExecutorService service = Executors.newSingleThreadExecutor();
         final BiFunc<String, String, Func<String, Response>> bifunc = new AsyncReports(
@@ -54,5 +55,4 @@ void rendersOneReport() throws Exception {
         ).affirm();
         service.shutdownNow();
     }
-
 }