diff --git a/lib/puppet/pops/evaluator/literal_evaluator.rb b/lib/puppet/pops/evaluator/literal_evaluator.rb index c3da4c0c226..295ba7db5f5 100644 --- a/lib/puppet/pops/evaluator/literal_evaluator.rb +++ b/lib/puppet/pops/evaluator/literal_evaluator.rb @@ -11,9 +11,8 @@ module Evaluator # QualifiedName # Default (produced :default) # Regular Expression (produces ruby regular expression) -# -# Not considered literal -# QualifiedReference # i.e. File, FooBar +# QualifiedReference +# AccessExpresion # class LiteralEvaluator @@ -67,6 +66,16 @@ def literal_LiteralRegularExpression(o) o.value end + def literal_QualifiedReference(o) + o.value + end + + def literal_AccessExpression(o) + # to prevent parameters with [[]] like Optional[[String]] + throw :not_literal if o.keys.size == 1 && o.keys[0].is_a?(Model::LiteralList) + o.keys.map {|v| literal(v) } + end + def literal_ConcatenatedString(o) # use double quoted string value if there is no interpolation throw :not_literal unless o.segments.size == 1 && o.segments[0].is_a?(Model::LiteralString) diff --git a/lib/puppet/pops/issues.rb b/lib/puppet/pops/issues.rb index b6196365681..9ac8abd936e 100644 --- a/lib/puppet/pops/issues.rb +++ b/lib/puppet/pops/issues.rb @@ -207,6 +207,10 @@ def self.hard_issue(issue_code, *args, &block) _("The numeric parameter name '$%{name}' cannot be used (clashes with numeric match result variables)") % { name: name } end + ILLEGAL_NONLITERAL_PARAMETER_TYPE = issue :ILLEGAL_NONLITERAL_PARAMETER_TYPE, :name, :type_class do + _("The parameter '$%{name}' must be a literal type, not %{type_class}") % { name: name, type_class: label.a_an(type_class) } + end + # In certain versions of Puppet it may be allowed to assign to a not already assigned key # in an array or a hash. This is an optional validation that may be turned on to prevent accidental # mutation. diff --git a/lib/puppet/pops/validation/checker4_0.rb b/lib/puppet/pops/validation/checker4_0.rb index eb7ac5cc778..d5a2cde0c4d 100644 --- a/lib/puppet/pops/validation/checker4_0.rb +++ b/lib/puppet/pops/validation/checker4_0.rb @@ -471,6 +471,7 @@ def check_HostClassDefinition(o) internal_check_parameter_name_uniqueness(o) internal_check_reserved_params(o) internal_check_no_idem_last(o) + internal_check_parameter_type_literal(o) end def check_ResourceTypeDefinition(o) @@ -481,6 +482,18 @@ def check_ResourceTypeDefinition(o) internal_check_no_idem_last(o) end + def internal_check_parameter_type_literal(o) + o.parameters.each do |p| + next unless p.type_expr + + type = nil + catch :not_literal do + type = literal(p.type_expr) + end + acceptor.accept(Issues::ILLEGAL_NONLITERAL_PARAMETER_TYPE, p, {name: p.name, type_class: p.type_expr.class}) if type.nil? + end + end + def internal_check_return_type(o) r = o.return_type internal_check_type_ref(o, r) unless r.nil? diff --git a/spec/unit/pops/evaluator/literal_evaluator_spec.rb b/spec/unit/pops/evaluator/literal_evaluator_spec.rb index 425fdf1d8ad..c9e8d661c9c 100644 --- a/spec/unit/pops/evaluator/literal_evaluator_spec.rb +++ b/spec/unit/pops/evaluator/literal_evaluator_spec.rb @@ -16,6 +16,8 @@ '"a"' => 'a', 'a' => 'a', 'a::b' => 'a::b', + 'Integer[1]' => [1], + 'File' => "file", # special values 'default' => :default, @@ -35,9 +37,9 @@ expect(leval.literal(parser.parse_string('undef'))).to be_nil end - ['1+1', 'File', '[1,2, 1+2]', '{a=>1+1}', 'Integer[1]', '"x$y"', '"x${y}z"'].each do |source| + ['1+1', '[1,2, 1+2]', '{a=>1+1}', '"x$y"', '"x${y}z"', 'Integer[1-3]', 'Optional[[String]]'].each do |source| it "throws :not_literal for non literal expression '#{source}'" do - expect{leval.literal(parser.parse_string('1+1'))}.to throw_symbol(:not_literal) + expect{leval.literal(parser.parse_string(source))}.to throw_symbol(:not_literal) end end end