chef 12.7.2 → 12.8.1

data/spec/integration/recipes/remote_directory.rb

@@ -1,74 +1,74 @@
-require "support/shared/integration/integration_helper"
-
-describe Chef::Resource::RemoteDirectory do
-  include IntegrationSupport
-  include Chef::Mixin::ShellOut
-
-  # Until Cheffish::RSpec has cookbook support, we have to run the whole client
-  let(:chef_dir) { File.join(File.dirname(__FILE__), "..", "..", "..", "bin") }
-
-  # Invoke `chef-client` as `ruby PATH/TO/chef-client`. This ensures the
-  # following constraints are satisfied:
-  # * Windows: windows can only run batch scripts as bare executables. Rubygems
-  # creates batch wrappers for installed gems, but we don't have batch wrappers
-  # in the source tree.
-  # * Other `chef-client` in PATH: A common case is running the tests on a
-  # machine that has omnibus chef installed. In that case we need to ensure
-  # we're running `chef-client` from the source tree and not the external one.
-  # cf. CHEF-4914
-  let(:chef_client) { "ruby '#{chef_dir}/chef-client' --minimal-ohai" }
-
-  when_the_repository "has a cookbook with a source_dir with two subdirectories, each with one file and subdir in a different alphabetical order" do
-    before do
-      file "config/client.rb", <<-EOM
-        local_mode true
-        cookbook_path "#{path_to('cookbooks')}"
-      EOM
-      directory "cookbooks/test" do
-        directory "files/default/source_dir" do
-          directory "sub1" do
-            file "aaa", ""
-            file "zzz/file", ""
-          end
-          directory "sub2" do
-            file "aaa/file", ""
-            file "zzz", ""
-          end
-        end
-      end
-    end
-
-    context "and a recipe is run with a remote_directory that syncs source_dir with different mode and file_mode" do
-      let!(:dest_dir) { path_to("dest_dir") }
-      before do
-        directory "cookbooks/test" do
-          file "recipes/default.rb", <<-EOM
-             remote_directory #{dest_dir.inspect} do
-               source "source_dir"
-               mode "0754"
-               files_mode 0777
-             end
-          EOM
-        end
-        shell_out!("#{chef_client} -c \"#{path_to('config/client.rb')}\" -o 'test::default'", :cwd => chef_dir)
-      end
-
-      def mode_of(path)
-        path = path_to(path)
-        stat = File.stat(path)
-        (stat.mode & 0777).to_s(8)
-      end
-
-      it "creates all directories and files with the correct permissions" do
-        expect(mode_of("dest_dir/sub1")).to eq "754"
-        expect(mode_of("dest_dir/sub1/aaa")).to eq "777"
-        expect(mode_of("dest_dir/sub1/zzz")).to eq "754"
-        expect(mode_of("dest_dir/sub1/zzz/file")).to eq "777"
-        expect(mode_of("dest_dir/sub2")).to eq "754"
-        expect(mode_of("dest_dir/sub2/aaa")).to eq "754"
-        expect(mode_of("dest_dir/sub2/aaa/file")).to eq "777"
-        expect(mode_of("dest_dir/sub2/zzz")).to eq "777"
-      end
-    end
-  end
-end
+require "support/shared/integration/integration_helper"
+
+describe Chef::Resource::RemoteDirectory do
+  include IntegrationSupport
+  include Chef::Mixin::ShellOut
+
+  # Until Cheffish::RSpec has cookbook support, we have to run the whole client
+  let(:chef_dir) { File.join(File.dirname(__FILE__), "..", "..", "..", "bin") }
+
+  # Invoke `chef-client` as `ruby PATH/TO/chef-client`. This ensures the
+  # following constraints are satisfied:
+  # * Windows: windows can only run batch scripts as bare executables. Rubygems
+  # creates batch wrappers for installed gems, but we don't have batch wrappers
+  # in the source tree.
+  # * Other `chef-client` in PATH: A common case is running the tests on a
+  # machine that has omnibus chef installed. In that case we need to ensure
+  # we're running `chef-client` from the source tree and not the external one.
+  # cf. CHEF-4914
+  let(:chef_client) { "ruby '#{chef_dir}/chef-client' --minimal-ohai" }
+
+  when_the_repository "has a cookbook with a source_dir with two subdirectories, each with one file and subdir in a different alphabetical order" do
+    before do
+      file "config/client.rb", <<-EOM
+        local_mode true
+        cookbook_path "#{path_to('cookbooks')}"
+      EOM
+      directory "cookbooks/test" do
+        directory "files/default/source_dir" do
+          directory "sub1" do
+            file "aaa", ""
+            file "zzz/file", ""
+          end
+          directory "sub2" do
+            file "aaa/file", ""
+            file "zzz", ""
+          end
+        end
+      end
+    end
+
+    context "and a recipe is run with a remote_directory that syncs source_dir with different mode and file_mode" do
+      let!(:dest_dir) { path_to("dest_dir") }
+      before do
+        directory "cookbooks/test" do
+          file "recipes/default.rb", <<-EOM
+             remote_directory #{dest_dir.inspect} do
+               source "source_dir"
+               mode "0754"
+               files_mode 0777
+             end
+          EOM
+        end
+        shell_out!("#{chef_client} -c \"#{path_to('config/client.rb')}\" -o 'test::default'", :cwd => chef_dir)
+      end
+
+      def mode_of(path)
+        path = path_to(path)
+        stat = File.stat(path)
+        (stat.mode & 0777).to_s(8)
+      end
+
+      it "creates all directories and files with the correct permissions" do
+        expect(mode_of("dest_dir/sub1")).to eq "754"
+        expect(mode_of("dest_dir/sub1/aaa")).to eq "777"
+        expect(mode_of("dest_dir/sub1/zzz")).to eq "754"
+        expect(mode_of("dest_dir/sub1/zzz/file")).to eq "777"
+        expect(mode_of("dest_dir/sub2")).to eq "754"
+        expect(mode_of("dest_dir/sub2/aaa")).to eq "754"
+        expect(mode_of("dest_dir/sub2/aaa/file")).to eq "777"
+        expect(mode_of("dest_dir/sub2/zzz")).to eq "777"
+      end
+    end
+  end
+end

data/spec/integration/recipes/resource_action_spec.rb

@@ -1,569 +1,569 @@
-require "support/shared/integration/integration_helper"
-
-# Houses any classes we declare
-module ResourceActionSpec
-
-  describe "Resource.action" do
-    include IntegrationSupport
-
-    shared_context "ActionJackson" do
-      it "the default action is the first declared action" do
-        converge <<-EOM, __FILE__, __LINE__ + 1
-        #{resource_dsl} "hi" do
-          foo "foo!"
-        end
-      EOM
-        expect(ActionJackson.ran_action).to eq :access_recipe_dsl
-        expect(ActionJackson.succeeded).to eq true
-      end
-
-      it "the action can access recipe DSL" do
-        converge <<-EOM, __FILE__, __LINE__ + 1
-        #{resource_dsl} "hi" do
-          foo "foo!"
-          action :access_recipe_dsl
-        end
-      EOM
-        expect(ActionJackson.ran_action).to eq :access_recipe_dsl
-        expect(ActionJackson.succeeded).to eq true
-      end
-
-      it "the action can access attributes" do
-        converge <<-EOM, __FILE__, __LINE__ + 1
-        #{resource_dsl} "hi" do
-          foo "foo!"
-          action :access_attribute
-        end
-      EOM
-        expect(ActionJackson.ran_action).to eq :access_attribute
-        expect(ActionJackson.succeeded).to eq "foo!"
-      end
-
-      it "the action can access public methods" do
-        converge <<-EOM, __FILE__, __LINE__ + 1
-        #{resource_dsl} "hi" do
-          foo "foo!"
-          action :access_method
-        end
-      EOM
-        expect(ActionJackson.ran_action).to eq :access_method
-        expect(ActionJackson.succeeded).to eq "foo_public!"
-      end
-
-      it "the action can access protected methods" do
-        converge <<-EOM, __FILE__, __LINE__ + 1
-        #{resource_dsl} "hi" do
-          foo "foo!"
-          action :access_protected_method
-        end
-      EOM
-        expect(ActionJackson.ran_action).to eq :access_protected_method
-        expect(ActionJackson.succeeded).to eq "foo_protected!"
-      end
-
-      it "the action cannot access private methods" do
-        expect {
-          converge(<<-EOM, __FILE__, __LINE__ + 1)
-          #{resource_dsl} "hi" do
-            foo "foo!"
-            action :access_private_method
-          end
-        EOM
-        }.to raise_error(NameError)
-        expect(ActionJackson.ran_action).to eq :access_private_method
-      end
-
-      it "the action cannot access resource instance variables" do
-        converge <<-EOM, __FILE__, __LINE__ + 1
-        #{resource_dsl} "hi" do
-          foo "foo!"
-          action :access_instance_variable
-        end
-      EOM
-        expect(ActionJackson.ran_action).to eq :access_instance_variable
-        expect(ActionJackson.succeeded).to be_nil
-      end
-
-      it "the action does not compile until the prior resource has converged" do
-        converge <<-EOM, __FILE__, __LINE__ + 1
-        ruby_block "wow" do
-          block do
-            ResourceActionSpec::ActionJackson.ruby_block_converged = "ruby_block_converged!"
-          end
-        end
-
-        #{resource_dsl} "hi" do
-          foo "foo!"
-          action :access_class_method
-        end
-      EOM
-        expect(ActionJackson.ran_action).to eq :access_class_method
-        expect(ActionJackson.succeeded).to eq "ruby_block_converged!"
-      end
-
-      it "the action's resources converge before the next resource converges" do
-        converge <<-EOM, __FILE__, __LINE__ + 1
-        #{resource_dsl} "hi" do
-          foo "foo!"
-          action :access_attribute
-        end
-
-        ruby_block "wow" do
-          block do
-            ResourceActionSpec::ActionJackson.ruby_block_converged = ResourceActionSpec::ActionJackson.succeeded
-          end
-        end
-      EOM
-        expect(ActionJackson.ran_action).to eq :access_attribute
-        expect(ActionJackson.succeeded).to eq "foo!"
-        expect(ActionJackson.ruby_block_converged).to eq "foo!"
-      end
-    end
-
-    context "With resource 'action_jackson'" do
-      class ActionJackson < Chef::Resource
-        use_automatic_resource_name
-        def foo(value = nil)
-          @foo = value if value
-          @foo
-        end
-
-        def blarghle(value = nil)
-          @blarghle = value if value
-          @blarghle
-        end
-
-        class <<self
-          attr_accessor :ran_action
-          attr_accessor :succeeded
-          attr_accessor :ruby_block_converged
-        end
-
-        public
-
-        def foo_public
-          "foo_public!"
-        end
-
-        protected
-
-        def foo_protected
-          "foo_protected!"
-        end
-
-        private
-
-        def foo_private
-          "foo_private!"
-        end
-
-        public
-
-        action :access_recipe_dsl do
-          ActionJackson.ran_action = :access_recipe_dsl
-          ruby_block "hi there" do
-            block do
-              ActionJackson.succeeded = true
-            end
-          end
-        end
-        action :access_attribute do
-          ActionJackson.ran_action = :access_attribute
-          ActionJackson.succeeded = foo
-          ActionJackson.succeeded += " #{blarghle}" if blarghle
-          ActionJackson.succeeded += " #{bar}" if respond_to?(:bar)
-        end
-        action :access_attribute2 do
-          ActionJackson.ran_action = :access_attribute2
-          ActionJackson.succeeded = foo
-          ActionJackson.succeeded += " #{blarghle}" if blarghle
-          ActionJackson.succeeded += " #{bar}" if respond_to?(:bar)
-        end
-        action :access_method do
-          ActionJackson.ran_action = :access_method
-          ActionJackson.succeeded = foo_public
-        end
-        action :access_protected_method do
-          ActionJackson.ran_action = :access_protected_method
-          ActionJackson.succeeded = foo_protected
-        end
-        action :access_private_method do
-          ActionJackson.ran_action = :access_private_method
-          ActionJackson.succeeded = foo_private
-        end
-        action :access_instance_variable do
-          ActionJackson.ran_action = :access_instance_variable
-          ActionJackson.succeeded = @foo
-        end
-        action :access_class_method do
-          ActionJackson.ran_action = :access_class_method
-          ActionJackson.succeeded = ActionJackson.ruby_block_converged
-        end
-      end
-
-      before(:each) {
-        ActionJackson.ran_action = :error
-        ActionJackson.succeeded = :error
-        ActionJackson.ruby_block_converged = :error
-      }
-
-      it_behaves_like "ActionJackson" do
-        let(:resource_dsl) { :action_jackson }
-      end
-
-      it "Can retrieve ancestors of action class without crashing" do
-        converge { action_jackson "hi" }
-        expect { ActionJackson.action_class.ancestors.join(",") }.not_to raise_error
-      end
-
-      context "And 'action_jackgrandson' inheriting from ActionJackson and changing nothing" do
-        before(:context) {
-          class ActionJackgrandson < ActionJackson
-            use_automatic_resource_name
-          end
-        }
-
-        it_behaves_like "ActionJackson" do
-          let(:resource_dsl) { :action_jackgrandson }
-        end
-      end
-
-      context "And 'action_jackalope' inheriting from ActionJackson with an extra attribute, action and custom method" do
-        class ActionJackalope < ActionJackson
-          use_automatic_resource_name
-
-          def foo(value = nil)
-            @foo = "#{value}alope" if value
-            @foo
-          end
-
-          def bar(value = nil)
-            @bar = "#{value}alope" if value
-            @bar
-          end
-          class <<self
-            attr_accessor :load_current_resource_ran
-            attr_accessor :jackalope_ran
-          end
-          action :access_jackalope do
-            ActionJackalope.jackalope_ran = :access_jackalope
-            ActionJackalope.succeeded = "#{foo} #{blarghle} #{bar}"
-          end
-          action :access_attribute do
-            super()
-            ActionJackalope.jackalope_ran = :access_attribute
-            ActionJackalope.succeeded = ActionJackson.succeeded
-          end
-        end
-        before do
-          ActionJackalope.jackalope_ran = nil
-          ActionJackalope.load_current_resource_ran = nil
-        end
-
-        context "action_jackson still behaves the same" do
-          it_behaves_like "ActionJackson" do
-            let(:resource_dsl) { :action_jackson }
-          end
-        end
-
-        it "the default action remains the same even though new actions were specified first" do
-          converge {
-            action_jackalope "hi" do
-              foo "foo!"
-              bar "bar!"
-            end
-          }
-          expect(ActionJackson.ran_action).to eq :access_recipe_dsl
-          expect(ActionJackson.succeeded).to eq true
-        end
-
-        it "new actions run, and can access overridden, new, and overridden attributes" do
-          converge {
-            action_jackalope "hi" do
-              foo "foo!"
-              bar "bar!"
-              blarghle "blarghle!"
-              action :access_jackalope
-            end
-          }
-          expect(ActionJackalope.jackalope_ran).to eq :access_jackalope
-          expect(ActionJackalope.succeeded).to eq "foo!alope blarghle! bar!alope"
-        end
-
-        it "overridden actions run, call super, and can access overridden, new, and overridden attributes" do
-          converge {
-            action_jackalope "hi" do
-              foo "foo!"
-              bar "bar!"
-              blarghle "blarghle!"
-              action :access_attribute
-            end
-          }
-          expect(ActionJackson.ran_action).to eq :access_attribute
-          expect(ActionJackson.succeeded).to eq "foo!alope blarghle! bar!alope"
-          expect(ActionJackalope.jackalope_ran).to eq :access_attribute
-          expect(ActionJackalope.succeeded).to eq "foo!alope blarghle! bar!alope"
-        end
-
-        it "non-overridden actions run and can access overridden and non-overridden variables (but not necessarily new ones)" do
-          converge {
-            action_jackalope "hi" do
-              foo "foo!"
-              bar "bar!"
-              blarghle "blarghle!"
-              action :access_attribute2
-            end
-          }
-          expect(ActionJackson.ran_action).to eq :access_attribute2
-          expect(ActionJackson.succeeded).to eq("foo!alope blarghle! bar!alope").or(eq("foo!alope blarghle!"))
-        end
-      end
-    end
-
-    context "With a resource with no actions" do
-      class NoActionJackson < Chef::Resource
-        use_automatic_resource_name
-
-        def foo(value = nil)
-          @foo = value if value
-          @foo
-        end
-
-        class <<self
-          attr_accessor :action_was
-        end
-      end
-
-      it "the default action is :nothing" do
-        converge {
-          no_action_jackson "hi" do
-            foo "foo!"
-            NoActionJackson.action_was = action
-          end
-        }
-        expect(NoActionJackson.action_was).to eq [:nothing]
-      end
-    end
-
-    context "With a resource with action a-b-c d" do
-      class WeirdActionJackson < Chef::Resource
-        use_automatic_resource_name
-
-        class <<self
-          attr_accessor :action_was
-        end
-
-        action "a-b-c d" do
-          WeirdActionJackson.action_was = action
-        end
-      end
-
-      it "Running the action works" do
-        expect_recipe {
-          weird_action_jackson "hi"
-        }.to be_up_to_date
-        expect(WeirdActionJackson.action_was).to eq :"a-b-c d"
-      end
-    end
-
-    context "With a resource with property x" do
-      class ResourceActionSpecWithX < Chef::Resource
-        resource_name :resource_action_spec_with_x
-        property :x, default: 20
-        action :set do
-          # Access x during converge to ensure that we emit no warnings there
-          x
-        end
-      end
-
-      context "And another resource with a property x and an action that sets property x to its value" do
-        class ResourceActionSpecAlsoWithX < Chef::Resource
-          resource_name :resource_action_spec_also_with_x
-          property :x
-          action :set_x_to_x do
-            resource_action_spec_with_x "hi" do
-              x x
-            end
-          end
-          def self.x_warning_line
-            __LINE__ - 4
-          end
-          action :set_x_to_x_in_non_initializer do
-            r = resource_action_spec_with_x "hi" do
-              x 10
-            end
-            x_times_2 = r.x * 2
-          end
-          action :set_x_to_10 do
-            resource_action_spec_with_x "hi" do
-              x 10
-            end
-          end
-        end
-
-        attr_reader :x_warning_line
-
-        it "Using the enclosing resource to set x to x emits a warning that you're using the wrong x" do
-          recipe = converge {
-            resource_action_spec_also_with_x "hi" do
-              x 1
-              action :set_x_to_x
-            end
-          }
-          warnings = recipe.logs.lines.select { |l| l =~ /warn/i }
-          expect(warnings.size).to eq 1
-          expect(warnings[0]).to match(/property x is declared in both resource_action_spec_with_x\[hi\] and resource_action_spec_also_with_x\[hi\] action :set_x_to_x. Use new_resource.x instead. At #{__FILE__}:#{ResourceActionSpecAlsoWithX.x_warning_line}/)
-        end
-
-        it "Using the enclosing resource to set x to x outside the initializer emits no warning" do
-          expect_recipe {
-            resource_action_spec_also_with_x "hi" do
-              x 1
-              action :set_x_to_x_in_non_initializer
-            end
-          }.to emit_no_warnings_or_errors
-        end
-
-        it "Using the enclosing resource to set x to 10 emits no warning" do
-          expect_recipe {
-            resource_action_spec_also_with_x "hi" do
-              x 1
-              action :set_x_to_10
-            end
-          }.to emit_no_warnings_or_errors
-        end
-
-        it "Using the enclosing resource to set x to 10 emits no warning" do
-          expect_recipe {
-            r = resource_action_spec_also_with_x "hi"
-            r.x 1
-            r.action :set_x_to_10
-          }.to emit_no_warnings_or_errors
-        end
-      end
-
-    end
-
-    context "With a resource with a set_or_return property named group (same name as a resource)" do
-      class ResourceActionSpecWithGroupAction < Chef::Resource
-        resource_name :resource_action_spec_set_group_to_nil
-        action :set_group_to_nil do
-          # Access x during converge to ensure that we emit no warnings there
-          resource_action_spec_with_group "hi" do
-            group nil
-            action :nothing
-          end
-        end
-      end
-
-      class ResourceActionSpecWithGroup < Chef::Resource
-        resource_name :resource_action_spec_with_group
-        def group(value = nil)
-          set_or_return(:group, value, {})
-        end
-      end
-
-      it "Setting group to nil in an action does not emit a warning about it being defined in two places" do
-        expect_recipe {
-          resource_action_spec_set_group_to_nil "hi" do
-            action :set_group_to_nil
-          end
-        }.to emit_no_warnings_or_errors
-      end
-    end
-
-    context "When a resource has a property with the same name as another resource" do
-      class HasPropertyNamedTemplate < Chef::Resource
-        use_automatic_resource_name
-        property :template
-        action :create do
-          template "x" do
-            "blah"
-          end
-        end
-      end
-
-      it "Raises an error when attempting to use a template in the action" do
-        expect_converge {
-          has_property_named_template "hi"
-        }.to raise_error(/Property template of has_property_named_template\[hi\] cannot be passed a block! If you meant to create a resource named template instead, you'll need to first rename the property./)
-      end
-    end
-
-    context "When a resource declares methods in action_class and declare_action_class" do
-      class DeclaresActionClassMethods < Chef::Resource
-        use_automatic_resource_name
-        property :x
-        action :create do
-          new_resource.x = a + b + c + d
-        end
-        action_class do
-          def a
-            1
-          end
-        end
-        declare_action_class do
-          def b
-            2
-          end
-        end
-        action_class do
-          def c
-            3
-          end
-        end
-        declare_action_class do
-          def d
-            4
-          end
-        end
-      end
-
-      it "the methods are not available on the resource" do
-        expect { DeclaresActionClassMethods.new("hi").a }.to raise_error(NameError)
-        expect { DeclaresActionClassMethods.new("hi").b }.to raise_error(NameError)
-        expect { DeclaresActionClassMethods.new("hi").c }.to raise_error(NameError)
-        expect { DeclaresActionClassMethods.new("hi").d }.to raise_error(NameError)
-      end
-
-      it "the methods are available to the action" do
-        r = nil
-        expect_recipe {
-          r = declares_action_class_methods "hi"
-        }.to emit_no_warnings_or_errors
-        expect(r.x).to eq(10)
-      end
-
-      context "And a subclass also creates a method" do
-        class DeclaresActionClassMethodsToo < DeclaresActionClassMethods
-          use_automatic_resource_name
-          action :create do
-            new_resource.x a + b + c + d + e
-          end
-          action_class do
-            def e
-              5
-            end
-          end
-        end
-
-        it "the methods are not available on the resource" do
-          expect { DeclaresActionClassMethods.new("hi").a }.to raise_error(NameError)
-          expect { DeclaresActionClassMethods.new("hi").b }.to raise_error(NameError)
-          expect { DeclaresActionClassMethods.new("hi").c }.to raise_error(NameError)
-          expect { DeclaresActionClassMethods.new("hi").d }.to raise_error(NameError)
-          expect { DeclaresActionClassMethods.new("hi").e }.to raise_error(NameError)
-        end
-
-        it "the methods are available to the action" do
-          r = nil
-          expect_recipe {
-            r = declares_action_class_methods_too "hi"
-          }.to emit_no_warnings_or_errors
-          expect(r.x).to eq(15)
-        end
-      end
-    end
-  end
-
-end
+require "support/shared/integration/integration_helper"
+
+# Houses any classes we declare
+module ResourceActionSpec
+
+  describe "Resource.action" do
+    include IntegrationSupport
+
+    shared_context "ActionJackson" do
+      it "the default action is the first declared action" do
+        converge <<-EOM, __FILE__, __LINE__ + 1
+        #{resource_dsl} "hi" do
+          foo "foo!"
+        end
+      EOM
+        expect(ActionJackson.ran_action).to eq :access_recipe_dsl
+        expect(ActionJackson.succeeded).to eq true
+      end
+
+      it "the action can access recipe DSL" do
+        converge <<-EOM, __FILE__, __LINE__ + 1
+        #{resource_dsl} "hi" do
+          foo "foo!"
+          action :access_recipe_dsl
+        end
+      EOM
+        expect(ActionJackson.ran_action).to eq :access_recipe_dsl
+        expect(ActionJackson.succeeded).to eq true
+      end
+
+      it "the action can access attributes" do
+        converge <<-EOM, __FILE__, __LINE__ + 1
+        #{resource_dsl} "hi" do
+          foo "foo!"
+          action :access_attribute
+        end
+      EOM
+        expect(ActionJackson.ran_action).to eq :access_attribute
+        expect(ActionJackson.succeeded).to eq "foo!"
+      end
+
+      it "the action can access public methods" do
+        converge <<-EOM, __FILE__, __LINE__ + 1
+        #{resource_dsl} "hi" do
+          foo "foo!"
+          action :access_method
+        end
+      EOM
+        expect(ActionJackson.ran_action).to eq :access_method
+        expect(ActionJackson.succeeded).to eq "foo_public!"
+      end
+
+      it "the action can access protected methods" do
+        converge <<-EOM, __FILE__, __LINE__ + 1
+        #{resource_dsl} "hi" do
+          foo "foo!"
+          action :access_protected_method
+        end
+      EOM
+        expect(ActionJackson.ran_action).to eq :access_protected_method
+        expect(ActionJackson.succeeded).to eq "foo_protected!"
+      end
+
+      it "the action cannot access private methods" do
+        expect {
+          converge(<<-EOM, __FILE__, __LINE__ + 1)
+          #{resource_dsl} "hi" do
+            foo "foo!"
+            action :access_private_method
+          end
+        EOM
+        }.to raise_error(NameError)
+        expect(ActionJackson.ran_action).to eq :access_private_method
+      end
+
+      it "the action cannot access resource instance variables" do
+        converge <<-EOM, __FILE__, __LINE__ + 1
+        #{resource_dsl} "hi" do
+          foo "foo!"
+          action :access_instance_variable
+        end
+      EOM
+        expect(ActionJackson.ran_action).to eq :access_instance_variable
+        expect(ActionJackson.succeeded).to be_nil
+      end
+
+      it "the action does not compile until the prior resource has converged" do
+        converge <<-EOM, __FILE__, __LINE__ + 1
+        ruby_block "wow" do
+          block do
+            ResourceActionSpec::ActionJackson.ruby_block_converged = "ruby_block_converged!"
+          end
+        end
+
+        #{resource_dsl} "hi" do
+          foo "foo!"
+          action :access_class_method
+        end
+      EOM
+        expect(ActionJackson.ran_action).to eq :access_class_method
+        expect(ActionJackson.succeeded).to eq "ruby_block_converged!"
+      end
+
+      it "the action's resources converge before the next resource converges" do
+        converge <<-EOM, __FILE__, __LINE__ + 1
+        #{resource_dsl} "hi" do
+          foo "foo!"
+          action :access_attribute
+        end
+
+        ruby_block "wow" do
+          block do
+            ResourceActionSpec::ActionJackson.ruby_block_converged = ResourceActionSpec::ActionJackson.succeeded
+          end
+        end
+      EOM
+        expect(ActionJackson.ran_action).to eq :access_attribute
+        expect(ActionJackson.succeeded).to eq "foo!"
+        expect(ActionJackson.ruby_block_converged).to eq "foo!"
+      end
+    end
+
+    context "With resource 'action_jackson'" do
+      class ActionJackson < Chef::Resource
+        use_automatic_resource_name
+        def foo(value = nil)
+          @foo = value if value
+          @foo
+        end
+
+        def blarghle(value = nil)
+          @blarghle = value if value
+          @blarghle
+        end
+
+        class <<self
+          attr_accessor :ran_action
+          attr_accessor :succeeded
+          attr_accessor :ruby_block_converged
+        end
+
+        public
+
+        def foo_public
+          "foo_public!"
+        end
+
+        protected
+
+        def foo_protected
+          "foo_protected!"
+        end
+
+        private
+
+        def foo_private
+          "foo_private!"
+        end
+
+        public
+
+        action :access_recipe_dsl do
+          ActionJackson.ran_action = :access_recipe_dsl
+          ruby_block "hi there" do
+            block do
+              ActionJackson.succeeded = true
+            end
+          end
+        end
+        action :access_attribute do
+          ActionJackson.ran_action = :access_attribute
+          ActionJackson.succeeded = foo
+          ActionJackson.succeeded += " #{blarghle}" if blarghle
+          ActionJackson.succeeded += " #{bar}" if respond_to?(:bar)
+        end
+        action :access_attribute2 do
+          ActionJackson.ran_action = :access_attribute2
+          ActionJackson.succeeded = foo
+          ActionJackson.succeeded += " #{blarghle}" if blarghle
+          ActionJackson.succeeded += " #{bar}" if respond_to?(:bar)
+        end
+        action :access_method do
+          ActionJackson.ran_action = :access_method
+          ActionJackson.succeeded = foo_public
+        end
+        action :access_protected_method do
+          ActionJackson.ran_action = :access_protected_method
+          ActionJackson.succeeded = foo_protected
+        end
+        action :access_private_method do
+          ActionJackson.ran_action = :access_private_method
+          ActionJackson.succeeded = foo_private
+        end
+        action :access_instance_variable do
+          ActionJackson.ran_action = :access_instance_variable
+          ActionJackson.succeeded = @foo
+        end
+        action :access_class_method do
+          ActionJackson.ran_action = :access_class_method
+          ActionJackson.succeeded = ActionJackson.ruby_block_converged
+        end
+      end
+
+      before(:each) {
+        ActionJackson.ran_action = :error
+        ActionJackson.succeeded = :error
+        ActionJackson.ruby_block_converged = :error
+      }
+
+      it_behaves_like "ActionJackson" do
+        let(:resource_dsl) { :action_jackson }
+      end
+
+      it "Can retrieve ancestors of action class without crashing" do
+        converge { action_jackson "hi" }
+        expect { ActionJackson.action_class.ancestors.join(",") }.not_to raise_error
+      end
+
+      context "And 'action_jackgrandson' inheriting from ActionJackson and changing nothing" do
+        before(:context) {
+          class ActionJackgrandson < ActionJackson
+            use_automatic_resource_name
+          end
+        }
+
+        it_behaves_like "ActionJackson" do
+          let(:resource_dsl) { :action_jackgrandson }
+        end
+      end
+
+      context "And 'action_jackalope' inheriting from ActionJackson with an extra attribute, action and custom method" do
+        class ActionJackalope < ActionJackson
+          use_automatic_resource_name
+
+          def foo(value = nil)
+            @foo = "#{value}alope" if value
+            @foo
+          end
+
+          def bar(value = nil)
+            @bar = "#{value}alope" if value
+            @bar
+          end
+          class <<self
+            attr_accessor :load_current_resource_ran
+            attr_accessor :jackalope_ran
+          end
+          action :access_jackalope do
+            ActionJackalope.jackalope_ran = :access_jackalope
+            ActionJackalope.succeeded = "#{foo} #{blarghle} #{bar}"
+          end
+          action :access_attribute do
+            super()
+            ActionJackalope.jackalope_ran = :access_attribute
+            ActionJackalope.succeeded = ActionJackson.succeeded
+          end
+        end
+        before do
+          ActionJackalope.jackalope_ran = nil
+          ActionJackalope.load_current_resource_ran = nil
+        end
+
+        context "action_jackson still behaves the same" do
+          it_behaves_like "ActionJackson" do
+            let(:resource_dsl) { :action_jackson }
+          end
+        end
+
+        it "the default action remains the same even though new actions were specified first" do
+          converge {
+            action_jackalope "hi" do
+              foo "foo!"
+              bar "bar!"
+            end
+          }
+          expect(ActionJackson.ran_action).to eq :access_recipe_dsl
+          expect(ActionJackson.succeeded).to eq true
+        end
+
+        it "new actions run, and can access overridden, new, and overridden attributes" do
+          converge {
+            action_jackalope "hi" do
+              foo "foo!"
+              bar "bar!"
+              blarghle "blarghle!"
+              action :access_jackalope
+            end
+          }
+          expect(ActionJackalope.jackalope_ran).to eq :access_jackalope
+          expect(ActionJackalope.succeeded).to eq "foo!alope blarghle! bar!alope"
+        end
+
+        it "overridden actions run, call super, and can access overridden, new, and overridden attributes" do
+          converge {
+            action_jackalope "hi" do
+              foo "foo!"
+              bar "bar!"
+              blarghle "blarghle!"
+              action :access_attribute
+            end
+          }
+          expect(ActionJackson.ran_action).to eq :access_attribute
+          expect(ActionJackson.succeeded).to eq "foo!alope blarghle! bar!alope"
+          expect(ActionJackalope.jackalope_ran).to eq :access_attribute
+          expect(ActionJackalope.succeeded).to eq "foo!alope blarghle! bar!alope"
+        end
+
+        it "non-overridden actions run and can access overridden and non-overridden variables (but not necessarily new ones)" do
+          converge {
+            action_jackalope "hi" do
+              foo "foo!"
+              bar "bar!"
+              blarghle "blarghle!"
+              action :access_attribute2
+            end
+          }
+          expect(ActionJackson.ran_action).to eq :access_attribute2
+          expect(ActionJackson.succeeded).to eq("foo!alope blarghle! bar!alope").or(eq("foo!alope blarghle!"))
+        end
+      end
+    end
+
+    context "With a resource with no actions" do
+      class NoActionJackson < Chef::Resource
+        use_automatic_resource_name
+
+        def foo(value = nil)
+          @foo = value if value
+          @foo
+        end
+
+        class <<self
+          attr_accessor :action_was
+        end
+      end
+
+      it "the default action is :nothing" do
+        converge {
+          no_action_jackson "hi" do
+            foo "foo!"
+            NoActionJackson.action_was = action
+          end
+        }
+        expect(NoActionJackson.action_was).to eq [:nothing]
+      end
+    end
+
+    context "With a resource with action a-b-c d" do
+      class WeirdActionJackson < Chef::Resource
+        use_automatic_resource_name
+
+        class <<self
+          attr_accessor :action_was
+        end
+
+        action "a-b-c d" do
+          WeirdActionJackson.action_was = action
+        end
+      end
+
+      it "Running the action works" do
+        expect_recipe {
+          weird_action_jackson "hi"
+        }.to be_up_to_date
+        expect(WeirdActionJackson.action_was).to eq :"a-b-c d"
+      end
+    end
+
+    context "With a resource with property x" do
+      class ResourceActionSpecWithX < Chef::Resource
+        resource_name :resource_action_spec_with_x
+        property :x, default: 20
+        action :set do
+          # Access x during converge to ensure that we emit no warnings there
+          x
+        end
+      end
+
+      context "And another resource with a property x and an action that sets property x to its value" do
+        class ResourceActionSpecAlsoWithX < Chef::Resource
+          resource_name :resource_action_spec_also_with_x
+          property :x
+          action :set_x_to_x do
+            resource_action_spec_with_x "hi" do
+              x x
+            end
+          end
+          def self.x_warning_line
+            __LINE__ - 4
+          end
+          action :set_x_to_x_in_non_initializer do
+            r = resource_action_spec_with_x "hi" do
+              x 10
+            end
+            x_times_2 = r.x * 2
+          end
+          action :set_x_to_10 do
+            resource_action_spec_with_x "hi" do
+              x 10
+            end
+          end
+        end
+
+        attr_reader :x_warning_line
+
+        it "Using the enclosing resource to set x to x emits a warning that you're using the wrong x" do
+          recipe = converge {
+            resource_action_spec_also_with_x "hi" do
+              x 1
+              action :set_x_to_x
+            end
+          }
+          warnings = recipe.logs.lines.select { |l| l =~ /warn/i }
+          expect(warnings.size).to eq 1
+          expect(warnings[0]).to match(/property x is declared in both resource_action_spec_with_x\[hi\] and resource_action_spec_also_with_x\[hi\] action :set_x_to_x. Use new_resource.x instead. At #{__FILE__}:#{ResourceActionSpecAlsoWithX.x_warning_line}/)
+        end
+
+        it "Using the enclosing resource to set x to x outside the initializer emits no warning" do
+          expect_recipe {
+            resource_action_spec_also_with_x "hi" do
+              x 1
+              action :set_x_to_x_in_non_initializer
+            end
+          }.to emit_no_warnings_or_errors
+        end
+
+        it "Using the enclosing resource to set x to 10 emits no warning" do
+          expect_recipe {
+            resource_action_spec_also_with_x "hi" do
+              x 1
+              action :set_x_to_10
+            end
+          }.to emit_no_warnings_or_errors
+        end
+
+        it "Using the enclosing resource to set x to 10 emits no warning" do
+          expect_recipe {
+            r = resource_action_spec_also_with_x "hi"
+            r.x 1
+            r.action :set_x_to_10
+          }.to emit_no_warnings_or_errors
+        end
+      end
+
+    end
+
+    context "With a resource with a set_or_return property named group (same name as a resource)" do
+      class ResourceActionSpecWithGroupAction < Chef::Resource
+        resource_name :resource_action_spec_set_group_to_nil
+        action :set_group_to_nil do
+          # Access x during converge to ensure that we emit no warnings there
+          resource_action_spec_with_group "hi" do
+            group nil
+            action :nothing
+          end
+        end
+      end
+
+      class ResourceActionSpecWithGroup < Chef::Resource
+        resource_name :resource_action_spec_with_group
+        def group(value = nil)
+          set_or_return(:group, value, {})
+        end
+      end
+
+      it "Setting group to nil in an action does not emit a warning about it being defined in two places" do
+        expect_recipe {
+          resource_action_spec_set_group_to_nil "hi" do
+            action :set_group_to_nil
+          end
+        }.to emit_no_warnings_or_errors
+      end
+    end
+
+    context "When a resource has a property with the same name as another resource" do
+      class HasPropertyNamedTemplate < Chef::Resource
+        use_automatic_resource_name
+        property :template
+        action :create do
+          template "x" do
+            "blah"
+          end
+        end
+      end
+
+      it "Raises an error when attempting to use a template in the action" do
+        expect_converge {
+          has_property_named_template "hi"
+        }.to raise_error(/Property template of has_property_named_template\[hi\] cannot be passed a block! If you meant to create a resource named template instead, you'll need to first rename the property./)
+      end
+    end
+
+    context "When a resource declares methods in action_class and declare_action_class" do
+      class DeclaresActionClassMethods < Chef::Resource
+        use_automatic_resource_name
+        property :x
+        action :create do
+          new_resource.x = a + b + c + d
+        end
+        action_class do
+          def a
+            1
+          end
+        end
+        declare_action_class do
+          def b
+            2
+          end
+        end
+        action_class do
+          def c
+            3
+          end
+        end
+        declare_action_class do
+          def d
+            4
+          end
+        end
+      end
+
+      it "the methods are not available on the resource" do
+        expect { DeclaresActionClassMethods.new("hi").a }.to raise_error(NameError)
+        expect { DeclaresActionClassMethods.new("hi").b }.to raise_error(NameError)
+        expect { DeclaresActionClassMethods.new("hi").c }.to raise_error(NameError)
+        expect { DeclaresActionClassMethods.new("hi").d }.to raise_error(NameError)
+      end
+
+      it "the methods are available to the action" do
+        r = nil
+        expect_recipe {
+          r = declares_action_class_methods "hi"
+        }.to emit_no_warnings_or_errors
+        expect(r.x).to eq(10)
+      end
+
+      context "And a subclass also creates a method" do
+        class DeclaresActionClassMethodsToo < DeclaresActionClassMethods
+          use_automatic_resource_name
+          action :create do
+            new_resource.x a + b + c + d + e
+          end
+          action_class do
+            def e
+              5
+            end
+          end
+        end
+
+        it "the methods are not available on the resource" do
+          expect { DeclaresActionClassMethods.new("hi").a }.to raise_error(NameError)
+          expect { DeclaresActionClassMethods.new("hi").b }.to raise_error(NameError)
+          expect { DeclaresActionClassMethods.new("hi").c }.to raise_error(NameError)
+          expect { DeclaresActionClassMethods.new("hi").d }.to raise_error(NameError)
+          expect { DeclaresActionClassMethods.new("hi").e }.to raise_error(NameError)
+        end
+
+        it "the methods are available to the action" do
+          r = nil
+          expect_recipe {
+            r = declares_action_class_methods_too "hi"
+          }.to emit_no_warnings_or_errors
+          expect(r.x).to eq(15)
+        end
+      end
+    end
+  end
+
+end