Merge pull request #96 from KnpLabs/fix/clear-headers

Fix clearHeaders to use array keys instead of plain text

Author: cursedcoder

lib/Github/HttpClient/HttpClient.php

@@ -72,8 +72,8 @@ public function setHeaders(array $headers)
     public function clearHeaders()
     {
         $this->headers = array(
-            sprintf('Accept: application/vnd.github.%s+json', $this->options['api_version']),
-            sprintf('User-Agent: %s', $this->options['user_agent']),
+            'Accept' => sprintf('application/vnd.github.%s+json', $this->options['api_version']),
+            'User-Agent' => sprintf('%s', $this->options['user_agent']),
         );
     }
 

test/Github/Tests/Functional/RepoTest.php

@@ -7,6 +7,23 @@
  */
 class RepoTest extends TestCase
 {
+    /**
+     * @test
+     */
+    public function shouldShowPRDiffIfHeaderIsPresent()
+    {
+        $this->client->setHeaders(
+            array('Accept' => sprintf(
+                'application/vnd.github.%s.diff',
+                $this->client->getOption('api_version')
+            ))
+        );
+
+        $diff = $this->client->api('pull_request')->show('KnpLabs', 'php-github-api', '92');
+
+        $this->assertTrue('string' === gettype($diff));
+    }
+
     /**
      * @test
      */