slack-notifier 2.2.2 → 2.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 8d741922d25b1c2a19c397045f22e37565f6f665
-  data.tar.gz: 25b452bb8bb9ffee8ed85c60a5c9e9dd522470a8
+  metadata.gz: 68824ffcac979f733a61c7b5a77d0d21e8ed1dfe
+  data.tar.gz: '08d9cee6b3a56238304f54756640794f443c78ec'
 SHA512:
-  metadata.gz: 2d54f11cc362ca57bd76792dc38d2e7755cc0f483144cce7721da028825a53227b4ecf703eac2fa910225a8a2ac02748e982b7fd3d372250cf9fe4bc1ce57519
-  data.tar.gz: c9eee1509416474730ff6a7f4aa63819c9f3297ce3720f28505489359cb12f7426e004b76ce3fc04b9047166eab36a2beed4c45e186ed5389211b4b5e806388e
+  metadata.gz: 65efd0c9618f26051b0f562f3ee46b5e6c697c479a6fe50d2014f77da3fdfb13c1ad6009f255cf6481cb4922d7bcc33b49ce8d4a65fb3372734f422f87189e7d
+  data.tar.gz: 9b636e5fcd4d0bc0c815fb6d3c0625a9f1fd62a366bd24e4f0cd3fbe368eadc0405410ce0786a87b12ccd5fbd4f9968b1d89239cd955e9ed62f6407982a76276

data/lib/slack-notifier.rb

@@ -42,9 +42,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).each do |pld|
+        params[:payload] = pld.to_json
+        client.post endpoint, params
+      end
     end
 
     private

data/lib/slack-notifier/config.rb

@@ -5,7 +5,12 @@ module Slack
       def initialize
         @http_client = Util::HTTPClient
         @defaults    = {}
-        @middleware  = [:format_message, :format_attachments, :at]
+        @middleware  = [
+          :format_message,
+          :format_attachments,
+          :at,
+          :channels,
+        ]
       end
 
       def http_client client=nil

data/lib/slack-notifier/payload_middleware.rb

@@ -20,3 +20,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/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/stack.rb

@@ -25,10 +25,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/version.rb

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

data/spec/integration/ping_integration_test.rb

@@ -10,5 +10,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,7 +46,7 @@ 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 [:format_message, :format_attachments, :at, :channels]
     end
 
     it "takes an array or a splat of args" do

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/stack_spec.rb

@@ -4,6 +4,10 @@ RSpec.describe Slack::Notifier::PayloadMiddleware::Stack 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 +22,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 +92,28 @@ 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_spec.rb

@@ -84,7 +84,7 @@ 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"),

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: slack-notifier
 version: !ruby/object:Gem::Version
-  version: 2.2.2
+  version: 2.3.0
 platform: ruby
 authors:
 - Steven Sloan
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-06-28 00:00:00.000000000 Z
+date: 2017-08-02 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
@@ -72,6 +74,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