Added method "testProperlyHandlesProxyHeadersWithCurlBug2"

Author: Stephen Seliuk

tests/HttpClients/AbstractTestHttpClient.php

@@ -7,6 +7,8 @@ abstract class AbstractTestHttpClient extends PHPUnit_Framework_TestCase
 Content-Type: text/html; charset=utf-8
 Location: https://foobar.com/\r\n\r\n";
   protected $fakeRawProxyHeader = "HTTP/1.0 200 Connection established\r\n\r\n";
+  protected $fakeRawProxyHeader2 = "HTTP/1.0 200 Connection established
+Proxy-agent: Kerio Control/7.1.1 build 1971\r\n\r\n";
   protected $fakeRawHeader = "HTTP/1.1 200 OK
 Etag: \"9d86b21aa74d74e574bbb35ba13524a52deb96e3\"
 Content-Type: text/javascript; charset=UTF-8

tests/HttpClients/FacebookCurlHttpClientTest.php

@@ -270,7 +270,7 @@ public function testProperlyHandlesProxyHeaders()
     $this->assertEquals($headers, $this->fakeHeadersAsArray);
   }
 
-  public function testProperlyHandlesProxyHeadersWithCurlBug()
+  public function testProperlyHandlesProxyHeadAdersWithCurlBug()
   {
     $rawHeader = $this->fakeRawProxyHeader . $this->fakeRawHeader;
     $this->curlMock
@@ -298,6 +298,34 @@ public function testProperlyHandlesProxyHeadersWithCurlBug()
     $this->assertEquals($headers, $this->fakeHeadersAsArray);
   }
 
+  public function testProperlyHandlesProxyHeadersWithCurlBug2()
+  {
+        $rawHeader = $this->fakeRawProxyHeader2 . $this->fakeRawHeader;
+        $this->curlMock
+            ->shouldReceive('getinfo')
+            ->with(CURLINFO_HEADER_SIZE)
+            ->once()
+            ->andReturn(mb_strlen($this->fakeRawHeader)); // Mimic bug that doesn't count proxy header
+        $this->curlMock
+            ->shouldReceive('version')
+            ->once()
+            ->andReturn(array('version_number' => self::CURL_VERSION_BUGGY));
+        $this->curlMock
+            ->shouldReceive('exec')
+            ->once()
+            ->andReturn($rawHeader . $this->fakeRawBody);
+
+        $this->curlClient->sendRequest();
+        list($rawHeaders, $rawBody) = $this->curlClient->extractResponseHeadersAndBody();
+
+        $this->assertEquals($rawHeaders, trim($rawHeader));
+        $this->assertEquals($rawBody, $this->fakeRawBody);
+
+        $headers = FacebookCurlHttpClient::headersToArray($rawHeaders);
+
+        $this->assertEquals($headers, $this->fakeHeadersAsArray);
+  }
+
   public function testProperlyHandlesRedirectHeaders()
   {
     $rawHeader = $this->fakeRawRedirectHeader . $this->fakeRawHeader;