[9.x] use static data providers in tests (#45049)

IonBazan · web-flow · commit 2641bc172640 · 2022-11-22T15:10:22.000-06:00
* use static data providers

* Fix remaining getTestDate() call
diff --git a/tests/Auth/AuthAccessGateTest.php b/tests/Auth/AuthAccessGateTest.php
@@ -570,7 +570,7 @@ public function testDefineSecondParameterShouldBeStringOrCallable($callback)
     /**
      * @return array
      */
-    public function notCallableDataProvider()
+    public static function notCallableDataProvider()
     {
         return [
             [1],
@@ -1051,7 +1051,7 @@ public function testHasAbilities($abilitiesToSet, $abilitiesToCheck, $expectedHa
         $this->assertEquals($expectedHasValue, $gate->has($abilitiesToCheck));
     }
 
-    public function hasAbilitiesTestDataProvider()
+    public static function hasAbilitiesTestDataProvider()
     {
         $abilities = ['foo' => 'foo', 'bar' => 'bar'];
         $noAbilities = [];
diff --git a/tests/Broadcasting/BroadcasterTest.php b/tests/Broadcasting/BroadcasterTest.php
@@ -322,7 +322,7 @@ public function testChannelNameMatchPattern($channel, $pattern, $shouldMatch)
         $this->assertEquals($shouldMatch, $this->broadcaster->channelNameMatchesPattern($channel, $pattern));
     }
 
-    public function channelNameMatchPatternProvider()
+    public static function channelNameMatchPatternProvider()
     {
         return [
             ['something', 'something', true],
diff --git a/tests/Broadcasting/UsePusherChannelsNamesTest.php b/tests/Broadcasting/UsePusherChannelsNamesTest.php
@@ -44,7 +44,7 @@ public function testIsGuardedChannel($requestChannelName, $_, $guarded)
         );
     }
 
-    public function channelsProvider()
+    public static function channelsProvider()
     {
         $prefixesInfos = [
             ['prefix' => 'private-', 'guarded' => true],
diff --git a/tests/Bus/BusBatchTest.php b/tests/Bus/BusBatchTest.php
@@ -448,7 +448,7 @@ public function test_options_unserialize_on_postgres($serialize, $options)
     /**
      * @return array
      */
-    public function serializedOptions()
+    public static function serializedOptions()
     {
         $options = [1, 2];
 
diff --git a/tests/Cache/CacheRepositoryTest.php b/tests/Cache/CacheRepositoryTest.php
@@ -27,7 +27,7 @@ protected function setUp(): void
     {
         parent::setUp();
 
-        Carbon::setTestNow(Carbon::parse($this->getTestDate()));
+        Carbon::setTestNow(Carbon::parse(self::getTestDate()));
     }
 
     protected function tearDown(): void
@@ -261,12 +261,12 @@ public function testAddWithDatetimeInPastOrZeroSecondsReturnsImmediately()
         $this->assertFalse($result);
     }
 
-    public function dataProviderTestGetSeconds()
+    public static function dataProviderTestGetSeconds()
     {
         return [
-            [Carbon::parse($this->getTestDate())->addMinutes(5)],
-            [(new DateTime($this->getTestDate()))->modify('+5 minutes')],
-            [(new DateTimeImmutable($this->getTestDate()))->modify('+5 minutes')],
+            [Carbon::parse(self::getTestDate())->addMinutes(5)],
+            [(new DateTime(self::getTestDate()))->modify('+5 minutes')],
+            [(new DateTimeImmutable(self::getTestDate()))->modify('+5 minutes')],
             [new DateInterval('PT5M')],
             [300],
         ];
@@ -438,7 +438,7 @@ protected function getRepository()
         return $repository;
     }
 
-    protected function getTestDate()
+    protected static function getTestDate()
     {
         return '2030-07-25 12:13:14 UTC';
     }
diff --git a/tests/Database/DatabaseConnectorTest.php b/tests/Database/DatabaseConnectorTest.php
@@ -45,7 +45,7 @@ public function testMySqlConnectCallsCreateConnectionWithProperArguments($dsn, $
         $this->assertSame($result, $connection);
     }
 
-    public function mySqlConnectProvider()
+    public static function mySqlConnectProvider()
     {
         return [
             ['mysql:host=foo;dbname=bar', ['host' => 'foo', 'database' => 'bar', 'collation' => 'utf8_unicode_ci', 'charset' => 'utf8']],
@@ -112,7 +112,7 @@ public function testPostgresSearchPathIsSet($searchPath, $expectedSql)
         $this->assertSame($result, $connection);
     }
 
-    public function provideSearchPaths()
+    public static function provideSearchPaths()
     {
         return [
             'all-lowercase' => [
diff --git a/tests/Database/DatabaseMySqlSchemaStateTest.php b/tests/Database/DatabaseMySqlSchemaStateTest.php
@@ -33,7 +33,7 @@ public function testConnectionString(string $expectedConnectionString, array $ex
         self::assertEquals($expectedVariables, $variables);
     }
 
-    public function provider(): Generator
+    public static function provider(): Generator
     {
         yield 'default' => [
             ' --user="${:LARAVEL_LOAD_USER}" --password="${:LARAVEL_LOAD_PASSWORD}" --host="${:LARAVEL_LOAD_HOST}" --port="${:LARAVEL_LOAD_PORT}"', [
diff --git a/tests/Http/HttpJsonResponseTest.php b/tests/Http/HttpJsonResponseTest.php
@@ -23,7 +23,7 @@ public function testSetAndRetrieveData($data)
         $this->assertSame('bar', $response->getData()->foo);
     }
 
-    public function setAndRetrieveDataProvider()
+    public static function setAndRetrieveDataProvider()
     {
         return [
             'Jsonable data' => [new JsonResponseTestJsonableObject],
@@ -85,7 +85,7 @@ public function testGracefullyHandledSomeJsonErrorsWithPartialOutputOnError($dat
         new JsonResponse(['data' => $data], 200, [], JSON_PARTIAL_OUTPUT_ON_ERROR);
     }
 
-    public function jsonErrorDataProvider()
+    public static function jsonErrorDataProvider()
     {
         // Resources can't be encoded
         $resource = tmpfile();
diff --git a/tests/Http/HttpRequestTest.php b/tests/Http/HttpRequestTest.php
@@ -89,7 +89,7 @@ public function testSegmentMethod($path, $segment, $expected)
         $this->assertEquals($expected, $request->segment($segment, 'default'));
     }
 
-    public function segmentProvider()
+    public static function segmentProvider()
     {
         return [
             ['', 1, 'default'],
@@ -111,7 +111,7 @@ public function testSegmentsMethod($path, $expected)
         $this->assertEquals(['foo', 'bar'], $request->segments());
     }
 
-    public function segmentsProvider()
+    public static function segmentsProvider()
     {
         return [
             ['', []],
@@ -1012,7 +1012,7 @@ public function testJSONEmulatingPHPBuiltInServer()
         $this->assertEquals($payload, $data);
     }
 
-    public function getPrefersCases()
+    public static function getPrefersCases()
     {
         return [
             ['application/json', ['json'], 'json'],
diff --git a/tests/Integration/Console/CommandSchedulingTest.php b/tests/Integration/Console/CommandSchedulingTest.php
@@ -95,7 +95,7 @@ public function testExecutionOrder($background)
         $this->assertLogged('before', 'handled', 'after');
     }
 
-    public function executionProvider()
+    public static function executionProvider()
     {
         return [
             'Foreground' => [false],
diff --git a/tests/Integration/Database/EloquentWhereHasTest.php b/tests/Integration/Database/EloquentWhereHasTest.php
@@ -87,7 +87,7 @@ public function testOrWhereRelationCallback($callbackEloquent, $callbackQuery)
         $this->assertEquals($userOrWhereHas->first()->id, $query->first()->id);
     }
 
-    public function dataProviderWhereRelationCallback()
+    public static function dataProviderWhereRelationCallback()
     {
         $callbackArray = function ($value) {
             $callbackEloquent = function (EloquentBuilder $builder) use ($value) {
diff --git a/tests/Integration/Database/MySql/DatabaseMySqlConnectionTest.php b/tests/Integration/Database/MySql/DatabaseMySqlConnectionTest.php
@@ -46,7 +46,7 @@ public function testJsonFloatComparison($value, $operator, $shouldMatch)
         );
     }
 
-    public function floatComparisonsDataProvider()
+    public static function floatComparisonsDataProvider()
     {
         return [
             [0.2, '=', true],
@@ -88,7 +88,7 @@ public function testJsonWhereNotNull($expected, $key, array $value = ['value' =>
         $this->assertSame(! $expected, DB::table(self::TABLE)->whereNotNull(self::JSON_COL.'->'.$key)->exists());
     }
 
-    public function jsonWhereNullDataProvider()
+    public static function jsonWhereNullDataProvider()
     {
         return [
             'key not exists' => [true, 'invalid'],
@@ -138,7 +138,7 @@ public function testWhereJsonContainsKey($count, $column)
         $this->assertSame($count, DB::table(self::TABLE)->whereJsonContainsKey($column)->count());
     }
 
-    public function jsonContainsKeyDataProvider()
+    public static function jsonContainsKeyDataProvider()
     {
         return [
             'string key' => [4, 'json_col->foo'],
diff --git a/tests/Integration/Database/Postgres/DatabasePostgresConnectionTest.php b/tests/Integration/Database/Postgres/DatabasePostgresConnectionTest.php
@@ -46,7 +46,7 @@ public function testJsonWhereNotNull($expected, $key, array $value = ['value' =>
         $this->assertSame(! $expected, DB::table('json_table')->whereNotNull("json_col->$key")->exists());
     }
 
-    public function jsonWhereNullDataProvider()
+    public static function jsonWhereNullDataProvider()
     {
         return [
             'key not exists' => [true, 'invalid'],
@@ -107,7 +107,7 @@ public function testWhereJsonContainsKey($count, $column)
         $this->assertSame($count, DB::table('json_table')->whereJsonContainsKey($column)->count());
     }
 
-    public function jsonContainsKeyDataProvider()
+    public static function jsonContainsKeyDataProvider()
     {
         return [
             'string key' => [4, 'json_col->foo'],
diff --git a/tests/Integration/Database/SqlServer/DatabaseSqlServerConnectionTest.php b/tests/Integration/Database/SqlServer/DatabaseSqlServerConnectionTest.php
@@ -42,7 +42,7 @@ public function testWhereJsonContainsKey($count, $column)
         $this->assertSame($count, DB::table('json_table')->whereJsonContainsKey($column)->count());
     }
 
-    public function jsonContainsKeyDataProvider()
+    public static function jsonContainsKeyDataProvider()
     {
         return [
             'string key' => [4, 'json_col->foo'],
diff --git a/tests/Integration/Database/Sqlite/DatabaseSqliteConnectionTest.php b/tests/Integration/Database/Sqlite/DatabaseSqliteConnectionTest.php
@@ -54,7 +54,7 @@ public function testWhereJsonContainsKey($count, $column)
         $this->assertSame($count, DB::table('json_table')->whereJsonContainsKey($column)->count());
     }
 
-    public function jsonContainsKeyDataProvider()
+    public static function jsonContainsKeyDataProvider()
     {
         return [
             'string key' => [4, 'json_col->foo'],
diff --git a/tests/Integration/Queue/CustomPayloadTest.php b/tests/Integration/Queue/CustomPayloadTest.php
@@ -18,7 +18,7 @@ protected function getPackageProviders($app)
         return [QueueServiceProvider::class];
     }
 
-    public function websites()
+    public static function websites()
     {
         yield ['laravel.com'];
 
diff --git a/tests/Mail/MailManagerTest.php b/tests/Mail/MailManagerTest.php
@@ -27,7 +27,7 @@ public function testEmptyTransportConfig($transport)
         $this->app['mail.manager']->mailer('custom_smtp');
     }
 
-    public function emptyTransportConfigDataProvider()
+    public static function emptyTransportConfigDataProvider()
     {
         return [
             [null], [''], [' '],
diff --git a/tests/Routing/RouteUriTest.php b/tests/Routing/RouteUriTest.php
@@ -20,7 +20,7 @@ public function testRouteUrisAreProperlyParsed($uri, $expectedParsedUri, $expect
     /**
      * @return array
      */
-    public function uriProvider()
+    public static function uriProvider()
     {
         return [
             [
diff --git a/tests/Routing/RoutingUrlGeneratorTest.php b/tests/Routing/RoutingUrlGeneratorTest.php
@@ -571,7 +571,7 @@ public function testRoutesWithDomainsThroughProxy()
         $this->assertSame('http://sub.foo.com/foo/bar', $url->route('foo'));
     }
 
-    public function providerRouteParameters()
+    public static function providerRouteParameters()
     {
         return [
             [['test' => 123]],
@@ -600,7 +600,7 @@ public function testUrlGenerationForControllersRequiresPassingOfRequiredParamete
         $this->assertSame('http://www.foo.com:8080/foo?test=123', $url->route('foo', $parameters));
     }
 
-    public function provideParametersAndExpectedMeaningfulExceptionMessages()
+    public static function provideParametersAndExpectedMeaningfulExceptionMessages()
     {
         return [
             'Missing parameters "one", "two" and "three"' => [
diff --git a/tests/Support/ConfigurationUrlParserTest.php b/tests/Support/ConfigurationUrlParserTest.php
@@ -45,7 +45,7 @@ public function testDriversAliases()
         ], (new ConfigurationUrlParser)->parseConfiguration('some-particular-alias://null'));
     }
 
-    public function databaseUrls()
+    public static function databaseUrls()
     {
         return [
             'simple URL' => [
diff --git a/tests/Support/SupportCollectionTest.php b/tests/Support/SupportCollectionTest.php
@@ -5464,7 +5464,7 @@ public function testUndot($collection)
      *
      * @return array
      */
-    public function collectionClassProvider()
+    public static function collectionClassProvider()
     {
         return [
             [Collection::class],
diff --git a/tests/Support/SupportHelpersTest.php b/tests/Support/SupportHelpersTest.php
@@ -892,7 +892,7 @@ public function testGetFromSERVERFirst()
         $this->assertSame('From $_SERVER', env('foo'));
     }
 
-    public function providesPregReplaceArrayData()
+    public static function providesPregReplaceArrayData()
     {
         $pointerArray = ['Taylor', 'Otwell'];
 
diff --git a/tests/Support/SupportStrTest.php b/tests/Support/SupportStrTest.php
@@ -849,7 +849,7 @@ public function testWordCount()
         $this->assertEquals(3, Str::wordCount('МАМА МЫЛА РАМУ', 'абвгдеёжзийклмнопрстуфхцчшщъыьэюяАБВГДЕЁЖЗИЙКЛМНОПРСТУФХЦЧШЩЪЫЬЭЮЯ'));
     }
 
-    public function validUuidList()
+    public static function validUuidList()
     {
         return [
             ['a0a2a2d2-0b87-4a18-83f2-2529882be2de'],
@@ -865,7 +865,7 @@ public function validUuidList()
         ];
     }
 
-    public function invalidUuidList()
+    public static function invalidUuidList()
     {
         return [
             ['not a valid uuid so we can test this'],
@@ -881,7 +881,7 @@ public function invalidUuidList()
         ];
     }
 
-    public function strContainsProvider()
+    public static function strContainsProvider()
     {
         return [
             ['Taylor', 'ylo', true, true],
@@ -900,7 +900,7 @@ public function strContainsProvider()
         ];
     }
 
-    public function strContainsAllProvider()
+    public static function strContainsAllProvider()
     {
         return [
             ['Taylor Otwell', ['taylor', 'otwell'], false, false],
@@ -938,7 +938,7 @@ public function testTransliterate(string $value, string $expected): void
         $this->assertSame($expected, Str::transliterate($value));
     }
 
-    public function specialCharacterProvider(): array
+    public static function specialCharacterProvider(): array
     {
         return [
             ['ⓐⓑⓒⓓⓔⓕⓖⓗⓘⓙⓚⓛⓜⓝⓞⓟⓠⓡⓢⓣⓤⓥⓦⓧⓨⓩ', 'abcdefghijklmnopqrstuvwxyz'],
diff --git a/tests/Testing/Concerns/TestDatabasesTest.php b/tests/Testing/Concerns/TestDatabasesTest.php
@@ -76,7 +76,7 @@ public function switchToDatabase($database)
         tap($method)->setAccessible(true)->invoke($instance, $database);
     }
 
-    public function databaseUrls()
+    public static function databaseUrls()
     {
         return [
             [
diff --git a/tests/Testing/ParallelTestingTest.php b/tests/Testing/ParallelTestingTest.php
@@ -86,7 +86,7 @@ public function testToken()
         $this->assertSame('1', (string) $parallelTesting->token());
     }
 
-    public function callbacks()
+    public static function callbacks()
     {
         return [
             ['setUpProcess'],
diff --git a/tests/Translation/TranslationMessageSelectorTest.php b/tests/Translation/TranslationMessageSelectorTest.php
@@ -17,7 +17,7 @@ public function testChoose($expected, $id, $number)
         $this->assertEquals($expected, $selector->choose($id, $number, 'en'));
     }
 
-    public function chooseTestData()
+    public static function chooseTestData()
     {
         return [
             ['first', 'first', 1],
diff --git a/tests/Validation/ValidationValidatorTest.php b/tests/Validation/ValidationValidatorTest.php
diff --git a/tests/View/Blade/BladeEchoHandlerTest.php b/tests/View/Blade/BladeEchoHandlerTest.php
diff --git a/tests/View/Blade/BladeForeachStatementsTest.php b/tests/View/Blade/BladeForeachStatementsTest.php
diff --git a/tests/View/Blade/BladeForelseStatementsTest.php b/tests/View/Blade/BladeForelseStatementsTest.php
diff --git a/tests/View/ViewBladeCompilerTest.php b/tests/View/ViewBladeCompilerTest.php
diff --git a/tests/View/ViewFileViewFinderTest.php b/tests/View/ViewFileViewFinderTest.php

Original file line number	Diff line number	Diff line change
`@@ -570,7 +570,7 @@ public function testDefineSecondParameterShouldBeStringOrCallable($callback)`
`570`	`570`	`/**`
`571`	`571`	`* @return array`
`572`	`572`	`*/`
`573`		`- public function notCallableDataProvider()`
	`573`	`+ public static function notCallableDataProvider()`
`574`	`574`	`{`
`575`	`575`	`return [`
`576`	`576`	`[1],`
`@@ -1051,7 +1051,7 @@ public function testHasAbilities($abilitiesToSet, $abilitiesToCheck, $expectedHa`
`1051`	`1051`	`$this->assertEquals($expectedHasValue, $gate->has($abilitiesToCheck));`
`1052`	`1052`	`}`
`1053`	`1053`
`1054`		`- public function hasAbilitiesTestDataProvider()`
	`1054`	`+ public static function hasAbilitiesTestDataProvider()`
`1055`	`1055`	`{`
`1056`	`1056`	`$abilities = ['foo' => 'foo', 'bar' => 'bar'];`
`1057`	`1057`	`$noAbilities = [];`
Original file line number	Diff line number	Diff line change
`@@ -322,7 +322,7 @@ public function testChannelNameMatchPattern($channel, $pattern, $shouldMatch)`
`322`	`322`	`$this->assertEquals($shouldMatch, $this->broadcaster->channelNameMatchesPattern($channel, $pattern));`
`323`	`323`	`}`
`324`	`324`
`325`		`- public function channelNameMatchPatternProvider()`
	`325`	`+ public static function channelNameMatchPatternProvider()`
`326`	`326`	`{`
`327`	`327`	`return [`
`328`	`328`	`['something', 'something', true],`
Original file line number	Diff line number	Diff line change
`@@ -44,7 +44,7 @@ public function testIsGuardedChannel($requestChannelName, $_, $guarded)`
`44`	`44`	`);`
`45`	`45`	`}`
`46`	`46`
`47`		`- public function channelsProvider()`
	`47`	`+ public static function channelsProvider()`
`48`	`48`	`{`
`49`	`49`	`$prefixesInfos = [`
`50`	`50`	`['prefix' => 'private-', 'guarded' => true],`
Original file line number	Diff line number	Diff line change
`@@ -448,7 +448,7 @@ public function test_options_unserialize_on_postgres($serialize, $options)`
`448`	`448`	`/**`
`449`	`449`	`* @return array`
`450`	`450`	`*/`
`451`		`- public function serializedOptions()`
	`451`	`+ public static function serializedOptions()`
`452`	`452`	`{`
`453`	`453`	`$options = [1, 2];`
`454`	`454`
Original file line number	Diff line number	Diff line change
`@@ -27,7 +27,7 @@ protected function setUp(): void`
`27`	`27`	`{`
`28`	`28`	`parent::setUp();`
`29`	`29`
`30`		`- Carbon::setTestNow(Carbon::parse($this->getTestDate()));`
	`30`	`+ Carbon::setTestNow(Carbon::parse(self::getTestDate()));`
`31`	`31`	`}`
`32`	`32`
`33`	`33`	`protected function tearDown(): void`
`@@ -261,12 +261,12 @@ public function testAddWithDatetimeInPastOrZeroSecondsReturnsImmediately()`
`261`	`261`	`$this->assertFalse($result);`
`262`	`262`	`}`
`263`	`263`
`264`		`- public function dataProviderTestGetSeconds()`
	`264`	`+ public static function dataProviderTestGetSeconds()`
`265`	`265`	`{`
`266`	`266`	`return [`
`267`		`- [Carbon::parse($this->getTestDate())->addMinutes(5)],`
`268`		`- [(new DateTime($this->getTestDate()))->modify('+5 minutes')],`
`269`		`- [(new DateTimeImmutable($this->getTestDate()))->modify('+5 minutes')],`
	`267`	`+ [Carbon::parse(self::getTestDate())->addMinutes(5)],`
	`268`	`+ [(new DateTime(self::getTestDate()))->modify('+5 minutes')],`
	`269`	`+ [(new DateTimeImmutable(self::getTestDate()))->modify('+5 minutes')],`
`270`	`270`	`[new DateInterval('PT5M')],`
`271`	`271`	`[300],`
`272`	`272`	`];`
`@@ -438,7 +438,7 @@ protected function getRepository()`
`438`	`438`	`return $repository;`
`439`	`439`	`}`
`440`	`440`
`441`		`- protected function getTestDate()`
	`441`	`+ protected static function getTestDate()`
`442`	`442`	`{`
`443`	`443`	`return '2030-07-25 12:13:14 UTC';`
`444`	`444`	`}`
Original file line number	Diff line number	Diff line change
`@@ -45,7 +45,7 @@ public function testMySqlConnectCallsCreateConnectionWithProperArguments($dsn, $`
`45`	`45`	`$this->assertSame($result, $connection);`
`46`	`46`	`}`
`47`	`47`
`48`		`- public function mySqlConnectProvider()`
	`48`	`+ public static function mySqlConnectProvider()`
`49`	`49`	`{`
`50`	`50`	`return [`
`51`	`51`	`['mysql:host=foo;dbname=bar', ['host' => 'foo', 'database' => 'bar', 'collation' => 'utf8_unicode_ci', 'charset' => 'utf8']],`
`@@ -112,7 +112,7 @@ public function testPostgresSearchPathIsSet($searchPath, $expectedSql)`
`112`	`112`	`$this->assertSame($result, $connection);`
`113`	`113`	`}`
`114`	`114`
`115`		`- public function provideSearchPaths()`
	`115`	`+ public static function provideSearchPaths()`
`116`	`116`	`{`
`117`	`117`	`return [`
`118`	`118`	`'all-lowercase' => [`
Original file line number	Diff line number	Diff line change
`@@ -33,7 +33,7 @@ public function testConnectionString(string $expectedConnectionString, array $ex`
`33`	`33`	`self::assertEquals($expectedVariables, $variables);`
`34`	`34`	`}`
`35`	`35`
`36`		`- public function provider(): Generator`
	`36`	`+ public static function provider(): Generator`
`37`	`37`	`{`
`38`	`38`	`yield 'default' => [`
`39`	`39`	`' --user="${:LARAVEL_LOAD_USER}" --password="${:LARAVEL_LOAD_PASSWORD}" --host="${:LARAVEL_LOAD_HOST}" --port="${:LARAVEL_LOAD_PORT}"', [`
Original file line number	Diff line number	Diff line change
`@@ -23,7 +23,7 @@ public function testSetAndRetrieveData($data)`
`23`	`23`	`$this->assertSame('bar', $response->getData()->foo);`
`24`	`24`	`}`
`25`	`25`
`26`		`- public function setAndRetrieveDataProvider()`
	`26`	`+ public static function setAndRetrieveDataProvider()`
`27`	`27`	`{`
`28`	`28`	`return [`
`29`	`29`	`'Jsonable data' => [new JsonResponseTestJsonableObject],`
`@@ -85,7 +85,7 @@ public function testGracefullyHandledSomeJsonErrorsWithPartialOutputOnError($dat`
`85`	`85`	`new JsonResponse(['data' => $data], 200, [], JSON_PARTIAL_OUTPUT_ON_ERROR);`
`86`	`86`	`}`
`87`	`87`
`88`		`- public function jsonErrorDataProvider()`
	`88`	`+ public static function jsonErrorDataProvider()`
`89`	`89`	`{`
`90`	`90`	`// Resources can't be encoded`
`91`	`91`	`$resource = tmpfile();`
Original file line number	Diff line number	Diff line change
`@@ -89,7 +89,7 @@ public function testSegmentMethod($path, $segment, $expected)`
`89`	`89`	`$this->assertEquals($expected, $request->segment($segment, 'default'));`
`90`	`90`	`}`
`91`	`91`
`92`		`- public function segmentProvider()`
	`92`	`+ public static function segmentProvider()`
`93`	`93`	`{`
`94`	`94`	`return [`
`95`	`95`	`['', 1, 'default'],`
`@@ -111,7 +111,7 @@ public function testSegmentsMethod($path, $expected)`
`111`	`111`	`$this->assertEquals(['foo', 'bar'], $request->segments());`
`112`	`112`	`}`
`113`	`113`
`114`		`- public function segmentsProvider()`
	`114`	`+ public static function segmentsProvider()`
`115`	`115`	`{`
`116`	`116`	`return [`
`117`	`117`	`['', []],`
`@@ -1012,7 +1012,7 @@ public function testJSONEmulatingPHPBuiltInServer()`
`1012`	`1012`	`$this->assertEquals($payload, $data);`
`1013`	`1013`	`}`
`1014`	`1014`
`1015`		`- public function getPrefersCases()`
	`1015`	`+ public static function getPrefersCases()`
`1016`	`1016`	`{`
`1017`	`1017`	`return [`
`1018`	`1018`	`['application/json', ['json'], 'json'],`
Original file line number	Diff line number	Diff line change
`@@ -95,7 +95,7 @@ public function testExecutionOrder($background)`
`95`	`95`	`$this->assertLogged('before', 'handled', 'after');`
`96`	`96`	`}`
`97`	`97`
`98`		`- public function executionProvider()`
	`98`	`+ public static function executionProvider()`
`99`	`99`	`{`
`100`	`100`	`return [`
`101`	`101`	`'Foreground' => [false],`