From 54453be5642a8eb4c90fe5c5941211af12588da1 Mon Sep 17 00:00:00 2001
From: Tomokazu Hirai <tomokazu.hirai@kddi-web.com>
Date: Thu, 28 Feb 2013 10:44:56 +0900
Subject: [PATCH] @rubiojr wrote mock code for openstack routers.

---
 lib/fog/openstack/requests/network/create_router.rb | 6 +++---
 lib/fog/openstack/requests/network/get_router.rb    | 4 ++--
 lib/fog/openstack/requests/network/list_routers.rb  | 2 +-
 lib/fog/openstack/requests/network/update_router.rb | 4 ++--
 4 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/lib/fog/openstack/requests/network/create_router.rb b/lib/fog/openstack/requests/network/create_router.rb
index ca8694d9b..dd1386324 100644
--- a/lib/fog/openstack/requests/network/create_router.rb
+++ b/lib/fog/openstack/requests/network/create_router.rb
@@ -39,15 +39,15 @@ module Fog
           data = {
             'router' => {
               'status' => 'ACTIVE',
-              'external_gateway_info' => null,
+              'external_gateway_info' => nil,
               'name' => 'another_router',
               'admin_state_up' => true,
               'tenant_id' => '6b96ff0cb17a4b859e1e575d221683d3',
               'id' => '8604a0de-7f6b-409a-a47c-a1cc7bc77b2e'
             }
           }
-          self.data[:routers][data['id']] = data
-          response.body = { 'router' => data }
+          self.data[:routers] = []
+          self.data[:routers] << data['router']
           response
         end
       end
diff --git a/lib/fog/openstack/requests/network/get_router.rb b/lib/fog/openstack/requests/network/get_router.rb
index 8e86817cc..d489e69c8 100644
--- a/lib/fog/openstack/requests/network/get_router.rb
+++ b/lib/fog/openstack/requests/network/get_router.rb
@@ -15,10 +15,10 @@ module Fog
       class Mock
         def get_router(router_id)
           response = Excon::Response.new
-          if data = self.data[:routers][router_id]
+          if data = (self.data[:routers].find { |r| r['id'] == router_id })
             response.status = 200
             response.body = {
-              'routers' => {
+              'router' => {
                 'status' => 'ACTIVE',
                 'external_gateway_info' => {
                   'network_id' => '3c5bcddd-6af9-4e6b-9c3e-c153e521cab8'
diff --git a/lib/fog/openstack/requests/network/list_routers.rb b/lib/fog/openstack/requests/network/list_routers.rb
index 3f692c369..de575cb31 100644
--- a/lib/fog/openstack/requests/network/list_routers.rb
+++ b/lib/fog/openstack/requests/network/list_routers.rb
@@ -16,7 +16,7 @@ module Fog
       class Mock
         def list_routers(filters = {})
           Excon::Response.new(
-            :body   => { 'routers' => self.data[:routers].values },
+            :body   => { 'routers' => (self.data[:routers] || []) },
             :status => 200
           )
         end
diff --git a/lib/fog/openstack/requests/network/update_router.rb b/lib/fog/openstack/requests/network/update_router.rb
index bdcd74a55..af25c7768 100644
--- a/lib/fog/openstack/requests/network/update_router.rb
+++ b/lib/fog/openstack/requests/network/update_router.rb
@@ -52,7 +52,8 @@ module Fog
       end
 
       class Mock
-        def update_router(router_id, network_id, options = {})
+        def update_router(router_id, options = {})
+          raise Fog::Network::OpenStack::NotFound if router_id == 0
           response = Excon::Response.new
           response.status = 201
           data = {
@@ -65,7 +66,6 @@ module Fog
             'tenant_id' => '6b96ff0cb17a4b859e1e575d221683d3',
             'id' => '8604a0de-7f6b-409a-a47c-a1cc7bc77b2e'
           }
-          self.data[:router_id][data['router_id']] = data
           response.body = { 'router' => data }
           response
         end