Apply fixed Coderrabitai review.

Author: terabytesoftw

composer.lock

@@ -91,9 +91,4 @@
             OrderedTraitsFixer::class,
             SingleQuoteFixer::class,
         ]
-    )
-    ->withSkip(
-        [
-            NullableTypeDeclarationFixer::class,
-        ]
     );

ecs.php

@@ -166,7 +166,7 @@ private function getByManager(string $key, mixed $value, mixed $default = null):
      */
     private function getDefaultValue(string $key, mixed $default = null): mixed
     {
-        $value = null === $default &&  array_key_exists($key, $this->defaults)
+        $value = null === $default && array_key_exists($key, $this->defaults)
             ? $this->defaults[$key]
             : $default;
 

src/Config/Config.php

@@ -4,8 +4,6 @@
 
 namespace Foxy;
 
-use Event;
-
 abstract class FoxyEvents
 {
     /**

src/FoxyEvents.php

@@ -13,10 +13,10 @@
 use Foxy\Asset\AssetManagerInterface;
 use Foxy\Config\Config;
 use Foxy\Event\{GetAssetsEvent, PostSolveEvent, PreSolveEvent};
+use Foxy\Exception\RuntimeException;
 use Foxy\Fallback\FallbackInterface;
 use Foxy\FoxyEvents;
 use Foxy\Util\AssetUtil;
-use RuntimeException;
 
 use function basename;
 use function copy;

src/Solver/Solver.php

@@ -76,7 +76,7 @@ public function testRestoreThrowsWhenRemoveFails(): void
             ->expects(self::once())
             ->method('remove')
             ->with('package.json')
-            ->willThrowException(new \RuntimeException('Remove failed.'));
+            ->willThrowException(new RuntimeException('Remove failed.'));
 
         $this->assetFallback->save();
 

tests/Fallback/AssetFallbackTest.php

@@ -78,7 +78,7 @@ public function testRestore(array $packages): void
         $this->input
             ->expects(self::any())
             ->method('getOption')
-            ->willReturnCallback(fn($option): ?bool => 'verbose' === $option ? false : null);
+            ->willReturnCallback(fn($option): bool|null => 'verbose' === $option ? false : null);
 
         $ed = $this->createMock(EventDispatcher::class);
 

tests/Fallback/ComposerFallbackTest.php

@@ -48,7 +48,7 @@ public function testConvertLockAlias(): void
         $convertedAliases = PackageUtil::convertLockAlias($lockData);
 
         self::assertArrayHasKey('aliases', $convertedAliases);
-        self::assertEquals($convertedAliases['aliases'], $expectedAliases);
+        self::assertEquals($expectedAliases, $convertedAliases['aliases']);
     }
 
     public function testLoadLockPackages(): void
@@ -75,8 +75,8 @@ public function testLoadLockPackages(): void
 
         self::assertArrayHasKey('packages', $lockDataLoaded);
         self::assertArrayHasKey('packages-dev', $lockDataLoaded);
-        self::assertEquals($lockDataLoaded['packages'], $expectedPackages);
-        self::assertEquals($lockDataLoaded['packages-dev'], $expectedDevPackages);
+        self::assertEquals($expectedPackages, $lockDataLoaded['packages']);
+        self::assertEquals($expectedDevPackages, $lockDataLoaded['packages-dev']);
     }
 
     public function testLoadLockPackagesWithoutPackages(): void