Chore: Add check executable, that will run all style checks and tests

Author: nikajorjika

bin/check

@@ -0,0 +1,12 @@
+#!/usr/bin/env bash
+
+set -e
+
+echo "Running Pint..."
+./vendor/bin/pint
+
+echo "Running PHPStan..."
+./vendor/bin/phpstan analyse --memory-limit=1G
+
+echo "Running Pest tests..."
+./vendor/bin/pest

composer.json

@@ -69,6 +69,9 @@
             ]
         }
     },
+    "bin": [
+        "bin/check"
+    ],
     "minimum-stability": "dev",
     "prefer-stable": true
 }

tests/Feature/ClearInboxControllerTest.php

@@ -11,61 +11,61 @@
             Route::post('/mailbox/clear', ClearInboxController::class)->name('inbox.clear');
         });
         config()->set('inbox.public', true);
-        
+
         // Clear any existing messages before each test
         $service = app(CaptureService::class);
         $service->clearAll();
     });
 
     it('empties store and returns success response', function () {
         $service = app(CaptureService::class);
-        
+
         // Store some messages first
         $payload1 = [
             'subject' => 'Test Email 1',
             'from' => [['email' => '[email protected]']],
             'raw' => 'Email 1 content',
         ];
         $payload2 = [
-            'subject' => 'Test Email 2', 
+            'subject' => 'Test Email 2',
             'from' => [['email' => '[email protected]']],
             'raw' => 'Email 2 content',
         ];
-        
+
         $service->store($payload1);
         $service->store($payload2);
-        
+
         // Verify messages exist
         $messagesBefore = $service->all();
         expect($messagesBefore)->toHaveCount(2);
-        
+
         // Clear the inbox
         $response = $this->post('/mailbox/clear');
-        
+
         $response->assertOk()
             ->assertJson([]);
-        
+
         // Verify messages are cleared
         $messagesAfter = $service->all();
         expect($messagesAfter)->toBeEmpty();
     });
 
     it('returns json response even when inbox is already empty', function () {
         $service = app(CaptureService::class);
-        
+
         // Ensure inbox is empty
         $messagesBefore = $service->all();
         expect($messagesBefore)->toBeEmpty();
-        
+
         $response = $this->post('/mailbox/clear');
-        
+
         $response->assertOk()
             ->assertJson([]);
     });
 
     it('handles large number of messages efficiently', function () {
         $service = app(CaptureService::class);
-        
+
         // Store multiple messages
         for ($i = 1; $i <= 10; $i++) {
             $payload = [
@@ -75,48 +75,48 @@
             ];
             $service->store($payload);
         }
-        
+
         // Verify messages exist
         $messagesBefore = $service->all();
         expect($messagesBefore)->toHaveCount(10);
-        
+
         // Clear all messages
         $response = $this->post('/mailbox/clear');
-        
+
         $response->assertOk();
-        
+
         // Verify all messages are cleared
         $messagesAfter = $service->all();
         expect($messagesAfter)->toBeEmpty();
     });
 
     it('calls CaptureService clearAll method correctly', function () {
         $service = app(CaptureService::class);
-        
+
         // Store a message to verify clearAll actually gets called
         $payload = [
             'subject' => 'Test Email',
             'from' => [['email' => '[email protected]']],
             'raw' => 'Test email content',
         ];
-        
+
         $key = $service->store($payload);
-        
+
         // Verify message exists
         expect($service->retrieve($key))->not->toBeNull();
-        
+
         $response = $this->post('/mailbox/clear');
         $response->assertOk();
-        
+
         // Verify clearAll was effective
         expect($service->retrieve($key))->toBeNull();
         expect($service->all())->toBeEmpty();
     });
 
     it('returns proper content-type header for json response', function () {
         $response = $this->post('/mailbox/clear');
-        
+
         $response->assertOk()
             ->assertHeader('content-type', 'application/json');
     });
-});
+});

tests/Feature/InboxControllerTest.php

@@ -1,21 +1,20 @@
 <?php
 
-use Illuminate\Support\Facades\View;
 use Redberry\MailboxForLaravel\CaptureService;
 use Redberry\MailboxForLaravel\Http\Controllers\InboxController;
 
 describe(InboxController::class, function () {
     beforeEach(function () {
         config()->set('inbox.public', true);
-        
+
         // Clear any existing messages before each test
         $service = app(CaptureService::class);
         $service->clearAll();
     });
 
     it('returns view with paginated list of messages sorted newest-first', function () {
         $service = app(CaptureService::class);
-        
+
         // Store messages with different timestamps to test sorting
         $payload1 = [
             'subject' => 'First Email',
@@ -24,23 +23,23 @@
             'raw' => 'Email 1',
         ];
         $payload2 = [
-            'subject' => 'Second Email', 
+            'subject' => 'Second Email',
             'timestamp' => 2000,
             'from' => [['email' => '[email protected]']],
             'raw' => 'Email 2',
         ];
-        
+
         $key1 = $service->store($payload1);
         $key2 = $service->store($payload2);
-        
-        $controller = new InboxController();
+
+        $controller = new InboxController;
         $request = request();
-        
+
         $result = $controller->__invoke($request, $service);
-        
+
         expect($result)->toBeInstanceOf(\Illuminate\Contracts\View\View::class);
         expect($result->name())->toBe('inbox::index');
-        
+
         $data = $result->getData();
         expect($data)->toHaveKey('data');
         expect($data['data'])->toHaveKey('messages');
@@ -50,14 +49,14 @@
 
     it('handles empty message list', function () {
         $service = app(CaptureService::class);
-        $controller = new InboxController();
+        $controller = new InboxController;
         $request = request();
-        
+
         $result = $controller->__invoke($request, $service);
-        
+
         expect($result)->toBeInstanceOf(\Illuminate\Contracts\View\View::class);
         expect($result->name())->toBe('inbox::index');
-        
+
         $data = $result->getData();
         expect($data)->toHaveKey('data');
         expect($data['data'])->toHaveKey('messages');
@@ -67,20 +66,20 @@
 
     it('normalizes message data structure for both paginated and non-paginated responses', function () {
         $service = app(CaptureService::class);
-        
+
         $payload = [
             'subject' => 'Test Email',
             'from' => [['email' => '[email protected]']],
             'raw' => 'Test email content',
         ];
-        
+
         $service->store($payload);
-        
-        $controller = new InboxController();
+
+        $controller = new InboxController;
         $request = request();
-        
+
         $result = $controller->__invoke($request, $service);
-        
+
         $data = $result->getData();
         // Should handle both paginated (with 'data' key) and non-paginated message arrays
         expect($data['data']['messages'])->toBeArray();
@@ -89,7 +88,7 @@
 
     it('uses CaptureService to retrieve all messages', function () {
         $service = app(CaptureService::class);
-        
+
         // Store multiple messages to verify the service is called correctly
         for ($i = 1; $i <= 3; $i++) {
             $payload = [
@@ -99,38 +98,38 @@
             ];
             $service->store($payload);
         }
-        
-        $controller = new InboxController();
+
+        $controller = new InboxController;
         $request = request();
-        
+
         $result = $controller->__invoke($request, $service);
-        
+
         $data = $result->getData();
         expect($data['data']['messages'])->toHaveCount(3);
     });
 
     it('properly formats data structure for Vue.js consumption', function () {
         $service = app(CaptureService::class);
-        
+
         $payload = [
             'subject' => 'Test Email',
             'from' => [['email' => '[email protected]', 'name' => 'Test User']],
             'to' => [['email' => '[email protected]']],
             'html' => '<p>Test content</p>',
             'raw' => 'Test email content',
         ];
-        
+
         $service->store($payload);
-        
-        $controller = new InboxController();
+
+        $controller = new InboxController;
         $request = request();
-        
+
         $result = $controller->__invoke($request, $service);
-        
+
         $data = $result->getData();
         expect($data)->toHaveKey('data');
         expect($data['data'])->toHaveKey('messages');
-        
+
         $message = $data['data']['messages'][0];
         expect($message)->toHaveKey('subject', 'Test Email');
         expect($message)->toHaveKey('from');
@@ -139,4 +138,4 @@
         expect($message)->toHaveKey('id');
         expect($message)->toHaveKey('timestamp');
     });
-});
+});