Merge pull request #9 from dgentilezza/master

Use of "body" is contemplated in method GET

Author: matricali

src/Client.php

@@ -46,7 +46,9 @@ public function __construct()
         $this->handle = curl_init();
 
         if (!is_resource($this->handle)) {
+            // @codeCoverageIgnoreStart
             throw new ClientException(curl_error($this->handle), 'curl');
+            // @codeCoverageIgnoreEnd
         }
 
         $this->setOptions([
@@ -172,18 +174,19 @@ public function sendRequest(RequestInterface $request)
      * Create a GET request for the client.
      *
      * @param string $uri     Resource URI.-
+     * @param string $body    Body to send in the request.-
      * @param array  $headers HTTP headers.-
      * @param array  $options Options to apply to the request.-
      *
      * @throws ClientException
      *
      * @return Response
      */
-    public function get($uri, array $headers = [], array $options = [])
+    public function get($uri, $body = '', array $headers = [], array $options = [])
     {
         $this->setOptions($options);
 
-        return $this->sendRequest(new Request(HttpMethod::GET, $uri, $headers));
+        return $this->sendRequest(new Request(HttpMethod::GET, $uri, $headers, $body));
     }
 
     /**

src/ClientInterface.php

@@ -15,12 +15,13 @@ public function setOptions(array $options);
      * Create a GET request for the client.
      *
      * @param string $uri     Resource URI.-
+     * @param string $body    Body to send in the request.-
      * @param array  $headers HTTP headers.-
      * @param array  $options Options to apply to the request.-
      *
      * @return Response
      */
-    public function get($uri, array $headers = [], array $options = []);
+    public function get($uri, $body = '', array $headers = [], array $options = []);
 
     /**
      * Create a POST request for the client.

tests/EnumTest.php

@@ -0,0 +1,62 @@
+<?php
+/*
+Copyright (c) 2017 Jorge Matricali <jorgematricali@gmail.com>
+
+Permission is hereby granted, free of charge, to any person obtaining a copy
+of this software and associated documentation files (the "Software"), to deal
+in the Software without restriction, including without limitation the rights
+to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+copies of the Software, and to permit persons to whom the Software is
+furnished to do so, subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included in
+all copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
+THE SOFTWARE.
+*/
+
+namespace Matricali\Http\Tests;
+
+use Matricali\Http\HttpMethod;
+use PHPUnit\Framework\TestCase;
+
+/**
+ * @group Client
+ */
+class EnumTest extends TestCase
+{
+    public function testGetByNameNoName()
+    {
+        $res = HttpMethod::getByName('noName');
+        $this->assertEmpty($res);
+    }
+
+    public function testGetByNameName()
+    {
+        $res = HttpMethod::getByName('GET');
+        $this->assertEquals($res, 'GET');
+    }
+
+    public function testIsValidName()
+    {
+        // True
+        $res = HttpMethod::isValidName('GET');
+        $this->assertTrue($res);
+        // False
+        $res = HttpMethod::isValidName('GOT');
+        $this->assertFalse($res);
+        // True
+        $res = HttpMethod::isValidName('GET',true);
+        $this->assertTrue($res);
+        // False
+        $res = HttpMethod::isValidName('get',true);
+        $this->assertFalse($res);
+    }
+
+}