feat: improve missing adapter exception

Author: innocenzi

src/Tempest/Storage/src/MissingAdapterException.php

@@ -3,13 +3,40 @@
 namespace Tempest\Storage;
 
 use Exception;
+use League\Flysystem\AwsS3V3\AwsS3V3Adapter;
+use League\Flysystem\AzureBlobStorage\AzureBlobStorageAdapter;
+use League\Flysystem\Ftp\FtpAdapter;
+use League\Flysystem\GoogleCloudStorage\GoogleCloudStorageAdapter;
+use League\Flysystem\InMemory\InMemoryFilesystemAdapter;
+use League\Flysystem\PhpseclibV3\SftpAdapter;
+use League\Flysystem\ReadOnly\ReadOnlyFilesystemAdapter;
+use League\Flysystem\ZipArchive\ZipArchiveAdapter;
 
 final class MissingAdapterException extends Exception implements StorageException
 {
-    public function __construct(string $missing)
+    public function __construct(
+        private readonly string $missing,
+    ) {
+        $packageName = $this->getPackageName();
+        $message = $packageName
+            ? sprintf('The `%s` adapter is missing. Install it using `composer require %s`.', $missing, $packageName)
+            : sprintf('The `%s` adapter is missing.', $missing);
+
+        parent::__construct($message);
+    }
+
+    private function getPackageName(): ?string
     {
-        parent::__construct(
-            message: sprintf('The `%s` adapter is missing.', $missing),
-        );
+        return match ($this->missing) {
+            AwsS3V3Adapter::class => 'league/flysystem-aws-s3-v3',
+            InMemoryFilesystemAdapter::class => 'league/flysystem-memory',
+            ReadOnlyFilesystemAdapter::class => 'league/flysystem-read-only',
+            SftpAdapter::class => 'league/flysystem-sftp',
+            ZipArchiveAdapter::class => 'league/flysystem-ziparchive',
+            AzureBlobStorageAdapter::class => 'league/flysystem-azure-blob-storage',
+            FtpAdapter::class => 'league/flysystem-ftp',
+            GoogleCloudStorageAdapter::class => 'league/flysystem-google-cloud-storage',
+            default => null,
+        };
     }
 }

tests/Integration/Storage/StorageTesterTest.php

@@ -4,7 +4,11 @@
 
 use DateTime;
 use DateTimeInterface;
+use League\Flysystem\FilesystemAdapter;
+use League\Flysystem\Local\LocalFilesystemAdapter;
+use Tempest\Storage\Config\StorageConfig;
 use Tempest\Storage\ForbiddenStorageUsageException;
+use Tempest\Storage\MissingAdapterException;
 use Tempest\Storage\Storage;
 use Tests\Tempest\Integration\FrameworkIntegrationTestCase;
 
@@ -101,4 +105,23 @@ public function test_prevent_usage_without_fake_with_fake(): void
 
         $this->storage->assertFileExists('bar.txt');
     }
+
+    public function test_no_adapter(): void
+    {
+        $this->expectException(MissingAdapterException::class);
+        $this->expectExceptionMessage('The `UnknownClass` adapter is missing');
+
+        $this->container->config(new class implements StorageConfig {
+            public string $adapter = 'UnknownClass';
+            public bool $readonly = false;
+
+            public function createAdapter(): FilesystemAdapter
+            {
+                return new LocalFilesystemAdapter(__DIR__);
+            }
+        });
+
+        $storage = $this->container->get(Storage::class);
+        $storage->write('bar.txt', 'baz');
+    }
 }