Use PHPUnit\Framework\TestCase instead of PHPUnit_Framework_TestCase

carusogabriel · carusogabriel · commit 28b431b7fcd0 · 2017-11-16T11:28:59.000-02:00
diff --git a/Tests/Command/BaseCommandTest.php b/Tests/Command/BaseCommandTest.php
@@ -2,7 +2,9 @@
 
 namespace OldSound\RabbitMqBundle\Tests\Command;
 
-abstract class BaseCommandTest extends \PHPUnit_Framework_TestCase
+use PHPUnit\Framework\TestCase;
+
+abstract class BaseCommandTest extends TestCase
 {
     protected $application;
     protected $definition;
diff --git a/Tests/Command/ConsumerCommandTest.php b/Tests/Command/ConsumerCommandTest.php
@@ -3,7 +3,6 @@
 namespace OldSound\RabbitMqBundle\Tests\Command;
 
 use OldSound\RabbitMqBundle\Command\ConsumerCommand;
-
 use Symfony\Component\Console\Input\InputOption;
 
 class ConsumerCommandTest extends BaseCommandTest
diff --git a/Tests/Command/DynamicConsumerCommandTest.php b/Tests/Command/DynamicConsumerCommandTest.php
@@ -3,11 +3,11 @@
 namespace OldSound\RabbitMqBundle\Tests\Command;
 
 use OldSound\RabbitMqBundle\Command\DynamicConsumerCommand;
-
 use Symfony\Component\Console\Input\InputOption;
 
-class DynamicConsumerCommandTest extends BaseCommandTest{
-    
+class DynamicConsumerCommandTest extends BaseCommandTest
+{
+
     protected function setUp()
     {
         parent::setUp();
@@ -25,7 +25,7 @@ protected function setUp()
         $this->command = new DynamicConsumerCommand();
         $this->command->setApplication($this->application);
     }
-    
+
     /**
      * testInputsDefinitionCommand
      */
@@ -35,7 +35,7 @@ public function testInputsDefinitionCommand()
         $definition = $this->command->getDefinition();
         $this->assertTrue($definition->hasArgument('name'));
         $this->assertTrue($definition->getArgument('name')->isRequired()); // Name is required to find the service
-        
+
         $this->assertTrue($definition->hasArgument('context'));
         $this->assertTrue($definition->getArgument('context')->isRequired()); // Context is required for the queue options provider
 
diff --git a/Tests/Command/MultipleConsumerCommandTest.php b/Tests/Command/MultipleConsumerCommandTest.php
@@ -3,11 +3,11 @@
 namespace OldSound\RabbitMqBundle\Tests\Command;
 
 use OldSound\RabbitMqBundle\Command\MultipleConsumerCommand;
-
 use Symfony\Component\Console\Input\InputOption;
 
-class MultipleConsumerCommandTest extends BaseCommandTest{
-    
+class MultipleConsumerCommandTest extends BaseCommandTest
+{
+
     protected function setUp()
     {
         parent::setUp();
@@ -25,7 +25,7 @@ protected function setUp()
         $this->command = new MultipleConsumerCommand();
         $this->command->setApplication($this->application);
     }
-    
+
     /**
      * testInputsDefinitionCommand
      */
@@ -35,7 +35,7 @@ public function testInputsDefinitionCommand()
         $definition = $this->command->getDefinition();
         $this->assertTrue($definition->hasArgument('name'));
         $this->assertTrue($definition->getArgument('name')->isRequired()); // Name is required to find the service
-        
+
         $this->assertTrue($definition->hasArgument('context'));
         $this->assertFalse($definition->getArgument('context')->isRequired()); // Context is required for the queue options provider
 
diff --git a/Tests/DependencyInjection/OldSoundRabbitMqExtensionTest.php b/Tests/DependencyInjection/OldSoundRabbitMqExtensionTest.php
@@ -9,8 +9,9 @@
 use Symfony\Component\DependencyInjection\Loader\YamlFileLoader;
 use OldSound\RabbitMqBundle\DependencyInjection\OldSoundRabbitMqExtension;
 use Symfony\Component\DependencyInjection\Reference;
+use PHPUnit\Framework\TestCase;
 
-class OldSoundRabbitMqExtensionTest extends \PHPUnit_Framework_TestCase
+class OldSoundRabbitMqExtensionTest extends TestCase
 {
     public function testFooConnectionDefinition()
     {
diff --git a/Tests/Event/AfterProcessingMessageEventTest.php b/Tests/Event/AfterProcessingMessageEventTest.php
@@ -5,13 +5,14 @@
 use OldSound\RabbitMqBundle\Event\AfterProcessingMessageEvent;
 use OldSound\RabbitMqBundle\RabbitMq\Consumer;
 use PhpAmqpLib\Message\AMQPMessage;
+use PHPUnit\Framework\TestCase;
 
 /**
  * Class AfterProcessingMessageEventTest
  *
  * @package OldSound\RabbitMqBundle\Tests\Event
  */
-class AfterProcessingMessageEventTest extends \PHPUnit_Framework_TestCase
+class AfterProcessingMessageEventTest extends TestCase
 {
     protected function getConsumer()
     {
@@ -33,4 +34,4 @@ public function testEvent()
         $this->assertSame($AMQPMessage, $event->getAMQPMessage());
         $this->assertSame($consumer, $event->getConsumer());
     }
-}
+}
diff --git a/Tests/Event/BeforeProcessingMessageEventTest.php b/Tests/Event/BeforeProcessingMessageEventTest.php
@@ -5,13 +5,14 @@
 use OldSound\RabbitMqBundle\Event\BeforeProcessingMessageEvent;
 use OldSound\RabbitMqBundle\RabbitMq\Consumer;
 use PhpAmqpLib\Message\AMQPMessage;
+use PHPUnit\Framework\TestCase;
 
 /**
  * Class BeforeProcessingMessageEventTest
  *
  * @package OldSound\RabbitMqBundle\Tests\Event
  */
-class BeforeProcessingMessageEventTest extends \PHPUnit_Framework_TestCase
+class BeforeProcessingMessageEventTest extends TestCase
 {
     protected function getConsumer()
     {
diff --git a/Tests/Event/OnIdleEventTest.php b/Tests/Event/OnIdleEventTest.php
@@ -4,13 +4,14 @@
 
 use OldSound\RabbitMqBundle\Event\OnIdleEvent;
 use OldSound\RabbitMqBundle\RabbitMq\Consumer;
+use PHPUnit\Framework\TestCase;
 
 /**
  * Class OnIdleEventTest
  *
  * @package OldSound\RabbitMqBundle\Tests\Event
  */
-class OnIdleEventTest extends \PHPUnit_Framework_TestCase
+class OnIdleEventTest extends TestCase
 {
     protected function getConsumer()
     {
diff --git a/Tests/Manager/MemoryCheckerTest.php b/Tests/Manager/MemoryCheckerTest.php
@@ -4,13 +4,14 @@
 
 use OldSound\RabbitMqBundle\MemoryChecker\MemoryConsumptionChecker;
 use OldSound\RabbitMqBundle\MemoryChecker\NativeMemoryUsageProvider;
+use PHPUnit\Framework\TestCase;
 
 /**
  * Class MemoryManagerTest
  *
  * @package OldSound\RabbitMqBundle\Tests\Manager
  */
-class MemoryConsumptionCheckerTest extends \PHPUnit_Framework_TestCase
+class MemoryConsumptionCheckerTest extends TestCase
 {
     public function testMemoryIsNotAlmostOverloaded()
     {
diff --git a/Tests/RabbitMq/AMQPConnectionFactoryTest.php b/Tests/RabbitMq/AMQPConnectionFactoryTest.php
@@ -5,8 +5,9 @@
 use OldSound\RabbitMqBundle\Provider\ConnectionParametersProviderInterface;
 use OldSound\RabbitMqBundle\RabbitMq\AMQPConnectionFactory;
 use OldSound\RabbitMqBundle\Tests\RabbitMq\Fixtures\AMQPConnection;
+use PHPUnit\Framework\TestCase;
 
-class AMQPConnectionFactoryTest extends \PHPUnit_Framework_TestCase
+class AMQPConnectionFactoryTest extends TestCase
 {
     public function testDefaultValues()
     {
diff --git a/Tests/RabbitMq/BaseAmqpTest.php b/Tests/RabbitMq/BaseAmqpTest.php
@@ -5,8 +5,9 @@
 use OldSound\RabbitMqBundle\Event\AMQPEvent;
 use OldSound\RabbitMqBundle\RabbitMq\BaseAmqp;
 use OldSound\RabbitMqBundle\RabbitMq\Consumer;
+use PHPUnit\Framework\TestCase;
 
-class BaseAmqpTest extends \PHPUnit_Framework_TestCase
+class BaseAmqpTest extends TestCase
 {
 
     public function testLazyConnection()
diff --git a/Tests/RabbitMq/BaseConsumerTest.php b/Tests/RabbitMq/BaseConsumerTest.php
@@ -3,8 +3,9 @@
 namespace OldSound\RabbitMqBundle\Tests\RabbitMq;
 
 use OldSound\RabbitMqBundle\RabbitMq\BaseConsumer;
+use PHPUnit\Framework\TestCase;
 
-class BaseConsumerTest extends \PHPUnit_Framework_TestCase
+class BaseConsumerTest extends TestCase
 {
     /** @var BaseConsumer */
     protected $consumer;
diff --git a/Tests/RabbitMq/BindingTest.php b/Tests/RabbitMq/BindingTest.php
@@ -2,10 +2,10 @@
 
 namespace OldSound\RabbitMqBundle\Tests\RabbitMq;
 
-
 use OldSound\RabbitMqBundle\RabbitMq\Binding;
+use PHPUnit\Framework\TestCase;
 
-class BindingTest extends \PHPUnit_Framework_TestCase
+class BindingTest extends TestCase
 {
 
     protected function getBinding($amqpConnection, $amqpChannel)
diff --git a/Tests/RabbitMq/ConsumerTest.php b/Tests/RabbitMq/ConsumerTest.php
@@ -10,9 +10,10 @@
 use PhpAmqpLib\Exception\AMQPTimeoutException;
 use PhpAmqpLib\Message\AMQPMessage;
 use OldSound\RabbitMqBundle\RabbitMq\ConsumerInterface;
+use PHPUnit\Framework\TestCase;
 
-class ConsumerTest extends \PHPUnit_Framework_TestCase
-{   
+class ConsumerTest extends TestCase
+{
     protected function getConsumer($amqpConnection, $amqpChannel)
     {
         return new Consumer($amqpConnection, $amqpChannel);
diff --git a/Tests/RabbitMq/DynamicConsumerTest.php b/Tests/RabbitMq/DynamicConsumerTest.php
@@ -5,12 +5,12 @@
 use OldSound\RabbitMqBundle\RabbitMq\DynamicConsumer;
 
 class DynamicConsumerTest extends ConsumerTest
-{   
+{
     public function getConsumer($amqpConnection, $amqpChannel)
     {
         return new DynamicConsumer($amqpConnection, $amqpChannel);
     }
-    
+
     /**
      * Preparing QueueOptionsProviderInterface instance
      *
@@ -21,14 +21,14 @@ private function prepareQueueOptionsProvider()
         return $this->getMockBuilder('\OldSound\RabbitMqBundle\Provider\QueueOptionsProviderInterface')
             ->getMock();
     }
-    
+
     public function testQueueOptionsPrivider()
     {
         $amqpConnection = $this->prepareAMQPConnection();
         $amqpChannel = $this->prepareAMQPChannel();
         $consumer = $this->getConsumer($amqpConnection, $amqpChannel);
         $consumer->setContext('foo');
-        
+
         $queueOptionsProvider = $this->prepareQueueOptionsProvider();
         $queueOptionsProvider->expects($this->once())
             ->method('getQueueOptions')
@@ -40,12 +40,12 @@ public function testQueueOptionsPrivider()
                     )
                 )
             ));
-        
+
         $consumer->setQueueOptionsProvider($queueOptionsProvider);
-        
+
         $reflectionClass = new \ReflectionClass(get_class($consumer));
         $reflectionMethod = $reflectionClass->getMethod('mergeQueueOptions');
         $reflectionMethod->setAccessible(true);
-        $reflectionMethod->invoke($consumer);        
+        $reflectionMethod->invoke($consumer);
     }
 }
diff --git a/Tests/RabbitMq/MultipleConsumerTest.php b/Tests/RabbitMq/MultipleConsumerTest.php
@@ -8,8 +8,9 @@
 use PhpAmqpLib\Channel\AMQPChannel;
 use PhpAmqpLib\Connection\AMQPConnection;
 use PhpAmqpLib\Message\AMQPMessage;
+use PHPUnit\Framework\TestCase;
 
-class MultipleConsumerTest extends \PHPUnit_Framework_TestCase
+class MultipleConsumerTest extends TestCase
 {
     /**
      * Multiple consumer
@@ -111,13 +112,13 @@ public function testQueuesProvider($processFlag, $expectedMethod, $expectedReque
         $this->multipleConsumer->processQueueMessage('test-1', $amqpMessage);
         $this->multipleConsumer->processQueueMessage('test-2', $amqpMessage);
     }
-    
+
     public function testQueuesPrivider()
     {
         $amqpConnection = $this->prepareAMQPConnection();
         $amqpChannel = $this->prepareAMQPChannel();
         $this->multipleConsumer->setContext('foo');
-        
+
         $queuesProvider = $this->prepareQueuesProvider();
         $queuesProvider->expects($this->once())
             ->method('getQueues')
@@ -126,13 +127,13 @@ public function testQueuesPrivider()
                     'queue_foo' => array()
                 )
             ));
-        
+
         $this->multipleConsumer->setQueuesProvider($queuesProvider);
-        
+
         $reflectionClass = new \ReflectionClass(get_class($this->multipleConsumer));
         $reflectionMethod = $reflectionClass->getMethod('mergeQueues');
         $reflectionMethod->setAccessible(true);
-        $reflectionMethod->invoke($this->multipleConsumer);        
+        $reflectionMethod->invoke($this->multipleConsumer);
     }
 
     /**
@@ -268,4 +269,4 @@ private function prepareCallback($processFlag)
             return $processFlag;
         };
     }
-} 
+}
diff --git a/Tests/RabbitMq/RpcClientTest.php b/Tests/RabbitMq/RpcClientTest.php
@@ -4,8 +4,9 @@
 
 use OldSound\RabbitMqBundle\RabbitMq\RpcClient;
 use PhpAmqpLib\Message\AMQPMessage;
+use PHPUnit\Framework\TestCase;
 
-class RpcClientTest extends \PHPUnit_Framework_TestCase
+class RpcClientTest extends TestCase
 {
     public function testProcessMessageWithCustomUnserializer()
     {
diff --git a/Tests/RabbitMq/RpcServerTest.php b/Tests/RabbitMq/RpcServerTest.php
@@ -4,8 +4,9 @@
 
 use OldSound\RabbitMqBundle\RabbitMq\RpcServer;
 use PhpAmqpLib\Message\AMQPMessage;
+use PHPUnit\Framework\TestCase;
 
-class RpcServerTest extends \PHPUnit_Framework_TestCase
+class RpcServerTest extends TestCase
 {
     public function testProcessMessageWithCustomSerializer()
     {
@@ -38,4 +39,3 @@ public function testProcessMessageWithCustomSerializer()
         $server->processMessage($message);
     }
 }
-

Original file line number	Diff line number	Diff line change
`@@ -2,7 +2,9 @@`
`2`	`2`
`3`	`3`	`namespace OldSound\RabbitMqBundle\Tests\Command;`
`4`	`4`
`5`		`-abstract class BaseCommandTest extends \PHPUnit_Framework_TestCase`
	`5`	`+use PHPUnit\Framework\TestCase;`
	`6`	`+`
	`7`	`+abstract class BaseCommandTest extends TestCase`
`6`	`8`	`{`
`7`	`9`	`protected $application;`
`8`	`10`	`protected $definition;`
Original file line number	Diff line number	Diff line change
`@@ -9,8 +9,9 @@`
`9`	`9`	`use Symfony\Component\DependencyInjection\Loader\YamlFileLoader;`
`10`	`10`	`use OldSound\RabbitMqBundle\DependencyInjection\OldSoundRabbitMqExtension;`
`11`	`11`	`use Symfony\Component\DependencyInjection\Reference;`
	`12`	`+use PHPUnit\Framework\TestCase;`
`12`	`13`
`13`		`-class OldSoundRabbitMqExtensionTest extends \PHPUnit_Framework_TestCase`
	`14`	`+class OldSoundRabbitMqExtensionTest extends TestCase`
`14`	`15`	`{`
`15`	`16`	`public function testFooConnectionDefinition()`
`16`	`17`	`{`
Original file line number	Diff line number	Diff line change
`@@ -5,13 +5,14 @@`
`5`	`5`	`use OldSound\RabbitMqBundle\Event\AfterProcessingMessageEvent;`
`6`	`6`	`use OldSound\RabbitMqBundle\RabbitMq\Consumer;`
`7`	`7`	`use PhpAmqpLib\Message\AMQPMessage;`
	`8`	`+use PHPUnit\Framework\TestCase;`
`8`	`9`
`9`	`10`	`/**`
`10`	`11`	`* Class AfterProcessingMessageEventTest`
`11`	`12`	`*`
`12`	`13`	`* @package OldSound\RabbitMqBundle\Tests\Event`
`13`	`14`	`*/`
`14`		`-class AfterProcessingMessageEventTest extends \PHPUnit_Framework_TestCase`
	`15`	`+class AfterProcessingMessageEventTest extends TestCase`
`15`	`16`	`{`
`16`	`17`	`protected function getConsumer()`
`17`	`18`	`{`
`@@ -33,4 +34,4 @@ public function testEvent()`
`33`	`34`	`$this->assertSame($AMQPMessage, $event->getAMQPMessage());`
`34`	`35`	`$this->assertSame($consumer, $event->getConsumer());`
`35`	`36`	`}`
`36`		`-}`
	`37`	`+}`
Original file line number	Diff line number	Diff line change
`@@ -5,13 +5,14 @@`
`5`	`5`	`use OldSound\RabbitMqBundle\Event\BeforeProcessingMessageEvent;`
`6`	`6`	`use OldSound\RabbitMqBundle\RabbitMq\Consumer;`
`7`	`7`	`use PhpAmqpLib\Message\AMQPMessage;`
	`8`	`+use PHPUnit\Framework\TestCase;`
`8`	`9`
`9`	`10`	`/**`
`10`	`11`	`* Class BeforeProcessingMessageEventTest`
`11`	`12`	`*`
`12`	`13`	`* @package OldSound\RabbitMqBundle\Tests\Event`
`13`	`14`	`*/`
`14`		`-class BeforeProcessingMessageEventTest extends \PHPUnit_Framework_TestCase`
	`15`	`+class BeforeProcessingMessageEventTest extends TestCase`
`15`	`16`	`{`
`16`	`17`	`protected function getConsumer()`
`17`	`18`	`{`
Original file line number	Diff line number	Diff line change
`@@ -4,13 +4,14 @@`
`4`	`4`
`5`	`5`	`use OldSound\RabbitMqBundle\Event\OnIdleEvent;`
`6`	`6`	`use OldSound\RabbitMqBundle\RabbitMq\Consumer;`
	`7`	`+use PHPUnit\Framework\TestCase;`
`7`	`8`
`8`	`9`	`/**`
`9`	`10`	`* Class OnIdleEventTest`
`10`	`11`	`*`
`11`	`12`	`* @package OldSound\RabbitMqBundle\Tests\Event`
`12`	`13`	`*/`
`13`		`-class OnIdleEventTest extends \PHPUnit_Framework_TestCase`
	`14`	`+class OnIdleEventTest extends TestCase`
`14`	`15`	`{`
`15`	`16`	`protected function getConsumer()`
`16`	`17`	`{`
Original file line number	Diff line number	Diff line change
`@@ -4,13 +4,14 @@`
`4`	`4`
`5`	`5`	`use OldSound\RabbitMqBundle\MemoryChecker\MemoryConsumptionChecker;`
`6`	`6`	`use OldSound\RabbitMqBundle\MemoryChecker\NativeMemoryUsageProvider;`
	`7`	`+use PHPUnit\Framework\TestCase;`
`7`	`8`
`8`	`9`	`/**`
`9`	`10`	`* Class MemoryManagerTest`
`10`	`11`	`*`
`11`	`12`	`* @package OldSound\RabbitMqBundle\Tests\Manager`
`12`	`13`	`*/`
`13`		`-class MemoryConsumptionCheckerTest extends \PHPUnit_Framework_TestCase`
	`14`	`+class MemoryConsumptionCheckerTest extends TestCase`
`14`	`15`	`{`
`15`	`16`	`public function testMemoryIsNotAlmostOverloaded()`
`16`	`17`	`{`
Original file line number	Diff line number	Diff line change
`@@ -5,8 +5,9 @@`
`5`	`5`	`use OldSound\RabbitMqBundle\Provider\ConnectionParametersProviderInterface;`
`6`	`6`	`use OldSound\RabbitMqBundle\RabbitMq\AMQPConnectionFactory;`
`7`	`7`	`use OldSound\RabbitMqBundle\Tests\RabbitMq\Fixtures\AMQPConnection;`
	`8`	`+use PHPUnit\Framework\TestCase;`
`8`	`9`
`9`		`-class AMQPConnectionFactoryTest extends \PHPUnit_Framework_TestCase`
	`10`	`+class AMQPConnectionFactoryTest extends TestCase`
`10`	`11`	`{`
`11`	`12`	`public function testDefaultValues()`
`12`	`13`	`{`