phpunit: Upgrade to version 8, fixed tests, formatting

Use `dms/phpunit-arraysubset-asserts` after `assertArraySubset` got deprecated

1 files changed, 21 insertions, 4 deletions

diff --git a/tests/Feature/Logger/EngelsystemLoggerTest.php b/tests/Feature/Logger/EngelsystemLoggerTest.php
index 9ec078a3..cb7e18f3 100644
--- a/tests/Feature/Logger/EngelsystemLoggerTest.php
+++ b/tests/Feature/Logger/EngelsystemLoggerTest.php
@@ -8,10 +8,12 @@ use Engelsystem\Test\Feature\ApplicationFeatureTest;
 use Psr\Log\InvalidArgumentException;
 use Psr\Log\LoggerInterface;
 use Psr\Log\LogLevel;
+use stdClass;
 
 class EngelsystemLoggerTest extends ApplicationFeatureTest
 {
     /**
+     * @covers \Engelsystem\Logger\EngelsystemLogger::__construct
      * @return LoggerInterface
      */
     public function getLogger()
@@ -20,6 +22,9 @@ class EngelsystemLoggerTest extends ApplicationFeatureTest
         return new EngelsystemLogger($logEntry);
     }
 
+    /**
+     * @covers \Engelsystem\Logger\EngelsystemLogger::__construct
+     */
     public function testImplements()
     {
         $this->assertInstanceOf(LoggerInterface::class, $this->getLogger());
@@ -43,6 +48,7 @@ class EngelsystemLoggerTest extends ApplicationFeatureTest
     }
 
     /**
+     * @covers       \Engelsystem\Models\LogEntry
      * @dataProvider provideLogLevels
      * @param string $level
      */
@@ -58,6 +64,9 @@ class EngelsystemLoggerTest extends ApplicationFeatureTest
         $this->assertCount(2, $entries);
     }
 
+    /**
+     * @covers \Engelsystem\Logger\EngelsystemLogger::log
+     */
     public function testContextReplacement()
     {
         LogEntry::query()->truncate();
@@ -78,12 +87,13 @@ class EngelsystemLoggerTest extends ApplicationFeatureTest
         return [
             ['Data and {context}', [], 'Data and {context}'],
             ['Data and {context}', ['context' => null], 'Data and '],
-            ['Data and {context}', ['context' => new \stdClass()], 'Data and {context}'],
+            ['Data and {context}', ['context' => new stdClass()], 'Data and {context}'],
             ['Some user asked: {question}', ['question' => 'Foo?'], 'Some user asked: Foo?'],
         ];
     }
 
     /**
+     * @covers       \Engelsystem\Logger\EngelsystemLogger::log
      * @dataProvider provideContextReplaceValues
      *
      * @param string   $message
@@ -99,6 +109,9 @@ class EngelsystemLoggerTest extends ApplicationFeatureTest
         $this->assertEquals($expected, $entry['message']);
     }
 
+    /**
+     * @covers \Engelsystem\Logger\EngelsystemLogger::log
+     */
     public function testContextToString()
     {
         LogEntry::query()->truncate();
@@ -119,19 +132,20 @@ class EngelsystemLoggerTest extends ApplicationFeatureTest
     }
 
     /**
-     * @expectedException InvalidArgumentException
+     * @covers \Engelsystem\Logger\EngelsystemLogger::log
      */
     public function testThrowExceptionOnInvalidLevel()
     {
         $logger = $this->getLogger();
 
+        $this->expectException(InvalidArgumentException::class);
         $logger->log('This log level should never be defined', 'Some message');
     }
 
     /**
      * @return array
      */
-    public function getLastEntry()
+    protected function getLastEntry()
     {
         $entries = LogEntry::all();
         $entry = $entries->last();
@@ -139,7 +153,10 @@ class EngelsystemLoggerTest extends ApplicationFeatureTest
         return $entry;
     }
 
-    public function tearDown()
+    /**
+     * Cleanup
+     */
+    protected function tearDown(): void
     {
         LogEntry::query()->truncate();
     }