Don't send an emtpy ref to the client

The api will return Not Found when sending an empty ref.

Author: NoUseFreak

lib/Github/Api/AbstractApi.php

@@ -69,6 +69,9 @@ protected function get($path, array $parameters = array(), $requestHeaders = arr
         if (null !== $this->perPage && !isset($parameters['per_page'])) {
             $parameters['per_page'] = $this->perPage;
         }
+        if (array_key_exists('ref', $parameters) && is_null($parameters['ref'])) {
+            unset($parameters['ref']);
+        }
         $response = $this->client->getHttpClient()->get($path, $parameters, $requestHeaders);
 
         return ResponseMediator::getContent($response);

test/Github/Tests/Api/AbstractApiTest.php

@@ -3,6 +3,7 @@
 namespace Github\Tests\Api;
 
 use Github\Api\AbstractApi;
+use Guzzle\Http\Message\Response;
 
 class AbstractApiTest extends \PHPUnit_Framework_TestCase
 {
@@ -111,6 +112,26 @@ public function shouldPassDELETERequestToClient()
         $this->assertEquals($expectedArray, $api->delete('/path', array('param1' => 'param1value'), array('option1' => 'option1value')));
     }
 
+    /**
+     * @test
+     */
+    public function shouldNotPassEmptyRefToClient()
+    {
+        $expectedResponse = new Response('value');
+
+        $httpClient = $this->getHttpMock();
+        $httpClient
+            ->expects($this->any())
+            ->method('get')
+            ->with('/path', array())
+            ->will($this->returnValue($expectedResponse));
+        $client = $this->getClientMock();
+        $client->setHttpClient($httpClient);
+
+        $api = new ExposedAbstractApiTestInstance($client);
+        $api->get('/path', array('ref' => null));
+    }
+
     protected function getAbstractApiObject($client)
     {
         return new AbstractApiTestInstance($client);
@@ -193,3 +214,14 @@ public function delete($path, array $parameters = array(), $requestHeaders = arr
         return $this->client->getHttpClient()->delete($path, $parameters, $requestHeaders);
     }
 }
+
+class ExposedAbstractApiTestInstance extends AbstractApi
+{
+    /**
+     * {@inheritDoc}
+     */
+    public function get($path, array $parameters = array(), $requestHeaders = array())
+    {
+        return parent::get($path, $parameters, $requestHeaders);
+    }
+}