Fix bug where error parser what-if output causes resource to be considered converged.

We now fall back to assuming the resource is not converged if we cannot parse
information about a dsc resource.

2 files changed, 8 insertions, 4 deletions

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