Fix styling

Author: nikajorjika

config/inbox.php

@@ -1,4 +1,5 @@
 <?php
+
 return [
     'store' => [
         'driver' => env('INBOX_STORE_DRIVER', 'file'),

src/CaptureService.php

@@ -6,20 +6,18 @@
 
 class CaptureService
 {
-    public function __construct(protected MessageStore $storage)
-    {
-    }
+    public function __construct(protected MessageStore $storage) {}
 
     /**
      * Persist the raw message and metadata.
      */
     public function storeRaw(string $raw): string
     {
-        $key = 'email_' . md5($raw) . '_' . microtime(true);
+        $key = 'email_'.md5($raw).'_'.microtime(true);
 
         $this->storage->store($key, [
             'timestamp' => time(),
-            'raw'       => $raw,
+            'raw' => $raw,
         ]);
 
         return $key;

src/Contracts/MessageStore.php

@@ -9,24 +9,20 @@ interface MessageStore
      *
      * @param  string  $key  Unique key (you generate it).
      * @param  array  $value  Associative array payload; MUST include at least: ['raw' => string, 'timestamp' => int]
-     *
-     * @return void
      */
     public function store(string $key, array $value): void;
 
     /**
      * Retrieve a single payload by key.
      *
      * @param  string  $key  Unique key to retrieve the payload.
-     *
      * @return array|null Returns the payload array or null if not found.
      */
     public function retrieve(string $key): ?array;
 
     /**
      * Retrieve multiple payload keys optionally filtered by a UNIX timestamp (>= since).
      *
-     * @param  int|null  $since
      *
      * @return iterable<string> Keys
      */

src/InboxServiceProvider.php

@@ -21,13 +21,13 @@ public function configurePackage(Package $package): void
     public function registeringPackage(): void
     {
         $this->app->singleton(MessageStore::class, function () {
-            return (new StoreManager())->create();
+            return (new StoreManager)->create();
         });
         $this->app->singleton(CaptureService::class, function () {
             return new CaptureService(app(MessageStore::class));
         });
 
-        $this->app->singleton(InboxTransport::class, fn() => new InboxTransport(app(CaptureService::class)));
+        $this->app->singleton(InboxTransport::class, fn () => new InboxTransport(app(CaptureService::class)));
 
         // Register the mail driver
         $this->app->afterResolving(MailManager::class, function (MailManager $manager) {

src/Storage/FileStorage.php

@@ -11,7 +11,7 @@ class FileStorage implements MessageStore
     public function __construct(?string $basePath = null)
     {
         $this->basePath = $basePath ?: storage_path('app/mail-inbox');
-        if (!is_dir($this->basePath)) {
+        if (! is_dir($this->basePath)) {
             @mkdir($this->basePath, 0775, true);
         }
     }
@@ -27,7 +27,7 @@ public function store(string $key, array $value): void
     public function retrieve(string $key): ?array
     {
         $path = $this->pathFor($key);
-        if (!is_file($path)) {
+        if (! is_file($path)) {
             return null;
         }
 
@@ -39,7 +39,7 @@ public function retrieve(string $key): ?array
 
     public function keys(?int $since = null): iterable
     {
-        if (!is_dir($this->basePath)) {
+        if (! is_dir($this->basePath)) {
             return [];
         }
 
@@ -49,7 +49,7 @@ public function keys(?int $since = null): iterable
 
             if ($since) {
                 $payload = $this->retrieve($key);
-                if (!$payload || ($payload['timestamp'] ?? 0) < $since) {
+                if (! $payload || ($payload['timestamp'] ?? 0) < $since) {
                     continue;
                 }
             }
@@ -72,7 +72,7 @@ public function purgeOlderThan(int $seconds): void
 
         foreach ($this->keys() as $key) {
             $payload = $this->retrieve($key);
-            if (!$payload) {
+            if (! $payload) {
                 continue;
             }
             if (($payload['timestamp'] ?? 0) < $cut) {

src/Transport/InboxTransport.php

@@ -10,9 +10,7 @@
 
 class InboxTransport implements TransportInterface
 {
-    public function __construct(protected CaptureService $mailbox)
-    {
-    }
+    public function __construct(protected CaptureService $mailbox) {}
 
     public function __toString(): string
     {

tests/Feature/MailerRegistrationTest.php

@@ -16,17 +16,20 @@ public function build()
     $captured = (object) ['raw' => null];
 
     $this->app->singleton(CaptureService::class, function () use ($captured) {
-        return new class($captured) extends CaptureService {
+        return new class($captured) extends CaptureService
+        {
             public function __construct(private object $captured) {}
+
             public function storeRaw(string $raw): string
             {
                 $this->captured->raw = $raw;
+
                 return 'k';
             }
         };
     });
 
-    Mail::to('[email protected]')->send(new TestMailable());
+    Mail::to('[email protected]')->send(new TestMailable);
 
     expect($captured->raw)->not->toBeNull()
         ->and($captured->raw)->toContain('Hello');

tests/TestCase.php

@@ -13,7 +13,7 @@ protected function setUp(): void
         parent::setUp();
 
         Factory::guessFactoryNamesUsing(
-            fn(string $modelName
+            fn (string $modelName
             ) => 'Redberry\\MailboxForLaravel\\Database\\Factories\\'.class_basename($modelName).'Factory'
         );
     }
@@ -31,6 +31,6 @@ public function getEnvironmentSetUp($app)
 
         $app['config']->set('mail.mailers.inbox', ['transport' => 'inbox']);
         $app['config']->set('mail.default', 'inbox');
-        
+
     }
 }

tests/Unit/CaptureServiceTest.php

@@ -7,28 +7,47 @@ class InMemoryStore implements MessageStore
 {
     public array $data = [];
 
-    public function store(string $key, array $value): void { $this->data[$key] = $value; }
-    public function retrieve(string $key): ?array { return $this->data[$key] ?? null; }
-    public function keys(?int $since = null): iterable {
+    public function store(string $key, array $value): void
+    {
+        $this->data[$key] = $value;
+    }
+
+    public function retrieve(string $key): ?array
+    {
+        return $this->data[$key] ?? null;
+    }
+
+    public function keys(?int $since = null): iterable
+    {
         foreach (array_keys($this->data) as $k) {
             if ($since) {
                 $ts = $this->data[$k]['timestamp'] ?? 0;
-                if ($ts < $since) { continue; }
+                if ($ts < $since) {
+                    continue;
+                }
             }
             yield $k;
         }
     }
-    public function delete(string $key): void { unset($this->data[$key]); }
-    public function purgeOlderThan(int $seconds): void {
+
+    public function delete(string $key): void
+    {
+        unset($this->data[$key]);
+    }
+
+    public function purgeOlderThan(int $seconds): void
+    {
         $cut = time() - $seconds;
         foreach ($this->data as $k => $v) {
-            if (($v['timestamp'] ?? 0) < $cut) unset($this->data[$k]);
+            if (($v['timestamp'] ?? 0) < $cut) {
+                unset($this->data[$k]);
+            }
         }
     }
 }
 
 it('stores raw message with timestamp and returns a key', function () {
-    $store = new InMemoryStore();
+    $store = new InMemoryStore;
     $svc = new CaptureService($store);
 
     $key = $svc->storeRaw("Subject: Hi\r\n\r\nBody");

tests/Unit/FileStorageTest.php

@@ -3,15 +3,19 @@
 use Redberry\MailboxForLaravel\Storage\FileStorage;
 
 beforeEach(function () {
-    $this->basePath = sys_get_temp_dir() . '/inbox-fs-' . uniqid();
+    $this->basePath = sys_get_temp_dir().'/inbox-fs-'.uniqid();
     @mkdir($this->basePath, 0777, true);
     $this->fs = new FileStorage($this->basePath);
 });
 
 afterEach(function () {
     if (is_dir($this->basePath)) {
-        $files = glob($this->basePath . '/*');
-        if ($files) { foreach ($files as $f) { @unlink($f); } }
+        $files = glob($this->basePath.'/*');
+        if ($files) {
+            foreach ($files as $f) {
+                @unlink($f);
+            }
+        }
         @rmdir($this->basePath);
     }
 });