Kouhei Sutou 2019-03-26 16:10:28 +0900 (Tue, 26 Mar 2019) Revision: 17f5aed282b88435a7e4cc77937e19fc92e89dd8 https://github.com/groonga/groonga-query-log/commit/17f5aed282b88435a7e4cc77937e19fc92e89dd8 Message: Move notify check to notifier Modified files: lib/groonga-query-log/command/run-regression-test.rb Modified: lib/groonga-query-log/command/run-regression-test.rb (+6 -4) =================================================================== --- lib/groonga-query-log/command/run-regression-test.rb 2019-03-26 16:09:02 +0900 (b0906f2) +++ lib/groonga-query-log/command/run-regression-test.rb 2019-03-26 16:10:28 +0900 (75bf878) @@ -90,10 +90,10 @@ module GroongaQueryLog tester_options) success = tester.run - if @notifier_options[:mail_to] - notifier = MailNotifier.new(success, Time.now - @start_time, @notifier_options) - notifier.notify - end + notifier = MailNotifier.new(success, + Time.now - @start_time, + @notifier_options) + notifier.notify success end @@ -661,6 +661,8 @@ module GroongaQueryLog end def notify(output=nil) + return unless @options[:mail_to] + format_log = "" @output = output || StringIO.new options = {:output => @output} -------------- next part -------------- An HTML attachment was scrubbed... URL: <https://lists.osdn.me/mailman/archives/groonga-commit/attachments/20190326/8d9031a0/attachment.html>
