Merge pull request #30 from clue-labs/resize

clue · clue · commit b0206d4a5c43 · 2016-04-21T01:24:25.000+02:00
Fix execResize() to issue POST request
diff --git a/src/Client.php b/src/Client.php
@@ -856,7 +856,7 @@ public function execStart($exec, $config)
      */
     public function execResize($exec, $w, $h)
     {
-        return $this->browser->get(
+        return $this->browser->post(
             $this->uri->expand(
                 '/exec/{exec}/resize{?w,h}',
                 array(
diff --git a/tests/ClientTest.php b/tests/ClientTest.php
@@ -371,7 +371,7 @@ public function testExecStart()
 
     public function testExecResize()
     {
-        $this->expectRequestFlow('get', '/exec/123/resize?w=800&h=600', $this->createResponse(), 'expectEmpty');
+        $this->expectRequestFlow('POST', '/exec/123/resize?w=800&h=600', $this->createResponse(), 'expectEmpty');
 
         $this->expectPromiseResolveWith('', $this->client->execResize(123, 800, 600));
     }

Original file line number	Diff line number	Diff line change
`@@ -856,7 +856,7 @@ public function execStart($exec, $config)`
`856`	`856`	`*/`
`857`	`857`	`public function execResize($exec, $w, $h)`
`858`	`858`	`{`
`859`		`- return $this->browser->get(`
	`859`	`+ return $this->browser->post(`
`860`	`860`	`$this->uri->expand(`
`861`	`861`	`'/exec/{exec}/resize{?w,h}',`
`862`	`862`	`array(`
Original file line number	Diff line number	Diff line change
`@@ -371,7 +371,7 @@ public function testExecStart()`
`371`	`371`
`372`	`372`	`public function testExecResize()`
`373`	`373`	`{`
`374`		`- $this->expectRequestFlow('get', '/exec/123/resize?w=800&h=600', $this->createResponse(), 'expectEmpty');`
	`374`	`+ $this->expectRequestFlow('POST', '/exec/123/resize?w=800&h=600', $this->createResponse(), 'expectEmpty');`
`375`	`375`
`376`	`376`	`$this->expectPromiseResolveWith('', $this->client->execResize(123, 800, 600));`
`377`	`377`	`}`