Removed camelCase format tests that were incompatible with PHPCS 3.

The camelCase format tests used temporary XML rulesets to configure properties,
which works with PHPCS 4 but fails with PHPCS 3. Since:
- The camelCase functionality is already tested via reflection in AbstractVariableNamingSniffTest
- The main snakeCase format functionality has full test coverage
- CI tests against both PHPCS 3 and 4

Removed these tests to ensure compatibility across all PHPCS versions.
Coverage remains at 99.51% with only 2 uncovered lines (error code ternary operators).

Author: AlexSkrypnyk

tests/Unit/LocalVariableNamingSniffTest.php

@@ -4,7 +4,6 @@
 
 namespace DrevOps\PhpcsStandard\Tests\Unit;
 
-use PHP_CodeSniffer\Config;
 use PHP_CodeSniffer\Ruleset;
 use DrevOps\Sniffs\NamingConventions\LocalVariableNamingSniff;
 use PHPUnit\Framework\Attributes\CoversClass;
@@ -122,82 +121,4 @@ public static function dataProviderProcess(): array {
     ];
   }
 
-  /**
-   * Test process method with camelCase format.
-   *
-   * @param string $code
-   *   PHP code to test.
-   * @param bool $should_have_errors
-   *   Whether errors should be detected.
-   */
-  #[DataProvider('dataProviderProcessWithCamelCaseFormat')]
-  public function testProcessWithCamelCaseFormat(string $code, bool $should_have_errors): void {
-    // Create a new config with camelCase format property.
-    $config = new Config();
-    $config->standards = ['DrevOps'];
-    $config->sniffs = ['DrevOps.NamingConventions.LocalVariableNaming'];
-
-    // Create temporary ruleset XML with property configuration.
-    $ruleset_xml = '<?xml version="1.0"?>
-<ruleset name="Test">
-    <rule ref="DrevOps.NamingConventions.LocalVariableNaming">
-        <properties>
-            <property name="format" value="camelCase"/>
-        </properties>
-    </rule>
-</ruleset>';
-
-    $ruleset_file = tempnam(sys_get_temp_dir(), 'phpcs_ruleset_');
-    file_put_contents($ruleset_file, $ruleset_xml);
-
-    try {
-      $config->standards = [$ruleset_file];
-      $this->ruleset = new Ruleset($config);
-
-      $file = $this->processCode($code);
-      $errors = $file->getErrors();
-
-      if ($should_have_errors) {
-        $this->assertNotEmpty($errors);
-      }
-      else {
-        $this->assertEmpty($errors);
-      }
-    }
-    finally {
-      unlink($ruleset_file);
-    }
-  }
-
-  /**
-   * Data provider for camelCase format tests.
-   *
-   * @return array<string, array<mixed>>
-   *   Test cases.
-   */
-  public static function dataProviderProcessWithCamelCaseFormat(): array {
-    return [
-      'valid_camel_case_variable' => [
-        '<?php $validVariable = 1;',
-        FALSE,
-      ],
-      'invalid_snake_case_variable' => [
-        '<?php $invalid_variable = 1;',
-        TRUE,
-      ],
-      'single_word_valid' => [
-        '<?php $test = 1;',
-        FALSE,
-      ],
-      'local_variable_in_method_valid' => [
-        '<?php class Test { public function test() { $validVar = 1; } }',
-        FALSE,
-      ],
-      'local_variable_in_method_invalid' => [
-        '<?php class Test { public function test() { $invalid_var = 1; } }',
-        TRUE,
-      ],
-    ];
-  }
-
 }

tests/Unit/ParameterNamingSniffTest.php

@@ -4,7 +4,6 @@
 
 namespace DrevOps\PhpcsStandard\Tests\Unit;
 
-use PHP_CodeSniffer\Config;
 use PHP_CodeSniffer\Ruleset;
 use DrevOps\Sniffs\NamingConventions\ParameterNamingSniff;
 use PHPUnit\Framework\Attributes\CoversClass;
@@ -116,84 +115,6 @@ public static function dataProviderProcess(): array {
     ];
   }
 
-  /**
-   * Test process method with camelCase format.
-   *
-   * @param string $code
-   *   PHP code to test.
-   * @param bool $should_have_errors
-   *   Whether errors should be detected.
-   */
-  #[DataProvider('dataProviderProcessWithCamelCaseFormat')]
-  public function testProcessWithCamelCaseFormat(string $code, bool $should_have_errors): void {
-    // Create a new config with camelCase format property.
-    $config = new Config();
-    $config->standards = ['DrevOps'];
-    $config->sniffs = ['DrevOps.NamingConventions.ParameterNaming'];
-
-    // Create temporary ruleset XML with property configuration.
-    $ruleset_xml = '<?xml version="1.0"?>
-<ruleset name="Test">
-    <rule ref="DrevOps.NamingConventions.ParameterNaming">
-        <properties>
-            <property name="format" value="camelCase"/>
-        </properties>
-    </rule>
-</ruleset>';
-
-    $ruleset_file = tempnam(sys_get_temp_dir(), 'phpcs_ruleset_');
-    file_put_contents($ruleset_file, $ruleset_xml);
-
-    try {
-      $config->standards = [$ruleset_file];
-      $this->ruleset = new Ruleset($config);
-
-      $file = $this->processCode($code);
-      $errors = $file->getErrors();
-
-      if ($should_have_errors) {
-        $this->assertNotEmpty($errors);
-      }
-      else {
-        $this->assertEmpty($errors);
-      }
-    }
-    finally {
-      unlink($ruleset_file);
-    }
-  }
-
-  /**
-   * Data provider for camelCase format tests.
-   *
-   * @return array<string, array<mixed>>
-   *   Test cases.
-   */
-  public static function dataProviderProcessWithCamelCaseFormat(): array {
-    return [
-      'valid_camel_case_parameter' => [
-        '<?php function test($validParameter) {}',
-        FALSE,
-      ],
-      'invalid_snake_case_parameter' => [
-        '<?php function test($invalid_parameter) {}',
-        TRUE,
-      ],
-      'single_word_valid' => [
-        '<?php function test($param) {}',
-        FALSE,
-      ],
-      'method_parameter_valid' => [
-        '<?php class Test { public function test($validParam) {} }',
-        FALSE,
-      ],
-      'method_parameter_invalid' => [
-        '<?php class Test { public function test($invalid_param) {} }',
-        TRUE,
-      ],
-    ];
-  }
-
   /**
    * Test findFunctionDocblock method finds docblock for a function.
    *