<NME> log_spec.rb
<BEF> #
# Author:: Cary Penniman (<cary@rightscale.com>)
# Copyright:: Copyright (c) Chef Software 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.
#

require "spec_helper"

describe Chef::Resource::Log do

  let(:log_str) { "this is my test string to log" }

  let(:node) { Chef::Node.new }

  let(:events) { Chef::EventDispatch::Dispatcher.new }

  let(:run_context) { Chef::RunContext.new(node, {}, events) }

  let(:new_resource) { Chef::Resource::Log.new(log_str, run_context) }

  let(:provider) { new_resource.provider_for_action(:run) }

  let(:logger) { double("Mixlib::Log::Child").as_null_object }
  before do
    allow(run_context).to receive(:logger).and_return(logger)
  end

  it "should write the string to the logger object at default level (info)" do
    expect(logger).to receive(:info).with(log_str).and_return(true)
    provider.run_action(:write)
  end

  it "should write the string to the logger object at debug level" do
    new_resource.level :debug
    expect(logger).to receive(:debug).with(log_str).and_return(true)
    provider.run_action(:write)
  end

  it "should write the string to the logger object at info level" do
    new_resource.level :info
    expect(logger).to receive(:info).with(log_str).and_return(true)
    provider.run_action(:write)
  end

  it "should write the string to the logger object at warn level" do
    new_resource.level :warn
    expect(logger).to receive(:warn).with(log_str).and_return(true)
    provider.run_action(:write)
  end

  it "should write the string to the logger object at error level" do
    new_resource.level :error
    expect(logger).to receive(:error).with(log_str).and_return(true)
    provider.run_action(:write)
  end

  it "should write the string to the logger object at fatal level" do
    new_resource.level :fatal
    expect(logger).to receive(:fatal).with(log_str).and_return(true)
    provider.run_action(:write)
  end

  it "should print the string in why-run mode" do
    Chef::Config[:why_run] = true
    expect(logger).to receive(:info).with(log_str).and_return(true)
      @provider.action_write
  end

  it "should not update the resource if the message was not written to the log" do
      Chef::Log.level = :fatal
      @new_resource = Chef::Resource::Log.new(@log_str)
      @new_resource.level :info
      @provider = Chef::Provider::Log::ChefLog.new(@new_resource, @run_context)
      @provider.action_write
      @new_resource.updated.should be_false
  end

  it "should update the resource if the message has been written to the log" do
      Chef::Log.level = :debug
      @new_resource = Chef::Resource::Log.new(@log_str)
      @new_resource.level :info
      @provider = Chef::Provider::Log::ChefLog.new(@new_resource, @run_context)
      @provider.action_write
      @new_resource.updated.should be_true
  end

end
    it "updates the resource count if count_log_resource_updates=true" do
      Chef::Config[:count_log_resource_updates] = true
      expect(new_resource).to receive(:updated_by_last_action)
      provider.run_action(:write)
    end

    it "doesn't update the resource count if count_log_resource_updates=false" do
      Chef::Config[:count_log_resource_updates] = false
      expect(new_resource).not_to receive(:updated_by_last_action)
      provider.run_action(:write)
    end
  end
end

<MSG> Revert "CHEF-4028 - Log resource no longer counts as an updated resource unless a write actually happens"

This reverts commit 77119f0f37cc100d450e4f2d2f9dcfb9a57a74a8.

<DFF> @@ -78,22 +78,4 @@ describe Chef::Provider::Log::ChefLog do
       @provider.action_write
   end
 
-  it "should not update the resource if the message was not written to the log" do
-      Chef::Log.level = :fatal
-      @new_resource = Chef::Resource::Log.new(@log_str)
-      @new_resource.level :info
-      @provider = Chef::Provider::Log::ChefLog.new(@new_resource, @run_context)
-      @provider.action_write
-      @new_resource.updated.should be_false
-  end
-
-  it "should update the resource if the message has been written to the log" do
-      Chef::Log.level = :debug
-      @new_resource = Chef::Resource::Log.new(@log_str)
-      @new_resource.level :info
-      @provider = Chef::Provider::Log::ChefLog.new(@new_resource, @run_context)
-      @provider.action_write
-      @new_resource.updated.should be_true
-  end
-
 end

{'addition_count': 0, 'commit_subject': 'Revert "CHEF-4028 - Log resource no longer counts as an updated resource unless a write actually happens"', 'deletion_count': 18, 'file_extension': '.rb', 'lang': 'rb', 'license': 'apache-2.0', 'repo_name': 'chef/chef'}