[HttpKernel] Code style fixes

ilyachase · ilyachase · commit 5ed5a4238057 · 2024-02-29T16:28:00.000+05:00
diff --git a/src/Symfony/Component/HttpKernel/Tests/Controller/ArgumentResolver/RequestValueResolverTest.php b/src/Symfony/Component/HttpKernel/Tests/Controller/ArgumentResolver/RequestValueResolverTest.php
@@ -20,7 +20,7 @@
 
 class RequestValueResolverTest extends TestCase
 {
-    public function testSameRequestReturned(): void
+    public function testSameRequestReturned()
     {
         $resolver = new RequestValueResolver();
         $expectedRequest = Request::create('/');
@@ -29,15 +29,15 @@ public function testSameRequestReturned(): void
         self::assertSame($expectedRequest, $actualRequest[0] ?? null);
     }
 
-    public function testRequestIsNotResolvedForRandomClass(): void
+    public function testRequestIsNotResolvedForRandomClass()
     {
         $resolver = new RequestValueResolver();
         $expectedRequest = Request::create('/');
         $actualRequest = $resolver->resolve($expectedRequest, new ArgumentMetadata('request', self::class, false, false, null));
         self::assertCount(0, $actualRequest);
     }
 
-    public function testExceptionThrownForRandomRequestClass(): void
+    public function testExceptionThrownForRandomRequestClass()
     {
         $resolver = new RequestValueResolver();
         $expectedRequest = Request::create('/');
diff --git a/src/Symfony/Component/HttpKernel/Tests/Controller/ArgumentResolver/ServiceValueResolverTest.php b/src/Symfony/Component/HttpKernel/Tests/Controller/ArgumentResolver/ServiceValueResolverTest.php
@@ -13,7 +13,6 @@
 
 use PHPUnit\Framework\TestCase;
 use Symfony\Component\DependencyInjection\ContainerBuilder;
-use Symfony\Component\DependencyInjection\Exception\RuntimeException;
 use Symfony\Component\DependencyInjection\ServiceLocator;
 use Symfony\Component\HttpFoundation\Request;
 use Symfony\Component\HttpKernel\Controller\ArgumentResolver\ServiceValueResolver;
diff --git a/src/Symfony/Component/HttpKernel/Tests/Controller/ArgumentResolverTest.php b/src/Symfony/Component/HttpKernel/Tests/Controller/ArgumentResolverTest.php
@@ -347,7 +347,7 @@ public function testUnknownTargetedResolver()
         $resolver->getArguments($request, $controller);
     }
 
-    public function testResolversChainCompletionWhenResolverThrowsSpecialException(): void
+    public function testResolversChainCompletionWhenResolverThrowsSpecialException()
     {
         $failingValueResolver = new class() implements ValueResolverInterface {
             public function resolve(Request $request, ArgumentMetadata $argument): iterable
@@ -367,7 +367,7 @@ public function resolve(Request $request, ArgumentMetadata $argument): iterable
         self::assertEquals([123], $actualArguments);
     }
 
-    public function testExceptionList(): void
+    public function testExceptionList()
     {
         $failingValueResolverOne = new class() implements ValueResolverInterface {
             public function resolve(Request $request, ArgumentMetadata $argument): iterable

Original file line number	Diff line number	Diff line change
`@@ -20,7 +20,7 @@`
`20`	`20`
`21`	`21`	`class RequestValueResolverTest extends TestCase`
`22`	`22`	`{`
`23`		`- public function testSameRequestReturned(): void`
	`23`	`+ public function testSameRequestReturned()`
`24`	`24`	`{`
`25`	`25`	`$resolver = new RequestValueResolver();`
`26`	`26`	`$expectedRequest = Request::create('/');`
`@@ -29,15 +29,15 @@ public function testSameRequestReturned(): void`
`29`	`29`	`self::assertSame($expectedRequest, $actualRequest[0] ?? null);`
`30`	`30`	`}`
`31`	`31`
`32`		`- public function testRequestIsNotResolvedForRandomClass(): void`
	`32`	`+ public function testRequestIsNotResolvedForRandomClass()`
`33`	`33`	`{`
`34`	`34`	`$resolver = new RequestValueResolver();`
`35`	`35`	`$expectedRequest = Request::create('/');`
`36`	`36`	`$actualRequest = $resolver->resolve($expectedRequest, new ArgumentMetadata('request', self::class, false, false, null));`
`37`	`37`	`self::assertCount(0, $actualRequest);`
`38`	`38`	`}`
`39`	`39`
`40`		`- public function testExceptionThrownForRandomRequestClass(): void`
	`40`	`+ public function testExceptionThrownForRandomRequestClass()`
`41`	`41`	`{`
`42`	`42`	`$resolver = new RequestValueResolver();`
`43`	`43`	`$expectedRequest = Request::create('/');`
Original file line number	Diff line number	Diff line change
`@@ -347,7 +347,7 @@ public function testUnknownTargetedResolver()`
`347`	`347`	`$resolver->getArguments($request, $controller);`
`348`	`348`	`}`
`349`	`349`
`350`		`- public function testResolversChainCompletionWhenResolverThrowsSpecialException(): void`
	`350`	`+ public function testResolversChainCompletionWhenResolverThrowsSpecialException()`
`351`	`351`	`{`
`352`	`352`	`$failingValueResolver = new class() implements ValueResolverInterface {`
`353`	`353`	`public function resolve(Request $request, ArgumentMetadata $argument): iterable`
`@@ -367,7 +367,7 @@ public function resolve(Request $request, ArgumentMetadata $argument): iterable`
`367`	`367`	`self::assertEquals([123], $actualArguments);`
`368`	`368`	`}`
`369`	`369`
`370`		`- public function testExceptionList(): void`
	`370`	`+ public function testExceptionList()`
`371`	`371`	`{`
`372`	`372`	`$failingValueResolverOne = new class() implements ValueResolverInterface {`
`373`	`373`	`public function resolve(Request $request, ArgumentMetadata $argument): iterable`