Merge pull request #1205 from smortex/to_python-nil

Fix serialization of undef in to_python()

Author: david22swan

lib/puppet/functions/to_python.rb

@@ -28,7 +28,7 @@ def to_python(object)
     case object
     when true then 'True'
     when false then 'False'
-    when :undef then 'None'
+    when nil then 'None'
     when Array then "[#{object.map { |x| to_python(x) }.join(', ')}]"
     when Hash then "{#{object.map { |k, v| "#{to_python(k)}: #{to_python(v)}" }.join(', ')}}"
     else object.inspect

lib/puppet/functions/to_ruby.rb

@@ -26,7 +26,6 @@
   #   The String representation of the object
   def to_ruby(object)
     case object
-    when :undef then 'nil'
     when Array then "[#{object.map { |x| to_ruby(x) }.join(', ')}]"
     when Hash then "{#{object.map { |k, v| "#{to_ruby(k)} => #{to_ruby(v)}" }.join(', ')}}"
     else object.inspect

spec/functions/to_python_spec.rb

@@ -5,7 +5,7 @@
 describe 'to_python' do
   it { is_expected.not_to eq(nil) }
   it { is_expected.to run.with_params('').and_return('""') }
-  it { is_expected.to run.with_params(:undef).and_return('None') }
+  it { is_expected.to run.with_params(nil).and_return('None') }
   it { is_expected.to run.with_params(true).and_return('True') }
   it { is_expected.to run.with_params(false).and_return('False') }
   it { is_expected.to run.with_params('one').and_return('"one"') }

spec/functions/to_ruby_spec.rb

@@ -5,7 +5,7 @@
 describe 'to_ruby' do
   it { is_expected.not_to eq(nil) }
   it { is_expected.to run.with_params('').and_return('""') }
-  it { is_expected.to run.with_params(:undef).and_return('nil') }
+  it { is_expected.to run.with_params(nil).and_return('nil') }
   it { is_expected.to run.with_params(true).and_return('true') }
   it { is_expected.to run.with_params('one').and_return('"one"') }
   it { is_expected.to run.with_params(42).and_return('42') }