it tests - wip

Author: andrzej-janczak

integration-tests/config-discover/expected/codacy.yaml

@@ -7,6 +7,6 @@ tools:
     - [email protected]
     - [email protected]
     - [email protected]
-    - pmd@6.55.0
+    - pmd@7.11.0
     - [email protected]
     - [email protected]

integration-tests/config-discover/expected/tools-configs/ruleset.xml

@@ -1,9 +1,9 @@
-<?xml version="1.0"?>
-<ruleset name="Codacy PMD Ruleset"
-xmlns="http://pmd.sourceforge.net/ruleset/2.0.0"
+<?xml version="1.0" encoding="UTF-8"?>
+<ruleset name="Codacy PMD 7 Ruleset"
+xmlns="https://pmd.github.io/ruleset/2.0.0"
 xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
-xsi:schemaLocation="http://pmd.sourceforge.net/ruleset/2.0.0 https://pmd.sourceforge.io/ruleset_2_0_0.xsd">
-<description>Codacy PMD Ruleset</description>
+xsi:schemaLocation="https://pmd.github.io/ruleset/2.0.0 https://pmd.github.io/schemas/pmd-7.0.0.xsd">
+<description>Codacy PMD 7 Ruleset</description>
     <rule ref="category/ecmascript/codestyle.xml/AssignmentInOperand">
         <properties>
             <property name="allowFor" value="false"/>
@@ -47,7 +47,7 @@ xsi:schemaLocation="http://pmd.sourceforge.net/ruleset/2.0.0 https://pmd.sourcef
     </rule>
     <rule ref="category/java/errorprone.xml/MissingStaticMethodInNonInstantiatableClass">
         <properties>
-            <property name="annotations" value="org.springframework.beans.factory.annotation.Autowired,javax.inject.Inject"/>
+            <property name="annotations" value="org.springframework.beans.factory.annotation.Autowired,javax.inject.Inject,com.google.inject.Inject,lombok.Builder"/>
         </properties>
     </rule>
     <rule ref="category/plsql/design.xml/TooManyMethods">
@@ -60,11 +60,6 @@ xsi:schemaLocation="http://pmd.sourceforge.net/ruleset/2.0.0 https://pmd.sourcef
             <property name="validTypes" value="pom,jar,maven-plugin,ejb,war,ear,rar,par"/>
         </properties>
     </rule>
-    <rule ref="category/vm/design.xml/ExcessiveTemplateLength">
-        <properties>
-            <property name="minimum" value="1000"/>
-        </properties>
-    </rule>
     <rule ref="category/xsl/performance.xml/AvoidAxisNavigation">
         <properties>
             <property name="checkSelfDescendantAbreviation" value="false"/>

integration-tests/run.sh

@@ -485,6 +485,16 @@ run_init_test() {
   [ -d "$test_dir" ] || { echo "❌ Test directory does not exist: $test_dir"; exit 1; }
 
   cd "$test_dir" || exit 1
+  
+  # Set up cleanup trap to ensure .codacy is always removed
+  cleanup_codacy() {
+    if [ -d ".codacy" ]; then
+      rm -rf .codacy
+      echo "🧹 Cleaned up .codacy directory"
+    fi
+  }
+  trap cleanup_codacy EXIT
+  
   rm -rf .codacy
 
   if [ "$use_token" = "true" ]; then
@@ -495,8 +505,11 @@ run_init_test() {
   fi
 
   compare_files "expected" ".codacy" "Test $test_name"
+  
   echo "✅ Test $test_name completed successfully"
   echo "----------------------------------------"
+  
+  # Cleanup will happen automatically via trap
 }
 
 run_config_discover_test() {
@@ -507,6 +520,16 @@ run_config_discover_test() {
   [ -d "$test_dir" ] || { echo "❌ Test directory does not exist: $test_dir"; exit 1; }
 
   cd "$test_dir" || exit 1
+  
+  # Set up cleanup trap to ensure .codacy is always removed
+  cleanup_codacy() {
+    if [ -d ".codacy" ]; then
+      rm -rf .codacy
+      echo "🧹 Cleaned up .codacy directory"
+    fi
+  }
+  trap cleanup_codacy EXIT
+  
   # Clean up previous test results
   rm -rf .codacy
 
@@ -553,8 +576,11 @@ EOF
   "$CLI_PATH" config discover .
 
   compare_files "expected" ".codacy" "Test $test_name"
+  
   echo "✅ Test $test_name completed successfully"
   echo "----------------------------------------"
+  
+  # Cleanup will happen automatically via trap
 }
 
 # Run all tests