fix: use json guzzle request option instead of form_params (#102)

* fix: use json guzzle request option instead of form_params * [test] assert using json in defer subscription:defer --------- Co-authored-by: Mike Mellor <[email protected]> Co-authored-by: imdhemy <[email protected]>
imdhemy · Dec 15, 2023 · bb94f3b · bb94f3b
1 parent a227174
commit bb94f3b
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 6 deletions.
diff --git a/src/Subscriptions/SubscriptionClient.php b/src/Subscriptions/SubscriptionClient.php
@@ -86,7 +86,7 @@ public function defer(SubscriptionDeferralInfo $subscriptionDeferralInfo): Time
     {
         $uri = $this->getEndpoint(self::URI_DEFER);
         $options = [
-            'form_params' => [
+            'json' => [
                 'deferralInfo' => $subscriptionDeferralInfo->toArray(),
             ],
         ];

diff --git a/tests/Subscriptions/SubscriptionClientTest.php b/tests/Subscriptions/SubscriptionClientTest.php
@@ -11,7 +11,6 @@
 use Imdhemy\GooglePlay\Subscriptions\SubscriptionPurchase;
 use Imdhemy\GooglePlay\ValueObjects\EmptyResponse;
 use Imdhemy\GooglePlay\ValueObjects\SubscriptionDeferralInfo;
-use Imdhemy\GooglePlay\ValueObjects\Time;
 use Tests\TestCase;
 
 class SubscriptionClientTest extends TestCase
@@ -113,10 +112,8 @@ public function cancel()
 
     /**
      * @test
-     *
-     * @throws GuzzleException
      */
-    public function defer()
+    public function defer(): void
     {
         $desiredExpiryTimeMillis = $this->faker->dateTime->getTimestamp() * 1000;
         $deferResponse = new Response(200, [], json_encode(['newExpiryTimeMillis' => $desiredExpiryTimeMillis]));
@@ -128,11 +125,11 @@ public function defer()
         $deferralInfo = new SubscriptionDeferralInfo(0, $desiredExpiryTimeMillis);
         $newExpiryTime = $subscriptionClient->defer($deferralInfo);
 
-        $this->assertInstanceOf(Time::class, $newExpiryTime);
         $this->assertEquals($desiredExpiryTimeMillis, $newExpiryTime->getCarbon()->getTimestampMs());
 
         /** @var Request $request */
         $request = $transactions[0]['request'];
+        $this->assertEquals($deferralInfo->toArray(), json_decode($request->getBody()->getContents(), true)['deferralInfo']);
         $uri = $this->getEndpoint(SubscriptionClient::URI_DEFER);
         $this->assertEquals($uri, (string)$request->getUri());
     }