Tests: as of PHPUnit 10 data provider methods should be static

Per the PHPUnit manual: "A data provider method must be public and static" (https://docs.phpunit.de/en/12.0/writing-tests-for-phpunit.html).

A similar change was done in the PHPCS repository: PHPCSStandards/PHP_CodeSniffer#255

Tests will continue to work when running with PHPUnit < 10.

Author: rodrigoprimo

Tests/DocsXsd/DocsXsdTest.php

@@ -60,7 +60,7 @@ public function testXsdValidationPassedWithValidXml($fixtureFile)
      *
      * @return array
      */
-    public function dataValidXsd()
+    public static function dataValidXsd()
     {
         return [
             'Valid docs example with single standard in the file' => [
@@ -110,7 +110,7 @@ public function testXsdValidationFailsForInvalidXml($fixtureFile, $expectedStdOu
      *
      * @return array
      */
-    public function dataInvalidXsd()
+    public static function dataInvalidXsd()
     {
         return [
             'Title attribute too long on <documentation> element' => [

Tests/FeatureComplete/Check/ColorTest.php

@@ -65,7 +65,7 @@ public function testColors($fixtureDir, $expectedOutput, $exitCode, $src = 'stdo
      *
      * @return array
      */
-    public function dataColors()
+    public static function dataColors()
     {
         return [
             'feature complete - header' => [

Tests/FeatureComplete/Check/MarkProgressTest.php

@@ -104,7 +104,7 @@ public function testProgress($nrOfFiles, $expectedOutput)
      *
      * @return array
      */
-    public function dataProgress()
+    public static function dataProgress()
     {
         return [
             'single line progress reporting (<= 60 sniffs)'      => [

Tests/FeatureComplete/Check/NoSniffsTest.php

@@ -77,7 +77,7 @@ public function testTargetDoesntContainSniffs($fixtureDir)
      *
      * @return array
      */
-    public function dataTargetDoesntContainSniffs()
+    public static function dataTargetDoesntContainSniffs()
     {
         return [
             'empty directory'                            => ['EmptyDir'],

Tests/FeatureComplete/Check/ValidStandardsTest.php

@@ -63,7 +63,7 @@ public function testFeatureCompleteStandard($fixtureDir, $testDocCount)
      *
      * @return array
      */
-    public function dataFeatureCompleteStandard()
+    public static function dataFeatureCompleteStandard()
     {
         return [
             'feature complete with CSS test files'      => [

Tests/FeatureComplete/Config/GetHelpTest.php

@@ -107,7 +107,7 @@ public function testShowHelpNoColors($command)
      *
      * @return array
      */
-    public function dataShowHelp()
+    public static function dataShowHelp()
     {
         return [
             '-h'     => [

Tests/FeatureComplete/Config/GetVersionTest.php

@@ -49,7 +49,7 @@ public function testShowVersion($command)
      *
      * @return array
      */
-    public function dataShowVersion()
+    public static function dataShowVersion()
     {
         return [
             '-V'        => [

Tests/FeatureComplete/Config/MagicMethodsTest.php

@@ -65,7 +65,7 @@ public function testGet($propertyName, $expected)
      *
      * @return void
      */
-    public function dataGet()
+    public static function dataGet()
     {
         return [
             'property which doesn\'t exist on the class' => [
@@ -104,7 +104,7 @@ public function testIsset($propertyName, $expected)
      *
      * @return void
      */
-    public function dataIsset()
+    public static function dataIsset()
     {
         return [
             'property which doesn\'t exist on the class' => [
@@ -167,7 +167,7 @@ public function testUnset($propertyName)
      *
      * @return void
      */
-    public function dataSetUnset()
+    public static function dataSetUnset()
     {
         return [
             'property which doesn\'t exist on the class' => ['doesnotexist'],

Tests/FeatureComplete/Config/ProcessCliCommandTest.php

@@ -73,7 +73,7 @@ public function testProcessCliCommandUnsupportedArgument($command)
      *
      * @return array
      */
-    public function dataProcessCliCommandUnsupportedArgument()
+    public static function dataProcessCliCommandUnsupportedArgument()
     {
         return [
             'Unsupported short arguments' => [
@@ -141,7 +141,7 @@ public function testProcessCliCommand($command, array $expectedChanged, $checkSh
      *
      * @return array
      */
-    public function dataProcessCliCommand()
+    public static function dataProcessCliCommand()
     {
         /*
          * For project root, we only really verify that it has been set as the value will depend
@@ -402,7 +402,7 @@ public function testProcessCliCommandOutputOnlyArgs($command, $expectedChanged)
      *
      * @return array
      */
-    public function dataProcessCliCommandOutputOnlyArgs()
+    public static function dataProcessCliCommandOutputOnlyArgs()
     {
         /*
          * For project root, we only verify that it has been set as the value will depend on the