Fix styling

Author: freekmurze

src/Drivers/HtmlDriver.php

@@ -15,7 +15,7 @@ public function serialize($data): string
             throw new CantBeSerialized('Only strings can be serialized to html');
         }
 
-        if ('' === $data) {
+        if ($data === '') {
             return "\n";
         }
 

src/Drivers/ImageDriver.php

@@ -13,8 +13,7 @@ class ImageDriver implements Driver
     public function __construct(
         protected float $threshold = 0.1,
         protected bool $includeAa = true,
-    )
-    {
+    ) {
     }
 
     public function serialize($data): string
@@ -31,10 +30,10 @@ public function match($expected, $actual)
     {
         $tempPath = sys_get_temp_dir();
 
-        $expectedTempPath = $tempPath . '/expected.png';
+        $expectedTempPath = $tempPath.'/expected.png';
         file_put_contents($expectedTempPath, $expected);
 
-        $actualTempPath = $tempPath . '/actual.png';
+        $actualTempPath = $tempPath.'/actual.png';
         file_put_contents($actualTempPath, $actual);
 
         $pixelMatch = Pixelmatch::new($expectedTempPath, $actualTempPath)
@@ -44,8 +43,7 @@ public function match($expected, $actual)
         try {
             $result = $pixelMatch->matches();
         } catch (CouldNotCompare $exception) {
-            throw new
-            ExpectationFailedException($exception->getMessage());
+            throw new ExpectationFailedException($exception->getMessage());
         }
 
         Assert::assertTrue($result);

src/MatchesSnapshots.php

@@ -51,7 +51,7 @@ public function markTestIncompleteIfSnapshotsHaveChanged()
 
     public function assertMatchesSnapshot($actual, Driver $driver = null): void
     {
-        if (!is_null($driver)) {
+        if (! is_null($driver)) {
             $this->doSnapshotAssertion($actual, $driver);
 
             return;
@@ -68,7 +68,7 @@ public function assertMatchesSnapshot($actual, Driver $driver = null): void
 
     public function assertMatchesFileHashSnapshot(string $filePath): void
     {
-        if (!file_exists($filePath)) {
+        if (! file_exists($filePath)) {
             $this->fail('File does not exist');
         }
 
@@ -116,8 +116,7 @@ public function assertMatchesImageSnapshot(
         $actual,
         float $threshold = 0.1,
         bool $includeAa = true
-    ): void
-    {
+    ): void {
         $this->assertMatchesSnapshot($actual, new ImageDriver(
             $threshold,
             $includeAa,
@@ -131,8 +130,8 @@ public function assertMatchesImageSnapshot(
      */
     protected function getFileSnapshotDirectory(): string
     {
-        return $this->getSnapshotDirectory() .
-            DIRECTORY_SEPARATOR .
+        return $this->getSnapshotDirectory().
+            DIRECTORY_SEPARATOR.
             'files';
     }
 
@@ -161,7 +160,7 @@ protected function shouldUpdateSnapshots(): bool
      */
     protected function shouldCreateSnapshots(): bool
     {
-        return !in_array('--without-creating-snapshots', $_SERVER['argv'], true)
+        return ! in_array('--without-creating-snapshots', $_SERVER['argv'], true)
             && getenv('CREATE_SNAPSHOTS') !== 'false';
     }
 
@@ -175,7 +174,7 @@ protected function doSnapshotAssertion(mixed $actual, Driver $driver)
             $driver
         );
 
-        if (!$snapshot->exists()) {
+        if (! $snapshot->exists()) {
             $this->assertSnapshotShouldBeCreated($snapshot->filename());
 
             $this->createSnapshotAndMarkTestIncomplete($snapshot, $actual);
@@ -203,7 +202,7 @@ protected function doSnapshotAssertion(mixed $actual, Driver $driver)
 
     protected function doFileSnapshotAssertion(string $filePath): void
     {
-        if (!file_exists($filePath)) {
+        if (! file_exists($filePath)) {
             $this->fail('File does not exist');
         }
 
@@ -217,7 +216,7 @@ protected function doFileSnapshotAssertion(string $filePath): void
 
         $this->snapshotIncrementor++;
 
-        $snapshotId = $this->getSnapshotId() . '.' . $fileExtension;
+        $snapshotId = $this->getSnapshotId().'.'.$fileExtension;
         $snapshotId = Filename::cleanFilename($snapshotId);
 
         // If $filePath has a different file extension than the snapshot, the test should fail
@@ -240,13 +239,13 @@ protected function doFileSnapshotAssertion(string $filePath): void
             $this->fail("File did not match the snapshot file extension (expected: {$expectedExtension}, was: {$fileExtension})");
         }
 
-        $failedSnapshotId = $snapshotId . '_failed.' . $fileExtension;
+        $failedSnapshotId = $snapshotId.'_failed.'.$fileExtension;
 
         if ($fileSystem->has($failedSnapshotId)) {
             $fileSystem->delete($failedSnapshotId);
         }
 
-        if (!$fileSystem->has($snapshotId)) {
+        if (! $fileSystem->has($snapshotId)) {
             $this->assertSnapshotShouldBeCreated($failedSnapshotId);
 
             $fileSystem->copy($filePath, $snapshotId);
@@ -256,7 +255,7 @@ protected function doFileSnapshotAssertion(string $filePath): void
             return;
         }
 
-        if (!$fileSystem->fileEquals($filePath, $snapshotId)) {
+        if (! $fileSystem->fileEquals($filePath, $snapshotId)) {
             if ($this->shouldUpdateSnapshots()) {
                 $fileSystem->copy($filePath, $snapshotId);
 
@@ -289,8 +288,8 @@ protected function updateSnapshotAndMarkTestIncomplete(Snapshot $snapshot, $actu
 
     protected function rethrowExpectationFailedExceptionWithUpdateSnapshotsPrompt($exception): void
     {
-        $newMessage = $exception->getMessage() . "\n\n" .
-            'Snapshots can be updated by passing ' .
+        $newMessage = $exception->getMessage()."\n\n".
+            'Snapshots can be updated by passing '.
             '`-d --update-snapshots` through PHPUnit\'s CLI arguments.';
 
         $exceptionReflection = new ReflectionObject($exception);
@@ -314,9 +313,9 @@ protected function assertSnapshotShouldBeCreated(string $snapshotFileName): void
         }
 
         $this->fail(
-            "Snapshot \"$snapshotFileName\" does not exist.\n" .
-            'You can automatically create it by removing ' .
-            'the `CREATE_SNAPSHOTS=false` env var, or ' .
+            "Snapshot \"$snapshotFileName\" does not exist.\n".
+            'You can automatically create it by removing '.
+            'the `CREATE_SNAPSHOTS=false` env var, or '.
             '`-d --without-creating-snapshots` of PHPUnit\'s CLI arguments.'
         );
     }

tests/Unit/Drivers/ImageDriverTest.php

@@ -14,9 +14,9 @@ public function setUp(): void
 
         $this->driver = new ImageDriver();
 
-        $this->pathToImageA = __DIR__ . '/../test_files/testA.png';
-        $this->pathToImageB = __DIR__ . '/../test_files/testB.png';
-        $this->pathToImageWithDifferentDimensions = __DIR__ . '/../test_files/testC.png';
+        $this->pathToImageA = __DIR__.'/../test_files/testA.png';
+        $this->pathToImageB = __DIR__.'/../test_files/testB.png';
+        $this->pathToImageWithDifferentDimensions = __DIR__.'/../test_files/testC.png';
 
     }