feature: transform both switchToXHR and removeXhr to a xhrRequest

Amrouche Hamza · Amrouche Hamza · commit 4ed08896fa35 · 2018-03-20T08:54:48.000+01:00
diff --git a/src/Symfony/Component/BrowserKit/Client.php b/src/Symfony/Component/BrowserKit/Client.php
@@ -151,14 +151,15 @@ public function getServerParameter($key, $default = '')
         return isset($this->server[$key]) ? $this->server[$key] : $default;
     }
 
-    public function switchToXHR()
+    public function xmlHttpRequest(string $method, string $uri, array $parameters = array(), array $files = array(), array $server = array(), string $content = null, bool $changeHistory = true): Crawler
     {
         $this->setServerParameter('HTTP_X_REQUESTED_WITH', 'XMLHttpRequest');
-    }
 
-    public function removeXHR()
-    {
-        unset($this->server['HTTP_X_REQUESTED_WITH']);
+        try {
+            return $this->request($method, $uri, $parameters, $files, $server, $content, $changeHistory);
+        } finally {
+            unset($this->server['HTTP_X_REQUESTED_WITH']);
+        }
     }
 
     /**
diff --git a/src/Symfony/Component/BrowserKit/Tests/ClientTest.php b/src/Symfony/Component/BrowserKit/Tests/ClientTest.php
@@ -104,13 +104,11 @@ public function testGetRequestNull()
         $this->assertNull($client->getRequest());
     }
 
-    public function testGetRequestWithXHR()
+    public function testXmlHttpRequest()
     {
         $client = new TestClient();
-        $client->switchToXHR();
-        $client->request('GET', 'http://example.com/', array(), array(), array(), null, true, true);
+        $client->xmlHttpRequest('GET', 'http://example.com/', array(), array(), array(), null, true);
         $this->assertEquals($client->getRequest()->getServer()['HTTP_X_REQUESTED_WITH'], 'XMLHttpRequest');
-        $client->removeXHR();
         $this->assertFalse($client->getServerParameter('HTTP_X_REQUESTED_WITH', false));
     }
 

Original file line number	Diff line number	Diff line change
`@@ -151,14 +151,15 @@ public function getServerParameter($key, $default = '')`
`151`	`151`	`return isset($this->server[$key]) ? $this->server[$key] : $default;`
`152`	`152`	`}`
`153`	`153`
`154`		`- public function switchToXHR()`
	`154`	`+ public function xmlHttpRequest(string $method, string $uri, array $parameters = array(), array $files = array(), array $server = array(), string $content = null, bool $changeHistory = true): Crawler`
`155`	`155`	`{`
`156`	`156`	`$this->setServerParameter('HTTP_X_REQUESTED_WITH', 'XMLHttpRequest');`
`157`		`- }`
`158`	`157`
`159`		`- public function removeXHR()`
`160`		`- {`
`161`		`- unset($this->server['HTTP_X_REQUESTED_WITH']);`
	`158`	`+ try {`
	`159`	`+ return $this->request($method, $uri, $parameters, $files, $server, $content, $changeHistory);`
	`160`	`+ } finally {`
	`161`	`+ unset($this->server['HTTP_X_REQUESTED_WITH']);`
	`162`	`+ }`
`162`	`163`	`}`
`163`	`164`
`164`	`165`	`/**`
Original file line number	Diff line number	Diff line change
`@@ -104,13 +104,11 @@ public function testGetRequestNull()`
`104`	`104`	`$this->assertNull($client->getRequest());`
`105`	`105`	`}`
`106`	`106`
`107`		`- public function testGetRequestWithXHR()`
	`107`	`+ public function testXmlHttpRequest()`
`108`	`108`	`{`
`109`	`109`	`$client = new TestClient();`
`110`		`- $client->switchToXHR();`
`111`		`- $client->request('GET', 'http://example.com/', array(), array(), array(), null, true, true);`
	`110`	`+ $client->xmlHttpRequest('GET', 'http://example.com/', array(), array(), array(), null, true);`
`112`	`111`	`$this->assertEquals($client->getRequest()->getServer()['HTTP_X_REQUESTED_WITH'], 'XMLHttpRequest');`
`113`		`- $client->removeXHR();`
`114`	`112`	`$this->assertFalse($client->getServerParameter('HTTP_X_REQUESTED_WITH', false));`
`115`	`113`	`}`
`116`	`114`