[Pkg-puppet-devel] [SCM] Puppet packaging for Debian branch, master, updated. debian/0.24.6-1-356-g5718585

[James Turnbull]

The following commit has been merged in the master branch:
commit ed4c4050a8ac46c509b20031814646a49ba86e54
Author: Luke Kanies <luke at madstop.com>
Date:   Thu Nov 20 14:37:16 2008 -0600

    Fixing #1755 - File modes (and other strange properties) will now display correctly
    in ralsh and generated manifests.
    
    Signed-off-by: Luke Kanies <luke at madstop.com>

diff --git a/lib/puppet/type.rb b/lib/puppet/type.rb
index 7e86549..7a1c8b4 100644
--- a/lib/puppet/type.rb
+++ b/lib/puppet/type.rb
@@ -2487,8 +2487,8 @@ class Type
         trans = TransObject.new(self.title, self.class.name)
 
         values = retrieve()
-        values.each do |name, value|
-            trans[name.name] = value
+        values.each do |param, value|
+            trans[param.name] = param.is_to_s(value)
         end
 
         @parameters.each do |name, param|
diff --git a/spec/unit/type.rb b/spec/unit/type.rb
index 5e6cf33..903936a 100755
--- a/spec/unit/type.rb
+++ b/spec/unit/type.rb
@@ -61,4 +61,18 @@ describe Puppet::Type do
             @catalog.clear(true)
         end
     end
+
+    describe "when converting to a transportable resource" do
+        before do
+            @user = Puppet::Type.type(:user).create(:name => "to_trans_testing", :ensure => :present)
+        end
+
+        it "should use each parameter's 'is_to_s' method to display the value" do
+            # Use 'ensure' because none of the other props will be tested if the user is absent.
+            @user.property(:ensure).expects(:retrieve).returns :absent
+            @user.property(:ensure).expects(:is_to_s).with(:absent)
+
+            @user.to_trans
+        end
+    end
 end

-- 
Puppet packaging for Debian