commerceguys / guzzle-oauth2-plugin

tests/MockOAuth2Server.php

Doc Comments +9 added lines, -9 removed lines

@@ -58,7 +58,7 @@ 
     }
 
     /**
-     * @return HandlerStack
+     * @return MockHandler
      */
     public function getHandlerStack()
     {
@@ -69,7 +69,7 @@ 
      * @param RequestInterface $request
      * @param array            $options
      *
-     * @return array
+     * @return Response
      */
     protected function getResult(RequestInterface $request, array $options)
     {
@@ -89,7 +89,7 @@ 
      * @param RequestInterface $request
      * @param array            $options
      *
-     * @return array
+     * @return Response
      */
     protected function oauth2Token(RequestInterface $request, $options)
     {
@@ -114,7 +114,7 @@ 
     }
 
     /**
-     * @return array
+     * @return Response
      */
     protected function validTokenResponse()
     {
@@ -136,7 +136,7 @@ 
     /**
      * @param array  $requestBody
      *
-     * @return array
+     * @return Response
      *   The response as expected by the MockHandler.
      */
     protected function grantTypePassword(array $requestBody)
@@ -152,7 +152,7 @@ 
     /**
      * @param array  $options
      *
-     * @return array
+     * @return Response
      *   The response as expected by the MockHandler.
      */
     protected function grantTypeClientCredentials(array $options)
@@ -168,7 +168,7 @@ 
     /**
      * @param array  $requestBody
      *
-     * @return array
+     * @return Response
      */
     protected function grantTypeRefreshToken(array $requestBody)
     {
@@ -182,7 +182,7 @@ 
     /**
      * @param array  $requestBody
      *
-     * @return array
+     * @return Response
      */
     protected function grantTypeJwtBearer(array $requestBody)
     {
@@ -196,7 +196,7 @@ 
     /**
      * @param RequestInterface $request
      *
-     * @return array
+     * @return Response
      */
     protected function mockApiCall(RequestInterface $request)
     {

Unused Use Statements -1 removed lines

@@ -3,7 +3,6 @@
 namespace CommerceGuys\Guzzle\Oauth2\Tests;
 
 
-use GuzzleHttp\Exception\RequestException;
 use GuzzleHttp\Handler\MockHandler;
 use GuzzleHttp\HandlerStack;
 use GuzzleHttp\Psr7\Response;

Spacing +1 added lines, -1 removed lines

@@ -47,7 +47,7 @@
                 array_fill(
                     0,
                     $this->options[self::KEY_EXPECTED_QUERY_COUNT],
-                    function (RequestInterface $request, array $options) {
+                    function(RequestInterface $request, array $options) {
                         return $this->getResult($request, $options);
                     }
                 )

tests/OAuthMiddlewareTest.php

Unused Use Statements -1 removed lines

@@ -9,7 +9,6 @@
 use CommerceGuys\Guzzle\Oauth2\Tests\MockOAuth2Server;
 use CommerceGuys\Guzzle\Oauth2\Tests\MockOAuthMiddleware;
 use CommerceGuys\Guzzle\Oauth2\Tests\TestBase;
-use GuzzleHttp\HandlerStack;
 use GuzzleHttp\RequestOptions;
 use Psr\Http\Message\ResponseInterface;
 

src/Middleware/OAuthMiddleware.php

Spacing +7 added lines, -7 removed lines

@@ -56,8 +56,8 @@ 
      */
     public function onBefore()
     {
-        return function (callable $handler) {
-            return function (RequestInterface $request, array $options) use ($handler) {
+        return function(callable $handler) {
+            return function(RequestInterface $request, array $options) use ($handler) {
                 if (
                     isset($options['auth']) &&
                     'oauth2' == $options['auth'] &&
@@ -66,7 +66,7 @@ 
                 ) {
                     $token = $this->getAccessToken();
                     if ($token !== null) {
-                        return $handler($request->withAddedHeader('Authorization', 'Bearer '.$token->getToken()), $options);
+                        return $handler($request->withAddedHeader('Authorization', 'Bearer ' . $token->getToken()), $options);
                     }
                 }
 
@@ -79,13 +79,13 @@ 
     {
         $calls = 0;
 
-        return function (callable $handler) use (&$calls, $limit) {
-            return function (RequestInterface $request, array $options) use ($handler, &$calls, $limit) {
+        return function(callable $handler) use (&$calls, $limit) {
+            return function(RequestInterface $request, array $options) use ($handler, &$calls, $limit) {
                 /* @var PromiseInterface */
                 $promise = $handler($request, $options);
 
                 return $promise->then(
-                    function (ResponseInterface $response) use ($request, $options, &$calls, $limit) {
+                    function(ResponseInterface $response) use ($request, $options, &$calls, $limit) {
                         if (
                             $response->getStatusCode() == 401 &&
                             isset($options['auth']) &&
@@ -99,7 +99,7 @@ 
                             }
 
                             if ($token = $this->getAccessToken()) {
-                                $response = $this->client->send($request->withHeader('Authorization', 'Bearer '.$token->getToken()), $options);
+                                $response = $this->client->send($request->withHeader('Authorization', 'Bearer ' . $token->getToken()), $options);
                             }
                         }
 

tests/MockOAuthMiddleware.php

Spacing +3 added lines, -3 removed lines

@@ -50,12 +50,12 @@
     {
         $calls = 0;
 
-        return function (callable $handler) use (&$calls) {
-            return function (RequestInterface $request, array $options) use ($handler, &$calls) {
+        return function(callable $handler) use (&$calls) {
+            return function(RequestInterface $request, array $options) use ($handler, &$calls) {
                 /** @var PromiseInterface */
                 $promise = $handler($request, $options);
                 return $promise->then(
-                    function (ResponseInterface $response) use ($request, $options, &$calls) {
+                    function(ResponseInterface $response) use ($request, $options, &$calls) {
                         if (
                             isset($this->options[self::KEY_TOKEN_EXPIRED_ON_FAILURE_COUNT]) &&
                             isset($options['auth']) &&