uniform_notifier 1.11.0 → 1.13.2

data/spec/uniform_notifier/rails_logger_spec.rb

@@ -1,20 +1,22 @@
-require "spec_helper"
+# frozen_string_literal: true
+
+require 'spec_helper'
 
 class Rails
   # mock Rails
 end
 
 RSpec.describe UniformNotifier::RailsLogger do
-  it "should not notify rails logger" do
-    expect(UniformNotifier::RailsLogger.out_of_channel_notify(:title => "notify rails logger")).to be_nil
+  it 'should not notify rails logger' do
+    expect(UniformNotifier::RailsLogger.out_of_channel_notify(title: 'notify rails logger')).to be_nil
   end
 
-  it "should notify rails logger" do
-    logger = double("logger")
+  it 'should notify rails logger' do
+    logger = double('logger')
     expect(Rails).to receive(:logger).and_return(logger)
-    expect(logger).to receive(:warn).with("notify rails logger")
+    expect(logger).to receive(:warn).with('notify rails logger')
 
     UniformNotifier.rails_logger = true
-    UniformNotifier::RailsLogger.out_of_channel_notify(:title => "notify rails logger")
+    UniformNotifier::RailsLogger.out_of_channel_notify(title: 'notify rails logger')
   end
 end

data/spec/uniform_notifier/raise_spec.rb

@@ -1,31 +1,30 @@
-require "spec_helper"
+# frozen_string_literal: true
+
+require 'spec_helper'
 
 RSpec.describe UniformNotifier::Raise do
-  it "should not notify message" do
-    expect(UniformNotifier::Raise.out_of_channel_notify(:title => "notification")).to be_nil
+  it 'should not notify message' do
+    expect(UniformNotifier::Raise.out_of_channel_notify(title: 'notification')).to be_nil
   end
 
-  it "should raise error of the default class" do
+  it 'should raise error of the default class' do
     UniformNotifier.raise = true
-    expect {
-      UniformNotifier::Raise.out_of_channel_notify(:title => "notification")
-    }.to raise_error(UniformNotifier::Exception, "notification")
+    expect { UniformNotifier::Raise.out_of_channel_notify(title: 'notification') }.to raise_error(
+      UniformNotifier::Exception,
+      'notification'
+    )
   end
 
-  it "allows the user to override the default exception class" do
-    klass = Class.new(Exception)
+  it 'allows the user to override the default exception class' do
+    klass = Class.new(RuntimeError)
     UniformNotifier.raise = klass
-    expect {
-      UniformNotifier::Raise.out_of_channel_notify(:title => "notification")
-    }.to raise_error(klass, "notification")
+    expect { UniformNotifier::Raise.out_of_channel_notify(title: 'notification') }.to raise_error(klass, 'notification')
   end
 
-  it "can be turned from on to off again" do
+  it 'can be turned from on to off again' do
     UniformNotifier.raise = true
     UniformNotifier.raise = false
 
-    expect {
-      UniformNotifier::Raise.out_of_channel_notify(:title => "notification")
-    }.not_to raise_error
+    expect { UniformNotifier::Raise.out_of_channel_notify(title: 'notification') }.not_to raise_error
   end
 end

data/spec/uniform_notifier/rollbar_spec.rb

@@ -1,18 +1,27 @@
-require "spec_helper"
+# frozen_string_literal: true
+
+require 'spec_helper'
 
 class Rollbar
   # mock Rollbar
 end
 
 RSpec.describe UniformNotifier::RollbarNotifier do
-  it "should not notify rollbar" do
-    expect(UniformNotifier::RollbarNotifier.out_of_channel_notify(:title => "notify rollbar")).to be_nil
+  it 'should not notify rollbar' do
+    expect(UniformNotifier::RollbarNotifier.out_of_channel_notify(title: 'notify rollbar')).to be_nil
   end
 
-  it "should notify rollbar" do
-    expect(Rollbar).to receive(:info).with(UniformNotifier::Exception.new("notify rollbar"))
+  it 'should notify rollbar' do
+    expect(Rollbar).to receive(:log).with('info', UniformNotifier::Exception.new('notify rollbar'))
 
     UniformNotifier.rollbar = true
-    UniformNotifier::RollbarNotifier.out_of_channel_notify(:title => "notify rollbar")
+    UniformNotifier::RollbarNotifier.out_of_channel_notify(title: 'notify rollbar')
+  end
+
+  it 'should notify rollbar' do
+    expect(Rollbar).to receive(:log).with('warning', UniformNotifier::Exception.new('notify rollbar'))
+
+    UniformNotifier.rollbar = { level: 'warning' }
+    UniformNotifier::RollbarNotifier.out_of_channel_notify(title: 'notify rollbar')
   end
 end

data/spec/uniform_notifier/sentry_spec.rb

@@ -1,25 +1,27 @@
-require "spec_helper"
+# frozen_string_literal: true
+
+require 'spec_helper'
 
 class Raven
   # mock Sentry
 end
 
 RSpec.describe UniformNotifier::SentryNotifier do
-  it "should not notify sentry" do
-    expect(UniformNotifier::SentryNotifier.out_of_channel_notify(:title => "notify sentry")).to be_nil
+  it 'should not notify sentry' do
+    expect(UniformNotifier::SentryNotifier.out_of_channel_notify(title: 'notify sentry')).to be_nil
   end
 
-  it "should notify sentry" do
-    expect(Raven).to receive(:capture_exception).with(UniformNotifier::Exception.new("notify sentry"), {})
+  it 'should notify sentry' do
+    expect(Raven).to receive(:capture_exception).with(UniformNotifier::Exception.new('notify sentry'), {})
 
     UniformNotifier.sentry = true
-    UniformNotifier::SentryNotifier.out_of_channel_notify(:title => "notify sentry")
+    UniformNotifier::SentryNotifier.out_of_channel_notify(title: 'notify sentry')
   end
 
-  it "should notify sentry" do
-    expect(Raven).to receive(:capture_exception).with(UniformNotifier::Exception.new("notify sentry"), :foo => :bar)
+  it 'should notify sentry' do
+    expect(Raven).to receive(:capture_exception).with(UniformNotifier::Exception.new('notify sentry'), foo: :bar)
 
-    UniformNotifier.sentry = { :foo => :bar }
-    UniformNotifier::SentryNotifier.out_of_channel_notify("notify sentry")
+    UniformNotifier.sentry = { foo: :bar }
+    UniformNotifier::SentryNotifier.out_of_channel_notify('notify sentry')
   end
 end

data/spec/uniform_notifier/slack_spec.rb

@@ -1,39 +1,43 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 RSpec.describe UniformNotifier::Slack do
   context 'not enabled' do
     it 'should not notify slack' do
       expect_any_instance_of(Slack::Notifier).to_not receive(:ping)
-      expect(UniformNotifier::Slack.out_of_channel_notify(:title => 'notify slack')).to be_nil
+      expect(UniformNotifier::Slack.out_of_channel_notify(title: 'notify slack')).to be_nil
     end
   end
 
   context 'configuration' do
     context 'no webhook_url is given' do
       it 'should raise an error' do
-        expect{ UniformNotifier.slack = {} }.to raise_error(UniformNotifier::NotificationError)
+        expect { UniformNotifier.slack = {} }.to raise_error(UniformNotifier::NotificationError)
       end
 
       it 'should not notify slack' do
         begin
           UniformNotifier.slack = {}
         rescue UniformNotifier::NotificationError
+
         ensure
           expect_any_instance_of(Slack::Notifier).to_not receive(:ping)
-          expect(UniformNotifier::Slack.out_of_channel_notify(:title => 'notify slack')).to be_nil
+          expect(UniformNotifier::Slack.out_of_channel_notify(title: 'notify slack')).to be_nil
         end
       end
     end
 
     it 'should remove invalid options' do
       expect(Slack::Notifier).to receive(:new).with('http://some.slack.url', {}).and_return(true)
-      UniformNotifier.slack = { :webhook_url => 'http://some.slack.url', :pizza => 'pepperoni' }
+      UniformNotifier.slack = { webhook_url: 'http://some.slack.url', pizza: 'pepperoni' }
       expect(UniformNotifier::Slack.active?).to eq true
     end
 
     it 'should allow username and channel config options' do
-      expect(Slack::Notifier).to receive(:new).with('http://some.slack.url', { :username => 'The Dude', :channel => '#carpets' }).and_return(true)
-      UniformNotifier.slack = { :webhook_url => 'http://some.slack.url', :username => 'The Dude', :channel => '#carpets' }
+      expect(Slack::Notifier).to receive(:new).with('http://some.slack.url', username: 'The Dude', channel: '#carpets')
+        .and_return(true)
+      UniformNotifier.slack = { webhook_url: 'http://some.slack.url', username: 'The Dude', channel: '#carpets' }
       expect(UniformNotifier::Slack.active?).to eq true
     end
   end
@@ -45,9 +49,9 @@ RSpec.describe UniformNotifier::Slack do
     end
 
     it 'should notify slack' do
-      UniformNotifier.slack = { :webhook_url => 'http://some.slack.url' }
+      UniformNotifier.slack = { webhook_url: 'http://some.slack.url' }
       expect_any_instance_of(Slack::Notifier).to receive(:ping)
-      expect(UniformNotifier::Slack.out_of_channel_notify(:title => @message)).to eq @message
+      expect(UniformNotifier::Slack.out_of_channel_notify(title: @message)).to eq @message
     end
   end
 end

data/spec/uniform_notifier/terminal_notifier_spec.rb

@@ -0,0 +1,27 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+RSpec.describe UniformNotifier::TerminalNotifier do
+  it 'should not notify terminal-notifier when disabled' do
+    expect(UniformNotifier::TerminalNotifier.out_of_channel_notify(title: 'notify terminal')).to be_nil
+  end
+
+  it "should raise an exception when terminal-notifier gem isn't available" do
+    UniformNotifier.terminal_notifier = true
+    expect {
+      UniformNotifier::TerminalNotifier.out_of_channel_notify(body: 'body', title: 'notify terminal')
+    }.to raise_error(UniformNotifier::NotificationError, /terminal-notifier gem/)
+  end
+
+  it 'should notify terminal-notifier when enabled' do
+    module TerminalNotifier
+      # mock TerminalNotifier
+    end
+
+    expect(TerminalNotifier).to receive(:notify).with('body', title: 'notify terminal')
+
+    UniformNotifier.terminal_notifier = true
+    UniformNotifier::TerminalNotifier.out_of_channel_notify(body: 'body', title: 'notify terminal')
+  end
+end

data/spec/uniform_notifier/xmpp_spec.rb

@@ -1,50 +1,56 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 RSpec.describe UniformNotifier::Xmpp do
-  it "should not notify xmpp" do
-    expect(UniformNotifier::Xmpp.out_of_channel_notify(:title => "notify xmpp")).to be_nil
+  it 'should not notify xmpp' do
+    expect(UniformNotifier::Xmpp.out_of_channel_notify(title: 'notify xmpp')).to be_nil
   end
 
-  it "should notify xmpp without online status" do
-    jid = double("jid")
-    xmpp = double("xmpp")
+  it 'should notify xmpp without online status' do
+    jid = double('jid')
+    xmpp = double('xmpp')
     expect(Jabber::JID).to receive(:new).with('from@gmail.com').and_return(jid)
     expect(Jabber::Client).to receive(:new).with(jid).and_return(xmpp)
     expect(xmpp).to receive(:connect)
     expect(xmpp).to receive(:auth).with('123456')
 
-    message = double("message")
+    message = double('message')
     expect(Jabber::Message).to receive(:new).with('to@gmail.com', 'notify xmpp').and_return(message)
     expect(message).to receive(:set_type).with(:normal).and_return(message)
     expect(message).to receive(:set_subject).with('Uniform Notifier').and_return(message)
     expect(xmpp).to receive(:send).with(message)
 
-    UniformNotifier.xmpp = {:account => 'from@gmail.com', :password => '123456', :receiver => 'to@gmail.com', :show_online_status => false}
-    UniformNotifier::Xmpp.out_of_channel_notify(:title => 'notify xmpp')
+    UniformNotifier.xmpp = {
+      account: 'from@gmail.com', password: '123456', receiver: 'to@gmail.com', show_online_status: false
+    }
+    UniformNotifier::Xmpp.out_of_channel_notify(title: 'notify xmpp')
   end
 
-  it "should notify xmpp with online status" do
-    jid = double("jid")
-    xmpp = double("xmpp")
+  it 'should notify xmpp with online status' do
+    jid = double('jid')
+    xmpp = double('xmpp')
     expect(Jabber::JID).to receive(:new).with('from@gmail.com').and_return(jid)
     expect(Jabber::Client).to receive(:new).with(jid).and_return(xmpp)
     expect(xmpp).to receive(:connect)
     expect(xmpp).to receive(:auth).with('123456')
 
-    presence = double("presence")
+    presence = double('presence')
     now = Time.now
     allow(Time).to receive(:now).and_return(now)
     expect(Jabber::Presence).to receive(:new).and_return(presence)
     expect(presence).to receive(:set_status).with("Uniform Notifier started on #{now}").and_return(presence)
     expect(xmpp).to receive(:send).with(presence)
 
-    message = double("message")
+    message = double('message')
     expect(Jabber::Message).to receive(:new).with('to@gmail.com', 'notify xmpp').and_return(message)
     expect(message).to receive(:set_type).with(:normal).and_return(message)
     expect(message).to receive(:set_subject).with('Uniform Notifier').and_return(message)
     expect(xmpp).to receive(:send).with(message)
 
-    UniformNotifier.xmpp = {:account => 'from@gmail.com', :password => '123456', :receiver => 'to@gmail.com', :show_online_status => true}
-    UniformNotifier::Xmpp.out_of_channel_notify(:title => 'notify xmpp')
+    UniformNotifier.xmpp = {
+      account: 'from@gmail.com', password: '123456', receiver: 'to@gmail.com', show_online_status: true
+    }
+    UniformNotifier::Xmpp.out_of_channel_notify(title: 'notify xmpp')
   end
 end

data/uniform_notifier.gemspec

@@ -1,28 +1,35 @@
-# -*- encoding: utf-8 -*-
-$:.push File.expand_path("../lib", __FILE__)
-require "uniform_notifier/version"
+# frozen_string_literal: true
+
+$LOAD_PATH.push File.expand_path('lib', __dir__)
+require 'uniform_notifier/version'
 
 Gem::Specification.new do |s|
-  s.name        = "uniform_notifier"
+  s.name        = 'uniform_notifier'
   s.version     = UniformNotifier::VERSION
   s.platform    = Gem::Platform::RUBY
-  s.authors     = ["Richard Huang"]
-  s.email       = ["flyerhzm@gmail.com"]
-  s.homepage    = "http://rubygems.org/gems/uniform_notifier"
-  s.summary     = %q{uniform notifier for rails logger, customized logger, javascript alert, javascript console, growl and xmpp}
-  s.description = %q{uniform notifier for rails logger, customized logger, javascript alert, javascript console, growl and xmpp}
+  s.authors     = ['Richard Huang']
+  s.email       = ['flyerhzm@gmail.com']
+  s.homepage    = 'http://rubygems.org/gems/uniform_notifier'
+  s.summary     = 'uniform notifier for rails logger, customized logger, javascript alert, javascript console, growl and xmpp'
+  s.description = 'uniform notifier for rails logger, customized logger, javascript alert, javascript console, growl and xmpp'
   s.license     = 'MIT'
 
-  s.rubyforge_project = "uniform_notifier"
+  s.required_ruby_version = '>= 2.3'
 
-  s.add_development_dependency %q<ruby-growl>, ["= 4.0"]
-  s.add_development_dependency %q<ruby_gntp>, ["= 0.3.4"]
-  s.add_development_dependency %q<xmpp4r>, ["= 0.5"]
-  s.add_development_dependency %q<slack-notifier>, [">= 1.0"]
-  s.add_development_dependency %q<rspec>, ["> 0"]
+  s.add_development_dependency 'rspec', ['> 0']
+  s.add_development_dependency 'ruby-growl', ['= 4.0']
+  s.add_development_dependency 'ruby_gntp', ['= 0.3.4']
+  s.add_development_dependency 'slack-notifier', ['>= 1.0']
+  s.add_development_dependency 'xmpp4r', ['= 0.5']
 
   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.require_paths = ["lib"]
+  s.executables   = `git ls-files -- bin/*`.split("\n").map { |f| File.basename(f) }
+  s.require_paths = ['lib']
+
+  if s.respond_to?(:metadata)
+    s.metadata['changelog_uri'] = 'https://github.com/flyerhzm/uniform_notifier/blob/master/CHANGELOG.md'
+    s.metadata['source_code_uri'] = 'https://github.com/flyerhzm/uniform_notifier'
+    s.metadata['bug_tracker_uri'] = 'https://github.com/flyerhzm/uniform_notifier/issues'
+  end
 end

metadata

@@ -1,57 +1,57 @@
 --- !ruby/object:Gem::Specification
 name: uniform_notifier
 version: !ruby/object:Gem::Version
-  version: 1.11.0
+  version: 1.13.2
 platform: ruby
 authors:
 - Richard Huang
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2017-11-13 00:00:00.000000000 Z
+date: 2021-01-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
-  name: ruby-growl
+  name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '='
+    - - ">"
       - !ruby/object:Gem::Version
-        version: '4.0'
+        version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '='
+    - - ">"
       - !ruby/object:Gem::Version
-        version: '4.0'
+        version: '0'
 - !ruby/object:Gem::Dependency
-  name: ruby_gntp
+  name: ruby-growl
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.3.4
+        version: '4.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.3.4
+        version: '4.0'
 - !ruby/object:Gem::Dependency
-  name: xmpp4r
+  name: ruby_gntp
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: '0.5'
+        version: 0.3.4
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: '0.5'
+        version: 0.3.4
 - !ruby/object:Gem::Dependency
   name: slack-notifier
   requirement: !ruby/object:Gem::Requirement
@@ -67,19 +67,19 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '1.0'
 - !ruby/object:Gem::Dependency
-  name: rspec
+  name: xmpp4r
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">"
+    - - '='
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '0.5'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">"
+    - - '='
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '0.5'
 description: uniform notifier for rails logger, customized logger, javascript alert,
   javascript console, growl and xmpp
 email:
@@ -110,6 +110,7 @@ files:
 - lib/uniform_notifier/rollbar.rb
 - lib/uniform_notifier/sentry.rb
 - lib/uniform_notifier/slack.rb
+- lib/uniform_notifier/terminal_notifier.rb
 - lib/uniform_notifier/version.rb
 - lib/uniform_notifier/xmpp.rb
 - spec/spec_helper.rb
@@ -126,13 +127,17 @@ files:
 - spec/uniform_notifier/rollbar_spec.rb
 - spec/uniform_notifier/sentry_spec.rb
 - spec/uniform_notifier/slack_spec.rb
+- spec/uniform_notifier/terminal_notifier_spec.rb
 - spec/uniform_notifier/xmpp_spec.rb
 - uniform_notifier.gemspec
 homepage: http://rubygems.org/gems/uniform_notifier
 licenses:
 - MIT
-metadata: {}
-post_install_message: 
+metadata:
+  changelog_uri: https://github.com/flyerhzm/uniform_notifier/blob/master/CHANGELOG.md
+  source_code_uri: https://github.com/flyerhzm/uniform_notifier
+  bug_tracker_uri: https://github.com/flyerhzm/uniform_notifier/issues
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -140,16 +145,15 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: '2.3'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: uniform_notifier
-rubygems_version: 2.6.13
-signing_key: 
+rubygems_version: 3.1.4
+signing_key:
 specification_version: 4
 summary: uniform notifier for rails logger, customized logger, javascript alert, javascript
   console, growl and xmpp
@@ -168,4 +172,5 @@ test_files:
 - spec/uniform_notifier/rollbar_spec.rb
 - spec/uniform_notifier/sentry_spec.rb
 - spec/uniform_notifier/slack_spec.rb
+- spec/uniform_notifier/terminal_notifier_spec.rb
 - spec/uniform_notifier/xmpp_spec.rb