diff options
-rw-r--r-- | lib/chef/exceptions.rb | 1 | ||||
-rw-r--r-- | lib/chef/util/dsc/lcm_output_parser.rb | 10 | ||||
-rw-r--r-- | lib/chef/util/dsc/local_configuration_manager.rb | 2 | ||||
-rw-r--r-- | spec/unit/util/dsc/lcm_output_parser_spec.rb | 19 |
4 files changed, 16 insertions, 16 deletions
diff --git a/lib/chef/exceptions.rb b/lib/chef/exceptions.rb index f5d91c24a6..48ab821cab 100644 --- a/lib/chef/exceptions.rb +++ b/lib/chef/exceptions.rb @@ -124,6 +124,7 @@ class Chef class DuplicateDataBagItem < RuntimeError; end class PowershellCmdletException < RuntimeError; end + class LCMParserException < RuntimeError; end class CannotDetermineHomebrewOwner < Package; end diff --git a/lib/chef/util/dsc/lcm_output_parser.rb b/lib/chef/util/dsc/lcm_output_parser.rb index f8f853a33a..37908d064b 100644 --- a/lib/chef/util/dsc/lcm_output_parser.rb +++ b/lib/chef/util/dsc/lcm_output_parser.rb @@ -18,6 +18,7 @@ require 'chef/log' require 'chef/util/dsc/resource_info' +require 'chef/exceptions' class Chef class Util @@ -53,8 +54,7 @@ class Chef # ] # def self.parse(lcm_output) - return [] unless lcm_output - + lcm_output ||= "" current_resource = Hash.new resources = [] @@ -96,7 +96,11 @@ class Chef resources.push(current_resource) end - build_resource_info(resources) + if resources.length > 0 + build_resource_info(resources) + else + raise Chef::Exceptions::LCMParserException, "Could not parse:\n#{lcm_output}" + end end def self.parse_line(line) diff --git a/lib/chef/util/dsc/local_configuration_manager.rb b/lib/chef/util/dsc/local_configuration_manager.rb index f498a2bfea..55ffc7943c 100644 --- a/lib/chef/util/dsc/local_configuration_manager.rb +++ b/lib/chef/util/dsc/local_configuration_manager.rb @@ -103,7 +103,7 @@ EOH Chef::Log.debug("DSC: DSC returned the following '-whatif' output from test operation:\n#{what_if_output}") begin Parser::parse(what_if_output) - rescue Chef::Util::DSC::LocalConfigurationManager::Parser => e + rescue Chef::Exceptions::LCMParserException => e Chef::Log::warn("Could not parse LCM output: #{e}") [Chef::Util::DSC::ResourceInfo.new('Unknown DSC Resources', true, ['Unknown changes because LCM output was not parsable.'])] end diff --git a/spec/unit/util/dsc/lcm_output_parser_spec.rb b/spec/unit/util/dsc/lcm_output_parser_spec.rb index ba4f40c4f7..426102caca 100644 --- a/spec/unit/util/dsc/lcm_output_parser_spec.rb +++ b/spec/unit/util/dsc/lcm_output_parser_spec.rb @@ -20,24 +20,19 @@ require 'chef/util/dsc/lcm_output_parser' describe Chef::Util::DSC::LocalConfigurationManager::Parser do context 'empty input parameter' do - it 'returns an empty array for a 0 length string' do - expect(Chef::Util::DSC::LocalConfigurationManager::Parser::parse('')).to be_empty + it 'raises an exception when there are no valid lines' do + str = <<-EOF + + EOF + expect {Chef::Util::DSC::LocalConfigurationManager::Parser::parse(str)}.to raise_error(Chef::Exceptions::LCMParserException) end - it 'returns an empty array for a nil input' do - expect(Chef::Util::DSC::LocalConfigurationManager::Parser::parse('')).to be_empty + it 'raises an exception for a nil input' do + expect {Chef::Util::DSC::LocalConfigurationManager::Parser::parse(nil)}.to raise_error(Chef::Exceptions::LCMParserException) end end context 'correctly formatted output from lcm' do - it 'returns an empty array for a log with no resources' do - str = <<EOF -logtype: [machinename]: LCM: [ Start Set ] -logtype: [machinename]: LCM: [ End Set ] -EOF - expect(Chef::Util::DSC::LocalConfigurationManager::Parser::parse(str)).to be_empty - end - it 'returns a single resource when only 1 logged with the correct name' do str = <<EOF logtype: [machinename]: LCM: [ Start Set ] |