Fixing backwards compatibility in mounts -- they were not correctly copying the path over to the name

git-svn-id: https://reductivelabs.com/svn/puppet/trunk@1888 980ebf18-57e1-0310-9a29-db15c13687c0

1 files changed, 24 insertions, 5 deletions

diff --git a/test/types/mount.rb b/test/types/mount.rb
index 8b8cf9ab5..1134b664a 100755
--- a/test/types/mount.rb
+++ b/test/types/mount.rb
@@ -76,6 +76,14 @@ class TestMounts < Test::Unit::TestCase
     def mkmount
         mount = nil
 
+        assert_nothing_raised {
+            mount = Puppet.type(:mount).create(mkmount_args)
+        }
+
+        return mount
+    end
+
+    def mkmount_args
         if defined? @pcount
             @pcount += 1
         else
@@ -95,11 +103,7 @@ class TestMounts < Test::Unit::TestCase
             end
         end
 
-        assert_nothing_raised {
-            mount = Puppet.type(:mount).create(args)
-        }
-
-        return mount
+        return args
     end
 
     def test_simplemount
@@ -223,6 +227,21 @@ class TestMounts < Test::Unit::TestCase
             "Mount was not actually removed")
     end
     end
+
+    # Make sure that the name gets correctly set if they set the path,
+    # which used to be the namevar.
+    def test_name_and_path
+        mount = nil
+        args = mkmount_args
+        args[:name] = "mount_name"
+        args[:path] = "mount_path"
+
+        assert_nothing_raised do
+            mount = @mount.create(args)
+        end
+
+        assert_equal("mount_path", mount[:name], "Name did not get copied over")
+    end
 end
 
 # $Id$