diff options
author | Tim Smith <tsmith@chef.io> | 2017-12-19 12:54:15 -0800 |
---|---|---|
committer | GitHub <noreply@github.com> | 2017-12-19 12:54:15 -0800 |
commit | 17488942f10938664ae642d7a56531665d77c14f (patch) | |
tree | cdc88508eb8eff1647e50604a922dc75d992b8d3 /spec/unit/resource/log_spec.rb | |
parent | 550132f4a37297d9b2b99dcf41ea9298506366ed (diff) | |
parent | 62068b35f7a03c1c0da9c8f213159c08a266d6f8 (diff) | |
download | chef-17488942f10938664ae642d7a56531665d77c14f.tar.gz |
Merge pull request #6692 from chef/resource_specs
Cleanup to some of the resource specs
Diffstat (limited to 'spec/unit/resource/log_spec.rb')
-rw-r--r-- | spec/unit/resource/log_spec.rb | 52 |
1 files changed, 24 insertions, 28 deletions
diff --git a/spec/unit/resource/log_spec.rb b/spec/unit/resource/log_spec.rb index 18a1eb65bf..4459762687 100644 --- a/spec/unit/resource/log_spec.rb +++ b/spec/unit/resource/log_spec.rb @@ -21,53 +21,49 @@ require "spec_helper" describe Chef::Resource::Log do - before(:each) do - @log_str = "this is my string to log" - @resource = Chef::Resource::Log.new(@log_str) - end + let(:log_str) { "this is my string to log" } + let(:resource) { Chef::Resource::Log.new(log_str) } - it "should create a new Chef::Resource::Log" do - expect(@resource).to be_a_kind_of(Chef::Resource) - expect(@resource).to be_a_kind_of(Chef::Resource::Log) + it "creates a new Chef::Resource::Log" do + expect(resource).to be_a_kind_of(Chef::Resource) + expect(resource).to be_a_kind_of(Chef::Resource::Log) end it "supports the :write actions" do - expect(@resource.allowed_actions).to include(:write) + expect(resource.allowed_actions).to include(:write) end - it "should have a name of log" do - expect(@resource.resource_name).to eq(:log) + it "has a name of log" do + expect(resource.resource_name).to eq(:log) end - it "should allow you to set a log string" do - expect(@resource.name).to eq(@log_str) + it "allows you to set a log string" do + expect(resource.name).to eq(log_str) end - it "should set the message to the first argument to new" do - expect(@resource.message).to eq(@log_str) + it "sets the message to the first argument to new" do + expect(resource.message).to eq(log_str) end - it "should accept a string for the log message" do - @resource.message "this is different" - expect(@resource.message).to eq("this is different") + it "accepts a string for the log message" do + resource.message "this is different" + expect(resource.message).to eq("this is different") end - it "should accept a vaild level option" do - @resource.level :debug - @resource.level :info - @resource.level :warn - @resource.level :error - @resource.level :fatal - expect { @resource.level :unsupported }.to raise_error(ArgumentError) + it "accepts a vaild level option" do + resource.level :debug + resource.level :info + resource.level :warn + resource.level :error + resource.level :fatal + expect { resource.level :unsupported }.to raise_error(ArgumentError) end describe "when the identity is defined" do - before do - @resource = Chef::Resource::Log.new("ery day I'm loggin-in") - end + let(:resource) { Chef::Resource::Log.new("ery day I'm loggin-in") } it "returns the log string as its identity" do - expect(@resource.identity).to eq("ery day I'm loggin-in") + expect(resource.identity).to eq("ery day I'm loggin-in") end end end |