add more tests

ildyria · ildyria · commit f55446cc680e · 2025-11-16T00:11:52.000+01:00
diff --git a/database/factories/OrderFactory.php b/database/factories/OrderFactory.php
@@ -189,7 +189,6 @@ public function closed(): static
 			]);
 	}
 
-
 	/**
 	 * Create a failed order.
 	 *
diff --git a/phpunit.ci.xml b/phpunit.ci.xml
@@ -25,7 +25,7 @@
     </testsuite>
     <testsuite name="Webshop">
       <directory suffix="Test.php">./tests/Webshop</directory>
-      <exclude>./tests/Webshop/BaseCheckoutControllerTest.php</exclude>
+      <exclude>./tests/Webshop/Checkout/BaseCheckoutControllerTest.php</exclude>
     </testsuite>
   </testsuites>
   <extensions>
diff --git a/phpunit.pgsql.xml b/phpunit.pgsql.xml
@@ -25,7 +25,7 @@
     </testsuite>
     <testsuite name="Webshop">
       <directory suffix="Test.php">./tests/Webshop</directory>
-      <exclude>./tests/Webshop/BaseCheckoutControllerTest.php</exclude>
+      <exclude>./tests/Webshop/Checkout/BaseCheckoutControllerTest.php</exclude>
     </testsuite>
   </testsuites>
   <extensions>
diff --git a/phpunit.xml b/phpunit.xml
@@ -25,7 +25,7 @@
     </testsuite>
     <testsuite name="Webshop">
       <directory suffix="Test.php">./tests/Webshop</directory>
-      <exclude>./tests/Webshop/BaseCheckoutControllerTest.php</exclude>
+      <exclude>./tests/Webshop/Checkout/BaseCheckoutControllerTest.php</exclude>
     </testsuite>
   </testsuites>
   <extensions>
diff --git a/tests/Webshop/Checkout/BaseCheckoutControllerTest.php b/tests/Webshop/Checkout/BaseCheckoutControllerTest.php
@@ -16,7 +16,7 @@
  * @noinspection PhpUnhandledExceptionInspection
  */
 
-namespace Tests\Webshop;
+namespace Tests\Webshop\Checkout;
 
 use App\Contracts\Http\Requests\RequestAttribute;
 use App\Models\Order;
diff --git a/tests/Webshop/Checkout/CheckoutControllerTest.php b/tests/Webshop/Checkout/CheckoutControllerTest.php
@@ -16,7 +16,7 @@
  * @noinspection PhpUnhandledExceptionInspection
  */
 
-namespace Tests\Webshop;
+namespace Tests\Webshop\Checkout;
 
 use Tests\Feature_v2\Base\BaseApiWithDataTest;
 use Tests\Traits\RequireSE;
diff --git a/tests/Webshop/Checkout/CheckoutCreateSessionControllerTest.php b/tests/Webshop/Checkout/CheckoutCreateSessionControllerTest.php
@@ -16,7 +16,7 @@
  * @noinspection PhpUnhandledExceptionInspection
  */
 
-namespace Tests\Webshop;
+namespace Tests\Webshop\Checkout;
 
 use App\Contracts\Http\Requests\RequestAttribute;
 use App\Enum\OmnipayProviderType;
diff --git a/tests/Webshop/Checkout/CheckoutFinalizeOrCancelControllerTest.php b/tests/Webshop/Checkout/CheckoutFinalizeOrCancelControllerTest.php
@@ -16,10 +16,11 @@
  * @noinspection PhpUnhandledExceptionInspection
  */
 
-namespace Tests\Webshop;
+namespace Tests\Webshop\Checkout;
 
 use App\Enum\OmnipayProviderType;
 use App\Enum\PaymentStatusType;
+use Illuminate\Support\Facades\Session;
 use Illuminate\Support\Str;
 
 /**
@@ -52,7 +53,7 @@ public function testFinalizePaymentSuccess(): void
 
 		$this->assertDatabaseHas('orders', ['transaction_id' => $transaction_id, 'status' => PaymentStatusType::PROCESSING->value]);
 
-		$response = $this->getJsonWithData('Shop/Checkout/Finalize/' . $provider . '/' . $transaction_id, [
+		Session::put('processing', [
 			'payment_id' => 'dummy-payment-123',
 			'status' => 'completed',
 			'transactionReference' => $this->test_order->transaction_id,
@@ -64,6 +65,8 @@ public function testFinalizePaymentSuccess(): void
 			],
 		]);
 
+		$response = $this->getJsonWithData('Shop/Checkout/Finalize/' . $provider . '/' . $transaction_id);
+
 		$this->assertOk($response);
 		$response->assertJsonStructure([
 			'is_success',
@@ -224,7 +227,7 @@ public function testTransactionIdGeneration(): void
 		$provider = OmnipayProviderType::DUMMY->value;
 		$transaction_id = $this->test_order->transaction_id;
 
-		$response = $this->getJsonWithData('Shop/Checkout/Finalize/' . $provider . '/' . $transaction_id, [
+		Session::put('processing', [
 			'payment_id' => 'dummy-payment-123',
 			'status' => 'completed',
 			'transactionReference' => $this->test_order->transaction_id,
@@ -236,6 +239,8 @@ public function testTransactionIdGeneration(): void
 			],
 		]);
 
+		$response = $this->getJsonWithData('Shop/Checkout/Finalize/' . $provider . '/' . $transaction_id);
+
 		$this->assertOk($response);
 		$response->assertJson([
 			'is_success' => true,
diff --git a/tests/Webshop/Checkout/CheckoutOfflineControllerTest.php b/tests/Webshop/Checkout/CheckoutOfflineControllerTest.php
@@ -16,7 +16,7 @@
  * @noinspection PhpUnhandledExceptionInspection
  */
 
-namespace Tests\Webshop;
+namespace Tests\Webshop\Checkout;
 
 use App\Contracts\Http\Requests\RequestAttribute;
 use App\Enum\PaymentStatusType;
diff --git a/tests/Webshop/Checkout/CheckoutProcessPaymentControllerTest.php b/tests/Webshop/Checkout/CheckoutProcessPaymentControllerTest.php
@@ -16,7 +16,7 @@
  * @noinspection PhpUnhandledExceptionInspection
  */
 
-namespace Tests\Webshop;
+namespace Tests\Webshop\Checkout;
 
 use App\Enum\OmnipayProviderType;
 use App\Enum\PaymentStatusType;
diff --git a/tests/Webshop/OrderManagement/FlushOldOrdersTest.php b/tests/Webshop/OrderManagement/FlushOldOrdersTest.php
diff --git a/tests/Webshop/OrderManagement/OrderControllerMarkAsDeliveredTest.php b/tests/Webshop/OrderManagement/OrderControllerMarkAsDeliveredTest.php
diff --git a/tests/Webshop/OrderManagement/OrderControllerMarkAsPaidTest.php b/tests/Webshop/OrderManagement/OrderControllerMarkAsPaidTest.php
diff --git a/tests/Webshop/OrderManagement/OrderListingAccessTest.php b/tests/Webshop/OrderManagement/OrderListingAccessTest.php
diff --git a/tests/Webshop/Purchasables/ShopManagementControllerTest.php b/tests/Webshop/Purchasables/ShopManagementControllerTest.php

Original file line number	Diff line number	Diff line change
`@@ -189,7 +189,6 @@ public function closed(): static`
`189`	`189`	`]);`
`190`	`190`	`}`
`191`	`191`
`192`		`-`
`193`	`192`	`/**`
`194`	`193`	`* Create a failed order.`
`195`	`194`	`*`