From 9e6a0558af8994d9e0fbe99d0d3c0bf9475cb81b Mon Sep 17 00:00:00 2001
From: Brian Nelson <bnelson@sugarcrm.com>
Date: Thu, 19 Jul 2012 23:19:54 -0700
Subject: [PATCH] fix for RDS mocking to avoid state flipping between
 "modifying" and "available"

---
 lib/fog/aws/requests/rds/describe_db_instances.rb | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/lib/fog/aws/requests/rds/describe_db_instances.rb b/lib/fog/aws/requests/rds/describe_db_instances.rb
index 2fff58771..a83a6a0e8 100644
--- a/lib/fog/aws/requests/rds/describe_db_instances.rb
+++ b/lib/fog/aws/requests/rds/describe_db_instances.rb
@@ -58,7 +58,7 @@ module Fog
                  end
               when "rebooting" # I don't know how to show rebooting just once before it changes to available
                 # it applies pending modified values
-                if server["PendingModifiedValues"]
+                unless server["PendingModifiedValues"].empty?
                   server.merge!(server["PendingModifiedValues"])
                   server["PendingModifiedValues"] = {}
                   self.data[:tmp] ||= Time.now + Fog::Mock.delay * 2
@@ -69,13 +69,13 @@ module Fog
                 end
               when "modifying"
                 # TODO there are some fields that only applied after rebooting
-                if server["PendingModifiedValues"]
+                unless server["PendingModifiedValues"].empty?
                   server.merge!(server["PendingModifiedValues"])
                   server["PendingModifiedValues"] = {}
                   server["DBInstanceStatus"] = 'available'
                 end
               when "available" # I'm not sure if amazon does this
-                if server["PendingModifiedValues"]
+                unless server["PendingModifiedValues"].empty?
                   server["DBInstanceStatus"] = 'modifying'
                 end