Minor fixes to compute key_pairs (#548)

* `delete_key_pair()` now passes options in query string if provided
* The Mock implementation for `get_key_pair()` is no longer misnamed

Author: otuz-godaddy

lib/fog/openstack/compute/requests/delete_key_pair.rb

@@ -8,13 +8,14 @@ def delete_key_pair(key_name, user_id = nil)
           request(
             :expects => [202, 204],
             :method  => 'DELETE',
-            :path    => "os-keypairs/#{Fog::OpenStack.escape(key_name)}"
+            :path    => "os-keypairs/#{Fog::OpenStack.escape(key_name)}",
+            :query   => options
           )
         end
       end
 
       class Mock
-        def delete_key_pair(_key_name)
+        def delete_key_pair(_key_name, _user_id = nil)
           response = Excon::Response.new
           response.status = 202
           response.headers = {

lib/fog/openstack/compute/requests/get_key_pair.rb

@@ -12,7 +12,7 @@ def get_key_pair(key_name)
       end
 
       class Mock
-        def delete_key_pair(key_name)
+        def get_key_pair(key_name)
           response = Excon::Response.new
           response.status = 200
           response.headers = {