Make ensure_packages an InternalFunction

To preserve original behaviour of the created package(s) being contained
within the class that called `ensure_packages`, the function needs to be
an `InternalFunction` and call `ensure_resource(s)` with the correct
scope.

Author: alexjfisher

lib/puppet/functions/ensure_packages.rb

@@ -4,19 +4,20 @@
 #
 # It optionally takes a hash as a second parameter that will be passed as the
 # third argument to the ensure_resource() function.
-Puppet::Functions.create_function(:ensure_packages) do
+Puppet::Functions.create_function(:ensure_packages, Puppet::Functions::InternalFunction) do
   # @param packages
   #   The packages to ensure are installed. If it's a Hash it will be passed to `ensure_resource`
   # @param default_attributes
   #   Default attributes to be passed to the `ensure_resource()` function
   # @return [Undef] Returns nothing.
   dispatch :ensure_packages do
+    scope_param
     param 'Variant[String[1], Array[String[1]], Hash[String[1], Any]]', :packages
     optional_param 'Hash', :default_attributes
     return_type 'Undef'
   end
 
-  def ensure_packages(packages, default_attributes = nil)
+  def ensure_packages(scope, packages, default_attributes = nil)
     if default_attributes
       defaults = { 'ensure' => 'installed' }.merge(default_attributes)
       if defaults['ensure'] == 'present'
@@ -27,10 +28,10 @@ def ensure_packages(packages, default_attributes = nil)
     end
 
     if packages.is_a?(Hash)
-      call_function('ensure_resources', 'package', packages.dup, defaults)
+      scope.call_function('ensure_resources', ['package', packages.dup, defaults])
     else
       Array(packages).each do |package_name|
-        call_function('ensure_resource', 'package', package_name, defaults)
+        scope.call_function('ensure_resource', ['package', package_name, defaults])
       end
     end
     nil