rename webhook route

Author: danilopolani

routes/api.php

@@ -2,8 +2,10 @@
 
 use Illuminate\Support\Facades\Route;
 use MailCarrier\Http\Controllers\MailCarrierController;
+use MailCarrier\Http\Controllers\WebhookController;
 use MailCarrier\Http\Middleware\ForceJsonRequest;
 
 Route::prefix('api')->middleware(ForceJsonRequest::class)->group(function () {
     Route::post('send', [MailCarrierController::class, 'send'])->name('mailcarrier.send');
+    Route::post('webhook', WebhookController::class)->name('mailcarrier.webhook.process');
 });

routes/web.php

@@ -5,7 +5,6 @@
 use MailCarrier\Http\Controllers\LogController;
 use MailCarrier\Http\Controllers\MailCarrierController;
 use MailCarrier\Http\Controllers\SocialAuthController;
-use MailCarrier\Http\Controllers\WebhookController;
 use MailCarrier\Livewire\PreviewTemplate;
 
 Route::middleware(['web', 'auth:' . Config::get('filament.auth.guard')])->group(function () {
@@ -23,5 +22,3 @@
 Route::middleware('web')->group(function () {
     Route::get('templates/preview', PreviewTemplate::class)->name('templates.preview');
 });
-
-Route::post('webhook', WebhookController::class)->name('webhook.process');

tests/Feature/Webhooks/ProcessTest.php

@@ -31,15 +31,15 @@
                 && $webhook->body === ['test' => 'data'];
         });
 
-    postJson(route('webhook.process'), ['test' => 'data'], ['x-custom-header' => 'custom-value'])
+    postJson(route('mailcarrier.webhook.process'), ['test' => 'data'], ['x-custom-header' => 'custom-value'])
         ->assertOk();
 });
 
 it('processes a valid webhook successfully', function () {
     $log = LogModel::factory()->create(['message_id' => 'test-message-id']);
     expect($log->events()->count())->toBe(0);
 
-    postJson(route('webhook.process'), [], ['header1' => 'value1'])
+    postJson(route('mailcarrier.webhook.process'), [], ['header1' => 'value1'])
         ->assertOk();
 
     expect($log->events()->first())
@@ -57,7 +57,7 @@
     $log = LogModel::factory()->create(['message_id' => 'test-message-id']);
     expect($log->events()->count())->toBe(0);
 
-    postJson(route('webhook.process'), [], ['header1' => 'value1'])
+    postJson(route('mailcarrier.webhook.process'), [], ['header1' => 'value1'])
         ->assertOk();
 
     expect($log->events()->first())
@@ -75,7 +75,7 @@
     $log = LogModel::factory()->create(['message_id' => 'test-message-id']);
     expect($log->events()->count())->toBe(0);
 
-    postJson(route('webhook.process'), [], ['header1' => 'value1'])
+    postJson(route('mailcarrier.webhook.process'), [], ['header1' => 'value1'])
         ->assertOk();
 
     expect($log->events()->first())
@@ -94,7 +94,7 @@
     $log = LogModel::factory()->create(['message_id' => 'test-message-id']);
     expect($log->events()->count())->toBe(0);
 
-    postJson(route('webhook.process'), [], ['header1' => 'value1'])
+    postJson(route('mailcarrier.webhook.process'), [], ['header1' => 'value1'])
         ->assertUnprocessable()
         ->assertJson(['message' => 'Webhook validation failed.']);
 
@@ -110,7 +110,7 @@
     $log = LogModel::factory()->create(['message_id' => 'test-message-id']);
     expect($log->events()->count())->toBe(0);
 
-    postJson(route('webhook.process'), [], ['header1' => 'value1'])
+    postJson(route('mailcarrier.webhook.process'), [], ['header1' => 'value1'])
         ->assertOk();
 
     expect($log->events()->count())->toBe(0);
@@ -120,7 +120,7 @@
 it('does nothing when log is not found', function () {
     expect(LogEvent::count())->toBe(0);
 
-    postJson(route('webhook.process'), [], ['header1' => 'value1'])
+    postJson(route('mailcarrier.webhook.process'), [], ['header1' => 'value1'])
         ->assertOk();
 
     expect(LogEvent::count())->toBe(0);