diff options
author | John Keiser <john@johnkeiser.com> | 2015-12-02 16:45:32 -0800 |
---|---|---|
committer | John Keiser <john@johnkeiser.com> | 2015-12-02 16:45:32 -0800 |
commit | 3d7440aae2f78f99d63ed4a24b5b1f3c5b1da8a8 (patch) | |
tree | 7ebfe5f31203e06088cd3d0d257dd366c6c599ef /spec/integration/recipes/lwrp_inline_resources_spec.rb | |
parent | de1f684f415faa54599c6b3abbe211d64a319aa6 (diff) | |
download | chef-3d7440aae2f78f99d63ed4a24b5b1f3c5b1da8a8.tar.gz |
Converge actions all together instead of in action :x (fixes #4124)jk/4124
Diffstat (limited to 'spec/integration/recipes/lwrp_inline_resources_spec.rb')
-rw-r--r-- | spec/integration/recipes/lwrp_inline_resources_spec.rb | 116 |
1 files changed, 93 insertions, 23 deletions
diff --git a/spec/integration/recipes/lwrp_inline_resources_spec.rb b/spec/integration/recipes/lwrp_inline_resources_spec.rb index e70605d3d3..73b7d428f0 100644 --- a/spec/integration/recipes/lwrp_inline_resources_spec.rb +++ b/spec/integration/recipes/lwrp_inline_resources_spec.rb @@ -18,35 +18,105 @@ describe "LWRPs with inline resources" do # cf. CHEF-4914 let(:chef_client) { "ruby '#{chef_dir}/chef-client' --minimal-ohai" } + context "with a use_inline_resources provider with 'def action_a' instead of action :a" do + class LwrpInlineResourcesTest < Chef::Resource::LWRPBase + resource_name :lwrp_inline_resources_test + actions :a, :nothing + default_action :a + property :ran_a + class Provider < Chef::Provider::LWRPBase + provides :lwrp_inline_resources_test + use_inline_resources + def action_a + r = new_resource + ruby_block 'run a' do + block { r.ran_a "ran a" } + end + end + end + end + + it "this is totally a bug, but for backcompat purposes, it adds the resources to the main resource collection and does not get marked updated" do + r = nil + expect_recipe { + r = lwrp_inline_resources_test 'hi' + }.to have_updated('ruby_block[run a]', :run) + expect(r.ran_a).to eq "ran a" + end + end + + context "with an inline_resources provider with two actions, one calling the other" do + class LwrpInlineResourcesTest2 < Chef::Resource::LWRPBase + resource_name :lwrp_inline_resources_test2 + actions :a, :b, :nothing + default_action :b + property :ran_a + property :ran_b + class Provider < Chef::Provider::LWRPBase + provides :lwrp_inline_resources_test2 + use_inline_resources + + action :a do + r = new_resource + ruby_block 'run a' do + block { r.ran_a "ran a" } + end + end + + action :b do + action_a + r = new_resource + # Grab ran_a right now, before we converge + ran_a = r.ran_a + ruby_block 'run b' do + block { r.ran_b "ran b: ran_a value was #{ran_a.inspect}" } + end + end + end + end + + it "resources declared in b are executed immediately inline", :focus do + r = nil + expect_recipe { + r = lwrp_inline_resources_test2 'hi' do + action :b + end + }.to have_updated('lwrp_inline_resources_test2[hi]', :b). + and have_updated('ruby_block[run a]', :run). + and have_updated('ruby_block[run b]', :run) + expect(r.ran_b).to eq "ran b: ran_a value was \"ran a\"" + end + end + when_the_repository "has a cookbook with a nested LWRP" do before do directory 'cookbooks/x' do - file 'resources/do_nothing.rb', <<EOM -actions :create, :nothing -default_action :create -EOM - file 'providers/do_nothing.rb', <<EOM -action :create do -end -EOM + file 'resources/do_nothing.rb', <<-EOM + actions :create, :nothing + default_action :create + EOM + file 'providers/do_nothing.rb', <<-EOM + action :create do + end + EOM - file 'resources/my_machine.rb', <<EOM -actions :create, :nothing -default_action :create -EOM - file 'providers/my_machine.rb', <<EOM -use_inline_resources -action :create do - x_do_nothing 'a' - x_do_nothing 'b' -end -EOM + file 'resources/my_machine.rb', <<-EOM + actions :create, :nothing + default_action :create + EOM + file 'providers/my_machine.rb', <<-EOM + use_inline_resources + action :create do + x_do_nothing 'a' + x_do_nothing 'b' + end + EOM - file 'recipes/default.rb', <<EOM -x_my_machine "me" -x_my_machine "you" -EOM + file 'recipes/default.rb', <<-EOM + x_my_machine "me" + x_my_machine "you" + EOM end # directory 'cookbooks/x' end |