puppet 8.5.0 → 8.6.0

data/lib/puppet/parser/script_compiler.rb

@@ -118,6 +118,6 @@ class Puppet::Parser::ScriptCompiler
   def evaluate_main
     @loaders.pre_load
     program = @loaders.load_main_manifest
-    return program.nil? ? nil : Puppet::Pops::Parser::EvaluatingParser.singleton.evaluator.evaluate(program, @topscope)
+    program.nil? ? nil : Puppet::Pops::Parser::EvaluatingParser.singleton.evaluator.evaluate(program, @topscope)
   end
 end

data/lib/puppet/parser/templatewrapper.rb

@@ -100,6 +100,6 @@ class Puppet::Parser::TemplateWrapper
   end
 
   def to_s
-    "template[#{(@__file__ ? @__file__ : "inline")}]"
+    "template[#{@__file__ || 'inline'}]"
   end
 end

data/lib/puppet/parser/type_loader.rb

@@ -67,27 +67,25 @@ class Puppet::Parser::TypeLoader
     return nil if fqname == "" # special-case main.
 
     files_to_try_for(fqname).each do |filename|
-      begin
-        imported_types = import_from_modules(filename)
-        result = imported_types.find { |t| t.type == type and t.name == fqname }
-        if result
-          Puppet.debug { "Automatically imported #{fqname} from #{filename} into #{environment}" }
-          return result
-        end
-      rescue TypeLoaderError
-        # I'm not convinced we should just drop these errors, but this
-        # preserves existing behaviours.
+      imported_types = import_from_modules(filename)
+      result = imported_types.find { |t| t.type == type and t.name == fqname }
+      if result
+        Puppet.debug { "Automatically imported #{fqname} from #{filename} into #{environment}" }
+        return result
       end
+    rescue TypeLoaderError
+      # I'm not convinced we should just drop these errors, but this
+      # preserves existing behaviours.
     end
     # Nothing found.
-    return nil
+    nil
   end
 
   def parse_file(file)
     Puppet.debug { "importing '#{file}' in environment #{environment}" }
     parser = Puppet::Parser::ParserFactory.parser
     parser.file = file
-    return parser.parse
+    parser.parse
   end
 
   private

data/lib/puppet/parser.rb

@@ -16,7 +16,7 @@ require_relative 'parser/relationship'
 require_relative '../puppet/resource/type'
 require 'monitor'
 
-require_relative 'parser/compiler/catalog_validator/relationship_validator.rb'
+require_relative 'parser/compiler/catalog_validator/relationship_validator'
 
 # PUP-3274 This should probably go someplace else
 class Puppet::LexError < RuntimeError; end

data/lib/puppet/pops/adaptable.rb

@@ -182,7 +182,7 @@ module Adaptable
     # Returns the name of the class, or the name of the type if the class represents an Object type
     # @return [String] the name of the class or type
     def self.type_name
-      self.name
+      name
     end
 
     # Returns a suitable instance variable name for the _name_ of this instance. The name is created by calling

data/lib/puppet/pops/evaluator/access_operator.rb

@@ -67,7 +67,7 @@ class AccessOperator
                k2 = k2 < 0 ? o.length - k1 + k2 + 1 : k2  # abs length (negative k2 is length from pos to end count)
                # if k1 is outside, adjust to first position, and adjust length
                if k1 < 0
-                 k2 = k2 + k1
+                 k2 += k1
                  k1 = 0
                end
                o[k1, k2]
@@ -116,7 +116,7 @@ class AccessOperator
       k2 = k2 < 0 ? o.length - k1 + k2 + 1 : k2  # abs length (negative k2 is length from pos to end count)
       # if k1 is outside, adjust to first position, and adjust length
       if k1 < 0
-        k2 = k2 + k1
+        k2 += k1
         k1 = 0
       end
       # Help ruby always return empty array when asking for a sub array
@@ -448,7 +448,7 @@ class AccessOperator
 
     keys.each_with_index do |x, index|
       fail(Issues::BAD_INTEGER_SLICE_TYPE, @semantic.keys[index],
-           { :actual => x.class }) unless (x.is_a?(Integer) || x == :default)
+           { :actual => x.class }) unless x.is_a?(Integer) || x == :default
     end
     Types::PIntegerType.new(*keys)
   end
@@ -461,7 +461,7 @@ class AccessOperator
 
     keys.each_with_index do |x, index|
       fail(Issues::BAD_FLOAT_SLICE_TYPE, @semantic.keys[index],
-           { :actual => x.class }) unless (x.is_a?(Float) || x.is_a?(Integer) || x == :default)
+           { :actual => x.class }) unless x.is_a?(Float) || x.is_a?(Integer) || x == :default
     end
     from, to = keys
     from = from == :default || from.nil? ? nil : Float(from)
@@ -556,7 +556,7 @@ class AccessOperator
     else
       keys.each_with_index do |x, index|
         fail(Issues::BAD_COLLECTION_SLICE_TYPE, @semantic.keys[start_index + index],
-             { :actual => x.class }) unless (x.is_a?(Integer) || x == :default)
+             { :actual => x.class }) unless x.is_a?(Integer) || x == :default
       end
       Types::PIntegerType.new(*keys)
     end
@@ -635,7 +635,7 @@ class AccessOperator
       return result_type_array ? [] : nil
     end
 
-    if !o.title.nil?
+    unless o.title.nil?
       # lookup resource and return one or more parameter values
       resource = find_resource(scope, o.type_name, o.title)
       unless resource
@@ -665,7 +665,7 @@ class AccessOperator
       Types::PResourceType.new(type_name, t == :no_title ? nil : t)
     end
     # returns single type if request was for a single entity, else an array of types (possibly empty)
-    return result_type_array ? result : result.pop
+    result_type_array ? result : result.pop
   end
 
   NS = '::'
@@ -725,7 +725,7 @@ class AccessOperator
     end
 
     # returns single type as type, else an array of types
-    return result_type_array ? result : result.pop
+    result_type_array ? result : result.pop
   end
 end
 end

data/lib/puppet/pops/evaluator/callable_signature.rb

@@ -22,7 +22,7 @@ class Puppet::Pops::Evaluator::CallableSignature
   # @api public
   #
   def parameter_names
-    raise NotImplementedError.new
+    raise NotImplementedError
   end
 
   # Returns a PCallableType with the type information, required and optional count, and type information about
@@ -34,7 +34,7 @@ class Puppet::Pops::Evaluator::CallableSignature
   # @api public
   #
   def type
-    raise NotImplementedError.new
+    raise NotImplementedError
   end
 
   # Returns the expected type for an optional block. The type may be nil, which means that the callable does
@@ -56,7 +56,7 @@ class Puppet::Pops::Evaluator::CallableSignature
   # @api public
   #
   def block_name
-    raise NotImplementedError.new
+    raise NotImplementedError
   end
 
   # Returns a range indicating the optionality of a block. One of [0,0] (does not accept block), [0,1] (optional
@@ -87,7 +87,7 @@ class Puppet::Pops::Evaluator::CallableSignature
   # @api public
   #
   def last_captures_rest?
-    raise NotImplementedError.new
+    raise NotImplementedError
   end
 
   # Returns true if the given x is infinity

data/lib/puppet/pops/evaluator/collector_transformer.rb

@@ -26,7 +26,7 @@ class CollectorTransformer
 
     fail "Resource type #{type} doesn't exist" unless resource_type
 
-    if !o.operations.empty?
+    unless o.operations.empty?
       overrides = {
         :parameters => o.operations.map { |x| @@evaluator.evaluate(x, scope) }.flatten,
         :file => o.file,
@@ -182,19 +182,19 @@ class CollectorTransformer
   def match_AndExpression(o, scope)
     left_match = match(o.left_expr, scope)
     right_match = match(o.right_expr, scope)
-    return [left_match, 'and', right_match]
+    [left_match, 'and', right_match]
   end
 
   def match_OrExpression(o, scope)
     left_match = match(o.left_expr, scope)
     right_match = match(o.right_expr, scope)
-    return [left_match, 'or', right_match]
+    [left_match, 'or', right_match]
   end
 
   def match_ComparisonExpression(o, scope)
     left_match = match(o.left_expr, scope)
     right_match = match(o.right_expr, scope)
-    return [left_match, o.operator.to_s, right_match]
+    [left_match, o.operator.to_s, right_match]
   end
 
   def match_VariableExpression(o, scope)

data/lib/puppet/pops/evaluator/collectors/abstract_collector.rb

@@ -28,7 +28,7 @@ class Puppet::Pops::Evaluator::Collectors::AbstractCollector
     @collected = {}
     @scope = scope
 
-    if !(overrides.nil? || overrides[:parameters])
+    unless overrides.nil? || overrides[:parameters]
       raise ArgumentError, _("Exported resource try to override without parameters")
     end
 
@@ -50,12 +50,12 @@ class Puppet::Pops::Evaluator::Collectors::AbstractCollector
       overrides[:source].override = true
 
       objects.each do |res|
-        unless @collected.include?(res.ref)
-          t = res.type
-          t = Puppet::Pops::Evaluator::Runtime3ResourceSupport.find_resource_type(scope, t)
-          newres = Puppet::Parser::Resource.new(t, res.title, @overrides)
-          scope.compiler.add_override(newres)
-        end
+        next if @collected.include?(res.ref)
+
+        t = res.type
+        t = Puppet::Pops::Evaluator::Runtime3ResourceSupport.find_resource_type(scope, t)
+        newres = Puppet::Parser::Resource.new(t, res.title, @overrides)
+        scope.compiler.add_override(newres)
       end
     end
 

data/lib/puppet/pops/evaluator/collectors/fixed_set_collector.rb

@@ -18,14 +18,14 @@ class Puppet::Pops::Evaluator::Collectors::FixedSetCollector < Puppet::Pops::Eva
     resolved = []
     result = @resources.each_with_object([]) do |ref, memo|
       res = @scope.findresource(ref.to_s)
-      if res
-        res.virtual = false
-        memo << res
-        resolved << ref
-      end
+      next unless res
+
+      res.virtual = false
+      memo << res
+      resolved << ref
     end
 
-    @resources = @resources - resolved
+    @resources -= resolved
 
     @scope.compiler.delete_collection(self) if @resources.empty?
 

data/lib/puppet/pops/evaluator/compare_operator.rb

@@ -50,7 +50,7 @@ class CompareOperator
   def cmp_String(a, b)
     return a.casecmp(b) if b.is_a?(String)
 
-    raise ArgumentError.new(_("A String is not comparable to a non String"))
+    raise ArgumentError, _("A String is not comparable to a non String")
   end
 
   # Equality is case independent.
@@ -67,7 +67,7 @@ class CompareOperator
     when Time::Timespan, Time::Timestamp
       -(b <=> a) # compare other way and invert result
     else
-      raise ArgumentError.new(_("A Numeric is not comparable to non Numeric"))
+      raise ArgumentError, _("A Numeric is not comparable to non Numeric")
     end
   end
 
@@ -97,30 +97,30 @@ class CompareOperator
     if b.is_a?(Symbol)
       a <=> b
     else
-      raise ArgumentError.new(_("Symbol not comparable to non Symbol"))
+      raise ArgumentError, _("Symbol not comparable to non Symbol")
     end
   end
 
   def cmp_Timespan(a, b)
-    raise ArgumentError.new(_('Timespans are only comparable to Timespans, Integers, and Floats')) unless b.is_a?(Time::Timespan) || b.is_a?(Integer) || b.is_a?(Float)
+    raise ArgumentError, _('Timespans are only comparable to Timespans, Integers, and Floats') unless b.is_a?(Time::Timespan) || b.is_a?(Integer) || b.is_a?(Float)
 
     a <=> b
   end
 
   def cmp_Timestamp(a, b)
-    raise ArgumentError.new(_('Timestamps are only comparable to Timestamps, Integers, and Floats')) unless b.is_a?(Time::Timestamp) || b.is_a?(Integer) || b.is_a?(Float)
+    raise ArgumentError, _('Timestamps are only comparable to Timestamps, Integers, and Floats') unless b.is_a?(Time::Timestamp) || b.is_a?(Integer) || b.is_a?(Float)
 
     a <=> b
   end
 
   def cmp_Version(a, b)
-    raise ArgumentError.new(_('Versions not comparable to non Versions')) unless b.is_a?(SemanticPuppet::Version)
+    raise ArgumentError, _('Versions not comparable to non Versions') unless b.is_a?(SemanticPuppet::Version)
 
     a <=> b
   end
 
   def cmp_Object(a, b)
-    raise ArgumentError.new(_('Only Strings, Numbers, Timespans, Timestamps, and Versions are comparable'))
+    raise ArgumentError, _('Only Strings, Numbers, Timespans, Timestamps, and Versions are comparable')
   end
 
   def equals_Object(a, b)
@@ -183,15 +183,15 @@ class CompareOperator
       end
       # Always set match data, a "not found" should not keep old match data visible
       set_match_data(matched, scope) # creates ephemeral
-      return !!matched
+      !!matched
     when String, SemanticPuppet::Version
       a.any? { |element| match(b, element, scope) }
     when Types::PAnyType
       a.each { |element| return true if b.instance?(element) }
-      return false
+      false
     else
       a.each { |element| return true if equals(element, b) }
-      return false
+      false
     end
   end
 

data/lib/puppet/pops/evaluator/deferred_resolver.rb

@@ -172,7 +172,7 @@ class DeferredResolver
     else
       # call the function later
       DeferredValue.new(
-        Proc.new {
+        proc {
           # deferred functions can have nested deferred arguments
           resolved_arguments = mapped_arguments.map { |arg| resolve_lazy_args(arg) }
           @scope.call_function(func_name, resolved_arguments)

data/lib/puppet/pops/evaluator/epp_evaluator.rb

@@ -31,7 +31,7 @@ class Puppet::Pops::Evaluator::EppEvaluator
 
     unless Puppet::FileSystem.exist?(file)
       unless file =~ /\.epp$/
-        file = file + ".epp"
+        file += ".epp"
       end
     end
 
@@ -82,7 +82,7 @@ class Puppet::Pops::Evaluator::EppEvaluator
     end
 
     # filter out all qualified names and set them in qualified_variables
-    # only pass unqualified (filtered) variable names to the the template
+    # only pass unqualified (filtered) variable names to the template
     filtered_args = {}
     template_args.each_pair do |k, v|
       if k =~ /::/

data/lib/puppet/pops/evaluator/evaluator_impl.rb

@@ -74,46 +74,44 @@ class EvaluatorImpl
   # @api public
   #
   def evaluate(target, scope)
-    begin
-      @@eval_visitor.visit_this_1(self, target, scope)
-    rescue SemanticError => e
-      # A raised issue may not know the semantic target, use errors call stack, but fill in the
-      # rest from a supplied semantic object, or the target instruction if there is not semantic
-      # object.
-      #
-      fail(e.issue, e.semantic || target, e.options, e)
-    rescue Puppet::PreformattedError => e
-      # Already formatted with location information, and with the wanted call stack.
-      # Note this is currently a specialized ParseError, so rescue-order is important
-      #
-      raise e
-    rescue Puppet::ParseError => e
-      # ParseError may be raised in ruby code without knowing the location
-      # in puppet code.
-      # Accept a ParseError that has file or line information available
-      # as an error that should be used verbatim. (Tests typically run without
-      # setting a file name).
-      # ParseError can supply an original - it is impossible to determine which
-      # call stack that should be propagated, using the ParseError's backtrace.
-      #
-      if e.file || e.line
-        raise e
-      else
-        # Since it had no location information, treat it as user intended a general purpose
-        # error. Pass on its call stack.
-        fail(Issues::RUNTIME_ERROR, target, { :detail => e.message }, e)
-      end
-    rescue Puppet::Error => e
-      # PuppetError has the ability to wrap an exception, if so, use the wrapped exception's
-      # call stack instead
-      fail(Issues::RUNTIME_ERROR, target, { :detail => e.message }, e.original || e)
-    rescue StopIteration => e
-      # Ensure these are not rescued as StandardError
+    @@eval_visitor.visit_this_1(self, target, scope)
+  rescue SemanticError => e
+    # A raised issue may not know the semantic target, use errors call stack, but fill in the
+    # rest from a supplied semantic object, or the target instruction if there is not semantic
+    # object.
+    #
+    fail(e.issue, e.semantic || target, e.options, e)
+  rescue Puppet::PreformattedError => e
+    # Already formatted with location information, and with the wanted call stack.
+    # Note this is currently a specialized ParseError, so rescue-order is important
+    #
+    raise e
+  rescue Puppet::ParseError => e
+    # ParseError may be raised in ruby code without knowing the location
+    # in puppet code.
+    # Accept a ParseError that has file or line information available
+    # as an error that should be used verbatim. (Tests typically run without
+    # setting a file name).
+    # ParseError can supply an original - it is impossible to determine which
+    # call stack that should be propagated, using the ParseError's backtrace.
+    #
+    if e.file || e.line
       raise e
-    rescue StandardError => e
-      # All other errors, use its message and call stack
+    else
+      # Since it had no location information, treat it as user intended a general purpose
+      # error. Pass on its call stack.
       fail(Issues::RUNTIME_ERROR, target, { :detail => e.message }, e)
     end
+  rescue Puppet::Error => e
+    # PuppetError has the ability to wrap an exception, if so, use the wrapped exception's
+    # call stack instead
+    fail(Issues::RUNTIME_ERROR, target, { :detail => e.message }, e.original || e)
+  rescue StopIteration => e
+    # Ensure these are not rescued as StandardError
+    raise e
+  rescue StandardError => e
+    # All other errors, use its message and call stack
+    fail(Issues::RUNTIME_ERROR, target, { :detail => e.message }, e)
   end
 
   # Assigns the given _value_ to the given _target_. The additional argument _o_ is the instruction that
@@ -693,7 +691,7 @@ class EvaluatorImpl
       the_default = nil
       if o.options.find do |co|
         # the first case option that matches
-        if co.values.find do |c|
+        next unless co.values.find do |c|
           c = unwind_parentheses(c)
           case c
           when Model::LiteralDefault
@@ -707,9 +705,9 @@ class EvaluatorImpl
             is_match?(test, evaluate(c, scope), c, co, scope)
           end
         end
-          result = evaluate(co.then_expr, scope)
-          true # the option was picked
-        end
+
+        result = evaluate(co.then_expr, scope)
+        true # the option was picked
       end
         result # an option was picked, and produced a result
       else
@@ -746,17 +744,15 @@ class EvaluatorImpl
   end
 
   def eval_Program(o, scope)
-    begin
-      file = o.locator.file
-      line = 0
-      # Add stack frame for "top scope" logic. See Puppet::Pops::PuppetStack
-      return Puppet::Pops::PuppetStack.stack(file, line, self, 'evaluate', [o.body, scope])
-      # evaluate(o.body, scope)
-    rescue Puppet::Pops::Evaluator::PuppetStopIteration => ex
-      # breaking out of a file level program is not allowed
-      # TRANSLATOR break() is a method that should not be translated
-      raise Puppet::ParseError.new(_("break() from context where this is illegal"), ex.file, ex.line)
-    end
+    file = o.locator.file
+    line = 0
+    # Add stack frame for "top scope" logic. See Puppet::Pops::PuppetStack
+    Puppet::Pops::PuppetStack.stack(file, line, self, 'evaluate', [o.body, scope])
+    # evaluate(o.body, scope)
+  rescue Puppet::Pops::Evaluator::PuppetStopIteration => ex
+    # breaking out of a file level program is not allowed
+    # TRANSLATOR break() is a method that should not be translated
+    raise Puppet::ParseError.new(_("break() from context where this is illegal"), ex.file, ex.line)
   end
 
   # Produces Array[PAnyType], an array of resource references
@@ -811,7 +807,7 @@ class EvaluatorImpl
 
     # This is a runtime check - the model is valid, but will have runtime issues when evaluated
     # and storeconfigs is not set.
-    if (o.exported)
+    if o.exported
       optionally_fail(Issues::RT_NO_STORECONFIGS_EXPORT, o);
     end
 
@@ -1068,10 +1064,10 @@ class EvaluatorImpl
   # Evaluates Puppet DSL `unless`
   def eval_UnlessExpression o, scope
     scope.with_guarded_scope do
-      unless is_true?(evaluate(o.test, scope), o.test)
-        evaluate(o.then_expr, scope)
-      else
+      if is_true?(evaluate(o.test, scope), o.test)
         evaluate(o.else_expr, scope)
+      else
+        evaluate(o.then_expr, scope)
       end
     end
   end
@@ -1236,15 +1232,15 @@ class EvaluatorImpl
               Hash[*y]
             end
           else
-            raise ArgumentError.new(_('Can only append Array or Hash to a Hash'))
+            raise ArgumentError, _('Can only append Array or Hash to a Hash')
           end
       x.merge y # new hash with overwrite
     when URI
-      raise ArgumentError.new(_('An URI can only be merged with an URI or String')) unless y.is_a?(String) || y.is_a?(URI)
+      raise ArgumentError, _('An URI can only be merged with an URI or String') unless y.is_a?(String) || y.is_a?(URI)
 
       x + y
     when Types::PBinaryType::Binary
-      raise ArgumentError.new(_('Can only append Binary to a Binary')) unless y.is_a?(Types::PBinaryType::Binary)
+      raise ArgumentError, _('Can only append Binary to a Binary') unless y.is_a?(Types::PBinaryType::Binary)
 
       Types::PBinaryType::Binary.from_binary_string(x.binary_buffer + y.binary_buffer)
     else
@@ -1276,7 +1272,7 @@ class EvaluatorImpl
           end
       y.each { |e| result.delete(e) }
     else
-      raise ArgumentError.new(_("Can only delete from an Array or Hash."))
+      raise ArgumentError, _("Can only delete from an Array or Hash.")
     end
     result
   end

data/lib/puppet/pops/evaluator/literal_evaluator.rb

@@ -76,6 +76,10 @@ class LiteralEvaluator
     o.keys.map { |v| literal(v) }
   end
 
+  def literal_UnaryMinusExpression(o)
+    -literal(o.expr)
+  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)

data/lib/puppet/pops/evaluator/relationship_operator.rb

@@ -47,7 +47,7 @@ class RelationshipOperator
   # Catch all non transformable objects
   # @api private
   def transform_Object(o, scope)
-    raise IllegalRelationshipOperandError.new(o)
+    raise IllegalRelationshipOperandError, o
   end
 
   # A Resource is by definition a Catalog type, but of 3.x type
@@ -98,7 +98,7 @@ class RelationshipOperator
   #
   def assert_catalog_type(o, scope)
     unless @type_calculator.assignable?(@catalog_type, o)
-      raise NotCatalogTypeError.new(o)
+      raise NotCatalogTypeError, o
     end
 
     # TODO must check if this is an abstract PResourceType (i.e. without a type_name) - which should fail ?

data/lib/puppet/pops/evaluator/runtime3_converter.rb

@@ -70,7 +70,7 @@ class Runtime3Converter
     return o unless o < MIN_INTEGER || o > MAX_INTEGER
 
     range_end = o > MAX_INTEGER ? 'max' : 'min'
-    raise Puppet::Error, "Use of a Ruby Integer outside of Puppet Integer #{range_end} range, got '#{"0x%x" % o}'"
+    raise Puppet::Error, "Use of a Ruby Integer outside of Puppet Integer #{range_end} range, got '#{'0x%x' % o}'"
   end
 
   def convert_BigDecimal(o, scope, undef_value)
@@ -165,7 +165,7 @@ class Runtime3Converter
     @convert_visitor = Puppet::Pops::Visitor.new(self, 'convert', 2, 2)
   end
 
-  @instance = self.new
+  @instance = new
 end
 
 # A Ruby function written for the 3.x API cannot be expected to handle extended data types. This
@@ -220,6 +220,6 @@ class Runtime3FunctionArgumentConverter < Runtime3Converter
     end
   end
 
-  @instance = self.new
+  @instance = new
 end
 end

data/lib/puppet/pops/evaluator/runtime3_support.rb

@@ -37,7 +37,7 @@ module Runtime3Support
     if except.nil? && diagnostic_producer.severity_producer[issue] == :error
       # Want a stacktrace, and it must be passed as an exception
       begin
-        raise EvaluationError.new()
+        raise EvaluationError
       rescue EvaluationError => e
         except = e
       end

data/lib/puppet/pops/functions/dispatcher.rb

@@ -62,15 +62,15 @@ class Puppet::Pops::Functions::Dispatcher
     # make a copy to make sure it can be contained by someone else (even if it is not contained here, it
     # should be treated as immutable).
     #
-    callables = dispatchers.map { |dispatch| dispatch.type }
+    callables = dispatchers.map(&:type)
 
     # multiple signatures, produce a Variant type of Callable1-n (must copy them)
     # single signature, produce single Callable
-    callables.size > 1 ?  Puppet::Pops::Types::TypeFactory.variant(*callables) : callables.pop
+    callables.size > 1 ? Puppet::Pops::Types::TypeFactory.variant(*callables) : callables.pop
   end
 
   # @api private
   def signatures
-    @dispatchers.reject { |dispatcher| dispatcher.argument_mismatch_handler? }
+    @dispatchers.reject(&:argument_mismatch_handler?)
   end
 end