test(stock): cover authoritative reservation expiration

tests/Feature/Cart/CartControllerTest.php

@@ -47,6 +47,7 @@ class CartControllerTest extends TestCase
         ]));
         $this->assertTrue(Schema::hasColumn('carritos', 'current_stock_reservation_id'));
         $this->assertTrue(Schema::hasColumn('compras', 'stock_reservation_id'));
+        $this->assertFalse(Schema::hasColumn('compras', 'expires_at'));
     }
 
     public function test_it_aggregates_shared_inventory_into_one_cart_reservation_line(): void

tests/Unit/Catalog/ExpireStockReservationsServiceTest.php

@@ -14,7 +14,9 @@ use Tests\TestCase;
 
 class ExpireStockReservationsServiceTest extends TestCase
 {
-    public function test_it_expires_purchases_before_abandoned_cart_reservations(): void
+    use RefreshDatabase;
+
+    public function test_it_expires_an_orphan_reservation_and_releases_its_inventory(): void
     {
         $inventory = Inventory::query()->create([
             'real_stock' => 10,
@@ -31,31 +33,13 @@ class ExpireStockReservationsServiceTest extends TestCase
             'tracks_inventory' => true,
         ]);
 
-        $carts = \Mockery::mock(ExpireCartReservationsService::class);
+        $result = app(ExpireStockReservationsService::class)->expireOverdue();
-        $carts->shouldReceive('expireOverdue')
-            ->once()
-            ->ordered()
-            ->andReturn(3);
-
-        $logger = \Mockery::mock(LoggerInterface::class);
-        Log::shouldReceive('channel')
-            ->once()
-            ->with('commands')
-            ->andReturn($logger);
-        $logger->shouldReceive('info')
-            ->once()
-            ->with('Stock reservation cleanup completed.', [
-                'command' => 'reservations:expire',
-                'expired_purchases' => 2,
-                'expired_cart_reservations' => 3,
-                'total_expired' => 5,
-            ]);
-
-        $result = (new ExpireStockReservationsService($checkout, $carts))->expireOverdue();
 
         $this->assertSame([
-            'purchases' => 2,
+            'purchases' => 0,
-            'cart_reservations' => 3,
+            'cart_reservations' => 0,
+            'orphan_reservations' => 1,
+            'failed' => 0,
         ], $result);
         $this->assertDatabaseHas('stock_reservations', [
             'id' => $reservation->id,
@@ -87,19 +71,12 @@ class ExpireStockReservationsServiceTest extends TestCase
                     throw new RuntimeException('Broken reservation.');
                 }
 
-        $logger = \Mockery::mock(LoggerInterface::class);
+                $reservation->update([
-        Log::shouldReceive('channel')
+                    'status' => StockReservation::STATUS_EXPIRED,
-            ->once()
+                    'expires_at' => null,
-            ->with('commands')
+                    'expired_at' => now(),
-            ->andReturn($logger);
+                ]);
-        $logger->shouldReceive('error')
+            });
-            ->once()
-            ->with('Stock reservation cleanup failed.', [
-                'command' => 'reservations:expire',
-                'expired_purchases' => 2,
-                'expired_cart_reservations' => null,
-                'exception' => $exception,
-            ]);
 
         $service = new ExpireStockReservationsService(
             \Mockery::mock(ReleaseCheckoutService::class),