Tweak tests

Author: Eugene Leonovich

tests/Unit/Di/ContainerBuilderTest.php

@@ -6,59 +6,59 @@
 use App\Di\ContainerBuilder;
 use PHPUnit\Framework\TestCase;
 
-class ContainerBuilderTest extends TestCase
+final class ContainerBuilderTest extends TestCase
 {
     public function testConstructorSetOptions(): void
     {
         $options = ['foo' => 'bar'];
         $builder = new ContainerBuilder($options);
 
-        $this->assertSame($options['foo'], $builder->get('foo'));
+        self::assertSame($options['foo'], $builder->get('foo'));
     }
 
     public function testSetGetEnv(): void
     {
         $builder = new ContainerBuilder();
         $env = 'test';
 
-        $this->assertNotSame($env, $builder->getEnv());
+        self::assertNotSame($env, $builder->getEnv());
         $builder->setEnv($env);
-        $this->assertSame($env, $builder->getEnv());
+        self::assertSame($env, $builder->getEnv());
     }
 
     public function testSetGetDebug(): void
     {
         $builder = new ContainerBuilder();
 
-        $this->assertNotTrue($builder->isDebug());
+        self::assertNotTrue($builder->isDebug());
         $builder->setDebug(true);
-        $this->assertTrue($builder->isDebug());
+        self::assertTrue($builder->isDebug());
         $builder->setDebug(false);
-        $this->assertFalse($builder->isDebug());
+        self::assertFalse($builder->isDebug());
     }
 
     public function testSetOptions(): void
     {
         $builder = new ContainerBuilder();
         $builder->setOptions(['foo' => 'bar']);
 
-        $this->assertSame('bar', $builder->get('foo'));
+        self::assertSame('bar', $builder->get('foo'));
     }
 
     public function testGetOptions(): void
     {
         $options = ['foo' => 'bar', 'baz' => 'qux'];
         $builder = new ContainerBuilder($options);
 
-        $this->assertSame($options, $builder->getOptions());
+        self::assertSame($options, $builder->getOptions());
     }
 
     public function testSet(): void
     {
         $builder = new ContainerBuilder();
         $builder->set('foo', 'bar');
 
-        $this->assertSame('bar', $builder->get('foo'));
+        self::assertSame('bar', $builder->get('foo'));
     }
 
     /**
@@ -76,7 +76,7 @@ public function testTryGet(): void
     {
         $builder = new ContainerBuilder();
 
-        $this->assertSame(42, $builder->tryGet('foo', 42));
+        self::assertSame(42, $builder->tryGet('foo', 42));
     }
 
     public function testBuild(): void
@@ -87,9 +87,9 @@ public function testBuild(): void
 
         $container = $builder->build();
 
-        $this->assertInstanceOf(Container::class, $container);
-        $this->assertSame('bar', $container->get('foo'));
-        $this->assertSame('baz', $container->getEnv());
-        $this->assertTrue($container->isDebug());
+        self::assertInstanceOf(Container::class, $container);
+        self::assertSame('bar', $container->get('foo'));
+        self::assertSame('baz', $container->getEnv());
+        self::assertTrue($container->isDebug());
     }
 }

tests/Unit/Di/ContainerTest.php

@@ -5,35 +5,35 @@
 use App\Di\Container;
 use PHPUnit\Framework\TestCase;
 
-class ContainerTest extends TestCase
+final class ContainerTest extends TestCase
 {
     public function testGet(): void
     {
         $container = new Container(['foo' => 'bar'], 'test', true);
 
-        $this->assertSame('bar', $container->get('foo'));
+        self::assertSame('bar', $container->get('foo'));
     }
 
     public function testGetEnv(): void
     {
         $container = new Container([], 'test', true);
 
-        $this->assertSame('test', $container->getEnv());
+        self::assertSame('test', $container->getEnv());
     }
 
     public function testIsDebug(): void
     {
         $container = new Container([], 'test', true);
 
-        $this->assertTrue($container->isDebug());
+        self::assertTrue($container->isDebug());
     }
 
     public function testGetOptions(): void
     {
         $options = ['foo' => 'bar', 'baz' => 'qux'];
         $container = new Container($options, 'test', true);
 
-        $this->assertSame($options, $container->getOptions());
+        self::assertSame($options, $container->getOptions());
     }
 
     /**
@@ -51,6 +51,6 @@ public function testTryGet(): void
     {
         $container = new Container([], 'test', true);
 
-        $this->assertSame(42, $container->tryGet('foo', 42));
+        self::assertSame(42, $container->tryGet('foo', 42));
     }
 }

tests/Unit/UseCase/Greet/GreetHandlerTest.php

@@ -6,12 +6,12 @@
 use PHPUnit\Framework\TestCase;
 use Psr\Log\LoggerInterface as Logger;
 
-class GreetHandlerTest extends TestCase
+final class GreetHandlerTest extends TestCase
 {
     public function testInvoke(): void
     {
         $logger = $this->createMock(Logger::class);
-        $logger->expects($this->once())->method('info')
+        $logger->expects(self::once())->method('info')
             ->with('Hello World');
 
         $handler = new GreetHandler(false);
@@ -21,7 +21,7 @@ public function testInvoke(): void
     public function testInvokeWithYell(): void
     {
         $logger = $this->createMock(Logger::class);
-        $logger->expects($this->once())->method('info')
+        $logger->expects(self::once())->method('info')
             ->with('HELLO WORLD');
 
         $handler = new GreetHandler(true);