Migrate to static data providers using rector/rector

OskarStark · nicolas-grekas · commit 105a7ac54eca · 2023-02-14T09:03:56.000+01:00
diff --git a/Tests/AbstractCrawlerTestCase.php b/Tests/AbstractCrawlerTestCase.php
@@ -912,7 +912,7 @@ public function testSiblings()
         }
     }
 
-    public function provideMatchTests()
+    public static function provideMatchTests()
     {
         yield ['#foo', true, '#foo'];
         yield ['#foo', true, '.foo'];
@@ -1160,7 +1160,7 @@ public function testBaseTag($baseValue, $linkValue, $expectedUri, $currentUri =
         $this->assertEquals($expectedUri, $crawler->filterXPath('//a')->link()->getUri(), $description);
     }
 
-    public function getBaseTagData()
+    public static function getBaseTagData()
     {
         return [
             ['http://base.com', 'link', 'http://base.com/link'],
@@ -1180,7 +1180,7 @@ public function testBaseTagWithForm($baseValue, $actionValue, $expectedUri, $cur
         $this->assertEquals($expectedUri, $crawler->filterXPath('//button')->form()->getUri(), $description);
     }
 
-    public function getBaseTagWithFormData()
+    public static function getBaseTagWithFormData()
     {
         return [
             ['https://base.com/', 'link/', 'https://base.com/link/', 'https://base.com/link/', '<base> tag does work with a path and relative form action'],
diff --git a/Tests/Field/FileFormFieldTest.php b/Tests/Field/FileFormFieldTest.php
@@ -78,7 +78,7 @@ public function testSetValue($method)
         );
     }
 
-    public function getSetValueMethods()
+    public static function getSetValueMethods()
     {
         return [
             ['setValue'],
diff --git a/Tests/FormTest.php b/Tests/FormTest.php
@@ -78,7 +78,7 @@ public function testConstructorThrowsExceptionIfNoRelatedForm(\DOMElement $node)
         new Form($node, 'http://example.com');
     }
 
-    public function constructorThrowsExceptionIfNoRelatedFormProvider()
+    public static function constructorThrowsExceptionIfNoRelatedFormProvider()
     {
         $dom = new \DOMDocument();
         $dom->loadHTML('
@@ -217,7 +217,7 @@ function ($field) {
         );
     }
 
-    public function provideInitializeValues()
+    public static function provideInitializeValues()
     {
         return [
             [
@@ -585,7 +585,7 @@ public function testGetUriWithActionOverride()
         $this->assertEquals('http://localhost/bar', $form->getUri(), '->getUri() returns absolute URIs');
     }
 
-    public function provideGetUriValues()
+    public static function provideGetUriValues()
     {
         return [
             [
diff --git a/Tests/Html5ParserCrawlerTest.php b/Tests/Html5ParserCrawlerTest.php
@@ -50,7 +50,7 @@ public function testHtml5ParserWithInvalidHeadedContent(string $content)
         self::assertEmpty($crawler->filterXPath('//h1')->text(), '->addHtmlContent failed as expected');
     }
 
-    public function validHtml5Provider(): iterable
+    public static function validHtml5Provider(): iterable
     {
         $html = self::getDoctype().'<html><body><h1><p>Foo</p></h1></body></html>';
         $BOM = \chr(0xEF).\chr(0xBB).\chr(0xBF);
@@ -63,7 +63,7 @@ public function validHtml5Provider(): iterable
         yield 'All together' => [$BOM.'  <!--c-->'.$html];
     }
 
-    public function invalidHtml5Provider(): iterable
+    public static function invalidHtml5Provider(): iterable
     {
         $html = self::getDoctype().'<html><body><h1><p>Foo</p></h1></body></html>';
 
diff --git a/Tests/ImageTest.php b/Tests/ImageTest.php
@@ -56,7 +56,7 @@ public function testGetUri($url, $currentUri, $expected)
         $this->assertEquals($expected, $image->getUri());
     }
 
-    public function getGetUriTests()
+    public static function getGetUriTests()
     {
         return [
             ['/foo.png', 'http://localhost/bar/foo/', 'http://localhost/foo.png'],
diff --git a/Tests/LinkTest.php b/Tests/LinkTest.php
@@ -105,7 +105,7 @@ public function testGetUriOnLink($url, $currentUri, $expected)
         $this->assertEquals($expected, $link->getUri());
     }
 
-    public function getGetUriTests()
+    public static function getGetUriTests()
     {
         return [
             ['/foo', 'http://localhost/bar/foo/', 'http://localhost/foo'],
diff --git a/Tests/UriResolverTest.php b/Tests/UriResolverTest.php
@@ -24,7 +24,7 @@ public function testResolver(string $uri, string $baseUri, string $expected)
         $this->assertEquals($expected, UriResolver::resolve($uri, $baseUri));
     }
 
-    public function provideResolverTests()
+    public static function provideResolverTests()
     {
         return [
             ['/foo', 'http://localhost/bar/foo/', 'http://localhost/foo'],

Original file line number	Diff line number	Diff line change
`@@ -912,7 +912,7 @@ public function testSiblings()`
`912`	`912`	`}`
`913`	`913`	`}`
`914`	`914`
`915`		`- public function provideMatchTests()`
	`915`	`+ public static function provideMatchTests()`
`916`	`916`	`{`
`917`	`917`	`yield ['#foo', true, '#foo'];`
`918`	`918`	`yield ['#foo', true, '.foo'];`
`@@ -1160,7 +1160,7 @@ public function testBaseTag($baseValue, $linkValue, $expectedUri, $currentUri =`
`1160`	`1160`	`$this->assertEquals($expectedUri, $crawler->filterXPath('//a')->link()->getUri(), $description);`
`1161`	`1161`	`}`
`1162`	`1162`
`1163`		`- public function getBaseTagData()`
	`1163`	`+ public static function getBaseTagData()`
`1164`	`1164`	`{`
`1165`	`1165`	`return [`
`1166`	`1166`	`['http://base.com', 'link', 'http://base.com/link'],`
`@@ -1180,7 +1180,7 @@ public function testBaseTagWithForm($baseValue, $actionValue, $expectedUri, $cur`
`1180`	`1180`	`$this->assertEquals($expectedUri, $crawler->filterXPath('//button')->form()->getUri(), $description);`
`1181`	`1181`	`}`
`1182`	`1182`
`1183`		`- public function getBaseTagWithFormData()`
	`1183`	`+ public static function getBaseTagWithFormData()`
`1184`	`1184`	`{`
`1185`	`1185`	`return [`
`1186`	`1186`	`['https://base.com/', 'link/', 'https://base.com/link/', 'https://base.com/link/', '<base> tag does work with a path and relative form action'],`
Original file line number	Diff line number	Diff line change
`@@ -78,7 +78,7 @@ public function testSetValue($method)`
`78`	`78`	`);`
`79`	`79`	`}`
`80`	`80`
`81`		`- public function getSetValueMethods()`
	`81`	`+ public static function getSetValueMethods()`
`82`	`82`	`{`
`83`	`83`	`return [`
`84`	`84`	`['setValue'],`
Original file line number	Diff line number	Diff line change
`@@ -78,7 +78,7 @@ public function testConstructorThrowsExceptionIfNoRelatedForm(\DOMElement $node)`
`78`	`78`	`new Form($node, 'http://example.com');`
`79`	`79`	`}`
`80`	`80`
`81`		`- public function constructorThrowsExceptionIfNoRelatedFormProvider()`
	`81`	`+ public static function constructorThrowsExceptionIfNoRelatedFormProvider()`
`82`	`82`	`{`
`83`	`83`	`$dom = new \DOMDocument();`
`84`	`84`	`$dom->loadHTML('`
`@@ -217,7 +217,7 @@ function ($field) {`
`217`	`217`	`);`
`218`	`218`	`}`
`219`	`219`
`220`		`- public function provideInitializeValues()`
	`220`	`+ public static function provideInitializeValues()`
`221`	`221`	`{`
`222`	`222`	`return [`
`223`	`223`	`[`
`@@ -585,7 +585,7 @@ public function testGetUriWithActionOverride()`
`585`	`585`	`$this->assertEquals('http://localhost/bar', $form->getUri(), '->getUri() returns absolute URIs');`
`586`	`586`	`}`
`587`	`587`
`588`		`- public function provideGetUriValues()`
	`588`	`+ public static function provideGetUriValues()`
`589`	`589`	`{`
`590`	`590`	`return [`
`591`	`591`	`[`
Original file line number	Diff line number	Diff line change
`@@ -50,7 +50,7 @@ public function testHtml5ParserWithInvalidHeadedContent(string $content)`
`50`	`50`	`self::assertEmpty($crawler->filterXPath('//h1')->text(), '->addHtmlContent failed as expected');`
`51`	`51`	`}`
`52`	`52`
`53`		`- public function validHtml5Provider(): iterable`
	`53`	`+ public static function validHtml5Provider(): iterable`
`54`	`54`	`{`
`55`	`55`	`$html = self::getDoctype().'<html><body><h1><p>Foo</p></h1></body></html>';`
`56`	`56`	`$BOM = \chr(0xEF).\chr(0xBB).\chr(0xBF);`
`@@ -63,7 +63,7 @@ public function validHtml5Provider(): iterable`
`63`	`63`	`yield 'All together' => [$BOM.' <!--c-->'.$html];`
`64`	`64`	`}`
`65`	`65`
`66`		`- public function invalidHtml5Provider(): iterable`
	`66`	`+ public static function invalidHtml5Provider(): iterable`
`67`	`67`	`{`
`68`	`68`	`$html = self::getDoctype().'<html><body><h1><p>Foo</p></h1></body></html>';`
`69`	`69`
Original file line number	Diff line number	Diff line change
`@@ -56,7 +56,7 @@ public function testGetUri($url, $currentUri, $expected)`
`56`	`56`	`$this->assertEquals($expected, $image->getUri());`
`57`	`57`	`}`
`58`	`58`
`59`		`- public function getGetUriTests()`
	`59`	`+ public static function getGetUriTests()`
`60`	`60`	`{`
`61`	`61`	`return [`
`62`	`62`	`['/foo.png', 'http://localhost/bar/foo/', 'http://localhost/foo.png'],`
Original file line number	Diff line number	Diff line change
`@@ -105,7 +105,7 @@ public function testGetUriOnLink($url, $currentUri, $expected)`
`105`	`105`	`$this->assertEquals($expected, $link->getUri());`
`106`	`106`	`}`
`107`	`107`
`108`		`- public function getGetUriTests()`
	`108`	`+ public static function getGetUriTests()`
`109`	`109`	`{`
`110`	`110`	`return [`
`111`	`111`	`['/foo', 'http://localhost/bar/foo/', 'http://localhost/foo'],`
Original file line number	Diff line number	Diff line change
`@@ -24,7 +24,7 @@ public function testResolver(string $uri, string $baseUri, string $expected)`
`24`	`24`	`$this->assertEquals($expected, UriResolver::resolve($uri, $baseUri));`
`25`	`25`	`}`
`26`	`26`
`27`		`- public function provideResolverTests()`
	`27`	`+ public static function provideResolverTests()`
`28`	`28`	`{`
`29`	`29`	`return [`
`30`	`30`	`['/foo', 'http://localhost/bar/foo/', 'http://localhost/foo'],`