Code Quality using Rector (#211)

## Description



## Checklist
- [ ] Updated CHANGELOG files
- [ ] Updated Documentation
- [ ] Unit Tests Created
- [x] php-cs-fixer

Author: JoshuaEstes

CommandMessageBus.php

@@ -24,13 +24,9 @@
  */
 class CommandMessageBus
 {
-    private MessageBusInterface $messageBus;
     private array $stamps = [];
 
-    public function __construct(MessageBusInterface $commandBus)
-    {
-        $this->messageBus = $commandBus;
-    }
+    public function __construct(private MessageBusInterface $messageBus) {}
 
     /**
      * Allows you to dispatch the command with additional stamps if

Tests/Command/AbstractOptionsResolverCommandMessageTest.php

@@ -4,16 +4,17 @@
 
 namespace SonsOfPHP\Bridge\Symfony\Cqrs\Tests\Command;
 
+use PHPUnit\Framework\Attributes\CoversClass;
 use PHPUnit\Framework\TestCase;
 use SonsOfPHP\Bridge\Symfony\Cqrs\Command\AbstractOptionsResolverCommandMessage;
 use SonsOfPHP\Bridge\Symfony\Cqrs\Tests\DummyCommand;
 use SonsOfPHP\Contract\Cqrs\CommandMessageInterface;
 
 /**
- * @coversDefaultClass \SonsOfPHP\Bridge\Symfony\Cqrs\Command\AbstractOptionsResolverCommandMessage
- *
  * @uses \SonsOfPHP\Bridge\Symfony\Cqrs\Command\AbstractOptionsResolverCommandMessage
+ * @coversNothing
  */
+#[CoversClass(AbstractOptionsResolverCommandMessage::class)]
 final class AbstractOptionsResolverCommandMessageTest extends TestCase
 {
     /**
@@ -26,12 +27,6 @@ public function testItHasTheCorrectInterface(): void
         $this->assertInstanceOf(CommandMessageInterface::class, $command); // @phpstan-ignore-line
     }
 
-    /**
-     * @covers ::__construct
-     * @covers ::configureOptions
-     * @covers ::getOption
-     * @covers ::getOptions
-     */
     public function testConstructor(): void
     {
         DummyCommand::setConfigureOptionsCallback(function ($resolver): void {
@@ -46,10 +41,6 @@ public function testConstructor(): void
         $this->assertSame('unique-id', $command->getOption('id'));
     }
 
-    /**
-     * @covers ::__get
-     * @covers ::getOption
-     */
     public function testMagicMethodGetWorks(): void
     {
         DummyCommand::setConfigureOptionsCallback(function ($resolver): void {
@@ -62,10 +53,6 @@ public function testMagicMethodGetWorks(): void
         $this->assertSame('unique-id', $command->id);
     }
 
-    /**
-     * @covers ::__isset
-     * @covers ::hasOption
-     */
     public function testMagicMethodIssetWorks(): void
     {
         DummyCommand::setConfigureOptionsCallback(function ($resolver): void {

Tests/CommandMessageBusTest.php

@@ -4,23 +4,21 @@
 
 namespace SonsOfPHP\Bridge\Symfony\Cqrs\Tests;
 
+use PHPUnit\Framework\Attributes\CoversClass;
 use PHPUnit\Framework\TestCase;
 use SonsOfPHP\Bridge\Symfony\Cqrs\CommandMessageBus;
+use stdClass;
 use Symfony\Component\Messenger\Envelope;
 use Symfony\Component\Messenger\MessageBusInterface;
 use Symfony\Component\Messenger\Stamp\StampInterface;
 
 /**
- * @coversDefaultClass \SonsOfPHP\Bridge\Symfony\Cqrs\CommandMessageBus
- *
  * @internal
+ * @coversNothing
  */
+#[CoversClass(CommandMessageBus::class)]
 final class CommandMessageBusTest extends TestCase
 {
-    /**
-     * @covers ::__construct
-     * @covers ::withStamps
-     */
     public function testWithStampsIsImmutable(): void
     {
         $commandBus = new CommandMessageBus($this->createMock(MessageBusInterface::class));
@@ -32,17 +30,13 @@ public function testWithStampsIsImmutable(): void
         $this->assertNotSame($commandBus, $anotherCommandBus);
     }
 
-    /**
-     * @covers ::__construct
-     * @covers ::dispatch
-     */
     public function testItWillUseMessageBusToDispatchCommand(): void
     {
         $bus = $this->createMock(MessageBusInterface::class);
         $bus->expects($this->once())->method('dispatch')->willReturn(new Envelope($bus));
 
         $commandBus = new CommandMessageBus($bus);
 
-        $commandBus->dispatch(new \stdClass());
+        $commandBus->dispatch(new stdClass());
     }
 }

Tests/QueryMessageBusTest.php

@@ -4,31 +4,29 @@
 
 namespace SonsOfPHP\Bridge\Symfony\Cqrs\Tests;
 
+use PHPUnit\Framework\Attributes\CoversClass;
 use PHPUnit\Framework\TestCase;
 use SonsOfPHP\Bridge\Symfony\Cqrs\QueryMessageBus;
+use stdClass;
 use Symfony\Component\Messenger\Envelope;
 use Symfony\Component\Messenger\MessageBusInterface;
 use Symfony\Component\Messenger\Stamp\HandledStamp;
 
 /**
- * @coversDefaultClass \SonsOfPHP\Bridge\Symfony\Cqrs\QueryMessageBus
- *
  * @internal
+ * @coversNothing
  */
+#[CoversClass(QueryMessageBus::class)]
 final class QueryMessageBusTest extends TestCase
 {
-    /**
-     * @covers ::__construct
-     * @covers ::handle
-     */
     public function testItWillUseMessageBusToDispatchQuery(): void
     {
         $bus = $this->createMock(MessageBusInterface::class);
         $bus->expects($this->once())->method('dispatch')->willReturn(
-            new Envelope(new \stdClass(), [new HandledStamp('result', 'handlerName')])
+            new Envelope(new stdClass(), [new HandledStamp('result', 'handlerName')])
         );
         $queryBus = new QueryMessageBus($bus);
 
-        $queryBus->handle(new \stdClass());
+        $queryBus->handle(new stdClass());
     }
 }