Merge pull request #3 from francisco-aguilar-olx/master

Add support for PATCH requests

Author: pproenca

src/HttpClient.php

@@ -49,6 +49,15 @@ public function put($uri)
         return new FluentRequest('PUT', $uri, $this->client);
     }
 
+    /**
+     * @param $uri
+     * @return FluentRequestInterface
+     */
+    public function patch($uri)
+    {
+        return new FluentRequest('PATCH', $uri, $this->client);
+    }
+
     /**
      * @param $uri
      * @return FluentRequestInterface

src/HttpClientInterface.php

@@ -23,6 +23,12 @@ public function post($uri);
      */
     public function put($uri);
 
+    /**
+     * @param $uri
+     * @return FluentRequestInterface
+     */
+    public function patch($uri);
+
     /**
      * @param $uri
      * @return FluentRequestInterface

src/Request/RequestExecutor.php

@@ -51,6 +51,9 @@ public function execute()
             case 'PUT':
                 $requestResponse = $this->getClient()->put($this->uri, $this->options);
                 break;
+            case 'PATCH':
+                $requestResponse = $this->getClient()->patch($this->uri, $this->options);
+                break;
             case 'DELETE':
                 $requestResponse = $this->getClient()->delete($this->uri, $this->options);
                 break;

tests/HttpClientTest.php

@@ -36,6 +36,14 @@ public function testPutMethodIsCalledShouldReturnFluentRequestWithHTTPPutMethod(
         $this->assertAttributeEquals('http://www.olx.com', 'uri', $requestObj);
     }
 
+    public function testPatchMethodIsCalledShouldReturnFluentRequestWithHTTPPatchMethod()
+    {
+        $requestObj = $this->httpClient->patch('http://www.olx.com');
+
+        $this->assertAttributeEquals('PATCH', 'method', $requestObj);
+        $this->assertAttributeEquals('http://www.olx.com', 'uri', $requestObj);
+    }
+
     public function testDeleteMethodIsCalledShouldReturnFluentRequestWithHTTPDeleteMethod()
     {
         $requestObj = $this->httpClient->delete('http://www.olx.com');

tests/Request/RequestExecutorTest.php

@@ -29,6 +29,13 @@ public function testExecuteGivenHTTPMethodIsPUT()
         $this->assertEquals('put', $executor());
     }
 
+    public function testExecuteGivenHTTPMethodIsPATCH()
+    {
+        $executor = $this->makeRequestExecutorFor('PATCH');
+
+        $this->assertEquals('patch', $executor());
+    }
+
     public function testExecuteGivenHTTPMethodIsDELETE()
     {
         $executor = $this->makeRequestExecutorFor('DELETE');
@@ -55,12 +62,13 @@ private function makeRequestExecutorFor($method)
     {
         // set the methods manually, since with >Guzzle 6.0 the client uses __call method which complicates a bit the mocking
         $httpClient = $this->getMockBuilder(Client::class)
-            ->setMethods(['get','post','put','delete'])
+            ->setMethods(['get','post','put','patch','delete'])
             ->getMock();
 
         $httpClient->method('get')->willReturn('get');
         $httpClient->method('post')->willReturn('post');
         $httpClient->method('put')->willReturn('put');
+        $httpClient->method('patch')->willReturn('patch');
         $httpClient->method('delete')->willReturn('delete');
 
         $requestExecutor = $this->getMockBuilder(RequestExecutor::class)