Merge pull request #26 from apoca/develop

closed: #25 - Checkout status must be a GET not a POST
apoca · Apr 12, 2019 · a6423eb · a6423eb
2 parents 015b0d8 + 9905087
commit a6423eb
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/Brands/Transaction.php b/src/Brands/Transaction.php
@@ -72,7 +72,7 @@ public function status(): object
         try {
             $client = new Client($this->clientConfig);
 
-            $response = $client->post($this->endpoint . "checkouts/{$this->getCheckoutId()}/payment?entityId=" . config('sibs.authentication.entityId'),
+            $response = $client->get($this->endpoint . "checkouts/{$this->getCheckoutId()}/payment?entityId=" . config('sibs.authentication.entityId'),
                 [
                     'headers' => [
                         'Authorization' => config('sibs.authentication.token'),

diff --git a/tests/Feature/SibsServiceTest.php b/tests/Feature/SibsServiceTest.php
@@ -33,6 +33,6 @@ public function verifyResponseStatusInvalidEntity(): void
         $response2 = Sibs::status($response->response->id);
 
         $this->assertSame($response->status, 200);
-        $this->assertSame($response2->status, 400);
+        $this->assertSame($response2->status, 200);
     }
 }