slack-notifier 2.2.1 → 2.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: f5599f6fb6901d7b9c7009d6343288916829b97a
-  data.tar.gz: 60d21036bd104a7bda04b8dbb9da0b4a0747a8b7
+SHA256:
+  metadata.gz: 2b0deceb91f70d0812f01b82db70eacb8dbf01d4d1985ca02375ac13b36d829e
+  data.tar.gz: 5ec7d0eb36e91bbee6f510411646b4c57935aa661706b791e1dc0941fa66dbd9
 SHA512:
-  metadata.gz: b6199f52c950a2470e651937f2f91ceb67abf95d0987b3c7cf441f9fe3e2e8b405a14a3570d110f367c308bdebafe7bd4f797661c9ce6f563d1ec5d54ea077bf
-  data.tar.gz: 9b7efc69cbeaa137a68e6240f94a198e8932ec4ea7deb48b45d9b46b5091b8118a1ad511d455c4eb91de7d38471c5e54119f5122586d7fed6b9463b6b4525daa
+  metadata.gz: 0d42ebede5966444cf3101f9410f442e3fad0cdd3a9a7a25f67ff8c068c474bbd49f08d120070cca8652ffd351d68e99f90f648090b97469e3519761b167f513
+  data.tar.gz: 950d6e0ebc557de14b56b8ea5982e6057eb54530ec7b5b406f3c28ca22c3aaf26744c2ce6d69654b39bc75c8525d43204687b8adb97c331cf21c91f9629bd97f

data/lib/slack-notifier.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require "uri"
 require "json"
 
@@ -42,9 +43,11 @@ module Slack
       payload = config.defaults.merge(payload)
 
       params[:http_options] = payload.delete(:http_options) if payload.key?(:http_options)
-      params[:payload]      = middleware.call(payload).to_json
 
-      client.post endpoint, params
+      middleware.call(payload).map do |pld|
+        params[:payload] = pld.to_json
+        client.post endpoint, params
+      end
     end
 
     private

data/lib/slack-notifier/config.rb

@@ -1,11 +1,17 @@
 # frozen_string_literal: true
+
 module Slack
   class Notifier
     class Config
       def initialize
         @http_client = Util::HTTPClient
         @defaults    = {}
-        @middleware  = [:format_message, :format_attachments, :at]
+        @middleware  = %i[
+          format_message
+          format_attachments
+          at
+          channels
+        ]
       end
 
       def http_client client=nil

data/lib/slack-notifier/payload_middleware.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module Slack
   class Notifier
     class PayloadMiddleware
@@ -20,3 +21,4 @@ require_relative "payload_middleware/base"
 require_relative "payload_middleware/format_message"
 require_relative "payload_middleware/format_attachments"
 require_relative "payload_middleware/at"
+require_relative "payload_middleware/channels"

data/lib/slack-notifier/payload_middleware/base.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module Slack
   class Notifier
     class PayloadMiddleware

data/lib/slack-notifier/payload_middleware/channels.rb

@@ -0,0 +1,21 @@
+# frozen_string_literal: true
+
+module Slack
+  class Notifier
+    class PayloadMiddleware
+      class Channels < Base
+        middleware_name :channels
+
+        def call payload={}
+          return payload unless payload[:channel].respond_to?(:to_ary)
+
+          payload[:channel].to_ary.map do |channel|
+            pld = payload.dup
+            pld[:channel] = channel
+            pld
+          end
+        end
+      end
+    end
+  end
+end

data/lib/slack-notifier/payload_middleware/format_attachments.rb

@@ -1,22 +1,29 @@
 # frozen_string_literal: true
+
 module Slack
   class Notifier
     class PayloadMiddleware
       class FormatAttachments < Base
         middleware_name :format_attachments
 
-        options formats: [:html, :markdown]
+        options formats: %i[html markdown]
 
         def call payload={}
-          attachments = payload.fetch(:attachments, payload["attachments"])
-          wrap_array(attachments).each do |attachment|
+          payload = payload.dup
+          attachments = payload.delete(:attachments)
+          attachments ||= payload.delete("attachments")
+
+          attachments = wrap_array(attachments).map do |attachment|
             ["text", :text].each do |key|
               if attachment.key?(key)
                 attachment[key] = Util::LinkFormatter.format(attachment[key], options)
               end
             end
+
+            attachment
           end
 
+          payload[:attachments] = attachments if attachments && !attachments.empty?
           payload
         end
 

data/lib/slack-notifier/payload_middleware/format_message.rb

@@ -1,11 +1,12 @@
 # frozen_string_literal: true
+
 module Slack
   class Notifier
     class PayloadMiddleware
       class FormatMessage < Base
         middleware_name :format_message
 
-        options formats: [:html, :markdown]
+        options formats: %i[html markdown]
 
         def call payload={}
           return payload unless payload[:text]

data/lib/slack-notifier/payload_middleware/stack.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module Slack
   class Notifier
     class PayloadMiddleware
@@ -25,10 +26,24 @@ module Slack
         end
 
         def call payload={}
-          stack.inject payload do |pld, middleware|
-            middleware.call(pld)
+          result = stack.inject payload do |pld, middleware|
+            as_array(pld).flat_map do |p|
+              middleware.call(p)
+            end
           end
+
+          as_array(result)
         end
+
+        private
+
+          def as_array args
+            if args.respond_to?(:to_ary)
+              args.to_ary
+            else
+              [args]
+            end
+          end
       end
     end
   end

data/lib/slack-notifier/util/escape.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module Slack
   class Notifier
     module Util

data/lib/slack-notifier/util/http_client.rb

@@ -2,7 +2,6 @@
 
 require "net/http"
 
-
 module Slack
   class Notifier
     class APIError < StandardError; end
@@ -23,6 +22,7 @@ module Slack
           @params       = params
         end
 
+        # rubocop:disable Layout/IndentHeredoc
         def call
           http_obj.request(request_obj).tap do |response|
             unless response.is_a?(Net::HTTPSuccess)
@@ -33,6 +33,7 @@ MSG
             end
           end
         end
+        # rubocop:enable Layout/IndentHeredoc
 
         private
 

data/lib/slack-notifier/util/link_formatter.rb

@@ -1,38 +1,55 @@
 # frozen_string_literal: true
+
 module Slack
   class Notifier
     module Util
       class LinkFormatter
         # http://rubular.com/r/19cNXW5qbH
-        HTML_PATTERN = / <a (?:.*?) href=['"](.+?)['"] (?:.*?)> (.+?) <\/a> /x
+        HTML_PATTERN = %r{
+          <a
+          (?:.*?)
+          href=['"](.+?)['"]
+          (?:.*?)>
+          (.+?)
+          </a>
+        }x
+
+        # the path portion of a url can contain these characters
+        VALID_PATH_CHARS = '\w\-\.\~\/\?\#\='
 
-        # http://rubular.com/r/guJbTK6x1f
-        MARKDOWN_PATTERN = /\[ ([^\[\]]*?) \] \( ((https?:\/\/.*?) | (mailto:.*?)) \) /x
+        # Attempt at only matching pairs of parens per
+        # the markdown spec http://spec.commonmark.org/0.27/#links
+        #
+        # http://rubular.com/r/y107aevxqT
+        MARKDOWN_PATTERN = %r{
+            \[ ([^\[\]]*?) \]
+            \( ((https?://.*?) | (mailto:.*?)) \)
+            (?! [#{VALID_PATH_CHARS}]* \) )
+        }x
 
         class << self
           def format string, opts={}
-            LinkFormatter.new(string, opts).formatted
+            LinkFormatter.new(string, **opts).formatted
           end
         end
 
         attr_reader :formats
 
-        def initialize string, formats: [:html, :markdown]
+        def initialize string, formats: %i[html markdown]
           @formats = formats
           @orig    = string.respond_to?(:scrub) ? string.scrub : string
         end
 
-        # rubocop:disable Style/GuardClause
+        # rubocop:disable Lint/RescueWithoutErrorClass
         def formatted
+          return @orig unless @orig.respond_to?(:gsub)
+
           sub_markdown_links(sub_html_links(@orig))
         rescue => e
-          if RUBY_VERSION < "2.1" && e.message.include?("invalid byte sequence")
-            raise e, "#{e.message}. Consider including the 'string-scrub' gem to strip invalid characters"
-          else
-            raise e
-          end
+          raise e unless RUBY_VERSION < "2.1" && e.message.include?("invalid byte sequence")
+          raise e, "#{e.message}. Consider including the 'string-scrub' gem to strip invalid characters"
         end
-        # rubocop:enable Style/GuardClause
+        # rubocop:enable Lint/RescueWithoutErrorClass
 
         private
 

data/lib/slack-notifier/version.rb

@@ -1,6 +1,7 @@
 # frozen_string_literal: true
+
 module Slack
   class Notifier
-    VERSION = "2.2.1".freeze # rubocop:disable Style/RedundantFreeze
+    VERSION = "2.4.0".freeze # rubocop:disable Style/RedundantFreeze
   end
 end

data/spec/end_to_end_spec.rb

@@ -1,5 +1,6 @@
 # frozen_string_literal: true
 # encoding: utf-8
+
 require "spec_helper"
 
 RSpec.describe Slack::Notifier do
@@ -31,6 +32,9 @@ RSpec.describe Slack::Notifier do
     { text: "hello", channel: "hodor" } =>
     { payload: { text: "hello", channel: "hodor" } },
 
+    { text: nil, attachments: [{ text: "attachment message" }] } =>
+    { payload: { text: nil, attachments: [{ text: "attachment message" }] } },
+
     { text: "the message", channel: "foo", attachments: [{ color: "#000",
                                                            text: "attachment message",
                                                            fallback: "fallback message" }] } =>
@@ -50,9 +54,16 @@ RSpec.describe Slack::Notifier do
     { attachments: { color: "#000",
                      text: "attachment message [hodor](http://winterfell.com)",
                      fallback: "fallback message" } } =>
-    { payload: { attachments: { color: "#000",
-                                text: "attachment message <http://winterfell.com|hodor>",
-                                fallback: "fallback message" } } },
+    { payload: { attachments: [{ color: "#000",
+                                 text: "attachment message <http://winterfell.com|hodor>",
+                                 fallback: "fallback message" }] } },
+
+    { attachments: { color: "#000",
+                     text: nil,
+                     fallback: "fallback message" } } =>
+    { payload: { attachments: [{ color: "#000",
+                                 text: nil,
+                                 fallback: "fallback message" }] } },
 
     { text: "hello", http_options: { timeout: 5 } } =>
     { http_options: { timeout: 5 }, payload: { text: "hello" } }

data/spec/integration/ping_integration_test.rb

@@ -1,5 +1,6 @@
 # frozen_string_literal: true
 # encoding: utf-8
+
 require_relative "../../lib/slack-notifier"
 
 ruby = if defined?(JRUBY_VERSION)
@@ -10,5 +11,6 @@ end
 puts "testing with #{ruby}"
 
 notifier = Slack::Notifier.new ENV["SLACK_WEBHOOK_URL"], username: "notifier"
+notifier.ping "hello", channel: ["#general", "#random"]
 notifier.ping "hello/こんにちは from notifier test script on #{ruby}\225"
 notifier.ping attachments: [{ color: "#1BF5AF", fallback: "fallback", text: "attachment" }]

data/spec/lib/slack-notifier/config_spec.rb

@@ -46,17 +46,17 @@ RSpec.describe Slack::Notifier::Config do
     it "is [:format_message, :format_attachments, :at] if not set" do
       subject = described_class.new
 
-      expect(subject.middleware).to eq [:format_message, :format_attachments, :at]
+      expect(subject.middleware).to eq %i[format_message format_attachments at channels]
     end
 
     it "takes an array or a splat of args" do
       subject = described_class.new
 
       subject.middleware :layer, :two
-      expect(subject.middleware).to eq [:layer, :two]
+      expect(subject.middleware).to eq %i[layer two]
 
-      subject.middleware [:one, :layer]
-      expect(subject.middleware).to eq [:one, :layer]
+      subject.middleware %i[one layer]
+      expect(subject.middleware).to eq %i[one layer]
     end
 
     it "allows passing options to middleware stack" do

data/spec/lib/slack-notifier/payload_middleware/at_spec.rb

@@ -3,7 +3,7 @@
 RSpec.describe Slack::Notifier::PayloadMiddleware::At do
   it "can handle array at" do
     subject = described_class.new(:notifier)
-    payload = { text: "hello", at: [:john, :ken, :here] }
+    payload = { text: "hello", at: %i[john ken here] }
 
     expect(subject.call(payload)).to eq text: "<@john> <@ken> <!here> hello"
   end

data/spec/lib/slack-notifier/payload_middleware/base_spec.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 RSpec.describe Slack::Notifier::PayloadMiddleware::Base do
   before(:each) do
     @registry_backup = Slack::Notifier::PayloadMiddleware.registry.dup

data/spec/lib/slack-notifier/payload_middleware/channels_spec.rb

@@ -0,0 +1,20 @@
+# frozen_string_literal: true
+
+RSpec.describe Slack::Notifier::PayloadMiddleware::Channels do
+  it "leaves string channels alone" do
+    subject = described_class.new(:notifier)
+    payload = { text: "hello", channel: "hodor" }
+
+    expect(subject.call(payload)).to eq text: "hello", channel: "hodor"
+  end
+
+  it "splits payload into multiple if given an array of channels" do
+    subject = described_class.new(:notifier)
+    payload = { text: "hello", channel: %w[foo hodor] }
+
+    expect(subject.call(payload)).to eq [
+      { text: "hello", channel: "foo" },
+      { text: "hello", channel: "hodor" }
+    ]
+  end
+end

data/spec/lib/slack-notifier/payload_middleware/format_attachments_spec.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 RSpec.describe Slack::Notifier::PayloadMiddleware::FormatAttachments do
   it "passes the text of attachments through linkformatter with options[:formats]" do
     subject = described_class.new(:notifier, formats: [:html])
@@ -10,22 +11,34 @@ RSpec.describe Slack::Notifier::PayloadMiddleware::FormatAttachments do
   it "searches through string or symbol keys" do
     subject = described_class.new(:notifier)
     expect(Slack::Notifier::Util::LinkFormatter).to receive(:format)
-      .with("hello", formats: [:html, :markdown])
+      .with("hello", formats: %i[html markdown])
     subject.call("attachments" => [{ "text" => "hello" }])
 
     subject = described_class.new(:notifier)
     expect(Slack::Notifier::Util::LinkFormatter).to receive(:format)
-      .with("hello", formats: [:html, :markdown])
+      .with("hello", formats: %i[html markdown])
     subject.call(attachments: [{ text: "hello" }])
   end
 
   it "can handle a single attachment" do
     subject = described_class.new(:notifier)
     expect(Slack::Notifier::Util::LinkFormatter).to receive(:format)
-      .with("hello", formats: [:html, :markdown])
+      .with("hello", formats: %i[html markdown])
     subject.call(attachments: { text: "hello" })
   end
 
+  it "wraps attachment into array if given as a single hash" do
+    params = {
+      attachments: { text: "hello" }
+    }
+    payload = {
+      attachments: [{ text: "hello" }]
+    }
+    subject = described_class.new(:notifier)
+
+    expect(subject.call(params)).to eq payload
+  end
+
   it "returns the payload unmodified if not :attachments key" do
     payload = { foo: :bar }
     subject = described_class.new(:notifier)

data/spec/lib/slack-notifier/payload_middleware/format_message_spec.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 RSpec.describe Slack::Notifier::PayloadMiddleware::FormatMessage do
   it "passes the text through linkformatter with options[:formats]" do
     subject = described_class.new(:notifier, formats: [:html])
@@ -8,7 +9,7 @@ RSpec.describe Slack::Notifier::PayloadMiddleware::FormatMessage do
 
     subject = described_class.new(:notifier)
     expect(Slack::Notifier::Util::LinkFormatter).to receive(:format)
-      .with("hello", formats: [:html, :markdown])
+      .with("hello", formats: %i[html markdown])
     subject.call(text: "hello")
 
     subject = described_class.new(:notifier, formats: [:markdown])

data/spec/lib/slack-notifier/payload_middleware/stack_spec.rb

@@ -1,9 +1,14 @@
 # frozen_string_literal: true
+
 RSpec.describe Slack::Notifier::PayloadMiddleware::Stack do
   let(:return_one) do
     double(call: 1)
   end
 
+  let(:return_one_twice) do
+    double(call: [1, 1])
+  end
+
   let(:return_two) do
     double(call: 2)
   end
@@ -18,6 +23,7 @@ RSpec.describe Slack::Notifier::PayloadMiddleware::Stack do
     Slack::Notifier::PayloadMiddleware.send(:remove_instance_variable, :@registry)
 
     Slack::Notifier::PayloadMiddleware.register return_one, :return_one
+    Slack::Notifier::PayloadMiddleware.register return_one_twice, :return_one_twice
     Slack::Notifier::PayloadMiddleware.register return_two, :return_two
     Slack::Notifier::PayloadMiddleware.register return_three, :return_three
   end
@@ -87,7 +93,27 @@ RSpec.describe Slack::Notifier::PayloadMiddleware::Stack do
       expect(return_three).to receive(:call).with(1)
       expect(return_two).to receive(:call).with(3)
 
-      expect(subject.call(5)).to eq 2
+      expect(subject.call(5)).to eq [2]
+    end
+
+    it "allows any middleware to return an array but other's don't need special behavior" do
+      allow(return_one_twice).to receive(:new).and_return(return_one_twice)
+      allow(return_two).to receive(:new).and_return(return_two)
+
+      subject = described_class.new(:notifier)
+      subject.set(:return_one_twice, :return_two)
+
+      expect(subject.call(5)).to eq [2, 2]
+    end
+
+    it "handles multiple middleware splitting payload" do
+      allow(return_one_twice).to receive(:new).and_return(return_one_twice)
+      allow(return_two).to receive(:new).and_return(return_two)
+
+      subject = described_class.new(:notifier)
+      subject.set(:return_one_twice, :return_one_twice, :return_two)
+
+      expect(subject.call(5)).to eq [2, 2, 2, 2]
     end
   end
 end

data/spec/lib/slack-notifier/payload_middleware_spec.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 RSpec.describe Slack::Notifier::PayloadMiddleware do
   before(:each) do
     @registry_backup = described_class.registry.dup

data/spec/lib/slack-notifier/util/http_client_spec.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 RSpec.describe Slack::Notifier::Util::HTTPClient do
   describe "::post" do
     it "initializes Util::HTTPClient with the given uri and params then calls" do
@@ -6,7 +7,7 @@ RSpec.describe Slack::Notifier::Util::HTTPClient do
 
       expect(described_class)
         .to receive(:new).with("uri", "params")
-        .and_return(http_post_double)
+                         .and_return(http_post_double)
       expect(http_post_double).to receive(:call)
 
       described_class.post "uri", "params"

data/spec/lib/slack-notifier/util/link_formatter_spec.rb

@@ -1,7 +1,19 @@
 # frozen_string_literal: true
 # encoding: utf-8
+
 # rubocop:disable Metrics/LineLength
 RSpec.describe Slack::Notifier::Util::LinkFormatter do
+  describe "initialize & formatted" do
+    it "can be initialized without format args" do
+      subject = described_class.new("Hello World")
+      expect(subject.formatted()).to eq("Hello World")
+    end
+
+    it "can be initialized with format args" do
+      subject = described_class.new("Hello World", formats: [:html])
+      expect(subject.formatted()).to eq("Hello World")
+    end
+  end
   describe "::format" do
     it "formats html links" do
       formatted = described_class.format("Hello World, enjoy <a href='http://example.com'>this</a>.")
@@ -56,7 +68,7 @@ RSpec.describe Slack::Notifier::Util::LinkFormatter do
       end
     end
 
-    it 'doesn\'t replace valid Japanese' do
+    it "doesn't replace valid Japanese" do
       formatted = described_class.format("こんにちは")
       expect(formatted).to eq "こんにちは"
     end
@@ -71,6 +83,68 @@ RSpec.describe Slack::Notifier::Util::LinkFormatter do
       expect(formatted).to eq "<mailto:john@example.com|John>"
     end
 
+    it "handles links with trailing parentheses" do
+      formatted = described_class.format("Hello World, enjoy [foo(bar)](http://example.com/foo(bar))<a href='http://example.com/bar(foo)'>bar(foo)</a>")
+      expect(formatted).to include("http://example.com/foo(bar)|foo(bar)")
+      expect(formatted).to include("http://example.com/bar(foo)|bar(foo)")
+    end
+
+    it "formats a number of differently formatted links" do
+      input_output = {
+        "Hello World, enjoy [this](http://example.com)." =>
+          "Hello World, enjoy <http://example.com|this>.",
+
+        "Hello World, enjoy [[this](http://example.com) in brackets]." =>
+          "Hello World, enjoy [<http://example.com|this> in brackets].",
+
+        "Hello World, enjoy ([this](http://example.com) in parens)." =>
+          "Hello World, enjoy (<http://example.com|this> in parens).",
+
+        "Hello World, enjoy [](http://example.com)." =>
+          "Hello World, enjoy <http://example.com>.",
+
+        "Hello World, enjoy [link with query](http://example.com?foo=bar)." =>
+          "Hello World, enjoy <http://example.com?foo=bar|link with query>.",
+
+        "Hello World, enjoy [link with fragment](http://example.com/#foo-bar)." =>
+          "Hello World, enjoy <http://example.com/#foo-bar|link with fragment>.",
+
+        "Hello World, enjoy [link with parens](http://example.com/foo(bar)/baz)." =>
+          "Hello World, enjoy <http://example.com/foo(bar)/baz|link with parens>.",
+
+        "Hello World, enjoy [link with query](http://example.com/(parens)?foo=bar)." =>
+          "Hello World, enjoy <http://example.com/(parens)?foo=bar|link with query>.",
+
+        "Hello World, enjoy [link with parens](http://example.com/baz?bang=foo(bar))." =>
+          "Hello World, enjoy <http://example.com/baz?bang=foo(bar)|link with parens>.",
+
+        "Hello World, enjoy [link with fragment](http://example.com/(parens)#foo-bar)." =>
+          "Hello World, enjoy <http://example.com/(parens)#foo-bar|link with fragment>.",
+
+        "Hello World, enjoy [link with fragment](http://example.com/#foo-bar=(baz))." =>
+          "Hello World, enjoy <http://example.com/#foo-bar=(baz)|link with fragment>.",
+
+        "Hello World, enjoy [this](http://example.com?foo=bar)[this2](http://example2.com)." =>
+          "Hello World, enjoy <http://example.com?foo=bar|this><http://example2.com|this2>.",
+
+        "Hello World, enjoy [this](http://example.com?foo=bar) [this2](http://example2.com/#fragment)." =>
+          "Hello World, enjoy <http://example.com?foo=bar|this> <http://example2.com/#fragment|this2>.",
+
+        "Hello World, enjoy [this](http://example.com)<a href='http://example2.com'>this2</a>." =>
+          "Hello World, enjoy <http://example.com|this><http://example2.com|this2>.",
+
+        "Hello world, [John](mailto:john@example.com)." =>
+          "Hello world, <mailto:john@example.com|John>.",
+
+        "Hello World, enjoy [foo(bar)](http://example.com/foo(bar))<a href='http://example.com/bar(foo)'>bar(foo)</a>" =>
+          "Hello World, enjoy <http://example.com/foo(bar)|foo(bar)><http://example.com/bar(foo)|bar(foo)>"
+      }
+
+      input_output.each do |input, output|
+        expect(described_class.format(input)).to eq output
+      end
+    end
+
     context "with a configured stack" do
       it "only formats html if html is the only item in formats" do
         formatted = described_class.format("Hello World, enjoy [this](http://example.com)<a href='http://example2.com'>this2</a>.", formats: [:html])

data/spec/lib/slack-notifier_spec.rb

@@ -84,14 +84,15 @@ RSpec.describe Slack::Notifier do
 
       expect(stack).to receive(:call)
         .with(channel: "default", user: "rocket")
-        .and_return(test: "stack")
+        .and_return([test: "stack"])
 
       expect(mock_http).to receive(:post).with(
         URI.parse("http://example.com"),
         payload: '{"test":"stack"}'
       )
 
-      subject.post
+      responses = subject.post
+      expect(responses).to eq([:posted])
     end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: slack-notifier
 version: !ruby/object:Gem::Version
-  version: 2.2.1
+  version: 2.4.0
 platform: ruby
 authors:
 - Steven Sloan
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2017-05-29 00:00:00.000000000 Z
+date: 2021-05-07 00:00:00.000000000 Z
 dependencies: []
 description: " A slim ruby wrapper for posting to slack webhooks "
 email:
@@ -22,6 +22,7 @@ files:
 - lib/slack-notifier/payload_middleware.rb
 - lib/slack-notifier/payload_middleware/at.rb
 - lib/slack-notifier/payload_middleware/base.rb
+- lib/slack-notifier/payload_middleware/channels.rb
 - lib/slack-notifier/payload_middleware/format_attachments.rb
 - lib/slack-notifier/payload_middleware/format_message.rb
 - lib/slack-notifier/payload_middleware/stack.rb
@@ -34,6 +35,7 @@ files:
 - spec/lib/slack-notifier/config_spec.rb
 - spec/lib/slack-notifier/payload_middleware/at_spec.rb
 - spec/lib/slack-notifier/payload_middleware/base_spec.rb
+- spec/lib/slack-notifier/payload_middleware/channels_spec.rb
 - spec/lib/slack-notifier/payload_middleware/format_attachments_spec.rb
 - spec/lib/slack-notifier/payload_middleware/format_message_spec.rb
 - spec/lib/slack-notifier/payload_middleware/stack_spec.rb
@@ -46,7 +48,7 @@ homepage: http://github.com/stevenosloan/slack-notifier
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -61,9 +63,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.5.2
-signing_key: 
+rubygems_version: 3.2.15
+signing_key:
 specification_version: 4
 summary: A slim ruby wrapper for posting to slack webhooks
 test_files:
@@ -72,6 +73,7 @@ test_files:
 - spec/lib/slack-notifier/config_spec.rb
 - spec/lib/slack-notifier/payload_middleware/at_spec.rb
 - spec/lib/slack-notifier/payload_middleware/base_spec.rb
+- spec/lib/slack-notifier/payload_middleware/channels_spec.rb
 - spec/lib/slack-notifier/payload_middleware/format_attachments_spec.rb
 - spec/lib/slack-notifier/payload_middleware/format_message_spec.rb
 - spec/lib/slack-notifier/payload_middleware/stack_spec.rb