Fix more setUp signatures

andrewmy · andrewmy · commit 3731d696da04 · 2021-01-22T15:12:15.000+02:00
diff --git a/docs/bundle/functional_testing.md b/docs/bundle/functional_testing.md
@@ -57,7 +57,7 @@ class FooTest extends WebTestCase
     /** @var  \Symfony\Bundle\FrameworkBundle\Client */
     private $client;
 
-    public function setUp()
+    public function setUp(): void
     {
         $this->client = static::createClient();
     }
diff --git a/pkg/amqp-ext/Tests/Functional/AmqpCommonUseCasesTest.php b/pkg/amqp-ext/Tests/Functional/AmqpCommonUseCasesTest.php
@@ -22,7 +22,7 @@ class AmqpCommonUseCasesTest extends TestCase
      */
     private $amqpContext;
 
-    public function setUp()
+    public function setUp(): void
     {
         $this->amqpContext = $this->buildAmqpContext();
 
diff --git a/pkg/amqp-ext/Tests/Functional/AmqpConsumptionUseCasesTest.php b/pkg/amqp-ext/Tests/Functional/AmqpConsumptionUseCasesTest.php
@@ -29,7 +29,7 @@ class AmqpConsumptionUseCasesTest extends TestCase
      */
     private $amqpContext;
 
-    public function setUp()
+    public function setUp(): void
     {
         $this->amqpContext = $this->buildAmqpContext();
 
diff --git a/pkg/amqp-ext/Tests/Functional/AmqpRpcUseCasesTest.php b/pkg/amqp-ext/Tests/Functional/AmqpRpcUseCasesTest.php
@@ -23,7 +23,7 @@ class AmqpRpcUseCasesTest extends TestCase
      */
     private $amqpContext;
 
-    public function setUp()
+    public function setUp(): void
     {
         $this->amqpContext = $this->buildAmqpContext();
 
diff --git a/pkg/amqp-tools/Tests/SignalSocketHelperTest.php b/pkg/amqp-tools/Tests/SignalSocketHelperTest.php
@@ -16,7 +16,7 @@ class SignalSocketHelperTest extends TestCase
 
     private $backupSigIntHandler;
 
-    public function setUp()
+    public function setUp(): void
     {
         parent::setUp();
 
diff --git a/pkg/async-event-dispatcher/Tests/Functional/UseCasesTest.php b/pkg/async-event-dispatcher/Tests/Functional/UseCasesTest.php
@@ -53,7 +53,7 @@ class UseCasesTest extends TestCase
      */
     protected $asyncProcessor;
 
-    public function setUp()
+    public function setUp(): void
     {
         (new Filesystem())->remove(__DIR__.'/queues/');
 
diff --git a/pkg/dbal/Tests/Functional/DbalConsumerTest.php b/pkg/dbal/Tests/Functional/DbalConsumerTest.php
@@ -21,7 +21,7 @@ class DbalConsumerTest extends TestCase
      */
     private $context;
 
-    public function setUp()
+    public function setUp(): void
     {
         $this->context = $this->createDbalContext();
     }
diff --git a/pkg/dbal/Tests/Spec/Mysql/DbalSendAndReceivePriorityMessagesFromQueueTest.php b/pkg/dbal/Tests/Spec/Mysql/DbalSendAndReceivePriorityMessagesFromQueueTest.php
@@ -16,7 +16,7 @@ class DbalSendAndReceivePriorityMessagesFromQueueTest extends SendAndReceivePrio
 
     private $publishedAt;
 
-    public function setUp()
+    public function setUp(): void
     {
         parent::setUp();
 
diff --git a/pkg/dbal/Tests/Spec/Postgresql/DbalSendAndReceivePriorityMessagesFromQueueTest.php b/pkg/dbal/Tests/Spec/Postgresql/DbalSendAndReceivePriorityMessagesFromQueueTest.php
@@ -16,7 +16,7 @@ class DbalSendAndReceivePriorityMessagesFromQueueTest extends SendAndReceivePrio
 
     private $publishedAt;
 
-    public function setUp()
+    public function setUp(): void
     {
         parent::setUp();
 
diff --git a/pkg/enqueue-bundle/Tests/Functional/Events/AsyncListenerTest.php b/pkg/enqueue-bundle/Tests/Functional/Events/AsyncListenerTest.php
@@ -16,7 +16,7 @@
  */
 class AsyncListenerTest extends WebTestCase
 {
-    public function setUp()
+    public function setUp(): void
     {
         parent::setUp();
 
diff --git a/pkg/enqueue-bundle/Tests/Functional/Events/AsyncProcessorTest.php b/pkg/enqueue-bundle/Tests/Functional/Events/AsyncProcessorTest.php
@@ -18,7 +18,7 @@
  */
 class AsyncProcessorTest extends WebTestCase
 {
-    public function setUp()
+    public function setUp(): void
     {
         parent::setUp();
 
diff --git a/pkg/enqueue-bundle/Tests/Functional/Events/AsyncSubscriberTest.php b/pkg/enqueue-bundle/Tests/Functional/Events/AsyncSubscriberTest.php
@@ -16,7 +16,7 @@
  */
 class AsyncSubscriberTest extends WebTestCase
 {
-    public function setUp()
+    public function setUp(): void
     {
         parent::setUp();
 
diff --git a/pkg/enqueue-bundle/Tests/Functional/LazyProducerTest.php b/pkg/enqueue-bundle/Tests/Functional/LazyProducerTest.php
@@ -11,7 +11,7 @@
  */
 class LazyProducerTest extends WebTestCase
 {
-    public function setUp()
+    public function setUp(): void
     {
         // do not call parent::setUp.
         // parent::setUp();
diff --git a/pkg/enqueue-bundle/Tests/Functional/UseCasesTest.php b/pkg/enqueue-bundle/Tests/Functional/UseCasesTest.php
@@ -20,7 +20,7 @@ class UseCasesTest extends WebTestCase
 {
     const RECEIVE_TIMEOUT = 500;
 
-    public function setUp()
+    public function setUp(): void
     {
         // do not call parent::setUp.
         // parent::setUp();
diff --git a/pkg/fs/Tests/Functional/FsCommonUseCasesTest.php b/pkg/fs/Tests/Functional/FsCommonUseCasesTest.php
@@ -17,7 +17,7 @@ class FsCommonUseCasesTest extends \PHPUnit\Framework\TestCase
      */
     private $fsContext;
 
-    public function setUp()
+    public function setUp(): void
     {
         $this->fsContext = (new FsConnectionFactory(['path' => sys_get_temp_dir()]))->createContext();
 
diff --git a/pkg/fs/Tests/Functional/FsConsumerTest.php b/pkg/fs/Tests/Functional/FsConsumerTest.php
@@ -15,7 +15,7 @@ class FsConsumerTest extends TestCase
      */
     private $fsContext;
 
-    public function setUp()
+    public function setUp(): void
     {
         $this->fsContext = (new FsConnectionFactory(['path' => sys_get_temp_dir()]))->createContext();
 
diff --git a/pkg/fs/Tests/Functional/FsConsumptionUseCasesTest.php b/pkg/fs/Tests/Functional/FsConsumptionUseCasesTest.php
@@ -25,7 +25,7 @@ class FsConsumptionUseCasesTest extends \PHPUnit\Framework\TestCase
      */
     private $fsContext;
 
-    public function setUp()
+    public function setUp(): void
     {
         $this->fsContext = (new FsConnectionFactory(['path' => sys_get_temp_dir()]))->createContext();
 
diff --git a/pkg/fs/Tests/Functional/FsProducerTest.php b/pkg/fs/Tests/Functional/FsProducerTest.php
@@ -14,7 +14,7 @@ class FsProducerTest extends TestCase
      */
     private $fsContext;
 
-    public function setUp()
+    public function setUp(): void
     {
         $this->fsContext = (new FsConnectionFactory(['path' => sys_get_temp_dir()]))->createContext();
 
diff --git a/pkg/fs/Tests/Functional/FsRpcUseCasesTest.php b/pkg/fs/Tests/Functional/FsRpcUseCasesTest.php
@@ -20,7 +20,7 @@ class FsRpcUseCasesTest extends TestCase
      */
     private $fsContext;
 
-    public function setUp()
+    public function setUp(): void
     {
         $this->fsContext = (new FsConnectionFactory(['path' => sys_get_temp_dir()]))->createContext();
 
diff --git a/pkg/gearman/Tests/SkipIfGearmanExtensionIsNotInstalledTrait.php b/pkg/gearman/Tests/SkipIfGearmanExtensionIsNotInstalledTrait.php
@@ -4,7 +4,7 @@
 
 trait SkipIfGearmanExtensionIsNotInstalledTrait
 {
-    public function setUp()
+    public function setUp(): void
     {
         if (false == class_exists(\GearmanClient::class)) {
             $this->markTestSkipped('The gearman extension is not installed');
diff --git a/pkg/gearman/Tests/Spec/GearmanSendToTopicAndReceiveFromQueueTest.php b/pkg/gearman/Tests/Spec/GearmanSendToTopicAndReceiveFromQueueTest.php
@@ -13,7 +13,7 @@ class GearmanSendToTopicAndReceiveFromQueueTest extends SendToTopicAndReceiveFro
 {
     private $time;
 
-    public function setUp()
+    public function setUp(): void
     {
         $this->time = time();
     }
diff --git a/pkg/gearman/Tests/Spec/GearmanSendToTopicAndReceiveNoWaitFromQueueTest.php b/pkg/gearman/Tests/Spec/GearmanSendToTopicAndReceiveNoWaitFromQueueTest.php
@@ -13,7 +13,7 @@ class GearmanSendToTopicAndReceiveNoWaitFromQueueTest extends SendToTopicAndRece
 {
     private $time;
 
-    public function setUp()
+    public function setUp(): void
     {
         $this->time = time();
     }
diff --git a/pkg/mongodb/Tests/Functional/MongodbConsumerTest.php b/pkg/mongodb/Tests/Functional/MongodbConsumerTest.php
@@ -19,7 +19,7 @@ class MongodbConsumerTest extends TestCase
      */
     private $context;
 
-    public function setUp()
+    public function setUp(): void
     {
         $this->context = $this->buildMongodbContext();
     }
diff --git a/pkg/mongodb/Tests/Spec/MongodbSendAndReceivePriorityMessagesFromQueueTest.php b/pkg/mongodb/Tests/Spec/MongodbSendAndReceivePriorityMessagesFromQueueTest.php
@@ -18,7 +18,7 @@ class MongodbSendAndReceivePriorityMessagesFromQueueTest extends SendAndReceiveP
 
     private $publishedAt;
 
-    public function setUp()
+    public function setUp(): void
     {
         parent::setUp();
 
diff --git a/pkg/pheanstalk/Tests/Spec/PheanstalkSendToTopicAndReceiveFromQueueTest.php b/pkg/pheanstalk/Tests/Spec/PheanstalkSendToTopicAndReceiveFromQueueTest.php
@@ -13,7 +13,7 @@ class PheanstalkSendToTopicAndReceiveFromQueueTest extends SendToTopicAndReceive
 {
     private $time;
 
-    public function setUp()
+    public function setUp(): void
     {
         $this->time = time();
     }
diff --git a/pkg/pheanstalk/Tests/Spec/PheanstalkSendToTopicAndReceiveNoWaitFromQueueTest.php b/pkg/pheanstalk/Tests/Spec/PheanstalkSendToTopicAndReceiveNoWaitFromQueueTest.php
@@ -13,7 +13,7 @@ class PheanstalkSendToTopicAndReceiveNoWaitFromQueueTest extends SendToTopicAndR
 {
     private $time;
 
-    public function setUp()
+    public function setUp(): void
     {
         $this->time = time();
     }
diff --git a/pkg/redis/Tests/Functional/PRedisCommonUseCasesTest.php b/pkg/redis/Tests/Functional/PRedisCommonUseCasesTest.php
@@ -19,7 +19,7 @@ class PRedisCommonUseCasesTest extends TestCase
      */
     private $context;
 
-    public function setUp()
+    public function setUp(): void
     {
         $this->context = $this->buildPRedisContext();
 
diff --git a/pkg/redis/Tests/Functional/PRedisConsumptionUseCasesTest.php b/pkg/redis/Tests/Functional/PRedisConsumptionUseCasesTest.php
@@ -19,7 +19,7 @@ class PRedisConsumptionUseCasesTest extends TestCase
      */
     private $context;
 
-    public function setUp()
+    public function setUp(): void
     {
         $this->context = $this->buildPRedisContext();
 
diff --git a/pkg/redis/Tests/Functional/PhpRedisCommonUseCasesTest.php b/pkg/redis/Tests/Functional/PhpRedisCommonUseCasesTest.php
@@ -19,7 +19,7 @@ class PhpRedisCommonUseCasesTest extends TestCase
      */
     private $context;
 
-    public function setUp()
+    public function setUp(): void
     {
         $this->context = $this->buildPhpRedisContext();
 
diff --git a/pkg/redis/Tests/Functional/PhpRedisConsumptionUseCasesTest.php b/pkg/redis/Tests/Functional/PhpRedisConsumptionUseCasesTest.php
@@ -19,7 +19,7 @@ class PhpRedisConsumptionUseCasesTest extends TestCase
      */
     private $context;
 
-    public function setUp()
+    public function setUp(): void
     {
         $this->context = $this->buildPhpRedisContext();
 
diff --git a/pkg/stomp/Tests/Functional/StompCommonUseCasesTest.php b/pkg/stomp/Tests/Functional/StompCommonUseCasesTest.php
@@ -20,7 +20,7 @@ class StompCommonUseCasesTest extends \PHPUnit\Framework\TestCase
      */
     private $stompContext;
 
-    public function setUp()
+    public function setUp(): void
     {
         $this->stompContext = $this->buildStompContext();
 
diff --git a/pkg/stomp/Tests/Functional/StompConsumptionUseCasesTest.php b/pkg/stomp/Tests/Functional/StompConsumptionUseCasesTest.php
@@ -28,7 +28,7 @@ class StompConsumptionUseCasesTest extends \PHPUnit\Framework\TestCase
      */
     private $stompContext;
 
-    public function setUp()
+    public function setUp(): void
     {
         $this->stompContext = $this->buildStompContext();
 
diff --git a/pkg/stomp/Tests/Functional/StompRpcUseCasesTest.php b/pkg/stomp/Tests/Functional/StompRpcUseCasesTest.php
@@ -22,7 +22,7 @@ class StompRpcUseCasesTest extends \PHPUnit\Framework\TestCase
      */
     private $stompContext;
 
-    public function setUp()
+    public function setUp(): void
     {
         $this->stompContext = $this->buildStompContext();
 

Original file line number	Diff line number	Diff line change
`@@ -57,7 +57,7 @@ class FooTest extends WebTestCase`
`57`	`57`	`/** @var \Symfony\Bundle\FrameworkBundle\Client */`
`58`	`58`	`private $client;`
`59`	`59`
`60`		`- public function setUp()`
	`60`	`+ public function setUp(): void`
`61`	`61`	`{`
`62`	`62`	`$this->client = static::createClient();`
`63`	`63`	`}`
Original file line number	Diff line number	Diff line change
`@@ -22,7 +22,7 @@ class AmqpCommonUseCasesTest extends TestCase`
`22`	`22`	`*/`
`23`	`23`	`private $amqpContext;`
`24`	`24`
`25`		`- public function setUp()`
	`25`	`+ public function setUp(): void`
`26`	`26`	`{`
`27`	`27`	`$this->amqpContext = $this->buildAmqpContext();`
`28`	`28`
Original file line number	Diff line number	Diff line change
`@@ -29,7 +29,7 @@ class AmqpConsumptionUseCasesTest extends TestCase`
`29`	`29`	`*/`
`30`	`30`	`private $amqpContext;`
`31`	`31`
`32`		`- public function setUp()`
	`32`	`+ public function setUp(): void`
`33`	`33`	`{`
`34`	`34`	`$this->amqpContext = $this->buildAmqpContext();`
`35`	`35`
Original file line number	Diff line number	Diff line change
`@@ -23,7 +23,7 @@ class AmqpRpcUseCasesTest extends TestCase`
`23`	`23`	`*/`
`24`	`24`	`private $amqpContext;`
`25`	`25`
`26`		`- public function setUp()`
	`26`	`+ public function setUp(): void`
`27`	`27`	`{`
`28`	`28`	`$this->amqpContext = $this->buildAmqpContext();`
`29`	`29`
Original file line number	Diff line number	Diff line change
`@@ -16,7 +16,7 @@ class SignalSocketHelperTest extends TestCase`
`16`	`16`
`17`	`17`	`private $backupSigIntHandler;`
`18`	`18`
`19`		`- public function setUp()`
	`19`	`+ public function setUp(): void`
`20`	`20`	`{`
`21`	`21`	`parent::setUp();`
`22`	`22`
Original file line number	Diff line number	Diff line change
`@@ -53,7 +53,7 @@ class UseCasesTest extends TestCase`
`53`	`53`	`*/`
`54`	`54`	`protected $asyncProcessor;`
`55`	`55`
`56`		`- public function setUp()`
	`56`	`+ public function setUp(): void`
`57`	`57`	`{`
`58`	`58`	`(new Filesystem())->remove(__DIR__.'/queues/');`
`59`	`59`
Original file line number	Diff line number	Diff line change
`@@ -21,7 +21,7 @@ class DbalConsumerTest extends TestCase`
`21`	`21`	`*/`
`22`	`22`	`private $context;`
`23`	`23`
`24`		`- public function setUp()`
	`24`	`+ public function setUp(): void`
`25`	`25`	`{`
`26`	`26`	`$this->context = $this->createDbalContext();`
`27`	`27`	`}`
Original file line number	Diff line number	Diff line change
`@@ -16,7 +16,7 @@ class DbalSendAndReceivePriorityMessagesFromQueueTest extends SendAndReceivePrio`
`16`	`16`
`17`	`17`	`private $publishedAt;`
`18`	`18`
`19`		`- public function setUp()`
	`19`	`+ public function setUp(): void`
`20`	`20`	`{`
`21`	`21`	`parent::setUp();`
`22`	`22`
Original file line number	Diff line number	Diff line change
`@@ -16,7 +16,7 @@`
`16`	`16`	`*/`
`17`	`17`	`class AsyncListenerTest extends WebTestCase`
`18`	`18`	`{`
`19`		`- public function setUp()`
	`19`	`+ public function setUp(): void`
`20`	`20`	`{`
`21`	`21`	`parent::setUp();`
`22`	`22`
Original file line number	Diff line number	Diff line change
`@@ -18,7 +18,7 @@`
`18`	`18`	`*/`
`19`	`19`	`class AsyncProcessorTest extends WebTestCase`
`20`	`20`	`{`
`21`		`- public function setUp()`
	`21`	`+ public function setUp(): void`
`22`	`22`	`{`
`23`	`23`	`parent::setUp();`
`24`	`24`