client: fixes legacy guzzle request objects instantiation

Theres a big difference between `5.*` and `6.*` guzzle majors. So, in the `PagarMe\Sdk\Client::buildRequest` method the sdk should instantiate the correct request object accordingly its contracts, in this case twice: `\GuzzleHttp\Message\Request` and `\GuzzleHttp\Psr7\Request`.

If the user define request options, in guzzle `5.*` this options should be within the request object. If the major is `6.*` the options should be informed on `\GuzzleHttp\Client::send()` method as a second argument.

This fixes the described behavior above informing the request options on the right places.

lib/Client.php

@@ -81,10 +81,14 @@ public function send(RequestInterface $apiRequest)
     private function buildRequest($apiRequest)
     {
         if (class_exists('\\GuzzleHttp\\Message\\Request')) {
+            $options = array_merge(
+                $this->requestOptions,
+                ['json' => $this->buildBody($apiRequest)]
+            );
             return $this->client->createRequest(
                 $apiRequest->getMethod(),
                 $apiRequest->getPath(),
-                ['json' => $this->buildBody($apiRequest)]
+                $options
             );
         }
 

tests/unit/ClientTest.php

@@ -69,7 +69,8 @@ public function mustSendRequestWithProperContent()
                         'json' => [
                             'content'        => self::CONTENT,
                             'api_key'        => self::API_KEY
-                        ]
+                        ],
+                        'timeout' => null
                     ]
                 )
                 ->willReturn($this->getMock('GuzzleHttp\Message\RequestInterface'));