cheffish 1.5.0 → 1.6.0

data/lib/cheffish/rspec/recipe_run_wrapper.rb

@@ -1,78 +1,78 @@
-require 'cheffish/chef_run'
-require 'forwardable'
-
-module Cheffish
-  module RSpec
-    class RecipeRunWrapper < ChefRun
-      def initialize(chef_config, example: nil, &recipe)
-        super(chef_config)
-        @recipe = recipe
-        @example = example || recipe.binding.eval('self')
-      end
-
-      attr_reader :recipe
-      attr_reader :example
-
-      def client
-        if !@client
-          super
-          example = self.example
-
-          #
-          # Support for both resources and rspec example's let variables:
-          #
-          # In 12.4, the elimination of a bunch of metaprogramming in 12.4
-          # changed how Chef DSL is defined in code: resource methods are now
-          # explicitly defined in `Chef::DSL::Recipe`. In 12.3, no actual
-          # methods were defined and `respond_to?(:file)` would return false.
-          # If we reach `method_missing` here, it means that we either have a
-          # 12.3-ish resource or we want to call a `let` variable.
-          #
-          @client.instance_eval { @rspec_example = example }
-          def @client.method_missing(name, *args, &block)
-            # If there is a let variable, call it. This is because in 12.4,
-            # the parent class is going to call respond_to?(name) to find out
-            # if someone was doing weird things, and then call send(). This
-            # would result in an infinite loop, coming right. Back. Here.
-            # A fix to chef is incoming, but we still need this if we want to
-            # work with Chef 12.4.
-            if Gem::Version.new(Chef::VERSION) >= Gem::Version.new('12.4')
-              if @rspec_example.respond_to?(name)
-                return @rspec_example.public_send(name, *args, &block)
-              end
-            end
-
-            # In 12.3 or below, method_missing was the only way to call
-            # resources. If we are in 12.4, we still need to call the crazy
-            # method_missing metaprogramming because backcompat.
-            begin
-              super
-            rescue NameError
-              if @rspec_example.respond_to?(name)
-                @rspec_example.public_send(name, *args, &block)
-              else
-                raise
-              end
-            end
-          end
-          # This is called by respond_to?, and is required to make sure the
-          # resource knows that we will in fact call the given method.
-          def @client.respond_to_missing?(name, include_private = false)
-            @rspec_example.respond_to?(name, include_private) || super
-          end
-
-          # Respond true to is_a?(Chef::Provider) so that Chef::Recipe::DSL.build_resource
-          # will hook resources up to the example let variables as well (via
-          # enclosing_provider).
-          # Please don't hurt me
-          def @client.is_a?(klass)
-            klass == Chef::Provider || super(klass)
-          end
-
-          @client.load_block(&recipe)
-        end
-        @client
-      end
-    end
-  end
-end
+require 'cheffish/chef_run'
+require 'forwardable'
+
+module Cheffish
+  module RSpec
+    class RecipeRunWrapper < ChefRun
+      def initialize(chef_config, example: nil, &recipe)
+        super(chef_config)
+        @recipe = recipe
+        @example = example || recipe.binding.eval('self')
+      end
+
+      attr_reader :recipe
+      attr_reader :example
+
+      def client
+        if !@client
+          super
+          example = self.example
+
+          #
+          # Support for both resources and rspec example's let variables:
+          #
+          # In 12.4, the elimination of a bunch of metaprogramming in 12.4
+          # changed how Chef DSL is defined in code: resource methods are now
+          # explicitly defined in `Chef::DSL::Recipe`. In 12.3, no actual
+          # methods were defined and `respond_to?(:file)` would return false.
+          # If we reach `method_missing` here, it means that we either have a
+          # 12.3-ish resource or we want to call a `let` variable.
+          #
+          @client.instance_eval { @rspec_example = example }
+          def @client.method_missing(name, *args, &block)
+            # If there is a let variable, call it. This is because in 12.4,
+            # the parent class is going to call respond_to?(name) to find out
+            # if someone was doing weird things, and then call send(). This
+            # would result in an infinite loop, coming right. Back. Here.
+            # A fix to chef is incoming, but we still need this if we want to
+            # work with Chef 12.4.
+            if Gem::Version.new(Chef::VERSION) >= Gem::Version.new('12.4')
+              if @rspec_example.respond_to?(name)
+                return @rspec_example.public_send(name, *args, &block)
+              end
+            end
+
+            # In 12.3 or below, method_missing was the only way to call
+            # resources. If we are in 12.4, we still need to call the crazy
+            # method_missing metaprogramming because backcompat.
+            begin
+              super
+            rescue NameError
+              if @rspec_example.respond_to?(name)
+                @rspec_example.public_send(name, *args, &block)
+              else
+                raise
+              end
+            end
+          end
+          # This is called by respond_to?, and is required to make sure the
+          # resource knows that we will in fact call the given method.
+          def @client.respond_to_missing?(name, include_private = false)
+            @rspec_example.respond_to?(name, include_private) || super
+          end
+
+          # Respond true to is_a?(Chef::Provider) so that Chef::Recipe::DSL.build_resource
+          # will hook resources up to the example let variables as well (via
+          # enclosing_provider).
+          # Please don't hurt me
+          def @client.is_a?(klass)
+            klass == Chef::Provider || super(klass)
+          end
+
+          @client.load_block(&recipe)
+        end
+        @client
+      end
+    end
+  end
+end

data/lib/cheffish/rspec/repository_support.rb

@@ -1,108 +1,108 @@
-module Cheffish
-  module RSpec
-    module RepositorySupport
-      def when_the_repository(desc, *tags, &block)
-        context("when the chef repo #{desc}", *tags) do
-          include_context "with a chef repo"
-          extend WhenTheRepositoryClassMethods
-          module_eval(&block)
-        end
-      end
-
-      ::RSpec.shared_context "with a chef repo" do
-        before :each do
-          raise "Can only create one directory per test" if @repository_dir
-          @repository_dir = Dir.mktmpdir('chef_repo')
-          Chef::Config.chef_repo_path = @repository_dir
-          %w(client cookbook data_bag environment node role user).each do |object_name|
-            Chef::Config.delete("#{object_name}_path".to_sym)
-          end
-        end
-
-        after :each do
-          if @repository_dir
-            begin
-              %w(client cookbook data_bag environment node role user).each do |object_name|
-                Chef::Config.delete("#{object_name}_path".to_sym)
-              end
-              Chef::Config.delete(:chef_repo_path)
-              FileUtils.remove_entry_secure(@repository_dir)
-            ensure
-              @repository_dir = nil
-            end
-          end
-          Dir.chdir(@old_cwd) if @old_cwd
-        end
-
-        def directory(relative_path, &block)
-          old_parent_path = @parent_path
-          @parent_path = path_to(relative_path)
-          FileUtils.mkdir_p(@parent_path)
-          instance_eval(&block) if block
-          @parent_path = old_parent_path
-        end
-
-        def file(relative_path, contents)
-          filename = path_to(relative_path)
-          dir = File.dirname(filename)
-          FileUtils.mkdir_p(dir) unless dir == '.'
-          File.open(filename, 'w') do |file|
-            raw = case contents
-                  when Hash
-                    JSON.pretty_generate(contents)
-                  when Array
-                    contents.join("\n")
-                  else
-                    contents
-                  end
-            file.write(raw)
-          end
-        end
-
-        def symlink(relative_path, relative_dest)
-          filename = path_to(relative_path)
-          dir = File.dirname(filename)
-          FileUtils.mkdir_p(dir) unless dir == '.'
-          dest_filename = path_to(relative_dest)
-          File.symlink(dest_filename, filename)
-        end
-
-        def path_to(relative_path)
-          File.expand_path(relative_path, (@parent_path || @repository_dir))
-        end
-
-        def cwd(relative_path)
-          @old_cwd = Dir.pwd
-          Dir.chdir(path_to(relative_path))
-        end
-
-        module WhenTheRepositoryClassMethods
-          def directory(*args, &block)
-            before :each do
-              directory(*args, &block)
-            end
-          end
-
-          def file(*args, &block)
-            before :each do
-              file(*args, &block)
-            end
-          end
-
-          def symlink(*args, &block)
-            before :each do
-              symlink(*args, &block)
-            end
-          end
-
-          def path_to(*args, &block)
-            before :each do
-              file(*args, &block)
-            end
-          end
-        end
-      end
-    end
-
-  end
-end
+module Cheffish
+  module RSpec
+    module RepositorySupport
+      def when_the_repository(desc, *tags, &block)
+        context("when the chef repo #{desc}", *tags) do
+          include_context "with a chef repo"
+          extend WhenTheRepositoryClassMethods
+          module_eval(&block)
+        end
+      end
+
+      ::RSpec.shared_context "with a chef repo" do
+        before :each do
+          raise "Can only create one directory per test" if @repository_dir
+          @repository_dir = Dir.mktmpdir('chef_repo')
+          Chef::Config.chef_repo_path = @repository_dir
+          %w(client cookbook data_bag environment node role user).each do |object_name|
+            Chef::Config.delete("#{object_name}_path".to_sym)
+          end
+        end
+
+        after :each do
+          if @repository_dir
+            begin
+              %w(client cookbook data_bag environment node role user).each do |object_name|
+                Chef::Config.delete("#{object_name}_path".to_sym)
+              end
+              Chef::Config.delete(:chef_repo_path)
+              FileUtils.remove_entry_secure(@repository_dir)
+            ensure
+              @repository_dir = nil
+            end
+          end
+          Dir.chdir(@old_cwd) if @old_cwd
+        end
+
+        def directory(relative_path, &block)
+          old_parent_path = @parent_path
+          @parent_path = path_to(relative_path)
+          FileUtils.mkdir_p(@parent_path)
+          instance_eval(&block) if block
+          @parent_path = old_parent_path
+        end
+
+        def file(relative_path, contents)
+          filename = path_to(relative_path)
+          dir = File.dirname(filename)
+          FileUtils.mkdir_p(dir) unless dir == '.'
+          File.open(filename, 'w') do |file|
+            raw = case contents
+                  when Hash
+                    JSON.pretty_generate(contents)
+                  when Array
+                    contents.join("\n")
+                  else
+                    contents
+                  end
+            file.write(raw)
+          end
+        end
+
+        def symlink(relative_path, relative_dest)
+          filename = path_to(relative_path)
+          dir = File.dirname(filename)
+          FileUtils.mkdir_p(dir) unless dir == '.'
+          dest_filename = path_to(relative_dest)
+          File.symlink(dest_filename, filename)
+        end
+
+        def path_to(relative_path)
+          File.expand_path(relative_path, (@parent_path || @repository_dir))
+        end
+
+        def cwd(relative_path)
+          @old_cwd = Dir.pwd
+          Dir.chdir(path_to(relative_path))
+        end
+
+        module WhenTheRepositoryClassMethods
+          def directory(*args, &block)
+            before :each do
+              directory(*args, &block)
+            end
+          end
+
+          def file(*args, &block)
+            before :each do
+              file(*args, &block)
+            end
+          end
+
+          def symlink(*args, &block)
+            before :each do
+              symlink(*args, &block)
+            end
+          end
+
+          def path_to(*args, &block)
+            before :each do
+              file(*args, &block)
+            end
+          end
+        end
+      end
+    end
+
+  end
+end

data/lib/cheffish/server_api.rb

@@ -1,52 +1,52 @@
-#
-# Author:: John Keiser (<jkeiser@opscode.com>)
-# Copyright:: Copyright (c) 2012 Opscode, Inc.
-# License:: Apache License, Version 2.0
-#
-# Licensed under the Apache License, Version 2.0 (the "License");
-# you may not use this file except in compliance with the License.
-# You may obtain a copy of the License at
-#
-#     http://www.apache.org/licenses/LICENSE-2.0
-#
-# Unless required by applicable law or agreed to in writing, software
-# distributed under the License is distributed on an "AS IS" BASIS,
-# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-# See the License for the specific language governing permissions and
-# limitations under the License.
-#
-
-require 'chef/version'
-require 'chef/http'
-require 'chef/http/authenticator'
-require 'chef/http/cookie_manager'
-require 'chef/http/decompressor'
-require 'chef/http/json_input'
-require 'chef/http/json_output'
-if Gem::Version.new(Chef::VERSION) >= Gem::Version.new('11.12')
-  require 'chef/http/remote_request_id'
-end
-
-module Cheffish
-  # Exactly like Chef::ServerAPI, but requires you to pass in what keys you want (no defaults)
-  class ServerAPI < Chef::HTTP
-
-    def initialize(url, options = {})
-      super(url, options)
-      root_url = URI.parse(url)
-      root_url.path = ''
-      @root_url = root_url.to_s
-    end
-
-    attr_reader :root_url
-
-    use Chef::HTTP::JSONInput
-    use Chef::HTTP::JSONOutput
-    use Chef::HTTP::CookieManager
-    use Chef::HTTP::Decompressor
-    use Chef::HTTP::Authenticator
-    if Gem::Version.new(Chef::VERSION) >= Gem::Version.new('11.12')
-      use Chef::HTTP::RemoteRequestID
-    end
-  end
-end
+#
+# Author:: John Keiser (<jkeiser@opscode.com>)
+# Copyright:: Copyright (c) 2012 Opscode, Inc.
+# License:: Apache License, Version 2.0
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+#
+
+require 'chef/version'
+require 'chef/http'
+require 'chef/http/authenticator'
+require 'chef/http/cookie_manager'
+require 'chef/http/decompressor'
+require 'chef/http/json_input'
+require 'chef/http/json_output'
+if Gem::Version.new(Chef::VERSION) >= Gem::Version.new('11.12')
+  require 'chef/http/remote_request_id'
+end
+
+module Cheffish
+  # Exactly like Chef::ServerAPI, but requires you to pass in what keys you want (no defaults)
+  class ServerAPI < Chef::HTTP
+
+    def initialize(url, options = {})
+      super(url, options)
+      root_url = URI.parse(url)
+      root_url.path = ''
+      @root_url = root_url.to_s
+    end
+
+    attr_reader :root_url
+
+    use Chef::HTTP::JSONInput
+    use Chef::HTTP::JSONOutput
+    use Chef::HTTP::CookieManager
+    use Chef::HTTP::Decompressor
+    use Chef::HTTP::Authenticator
+    if Gem::Version.new(Chef::VERSION) >= Gem::Version.new('11.12')
+      use Chef::HTTP::RemoteRequestID
+    end
+  end
+end