Fixing #923 (again). The host storage method was not

correctly searching for the host, so it was creating
a new host on each run, which is what was causing the
conflict.

1 files changed, 5 insertions, 4 deletions

diff --git a/test/rails/host.rb b/test/rails/host.rb
index 5853e8219..582bebcb2 100755
--- a/test/rails/host.rb
+++ b/test/rails/host.rb
@@ -51,8 +51,7 @@ class TestRailsHost < PuppetTest::TestCase
         }
 
         # Now collect our facts
-        facts = {"hostname" => Facter.value(:hostname), "test1" => "funtest",
-            "ipaddress" => Facter.value(:ipaddress)}
+        facts = {"hostname" => "myhost", "test1" => "funtest", "ipaddress" => "192.168.0.1"}
 
         # Now try storing our crap
         host = nil
@@ -120,13 +119,15 @@ class TestRailsHost < PuppetTest::TestCase
         facts["test3"] = "funtest"
         facts["test1"] = "changedfact"
         facts.delete("ipaddress")
-        host = nil
         node = mknode(facts["hostname"])
         node.parameters = facts
+        newhost = nil
         assert_nothing_raised {
-            host = Puppet::Rails::Host.store(node, resources)
+            newhost = Puppet::Rails::Host.store(node, resources)
         }
 
+        assert_equal(host.id, newhost.id, "Created new host instance)")
+
         # Make sure it sets the last_compile time
         assert_nothing_raised do
             assert_instance_of(Time, host.last_compile, "did not set last_compile")