rspec-hiera-puppet 0.3.0 → 1.0.0

data/README.md

@@ -106,7 +106,6 @@ demonstrates the use of a shared context.
 
   _spec/spec\_helper.rb_
 
-    require 'rspec-hiera'
     require 'rspec-hiera-puppet'
 
     fixture_path = File.expand_path(File.join(__FILE__, '..', 'fixtures'))

data/lib/hiera/backend/rspec_backend.rb

@@ -6,7 +6,7 @@ class Hiera
       end
 
       def lookup(key, scope, order_override, resolution_type)
-        answer = Backend.empty_answer(resolution_type)
+        answer = nil
 
         Hiera.debug("Looking up #{key} in RSpec backend")
 
@@ -26,6 +26,12 @@ class Hiera
             next
           end
 
+          # Extra logging that we found the key. This can be outputted
+          # multiple times if the resolution type is array or hash but that
+          # should be expected as the logging will then tell the user ALL the
+          # places where the key is found.
+          Hiera.debug("Found #{key} in #{source}")
+
           # for array resolution we just append to the array whatever
           # we find, we then goes onto the next file and keep adding to
           # the array
@@ -35,9 +41,11 @@ class Hiera
           case resolution_type
           when :array
             raise Exception, "Hiera type mismatch: expected Array and got #{new_answer.class}" unless new_answer.kind_of? Array or new_answer.kind_of? String
+            answer ||= []
             answer << new_answer
           when :hash
             raise Exception, "Hiera type mismatch: expected Hash and got #{new_answer.class}" unless new_answer.kind_of? Hash
+            answer ||= {}
             answer = new_answer.merge answer
           else
             answer = new_answer

data/lib/rspec-hiera-puppet/puppet.rb

@@ -18,126 +18,33 @@ class Puppet::Parser::Compiler
 
   def register_function_hiera(spec)
     Puppet::Parser::Functions.newfunction(:hiera, :type => :rvalue) do |*args|
-      # Functions called from puppet manifests that look like this:
-      #   lookup("foo", "bar")
-      # internally in puppet are invoked:  func(["foo", "bar"])
-      #
-      # where as calling from templates should work like this:
-      #   scope.function_lookup("foo", "bar")
-      #
-      #  Therefore, declare this function with args '*args' to accept any number
-      #  of arguments and deal with puppet's special calling mechanism now:
-      if args[0].is_a?(Array)
-        args = args[0]
-      end
-
-      key = args[0]
-      default = args[1]
-      override = args[2]
-
-      require 'hiera'
-      require 'hiera/scope'
-
-      hiera = Hiera.new(:config => spec.hiera_config.merge(:logger => 'puppet'))
-
-      if self.respond_to?("[]")
-        hiera_scope = self
-      else
-        hiera_scope = Hiera::Scope.new(self)
-      end
-
-      answer = hiera.lookup(key, default, hiera_scope, override, :priority)
-
-      raise(Puppet::ParseError, "Could not find data item #{key} in any Hiera data file and no default supplied") if answer.nil?
-
-      return answer
+      require 'hiera_puppet'
+      key, default, override = HieraPuppet.parse_args(args)
+      HieraPuppet.lookup(key, default, self, override, :priority)
     end
   end
 
   def register_function_hiera_array(spec)
     Puppet::Parser::Functions.newfunction(:hiera_array, :type => :rvalue) do |*args|
-      if args[0].is_a?(Array)
-        args = args[0]
-      end
-
-      key = args[0]
-      default = args[1]
-      override = args[2]
-
-      require 'hiera'
-      require 'hiera/scope'
-
-      hiera = Hiera.new(:config => spec.hiera_config.merge(:logger => 'puppet'))
-
-      if self.respond_to?("[]")
-        hiera_scope = self
-      else
-        hiera_scope = Hiera::Scope.new(self)
-      end
-
-      answer = hiera.lookup(key, default, hiera_scope, override, :array)
-
-      raise(Puppet::ParseError, "Could not find data item #{key} in any Hiera data file and no default supplied") if answer.empty?
-
-      answer
+      require 'hiera_puppet'
+      key, default, override = HieraPuppet.parse_args(args)
+      HieraPuppet.lookup(key, default, self, override, :array)
     end
   end
 
   def register_function_hiera_hash(spec)
     Puppet::Parser::Functions.newfunction(:hiera_hash, :type => :rvalue) do |*args|
-      if args[0].is_a?(Array)
-        args = args[0]
-      end
-
-      raise(Puppet::ParseError, "Please supply a parameter to perform a Hiera lookup") if args.empty?
-
-      key = args[0]
-      default = args[1]
-      override = args[2]
-
-      require 'hiera'
-      require 'hiera/scope'
-
-      hiera = Hiera.new(:config => spec.hiera_config.merge(:logger => 'puppet'))
-
-      if self.respond_to?("{}")
-        hiera_scope = self
-      else
-        hiera_scope = Hiera::Scope.new(self)
-      end
-
-      answer = hiera.lookup(key, default, hiera_scope, override, :hash)
-
-      raise(Puppet::ParseError, "Could not find data item #{key} in any Hiera data file and no default supplied") if answer.empty?
-
-      answer
+      require 'hiera_puppet'
+      key, default, override = HieraPuppet.parse_args(args)
+      HieraPuppet.lookup(key, default, self, override, :hash)
     end
   end
 
   def register_function_hiera_include(spec)
     Puppet::Parser::Functions.newfunction(:hiera_include) do |*args|
-      if args[0].is_a?(Array)
-        args = args[0]
-      end
-
-      key = args[0]
-      default = args[1]
-      override = args[2]
-
-      require 'hiera'
-      require 'hiera/scope'
-
-      hiera = Hiera.new(:config => spec.hiera_config.merge(:logger => 'puppet'))
-
-      if self.respond_to?("[]")
-        hiera_scope = self
-      else
-        hiera_scope = Hiera::Scope.new(self)
-      end
-
-      answer = hiera.lookup(key, default, hiera_scope, override, :array)
-
-      raise(Puppet::ParseError, "Could not find data item #{key} in any Hiera data file and no default supplied") if answer.empty?
+      require 'hiera_puppet'
+      key, default, override = HieraPuppet.parse_args(args)
+      answer = HieraPuppet.lookup(key, default, self, override, :array)
 
       method = Puppet::Parser::Functions.function(:include)
       send(method, answer)

data/lib/rspec-hiera-puppet/version.rb

@@ -1,7 +1,7 @@
 module Rspec
   module Hiera
     module Puppet
-      VERSION = "0.3.0"
+      VERSION = "1.0.0"
     end
   end
 end

data/rspec-hiera-puppet.gemspec

@@ -15,9 +15,9 @@ Gem::Specification.new do |gem|
   gem.require_paths = ["lib"]
   gem.version       = Rspec::Hiera::Puppet::VERSION
 
-  gem.add_dependency('puppet', '>= 2.7')
-  gem.add_dependency('hiera', '>= 0.3')
-  gem.add_dependency('hiera-puppet', '>= 0.3')
+  gem.add_dependency('puppet', '>= 3.0')
+  gem.add_dependency('hiera', '>= 1.0')
+  gem.add_dependency('hiera-puppet', '>= 1.0')
   gem.add_dependency('rspec')
   gem.add_dependency('rspec-puppet')
 end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: rspec-hiera-puppet
 version: !ruby/object:Gem::Version 
-  hash: 19
+  hash: 23
   prerelease: 
   segments: 
+  - 1
   - 0
-  - 3
   - 0
-  version: 0.3.0
+  version: 1.0.0
 platform: ruby
 authors: 
 - Michael Franz Aigner
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2012-08-05 00:00:00 Z
+date: 2012-10-19 00:00:00 Z
 dependencies: 
 - !ruby/object:Gem::Dependency 
   name: puppet
@@ -25,11 +25,11 @@ dependencies:
     requirements: 
     - - ">="
       - !ruby/object:Gem::Version 
-        hash: 13
+        hash: 7
         segments: 
-        - 2
-        - 7
-        version: "2.7"
+        - 3
+        - 0
+        version: "3.0"
   type: :runtime
   version_requirements: *id001
 - !ruby/object:Gem::Dependency 
@@ -40,11 +40,11 @@ dependencies:
     requirements: 
     - - ">="
       - !ruby/object:Gem::Version 
-        hash: 13
+        hash: 15
         segments: 
+        - 1
         - 0
-        - 3
-        version: "0.3"
+        version: "1.0"
   type: :runtime
   version_requirements: *id002
 - !ruby/object:Gem::Dependency 
@@ -55,11 +55,11 @@ dependencies:
     requirements: 
     - - ">="
       - !ruby/object:Gem::Version 
-        hash: 13
+        hash: 15
         segments: 
+        - 1
         - 0
-        - 3
-        version: "0.3"
+        version: "1.0"
   type: :runtime
   version_requirements: *id003
 - !ruby/object:Gem::Dependency