feat: @PhpCsFixer ruleset - enable php_unit_data_provider_static (#…

…7685)
PHP-CS-Fixer · Jan 20, 2024 · 27c0ba8 · 27c0ba8
1 parent 48a62f0
commit 27c0ba8
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 1 deletion.
diff --git a/doc/ruleSets/PhpCsFixerRisky.rst b/doc/ruleSets/PhpCsFixerRisky.rst
@@ -31,6 +31,10 @@ Rules
 - `no_unset_on_property <./../rules/language_construct/no_unset_on_property.rst>`_
 - `php_unit_data_provider_name <./../rules/php_unit/php_unit_data_provider_name.rst>`_
 - `php_unit_data_provider_return_type <./../rules/php_unit/php_unit_data_provider_return_type.rst>`_
+- `php_unit_data_provider_static <./../rules/php_unit/php_unit_data_provider_static.rst>`_ with config:
+
+  ``['force' => true]``
+
 - `php_unit_strict <./../rules/php_unit/php_unit_strict.rst>`_
 - `php_unit_test_case_static_method_calls <./../rules/php_unit/php_unit_test_case_static_method_calls.rst>`_ with config:
 

diff --git a/doc/rules/php_unit/php_unit_data_provider_static.rst b/doc/rules/php_unit/php_unit_data_provider_static.rst
@@ -94,12 +94,16 @@ With configuration: ``['force' => false]``.
 Rule sets
 ---------
 
-The rule is part of the following rule set:
+The rule is part of the following rule sets:
 
 - `@PHPUnit100Migration:risky <./../../ruleSets/PHPUnit100MigrationRisky.rst>`_ with config:
 
   ``['force' => true]``
 
+- `@PhpCsFixer:risky <./../../ruleSets/PhpCsFixerRisky.rst>`_ with config:
+
+  ``['force' => true]``
+
 
 References
 ----------

diff --git a/src/RuleSet/Sets/PhpCsFixerRiskySet.php b/src/RuleSet/Sets/PhpCsFixerRiskySet.php
@@ -51,6 +51,7 @@ public function getRules(): array
             'no_unset_on_property' => true,
             'php_unit_data_provider_name' => true,
             'php_unit_data_provider_return_type' => true,
+            'php_unit_data_provider_static' => ['force' => true],
             'php_unit_strict' => true,
             'php_unit_test_case_static_method_calls' => ['call_type' => 'self'],
             'static_lambda' => true,