Migrate to static data providers using rector/rector

OskarStark · nicolas-grekas · commit 95f3c7468db1 · 2023-02-14T09:03:56.000+01:00
diff --git a/Tests/CssSelectorConverterTest.php b/Tests/CssSelectorConverterTest.php
@@ -60,7 +60,7 @@ public function testCssToXPathWithoutPrefix($css, $xpath)
         $this->assertEquals($xpath, $converter->toXPath($css, ''), '->parse() parses an input string and returns a node');
     }
 
-    public function getCssToXPathWithoutPrefixTestData()
+    public static function getCssToXPathWithoutPrefixTestData()
     {
         return [
             ['h1', 'h1'],
diff --git a/Tests/Node/SpecificityTest.php b/Tests/Node/SpecificityTest.php
@@ -28,7 +28,7 @@ public function testPlusValue(Specificity $specificity, $value)
         $this->assertEquals($value + 123, $specificity->plus(new Specificity(1, 2, 3))->getValue());
     }
 
-    public function getValueTestData()
+    public static function getValueTestData()
     {
         return [
             [new Specificity(0, 0, 0), 0],
@@ -45,7 +45,7 @@ public function testCompareTo(Specificity $a, Specificity $b, $result)
         $this->assertEquals($result, $a->compareTo($b));
     }
 
-    public function getCompareTestData()
+    public static function getCompareTestData()
     {
         return [
             [new Specificity(0, 0, 0), new Specificity(0, 0, 0), 0],
diff --git a/Tests/Parser/Handler/AbstractHandlerTestCase.php b/Tests/Parser/Handler/AbstractHandlerTestCase.php
@@ -43,9 +43,9 @@ public function testDontHandleValue($value)
         $this->assertRemainingContent($reader, $value);
     }
 
-    abstract public function getHandleValueTestData();
+    abstract public static function getHandleValueTestData();
 
-    abstract public function getDontHandleValueTestData();
+    abstract public static function getDontHandleValueTestData();
 
     abstract protected function generateHandler();
 
diff --git a/Tests/Parser/Handler/CommentHandlerTest.php b/Tests/Parser/Handler/CommentHandlerTest.php
@@ -30,7 +30,7 @@ public function testHandleValue($value, Token $unusedArgument, $remainingContent
         $this->assertRemainingContent($reader, $remainingContent);
     }
 
-    public function getHandleValueTestData()
+    public static function getHandleValueTestData()
     {
         return [
             // 2nd argument only exists for inherited method compatibility
@@ -39,7 +39,7 @@ public function getHandleValueTestData()
         ];
     }
 
-    public function getDontHandleValueTestData()
+    public static function getDontHandleValueTestData()
     {
         return [
             ['>'],
diff --git a/Tests/Parser/Handler/HashHandlerTest.php b/Tests/Parser/Handler/HashHandlerTest.php
@@ -18,7 +18,7 @@
 
 class HashHandlerTest extends AbstractHandlerTestCase
 {
-    public function getHandleValueTestData()
+    public static function getHandleValueTestData()
     {
         return [
             ['#id', new Token(Token::TYPE_HASH, 'id', 0), ''],
@@ -29,7 +29,7 @@ public function getHandleValueTestData()
         ];
     }
 
-    public function getDontHandleValueTestData()
+    public static function getDontHandleValueTestData()
     {
         return [
             ['id'],
diff --git a/Tests/Parser/Handler/IdentifierHandlerTest.php b/Tests/Parser/Handler/IdentifierHandlerTest.php
@@ -18,7 +18,7 @@
 
 class IdentifierHandlerTest extends AbstractHandlerTestCase
 {
-    public function getHandleValueTestData()
+    public static function getHandleValueTestData()
     {
         return [
             ['foo', new Token(Token::TYPE_IDENTIFIER, 'foo', 0), ''],
@@ -29,7 +29,7 @@ public function getHandleValueTestData()
         ];
     }
 
-    public function getDontHandleValueTestData()
+    public static function getDontHandleValueTestData()
     {
         return [
             ['>'],
diff --git a/Tests/Parser/Handler/NumberHandlerTest.php b/Tests/Parser/Handler/NumberHandlerTest.php
@@ -17,7 +17,7 @@
 
 class NumberHandlerTest extends AbstractHandlerTestCase
 {
-    public function getHandleValueTestData()
+    public static function getHandleValueTestData()
     {
         return [
             ['12', new Token(Token::TYPE_NUMBER, '12', 0), ''],
@@ -30,7 +30,7 @@ public function getHandleValueTestData()
         ];
     }
 
-    public function getDontHandleValueTestData()
+    public static function getDontHandleValueTestData()
     {
         return [
             ['hello'],
diff --git a/Tests/Parser/Handler/StringHandlerTest.php b/Tests/Parser/Handler/StringHandlerTest.php
@@ -18,7 +18,7 @@
 
 class StringHandlerTest extends AbstractHandlerTestCase
 {
-    public function getHandleValueTestData()
+    public static function getHandleValueTestData()
     {
         return [
             ['"hello"', new Token(Token::TYPE_STRING, 'hello', 1), ''],
@@ -31,7 +31,7 @@ public function getHandleValueTestData()
         ];
     }
 
-    public function getDontHandleValueTestData()
+    public static function getDontHandleValueTestData()
     {
         return [
             ['hello'],
diff --git a/Tests/Parser/Handler/WhitespaceHandlerTest.php b/Tests/Parser/Handler/WhitespaceHandlerTest.php
@@ -16,7 +16,7 @@
 
 class WhitespaceHandlerTest extends AbstractHandlerTestCase
 {
-    public function getHandleValueTestData()
+    public static function getHandleValueTestData()
     {
         return [
             [' ', new Token(Token::TYPE_WHITESPACE, ' ', 0), ''],
@@ -28,7 +28,7 @@ public function getHandleValueTestData()
         ];
     }
 
-    public function getDontHandleValueTestData()
+    public static function getDontHandleValueTestData()
     {
         return [
             ['>'],
diff --git a/Tests/Parser/ParserTest.php b/Tests/Parser/ParserTest.php
@@ -93,7 +93,7 @@ public function testParseSeriesException($series)
         Parser::parseSeries($function->getArguments());
     }
 
-    public function getParserTestData()
+    public static function getParserTestData()
     {
         return [
             ['*', ['Element[*]']],
@@ -151,7 +151,7 @@ public function getParserTestData()
         ];
     }
 
-    public function getParserExceptionTestData()
+    public static function getParserExceptionTestData()
     {
         return [
             ['attributes(href)/html/body/a', SyntaxErrorException::unexpectedToken('selector', new Token(Token::TYPE_DELIMITER, '(', 10))->getMessage()],
@@ -181,7 +181,7 @@ public function getParserExceptionTestData()
         ];
     }
 
-    public function getPseudoElementsTestData()
+    public static function getPseudoElementsTestData()
     {
         return [
             ['foo', 'Element[foo]', ''],
@@ -203,7 +203,7 @@ public function getPseudoElementsTestData()
         ];
     }
 
-    public function getSpecificityTestData()
+    public static function getSpecificityTestData()
     {
         return [
             ['*', 0],
@@ -231,7 +231,7 @@ public function getSpecificityTestData()
         ];
     }
 
-    public function getParseSeriesTestData()
+    public static function getParseSeriesTestData()
     {
         return [
             ['1n+3', 1, 3],
@@ -253,7 +253,7 @@ public function getParseSeriesTestData()
         ];
     }
 
-    public function getParseSeriesExceptionTestData()
+    public static function getParseSeriesExceptionTestData()
     {
         return [
             ['foo'],
diff --git a/Tests/Parser/Shortcut/ClassParserTest.php b/Tests/Parser/Shortcut/ClassParserTest.php
@@ -32,7 +32,7 @@ public function testParse($source, $representation)
         $this->assertEquals($representation, (string) $selector->getTree());
     }
 
-    public function getParseTestData()
+    public static function getParseTestData()
     {
         return [
             ['.testclass', 'Class[Element[*].testclass]'],
diff --git a/Tests/Parser/Shortcut/ElementParserTest.php b/Tests/Parser/Shortcut/ElementParserTest.php
@@ -32,7 +32,7 @@ public function testParse($source, $representation)
         $this->assertEquals($representation, (string) $selector->getTree());
     }
 
-    public function getParseTestData()
+    public static function getParseTestData()
     {
         return [
             ['*', 'Element[*]'],
diff --git a/Tests/Parser/Shortcut/HashParserTest.php b/Tests/Parser/Shortcut/HashParserTest.php
@@ -32,7 +32,7 @@ public function testParse($source, $representation)
         $this->assertEquals($representation, (string) $selector->getTree());
     }
 
-    public function getParseTestData()
+    public static function getParseTestData()
     {
         return [
             ['#testid', 'Hash[Element[*]#testid]'],
diff --git a/Tests/XPath/TranslatorTest.php b/Tests/XPath/TranslatorTest.php
@@ -165,7 +165,7 @@ public function testOnlyOfTypeFindsSingleChildrenOfGivenType()
         $this->assertSame('A', $nodeList->item(0)->textContent);
     }
 
-    public function getXpathLiteralTestData()
+    public static function getXpathLiteralTestData()
     {
         return [
             ['foo', "'foo'"],
@@ -175,7 +175,7 @@ public function getXpathLiteralTestData()
         ];
     }
 
-    public function getCssToXPathTestData()
+    public static function getCssToXPathTestData()
     {
         return [
             ['*', '*'],
@@ -222,7 +222,7 @@ public function getCssToXPathTestData()
         ];
     }
 
-    public function getXmlLangTestData()
+    public static function getXmlLangTestData()
     {
         return [
             [':lang("EN")', ['first', 'second', 'third', 'fourth']],
@@ -237,7 +237,7 @@ public function getXmlLangTestData()
         ];
     }
 
-    public function getHtmlIdsTestData()
+    public static function getHtmlIdsTestData()
     {
         return [
             ['div', ['outer-div', 'li-div', 'foobar-div']],
@@ -362,7 +362,7 @@ public function getHtmlIdsTestData()
         ];
     }
 
-    public function getHtmlShakespearTestData()
+    public static function getHtmlShakespearTestData()
     {
         return [
             ['*', 246],

Original file line number	Diff line number	Diff line change
`@@ -60,7 +60,7 @@ public function testCssToXPathWithoutPrefix($css, $xpath)`
`60`	`60`	`$this->assertEquals($xpath, $converter->toXPath($css, ''), '->parse() parses an input string and returns a node');`
`61`	`61`	`}`
`62`	`62`
`63`		`- public function getCssToXPathWithoutPrefixTestData()`
	`63`	`+ public static function getCssToXPathWithoutPrefixTestData()`
`64`	`64`	`{`
`65`	`65`	`return [`
`66`	`66`	`['h1', 'h1'],`
Original file line number	Diff line number	Diff line change
`@@ -28,7 +28,7 @@ public function testPlusValue(Specificity $specificity, $value)`
`28`	`28`	`$this->assertEquals($value + 123, $specificity->plus(new Specificity(1, 2, 3))->getValue());`
`29`	`29`	`}`
`30`	`30`
`31`		`- public function getValueTestData()`
	`31`	`+ public static function getValueTestData()`
`32`	`32`	`{`
`33`	`33`	`return [`
`34`	`34`	`[new Specificity(0, 0, 0), 0],`
`@@ -45,7 +45,7 @@ public function testCompareTo(Specificity $a, Specificity $b, $result)`
`45`	`45`	`$this->assertEquals($result, $a->compareTo($b));`
`46`	`46`	`}`
`47`	`47`
`48`		`- public function getCompareTestData()`
	`48`	`+ public static function getCompareTestData()`
`49`	`49`	`{`
`50`	`50`	`return [`
`51`	`51`	`[new Specificity(0, 0, 0), new Specificity(0, 0, 0), 0],`
Original file line number	Diff line number	Diff line change
`@@ -43,9 +43,9 @@ public function testDontHandleValue($value)`
`43`	`43`	`$this->assertRemainingContent($reader, $value);`
`44`	`44`	`}`
`45`	`45`
`46`		`- abstract public function getHandleValueTestData();`
	`46`	`+ abstract public static function getHandleValueTestData();`
`47`	`47`
`48`		`- abstract public function getDontHandleValueTestData();`
	`48`	`+ abstract public static function getDontHandleValueTestData();`
`49`	`49`
`50`	`50`	`abstract protected function generateHandler();`
`51`	`51`
Original file line number	Diff line number	Diff line change
`@@ -30,7 +30,7 @@ public function testHandleValue($value, Token $unusedArgument, $remainingContent`
`30`	`30`	`$this->assertRemainingContent($reader, $remainingContent);`
`31`	`31`	`}`
`32`	`32`
`33`		`- public function getHandleValueTestData()`
	`33`	`+ public static function getHandleValueTestData()`
`34`	`34`	`{`
`35`	`35`	`return [`
`36`	`36`	`// 2nd argument only exists for inherited method compatibility`
`@@ -39,7 +39,7 @@ public function getHandleValueTestData()`
`39`	`39`	`];`
`40`	`40`	`}`
`41`	`41`
`42`		`- public function getDontHandleValueTestData()`
	`42`	`+ public static function getDontHandleValueTestData()`
`43`	`43`	`{`
`44`	`44`	`return [`
`45`	`45`	`['>'],`
Original file line number	Diff line number	Diff line change
`@@ -18,7 +18,7 @@`
`18`	`18`
`19`	`19`	`class HashHandlerTest extends AbstractHandlerTestCase`
`20`	`20`	`{`
`21`		`- public function getHandleValueTestData()`
	`21`	`+ public static function getHandleValueTestData()`
`22`	`22`	`{`
`23`	`23`	`return [`
`24`	`24`	`['#id', new Token(Token::TYPE_HASH, 'id', 0), ''],`
`@@ -29,7 +29,7 @@ public function getHandleValueTestData()`
`29`	`29`	`];`
`30`	`30`	`}`
`31`	`31`
`32`		`- public function getDontHandleValueTestData()`
	`32`	`+ public static function getDontHandleValueTestData()`
`33`	`33`	`{`
`34`	`34`	`return [`
`35`	`35`	`['id'],`
Original file line number	Diff line number	Diff line change
`@@ -17,7 +17,7 @@`
`17`	`17`
`18`	`18`	`class NumberHandlerTest extends AbstractHandlerTestCase`
`19`	`19`	`{`
`20`		`- public function getHandleValueTestData()`
	`20`	`+ public static function getHandleValueTestData()`
`21`	`21`	`{`
`22`	`22`	`return [`
`23`	`23`	`['12', new Token(Token::TYPE_NUMBER, '12', 0), ''],`
`@@ -30,7 +30,7 @@ public function getHandleValueTestData()`
`30`	`30`	`];`
`31`	`31`	`}`
`32`	`32`
`33`		`- public function getDontHandleValueTestData()`
	`33`	`+ public static function getDontHandleValueTestData()`
`34`	`34`	`{`
`35`	`35`	`return [`
`36`	`36`	`['hello'],`
Original file line number	Diff line number	Diff line change
`@@ -16,7 +16,7 @@`
`16`	`16`
`17`	`17`	`class WhitespaceHandlerTest extends AbstractHandlerTestCase`
`18`	`18`	`{`
`19`		`- public function getHandleValueTestData()`
	`19`	`+ public static function getHandleValueTestData()`
`20`	`20`	`{`
`21`	`21`	`return [`
`22`	`22`	`[' ', new Token(Token::TYPE_WHITESPACE, ' ', 0), ''],`
`@@ -28,7 +28,7 @@ public function getHandleValueTestData()`
`28`	`28`	`];`
`29`	`29`	`}`
`30`	`30`
`31`		`- public function getDontHandleValueTestData()`
	`31`	`+ public static function getDontHandleValueTestData()`
`32`	`32`	`{`
`33`	`33`	`return [`
`34`	`34`	`['>'],`
Original file line number	Diff line number	Diff line change
`@@ -93,7 +93,7 @@ public function testParseSeriesException($series)`
`93`	`93`	`Parser::parseSeries($function->getArguments());`
`94`	`94`	`}`
`95`	`95`
`96`		`- public function getParserTestData()`
	`96`	`+ public static function getParserTestData()`
`97`	`97`	`{`
`98`	`98`	`return [`
`99`	`99`	`['', ['Element[]']],`
`@@ -151,7 +151,7 @@ public function getParserTestData()`
`151`	`151`	`];`
`152`	`152`	`}`
`153`	`153`
`154`		`- public function getParserExceptionTestData()`
	`154`	`+ public static function getParserExceptionTestData()`
`155`	`155`	`{`
`156`	`156`	`return [`
`157`	`157`	`['attributes(href)/html/body/a', SyntaxErrorException::unexpectedToken('selector', new Token(Token::TYPE_DELIMITER, '(', 10))->getMessage()],`
`@@ -181,7 +181,7 @@ public function getParserExceptionTestData()`
`181`	`181`	`];`
`182`	`182`	`}`
`183`	`183`
`184`		`- public function getPseudoElementsTestData()`
	`184`	`+ public static function getPseudoElementsTestData()`
`185`	`185`	`{`
`186`	`186`	`return [`
`187`	`187`	`['foo', 'Element[foo]', ''],`
`@@ -203,7 +203,7 @@ public function getPseudoElementsTestData()`
`203`	`203`	`];`
`204`	`204`	`}`
`205`	`205`
`206`		`- public function getSpecificityTestData()`
	`206`	`+ public static function getSpecificityTestData()`
`207`	`207`	`{`
`208`	`208`	`return [`
`209`	`209`	`['*', 0],`
`@@ -231,7 +231,7 @@ public function getSpecificityTestData()`
`231`	`231`	`];`
`232`	`232`	`}`
`233`	`233`
`234`		`- public function getParseSeriesTestData()`
	`234`	`+ public static function getParseSeriesTestData()`
`235`	`235`	`{`
`236`	`236`	`return [`
`237`	`237`	`['1n+3', 1, 3],`
`@@ -253,7 +253,7 @@ public function getParseSeriesTestData()`
`253`	`253`	`];`
`254`	`254`	`}`
`255`	`255`
`256`		`- public function getParseSeriesExceptionTestData()`
	`256`	`+ public static function getParseSeriesExceptionTestData()`
`257`	`257`	`{`
`258`	`258`	`return [`
`259`	`259`	`['foo'],`
Original file line number	Diff line number	Diff line change
`@@ -32,7 +32,7 @@ public function testParse($source, $representation)`
`32`	`32`	`$this->assertEquals($representation, (string) $selector->getTree());`
`33`	`33`	`}`
`34`	`34`
`35`		`- public function getParseTestData()`
	`35`	`+ public static function getParseTestData()`
`36`	`36`	`{`
`37`	`37`	`return [`
`38`	`38`	`['.testclass', 'Class[Element[*].testclass]'],`
Original file line number	Diff line number	Diff line change
`@@ -165,7 +165,7 @@ public function testOnlyOfTypeFindsSingleChildrenOfGivenType()`
`165`	`165`	`$this->assertSame('A', $nodeList->item(0)->textContent);`
`166`	`166`	`}`
`167`	`167`
`168`		`- public function getXpathLiteralTestData()`
	`168`	`+ public static function getXpathLiteralTestData()`
`169`	`169`	`{`
`170`	`170`	`return [`
`171`	`171`	`['foo', "'foo'"],`
`@@ -175,7 +175,7 @@ public function getXpathLiteralTestData()`
`175`	`175`	`];`
`176`	`176`	`}`
`177`	`177`
`178`		`- public function getCssToXPathTestData()`
	`178`	`+ public static function getCssToXPathTestData()`
`179`	`179`	`{`
`180`	`180`	`return [`
`181`	`181`	`['', ''],`
`@@ -222,7 +222,7 @@ public function getCssToXPathTestData()`
`222`	`222`	`];`
`223`	`223`	`}`
`224`	`224`
`225`		`- public function getXmlLangTestData()`
	`225`	`+ public static function getXmlLangTestData()`
`226`	`226`	`{`
`227`	`227`	`return [`
`228`	`228`	`[':lang("EN")', ['first', 'second', 'third', 'fourth']],`
`@@ -237,7 +237,7 @@ public function getXmlLangTestData()`
`237`	`237`	`];`
`238`	`238`	`}`
`239`	`239`
`240`		`- public function getHtmlIdsTestData()`
	`240`	`+ public static function getHtmlIdsTestData()`
`241`	`241`	`{`
`242`	`242`	`return [`
`243`	`243`	`['div', ['outer-div', 'li-div', 'foobar-div']],`
`@@ -362,7 +362,7 @@ public function getHtmlIdsTestData()`
`362`	`362`	`];`
`363`	`363`	`}`
`364`	`364`
`365`		`- public function getHtmlShakespearTestData()`
	`365`	`+ public static function getHtmlShakespearTestData()`
`366`	`366`	`{`
`367`	`367`	`return [`
`368`	`368`	`['*', 246],`