lib/roku_builder/plugins/analyzer.rb in roku_builder-4.9.2 vs lib/roku_builder/plugins/analyzer.rb in roku_builder-4.9.3

@@ -63,11 +63,12 @@
         end
         unless has_source_dir
           add_warning(warning: :packageSourceDirectory, path: "source")
         end
         @warnings.concat(raf_inspector.run(analyzer_config[:inspectors]))
-        print_warnings(dir) unless quiet
+        format_messages(dir)
+        print_warnings unless quiet
       end
       @warnings
     end
 
     private
@@ -84,30 +85,36 @@
     def add_warning(warning:, path:)
       @warnings.push(@inspector_config[warning].deep_dup)
       @warnings.last[:path] = path
     end
 
-    def print_warnings(dir)
+    def print_warnings
       logger = ::Logger.new(STDOUT)
       logger.level  = @logger.level
       logger.formatter = proc {|severity, _datetime, _progname, msg|
         "%5s: %s\n\r" % [severity, msg]
       }
+      @logger.unknown "====== Analysis Results ======"
       @warnings.each do |warning|
         message = warning[:message]
-        if warning[:path]
-          warning[:path].slice!(dir) if dir
-          warning[:path].slice!(/^\//)
-          message += ". pkg:/"+warning[:path]
-          message += ":"+(warning[:line]+1).to_s if warning[:line]
-        end
         case(warning[:severity])
         when "error"
           logger.error(message)
         when "warning"
           logger.warn(message)
         when "info"
           logger.info(message)
+        end
+      end
+    end
+
+    def format_messages(dir)
+      @warnings.each do |warning|
+        if warning[:path]
+          warning[:path].slice!(dir) if dir
+          warning[:path].slice!(/^\//)
+          warning[:message] += ". pkg:/"+warning[:path]
+          warning[:message] += ":"+(warning[:line]+1).to_s if warning[:line]
         end
       end
     end
   end
   RokuBuilder.register_plugin(Analyzer)