test_notifier 2.0.3 → 2.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 5d8fd6d8f3c507c644caa30c41ecc00286ed974b
-  data.tar.gz: c6985dcc140c2b59dded50c59bcc538a217778aa
+SHA256:
+  metadata.gz: 48e80e39f269e2232354733498a32f2752e2d119786f5e6ed3694010194c006d
+  data.tar.gz: e24235a4358017441c12aab610ace1b43bfbaab5b65c899c3e128af53b9f7edf
 SHA512:
-  metadata.gz: ef1b1773a57e18902b2264e41464f08998fb72cbb4c74cb9b4688fb79d4572f646ab085f82646047e654182b97527bcaffba8889bbea4ec7f2b945c01118e387
-  data.tar.gz: 905c8b9129e4ca416ac8d0734beeb11cebcef1bb0c77e051596f22e34ef2aa05a4cf93236c633db52254ee13137a7c40ce900bdcc79afbd2948c7a9c30ca178b
+  metadata.gz: 80664726eaaa82e62db4970584a1cbe95246c2142ea09e88d0d8fbc8de438b1420ae9ad270f05d90a3d5d2c2543165942957915ef2f56114efbfa66605719179
+  data.tar.gz: 929599d9ea690c9e9bf7e189ce42c735abf9d83f961b16c70a3d1967053d516deb6d18bea39a212e4cd74412ff4b1e9669f0d41c85ef9920d6faea0ec4b96828

data/.gitignore

@@ -1,3 +1,4 @@
 *.gem
 pkg
-nbproject
+nbproject
+*.lock

data/.rubocop.yml

@@ -0,0 +1,9 @@
+---
+inherit_gem:
+  rubocop-fnando: .rubocop.yml
+
+AllCops:
+  TargetRubyVersion: 2.7
+  NewCops: enable
+  Exclude:
+    - vendor/**/*

data/Gemfile

@@ -1,2 +1,4 @@
-source "http://rubygems.org"
+# frozen_string_literal: true
+
+source "https://rubygems.org"
 gemspec

data/README.md

@@ -0,0 +1,81 @@
+# Test Notifier
+
+Inspired by
+http://railstips.org/2007/7/23/autotest-growl-pass-fail-notifications
+
+After using Growl notification, I decided to write my own plugin because I have
+to work on Ubuntu and Mac OS X and I missed the notification on my Linux box.
+This plugin works with Linux, Mac OS X and Windows. All you need to do is
+install the specific notification library for your OS.
+
+Instead of displaying lots of notifications for each failure, I prefer to be
+notified about the whole test result (you'll have to check your log file anyway
+in order to clean up the failures/errors).
+
+## Installation
+
+```
+gem install test_notifier
+```
+
+Check <https://github.com/fnando/notifier> to see how you can configure a
+notifier for your OS.
+
+## Usage
+
+If you're using Test::Unit you should add
+`require "test_notifier/runner/test_unit"` to your `test_helper.rb` file.
+
+If you're using RSpec you should add `require "test_notifier/runner/spec"` to
+your `spec_helper.rb` file. If you're using RSpec 2-3, you need to add
+`require "test_notifier/runner/rspec"` instead.
+
+If you're using Autotest you should add
+`require "test_notifier/runner/autotest"` to the file `~/.autotest`
+
+If you're using MiniTest you should add
+`require "test_notifier/runner/minitest"` to your `test_helper.rb` file.
+
+You can define your notifier.
+
+```ruby
+TestNotifier.default_notifier = :terminal_notifier
+```
+
+The available notifiers are `:terminal_notifier`, `:hud`, `:kdialog`,
+`:knotify`, `:notify_send`, `:osd_cat`, and `:snarl`.
+
+If you'd like to make Test Notifier optional for your project:
+
+```ruby
+TestNotifier.silence_no_notifier_warning = true
+```
+
+## Maintainer
+
+- Nando Vieira - https://nandovieira.com
+
+## Collaborators
+
+https://github.com/fnando/test_notifier/graphs/contributors
+
+## License
+
+(The MIT License)
+
+Permission is hereby granted, free of charge, to any person obtaining a copy of
+this software and associated documentation files (the 'Software'), to deal in
+the Software without restriction, including without limitation the rights to
+use, copy, modify, merge, publish, distribute, sublicense, and/or sell copies of
+the Software, and to permit persons to whom the Software is furnished to do so,
+subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included in all
+copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED 'AS IS', WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, FITNESS
+FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR
+COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER
+IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN
+CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.

data/Rakefile

@@ -1,5 +1,12 @@
+# frozen_string_literal: true
+
 require "bundler"
 Bundler::GemHelper.install_tasks
 
 require "rspec/core/rake_task"
 RSpec::Core::RakeTask.new
+
+require "rubocop/rake_task"
+RuboCop::RakeTask.new
+
+task default: %i[spec rubocop]

data/lib/test_notifier/runner/autotest.rb

@@ -1,36 +1,49 @@
+# frozen_string_literal: true
+
 require "test_notifier"
 
 Autotest.add_hook :ran_command do |at|
-  begin
-    content = at.results.to_s
+  content = at.results.to_s
+
+  rspec_re = /(\d+) examples?, (\d+) failures?(, (\d+) pendings?)?/
+  test_unit_re = /(\d+) tests, (\d+) assertions, (\d+) failures, (\d+) errors/
 
-    rspec_matches = content.match(/(\d+) examples?, (\d+) failures?(, (\d+) pendings?)?/)
-    test_unit_matches = content.match(/(\d+) tests, (\d+) assertions, (\d+) failures, (\d+) errors/)
+  rspec_matches = content.match(rspec_re)
+  test_unit_matches = content.match(test_unit_re)
 
-    if rspec_matches
-      _, examples, failures, _, pending = *rspec_matches
+  if rspec_matches
+    _, examples, failures, _, pending = *rspec_matches
 
-      stats = TestNotifier::Stats.new(:spec, {
-        :count    => examples,
-        :failures => failures,
-        :pending  => pending
-      })
+    stats = TestNotifier::Stats.new(
+      :spec,
+      {
+        count: examples,
+        failures: failures,
+        pending: pending
+      }
+    )
 
-      TestNotifier.notify(:status => stats.status, :message => stats.message) unless examples.to_i.zero?
-    elsif test_unit_matches
-      _, tests, assertions, failures, errors = *test_unit_matches
+    unless examples.to_i.zero?
+      TestNotifier.notify(status: stats.status, message: stats.message)
+    end
+  elsif test_unit_matches
+    _, tests, assertions, failures, errors = *test_unit_matches
 
-      stats = TestNotifier::Stats.new(:test_unit, {
-        :count      => tests,
-        :assertions => assertions,
-        :failures   => failures,
-        :errors     => errors
-      })
+    stats = TestNotifier::Stats.new(
+      :test_unit,
+      {
+        count: tests,
+        assertions: assertions,
+        failures: failures,
+        errors: errors
+      }
+    )
 
-      TestNotifier.notify(:status => stats.status, :message => stats.message) unless tests.to_i.zero?
+    unless tests.to_i.zero?
+      TestNotifier.notify(status: stats.status, message: stats.message)
     end
-  rescue => e
-    puts e
-    puts e.backtrace
   end
+rescue StandardError => error
+  puts error
+  puts error.backtrace
 end

data/lib/test_notifier/runner/minitest.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "test_notifier"
 require "minitest/unit"
 
@@ -5,11 +7,11 @@ MiniTest::Unit.after_tests do
   runner = MiniTest::Unit.runner
 
   stats = TestNotifier::Stats.new(:minitest, {
-    :count      => runner.test_count,
-    :assertions => runner.assertion_count,
-    :failures   => runner.failures,
-    :errors     => runner.errors
-  })
+                                    count: runner.test_count,
+                                    assertions: runner.assertion_count,
+                                    failures: runner.failures,
+                                    errors: runner.errors
+                                  })
 
-  TestNotifier.notify(:status => stats.status, :message => stats.message)
+  TestNotifier.notify(status: stats.status, message: stats.message)
 end

data/lib/test_notifier/runner/rspec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "rspec/core/version"
 
 if RSpec::Core::Version::STRING >= "3.0.0"

data/lib/test_notifier/runner/rspec2.rb

@@ -1,29 +1,50 @@
+# frozen_string_literal: true
+
 require "test_notifier"
 require "rspec/core/formatters/base_text_formatter"
 
-class RSpec::Core::Formatters::BaseTextFormatter
-  alias dump_summary_original dump_summary
+module RSpec
+  module Core
+    module Formatters
+      class BaseTextFormatter
+        alias dump_summary_original dump_summary
 
-  def dump_summary(duration, example_count, failure_count, pending_count)
-    dump_summary_original(duration, example_count, failure_count, pending_count)
+        def dump_summary(duration, example_count, failure_count, pending_count)
+          dump_summary_original(
+            duration,
+            example_count,
+            failure_count,
+            pending_count
+          )
 
-    return if example_count.zero?
+          return if example_count.zero?
 
-    failure_filter = proc {|e|
-      e.instance_variable_get("@exception").class.name == "RSpec::Expectations::ExpectationNotMetError"
-    }
+          failure_filter = proc do |error|
+            error
+              .instance_variable_get(:@exception)
+              .instance_of?(RSpec::Expectations::ExpectationNotMetError)
+          end
 
-    error_filter = proc {|e|
-      %w[RSpec::Expectations::ExpectationNotMetError NilClass].include?(e.instance_variable_get("@exception").class.name)
-    }
+          error_filter = proc do |error|
+            %w[
+              RSpec::Expectations::ExpectationNotMetError
+              NilClass
+            ].include?(error.instance_variable_get(:@exception).class.name)
+          end
 
-    stats = TestNotifier::Stats.new(:rspec, {
-      :count    => example_count,
-      :failures => examples.select(&failure_filter).count,
-      :pending  => pending_count,
-      :errors   => examples.reject(&error_filter).count
-    })
+          stats = TestNotifier::Stats.new(
+            :rspec,
+            {
+              count: example_count,
+              failures: examples.count(&failure_filter),
+              pending: pending_count,
+              errors: examples.count(&error_filter)
+            }
+          )
 
-    TestNotifier.notify(:status => stats.status, :message => stats.message)
+          TestNotifier.notify(status: stats.status, message: stats.message)
+        end
+      end
+    end
   end
 end

data/lib/test_notifier/runner/rspec3.rb

@@ -1,35 +1,50 @@
+# frozen_string_literal: true
+
 require "test_notifier"
 require "rspec/core"
 require "rspec/core/formatters/base_text_formatter"
 
-class RSpec::Core::Formatters::BaseTextFormatter
-  alias dump_summary_original dump_summary
-
-  def dump_summary(options)
-    dump_summary_original(options)
-
-    example_count = options.example_count
-    failure_count = options.failure_count
-    pending_count = options.pending_count
-    examples = options.examples
-
-    return if example_count.zero?
-
-    failure_filter = proc {|e|
-      e.instance_variable_get("@exception").class.name == "RSpec::Expectations::ExpectationNotMetError"
-    }
-
-    error_filter = proc {|e|
-      %w[RSpec::Expectations::ExpectationNotMetError NilClass].include?(e.instance_variable_get("@exception").class.name)
-    }
-
-    stats = TestNotifier::Stats.new(:rspec, {
-      :count    => example_count,
-      :failures => examples.select(&failure_filter).count,
-      :pending  => pending_count,
-      :errors   => examples.reject(&error_filter).count
-    })
-
-    TestNotifier.notify(:status => stats.status, :message => stats.message)
+module RSpec
+  module Core
+    module Formatters
+      class BaseTextFormatter
+        alias dump_summary_original dump_summary
+
+        def dump_summary(options)
+          dump_summary_original(options)
+
+          example_count = options.example_count
+          pending_count = options.pending_count
+          examples = options.examples
+
+          return if example_count.zero?
+
+          failure_filter = proc do |error|
+            error
+              .instance_variable_get(:@exception)
+              .instance_of?(RSpec::Expectations::ExpectationNotMetError)
+          end
+
+          error_filter = proc {|e|
+            %w[
+              RSpec::Expectations::ExpectationNotMetError
+              NilClass
+            ].include?(e.instance_variable_get(:@exception).class.name)
+          }
+
+          stats = TestNotifier::Stats.new(
+            :rspec,
+            {
+              count: example_count,
+              failures: examples.count(&failure_filter),
+              pending: pending_count,
+              errors: examples.count(&error_filter)
+            }
+          )
+
+          TestNotifier.notify(status: stats.status, message: stats.message)
+        end
+      end
+    end
   end
 end

data/lib/test_notifier/runner/spec.rb

@@ -1,21 +1,30 @@
+# frozen_string_literal: true
+
 require "test_notifier"
 require "spec/runner/formatter/base_text_formatter"
 
-class Spec::Runner::Formatter::BaseTextFormatter
-  alias dump_summary_original dump_summary
+module Spec
+  module Runner
+    module Formatter
+      class BaseTextFormatter
+        alias dump_summary_original dump_summary
 
-  def dump_summary(duration, example_count, failure_count, pending_count)
-    dump_summary_original(duration, example_count, failure_count, pending_count)
+        def dump_summary(duration, example_count, failure_count, pending_count)
+          dump_summary_original(duration, example_count, failure_count,
+                                pending_count)
 
-    return if example_count.zero?
+          return if example_count.zero?
 
-    stats = TestNotifier::Stats.new(:spec, {
-      :count    => example_count,
-      :failures => failure_count,
-      :pending  => pending_count,
-      :errors   => nil
-    })
+          stats = TestNotifier::Stats.new(:spec, {
+                                            count: example_count,
+                                            failures: failure_count,
+                                            pending: pending_count,
+                                            errors: nil
+                                          })
 
-    TestNotifier.notify(:status => stats.status, :message => stats.message)
+          TestNotifier.notify(status: stats.status, message: stats.message)
+        end
+      end
+    end
   end
 end

data/lib/test_notifier/runner/test_unit.rb

@@ -1,29 +1,38 @@
+# frozen_string_literal: true
+
 require "test_notifier"
 require "test/unit/ui/console/testrunner"
 
-class Test::Unit::UI::Console::TestRunner
-  alias finished_original finished
-
-  def finished(elapsed_time)
-    finished_original(elapsed_time)
+module Test
+  module Unit
+    module UI
+      module Console
+        class TestRunner
+          alias finished_original finished
 
-    begin
-      re = /(\d+) tests, (\d+) assertions, (\d+) failures, (\d+) errors/
-      _, tests, assertions, failures, errors = *@result.to_s.match(re)
-      return if tests.to_i.zero?
+          def finished(elapsed_time)
+            finished_original(elapsed_time)
 
+            begin
+              re = /(\d+) tests, (\d+) assertions, (\d+) failures, (\d+) errors/
+              _, tests, assertions, failures, errors = *@result.to_s.match(re)
+              return if tests.to_i.zero?
 
-      stats = TestNotifier::Stats.new(:test_unit, {
-        :count      => tests,
-        :assertions => assertions,
-        :failures   => failures,
-        :errors     => errors
-      })
+              stats = TestNotifier::Stats.new(:test_unit, {
+                                                count: tests,
+                                                assertions: assertions,
+                                                failures: failures,
+                                                errors: errors
+                                              })
 
-      TestNotifier.notify(:status => stats.status, :message => stats.message)
-    rescue => e
-      puts e
-      puts e.backtrace
+              TestNotifier.notify(status: stats.status, message: stats.message)
+            rescue StandardError => error
+              puts error
+              puts error.backtrace
+            end
+          end
+        end
+      end
     end
   end
 end

data/lib/test_notifier/runner.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module TestNotifier
   module Runner
     autoload :RSpec,    "test_notifier/runner/rspec"

data/lib/test_notifier/stats.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module TestNotifier
   class Stats
     attr_accessor :adapter, :options
@@ -17,15 +19,16 @@ module TestNotifier
       send("message_for_#{adapter}")
     end
 
-    private
-    def normalize(options)
-      [:count, :success, :failures, :pending, :errors, :assertions].inject({}) do |buffer, key|
+    private def normalize(options)
+      %i[
+        count success failures pending errors
+        assertions
+      ].each_with_object({}) do |key, buffer|
         buffer[key] = options[key].to_i
-        buffer
       end
     end
 
-    def status_for_minitest
+    private def status_for_minitest
       if options[:errors].nonzero?
         :error
       elsif options[:failures].nonzero?
@@ -35,7 +38,7 @@ module TestNotifier
       end
     end
 
-    def status_for_test_unit
+    private def status_for_test_unit
       if options[:errors].nonzero?
         :error
       elsif options[:failures].nonzero?
@@ -45,7 +48,7 @@ module TestNotifier
       end
     end
 
-    def status_for_rspec
+    private def status_for_rspec
       if options[:errors].nonzero?
         :error
       elsif options[:failures].nonzero?
@@ -55,7 +58,7 @@ module TestNotifier
       end
     end
 
-    def status_for_spec
+    private def status_for_spec
       if options[:failures].nonzero?
         :fail
       else
@@ -63,44 +66,56 @@ module TestNotifier
       end
     end
 
-    def message_for_rspec
-      options[:success] = options[:count] - (options[:failures] + options[:errors])
+    private def message_for_rspec
+      options[:success] =
+        options[:count] - (options[:failures] + options[:errors])
 
       text = []
-      text << "#{options[:count]} " + pluralize(options[:count], "example")
+      text << ("#{options[:count]} " + pluralize(options[:count], "example"))
       text << "#{options[:failures]} failed" unless options[:failures].zero?
       text << "#{options[:pending]} pending" unless options[:pending].zero?
-      text << "#{options[:errors]} " + pluralize(options[:errors], "error") unless options[:errors].zero?
+      unless options[:errors].zero?
+        text << ("#{options[:errors]} " + pluralize(options[:errors],
+                                                    "error"))
+      end
       text.join(", ")
     end
 
-    def message_for_spec
+    private def message_for_spec
       text = []
-      text << "#{options[:count]} " + pluralize(options[:count], "example")
+      text << ("#{options[:count]} " + pluralize(options[:count], "example"))
       text << "#{options[:failures]} failed" unless options[:failures].zero?
       text << "#{options[:pending]} pending" unless options[:pending].zero?
       text.join(", ")
     end
 
-    def message_for_test_unit
+    private def message_for_test_unit
       text = []
-      text << "#{options[:count]} " + pluralize(options[:count], "test")
-      text << "#{options[:assertions]} " + pluralize(options[:assertions], "assertion")
+      text << ("#{options[:count]} " + pluralize(options[:count], "test"))
+      text << ("#{options[:assertions]} " + pluralize(options[:assertions],
+                                                      "assertion"))
       text << "#{options[:failures]} failed" unless options[:failures].zero?
-      text << "#{options[:errors]} " + pluralize(options[:errors], "error") unless options[:errors].zero?
+      unless options[:errors].zero?
+        text << ("#{options[:errors]} " + pluralize(options[:errors],
+                                                    "error"))
+      end
       text.join(", ")
     end
 
-    def message_for_minitest
+    private def message_for_minitest
       text = []
-      text << "#{options[:count]} " + pluralize(options[:count], "test")
-      text << "#{options[:assertions]} " + pluralize(options[:assertions], "assertion")
+      text << ("#{options[:count]} " + pluralize(options[:count], "test"))
+      text << ("#{options[:assertions]} " + pluralize(options[:assertions],
+                                                      "assertion"))
       text << "#{options[:failures]} failed" unless options[:failures].zero?
-      text << "#{options[:errors]} " + pluralize(options[:errors], "error") unless options[:errors].zero?
+      unless options[:errors].zero?
+        text << ("#{options[:errors]} " + pluralize(options[:errors],
+                                                    "error"))
+      end
       text.join(", ")
     end
 
-    def pluralize(count, text)
+    private def pluralize(count, text)
       count > 1 ? "#{text}s" : text
     end
   end

data/lib/test_notifier/version.rb

@@ -1,8 +1,10 @@
+# frozen_string_literal: true
+
 module TestNotifier
   module Version
     MAJOR = 2
-    MINOR = 0
-    PATCH = 3
+    MINOR = 1
+    PATCH = 0
     STRING = "#{MAJOR}.#{MINOR}.#{PATCH}"
   end
 end

data/lib/test_notifier.rb

@@ -1,42 +1,53 @@
+# frozen_string_literal: true
+
 require "notifier"
 
 module TestNotifier
-  class << self
-    attr_accessor :silence_no_notifier_warning
-  end
-
   extend self
 
-  NO_NOTIFIERS_MESSAGE = "[TEST NOTIFIER] You have no supported notifiers installed. Please read documentation.\n"
+  NO_NOTIFIERS_MESSAGE = "[TEST NOTIFIER] You have no supported notifiers " \
+                         "installed. Please read documentation.\n"
 
   IMAGES = {
-    :fail    => File.dirname(__FILE__) + "/../resources/fail.png",
-    :error   => File.dirname(__FILE__) + "/../resources/error.png",
-    :success => File.dirname(__FILE__) + "/../resources/success.png"
-  }
+    fail: File.expand_path("#{__dir__}/../resources/fail.png"),
+    error: File.expand_path("#{__dir__}/../resources/error.png"),
+    success: File.expand_path("#{__dir__}/../resources/success.png")
+  }.freeze
+
+  HUD_SYMBOLS = {
+    fail: "exclamationmark.triangle",
+    error: "xmark.octagon.fill",
+    success: "checkmark.circle"
+  }.freeze
 
   TITLES = {
-    :fail    => "Failed!",
-    :success => "Passed!",
-    :error   => "Error!"
-  }
+    fail: "Failed!",
+    success: "Passed!",
+    error: "Error!"
+  }.freeze
 
   COLORS = {
-    :fail    => "orange",
-    :success => "green",
-    :error   => "red"
-  }
+    fail: "orange",
+    success: "green",
+    error: "red"
+  }.freeze
+
+  attr_accessor :silence_no_notifier_warning
 
   def default_notifier=(notifier)
     Notifier.default_notifier = notifier
   end
 
   def notify(options)
-    options.merge!({
-      :title => TITLES[options[:status]],
-      :image => IMAGES[options[:status]],
-      :color => COLORS[options[:status]]
-    })
+    options = options.merge(
+      title: TITLES[options[:status]],
+      image: IMAGES[options[:status]],
+      color: COLORS[options[:status]]
+    )
+
+    if Notifier.notifier == Notifier::Hud
+      options[:image] = HUD_SYMBOLS[options[:status]]
+    end
 
     notifier.notify(options)
   end
@@ -45,7 +56,7 @@ module TestNotifier
     notifier = Notifier.notifier
 
     if notifier == Notifier::Placebo && !silence_no_notifier_warning
-      STDERR << NO_NOTIFIERS_MESSAGE
+      $stderr << NO_NOTIFIERS_MESSAGE
     end
 
     notifier

data/spec/spec_helper.rb

@@ -1,9 +1,14 @@
+# frozen_string_literal: true
+
+require "bundler/setup"
 require "test_notifier"
 
 module SpecHelpers
   def unsupport_all_notifiers
     Notifier.notifiers.each do |notifier|
-      allow(notifier).to receive(:supported?).and_return(false) unless notifier == Notifier::Placebo
+      unless notifier == Notifier::Placebo
+        allow(notifier).to receive(:supported?).and_return(false)
+      end
     end
   end
 end

data/spec/stats/minitest_spec.rb

@@ -1,26 +1,30 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 describe TestNotifier::Stats, "MiniTest" do
   subject(:stats) { TestNotifier::Stats.new(:minitest) }
 
   it "returns success message" do
-    stats.options = { :count => 10, :assertions => 20 }
+    stats.options = {count: 10, assertions: 20}
     expect(stats.message).to eql("10 tests, 20 assertions")
   end
 
   it "message with failing examples" do
-    stats.options = { :count => 10, :assertions => 20, :failures => 5 }
+    stats.options = {count: 10, assertions: 20, failures: 5}
     expect(stats.message).to eql("10 tests, 20 assertions, 5 failed")
   end
 
   it "message with error examples" do
-    stats.options = { :count => 10, :assertions => 20, :errors => 5 }
+    stats.options = {count: 10, assertions: 20, errors: 5}
     expect(stats.message).to eql("10 tests, 20 assertions, 5 errors")
   end
 
   it "message with all types" do
-    stats.options = { :count => 6, :failures => 2, :errors => 3, :assertions => 20 }
+    stats.options = {
+      count: 6, failures: 2, errors: 3,
+      assertions: 20
+    }
     expect(stats.message).to eql("6 tests, 20 assertions, 2 failed, 3 errors")
   end
 end
-

data/spec/stats/rspec1_spec.rb

@@ -1,25 +1,27 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 describe TestNotifier::Stats, "RSpec 1" do
   subject(:stats) { TestNotifier::Stats.new(:spec) }
 
   it "returns success message" do
-    stats.options = { :count => 10 }
+    stats.options = {count: 10}
     expect(stats.message).to eql("10 examples")
   end
 
   it "returns message with failing examples" do
-    stats.options = { :count => 10, :failures => 5 }
+    stats.options = {count: 10, failures: 5}
     expect(stats.message).to eql("10 examples, 5 failed")
   end
 
   it "returns message with pending examples" do
-    stats.options = { :count => 10, :pending => 5 }
+    stats.options = {count: 10, pending: 5}
     expect(stats.message).to eql("10 examples, 5 pending")
   end
 
   it "returns message with all types" do
-    stats.options = { :count => 6, :failures => 2, :pending => 3 }
+    stats.options = {count: 6, failures: 2, pending: 3}
     expect(stats.message).to eql("6 examples, 2 failed, 3 pending")
   end
 end

data/spec/stats/rspec_spec.rb

@@ -1,30 +1,32 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 describe TestNotifier::Stats, "RSpec" do
   subject(:stats) { TestNotifier::Stats.new(:rspec) }
 
   it "returns success message" do
-    stats.options = { :count => 10 }
+    stats.options = {count: 10}
     expect(stats.message).to eql("10 examples")
   end
 
   it "returns message with failing examples" do
-    stats.options = { :count => 10, :failures => 5 }
+    stats.options = {count: 10, failures: 5}
     expect(stats.message).to eql("10 examples, 5 failed")
   end
 
   it "returns message with pending examples" do
-    stats.options = { :count => 10, :pending => 5 }
+    stats.options = {count: 10, pending: 5}
     expect(stats.message).to eql("10 examples, 5 pending")
   end
 
   it "returns message with error examples" do
-    stats.options = { :count => 10, :failures => 5, :errors => 5 }
+    stats.options = {count: 10, failures: 5, errors: 5}
     expect(stats.message).to eql("10 examples, 5 failed, 5 errors")
   end
 
   it "returns message with all types" do
-    stats.options = { :count => 6, :failures => 3, :errors => 2, :pending => 3 }
+    stats.options = {count: 6, failures: 3, errors: 2, pending: 3}
     expect(stats.message).to eql("6 examples, 3 failed, 3 pending, 2 errors")
   end
 end

data/spec/stats/test_unit_spec.rb

@@ -1,25 +1,30 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 describe TestNotifier::Stats, "TestUnit" do
   subject(:stats) { TestNotifier::Stats.new(:test_unit) }
 
   it "returns success message" do
-    stats.options = { :count => 10, :assertions => 20 }
+    stats.options = {count: 10, assertions: 20}
     expect(stats.message).to eql("10 tests, 20 assertions")
   end
 
   it "returns message with failing examples" do
-    stats.options = { :count => 10, :assertions => 20, :failures => 5 }
+    stats.options = {count: 10, assertions: 20, failures: 5}
     expect(stats.message).to eql("10 tests, 20 assertions, 5 failed")
   end
 
   it "message with error examples" do
-    stats.options = { :count => 10, :assertions => 20, :errors => 5 }
+    stats.options = {count: 10, assertions: 20, errors: 5}
     expect(stats.message).to eql("10 tests, 20 assertions, 5 errors")
   end
 
   it "message with all types" do
-    stats.options = { :count => 6, :failures => 2, :errors => 3, :assertions => 20 }
+    stats.options = {
+      count: 6, failures: 2, errors: 3,
+      assertions: 20
+    }
     expect(stats.message).to eql("6 tests, 20 assertions, 2 failed, 3 errors")
   end
 end

data/spec/test_notifier_spec.rb

@@ -1,10 +1,12 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 describe TestNotifier do
   before { unsupport_all_notifiers }
 
   it "uses default notifier" do
-    allow(Notifier::Growl).to receive(:supported?).and_return(true)
+    allow(Notifier::Hud).to receive(:supported?).and_return(true)
     allow(Notifier::Snarl).to receive(:supported?).and_return(true)
     TestNotifier.default_notifier = :snarl
 
@@ -12,40 +14,78 @@ describe TestNotifier do
   end
 
   it "outputs error message to $stderr when there's no supported notifier" do
-    expect(STDERR).to receive(:<<).with(TestNotifier::NO_NOTIFIERS_MESSAGE)
+    expect($stderr).to receive(:<<).with(TestNotifier::NO_NOTIFIERS_MESSAGE)
     expect(Notifier::Placebo).to receive(:notify)
 
-    TestNotifier.notify :status => :fail, :message => "You have failed!"
+    TestNotifier.notify status: :fail, message: "You have failed!"
   end
 
-  it "outputs error message won't display when silence_no_notifier_warning is true" do
+  it "doesn't output error message when silence_no_notifier_warning is true" do
     TestNotifier.silence_no_notifier_warning = true
 
-    expect(STDERR).not_to receive(:<<)
+    expect($stderr).not_to receive(:<<)
     expect(Notifier::Placebo).to receive(:notify)
 
-    TestNotifier.notify :status => :fail, :message => "You have failed!"
+    TestNotifier.notify status: :fail, message: "You have failed!"
   end
 
-  it "outputs error message won't display when silence_no_notifier_warning is true" do
-    TestNotifier.silence_no_notifier_warning = true
+  it "sends notification to supported notifier" do
+    args = {
+      status: :fail,
+      message: "You have failed!",
+      title: TestNotifier::TITLES[:fail],
+      image: TestNotifier::IMAGES[:fail],
+      color: TestNotifier::COLORS[:fail]
+    }
 
-    expect(STDERR).not_to receive(:<<)
-    expect(Notifier::Placebo).to receive(:notify)
+    allow(Notifier::Snarl).to receive(:supported?).and_return(true)
+    expect(Notifier::Snarl).to receive(:notify).with(args)
 
-    TestNotifier.notify :status => :fail, :message => "You have failed!"
+    TestNotifier.notify status: :fail, message: "You have failed!"
   end
 
-  it "sends notification to supported notifier" do
-    allow(Notifier::Snarl).to receive(:supported?).and_return(true)
-    expect(Notifier::Snarl).to receive(:notify).with({
-      :status  => :fail,
-      :message => "You have failed!",
-      :title   => TestNotifier::TITLES[:fail],
-      :image   => TestNotifier::IMAGES[:fail],
-      :color   => TestNotifier::COLORS[:fail]
-    })
-
-    TestNotifier.notify :status => :fail, :message => "You have failed!"
+  it "sets symbol name for hud notifier (failed)" do
+    args = {
+      message: "You have failed!",
+      title: "Failed!",
+      image: "exclamationmark.triangle",
+      color: "orange",
+      status: :fail
+    }
+
+    allow(Notifier::Hud).to receive(:supported?).and_return(true)
+    expect(Notifier::Hud).to receive(:notify).with(args)
+
+    TestNotifier.notify status: :fail, message: "You have failed!"
+  end
+
+  it "sets symbol name for hud notifier (success)" do
+    args = {
+      message: "You passed!",
+      title: "Passed!",
+      image: "checkmark.circle",
+      color: "green",
+      status: :success
+    }
+
+    allow(Notifier::Hud).to receive(:supported?).and_return(true)
+    expect(Notifier::Hud).to receive(:notify).with(args)
+
+    TestNotifier.notify status: :success, message: "You passed!"
+  end
+
+  it "sets symbol name for hud notifier (error)" do
+    args = {
+      message: "You have errored!",
+      title: "Error!",
+      image: "xmark.octagon.fill",
+      color: "red",
+      status: :error
+    }
+
+    allow(Notifier::Hud).to receive(:supported?).and_return(true)
+    expect(Notifier::Hud).to receive(:notify).with(args)
+
+    TestNotifier.notify status: :error, message: "You have errored!"
   end
 end

data/test_notifier.gemspec

@@ -1,27 +1,32 @@
-# -*- encoding: utf-8 -*-
-$:.push File.expand_path("../lib", __FILE__)
-require "test_notifier/version"
+# frozen_string_literal: true
+
+require_relative "lib/test_notifier/version"
 
 Gem::Specification.new do |s|
   s.name        = "test_notifier"
-  s.version     = "#{TestNotifier::Version::STRING}"
+  s.version     = TestNotifier::Version::STRING.to_s
   s.platform    = Gem::Platform::RUBY
   s.authors     = ["Nando Vieira"]
+  s.required_ruby_version = ">= 2.7"
   s.email       = ["fnando.vieira@gmail.com"]
   s.homepage    = "http://rubygems.org/gems/test_notifier"
-  s.summary     = "Display system notifications (dbus, growl and snarl) after running tests."
-  s.description = <<-DESC
-Display system notifications (dbus, growl and snarl) after
-running tests. It works on Mac OS X, Linux and Windows. Powerful when used
-with Autotest ZenTest gem and alike for Rails apps.
-DESC
+  s.summary     = "Display system notifications (dbus, terminal notifier, " \
+                  "snarl, and more) after running tests."
+  s.description = "Display system notifications after running tests. It " \
+                  "works on Mac OS X, Linux and Windows. Powerful when used " \
+                  "with autotest or guard."
 
+  s.metadata["rubygems_mfa_required"] = "true"
   s.files         = `git ls-files`.split("\n")
   s.test_files    = `git ls-files -- {test,spec,features}/*`.split("\n")
-  s.executables   = `git ls-files -- bin/*`.split("\n").map{ |f| File.basename(f) }
+  s.executables   = `git ls-files -- bin/*`.split("\n").map do |f|
+    File.basename(f)
+  end
   s.require_paths = ["lib"]
 
   s.add_dependency "notifier"
-  s.add_development_dependency "rspec", ">= 3.0.0"
   s.add_development_dependency "rake"
+  s.add_development_dependency "rspec", ">= 3.0.0"
+  s.add_development_dependency "rubocop"
+  s.add_development_dependency "rubocop-fnando"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: test_notifier
 version: !ruby/object:Gem::Version
-  version: 2.0.3
+  version: 2.1.0
 platform: ruby
 authors:
 - Nando Vieira
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2014-06-11 00:00:00.000000000 Z
+date: 2022-02-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: notifier
@@ -24,6 +24,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: rake
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
@@ -39,7 +53,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: 3.0.0
 - !ruby/object:Gem::Dependency
-  name: rake
+  name: rubocop
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -52,10 +66,22 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-description: |
-  Display system notifications (dbus, growl and snarl) after
-  running tests. It works on Mac OS X, Linux and Windows. Powerful when used
-  with Autotest ZenTest gem and alike for Rails apps.
+- !ruby/object:Gem::Dependency
+  name: rubocop-fnando
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+description: Display system notifications after running tests. It works on Mac OS
+  X, Linux and Windows. Powerful when used with autotest or guard.
 email:
 - fnando.vieira@gmail.com
 executables: []
@@ -64,9 +90,9 @@ extra_rdoc_files: []
 files:
 - ".gitignore"
 - ".rspec"
+- ".rubocop.yml"
 - Gemfile
-- Gemfile.lock
-- README.rdoc
+- README.md
 - Rakefile
 - lib/test_notifier.rb
 - lib/test_notifier/runner.rb
@@ -92,8 +118,9 @@ files:
 - test_notifier.gemspec
 homepage: http://rubygems.org/gems/test_notifier
 licenses: []
-metadata: {}
-post_install_message: 
+metadata:
+  rubygems_mfa_required: 'true'
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -101,18 +128,18 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: '2.7'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.2.2
-signing_key: 
+rubygems_version: 3.3.7
+signing_key:
 specification_version: 4
-summary: Display system notifications (dbus, growl and snarl) after running tests.
+summary: Display system notifications (dbus, terminal notifier, snarl, and more) after
+  running tests.
 test_files:
 - spec/spec_helper.rb
 - spec/stats/minitest_spec.rb
@@ -120,4 +147,3 @@ test_files:
 - spec/stats/rspec_spec.rb
 - spec/stats/test_unit_spec.rb
 - spec/test_notifier_spec.rb
-has_rdoc: 

data/Gemfile.lock

@@ -1,32 +0,0 @@
-PATH
-  remote: .
-  specs:
-    test_notifier (2.0.3)
-      notifier
-
-GEM
-  remote: http://rubygems.org/
-  specs:
-    diff-lcs (1.2.5)
-    notifier (0.5.0)
-    rake (10.3.2)
-    rspec (3.0.0)
-      rspec-core (~> 3.0.0)
-      rspec-expectations (~> 3.0.0)
-      rspec-mocks (~> 3.0.0)
-    rspec-core (3.0.0)
-      rspec-support (~> 3.0.0)
-    rspec-expectations (3.0.0)
-      diff-lcs (>= 1.2.0, < 2.0)
-      rspec-support (~> 3.0.0)
-    rspec-mocks (3.0.1)
-      rspec-support (~> 3.0.0)
-    rspec-support (3.0.0)
-
-PLATFORMS
-  ruby
-
-DEPENDENCIES
-  rake
-  rspec (>= 3.0.0)
-  test_notifier!

data/README.rdoc

@@ -1,74 +0,0 @@
-= Test Notifier
-
-Inspired by http://railstips.org/2007/7/23/autotest-growl-pass-fail-notifications
-
-After using Growl notification, I decided to write my own plugin because I have
-to work on Ubuntu and Mac OS X and I missed the notification on my Linux box.
-This plugin works with Linux, Mac OS X and Windows. All you need to do is
-install the specific notification library for your OS.
-
-Instead of displaying lots of notifications for each failure, I prefer to be
-notified about the whole test result (you'll have to check your log
-file anyway in order to clean up the failures/errors).
-
-== Installation
-
-  gem install test_notifier
-
-Check http://github.com/fnando/notifier to see how you can configure a notifier for your OS.
-
-== Usage
-
-If you're using Test::Unit you should add <tt>require "test_notifier/runner/test_unit"</tt> to your <tt>test_helper.rb</tt> file.
-
-If you're using RSpec you should add <tt>require "test_notifier/runner/spec"</tt> to your <tt>spec_helper.rb</tt> file. If you're using RSpec 2-3, you need to add <tt>require "test_notifier/runner/rspec"</tt> instead.
-
-If you're using Autotest you should add <tt>require "test_notifier/runner/autotest"</tt> to
-the file <tt>~/.autotest</tt>
-
-If you're using MiniTest you should add <tt>require "test_notifier/runner/minitest"</tt> to your <tt>test_helper.rb</tt> file.
-
-You can define your notifier.
-
-  TestNotifier.default_notifier = :growl
-
-The available notifiers are <tt>:growl</tt>, <tt>:kdialog</tt>, <tt>:knotify</tt>, <tt>:notify_send</tt>, <tt>:osd_cat</tt>, and <tt>:snarl</tt>.
-
-If you'd like to make Test Notifier optional for your project:
-
-  TestNotifier.silence_no_notifier_warning = true
-
-== Maintainer
-
-* Nando Vieira - http://simplesideias.com.br
-
-== Collaborators
-
-* Szymon (jeznet) Jeż - http://github.com/jeznet
-* Steve Halasz - http://github.com/woodchuck
-* Khaja Minhajuddin - http://minhajuddin.com/
-* Steve Sloan - http://github.com/CodeMonkeySteve
-* Jordan Byron - http://jordanbyron.com
-
-== License
-
-(The MIT License)
-
-Permission is hereby granted, free of charge, to any person obtaining
-a copy of this software and associated documentation files (the
-'Software'), to deal in the Software without restriction, including
-without limitation the rights to use, copy, modify, merge, publish,
-distribute, sublicense, and/or sell copies of the Software, and to
-permit persons to whom the Software is furnished to do so, subject to
-the following conditions:
-
-The above copyright notice and this permission notice shall be
-included in all copies or substantial portions of the Software.
-
-THE SOFTWARE IS PROVIDED 'AS IS', WITHOUT WARRANTY OF ANY KIND,
-EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
-MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.
-IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY
-CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT,
-TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE
-SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.