Merge pull request #7 from petrknap/molireali

Code style chores

composer.json

@@ -1,5 +1,5 @@
 {
-  "WARNING": "This file is updated automatically. All keys will be overwritten, except of 'conflict', 'keywords', 'require', 'require-dev' and 'scripts'.",
+  "WARNING": "This file is updated automatically. All keys will be overwritten, except of 'conflict', 'keywords', 'require', 'require-dev', 'scripts' and 'suggest'.",
   "autoload": {
     "psr-4": {
       "PetrKnap\\CriticalSection\\": "src"
@@ -34,19 +34,20 @@
     "symfony/lock": "^6.0"
   },
   "require-dev": {
-    "nunomaduro/phpinsights": "^2.9",
-    "petrknap/shorts": "^1.3",
+    "nunomaduro/phpinsights": "^2.11",
+    "petrknap/shorts": "^2.1",
     "phpstan/phpstan": "^1.10",
     "phpunit/phpunit": "^9.6",
     "squizlabs/php_codesniffer": "^3.7"
   },
   "scripts": {
     "test": "phpunit --colors=always --testdox tests",
     "validate": [
-      "phpcs --colors --standard=PSR12 --exclude=PSR12.Files.OpenTag,PSR12.Files.FileHeader,Generic.Files.LineLength src tests",
+      "composer outdated \"petrknap/*\" --major-only --strict --ansi --no-interaction",
+      "phpcs --colors --standard=PSR12 --exclude=Generic.Files.LineLength src tests",
       "phpstan analyse --level max src",
       "phpstan analyse --level 5 tests",
-      "phpinsights analyse src"
+      "phpinsights analyse src --ansi --no-interaction"
     ],
     "ci-script": [
       "@validate",

tests/CriticalSectionTest.php

@@ -1,4 +1,4 @@
-<?php /** @noinspection PhpUnhandledExceptionInspection */
+<?php
 
 declare(strict_types=1);
 

tests/NonCriticalSectionTest.php

@@ -1,4 +1,4 @@
-<?php /** @noinspection PhpUnhandledExceptionInspection */
+<?php
 
 declare(strict_types=1);
 

tests/ReadmeTest.php

@@ -1,4 +1,6 @@
-<?php declare(strict_types=1);
+<?php
+
+declare(strict_types=1);
 
 namespace PetrKnap\CriticalSection;
 

tests/SymfonyLockCriticalSectionTest.php

@@ -1,4 +1,4 @@
-<?php /** @noinspection PhpUnhandledExceptionInspection */
+<?php
 
 declare(strict_types=1);
 
@@ -8,6 +8,7 @@
 use PetrKnap\CriticalSection\Exception\CouldNotEnterCriticalSection;
 use PetrKnap\CriticalSection\Exception\CouldNotLeaveCriticalSection;
 use PHPUnit\Framework\TestCase;
+use stdClass;
 use Symfony\Component\Lock\Exception\LockAcquiringException;
 use Symfony\Component\Lock\Exception\LockConflictedException;
 use Symfony\Component\Lock\Exception\LockReleasingException;
@@ -17,7 +18,7 @@ final class SymfonyLockCriticalSectionTest extends TestCase
 {
     public function testAcquiresLockBeforeCriticalSectionIsExecuted(): void
     {
-        $shared = new \stdClass();
+        $shared = new stdClass();
         $shared->isLocked = false;
         $lock = self::createMock(LockInterface::class);
         $lock->expects(self::once())
@@ -34,7 +35,7 @@ public function testAcquiresLockBeforeCriticalSectionIsExecuted(): void
 
     public function testReleasesLockAfterCriticalSectionWasExecuted(): void
     {
-        $shared = new \stdClass();
+        $shared = new stdClass();
         $shared->isLocked = true;
         $lock = self::createMock(LockInterface::class);
         $lock->method('acquire')->willReturn(true);
@@ -53,7 +54,7 @@ public function testReleasesLockAfterCriticalSectionWasExecuted(): void
 
     public function testReleasesLockAndThrowsWhenCriticalSectionThrows(): void
     {
-        $shared = new \stdClass();
+        $shared = new stdClass();
         $shared->isLocked = true;
         $lock = self::createMock(LockInterface::class);
         $lock->method('acquire')->willReturn(true);

tests/WrappingCriticalSectionTest.php

@@ -1,4 +1,4 @@
-<?php /** @noinspection PhpUnhandledExceptionInspection */
+<?php
 
 declare(strict_types=1);
 
@@ -10,6 +10,7 @@
 final class WrappingCriticalSectionTest extends TestCase
 {
     private const FOO = 'bar';
+
     public function testReturnsValueReturnedByExecutedCriticalSection(): void
     {
         self::assertSame(