Merge pull request #1614 from blowmage/fix_openstack_mock_methods

Fix openstack mock methods

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

@@ -16,7 +16,7 @@ module Fog
       end
 
       class Mock
-        def allocate_address
+        def allocate_address(pool = nil)
           response = Excon::Response.new
           response.status = 200
           response.headers = {

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

@@ -16,7 +16,7 @@ module Fog
 
       class Mock
 
-        def get_snapshot_details(detailed=true)
+        def get_snapshot_details(snapshot_id)
           response = Excon::Response.new
           response.status = 200
           response.body = {

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

@@ -16,7 +16,7 @@ module Fog
 
       class Mock
 
-        def get_volume_details(detailed=true)
+        def get_volume_details(volume_id)
           response = Excon::Response.new
           response.status = 200
           response.body = {

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

@@ -19,7 +19,7 @@ module Fog
 
       class Mock
 
-        def list_servers
+        def list_servers(options = {})
           response = Excon::Response.new
           data = list_servers_detail.body['servers']
           servers = []

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

@@ -20,7 +20,7 @@ module Fog
       end # class Real
 
       class Mock
-        def remove_fixed_ip(server_id, network_id)
+        def remove_fixed_ip(server_id, address)
           true
         end # def remove_fixed_ip
       end # class Mock

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

@@ -16,7 +16,7 @@ module Fog
 
       class Mock
 
-        def update_server(server_id, options)
+        def update_server(server_id, options = {})
           response = Excon::Response.new
           if server = list_servers_detail.body['servers'].detect {|_| _['id'] == server_id}
             if options['name']