chef-rewind 0.0.5 → 0.0.6

data/spec/support/shared/unit/api_error_inspector.rb

@@ -0,0 +1,192 @@
+#
+# Author:: Daniel DeLeo (<dan@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.
+#
+
+
+
+# == API Error Inspector Examples
+# These tests are work in progress. They exercise the code enough to ensure it
+# runs without error, but don't make assertions about the output. This is
+# because aspects such as how information gets formatted, what's included, etc.
+# are still in flux. When testing an inspector, change the outputter to use
+# STDOUT and manually check the ouput.
+
+shared_examples_for "an api error inspector" do
+
+  before do
+    @node_name = "test-node.example.com"
+    @config = {
+      :validation_client_name => "testorg-validator",
+      :validation_key => "/etc/chef/testorg-validator.pem",
+      :chef_server_url => "https://chef-api.example.com",
+      :node_name => "testnode-name",
+      :client_key => "/etc/chef/client.pem"
+    }
+    @description = Chef::Formatters::ErrorDescription.new("Error registering the node:")
+    @outputter = Chef::Formatters::Outputter.new(StringIO.new, STDERR)
+    #@outputter = Chef::Formatters::Outputter.new(STDOUT, STDERR)
+
+  end
+
+  describe "when explaining a network error" do
+    before do
+      @exception = Errno::ECONNREFUSED.new("connection refused")
+      @inspector = described_class.new(@node_name, @exception, @config)
+      @inspector.add_explanation(@description)
+    end
+
+    it "prints a nice message" do
+      @description.display(@outputter)
+    end
+
+  end
+
+  describe "when explaining a 'private key missing' error" do
+    before do
+      @exception = Chef::Exceptions::PrivateKeyMissing.new("no private key yo")
+      @inspector = described_class.new(@node_name, @exception, @config)
+      @inspector.add_explanation(@description)
+    end
+
+    it "prints a nice message" do
+      @description.display(@outputter)
+    end
+
+  end
+
+  describe "when explaining a 401 caused by clock skew" do
+    before do
+      @response_body = "synchronize the clock on your host"
+      @response = Net::HTTPUnauthorized.new("1.1", "401", "(response) unauthorized")
+      @response.stub!(:body).and_return(@response_body)
+      @exception = Net::HTTPServerException.new("(exception) unauthorized", @response)
+      @inspector = described_class.new(@node_name, @exception, @config)
+      @inspector.add_explanation(@description)
+    end
+
+    it "prints a nice message" do
+      @description.display(@outputter)
+    end
+
+  end
+
+  describe "when explaining a 401 (no clock skew)" do
+    before do
+      @response_body = "check your key and node name"
+      @response = Net::HTTPUnauthorized.new("1.1", "401", "(response) unauthorized")
+      @response.stub!(:body).and_return(@response_body)
+      @exception = Net::HTTPServerException.new("(exception) unauthorized", @response)
+      @inspector = described_class.new(@node_name, @exception, @config)
+      @inspector.add_explanation(@description)
+    end
+
+    it "prints a nice message" do
+      @description.display(@outputter)
+    end
+
+  end
+
+  describe "when explaining a 403" do
+    before do
+      @response_body = "forbidden"
+      @response = Net::HTTPForbidden.new("1.1", "403", "(response) forbidden")
+      @response.stub!(:body).and_return(@response_body)
+      @exception = Net::HTTPServerException.new("(exception) forbidden", @response)
+      @inspector = described_class.new(@node_name, @exception, @config)
+      @inspector.add_explanation(@description)
+    end
+
+    it "prints a nice message" do
+      @description.display(@outputter)
+    end
+
+  end
+
+  describe "when explaining a 400" do
+    before do
+      @response_body = "didn't like your data"
+      @response = Net::HTTPBadRequest.new("1.1", "400", "(response) bad request")
+      @response.stub!(:body).and_return(@response_body)
+      @exception = Net::HTTPServerException.new("(exception) bad request", @response)
+      @inspector = described_class.new(@node_name, @exception, @config)
+      @inspector.add_explanation(@description)
+    end
+
+    it "prints a nice message" do
+      @description.display(@outputter)
+    end
+
+  end
+
+  describe "when explaining a 404" do
+    before do
+      @response_body = "probably caused by a redirect to a get"
+      @response = Net::HTTPNotFound.new("1.1", "404", "(response) not found")
+      @response.stub!(:body).and_return(@response_body)
+      @exception = Net::HTTPServerException.new("(exception) not found", @response)
+      @inspector = described_class.new(@node_name, @exception, @config)
+      @inspector.add_explanation(@description)
+    end
+
+    it "prints a nice message" do
+      @description.display(@outputter)
+    end
+  end
+
+  describe "when explaining a 500" do
+    before do
+      @response_body = "sad trombone"
+      @response = Net::HTTPInternalServerError.new("1.1", "500", "(response) internal server error")
+      @response.stub!(:body).and_return(@response_body)
+      @exception = Net::HTTPFatalError.new("(exception) internal server error", @response)
+      @inspector = described_class.new(@node_name, @exception, @config)
+      @inspector.add_explanation(@description)
+    end
+
+    it "prints a nice message" do
+      @description.display(@outputter)
+    end
+  end
+
+  describe "when explaining a 503" do
+    before do
+      @response_body = "sad trombone orchestra"
+      @response = Net::HTTPBadGateway.new("1.1", "502", "(response) bad gateway")
+      @response.stub!(:body).and_return(@response_body)
+      @exception = Net::HTTPFatalError.new("(exception) bad gateway", @response)
+      @inspector = described_class.new(@node_name, @exception, @config)
+      @inspector.add_explanation(@description)
+    end
+
+    it "prints a nice message" do
+      @description.display(@outputter)
+    end
+  end
+
+  describe "when explaining an unknown error" do
+    before do
+      @exception = RuntimeError.new("(exception) something went wrong")
+      @inspector = described_class.new(@node_name, @exception, @config)
+      @inspector.add_explanation(@description)
+    end
+
+    it "prints a nice message" do
+      @description.display(@outputter)
+    end
+  end
+
+end

data/spec/support/shared/unit/platform_introspector.rb

@@ -0,0 +1,162 @@
+#
+# Author:: Seth Falcon (<seth@opscode.com>)
+# Author:: Daniel DeLeo (<dan@opscode.com>)
+# Copyright:: Copyright (c) 2010, 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.
+#
+
+
+shared_examples_for "a platform introspector" do
+  before(:each) do
+    @platform_hash = {}
+    %w{openbsd freebsd}.each do |x|
+      @platform_hash[x] = {
+        "default" => x,
+        "1.2.3" => "#{x}-1.2.3"
+      }
+    end
+    @platform_hash["debian"] = {["5", "6"] => "debian-5/6", "default" => "debian"}
+    @platform_hash["default"] = "default"
+
+    @platform_family_hash = {
+      "debian" => "debian value",
+      [:rhel, :fedora] => "redhatty value",
+      "suse" => "suse value",
+      :default => "default value"
+    }
+  end
+
+  it "returns a default value when there is no known platform" do
+    node = Hash.new
+    platform_introspector.value_for_platform(@platform_hash).should == "default"
+  end
+
+  it "returns a default value when there is no known platform family" do
+    platform_introspector.value_for_platform_family(@platform_family_hash).should == "default value"
+  end
+
+  it "returns a default value when the current platform doesn't match" do
+    node.automatic_attrs[:platform] = "not-a-known-platform"
+    platform_introspector.value_for_platform(@platform_hash).should == "default"
+  end
+
+  it "returns a default value when current platform_family doesn't match" do
+    node.automatic_attrs[:platform_family] = "ultra-derived-linux"
+    platform_introspector.value_for_platform_family(@platform_family_hash).should == "default value"
+  end
+
+  it "returns a value based on the current platform" do
+    node.automatic_attrs[:platform] = "openbsd"
+    platform_introspector.value_for_platform(@platform_hash).should == "openbsd"
+  end
+
+  it "returns a value based on the current platform family" do
+    node.automatic_attrs[:platform_family] = "debian"
+    platform_introspector.value_for_platform_family(@platform_family_hash).should == "debian value"
+  end
+
+  it "returns a version-specific value based on the current platform" do
+    node.automatic_attrs[:platform] = "openbsd"
+    node.automatic_attrs[:platform_version] = "1.2.3"
+    platform_introspector.value_for_platform(@platform_hash).should == "openbsd-1.2.3"
+  end
+
+  it "returns a value based on the current platform if version not found" do
+    node.automatic_attrs[:platform] = "openbsd"
+    node.automatic_attrs[:platform_version] = "0.0.0"
+    platform_introspector.value_for_platform(@platform_hash).should == "openbsd"
+  end
+
+  describe "when platform versions is an array" do
+    it "returns a version-specific value based on the current platform" do
+      node.automatic_attrs[:platform] = "debian"
+      node.automatic_attrs[:platform_version] = "6"
+      platform_introspector.value_for_platform(@platform_hash).should == "debian-5/6"
+    end
+
+    it "returns a value based on the current platform if version not found" do
+      node.automatic_attrs[:platform] = "debian"
+      node.automatic_attrs[:platform_version] = "0.0.0"
+      platform_introspector.value_for_platform(@platform_hash).should == "debian"
+    end
+  end
+
+  describe "when checking platform?" do
+
+    it "returns true if the node is a provided platform and platforms are provided as symbols" do
+      node.automatic_attrs[:platform] = 'ubuntu'
+      platform_introspector.platform?([:redhat, :ubuntu]).should == true
+    end
+
+    it "returns true if the node is a provided platform and platforms are provided as strings" do
+      node.automatic_attrs[:platform] = 'ubuntu'
+      platform_introspector.platform?(["redhat", "ubuntu"]).should == true
+    end
+
+    it "returns false if the node is not of the provided platforms" do
+      node.automatic_attrs[:platform] = 'ubuntu'
+      platform_introspector.platform?(:splatlinux).should == false
+    end
+  end
+
+  describe "when checking platform_family?" do
+
+    it "returns true if the node is in a provided platform family and families are provided as symbols" do
+      node.automatic_attrs[:platform_family] = 'debian'
+      platform_introspector.platform_family?([:rhel, :debian]).should == true
+    end
+
+    it "returns true if the node is a provided platform and platforms are provided as strings" do
+      node.automatic_attrs[:platform_family] = 'rhel'
+      platform_introspector.platform_family?(["rhel", "debian"]).should == true
+    end
+
+    it "returns false if the node is not of the provided platforms" do
+      node.automatic_attrs[:platform_family] = 'suse'
+      platform_introspector.platform_family?(:splatlinux).should == false
+    end
+
+    it "returns false if the node is not of the provided platforms and platform_family is not set" do
+      platform_introspector.platform_family?(:splatlinux).should == false
+    end
+
+  end
+  # NOTE: this is a regression test for bug CHEF-1514
+  describe "when the value is an array" do
+    before do
+      @platform_hash = {
+        "debian" => { "4.0" => [ :restart, :reload ], "default" => [ :restart, :reload, :status ] },
+        "ubuntu" => { "default" => [ :restart, :reload, :status ] },
+        "centos" => { "default" => [ :restart, :reload, :status ] },
+        "redhat" => { "default" => [ :restart, :reload, :status ] },
+        "fedora" => { "default" => [ :restart, :reload, :status ] },
+        "default" => { "default" => [:restart, :reload ] }}
+    end
+
+    it "returns the correct default for a given platform" do
+      node.automatic_attrs[:platform] = "debian"
+      node.automatic_attrs[:platform_version] = '9000'
+      platform_introspector.value_for_platform(@platform_hash).should == [ :restart, :reload, :status ]
+    end
+
+    it "returns the correct platform+version specific value " do
+      node.automatic_attrs[:platform] = "debian"
+      node.automatic_attrs[:platform_version] = '4.0'
+      platform_introspector.value_for_platform(@platform_hash).should == [:restart, :reload]
+    end
+  end
+
+end
+

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: chef-rewind
 version: !ruby/object:Gem::Version
-  version: 0.0.5
+  version: 0.0.6
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-11-11 00:00:00.000000000 Z
+date: 2013-09-20 00:00:00.000000000 Z
 dependencies: []
 description: Monkey patches Chef to allow rewinding of existing resources
 email:
@@ -19,12 +19,60 @@ extensions: []
 extra_rdoc_files: []
 files:
 - .gitignore
+- CHANGELOG
 - Gemfile
 - LICENSE
 - README.md
 - Rakefile
 - chef-rewind.gemspec
 - lib/chef/rewind.rb
+- spec/data/cookbooks/angrybash/recipes/default.rb
+- spec/data/cookbooks/apache2/files/default/apache2_module_conf_generate.pl
+- spec/data/cookbooks/apache2/recipes/default.rb
+- spec/data/cookbooks/borken/recipes/default.rb
+- spec/data/cookbooks/borken/templates/default/borken.erb
+- spec/data/cookbooks/chefignore
+- spec/data/cookbooks/java/files/default/java.response
+- spec/data/cookbooks/openldap/attributes/default.rb
+- spec/data/cookbooks/openldap/attributes/smokey.rb
+- spec/data/cookbooks/openldap/definitions/client.rb
+- spec/data/cookbooks/openldap/definitions/server.rb
+- spec/data/cookbooks/openldap/files/default/.dotfile
+- spec/data/cookbooks/openldap/files/default/.ssh/id_rsa
+- spec/data/cookbooks/openldap/files/default/remotedir/.a_dotdir/.a_dotfile_in_a_dotdir
+- spec/data/cookbooks/openldap/files/default/remotedir/remote_dir_file1.txt
+- spec/data/cookbooks/openldap/files/default/remotedir/remote_dir_file2.txt
+- spec/data/cookbooks/openldap/files/default/remotedir/remotesubdir/.a_dotfile
+- spec/data/cookbooks/openldap/files/default/remotedir/remotesubdir/remote_subdir_file1.txt
+- spec/data/cookbooks/openldap/files/default/remotedir/remotesubdir/remote_subdir_file2.txt
+- spec/data/cookbooks/openldap/files/default/remotedir/subdir_with_no_file_just_a_subsubdir/the_subsubdir/some_file.txt
+- spec/data/cookbooks/openldap/recipes/default.rb
+- spec/data/cookbooks/openldap/recipes/gigantor.rb
+- spec/data/cookbooks/openldap/recipes/one.rb
+- spec/data/cookbooks/openldap/templates/default/openldap_stuff.conf.erb
+- spec/data/cookbooks/openldap/templates/default/openldap_variable_stuff.conf.erb
+- spec/data/cookbooks/openldap/templates/default/test.erb
+- spec/rewind_recipe_spec.rb
+- spec/rewind_resource_spec.rb
+- spec/spec_helper.rb
+- spec/support/chef_helpers.rb
+- spec/support/lib/chef/provider/easy.rb
+- spec/support/lib/chef/provider/snakeoil.rb
+- spec/support/lib/chef/resource/cat.rb
+- spec/support/lib/chef/resource/one_two_three_four.rb
+- spec/support/lib/chef/resource/zen_master.rb
+- spec/support/matchers/leak.rb
+- spec/support/mock/constant.rb
+- spec/support/mock/platform.rb
+- spec/support/platform_helpers.rb
+- spec/support/platforms/prof/gc.rb
+- spec/support/platforms/prof/win32.rb
+- spec/support/shared/functional/directory_resource.rb
+- spec/support/shared/functional/file_resource.rb
+- spec/support/shared/functional/knife.rb
+- spec/support/shared/functional/securable_resource.rb
+- spec/support/shared/unit/api_error_inspector.rb
+- spec/support/shared/unit/platform_introspector.rb
 homepage: ''
 licenses: []
 post_install_message: 
@@ -45,9 +93,56 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 1.8.23
+rubygems_version: 1.8.25
 signing_key: 
 specification_version: 3
 summary: Monkey patches Chef to allow rewinding of existing resources
-test_files: []
+test_files:
+- spec/data/cookbooks/angrybash/recipes/default.rb
+- spec/data/cookbooks/apache2/files/default/apache2_module_conf_generate.pl
+- spec/data/cookbooks/apache2/recipes/default.rb
+- spec/data/cookbooks/borken/recipes/default.rb
+- spec/data/cookbooks/borken/templates/default/borken.erb
+- spec/data/cookbooks/chefignore
+- spec/data/cookbooks/java/files/default/java.response
+- spec/data/cookbooks/openldap/attributes/default.rb
+- spec/data/cookbooks/openldap/attributes/smokey.rb
+- spec/data/cookbooks/openldap/definitions/client.rb
+- spec/data/cookbooks/openldap/definitions/server.rb
+- spec/data/cookbooks/openldap/files/default/.dotfile
+- spec/data/cookbooks/openldap/files/default/.ssh/id_rsa
+- spec/data/cookbooks/openldap/files/default/remotedir/.a_dotdir/.a_dotfile_in_a_dotdir
+- spec/data/cookbooks/openldap/files/default/remotedir/remote_dir_file1.txt
+- spec/data/cookbooks/openldap/files/default/remotedir/remote_dir_file2.txt
+- spec/data/cookbooks/openldap/files/default/remotedir/remotesubdir/.a_dotfile
+- spec/data/cookbooks/openldap/files/default/remotedir/remotesubdir/remote_subdir_file1.txt
+- spec/data/cookbooks/openldap/files/default/remotedir/remotesubdir/remote_subdir_file2.txt
+- spec/data/cookbooks/openldap/files/default/remotedir/subdir_with_no_file_just_a_subsubdir/the_subsubdir/some_file.txt
+- spec/data/cookbooks/openldap/recipes/default.rb
+- spec/data/cookbooks/openldap/recipes/gigantor.rb
+- spec/data/cookbooks/openldap/recipes/one.rb
+- spec/data/cookbooks/openldap/templates/default/openldap_stuff.conf.erb
+- spec/data/cookbooks/openldap/templates/default/openldap_variable_stuff.conf.erb
+- spec/data/cookbooks/openldap/templates/default/test.erb
+- spec/rewind_recipe_spec.rb
+- spec/rewind_resource_spec.rb
+- spec/spec_helper.rb
+- spec/support/chef_helpers.rb
+- spec/support/lib/chef/provider/easy.rb
+- spec/support/lib/chef/provider/snakeoil.rb
+- spec/support/lib/chef/resource/cat.rb
+- spec/support/lib/chef/resource/one_two_three_four.rb
+- spec/support/lib/chef/resource/zen_master.rb
+- spec/support/matchers/leak.rb
+- spec/support/mock/constant.rb
+- spec/support/mock/platform.rb
+- spec/support/platform_helpers.rb
+- spec/support/platforms/prof/gc.rb
+- spec/support/platforms/prof/win32.rb
+- spec/support/shared/functional/directory_resource.rb
+- spec/support/shared/functional/file_resource.rb
+- spec/support/shared/functional/knife.rb
+- spec/support/shared/functional/securable_resource.rb
+- spec/support/shared/unit/api_error_inspector.rb
+- spec/support/shared/unit/platform_introspector.rb
 has_rdoc: