update test

dystcz · Nov 20, 2023 · acad37a · acad37a
1 parent 9369169
commit acad37a
Show file tree

Hide file tree

Showing 3 changed files with 45 additions and 9 deletions.
diff --git a/src/StripePaymentAdapter.php b/src/StripePaymentAdapter.php
@@ -83,7 +83,7 @@ public function handleWebhook(Request $request): JsonResponse
 
         $paymentIntentStatus = match (true) {
             in_array($event->type, array_keys($statusMap)) => $statusMap[$event->type],
-            default => 'intent',
+            default => 'unknown',
         };
 
         $paymentIntent = new PaymentIntent(

diff --git a/tests/HandleStripeWebhookTest.php b/tests/HandleStripeWebhookTest.php
@@ -30,6 +30,16 @@
     Config::set('lunar-api.stripe.automatic_payment_methods', false);
     $intentId = App::make(StripePaymentAdapter::class)->createIntent($cart)->id;
 
+    $this->app->bind(\Lunar\Stripe\Concerns\ConstructsWebhookEvent::class, function ($app) {
+        return new class implements \Lunar\Stripe\Concerns\ConstructsWebhookEvent
+        {
+            public function constructEvent(string $jsonPayload, string $signature, string $secret)
+            {
+                return \Stripe\Event::constructFrom([]);
+            }
+        };
+    });
+
     $this->intent = PaymentIntent::retrieve($intentId);
 
     $this->cart = $cart;
@@ -56,7 +66,7 @@
         ->post(
             '/stripe/webhook',
             $data,
-            ['Stripe-Signature' => 'foobar'],
+            ['Stripe-Signature' => $this->determineStripeSignature($data)],
         );
 
     $response->assertSuccessful();
@@ -74,7 +84,11 @@
     $data['data']['object']['id'] = $paymentIntentId;
 
     $response = $this
-        ->post('/stripe/webhook', $data);
+        ->post(
+            '/stripe/webhook',
+            $data,
+            ['Stripe-Signature' => $this->determineStripeSignature($data)],
+        );
 
     $response->assertSuccessful();
 
@@ -90,23 +104,29 @@
     $data['data']['object']['id'] = $this->cart->meta['payment_intent'];
 
     $response = $this
-        ->post('/stripe/webhook', $data);
+        ->post(
+            '/stripe/webhook',
+            $data,
+            ['Stripe-Signature' => $this->determineStripeSignature($data)],
+        );
 
     $response->assertSuccessful();
 
     Event::assertDispatched(OrderPaymentFailed::class);
 })->todo();
 
 it('can handle any other event', function () {
+    $events = Event::fake();
+
     /** @var TestCase $this */
     $data = json_decode(file_get_contents(__DIR__.'/Stubs/Stripe/charge.succeeded.json'), true);
 
-    $data['data']['object']['id'] = $this->cart->meta['payment_intent'];
-
     $response = $this
-        ->post('/stripe/webhook', $data);
+        ->post(
+            '/stripe/webhook',
+            $data,
+            ['Stripe-Signature' => $this->determineStripeSignature($data)],
+        );
 
     $response->assertSuccessful();
-
-    Event::assertNothingDispatched();
 })->todo();
diff --git a/tests/TestCase.php b/tests/TestCase.php
@@ -149,6 +149,22 @@ public function getEnvironmentSetUp($app): void
         ]);
     }
 
+    /**
+     * Determine the Stripe signature.
+     */
+    protected function determineStripeSignature(array $payload, string $configKey = null): string
+    {
+        $secret = Config::get('services.stripe.webhooks.'.($configKey ?? 'payment_intent'));
+
+        $timestamp = time();
+
+        $timestampedPayload = $timestamp.'.'.json_encode($payload);
+
+        $signature = hash_hmac('sha256', $timestampedPayload, $secret);
+
+        return "t={$timestamp},v1={$signature}";
+    }
+
     /**
      * Define database migrations.
      */