[Groonga-commit] groonga/grntest at fce0910 [master] Fix a bug that bracktrace log isn't detected

Back to archive index

Kouhei Sutou null+****@clear*****
Sat Aug 16 22:15:39 JST 2014


Kouhei Sutou	2014-08-16 22:15:39 +0900 (Sat, 16 Aug 2014)

  New Revision: fce0910de2581353d066fae956b7d5f839cd825b
  https://github.com/groonga/grntest/commit/fce0910de2581353d066fae956b7d5f839cd825b

  Message:
    Fix a bug that bracktrace log isn't detected
    
    `message` had new line.
    
    [groonga-dev,02663]
    
    Reported by Naoya Murakami. Thanks!!!

  Modified files:
    lib/grntest/executors/base-executor.rb

  Modified: lib/grntest/executors/base-executor.rb (+2 -1)
===================================================================
--- lib/grntest/executors/base-executor.rb    2014-08-16 15:32:32 +0900 (4806f69)
+++ lib/grntest/executors/base-executor.rb    2014-08-16 22:15:39 +0900 (abede82)
@@ -276,11 +276,12 @@ module Grntest
         log.each_line do |line|
           timestamp, log_level, message = line.split(/\|\s*/, 3)
           _ = timestamp # suppress warning
+          message = message.chomp
           next unless important_log_level?(log_level)
           next if backtrace_log_message?(message)
           important_messages << "\#|#{log_level}| #{message}"
         end
-        important_messages.chomp
+        important_messages
       end
 
       def read_all_readable_content(output, options={})
-------------- next part --------------
HTML����������������������������...
Download 



More information about the Groonga-commit mailing list
Back to archive index