Change DeployKeys::update method to match GitHub API (#797)

* Change DeployKeys::update method to first remove the existing key and then create a new one, in accordance with the supported GitHub APIs

* Change DeployKeys::update method to first remove the existing key and then create a new one, in accordance with the supported GitHub APIs

* Convert tabs to spaces

Author: Mitchell Davis

lib/Github/Api/Repository/DeployKeys.php

@@ -37,7 +37,9 @@ public function update($username, $repository, $id, array $params)
             throw new MissingArgumentException(['title', 'key']);
         }
 
-        return $this->patch('/repos/'.rawurlencode($username).'/'.rawurlencode($repository).'/keys/'.rawurlencode($id), $params);
+        $this->remove($username, $repository, $id);
+
+        return $this->create($username, $repository, $params);
     }
 
     public function remove($username, $repository, $id)

test/Github/Tests/Api/Repository/DeployKeysTest.php

@@ -111,7 +111,9 @@ public function shouldNotUpdateDeployKeyWithoutTitle()
 
         $api = $this->getApiMock();
         $api->expects($this->never())
-            ->method('patch');
+            ->method('delete');
+        $api->expects($this->never())
+            ->method('post');
 
         $api->update('KnpLabs', 'php-github-api', 123, $data);
     }
@@ -126,7 +128,9 @@ public function shouldNotUpdateDeployKeyWithoutKey()
 
         $api = $this->getApiMock();
         $api->expects($this->never())
-            ->method('patch');
+            ->method('delete');
+        $api->expects($this->never())
+            ->method('post');
 
         $api->update('KnpLabs', 'php-github-api', 123, $data);
     }
@@ -141,8 +145,12 @@ public function shouldUpdateDeployKey()
 
         $api = $this->getApiMock();
         $api->expects($this->once())
-            ->method('patch')
-            ->with('/repos/KnpLabs/php-github-api/keys/123', $data)
+            ->method('delete')
+            ->with('/repos/KnpLabs/php-github-api/keys/123')
+            ->will($this->returnValue($expectedValue));
+        $api->expects($this->once())
+            ->method('post')
+            ->with('/repos/KnpLabs/php-github-api/keys', $data)
             ->will($this->returnValue($expectedValue));
 
         $this->assertEquals($expectedValue, $api->update('KnpLabs', 'php-github-api', 123, $data));