Add stale-read chargeback coverage and simplify comparison

Replace `! lessThan` with `greaterThanOrEqual` for readability in the
aftercare chargeback branch, and add a stale-read regression test that
proves concurrent deliveries dispatch `ChargebackReceived` only once.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Author: sandervanhooft

src/Http/Controllers/AftercareWebhookController.php

@@ -70,7 +70,7 @@ public function handleWebhook(Request $request)
 
                 $locallyKnownAmountChargedBack = $localPayment->getAmountChargedBack();
 
-                if (! $locallyKnownAmountChargedBack->lessThan($molliePaymentAmountChargedBackTotal)) {
+                if ($locallyKnownAmountChargedBack->greaterThanOrEqual($molliePaymentAmountChargedBackTotal)) {
                     return;
                 }
 

tests/Fixtures/ChargebackStaleReadPayment.php

@@ -0,0 +1,40 @@
+<?php
+
+namespace Laravel\Cashier\Tests\Fixtures;
+
+class ChargebackStaleReadPayment extends Payment
+{
+    protected static int $remainingStaleReads = 0;
+
+    protected static int $staleAmountChargedBack = 0;
+
+    public static function returnStaleChargebackAmountOnNextReads(int $reads, int $amountChargedBack = 0): void
+    {
+        static::$remainingStaleReads = $reads;
+        static::$staleAmountChargedBack = $amountChargedBack;
+    }
+
+    public static function resetStaleChargebackReads(): void
+    {
+        static::$remainingStaleReads = 0;
+        static::$staleAmountChargedBack = 0;
+    }
+
+    public static function findByPaymentId($id): ?self
+    {
+        /** @var self|null $payment */
+        $payment = parent::findByPaymentId($id);
+
+        if (! $payment || static::$remainingStaleReads <= 0) {
+            return $payment;
+        }
+
+        static::$remainingStaleReads--;
+
+        /** @var self $stalePayment */
+        $stalePayment = clone $payment;
+        $stalePayment->amount_charged_back = static::$staleAmountChargedBack;
+
+        return $stalePayment;
+    }
+}

tests/Http/Controllers/AftercareWebhookControllerTest.php

@@ -8,12 +8,14 @@
 use Illuminate\Support\Facades\Event;
 use Laravel\Cashier\Cashier;
 use Laravel\Cashier\Events\ChargebackReceived;
+use Laravel\Cashier\Payment;
 use Laravel\Cashier\Events\RefundProcessed;
 use Laravel\Cashier\Http\Controllers\AftercareWebhookController;
 use Laravel\Cashier\Order\OrderItemCollection;
 use Laravel\Cashier\Refunds\RefundItemCollection;
 use Laravel\Cashier\Tests\BaseTestCase;
 use Laravel\Cashier\Tests\Database\Factories\SubscriptionFactory;
+use Laravel\Cashier\Tests\Fixtures\ChargebackStaleReadPayment;
 use Laravel\Cashier\Tests\Fixtures\User;
 use Mollie\Api\MollieApiClient;
 use Mollie\Api\Resources\Payment as MolliePayment;
@@ -25,6 +27,14 @@
 
 class AftercareWebhookControllerTest extends BaseTestCase
 {
+    protected function tearDown(): void
+    {
+        ChargebackStaleReadPayment::resetStaleChargebackReads();
+        Cashier::usePaymentModel(Payment::class);
+
+        parent::tearDown();
+    }
+
     #[Test]
     public function itDetectsNewChargebacks()
     {
@@ -66,6 +76,47 @@ public function itDetectsNewChargebacks()
         Event::assertDispatched(ChargebackReceived::class);
     }
 
+    #[Test]
+    public function itHandlesDuplicateChargebackDeliveriesWhenTheOuterPaymentReadIsStale()
+    {
+        Event::fake();
+        Cashier::usePaymentModel(ChargebackStaleReadPayment::class);
+
+        $molliePayment = new MolliePayment(new MollieApiClient);
+        $molliePayment->id = 'tr_duplicate_chargeback';
+        $molliePayment->status = 'paid';
+        $molliePayment->amount = (object) [
+            'currency' => 'EUR',
+            'value' => '20.00',
+        ];
+        $molliePayment->amountRefunded = null;
+        $molliePayment->amountChargedBack = null;
+        $molliePayment->_links = (object) [];
+
+        $localPayment = Cashier::$paymentModel::createFromMolliePayment($molliePayment, $this->getUser());
+        $molliePayment->amountChargedBack = (object) [
+            'value' => '10.00',
+            'currency' => 'EUR',
+        ];
+        $molliePayment->_links->chargebacks = (object) [
+            'href' => 'https://www.mollie.com/dashboard/org_12345678/payments/tr_WDqYK6vllg',
+            'type' => 'application/json',
+        ];
+
+        $this->withMockedGetMolliePayment(2, $molliePayment);
+        ChargebackStaleReadPayment::returnStaleChargebackAmountOnNextReads(2);
+
+        /** @var AftercareWebhookController $controller */
+        $controller = $this->app->make(AftercareWebhookController::class);
+
+        $controller->handleWebhook($this->getWebhookRequest($molliePayment->id));
+        $controller->handleWebhook($this->getWebhookRequest($molliePayment->id));
+
+        $localPayment->refresh();
+        $this->assertMoney(1000, 'EUR', $localPayment->getAmountChargedBack());
+        Event::assertDispatchedTimes(ChargebackReceived::class, 1);
+    }
+
     #[Test]
     public function itDetectsNewRefunds()
     {