From 7661f96ed40d3139604bc51ea192d2246bdf4a71 Mon Sep 17 00:00:00 2001
From: Yohan Giarelli <yohan@les-tilleuls.coop>
Date: Wed, 15 Jan 2025 22:57:44 +0100
Subject: [PATCH] ci: Added PHP CS Fixer

---
 .github/workflows/ci.yml                      | 18 ++++++
 .gitignore                                    |  1 +
 .php-cs-fixer.dist.php                        | 35 +++++++++++
 composer.json                                 |  3 +-
 examples/basic-graph.php                      | 24 ++++----
 src/Dumper/Dumper.php                         |  2 +
 src/Dumper/MermaidDumper.php                  |  8 ++-
 src/Event/CanTransitionEvent.php              |  2 +
 src/Event/Event.php                           |  5 +-
 src/Event/EventDispatcher.php                 |  6 +-
 src/Event/PostTransitionEvent.php             |  2 +
 src/Event/PreTransitionEvent.php              |  2 +
 src/Event/TransitionEvent.php                 |  5 +-
 src/Exception/BadStateClassException.php      |  3 +-
 src/Exception/FiniteException.php             |  2 +
 src/Exception/NoStateFoundException.php       |  2 +
 src/Exception/NonUniqueStateException.php     |  2 +
 src/Exception/PropertyNotFoundException.php   |  2 +
 .../TransitionNotReachableException.php       |  2 +
 .../DependencyInjection/FiniteExtension.php   |  1 +
 src/Extension/Symfony/Bundle/FiniteBundle.php |  3 +-
 .../Command/DumpStateMachineCommand.php       | 12 ++--
 src/Extension/Twig/FiniteExtension.php        |  5 +-
 src/State.php                                 |  3 +-
 src/StateMachine.php                          | 46 +++++++-------
 src/Transition/Transition.php                 |  9 +--
 src/Transition/TransitionInterface.php        |  2 +
 tests/Dumper/MermaidDumperTest.php            |  6 +-
 tests/E2E/AlternativeGraphTest.php            | 16 ++---
 tests/E2E/BasicGraphTest.php                  | 22 +++----
 tests/Event/EventDispatcherTest.php           | 14 +++--
 .../FiniteExtensionTest.php                   |  5 +-
 .../Command/DumpStateMachineCommandTest.php   |  8 ++-
 .../Fixtures/Controller/FiniteController.php  |  6 +-
 .../Symfony/Fixtures/Model/Document.php       |  1 +
 .../Symfony/Fixtures/State/DocumentState.php  |  1 +
 .../Symfony/Fixtures/app/AppKernel.php        |  5 +-
 .../Symfony/Fixtures/app/bootstrap.php        |  3 +-
 .../Symfony/Functional/FiniteTest.php         |  3 +-
 tests/Extension/Symfony/ServiceTest.php       |  9 ++-
 tests/Extension/Twig/FiniteExtensionTest.php  |  5 +-
 tests/Fixtures/AlternativeArticle.php         |  8 ++-
 tests/Fixtures/AlternativeArticleState.php    | 14 +++--
 tests/Fixtures/Article.php                    | 10 +--
 tests/Fixtures/SimpleArticleState.php         | 20 +++---
 tests/StateMachineTest.php                    | 61 ++++++++++---------
 tests/Transition/TransitionTest.php           |  9 +--
 47 files changed, 274 insertions(+), 159 deletions(-)
 create mode 100644 .php-cs-fixer.dist.php

diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index 7a708f8..e7f7c1a 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -5,6 +5,24 @@ on:
     pull_request:
 
 jobs:
+    cs:
+        runs-on: ubuntu-latest
+        name: Code Style
+        steps:
+            -   name: Checkout
+                uses: actions/checkout@v4
+
+            -   name: Setup PHP
+                uses: shivammathur/setup-php@v2
+                with:
+                    php-version: '8.3'
+
+            -   name: Install dependencies
+                uses: ramsey/composer-install@v3
+
+            -   name: PHP CS Fixer
+                run: ./vendor/bin/php-cs-fixer fix --dry-run --diff --ansi
+
     psalm:
         runs-on: ubuntu-latest
         permissions:
diff --git a/.gitignore b/.gitignore
index 176e243..baf5363 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1,4 +1,5 @@
 tests/Extension/Symfony/Fixtures/app/var/
 vendor/
 .phpunit.result.cache
+.php-cs-fixer.cache
 composer.lock
diff --git a/.php-cs-fixer.dist.php b/.php-cs-fixer.dist.php
new file mode 100644
index 0000000..34ac90f
--- /dev/null
+++ b/.php-cs-fixer.dist.php
@@ -0,0 +1,35 @@
+<?php
+
+$finder = PhpCsFixer\Finder::create()
+    ->in(__DIR__)
+    ->exclude('tests/Extension/Symfony/Fixtures/app/var')
+    ->exclude('vendor');
+
+return (new PhpCsFixer\Config())
+    ->setRiskyAllowed(true)
+    ->setRules([
+        '@Symfony' => true,
+        '@Symfony:risky' => true,
+        'array_syntax' => [
+            'syntax' => 'short',
+        ],
+        'braces' => [
+            'allow_single_line_closure' => true,
+        ],
+        'declare_strict_types' => true,
+        'modernize_types_casting' => true,
+        'native_function_invocation' => ['include' => ['@compiler_optimized'], 'scope' => 'namespaced'],
+        'no_useless_else' => true,
+        'no_useless_return' => true,
+        'ordered_imports' => true,
+        'phpdoc_add_missing_param_annotation' => [
+            'only_untyped' => true,
+        ],
+        'phpdoc_order' => true,
+        'phpdoc_to_comment' => false,
+        'semicolon_after_instruction' => true,
+        'strict_comparison' => true,
+        'strict_param' => true,
+        'ternary_to_null_coalescing' => true,
+    ])
+    ->setFinder($finder);
diff --git a/composer.json b/composer.json
index 801e219..8a4f22a 100644
--- a/composer.json
+++ b/composer.json
@@ -36,7 +36,8 @@
         "symfony/twig-bundle": ">=5.4,<8",
         "symfony/browser-kit": ">=5.4,<8",
         "symfony/css-selector": ">=5.4,<8",
-        "symfony/console": ">=5.4,<8"
+        "symfony/console": ">=5.4,<8",
+        "friendsofphp/php-cs-fixer": "^3.68"
     },
     "autoload": {
         "psr-4": {
diff --git a/examples/basic-graph.php b/examples/basic-graph.php
index 9f45c0e..49e83f9 100755
--- a/examples/basic-graph.php
+++ b/examples/basic-graph.php
@@ -1,18 +1,20 @@
 #!/usr/bin/env php
 <?php
 
+declare(strict_types=1);
+
 use Finite\State;
 use Finite\Transition\Transition;
 
-require_once __DIR__ . '/../vendor/autoload.php';
+require_once __DIR__.'/../vendor/autoload.php';
 
 // Implement your State
 enum DocumentState: string implements State
 {
-    const PUBLISH = 'publish';
-    const CLEAR = 'clear';
-    const REPORT = 'report';
-    const DISABLE = 'disable';
+    public const PUBLISH = 'publish';
+    public const CLEAR = 'clear';
+    public const REPORT = 'report';
+    public const DISABLE = 'disable';
 
     case DRAFT = 'draft';
     case PUBLISHED = 'published';
@@ -21,12 +23,12 @@ enum DocumentState: string implements State
 
     public function isDeletable(): bool
     {
-        return in_array($this, [self::DRAFT, self::DISABLED]);
+        return in_array($this, [self::DRAFT, self::DISABLED], true);
     }
 
     public function isPrintable(): bool
     {
-        return in_array($this, [self::PUBLISHED, self::REPORTED]);
+        return in_array($this, [self::PUBLISHED, self::REPORTED], true);
     }
 
     public static function getTransitions(): array
@@ -57,16 +59,14 @@ public function setState(DocumentState $state): void
 }
 
 // Configure your graph
-$document     = new Document;
-$stateMachine = new Finite\StateMachine;
-
+$document = new Document();
+$stateMachine = new Finite\StateMachine();
 
 // Working with workflow
 
 // Current state
 var_dump($document->getState());
 
-
 // Available transitions
 var_dump($stateMachine->getReachablesTransitions($document));
 var_dump($stateMachine->can($document, DocumentState::PUBLISH));
@@ -77,7 +77,7 @@ public function setState(DocumentState $state): void
 // Apply transitions
 try {
     $stateMachine->apply($document, DocumentState::REPORT);
-} catch (\Exception $e) {
+} catch (Exception $e) {
     echo $e->getMessage(), "\n";
 }
 
diff --git a/src/Dumper/Dumper.php b/src/Dumper/Dumper.php
index a025ccd..e1bbae3 100644
--- a/src/Dumper/Dumper.php
+++ b/src/Dumper/Dumper.php
@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 namespace Finite\Dumper;
 
 use Finite\State;
diff --git a/src/Dumper/MermaidDumper.php b/src/Dumper/MermaidDumper.php
index 5fcb67f..3a241b6 100644
--- a/src/Dumper/MermaidDumper.php
+++ b/src/Dumper/MermaidDumper.php
@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 namespace Finite\Dumper;
 
 use Finite\State;
@@ -13,14 +15,14 @@ public function dump(string $stateEnum): string
     {
         $output = [
             '---',
-            'title: ' . $stateEnum,
+            'title: '.$stateEnum,
             '---',
             'stateDiagram-v2',
         ];
 
         foreach ($stateEnum::getTransitions() as $transition) {
             foreach ($transition->getSourceStates() as $state) {
-                $output[] = sprintf(
+                $output[] = \sprintf(
                     '    %s --> %s: %s',
                     $state->value,
                     $transition->getTargetState()->value,
@@ -29,6 +31,6 @@ public function dump(string $stateEnum): string
             }
         }
 
-        return implode(PHP_EOL, $output);
+        return implode(\PHP_EOL, $output);
     }
 }
diff --git a/src/Event/CanTransitionEvent.php b/src/Event/CanTransitionEvent.php
index f1afc0e..7d7bdae 100644
--- a/src/Event/CanTransitionEvent.php
+++ b/src/Event/CanTransitionEvent.php
@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 namespace Finite\Event;
 
 final class CanTransitionEvent extends TransitionEvent
diff --git a/src/Event/Event.php b/src/Event/Event.php
index fb6f3ea..469a861 100644
--- a/src/Event/Event.php
+++ b/src/Event/Event.php
@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 namespace Finite\Event;
 
 use Psr\EventDispatcher\StoppableEventInterface;
@@ -11,8 +13,7 @@ abstract class Event implements StoppableEventInterface
     public function __construct(
         private readonly object $object,
         private readonly ?string $stateClass = null,
-    )
-    {
+    ) {
     }
 
     public function getObject(): object
diff --git a/src/Event/EventDispatcher.php b/src/Event/EventDispatcher.php
index a375ce6..41c7a0c 100644
--- a/src/Event/EventDispatcher.php
+++ b/src/Event/EventDispatcher.php
@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 namespace Finite\Event;
 
 use Psr\EventDispatcher\EventDispatcherInterface;
@@ -23,11 +25,11 @@ public function addEventListener(string $eventClass, callable $listener): void
 
     public function dispatch(object $event): void
     {
-        if (!isset($this->listeners[get_class($event)])) {
+        if (!isset($this->listeners[$event::class])) {
             return;
         }
 
-        foreach ($this->listeners[get_class($event)] as $listener) {
+        foreach ($this->listeners[$event::class] as $listener) {
             $listener($event);
 
             if ($event instanceof StoppableEventInterface && $event->isPropagationStopped()) {
diff --git a/src/Event/PostTransitionEvent.php b/src/Event/PostTransitionEvent.php
index fcbc934..08e65b2 100644
--- a/src/Event/PostTransitionEvent.php
+++ b/src/Event/PostTransitionEvent.php
@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 namespace Finite\Event;
 
 final class PostTransitionEvent extends TransitionEvent
diff --git a/src/Event/PreTransitionEvent.php b/src/Event/PreTransitionEvent.php
index dad9038..c85e5cc 100644
--- a/src/Event/PreTransitionEvent.php
+++ b/src/Event/PreTransitionEvent.php
@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 namespace Finite\Event;
 
 final class PreTransitionEvent extends TransitionEvent
diff --git a/src/Event/TransitionEvent.php b/src/Event/TransitionEvent.php
index 0e60c0e..3f2a948 100644
--- a/src/Event/TransitionEvent.php
+++ b/src/Event/TransitionEvent.php
@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 namespace Finite\Event;
 
 abstract class TransitionEvent extends Event
@@ -8,8 +10,7 @@ public function __construct(
         object $object,
         private readonly string $transitionName,
         ?string $stateClass = null,
-    )
-    {
+    ) {
         parent::__construct($object, $stateClass);
     }
 
diff --git a/src/Exception/BadStateClassException.php b/src/Exception/BadStateClassException.php
index 5b4098d..7059efe 100644
--- a/src/Exception/BadStateClassException.php
+++ b/src/Exception/BadStateClassException.php
@@ -1,8 +1,9 @@
 <?php
 
+declare(strict_types=1);
+
 namespace Finite\Exception;
 
 class BadStateClassException extends \InvalidArgumentException implements FiniteException
 {
-
 }
diff --git a/src/Exception/FiniteException.php b/src/Exception/FiniteException.php
index a318fa2..1578c0c 100644
--- a/src/Exception/FiniteException.php
+++ b/src/Exception/FiniteException.php
@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 namespace Finite\Exception;
 
 interface FiniteException
diff --git a/src/Exception/NoStateFoundException.php b/src/Exception/NoStateFoundException.php
index fca4fb1..aacfca2 100644
--- a/src/Exception/NoStateFoundException.php
+++ b/src/Exception/NoStateFoundException.php
@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 namespace Finite\Exception;
 
 class NoStateFoundException extends \InvalidArgumentException implements FiniteException
diff --git a/src/Exception/NonUniqueStateException.php b/src/Exception/NonUniqueStateException.php
index c578bb0..934edaf 100644
--- a/src/Exception/NonUniqueStateException.php
+++ b/src/Exception/NonUniqueStateException.php
@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 namespace Finite\Exception;
 
 final class NonUniqueStateException extends \InvalidArgumentException implements FiniteException
diff --git a/src/Exception/PropertyNotFoundException.php b/src/Exception/PropertyNotFoundException.php
index 395ae8e..b3718fc 100644
--- a/src/Exception/PropertyNotFoundException.php
+++ b/src/Exception/PropertyNotFoundException.php
@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 namespace Finite\Exception;
 
 class PropertyNotFoundException extends \OutOfBoundsException implements FiniteException
diff --git a/src/Exception/TransitionNotReachableException.php b/src/Exception/TransitionNotReachableException.php
index 1838acc..8aafdb1 100644
--- a/src/Exception/TransitionNotReachableException.php
+++ b/src/Exception/TransitionNotReachableException.php
@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 namespace Finite\Exception;
 
 final class TransitionNotReachableException extends \InvalidArgumentException implements FiniteException
diff --git a/src/Extension/Symfony/Bundle/DependencyInjection/FiniteExtension.php b/src/Extension/Symfony/Bundle/DependencyInjection/FiniteExtension.php
index 58240ec..c898155 100644
--- a/src/Extension/Symfony/Bundle/DependencyInjection/FiniteExtension.php
+++ b/src/Extension/Symfony/Bundle/DependencyInjection/FiniteExtension.php
@@ -1,4 +1,5 @@
 <?php
+
 declare(strict_types=1);
 
 namespace Finite\Extension\Symfony\Bundle\DependencyInjection;
diff --git a/src/Extension/Symfony/Bundle/FiniteBundle.php b/src/Extension/Symfony/Bundle/FiniteBundle.php
index e70e7d8..5a7f49e 100644
--- a/src/Extension/Symfony/Bundle/FiniteBundle.php
+++ b/src/Extension/Symfony/Bundle/FiniteBundle.php
@@ -1,4 +1,5 @@
 <?php
+
 declare(strict_types=1);
 
 namespace Finite\Extension\Symfony\Bundle;
@@ -11,6 +12,6 @@ final class FiniteBundle extends Bundle
 {
     public function registerCommands(Application $application): void
     {
-        $application->add(new DumpStateMachineCommand);
+        $application->add(new DumpStateMachineCommand());
     }
 }
diff --git a/src/Extension/Symfony/Command/DumpStateMachineCommand.php b/src/Extension/Symfony/Command/DumpStateMachineCommand.php
index ec05021..f342b77 100644
--- a/src/Extension/Symfony/Command/DumpStateMachineCommand.php
+++ b/src/Extension/Symfony/Command/DumpStateMachineCommand.php
@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 namespace Finite\Extension\Symfony\Command;
 
 use Finite\Dumper\MermaidDumper;
@@ -27,23 +29,23 @@ protected function configure(): void
     protected function execute(InputInterface $input, OutputInterface $output): int
     {
         $io = new SymfonyStyle($input, $output);
-        $stateEnum = (string)$input->getArgument('state_enum');
+        $stateEnum = (string) $input->getArgument('state_enum');
 
         if (!(enum_exists($stateEnum) && is_subclass_of($stateEnum, State::class))) {
-            $io->error('The state enum "' . $stateEnum . '" does not exist.');
+            $io->error('The state enum "'.$stateEnum.'" does not exist.');
 
             return self::FAILURE;
         }
 
-        $format = (string)$input->getArgument('format');
+        $format = (string) $input->getArgument('format');
         switch ($format) {
             case self::FORMAT_MERMAID:
                 /** @psalm-suppress ArgumentTypeCoercion Type is enforced upper but not detected by psalm */
-                $output->writeln((new MermaidDumper)->dump($stateEnum));
+                $output->writeln((new MermaidDumper())->dump($stateEnum));
 
                 break;
             default:
-                $output->writeln('Unknown format "' . $format . '". Supported formats are: ' . implode(', ', self::FORMATS));
+                $output->writeln('Unknown format "'.$format.'". Supported formats are: '.implode(', ', self::FORMATS));
 
                 return self::FAILURE;
         }
diff --git a/src/Extension/Twig/FiniteExtension.php b/src/Extension/Twig/FiniteExtension.php
index e0abd34..c8281ab 100644
--- a/src/Extension/Twig/FiniteExtension.php
+++ b/src/Extension/Twig/FiniteExtension.php
@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 namespace Finite\Extension\Twig;
 
 use Finite\StateMachine;
@@ -10,8 +12,7 @@ class FiniteExtension extends AbstractExtension
 {
     public function __construct(
         private readonly StateMachine $stateMachine,
-    )
-    {
+    ) {
     }
 
     public function getFunctions(): array
diff --git a/src/State.php b/src/State.php
index dcd2adc..3851328 100644
--- a/src/State.php
+++ b/src/State.php
@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 namespace Finite;
 
 use Finite\Transition\TransitionInterface;
@@ -11,4 +13,3 @@ interface State
      */
     public static function getTransitions(): array;
 }
-
diff --git a/src/StateMachine.php b/src/StateMachine.php
index 2847231..bf57dc5 100644
--- a/src/StateMachine.php
+++ b/src/StateMachine.php
@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 namespace Finite;
 
 use Finite\Event\CanTransitionEvent;
@@ -17,9 +19,8 @@
 class StateMachine
 {
     public function __construct(
-        private readonly EventDispatcherInterface $dispatcher = new EventDispatcher,
-    )
-    {
+        private readonly EventDispatcherInterface $dispatcher = new EventDispatcher(),
+    ) {
     }
 
     /**
@@ -28,16 +29,16 @@ public function __construct(
     public function apply(object $object, string $transitionName, ?string $stateClass = null): void
     {
         if (!$this->can($object, $transitionName, $stateClass)) {
-            throw new TransitionNotReachableException('Unable to apply transition ' . $transitionName);
+            throw new TransitionNotReachableException('Unable to apply transition '.$transitionName);
         }
 
-        $property   = $this->extractStateProperty($object, $stateClass);
+        $property = $this->extractStateProperty($object, $stateClass);
         $transition = array_values(
-                          array_filter(
-                              $this->extractState($object, $stateClass)::getTransitions(),
-                              fn(TransitionInterface $transition) => $transition->getName() === $transitionName,
-                          )
-                      )[0];
+            array_filter(
+                $this->extractState($object, $stateClass)::getTransitions(),
+                fn (TransitionInterface $transition) => $transition->getName() === $transitionName,
+            )
+        )[0];
 
         $this->dispatcher->dispatch(new PreTransitionEvent($object, $transitionName, $stateClass));
 
@@ -63,7 +64,7 @@ public function can(object $object, string $transitionName, ?string $stateClass
                 continue;
             }
 
-            if (!in_array($state, $transition->getSourceStates())) {
+            if (!\in_array($state, $transition->getSourceStates(), true)) {
                 return false;
             }
 
@@ -73,7 +74,7 @@ public function can(object $object, string $transitionName, ?string $stateClass
             return !$event->isTransitionBlocked();
         }
 
-        throw new TransitionNotReachableException(sprintf('No transition "%s" found', $transitionName));
+        throw new TransitionNotReachableException(\sprintf('No transition "%s" found', $transitionName));
     }
 
     /**
@@ -85,20 +86,21 @@ public function getReachablesTransitions(object $object, ?string $stateClass = n
 
         return array_filter(
             $state->getTransitions(),
-            fn(TransitionInterface $transition) => $this->can($object, $transition->getName(), $stateClass),
+            fn (TransitionInterface $transition) => $this->can($object, $transition->getName(), $stateClass),
         );
     }
 
     /**
      * @psalm-suppress MoreSpecificReturnType
      * @psalm-suppress LessSpecificReturnStatement
+     *
      * @return array<int, enum-string<State>>
      */
     public function getStateClasses(object $object): array
     {
         return array_filter(
             array_map(
-                fn(\ReflectionProperty $property): string => (string)$property->getType(),
+                fn (\ReflectionProperty $property): string => (string) $property->getType(),
                 $this->extractStateProperties($object),
             ),
             fn (?string $name): bool => enum_exists($name),
@@ -107,7 +109,7 @@ public function getStateClasses(object $object): array
 
     public function hasState(object $object): bool
     {
-        return count($this->extractStateProperties($object)) > 0;
+        return \count($this->extractStateProperties($object)) > 0;
     }
 
     public function getDispatcher(): EventDispatcherInterface
@@ -132,29 +134,29 @@ private function extractState(object $object, ?string $stateClass = null): State
     private function extractStateProperty(object $object, ?string $stateClass = null): \ReflectionProperty
     {
         if ($stateClass && !enum_exists($stateClass)) {
-            throw new NoStateFoundException(sprintf('Enum "%s" does not exists', $stateClass));
+            throw new NoStateFoundException(\sprintf('Enum "%s" does not exists', $stateClass));
         }
 
         $properties = $this->extractStateProperties($object);
         if (null !== $stateClass) {
             foreach ($properties as $property) {
-                if ((string)($property->getType()) === $stateClass) {
+                if ((string) $property->getType() === $stateClass) {
                     return $property;
                 }
             }
 
-            throw new BadStateClassException(sprintf('Found no state on object "%s" with class "%s"', get_class($object), $stateClass));
+            throw new BadStateClassException(\sprintf('Found no state on object "%s" with class "%s"', $object::class, $stateClass));
         }
 
-        if (1 === count($properties)) {
+        if (1 === \count($properties)) {
             return $properties[0];
         }
 
-        if (count($properties) > 1) {
-            throw new NonUniqueStateException('Found multiple states on object ' . get_class($object));
+        if (\count($properties) > 1) {
+            throw new NonUniqueStateException('Found multiple states on object '.$object::class);
         }
 
-        throw new NoStateFoundException('Found no state on object ' . get_class($object));
+        throw new NoStateFoundException('Found no state on object '.$object::class);
     }
 
     /**
diff --git a/src/Transition/Transition.php b/src/Transition/Transition.php
index c8aef70..3eb6f10 100644
--- a/src/Transition/Transition.php
+++ b/src/Transition/Transition.php
@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 namespace Finite\Transition;
 
 use Finite\Exception\PropertyNotFoundException;
@@ -19,9 +21,8 @@ public function __construct(
         public readonly array $sourceStates,
         public readonly State&\BackedEnum $targetState,
         /** @var array<string, string> */
-        public readonly array $properties = []
-    )
-    {
+        public readonly array $properties = [],
+    ) {
     }
 
     public function getSourceStates(): array
@@ -51,7 +52,7 @@ public function hasProperty(string $name): bool
     public function getPropertyValue(string $name): mixed
     {
         if (!$this->hasProperty($name)) {
-            throw new PropertyNotFoundException(sprintf('Property "%s" does not exist', $name));
+            throw new PropertyNotFoundException(\sprintf('Property "%s" does not exist', $name));
         }
 
         return $this->properties[$name];
diff --git a/src/Transition/TransitionInterface.php b/src/Transition/TransitionInterface.php
index eea9fd3..c4fd924 100644
--- a/src/Transition/TransitionInterface.php
+++ b/src/Transition/TransitionInterface.php
@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 namespace Finite\Transition;
 
 use Finite\State;
diff --git a/tests/Dumper/MermaidDumperTest.php b/tests/Dumper/MermaidDumperTest.php
index 86d143b..8da57a9 100644
--- a/tests/Dumper/MermaidDumperTest.php
+++ b/tests/Dumper/MermaidDumperTest.php
@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 namespace Finite\Tests\Dumper;
 
 use Finite\Dumper\MermaidDumper;
@@ -8,7 +10,7 @@
 
 class MermaidDumperTest extends TestCase
 {
-    public function test_it_dumps(): void
+    public function testItDumps(): void
     {
         $this->assertSame(
             <<<MERMAID
@@ -23,7 +25,7 @@ public function test_it_dumps(): void
                 reported --> disabled: disable
                 published --> disabled: disable
             MERMAID,
-            (new MermaidDumper)->dump(SimpleArticleState::class)
+            (new MermaidDumper())->dump(SimpleArticleState::class)
         );
     }
 }
diff --git a/tests/E2E/AlternativeGraphTest.php b/tests/E2E/AlternativeGraphTest.php
index b009ec8..fbf61d7 100644
--- a/tests/E2E/AlternativeGraphTest.php
+++ b/tests/E2E/AlternativeGraphTest.php
@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 namespace Finite\Tests\E2E;
 
 use Finite\StateMachine;
@@ -10,15 +12,15 @@
 class AlternativeGraphTest extends TestCase
 {
     private AlternativeArticle $article;
-    private StateMachine       $stateMachine;
+    private StateMachine $stateMachine;
 
     protected function setUp(): void
     {
-        $this->article      = new AlternativeArticle('Hi ! I\'m an article.');
-        $this->stateMachine = new StateMachine;
+        $this->article = new AlternativeArticle('Hi ! I\'m an article.');
+        $this->stateMachine = new StateMachine();
     }
 
-    public function test_it_has_transitions(): void
+    public function testItHasTransitions(): void
     {
         $this->assertCount(2, $this->article->getAlternativeState()::getTransitions());
         $this->assertCount(1, $this->stateMachine->getReachablesTransitions($this->article, AlternativeArticleState::class));
@@ -29,20 +31,20 @@ public function test_it_has_transitions(): void
         );
     }
 
-    public function test_it_allows_to_transition(): void
+    public function testItAllowsToTransition(): void
     {
         $this->assertTrue($this->stateMachine->can($this->article, AlternativeArticleState::MARK_READ, AlternativeArticleState::class));
         $this->assertFalse($this->stateMachine->can($this->article, AlternativeArticleState::MARK_OLD, AlternativeArticleState::class));
     }
 
-    public function test_it_applies_transition(): void
+    public function testItAppliesTransition(): void
     {
         $this->stateMachine->apply($this->article, AlternativeArticleState::MARK_READ, AlternativeArticleState::class);
 
         $this->assertSame(AlternativeArticleState::READ, $this->article->getAlternativeState());
     }
 
-    public function test_it_reject_bad_transition(): void
+    public function testItRejectBadTransition(): void
     {
         $this->expectException(\InvalidArgumentException::class);
         $this->stateMachine->apply($this->article, AlternativeArticleState::MARK_OLD, AlternativeArticleState::class);
diff --git a/tests/E2E/BasicGraphTest.php b/tests/E2E/BasicGraphTest.php
index f5eb41c..8aeb13a 100644
--- a/tests/E2E/BasicGraphTest.php
+++ b/tests/E2E/BasicGraphTest.php
@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 namespace Finite\Tests\E2E;
 
 use Finite\Exception\FiniteException;
@@ -11,23 +13,21 @@
 
 class BasicGraphTest extends TestCase
 {
-    private Article      $article;
+    private Article $article;
     private StateMachine $stateMachine;
 
     protected function setUp(): void
     {
-        $this->article      = new Article('Hi ! I\'m an article.');
-        $this->stateMachine = new StateMachine;
+        $this->article = new Article('Hi ! I\'m an article.');
+        $this->stateMachine = new StateMachine();
     }
 
-
-    public function test_instantiate_and_have_state(): void
+    public function testInstantiateAndHaveState(): void
     {
-
         $this->assertSame('draft', $this->article->getState()->value);
     }
 
-    public function test_it_has_transitions(): void
+    public function testItHasTransitions(): void
     {
         $this->assertCount(4, $this->article->getState()::getTransitions());
         $this->assertCount(1, $this->stateMachine->getReachablesTransitions($this->article));
@@ -35,7 +35,7 @@ public function test_it_has_transitions(): void
         $this->assertSame(SimpleArticleState::PUBLISHED, $this->stateMachine->getReachablesTransitions($this->article)[0]->getTargetState());
     }
 
-    public function test_it_reject_bad_transitions(): void
+    public function testItRejectBadTransitions(): void
     {
         $this->expectException(\InvalidArgumentException::class);
         $this->expectException(FiniteException::class);
@@ -44,20 +44,20 @@ public function test_it_reject_bad_transitions(): void
         $this->stateMachine->can($this->article, 'touch this');
     }
 
-    public function test_it_allows_to_transition(): void
+    public function testItAllowsToTransition(): void
     {
         $this->assertTrue($this->stateMachine->can($this->article, SimpleArticleState::PUBLISH));
         $this->assertFalse($this->stateMachine->can($this->article, SimpleArticleState::REPORT));
     }
 
-    public function test_it_applies_transition(): void
+    public function testItAppliesTransition(): void
     {
         $this->stateMachine->apply($this->article, SimpleArticleState::PUBLISH);
 
         $this->assertSame(SimpleArticleState::PUBLISHED, $this->article->getState());
     }
 
-    public function test_it_reject_bad_transition(): void
+    public function testItRejectBadTransition(): void
     {
         $this->expectException(\InvalidArgumentException::class);
 
diff --git a/tests/Event/EventDispatcherTest.php b/tests/Event/EventDispatcherTest.php
index 1b2bc26..630ec83 100644
--- a/tests/Event/EventDispatcherTest.php
+++ b/tests/Event/EventDispatcherTest.php
@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 namespace Finite\Tests\Event;
 
 use Finite\Event\Event;
@@ -8,7 +10,7 @@
 
 class EventDispatcherTest extends TestCase
 {
-    public function test_it_dispatch_event(): void
+    public function testItDispatchEvent(): void
     {
         $objectMock = $this->getMockBuilder(Event::class)->disableOriginalConstructor()->getMock();
 
@@ -18,14 +20,14 @@ public function test_it_dispatch_event(): void
         $badListenerMock = $this->getMockBuilder(\stdClass::class)->addMethods(['__invoke'])->getMock();
         $badListenerMock->expects($this->never())->method('__invoke')->with($objectMock);
 
-        $dispatcher = new EventDispatcher;
+        $dispatcher = new EventDispatcher();
         $dispatcher->addEventListener($objectMock::class, $listenerMock);
         $dispatcher->addEventListener(Event::class, $badListenerMock);
 
         $dispatcher->dispatch($objectMock);
     }
 
-    public function test_it_dispatch_multiple_event(): void
+    public function testItDispatchMultipleEvent(): void
     {
         $objectMock = $this->getMockBuilder(Event::class)->disableOriginalConstructor()->getMock();
 
@@ -35,14 +37,14 @@ public function test_it_dispatch_multiple_event(): void
         $otherListenerMock = $this->getMockBuilder(\stdClass::class)->addMethods(['__invoke'])->getMock();
         $otherListenerMock->expects($this->once())->method('__invoke')->with($objectMock);
 
-        $dispatcher = new EventDispatcher;
+        $dispatcher = new EventDispatcher();
         $dispatcher->addEventListener($objectMock::class, $listenerMock);
         $dispatcher->addEventListener($objectMock::class, $otherListenerMock);
 
         $dispatcher->dispatch($objectMock);
     }
 
-    public function test_it_stops_propagation(): void
+    public function testItStopsPropagation(): void
     {
         $objectMock = $this->getMockBuilder(Event::class)->disableOriginalConstructor()->getMock();
         $objectMock->expects($this->once())->method('stopPropagation');
@@ -58,7 +60,7 @@ public function test_it_stops_propagation(): void
         $otherListenerMock = $this->getMockBuilder(\stdClass::class)->addMethods(['__invoke'])->getMock();
         $otherListenerMock->expects($this->never())->method('__invoke')->with($objectMock);
 
-        $dispatcher = new EventDispatcher;
+        $dispatcher = new EventDispatcher();
         $dispatcher->addEventListener($objectMock::class, $listenerMock);
         $dispatcher->addEventListener($objectMock::class, $otherListenerMock);
 
diff --git a/tests/Extension/Symfony/Bundle/DependencyInjection/FiniteExtensionTest.php b/tests/Extension/Symfony/Bundle/DependencyInjection/FiniteExtensionTest.php
index 143833e..a455bed 100644
--- a/tests/Extension/Symfony/Bundle/DependencyInjection/FiniteExtensionTest.php
+++ b/tests/Extension/Symfony/Bundle/DependencyInjection/FiniteExtensionTest.php
@@ -1,4 +1,5 @@
 <?php
+
 declare(strict_types=1);
 
 namespace Finite\Tests\Extension\Symfony\Bundle\DependencyInjection;
@@ -11,7 +12,7 @@
 
 class FiniteExtensionTest extends TestCase
 {
-    public function test_it_loads_services(): void
+    public function testItLoadsServices(): void
     {
         $container = $this->getMockBuilder(ContainerBuilder::class)
             ->disableOriginalConstructor()
@@ -25,7 +26,7 @@ public function test_it_loads_services(): void
             ),
         );
 
-        $extension = new FiniteExtension;
+        $extension = new FiniteExtension();
         $extension->load([], $container);
     }
 }
diff --git a/tests/Extension/Symfony/Command/DumpStateMachineCommandTest.php b/tests/Extension/Symfony/Command/DumpStateMachineCommandTest.php
index 06ce63a..9266553 100644
--- a/tests/Extension/Symfony/Command/DumpStateMachineCommandTest.php
+++ b/tests/Extension/Symfony/Command/DumpStateMachineCommandTest.php
@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 namespace Finite\Tests\Extension\Symfony\Command;
 
 use Finite\Tests\Extension\Symfony\Fixtures\State\DocumentState;
@@ -22,7 +24,7 @@ protected function setUp(): void
         $this->commandTester = new CommandTester($command);
     }
 
-    public function test_it_returns_mermaid_dump(): void
+    public function testItReturnsMermaidDump(): void
     {
         $this->commandTester->execute([
             'state_enum' => DocumentState::class,
@@ -32,7 +34,7 @@ public function test_it_returns_mermaid_dump(): void
         $this->commandTester->assertCommandIsSuccessful();
     }
 
-    public function test_it_fails_with_unknown_state_enum(): void
+    public function testItFailsWithUnknownStateEnum(): void
     {
         $this->commandTester->execute([
             'state_enum' => 'UnknownStateEnum',
@@ -42,7 +44,7 @@ public function test_it_fails_with_unknown_state_enum(): void
         $this->assertSame(1, $this->commandTester->getStatusCode());
     }
 
-    public function test_it_fails_with_unknown_format(): void
+    public function testItFailsWithUnknownFormat(): void
     {
         $this->commandTester->execute([
             'state_enum' => DocumentState::class,
diff --git a/tests/Extension/Symfony/Fixtures/Controller/FiniteController.php b/tests/Extension/Symfony/Fixtures/Controller/FiniteController.php
index 7012fb7..10c9f7a 100644
--- a/tests/Extension/Symfony/Fixtures/Controller/FiniteController.php
+++ b/tests/Extension/Symfony/Fixtures/Controller/FiniteController.php
@@ -1,4 +1,5 @@
 <?php
+
 declare(strict_types=1);
 
 namespace Finite\Tests\Extension\Symfony\Fixtures\Controller;
@@ -11,13 +12,12 @@ final class FiniteController
 {
     public function __construct(
         private readonly Environment $twig,
-    )
-    {
+    ) {
     }
 
     public function __invoke(): Response
     {
-        $document = new Document;
+        $document = new Document();
 
         return new Response(
             $this->twig->render(
diff --git a/tests/Extension/Symfony/Fixtures/Model/Document.php b/tests/Extension/Symfony/Fixtures/Model/Document.php
index 58a927f..4666cdc 100644
--- a/tests/Extension/Symfony/Fixtures/Model/Document.php
+++ b/tests/Extension/Symfony/Fixtures/Model/Document.php
@@ -1,4 +1,5 @@
 <?php
+
 declare(strict_types=1);
 
 namespace Finite\Tests\Extension\Symfony\Fixtures\Model;
diff --git a/tests/Extension/Symfony/Fixtures/State/DocumentState.php b/tests/Extension/Symfony/Fixtures/State/DocumentState.php
index ebd0880..93981f7 100644
--- a/tests/Extension/Symfony/Fixtures/State/DocumentState.php
+++ b/tests/Extension/Symfony/Fixtures/State/DocumentState.php
@@ -1,4 +1,5 @@
 <?php
+
 declare(strict_types=1);
 
 namespace Finite\Tests\Extension\Symfony\Fixtures\State;
diff --git a/tests/Extension/Symfony/Fixtures/app/AppKernel.php b/tests/Extension/Symfony/Fixtures/app/AppKernel.php
index 3dfe278..319a341 100644
--- a/tests/Extension/Symfony/Fixtures/app/AppKernel.php
+++ b/tests/Extension/Symfony/Fixtures/app/AppKernel.php
@@ -1,4 +1,5 @@
 <?php
+
 declare(strict_types=1);
 
 use Finite\Tests\Extension\Symfony\Fixtures\Controller\FiniteController;
@@ -36,7 +37,7 @@ protected function configureContainer(ContainerBuilder $c, LoaderInterface $load
                 FiniteController::class => (new Definition(FiniteController::class))
                     ->setAutowired(true)
                     ->addTag('controller.service_arguments'),
-                'logger' => (new Definition(NullLogger::class))
+                'logger' => (new Definition(NullLogger::class)),
             ]
         );
     }
@@ -48,7 +49,7 @@ public function getProjectDir(): string
 
     public function loadRoutes(LoaderInterface $loader): RouteCollection
     {
-        $collection = new RouteCollection;
+        $collection = new RouteCollection();
         $collection->add(
             'finite',
             new Route('/finite', ['_controller' => FiniteController::class]),
diff --git a/tests/Extension/Symfony/Fixtures/app/bootstrap.php b/tests/Extension/Symfony/Fixtures/app/bootstrap.php
index b58eb5b..3e7798e 100644
--- a/tests/Extension/Symfony/Fixtures/app/bootstrap.php
+++ b/tests/Extension/Symfony/Fixtures/app/bootstrap.php
@@ -1,7 +1,8 @@
 <?php
+
 declare(strict_types=1);
 
-$loader = require __DIR__ . '/../../../../../vendor/autoload.php';
+$loader = require __DIR__.'/../../../../../vendor/autoload.php';
 require __DIR__.'/AppKernel.php';
 
 return $loader;
diff --git a/tests/Extension/Symfony/Functional/FiniteTest.php b/tests/Extension/Symfony/Functional/FiniteTest.php
index 743834d..78c92d9 100644
--- a/tests/Extension/Symfony/Functional/FiniteTest.php
+++ b/tests/Extension/Symfony/Functional/FiniteTest.php
@@ -1,4 +1,5 @@
 <?php
+
 declare(strict_types=1);
 
 namespace Finite\Tests\Extension\Symfony\Functional;
@@ -7,7 +8,7 @@
 
 class FiniteTest extends WebTestCase
 {
-    public function test_finite_controller(): void
+    public function testFiniteController(): void
     {
         $client = $this->createClient();
         $client->request('GET', '/finite');
diff --git a/tests/Extension/Symfony/ServiceTest.php b/tests/Extension/Symfony/ServiceTest.php
index 3e18d95..c306c43 100644
--- a/tests/Extension/Symfony/ServiceTest.php
+++ b/tests/Extension/Symfony/ServiceTest.php
@@ -1,4 +1,5 @@
 <?php
+
 declare(strict_types=1);
 
 namespace Finite\Tests\Extension\Symfony;
@@ -8,14 +9,12 @@
 use Finite\StateMachine;
 use Finite\Tests\Extension\Symfony\Fixtures\Model\Document;
 use Symfony\Bundle\FrameworkBundle\Test\KernelTestCase;
-use Symfony\Bundle\FrameworkBundle\Test\TestContainer;
 use Symfony\Component\EventDispatcher\Debug\TraceableEventDispatcher;
 use Symfony\Contracts\EventDispatcher\EventDispatcherInterface;
-use Twig\Environment;
 
 class ServiceTest extends KernelTestCase
 {
-    public function test_services_are_registered(): void
+    public function testServicesAreRegistered(): void
     {
         $container = static::getContainer();
 
@@ -25,13 +24,13 @@ public function test_services_are_registered(): void
         $this->assertArrayHasKey(FiniteExtension::class, $container->get('twig')->getExtensions());
     }
 
-    public function test_it_uses_the_symfony_dispatcher(): void
+    public function testItUsesTheSymfonyDispatcher(): void
     {
         $container = static::getContainer();
 
         /** @var StateMachine $stateMachine */
         $stateMachine = $container->get(StateMachine::class);
-        $stateMachine->can(new Document, 'publish');
+        $stateMachine->can(new Document(), 'publish');
 
         /** @var TraceableEventDispatcher $debugDispatcher */
         $debugDispatcher = $container->get('debug.event_dispatcher');
diff --git a/tests/Extension/Twig/FiniteExtensionTest.php b/tests/Extension/Twig/FiniteExtensionTest.php
index d0d8923..a3a675b 100644
--- a/tests/Extension/Twig/FiniteExtensionTest.php
+++ b/tests/Extension/Twig/FiniteExtensionTest.php
@@ -1,4 +1,5 @@
 <?php
+
 declare(strict_types=1);
 
 namespace Finite\Tests\Extension\Twig;
@@ -9,7 +10,7 @@
 
 class FiniteExtensionTest extends TestCase
 {
-    public function test_it_declare_twig_functions(): void
+    public function testItDeclareTwigFunctions(): void
     {
         $object = $this->createMock(\stdClass::class);
         $stateMachine = $this->createMock(StateMachine::class);
@@ -17,7 +18,7 @@ public function test_it_declare_twig_functions(): void
         $functions = (new FiniteExtension($stateMachine))->getFunctions();
 
         $functions = array_combine(
-            array_map(fn($function) => $function->getName(), $functions),
+            array_map(fn ($function) => $function->getName(), $functions),
             $functions,
         );
         $this->assertArrayHasKey('finite_can', $functions);
diff --git a/tests/Fixtures/AlternativeArticle.php b/tests/Fixtures/AlternativeArticle.php
index 0324834..89bcb61 100644
--- a/tests/Fixtures/AlternativeArticle.php
+++ b/tests/Fixtures/AlternativeArticle.php
@@ -1,16 +1,18 @@
 <?php
 
+declare(strict_types=1);
+
 namespace Finite\Tests\Fixtures;
 
 class AlternativeArticle
 {
-    private SimpleArticleState          $state            = SimpleArticleState::DRAFT;
-    private AlternativeArticleState     $alternativeState = AlternativeArticleState::NEW;
+    private SimpleArticleState $state = SimpleArticleState::DRAFT;
+    private AlternativeArticleState $alternativeState = AlternativeArticleState::NEW;
     private readonly \DateTimeInterface $createdAt;
 
     public function __construct(public readonly string $title)
     {
-        $this->createdAt = new \DateTimeImmutable;
+        $this->createdAt = new \DateTimeImmutable();
     }
 
     public function getTitle(): string
diff --git a/tests/Fixtures/AlternativeArticleState.php b/tests/Fixtures/AlternativeArticleState.php
index db3f539..1b41138 100644
--- a/tests/Fixtures/AlternativeArticleState.php
+++ b/tests/Fixtures/AlternativeArticleState.php
@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 namespace Finite\Tests\Fixtures;
 
 use Finite\State;
@@ -7,21 +9,21 @@
 
 enum AlternativeArticleState: string implements State
 {
-    case NEW  = 'new';
+    case NEW = 'new';
     case READ = 'read';
-    case OLD  = 'old';
+    case OLD = 'old';
 
-    const MARK_READ = 'mark_read';
-    const MARK_OLD  = 'old';
+    public const MARK_READ = 'mark_read';
+    public const MARK_OLD = 'old';
 
     public function isFeatured(): bool
     {
-        return $this === self::NEW;
+        return self::NEW === $this;
     }
 
     public function isVisible(): bool
     {
-        return in_array($this, [self::NEW, self::READ]);
+        return \in_array($this, [self::NEW, self::READ], true);
     }
 
     public static function getTransitions(): array
diff --git a/tests/Fixtures/Article.php b/tests/Fixtures/Article.php
index e3d4698..0829b7f 100644
--- a/tests/Fixtures/Article.php
+++ b/tests/Fixtures/Article.php
@@ -1,14 +1,16 @@
 <?php
 
+declare(strict_types=1);
+
 namespace Finite\Tests\Fixtures;
 
 class Article
 {
-    public  $noTypeHere = null;
+    public $noTypeHere;
 
-    public  int|float $unionType = 0;
+    public int|float $unionType = 0;
 
-    public  \Traversable&\Countable $intersectionType;
+    public \Traversable&\Countable $intersectionType;
 
     public string $namedType = 'named';
 
@@ -18,7 +20,7 @@ class Article
 
     public function __construct(public readonly string $title)
     {
-        $this->createdAt = new \DateTimeImmutable;
+        $this->createdAt = new \DateTimeImmutable();
     }
 
     public function getTitle(): string
diff --git a/tests/Fixtures/SimpleArticleState.php b/tests/Fixtures/SimpleArticleState.php
index 3c88cab..cfa34ac 100644
--- a/tests/Fixtures/SimpleArticleState.php
+++ b/tests/Fixtures/SimpleArticleState.php
@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 namespace Finite\Tests\Fixtures;
 
 use Finite\State;
@@ -7,24 +9,24 @@
 
 enum SimpleArticleState: string implements State
 {
-    case DRAFT     = 'draft';
+    case DRAFT = 'draft';
     case PUBLISHED = 'published';
-    case REPORTED  = 'reported';
-    case DISABLED  = 'disabled';
+    case REPORTED = 'reported';
+    case DISABLED = 'disabled';
 
-    const PUBLISH = 'publish';
-    const CLEAR   = 'clear';
-    const REPORT  = 'report';
-    const DISABLE = 'disable';
+    public const PUBLISH = 'publish';
+    public const CLEAR = 'clear';
+    public const REPORT = 'report';
+    public const DISABLE = 'disable';
 
     public function isVisible(): bool
     {
-        return in_array($this, [self::PUBLISHED, self::REPORTED]);
+        return \in_array($this, [self::PUBLISHED, self::REPORTED], true);
     }
 
     public function isReviewable(): bool
     {
-        return in_array($this, [self::DRAFT, self::REPORTED]);
+        return \in_array($this, [self::DRAFT, self::REPORTED], true);
     }
 
     public static function getTransitions(): array
diff --git a/tests/StateMachineTest.php b/tests/StateMachineTest.php
index 7016387..6920a86 100644
--- a/tests/StateMachineTest.php
+++ b/tests/StateMachineTest.php
@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 namespace Finite\Tests;
 
 use Finite\Event\CanTransitionEvent;
@@ -9,7 +11,6 @@
 use Finite\Exception\FiniteException;
 use Finite\Exception\NonUniqueStateException;
 use Finite\Exception\NoStateFoundException;
-use Finite\Exception\TransitionNotReachableException;
 use Finite\StateMachine;
 use Finite\Tests\Fixtures\AlternativeArticle;
 use Finite\Tests\Fixtures\AlternativeArticleState;
@@ -20,20 +21,20 @@
 
 class StateMachineTest extends TestCase
 {
-    public function test_it_instantiate_event_dispatcher(): void
+    public function testItInstantiateEventDispatcher(): void
     {
-        $this->assertInstanceOf(EventDispatcher::class, (new StateMachine)->getDispatcher());
-        $this->assertInstanceOf(EventDispatcherInterface::class, (new StateMachine)->getDispatcher());
+        $this->assertInstanceOf(EventDispatcher::class, (new StateMachine())->getDispatcher());
+        $this->assertInstanceOf(EventDispatcherInterface::class, (new StateMachine())->getDispatcher());
     }
 
-    public function test_it_use_constructor_event_dispatcher(): void
+    public function testItUseConstructorEventDispatcher(): void
     {
         $eventDispatcher = $this->getMockBuilder(EventDispatcherInterface::class)->getMock();
 
         $this->assertSame($eventDispatcher, (new StateMachine($eventDispatcher))->getDispatcher());
     }
 
-    public function test_it_can_transition(): void
+    public function testItCanTransition(): void
     {
         $object = new Article('Hi !');
 
@@ -55,7 +56,7 @@ public function test_it_can_transition(): void
         $this->assertTrue($stateMachine->can($object, SimpleArticleState::PUBLISH));
     }
 
-    public function test_it_blocks_transition(): void
+    public function testItBlocksTransition(): void
     {
         $object = new Article('Hi !');
 
@@ -64,15 +65,14 @@ public function test_it_blocks_transition(): void
             ->expects($this->once())
             ->method('dispatch')
             ->with($this->isInstanceOf(CanTransitionEvent::class))
-            ->willReturnCallback(fn(CanTransitionEvent $event) => $event->blockTransition());
-
+            ->willReturnCallback(fn (CanTransitionEvent $event) => $event->blockTransition());
 
         $stateMachine = new StateMachine($eventDispatcher);
 
         $this->assertFalse($stateMachine->can($object, SimpleArticleState::PUBLISH));
     }
 
-    public function test_it_applies_transition(): void
+    public function testItAppliesTransition(): void
     {
         $object = new Article('Hi !');
 
@@ -83,7 +83,7 @@ public function test_it_applies_transition(): void
             ->expects($matcher)
             ->method('dispatch')
             ->willReturnCallback(
-                fn(TransitionEvent $e) => match ($matcher->numberOfInvocations()) {
+                fn (TransitionEvent $e) => match ($matcher->numberOfInvocations()) {
                     1, 2, 3 => SimpleArticleState::PUBLISH === $e->getTransitionName(),
                     4, 5, 6 => SimpleArticleState::REPORT === $e->getTransitionName(),
                 }
@@ -95,13 +95,13 @@ public function test_it_applies_transition(): void
         $stateMachine->apply($object, SimpleArticleState::REPORT);
     }
 
-    public function test_it_rejects_non_stateful_object()
+    public function testItRejectsNonStatefulObject()
     {
         $this->expectException(\InvalidArgumentException::class);
         $this->expectException(FiniteException::class);
         $this->expectException(NoStateFoundException::class);
 
-        (new StateMachine)->can(
+        (new StateMachine())->can(
             new class {
                 public string $title = 'Foobar';
             },
@@ -109,65 +109,66 @@ public function test_it_rejects_non_stateful_object()
         );
     }
 
-    public function test_it_rejects_unexistant_state_class()
+    public function testItRejectsUnexistantStateClass()
     {
         $this->expectException(\InvalidArgumentException::class);
         $this->expectException(FiniteException::class);
         $this->expectException(NoStateFoundException::class);
 
-        (new StateMachine)->can(new \stdClass, 'transition', 'Unexistant enum');
+        (new StateMachine())->can(new \stdClass(), 'transition', 'Unexistant enum');
     }
 
-    public function test_it_throws_if_no_state(): void
+    public function testItThrowsIfNoState(): void
     {
         $this->expectException(\InvalidArgumentException::class);
         $this->expectException(FiniteException::class);
         $this->expectException(NoStateFoundException::class);
 
-        $stateMachine = new StateMachine;
-        $stateMachine->can(new class extends \stdClass {}, 'transition');
+        $stateMachine = new StateMachine();
+        $stateMachine->can(new class extends \stdClass {
+        }, 'transition');
     }
 
-    public function test_it_throws_if_bad_state(): void
+    public function testItThrowsIfBadState(): void
     {
         $this->expectException(\InvalidArgumentException::class);
         $this->expectException(FiniteException::class);
         $this->expectException(BadStateClassException::class);
 
-        $stateMachine = new StateMachine;
+        $stateMachine = new StateMachine();
         $stateMachine->can(new Article('test'), 'publish', AlternativeArticleState::class);
     }
 
-    public function test_it_throws_if_many_state_and_none_given(): void
+    public function testItThrowsIfManyStateAndNoneGiven(): void
     {
         $this->expectException(\InvalidArgumentException::class);
         $this->expectException(FiniteException::class);
         $this->expectException(NonUniqueStateException::class);
 
-        $stateMachine = new StateMachine;
+        $stateMachine = new StateMachine();
         $stateMachine->can(new AlternativeArticle('test'), 'publish');
     }
 
-    public function test_it_returns_class_state_classes(): void
+    public function testItReturnsClassStateClasses(): void
     {
         $this->assertSame(
             [SimpleArticleState::class],
-            (new StateMachine)->getStateClasses(new Article('Hi !')),
+            (new StateMachine())->getStateClasses(new Article('Hi !')),
         );
         $this->assertSame(
             [SimpleArticleState::class, AlternativeArticleState::class],
-            (new StateMachine)->getStateClasses(new AlternativeArticle('Hi !')),
+            (new StateMachine())->getStateClasses(new AlternativeArticle('Hi !')),
         );
         $this->assertSame(
             [],
-            (new StateMachine)->getStateClasses(new \stdClass),
+            (new StateMachine())->getStateClasses(new \stdClass()),
         );
     }
 
-    public function test_it_returns_if_object_has_state(): void
+    public function testItReturnsIfObjectHasState(): void
     {
-        $this->assertTrue((new StateMachine)->hasState(new Article('Hi !')));
-        $this->assertTrue((new StateMachine)->hasState(new AlternativeArticle('Hi !')));
-        $this->assertFalse((new StateMachine)->hasState(new \stdClass));
+        $this->assertTrue((new StateMachine())->hasState(new Article('Hi !')));
+        $this->assertTrue((new StateMachine())->hasState(new AlternativeArticle('Hi !')));
+        $this->assertFalse((new StateMachine())->hasState(new \stdClass()));
     }
 }
diff --git a/tests/Transition/TransitionTest.php b/tests/Transition/TransitionTest.php
index 18fe64e..6d8be2e 100644
--- a/tests/Transition/TransitionTest.php
+++ b/tests/Transition/TransitionTest.php
@@ -1,4 +1,5 @@
 <?php
+
 declare(strict_types=1);
 
 namespace Finite\Tests\Transition;
@@ -23,20 +24,20 @@ protected function setUp(): void
         );
     }
 
-    public function test_it_has_properties(): void
+    public function testItHasProperties(): void
     {
         $this->assertTrue($this->object->hasProperty('property'));
         $this->assertTrue($this->object->hasProperty('property2'));
         $this->assertFalse($this->object->hasProperty('property3'));
     }
 
-    public function test_it_returns_property_value(): void
+    public function testItReturnsPropertyValue(): void
     {
         $this->assertSame('value', $this->object->getPropertyValue('property'));
         $this->assertSame('value2', $this->object->getPropertyValue('property2'));
     }
 
-    public function test_it_throws_exception_when_property_does_not_exist(): void
+    public function testItThrowsExceptionWhenPropertyDoesNotExist(): void
     {
         $this->expectException(\OutOfBoundsException::class);
         $this->expectException(FiniteException::class);
@@ -47,7 +48,7 @@ public function test_it_throws_exception_when_property_does_not_exist(): void
         $this->object->getPropertyValue('property3');
     }
 
-    public function test_it_returns_property_names(): void
+    public function testItReturnsPropertyNames(): void
     {
         $this->assertSame(['property', 'property2'], $this->object->getPropertyNames());
     }