Anonymous class CS fixes

GrahamCampbell · GrahamCampbell · commit cee4dae42f4a · 2021-05-02T20:56:46.000+01:00
diff --git a/src/Illuminate/Foundation/Testing/Concerns/InteractsWithExceptionHandling.php b/src/Illuminate/Foundation/Testing/Concerns/InteractsWithExceptionHandling.php
@@ -64,7 +64,8 @@ protected function withoutExceptionHandling(array $except = [])
             $this->originalExceptionHandler = app(ExceptionHandler::class);
         }
 
-        $this->app->instance(ExceptionHandler::class, new class($this->originalExceptionHandler, $except) implements ExceptionHandler {
+        $this->app->instance(ExceptionHandler::class, new class($this->originalExceptionHandler, $except) implements ExceptionHandler
+        {
             protected $except;
             protected $originalHandler;
 
diff --git a/src/Illuminate/Foundation/Testing/Concerns/MakesHttpRequests.php b/src/Illuminate/Foundation/Testing/Concerns/MakesHttpRequests.php
@@ -121,7 +121,8 @@ public function withoutMiddleware($middleware = null)
         }
 
         foreach ((array) $middleware as $abstract) {
-            $this->app->instance($abstract, new class {
+            $this->app->instance($abstract, new class
+            {
                 public function handle($request, $next)
                 {
                     return $next($request);
diff --git a/tests/Auth/AuthenticatableTest.php b/tests/Auth/AuthenticatableTest.php
@@ -23,7 +23,8 @@ public function testItReturnsStringAsRememberTokenWhenItWasSetToTrue()
 
     public function testItReturnsNullWhenRememberTokenNameWasSetToEmpty()
     {
-        $user = new class extends User {
+        $user = new class extends User
+        {
             public function getRememberTokenName()
             {
                 return '';
diff --git a/tests/Console/CommandTest.php b/tests/Console/CommandTest.php
@@ -48,7 +48,8 @@ public function testCallingClassCommandResolveCommandViaApplicationResolution()
 
     public function testGettingCommandArgumentsAndOptionsByClass()
     {
-        $command = new class extends Command {
+        $command = new class extends Command
+        {
             public function handle()
             {
             }
diff --git a/tests/Database/DatabaseEloquentCollectionTest.php b/tests/Database/DatabaseEloquentCollectionTest.php
@@ -454,17 +454,22 @@ public function testQueueableCollectionImplementationThrowsExceptionOnMultipleMo
     public function testQueueableRelationshipsReturnsOnlyRelationsCommonToAllModels()
     {
         // This is needed to prevent loading non-existing relationships on polymorphic model collections (#26126)
-        $c = new Collection([new class {
-            public function getQueueableRelations()
+        $c = new Collection([
+            new class
             {
-                return ['user'];
-            }
-        }, new class {
-            public function getQueueableRelations()
+                public function getQueueableRelations()
+                {
+                    return ['user'];
+                }
+            },
+            new class
             {
-                return ['user', 'comments'];
-            }
-        }]);
+                public function getQueueableRelations()
+                {
+                    return ['user', 'comments'];
+                }
+            },
+        ]);
 
         $this->assertEquals(['user'], $c->getQueueableRelations());
     }
diff --git a/tests/Database/DatabaseSoftDeletingTest.php b/tests/Database/DatabaseSoftDeletingTest.php
@@ -18,7 +18,8 @@ public function testDeletedAtIsAddedToDateCasts()
 
     public function testDeletedAtIsUniqueWhenAlreadyExists()
     {
-        $model = new class extends SoftDeletingModel {
+        $model = new class extends SoftDeletingModel
+        {
             protected $dates = ['deleted_at'];
         };
         $entries = array_filter($model->getDates(), function ($attribute) {
@@ -40,7 +41,8 @@ public function testDeletedAtIsCastToCarbonInstance()
 
     public function testExistingCastOverridesAddedDateCast()
     {
-        $model = new class(['deleted_at' => '2018-12-29 13:59:39']) extends SoftDeletingModel {
+        $model = new class(['deleted_at' => '2018-12-29 13:59:39']) extends SoftDeletingModel
+        {
             protected $casts = ['deleted_at' => 'bool'];
         };
 
@@ -49,7 +51,8 @@ public function testExistingCastOverridesAddedDateCast()
 
     public function testExistingMutatorOverridesAddedDateCast()
     {
-        $model = new class(['deleted_at' => '2018-12-29 13:59:39']) extends SoftDeletingModel {
+        $model = new class(['deleted_at' => '2018-12-29 13:59:39']) extends SoftDeletingModel
+        {
             protected function getDeletedAtAttribute()
             {
                 return 'expected';
@@ -61,7 +64,8 @@ protected function getDeletedAtAttribute()
 
     public function testCastingToStringOverridesAutomaticDateCastingToRetainPreviousBehaviour()
     {
-        $model = new class(['deleted_at' => '2018-12-29 13:59:39']) extends SoftDeletingModel {
+        $model = new class(['deleted_at' => '2018-12-29 13:59:39']) extends SoftDeletingModel
+        {
             protected $casts = ['deleted_at' => 'string'];
         };
 
diff --git a/tests/Foundation/FoundationApplicationTest.php b/tests/Foundation/FoundationApplicationTest.php
@@ -45,7 +45,8 @@ public function testServiceProvidersAreCorrectlyRegistered()
     public function testClassesAreBoundWhenServiceProviderIsRegistered()
     {
         $app = new Application;
-        $app->register($provider = new class($app) extends ServiceProvider {
+        $app->register($provider = new class($app) extends ServiceProvider
+        {
             public $bindings = [
                 AbstractClass::class => ConcreteClass::class,
             ];
@@ -62,7 +63,8 @@ public function testClassesAreBoundWhenServiceProviderIsRegistered()
     public function testSingletonsAreCreatedWhenServiceProviderIsRegistered()
     {
         $app = new Application;
-        $app->register($provider = new class($app) extends ServiceProvider {
+        $app->register($provider = new class($app) extends ServiceProvider
+        {
             public $singletons = [
                 AbstractClass::class => ConcreteClass::class,
             ];
diff --git a/tests/Foundation/FoundationTestResponseTest.php b/tests/Foundation/FoundationTestResponseTest.php
@@ -39,7 +39,8 @@ public function testAssertViewHas()
 
     public function testAssertViewHasModel()
     {
-        $model = new class extends Model {
+        $model = new class extends Model
+        {
             public function is($model)
             {
                 return $this == $model;
diff --git a/tests/Foundation/Http/Middleware/CheckForMaintenanceModeTest.php b/tests/Foundation/Http/Middleware/CheckForMaintenanceModeTest.php
@@ -105,7 +105,8 @@ public function testApplicationAllowsSomeURIs()
     {
         $app = $this->createMaintenanceApplication();
 
-        $middleware = new class($app) extends CheckForMaintenanceMode {
+        $middleware = new class($app) extends CheckForMaintenanceMode
+        {
             public function __construct($app)
             {
                 parent::__construct($app);
diff --git a/tests/Integration/Foundation/FoundationHelpersTest.php b/tests/Integration/Foundation/FoundationHelpersTest.php
@@ -29,7 +29,8 @@ public function testRescue()
             return 'no need to rescue';
         }, 'rescued!'), 'no need to rescue');
 
-        $testClass = new class {
+        $testClass = new class
+        {
             public function test(int $a)
             {
                 return $a;
diff --git a/tests/Integration/Http/ResourceTest.php b/tests/Integration/Http/ResourceTest.php
@@ -791,7 +791,8 @@ public function testKeysArePreservedInAnAnonymousColletionIfTheResourceIsFlagged
 
     public function testLeadingMergeKeyedValueIsMergedCorrectly()
     {
-        $filter = new class {
+        $filter = new class
+        {
             use ConditionallyLoadsAttributes;
 
             public function work()
@@ -811,7 +812,8 @@ public function work()
 
     public function testLeadingMergeKeyedValueIsMergedCorrectlyWhenFirstValueIsMissing()
     {
-        $filter = new class {
+        $filter = new class
+        {
             use ConditionallyLoadsAttributes;
 
             public function work()
@@ -835,7 +837,8 @@ public function work()
 
     public function testLeadingMergeValueIsMergedCorrectly()
     {
-        $filter = new class {
+        $filter = new class
+        {
             use ConditionallyLoadsAttributes;
 
             public function work()
@@ -860,7 +863,8 @@ public function work()
 
     public function testMergeValuesMayBeMissing()
     {
-        $filter = new class {
+        $filter = new class
+        {
             use ConditionallyLoadsAttributes;
 
             public function work()
@@ -885,7 +889,8 @@ public function work()
 
     public function testInitialMergeValuesMayBeMissing()
     {
-        $filter = new class {
+        $filter = new class
+        {
             use ConditionallyLoadsAttributes;
 
             public function work()
@@ -910,7 +915,8 @@ public function work()
 
     public function testMergeValueCanMergeJsonSerializable()
     {
-        $filter = new class {
+        $filter = new class
+        {
             use ConditionallyLoadsAttributes;
 
             public function work()
@@ -941,7 +947,8 @@ public function work()
 
     public function testMergeValueCanMergeCollectionOfJsonSerializable()
     {
-        $filter = new class {
+        $filter = new class
+        {
             use ConditionallyLoadsAttributes;
 
             public function work()
@@ -967,7 +974,8 @@ public function work()
 
     public function testAllMergeValuesMayBeMissing()
     {
-        $filter = new class {
+        $filter = new class
+        {
             use ConditionallyLoadsAttributes;
 
             public function work()
@@ -992,7 +1000,8 @@ public function work()
 
     public function testNestedMerges()
     {
-        $filter = new class {
+        $filter = new class
+        {
             use ConditionallyLoadsAttributes;
 
             public function work()
diff --git a/tests/Integration/Queue/CallQueuedHandlerTest.php b/tests/Integration/Queue/CallQueuedHandlerTest.php
@@ -156,7 +156,8 @@ public function handle()
     public function middleware()
     {
         return [
-            new class {
+            new class
+            {
                 public function handle($command, $next)
                 {
                     CallQueuedHandlerTestJobWithMiddleware::$middlewareCommand = $command;
diff --git a/tests/Pagination/PaginatorLoadMorphTest.php b/tests/Pagination/PaginatorLoadMorphTest.php
@@ -19,7 +19,8 @@ public function testCollectionLoadMorphCanChainOnThePaginator()
         $items = m::mock(Collection::class);
         $items->shouldReceive('loadMorph')->once()->with('parentable', $relations);
 
-        $p = (new class extends AbstractPaginator {
+        $p = (new class extends AbstractPaginator
+        {
             //
         })->setCollection($items);
 
diff --git a/tests/Support/SupportCollectionTest.php b/tests/Support/SupportCollectionTest.php
@@ -495,15 +495,17 @@ public function testHigherOrderUnique($collection)
     public function testHigherOrderFilter($collection)
     {
         $c = new $collection([
-            new class {
+            new class
+            {
                 public $name = 'Alex';
 
                 public function active()
                 {
                     return true;
                 }
             },
-            new class {
+            new class
+            {
                 public $name = 'John';
 
                 public function active()
diff --git a/tests/Support/SupportHelpersTest.php b/tests/Support/SupportHelpersTest.php
@@ -385,7 +385,8 @@ public function testOptional()
     {
         $this->assertNull(optional(null)->something());
 
-        $this->assertEquals(10, optional(new class {
+        $this->assertEquals(10, optional(new class
+        {
             public function something()
             {
                 return 10;
@@ -463,10 +464,12 @@ public function testOptionalIsMacroable()
 
         $this->assertNull(optional(null)->present()->something());
 
-        $this->assertSame('$10.00', optional(new class {
+        $this->assertSame('$10.00', optional(new class
+        {
             public function present()
             {
-                return new class {
+                return new class
+                {
                     public function something()
                     {
                         return '$10.00';
diff --git a/tests/Validation/ValidationValidatorTest.php b/tests/Validation/ValidationValidatorTest.php
@@ -2403,7 +2403,8 @@ public function testValidateEmail()
         $this->assertFalse($v->passes());
 
         $v = new Validator($trans, [
-            'x' => new class {
+            'x' => new class
+            {
                 public function __toString()
                 {
                     return 'aslsdlks';
@@ -2413,7 +2414,8 @@ public function __toString()
         $this->assertFalse($v->passes());
 
         $v = new Validator($trans, [
-            'x' => new class {
+            'x' => new class
+            {
                 public function __toString()
                 {
                     return 'foo@gmail.com';
@@ -4792,7 +4794,8 @@ public function testCustomValidationObject()
             $this->getIlluminateArrayTranslator(),
             ['name' => 'taylor'],
             [
-                'name' => new class implements Rule {
+                'name' => new class implements Rule
+                {
                     public function passes($attribute, $value)
                     {
                         return $value === 'taylor';
@@ -4814,7 +4817,8 @@ public function message()
             ['name' => 'adam'],
             [
                 'name' => [
-                    new class implements Rule {
+                    new class implements Rule
+                    {
                         public function passes($attribute, $value)
                         {
                             return $value === 'taylor';
@@ -4868,7 +4872,8 @@ public function message()
             $this->getIlluminateArrayTranslator(),
             ['name' => 'taylor', 'states' => ['AR', 'TX'], 'number' => 9],
             [
-                'states.*' => new class implements Rule {
+                'states.*' => new class implements Rule
+                {
                     public function passes($attribute, $value)
                     {
                         return in_array($value, ['AK', 'HI']);
@@ -4906,7 +4911,8 @@ function ($attribute, $value, $fail) {
             $this->getIlluminateArrayTranslator(),
             ['name' => 42],
             [
-                'name' => new class implements Rule {
+                'name' => new class implements Rule
+                {
                     public function passes($attribute, $value)
                     {
                         return $value === 'taylor';
@@ -4930,7 +4936,8 @@ public function message()
             ['name' => 42],
             [
                 'name' => [
-                    new class implements Rule {
+                    new class implements Rule
+                    {
                         public function passes($attribute, $value)
                         {
                             return $value === 'taylor';
@@ -4958,7 +4965,8 @@ public function testImplicitCustomValidationObjects()
             $this->getIlluminateArrayTranslator(),
             ['name' => ''],
             [
-                'name' => $rule = new class implements ImplicitRule {
+                'name' => $rule = new class implements ImplicitRule
+                {
                     public $called = false;
 
                     public function passes($attribute, $value)
diff --git a/tests/View/ViewFactoryTest.php b/tests/View/ViewFactoryTest.php

Original file line number	Diff line number	Diff line change
`@@ -64,7 +64,8 @@ protected function withoutExceptionHandling(array $except = [])`
`64`	`64`	`$this->originalExceptionHandler = app(ExceptionHandler::class);`
`65`	`65`	`}`
`66`	`66`
`67`		`- $this->app->instance(ExceptionHandler::class, new class($this->originalExceptionHandler, $except) implements ExceptionHandler {`
	`67`	`+ $this->app->instance(ExceptionHandler::class, new class($this->originalExceptionHandler, $except) implements ExceptionHandler`
	`68`	`+ {`
`68`	`69`	`protected $except;`
`69`	`70`	`protected $originalHandler;`
`70`	`71`
Original file line number	Diff line number	Diff line change
`@@ -121,7 +121,8 @@ public function withoutMiddleware($middleware = null)`
`121`	`121`	`}`
`122`	`122`
`123`	`123`	`foreach ((array) $middleware as $abstract) {`
`124`		`- $this->app->instance($abstract, new class {`
	`124`	`+ $this->app->instance($abstract, new class`
	`125`	`+ {`
`125`	`126`	`public function handle($request, $next)`
`126`	`127`	`{`
`127`	`128`	`return $next($request);`
Original file line number	Diff line number	Diff line change
`@@ -23,7 +23,8 @@ public function testItReturnsStringAsRememberTokenWhenItWasSetToTrue()`
`23`	`23`
`24`	`24`	`public function testItReturnsNullWhenRememberTokenNameWasSetToEmpty()`
`25`	`25`	`{`
`26`		`- $user = new class extends User {`
	`26`	`+ $user = new class extends User`
	`27`	`+ {`
`27`	`28`	`public function getRememberTokenName()`
`28`	`29`	`{`
`29`	`30`	`return '';`
Original file line number	Diff line number	Diff line change
`@@ -48,7 +48,8 @@ public function testCallingClassCommandResolveCommandViaApplicationResolution()`
`48`	`48`
`49`	`49`	`public function testGettingCommandArgumentsAndOptionsByClass()`
`50`	`50`	`{`
`51`		`- $command = new class extends Command {`
	`51`	`+ $command = new class extends Command`
	`52`	`+ {`
`52`	`53`	`public function handle()`
`53`	`54`	`{`
`54`	`55`	`}`
Original file line number	Diff line number	Diff line change
`@@ -39,7 +39,8 @@ public function testAssertViewHas()`
`39`	`39`
`40`	`40`	`public function testAssertViewHasModel()`
`41`	`41`	`{`
`42`		`- $model = new class extends Model {`
	`42`	`+ $model = new class extends Model`
	`43`	`+ {`
`43`	`44`	`public function is($model)`
`44`	`45`	`{`
`45`	`46`	`return $this == $model;`
Original file line number	Diff line number	Diff line change
`@@ -105,7 +105,8 @@ public function testApplicationAllowsSomeURIs()`
`105`	`105`	`{`
`106`	`106`	`$app = $this->createMaintenanceApplication();`
`107`	`107`
`108`		`- $middleware = new class($app) extends CheckForMaintenanceMode {`
	`108`	`+ $middleware = new class($app) extends CheckForMaintenanceMode`
	`109`	`+ {`
`109`	`110`	`public function __construct($app)`
`110`	`111`	`{`
`111`	`112`	`parent::__construct($app);`
Original file line number	Diff line number	Diff line change
`@@ -29,7 +29,8 @@ public function testRescue()`
`29`	`29`	`return 'no need to rescue';`
`30`	`30`	`}, 'rescued!'), 'no need to rescue');`
`31`	`31`
`32`		`- $testClass = new class {`
	`32`	`+ $testClass = new class`
	`33`	`+ {`
`33`	`34`	`public function test(int $a)`
`34`	`35`	`{`
`35`	`36`	`return $a;`