test it against php7.4 (#15)

* test it against php7.4

* upgrade to phpunit 8

* make compatible with phpunit 8

Author: mihaileu

.travis.yml

@@ -16,6 +16,8 @@ matrix:
       env: SYMFONY_VERSION=^4.3
     - php: 7.2
       env: SYMFONY_VERSION=^5.0
+    - php: 7.4
+      env: SYMFONY_VERSION=^5.0
 
   allow_failures:
     - php: nightly

Tests/Command/BaseCommandTest.php

@@ -11,7 +11,7 @@ abstract class BaseCommandTest extends TestCase
     protected $helperSet;
     protected $command;
 
-    protected function setUp()
+    protected function setUp(): void
     {
         $this->application = $this->getMockBuilder('Symfony\\Component\\Console\\Application')
             ->disableOriginalConstructor()

Tests/Command/ConsumerCommandTest.php

@@ -7,7 +7,7 @@
 
 class ConsumerCommandTest extends BaseCommandTest
 {
-    protected function setUp()
+    protected function setUp(): void
     {
         parent::setUp();
         $this->definition->expects($this->any())

Tests/Command/DynamicConsumerCommandTest.php

@@ -8,7 +8,7 @@
 class DynamicConsumerCommandTest extends BaseCommandTest
 {
 
-    protected function setUp()
+    protected function setUp(): void
     {
         parent::setUp();
         $this->definition->expects($this->any())

Tests/Command/MultipleConsumerCommandTest.php

@@ -8,7 +8,7 @@
 class MultipleConsumerCommandTest extends BaseCommandTest
 {
 
-    protected function setUp()
+    protected function setUp(): void
     {
         parent::setUp();
         $this->definition->expects($this->any())

Tests/Command/PurgeCommandTest.php

@@ -7,7 +7,7 @@
 
 class PurgeCommandTest extends BaseCommandTest
 {
-    protected function setUp()
+    protected function setUp(): void
     {
         parent::setUp();
         $this->definition->expects($this->any())

Tests/DependencyInjection/OldSoundRabbitMqExtensionTest.php

@@ -442,7 +442,7 @@ public function testConsumerWithQosOptions()
             }
         }
 
-        $this->assertInternalType('array', $setQosParameters);
+        $this->assertIsArray($setQosParameters);
         $this->assertEquals(
             array(
                 1024,

Tests/RabbitMq/AMQPConnectionFactoryTest.php

@@ -247,7 +247,7 @@ public function testSSLConnectionParameters()
         $context = $instance->constructParams[11];
         // unset to check whole array at once later
         $instance->constructParams[11] = null;
-        $this->assertInternalType('resource', $context);
+        $this->assertIsResource($context);
         $this->assertEquals('stream-context', get_resource_type($context));
         $options = stream_context_get_options($context);
         $this->assertEquals(array('ssl' => array('verify_peer' => false)), $options);

Tests/RabbitMq/BaseConsumerTest.php

@@ -10,9 +10,9 @@ class BaseConsumerTest extends TestCase
     /** @var BaseConsumer */
     protected $consumer;
 
-    protected function setUp()
+    protected function setUp(): void
     {
-        $amqpConnection =  $this->getMockBuilder('\PhpAmqpLib\Connection\AMQPConnection')
+        $amqpConnection = $this->getMockBuilder('\PhpAmqpLib\Connection\AMQPConnection')
             ->disableOriginalConstructor()
             ->getMock();
 

Tests/RabbitMq/MultipleConsumerTest.php

@@ -39,7 +39,7 @@ class MultipleConsumerTest extends TestCase
      *
      * @return void
      */
-    public function setUp()
+    public function setUp(): void
     {
         $this->amqpConnection = $this->prepareAMQPConnection();
         $this->amqpChannel = $this->prepareAMQPChannel();
@@ -246,14 +246,14 @@ private function prepareAMQPChannelExpectations($expectedMethod, $expectedRequeu
     {
         $this->amqpChannel->expects($this->any())
             ->method('basic_reject')
-            ->will($this->returnCallback(function($delivery_tag, $requeue) use ($expectedMethod, $expectedRequeue) {
+            ->will($this->returnCallback(function ($delivery_tag, $requeue) use ($expectedMethod, $expectedRequeue) {
                 Assert::assertSame($expectedMethod, 'basic_reject'); // Check if this function should be called.
                 Assert::assertSame($requeue, $expectedRequeue); // Check if the message should be requeued.
             }));
 
         $this->amqpChannel->expects($this->any())
             ->method('basic_ack')
-            ->will($this->returnCallback(function($delivery_tag) use ($expectedMethod) {
+            ->will($this->returnCallback(function ($delivery_tag) use ($expectedMethod) {
                 Assert::assertSame($expectedMethod, 'basic_ack'); // Check if this function should be called.
             }));
     }
@@ -266,7 +266,7 @@ private function prepareAMQPChannelExpectations($expectedMethod, $expectedRequeu
      */
     private function prepareCallback($processFlag)
     {
-        return function($msg) use (&$lastQueue, $processFlag) {
+        return function ($msg) use (&$lastQueue, $processFlag) {
             return $processFlag;
         };
     }