Migrate to static data providers using rector/rector

OskarStark · nicolas-grekas · commit 1504b6773c6b · 2023-02-14T09:03:56.000+01:00
diff --git a/Tests/PackageTest.php b/Tests/PackageTest.php
@@ -27,7 +27,7 @@ public function testGetUrl($version, $format, $path, $expected)
         $this->assertSame($expected, $package->getUrl($path));
     }
 
-    public function getConfigs()
+    public static function getConfigs()
     {
         return [
             ['v1', '', 'http://example.com/foo', 'http://example.com/foo'],
diff --git a/Tests/PathPackageTest.php b/Tests/PathPackageTest.php
@@ -28,7 +28,7 @@ public function testGetUrl($basePath, $format, $path, $expected)
         $this->assertSame($expected, $package->getUrl($path));
     }
 
-    public function getConfigs()
+    public static function getConfigs()
     {
         return [
             ['/foo', '', 'http://example.com/foo', 'http://example.com/foo'],
@@ -60,7 +60,7 @@ public function testGetUrlWithContext($basePathRequest, $basePath, $format, $pat
         $this->assertSame($expected, $package->getUrl($path));
     }
 
-    public function getContextConfigs()
+    public static function getContextConfigs()
     {
         return [
             ['', '/foo', '', '/baz', '/baz?v1'],
diff --git a/Tests/UrlPackageTest.php b/Tests/UrlPackageTest.php
@@ -31,7 +31,7 @@ public function testGetUrl($baseUrls, $format, $path, $expected)
         $this->assertSame($expected, $package->getUrl($path));
     }
 
-    public function getConfigs()
+    public static function getConfigs()
     {
         return [
             ['http://example.net', '', 'http://example.com/foo', 'http://example.com/foo'],
@@ -72,7 +72,7 @@ public function testGetUrlWithContext($secure, $baseUrls, $format, $path, $expec
         $this->assertSame($expected, $package->getUrl($path));
     }
 
-    public function getContextConfigs()
+    public static function getContextConfigs()
     {
         return [
             [false, 'http://example.com', '', 'foo', 'http://example.com/foo?v1'],
@@ -114,7 +114,7 @@ public function testWrongBaseUrl($baseUrls)
         new UrlPackage($baseUrls, new EmptyVersionStrategy());
     }
 
-    public function getWrongBaseUrlConfig()
+    public static function getWrongBaseUrlConfig()
     {
         return [
             ['not-a-url'],
diff --git a/Tests/VersionStrategy/StaticVersionStrategyTest.php b/Tests/VersionStrategy/StaticVersionStrategyTest.php
@@ -34,7 +34,7 @@ public function testApplyVersion($path, $version, $format)
         $this->assertSame($formatted, $staticVersionStrategy->applyVersion($path));
     }
 
-    public function getConfigs()
+    public static function getConfigs()
     {
         return [
             ['test-path', 'v1', null],

Original file line number	Diff line number	Diff line change
`@@ -27,7 +27,7 @@ public function testGetUrl($version, $format, $path, $expected)`
`27`	`27`	`$this->assertSame($expected, $package->getUrl($path));`
`28`	`28`	`}`
`29`	`29`
`30`		`- public function getConfigs()`
	`30`	`+ public static function getConfigs()`
`31`	`31`	`{`
`32`	`32`	`return [`
`33`	`33`	`['v1', '', 'http://example.com/foo', 'http://example.com/foo'],`
Original file line number	Diff line number	Diff line change
`@@ -28,7 +28,7 @@ public function testGetUrl($basePath, $format, $path, $expected)`
`28`	`28`	`$this->assertSame($expected, $package->getUrl($path));`
`29`	`29`	`}`
`30`	`30`
`31`		`- public function getConfigs()`
	`31`	`+ public static function getConfigs()`
`32`	`32`	`{`
`33`	`33`	`return [`
`34`	`34`	`['/foo', '', 'http://example.com/foo', 'http://example.com/foo'],`
`@@ -60,7 +60,7 @@ public function testGetUrlWithContext($basePathRequest, $basePath, $format, $pat`
`60`	`60`	`$this->assertSame($expected, $package->getUrl($path));`
`61`	`61`	`}`
`62`	`62`
`63`		`- public function getContextConfigs()`
	`63`	`+ public static function getContextConfigs()`
`64`	`64`	`{`
`65`	`65`	`return [`
`66`	`66`	`['', '/foo', '', '/baz', '/baz?v1'],`
Original file line number	Diff line number	Diff line change
`@@ -31,7 +31,7 @@ public function testGetUrl($baseUrls, $format, $path, $expected)`
`31`	`31`	`$this->assertSame($expected, $package->getUrl($path));`
`32`	`32`	`}`
`33`	`33`
`34`		`- public function getConfigs()`
	`34`	`+ public static function getConfigs()`
`35`	`35`	`{`
`36`	`36`	`return [`
`37`	`37`	`['http://example.net', '', 'http://example.com/foo', 'http://example.com/foo'],`
`@@ -72,7 +72,7 @@ public function testGetUrlWithContext($secure, $baseUrls, $format, $path, $expec`
`72`	`72`	`$this->assertSame($expected, $package->getUrl($path));`
`73`	`73`	`}`
`74`	`74`
`75`		`- public function getContextConfigs()`
	`75`	`+ public static function getContextConfigs()`
`76`	`76`	`{`
`77`	`77`	`return [`
`78`	`78`	`[false, 'http://example.com', '', 'foo', 'http://example.com/foo?v1'],`
`@@ -114,7 +114,7 @@ public function testWrongBaseUrl($baseUrls)`
`114`	`114`	`new UrlPackage($baseUrls, new EmptyVersionStrategy());`
`115`	`115`	`}`
`116`	`116`
`117`		`- public function getWrongBaseUrlConfig()`
	`117`	`+ public static function getWrongBaseUrlConfig()`
`118`	`118`	`{`
`119`	`119`	`return [`
`120`	`120`	`['not-a-url'],`
Original file line number	Diff line number	Diff line change
`@@ -34,7 +34,7 @@ public function testApplyVersion($path, $version, $format)`
`34`	`34`	`$this->assertSame($formatted, $staticVersionStrategy->applyVersion($path));`
`35`	`35`	`}`
`36`	`36`
`37`		`- public function getConfigs()`
	`37`	`+ public static function getConfigs()`
`38`	`38`	`{`
`39`	`39`	`return [`
`40`	`40`	`['test-path', 'v1', null],`