diff options
| -rw-r--r-- | lib/chef/knife/core/node_presenter.rb | 2 | ||||
| -rw-r--r-- | lib/chef/recipe.rb | 6 | ||||
| -rw-r--r-- | spec/unit/node_spec.rb | 2 | ||||
| -rw-r--r-- | spec/unit/recipe_spec.rb | 12 |
4 files changed, 11 insertions, 11 deletions
diff --git a/lib/chef/knife/core/node_presenter.rb b/lib/chef/knife/core/node_presenter.rb index d9ea8c7669..5192c53f51 100644 --- a/lib/chef/knife/core/node_presenter.rb +++ b/lib/chef/knife/core/node_presenter.rb @@ -125,7 +125,7 @@ ROLES summarized << <<-SUMMARY #{key('Recipes:')} #{Array(node[:recipes]).join(', ')} #{key('Platform:')} #{node[:platform]} #{node[:platform_version]} -#{key('Tags:')} #{Array(node[:tags]).join(', ')} +#{key('Tags:')} #{node.tags.join(', ')} SUMMARY if config[:medium_output] || config[:long_output] summarized +=<<-MORE diff --git a/lib/chef/recipe.rb b/lib/chef/recipe.rb index 83f3f5564b..0f9bf2655b 100644 --- a/lib/chef/recipe.rb +++ b/lib/chef/recipe.rb @@ -98,7 +98,7 @@ class Chef # false<FalseClass>:: If any of the parameters are missing def tagged?(*tags) tags.each do |tag| - return false unless run_context.node[:tags].include?(tag) + return false unless run_context.node.tags.include?(tag) end true end @@ -109,10 +109,10 @@ class Chef # tags<Array>:: A list of tags # # === Returns - # tags<Array>:: The current list of run_context.node[:tags] + # tags<Array>:: The current list of run_context.node.tags def untag(*tags) tags.each do |tag| - run_context.node.normal[:tags].delete(tag) + run_context.node.tags.delete(tag) end end diff --git a/spec/unit/node_spec.rb b/spec/unit/node_spec.rb index 5f3bed2833..9e78223318 100644 --- a/spec/unit/node_spec.rb +++ b/spec/unit/node_spec.rb @@ -785,7 +785,7 @@ describe Chef::Node do end it "should not set the tags attribute to an empty array if it is already defined" do - node.normal[:tags] = [ "radiohead" ] + node.tag("radiohead") node.consume_external_attrs(@ohai_data, {}) expect(node.tags).to eql([ "radiohead" ]) end diff --git a/spec/unit/recipe_spec.rb b/spec/unit/recipe_spec.rb index da6cf561f5..b242f28e11 100644 --- a/spec/unit/recipe_spec.rb +++ b/spec/unit/recipe_spec.rb @@ -621,19 +621,19 @@ describe Chef::Recipe do it "should set tags via tag" do recipe.tag "foo" - expect(node[:tags]).to include("foo") + expect(node.tags).to include("foo") end it "should set multiple tags via tag" do recipe.tag "foo", "bar" - expect(node[:tags]).to include("foo") - expect(node[:tags]).to include("bar") + expect(node.tags).to include("foo") + expect(node.tags).to include("bar") end it "should not set the same tag twice via tag" do recipe.tag "foo" recipe.tag "foo" - expect(node[:tags]).to eql([ "foo" ]) + expect(node.tags).to eql([ "foo" ]) end it "should return the current list of tags from tag with no arguments" do @@ -657,13 +657,13 @@ describe Chef::Recipe do it "should remove a tag from the tag list via untag" do recipe.tag "foo" recipe.untag "foo" - expect(node[:tags]).to eql([]) + expect(node.tags).to eql([]) end it "should remove multiple tags from the tag list via untag" do recipe.tag "foo", "bar" recipe.untag "bar", "foo" - expect(node[:tags]).to eql([]) + expect(node.tags).to eql([]) end end |