Add new architecture test for Flutterwave payment handler

damms005 · Oct 25, 2024 · 210a98a · 210a98a
1 parent 75f2b3f
commit 210a98a
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 2 deletions.
diff --git a/src/Services/PaymentHandlers/Flutterwave.php b/src/Services/PaymentHandlers/Flutterwave.php
@@ -81,8 +81,6 @@ protected function sendUserToPaymentGateway(string $redirect_or_callback_url, Pa
         $payment->save();
 
         header('Location: ' . $url);
-
-        exit;
     }
 
     public function confirmResponseCanBeHandledAndUpdateDatabaseWithTransactionOutcome(Request $paymentGatewayServerResponse): ?Payment

diff --git a/tests/ArchitectureTest.php b/tests/ArchitectureTest.php
@@ -0,0 +1,11 @@
+<?php
+
+describe('architecture tests', function () {
+    arch()
+        ->expect('Damms005\LaravelMultipay')
+        ->not->toUse(['die', 'dd', 'dump']);
+
+    arch()->preset()->php();
+    arch()->preset()->laravel()->ignoring('Damms005\LaravelMultipay\LaravelMultipayServiceProvider');
+    arch()->preset()->security()->ignoring('md5');
+});
-Original file line number
+Diff line change
@@ Expand Up @@
             $payment->save();
             header('Location: ' . $url);
-            exit;
         }
         public function confirmResponseCanBeHandledAndUpdateDatabaseWithTransactionOutcome(Request $paymentGatewayServerResponse): ?Payment
@@ Expand Down @@