groonga-query-log 1.6.1 → 1.6.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7420f29c336d80bf892fa8c8a5a8368ee6344f2e6d02846f7da1172daca83d7b
-  data.tar.gz: 0a6833bd88e699ea7e102a0466b4db7944d273a84adb6ccc7e9262795cf1e5c0
+  metadata.gz: 292e6cac875522bbe44ce9e3ba9720662607e73e2caff72dbb64ef01b403c353
+  data.tar.gz: 7c8ca5a6868504360814aae0c8726cdcbe909ba1806aa8a4c99e1b5324b85087
 SHA512:
-  metadata.gz: e4e4fbae0c9fb4b9fe3d6b474b2a77fb250f4a48c7a1775fd9aaed07a527a35c40e543fb21a250a4ef47ebadd75ef7626da17630462b52eef9395b9149ddef0a
-  data.tar.gz: 06fcd20a1e52b0edcd1697a4712ac8781b624a915fef57963518054fb501b1ec93c0b3d7195fbd3c898c5e6231139c5f1fd4781bf65fdb98c4a939057e620974
+  metadata.gz: 7ba5b0df58160f31cee7f9acb675bbd5c79b4f878550b184112ca8b557b591007cf8d32477beb7455e21737924785dba8305c2242a7e267a0e12b67bd3c14de6
+  data.tar.gz: 5be7b4777f3153332cea06f6b942b121a4983136dcf6b804f8c0e699b43b2fe70636a7cd8a0da9f4c240a6d2103c0b316e96c8afab659821d40cf0c78a6ea1b2

data/doc/text/news.md

@@ -1,5 +1,13 @@
 # News
 
+## 1.6.2: 2020-05-09
+
+### Improvements
+
+  * `run-regression-test`: Added leaked information to the first line.
+
+  * `run-regression-test`: Added `--mail-only-on-failure` option.
+
 ## 1.6.1: 2020-04-28
 
 ### Improvements

data/lib/groonga-query-log/command/run-regression-test.rb

@@ -79,6 +79,7 @@ module GroongaQueryLog
           mail_subject_on_failure: "Failure",
           mail_from: "groonga-query-log@#{Socket.gethostname}",
           mail_to: nil,
+          mail_only_on_failure: false,
           smtp_server: "localhost",
           smtp_auth_user: nil,
           smtp_auth_password: nil,
@@ -387,6 +388,11 @@ module GroongaQueryLog
                   "(#{@notifier_options[:mail_subject_on_failure]})") do |subject|
           @notifier_options[:mail_subject_on_failure] = subject
         end
+        parser.on("--[no-]mail-only-on-failure",
+                  "Send a notification e-mail only on failure",
+                  "(#{@notifier_options[:mail_only_on_failure]})") do |boolean|
+          @notifier_options[:mail_only_on_failure] = boolean
+        end
         parser
       end
 
@@ -464,10 +470,13 @@ module GroongaQueryLog
         formatted = format_elapsed_time(elapsed_time)
         formatted << "The number of executed commands: #{n_executed_commands}\n"
         if success
-          formatted << "Success\n"
+          formatted << "Success"
+          formatted << " but leaked" if n_leaked_objects > 0
         else
-          formatted << "Failure\n"
+          formatted << "Failure"
+          formatted << " and leaked" if n_leaked_objects > 0
         end
+        formatted << "\n"
         unless n_leaked_objects.zero?
           formatted << "\nLeaked: #{n_leaked_objects}"
         end
@@ -612,12 +621,12 @@ module GroongaQueryLog
                 input.close
                 begin
                   pid, status = Process.waitpid2(pid)
-                  filter_pid, filter_status = Process.waitpid2(filter_pid)
+                  filter_pid, _filter_status = Process.waitpid2(filter_pid)
                 rescue Interrupt
                   Process.kill(:TERM, pid)
                   Process.kill(:TERM, filter_pid)
                   pid, status = Process.waitpid2(pid)
-                  filter_pid, filter_status = Process.waitpid2(filter_pid)
+                  filter_pid, _filter_status = Process.waitpid2(filter_pid)
                 end
               end
             else
@@ -897,6 +906,7 @@ module GroongaQueryLog
 
         def notify_started
           return unless @options[:mail_to]
+          return if @options[:mail_only_on_failure]
 
           subject = @options[:mail_subject_on_start]
           send_mail(subject, "")
@@ -907,6 +917,7 @@ module GroongaQueryLog
 
           if success
             subject = @options[:mail_subject_on_success]
+            return if @options[:mail_only_on_failure]
           else
             subject = @options[:mail_subject_on_failure]
           end

data/lib/groonga-query-log/version.rb

@@ -15,5 +15,5 @@
 # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301  USA
 
 module GroongaQueryLog
-  VERSION = "1.6.1"
+  VERSION = "1.6.2"
 end

data/test/command/test-run-regression-test.rb

@@ -217,6 +217,7 @@ QUIT
         :mail_to => "noreply@example.com",
         :mail_subject_on_success => "Success",
         :mail_subject_on_failure => "Failure",
+        :mail_only_on_failure => true,
         :path => fixture_path("mail-notifier/failure.log"),
       }
       notifier = MailNotifier.new(options)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: groonga-query-log
 version: !ruby/object:Gem::Version
-  version: 1.6.1
+  version: 1.6.2
 platform: ruby
 authors:
 - Kouhei Sutou
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-04-27 00:00:00.000000000 Z
+date: 2020-05-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: charty