flipper-echo 0.0.1 → 0.0.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 3d0193b01d3747866567392b00529405385d4b23
-  data.tar.gz: 03d2d537df9c51aca849cc279245dd39f63835cb
+  metadata.gz: 8b2f87cd93693142ff67b6ceeed1444b6f7b012f
+  data.tar.gz: 31f38685142d8e670acaf735beaea1432810b873
 SHA512:
-  metadata.gz: 361febaf3cf8772c5eb2f723a3896068c00588e9f6e2aaf2003d3a52fb00a65356ba0dd3c94fb70b08767bfacdf3dd209e5f1e649140c95ec3b7b0b35d355982
-  data.tar.gz: 94c0a649e23ff84a9b9c41fce70c1145c3a24a9222377c073e237c4fae93ceabb7a12f1fa82cfdd4f8594d2a21c89d220e80f11256eb57c01c39b26ad25ea378
+  metadata.gz: 5d465bf8cf158efb1f6edb3c57ef6f952d9f6b2667cf39c59a1040884b7cbab83b69f2bd33758bd50187b47604b9385b635e65894bd91dcd92695e61582c59bb
+  data.tar.gz: 318e9de9d3ce75c78bd78d2802fecd91eb855f0120b55793428e7a3ee01896f3f56cbf5c004ed5855ba524969a931613f03cf1c4aec1e3faef2113167a0400ed

data/.gitignore

@@ -12,4 +12,5 @@
 *.o
 *.a
 mkmf.log
+assets
 *.gem

data/README.md

@@ -1,12 +1,15 @@
 # Flipper::Echo
 
+[![Gem Version](https://badge.fury.io/rb/flipper-echo.svg)](http://badge.fury.io/rb/flipper-echo)
+
 This gem adds a simple callback interface for
 [Flipper](https://github.com/jnunemaker/flipper) adapter events.
 
 For example, when a Flipper feature is changed, you can:
 
-* send a Slack notification
+* send a Slack notification (built-in)
 * write the change to a database or log file
+* notify a performance monitoring application
 * send a YO
 
 ## Installation
@@ -34,9 +37,9 @@ normally would (any adapter will do), e.g.:
 FLIPPER = Flipper.new(Flipper::Adapters::Memory.new)
 ```
 
-Then configure `Flipper::Echo`:
+Then configure this gem in any of the following ways:
 
-#### Option 1: handle event with a proc
+#### Handle event with a proc
 
 ```ruby
 Flipper::Echo.configure do |config|
@@ -50,12 +53,12 @@ Flipper::Echo.configure do |config|
 end
 ```
 
-#### Option 2: handle event with a notifier class
+#### Handle event with an object
 
-`Flipper::Echo` can also use any object that has a `notify` method:
+You can provide any Ruby object with a `notify` method:
 
 ```ruby
-class FlipperNotifier
+class CustomNotifier
   def notify(event)
     # Do something with the event...
   end
@@ -63,10 +66,46 @@ end
 
 Flipper::Echo.configure do |config|
   config.flipper  = FLIPPER
-  config.notifier = FlipperNotifier.new
+  config.notifier = CustomNotifier.new
+end
+```
+
+#### Configure multiple notifiers
+
+```ruby
+Flipper::Echo.configure do |config|
+  config.flipper = FLIPPER
+
+  config.notifiers << NewrelicNotifier.new
+  config.notifiers << GraphiteNotifier.new
+  config.notifiers << Flipper::Echo::Stdout::Notifier.new
+end
+```
+
+## Built-in notifiers
+
+#### Slack
+
+First [create a Slack webhook url](https://slack.com/services/new/incoming-webhook),
+then use it in your configuration:
+
+```ruby
+Flipper::Echo.configure do |config|
+  config.notifiers << Flipper::Echo::Slack::Notifier.new(
+    'https://hooks.slack.com/your/webhook...', channel: '#eng')
 end
 ```
 
+![Slack example 1](https://s3-us-west-2.amazonaws.com/mode.production/flipper-echo/prod-search-admins.png)
+
+![Slack example 2](https://s3-us-west-2.amazonaws.com/mode.production/flipper-echo/prod-risky-actors.png)
+
+![Slack example 3](https://s3-us-west-2.amazonaws.com/mode.production/flipper-echo/staging-rolled-out-removed.png)
+
+## Documentation
+
+[http://www.rubydoc.info/github/mode/flipper-echo/master](http://www.rubydoc.info/github/mode/flipper-echo/master)
+
 ## Contributing
 
 1. Fork it ( https://github.com/mode/flipper-echo/fork )
@@ -74,3 +113,7 @@ end
 3. Commit your changes (`git commit -am 'Add some feature'`)
 4. Push to the branch (`git push origin my-new-feature`)
 5. Create a new Pull Request
+
+## Actual screenshots
+
+![Newrelic example 1](https://s3-us-west-2.amazonaws.com/mode.production/flipper-echo/actual-newrelic-screenshot.png)

data/lib/flipper/echo.rb

@@ -2,6 +2,8 @@
 #
 require 'flipper/echo/configuration'
 require 'flipper/echo/event'
+require 'flipper/echo/slack'
+require 'flipper/echo/stdout'
 require 'flipper/echo/version'
 
 # Flipper namespace
@@ -32,7 +34,7 @@ module Flipper
       def enable(feature, gate, target)
         super.tap do
           Flipper::Echo::Event.new(
-            feature, :enable, gate: gate, target: target).notify
+            feature, :enabled, gate: gate, target: target).notify
         end
       end
 
@@ -41,20 +43,20 @@ module Flipper
       def disable(feature, gate, target)
         super.tap do
           Flipper::Echo::Event.new(
-            feature, :disable, gate: gate, target: target).notify
+            feature, :disabled, gate: gate, target: target).notify
         end
       end
 
       # Notify adapter remove events
       #
       def remove(feature)
-        super.tap { Flipper::Echo::Event.new(feature, :remove).notify }
+        super.tap { Flipper::Echo::Event.new(feature, :removed).notify }
       end
 
       # Notify adapter clear events
       #
       def clear(feature)
-        super.tap { Flipper::Echo::Event.new(feature, :clear).notify }
+        super.tap { Flipper::Echo::Event.new(feature, :cleared).notify }
       end
     end
   end

data/lib/flipper/echo/configuration.rb

@@ -10,7 +10,33 @@ module Flipper
     # intended behavior.
     #
     class Configuration
-      attr_accessor :notifier
+      # All configured notifiers
+      #
+      # @return [Array]
+      #
+      def notifiers
+        @notifiers ||= []
+      end
+
+      # Convenience method for assigning a single notifier
+      #
+      # @return [Array] all notifiers
+      #
+      def notifier=(notifier)
+        notifiers << notifier
+      end
+
+      # Optional environment name
+      #
+      attr_writer :environment
+
+      # Configured environment name
+      #
+      # @return [String]
+      #
+      def environment
+        @environment ||= ENV['FLIPPER_ECHO_ENVIRONMENT']
+      end
 
       # Assign Flipper instance
       #

data/lib/flipper/echo/event.rb

@@ -1,5 +1,7 @@
 # encoding: utf-8
 #
+require 'flipper'
+
 module Flipper
   module Echo
     # Encapsulates relevant information about a Flipper adapter event
@@ -11,7 +13,7 @@ module Flipper
       #
       # @param feature [Flipper::Feature] the feature that was changed
       # @param action [Symbol] the action
-      #   (`:enable`, `:disable`, `:remove` or `:clear`)
+      #   (`:enabled`, `:disabled`, `:removed` or `:cleared`)
       # @param options [optional, Hash] hash of options
       #
       # @option options [Flipper::Type] :target the event target
@@ -68,10 +70,14 @@ module Flipper
       # Passes this event to the configured notifier
       #
       def notify
-        if notifier.is_a?(Proc)
-          notifier.call(self)
-        elsif notifier.respond_to?(:notify)
-          notifier.notify(self)
+        notifiers.each do |notifier|
+          method = if notifier.is_a?(Proc)
+                     :call
+                   elsif notifier.respond_to?(:notify)
+                     :notify
+                   end
+
+          notifier.send(method, self) if method
         end
       end
 
@@ -93,8 +99,8 @@ module Flipper
 
       private
 
-      def notifier
-        Flipper::Echo.configuration.notifier
+      def notifiers
+        Flipper::Echo.configuration.notifiers
       end
     end
   end

data/lib/flipper/echo/slack.rb

@@ -0,0 +1,13 @@
+# encoding: utf-8
+#
+require 'flipper/echo/slack/message'
+require 'flipper/echo/slack/notifier'
+
+module Flipper
+  module Echo
+    # Slack notifier add-on module
+    #
+    module Slack
+    end
+  end
+end

data/lib/flipper/echo/slack/message.rb

@@ -0,0 +1,62 @@
+# encoding: utf-8
+#
+module Flipper
+  module Echo
+    module Slack
+      # Slack message formatter
+      #
+      class Message
+        attr_reader :event
+
+        # Construct a new Message instance
+        #
+        # @param event [Flipper::Echo::Event] the Flipper event
+        #
+        # @return [Flipper::Echo::Slack::Message] the instance
+        #
+        def initialize(event)
+          @event = event
+        end
+
+        # Build message body
+        #
+        # @return [String] the message body
+        #
+        def to_s
+          [base, event.action, filters].compact.join(' ')
+        end
+
+        private
+
+        # @return [String]
+        #
+        def base
+          parts = []
+          parts << "[*#{environment}*]" if environment
+          parts << "Feature `#{event.feature.name}`"
+          parts.join(' ')
+        end
+
+        # @return [String, nil]
+        #
+        def filters
+          if event.group
+            "for group `#{event.group.name}`"
+          elsif event.actor
+            "for actor `#{event.actor.flipper_id}`"
+          elsif event.percentage_of_actors
+            "for #{event.percentage_of_actors.value}% of actors"
+          elsif event.percentage_of_random
+            "for #{event.percentage_of_random.value}% of random"
+          end
+        end
+
+        # @return [String, nil]
+        #
+        def environment
+          Flipper::Echo.configuration.environment
+        end
+      end
+    end
+  end
+end

data/lib/flipper/echo/slack/notifier.rb

@@ -0,0 +1,143 @@
+# encoding: utf-8
+#
+require 'json'
+require 'net/https'
+require 'uri'
+
+module Flipper
+  module Echo
+    module Slack
+      # Customizable Slack webhook notifier
+      #
+      class Notifier
+        attr_reader :webhook_url, :options
+
+        # Construct a new Notifier instance
+        #
+        # @param webhook_url [String] the Slack webhook url
+        # @param options [optional, Hash] hash of options
+        #
+        # @option options [String] :username Slack bot username
+        # @option options [String] :channel name of Slack channel
+        # @option options [String] :icon_url the message icon url
+        #
+        # @return [Flipper::Echo::Slack::Notifier] the instance
+        #
+        def initialize(webhook_url, options = {})
+          @webhook_url = webhook_url
+          @options     = self.class.symbolize_keys(options)
+        end
+
+        # Normalized channel name
+        #
+        # @return [String, nil]
+        #
+        def channel
+          @channel ||= self.class.normalize_channel(options[:channel])
+        end
+
+        # Slack message username
+        #
+        # @return [String, nil]
+        #
+        def username
+          options.fetch(:username, DEFAULT_USERNAME)
+        end
+
+        # Slack message icon url
+        #
+        # @return [String, nil]
+        #
+        def icon_url
+          options.fetch(:icon_url, DEFAULT_ICON_URL)
+        end
+
+        # Post notification to Slack webhook url
+        #
+        # @return [HTTPResponse]
+        #
+        def notify(event)
+          message = Flipper::Echo::Slack::Message.new(event)
+
+          payload = {
+            text:     message.to_s,
+            channel:  channel,
+            username: username,
+            icon_url: icon_url
+          }
+
+          post(payload)
+        end
+
+        class << self
+          # Normalize Slack channel name for API
+          #
+          # @param name [String] the original name
+          #
+          # @return [String] the normalized name
+          #
+          def normalize_channel(name)
+            return unless name && name.size > 0
+
+            name =~ /\A[#@]/ ? name : "##{name}"
+          end
+
+          # Symbolize hash keys
+          #
+          # @param hash [Hash] the original hash
+          #
+          # @return [Hash] hash with symbolized keys
+          #
+          def symbolize_keys(hash)
+            hash.each_with_object({}) do |(key, value), result|
+              result[key.to_sym] = value
+            end
+          end
+        end
+
+        private
+
+        DEFAULT_USERNAME = 'Flipper'
+
+        DEFAULT_ICON_URL =
+          'https://s3-us-west-2.amazonaws.com/mode.production/' \
+          'flipper-echo/flipper-echo-icon-132.png'
+
+        # @return [Net::HTTP]
+        #
+        def http
+          @http ||= Net::HTTP.new(uri.host, uri.port).tap do |result|
+            result.use_ssl = ssl?
+          end
+        end
+
+        # @return [URI]
+        #
+        def uri
+          @uri ||= URI.parse(webhook_url)
+        end
+
+        # @return [true, false]
+        #
+        def ssl?
+          uri.scheme == 'https'
+        end
+
+        # @return [HTTPResponse]
+        #
+        def post(payload)
+          request = Net::HTTP::Post.new(uri.request_uri)
+          request.set_content_type('application/json')
+          request.body = payload.to_json
+
+          http.request(request).tap do |response|
+            unless response.code == '200'
+              $stderr.puts 'Unexpected Slack notifier response: ' \
+                           "#{response.code}"
+            end
+          end
+        end
+      end
+    end
+  end
+end

data/lib/flipper/echo/stdout.rb

@@ -0,0 +1,12 @@
+# encoding: utf-8
+#
+require 'flipper/echo/stdout/notifier'
+
+module Flipper
+  module Echo
+    # A stdout notifier add-on module
+    #
+    module Stdout
+    end
+  end
+end

data/lib/flipper/echo/stdout/notifier.rb

@@ -0,0 +1,45 @@
+# encoding: utf-8
+#
+module Flipper
+  module Echo
+    module Stdout
+      # Simple stdout notifier intended as a basis for more advanced custom
+      # notifiers
+      #
+      class Notifier
+        # Send notification message to stdout
+        #
+        # @return [nil]
+        #
+        def notify(event)
+          parts = []
+          parts << '[Flipper]'
+          parts << "Feature \"#{event.feature.name}\""
+          parts << event.action
+
+          target = target_message(event)
+
+          parts << "for #{target}" if target
+
+          $stdout.puts(parts.join(' '))
+        end
+
+        private
+
+        # @return [String, nil]
+        #
+        def target_message(event)
+          if event.group
+            "group #{event.group.name}"
+          elsif event.actor
+            "actor #{event.actor.flipper_id}"
+          elsif event.percentage_of_actors
+            "#{event.percentage_of_actors.value}% of actors"
+          elsif event.percentage_of_random
+            "#{event.percentage_of_random.value}% of random"
+          end
+        end
+      end
+    end
+  end
+end

data/lib/flipper/echo/version.rb

@@ -4,6 +4,6 @@ module Flipper
   module Echo
     # Current gem version
     #
-    VERSION = '0.0.1'
+    VERSION = '0.0.2'
   end
 end

data/spec/flipper/echo/configuration_spec.rb

@@ -30,4 +30,16 @@ describe Flipper::Echo::Configuration do
       configuration.flipper = flipper
     end
   end
+
+  describe '#notifier=' do
+    it 'appends notifier' do
+      notifier = double(:notifier)
+
+      expect(configuration.notifiers).to eq([])
+
+      configuration.notifier = notifier
+
+      expect(configuration.notifiers).to eq([notifier])
+    end
+  end
 end

data/spec/flipper/echo/event_spec.rb

@@ -123,7 +123,7 @@ describe Flipper::Echo::Event do
       it 'calls proc' do
         procedure = proc {}
 
-        allow(event).to receive(:notifier).and_return(procedure)
+        allow(event).to receive(:notifiers).and_return([procedure])
 
         expect(procedure).to receive(:call)
 
@@ -137,7 +137,7 @@ describe Flipper::Echo::Event do
       end
 
       it 'calls instance method if it exists' do
-        allow(event).to receive(:notifier).and_return(notifier)
+        allow(event).to receive(:notifiers).and_return([notifier])
 
         expect(notifier).to receive(:notify)
 
@@ -160,18 +160,18 @@ describe Flipper::Echo::Event do
     end
   end
 
-  describe '#notifier' do
+  describe '#notifiers' do
     let :notifier do
       double(:notifier)
     end
 
     it 'returns notifier from config' do
       allow(Flipper::Echo.configuration).to(
-        receive(:notifier).and_return(notifier))
+        receive(:notifiers).and_return([notifier]))
 
       event = Flipper::Echo::Event.new(feature, action)
 
-      expect(event.send(:notifier)).to eq(notifier)
+      expect(event.send(:notifiers)).to eq([notifier])
     end
   end
 end

data/spec/flipper/echo/slack/message_spec.rb

@@ -0,0 +1,82 @@
+# encoding: utf-8
+#
+require 'spec_helper'
+
+describe Flipper::Echo::Slack::Message do
+  let :feature do
+    double(:feature, name: :experimental)
+  end
+
+  describe '#base' do
+    let :event do
+      Flipper::Echo::Event.new(feature, :enabled)
+    end
+
+    it 'builds expected string without environment' do
+      message = Flipper::Echo::Slack::Message.new(event)
+
+      expect(message.send(:base)).to eq('Feature `experimental`')
+    end
+
+    it 'builds expected string with environment' do
+      message = Flipper::Echo::Slack::Message.new(event)
+
+      allow(message).to receive(:environment).and_return('staging')
+
+      expect(message.send(:base)).to eq('[*staging*] Feature `experimental`')
+    end
+  end
+
+  describe '#filters' do
+    it 'builds group string' do
+      event   = Flipper::Echo::Event.new(feature, :disabled)
+      message = Flipper::Echo::Slack::Message.new(event)
+
+      group = double(:group, name: 'admins')
+      allow(event).to receive(:group).and_return(group)
+
+      expect(message.send(:filters)).to eq('for group `admins`')
+    end
+
+    it 'builds actor string' do
+      event   = Flipper::Echo::Event.new(feature, :disabled)
+      message = Flipper::Echo::Slack::Message.new(event)
+
+      actor = double(:actor, flipper_id: 'Actor:1')
+      allow(event).to receive(:actor).and_return(actor)
+
+      expect(message.send(:filters)).to eq('for actor `Actor:1`')
+    end
+
+    it 'builds percentage of actors string' do
+      event   = Flipper::Echo::Event.new(feature, :disabled)
+      message = Flipper::Echo::Slack::Message.new(event)
+
+      percentage = double(:percentage, value: '10')
+      allow(event).to receive(:percentage_of_actors).and_return(percentage)
+
+      expect(message.send(:filters)).to eq('for 10% of actors')
+    end
+
+    it 'builds percentage of random string' do
+      event   = Flipper::Echo::Event.new(feature, :disabled)
+      message = Flipper::Echo::Slack::Message.new(event)
+
+      percentage = double(:percentage, value: '10')
+      allow(event).to receive(:percentage_of_random).and_return(percentage)
+
+      expect(message.send(:filters)).to eq('for 10% of random')
+    end
+  end
+
+  describe '#environment' do
+    it 'returns configured environment' do
+      allow(Flipper::Echo.configuration).to(
+        receive(:environment).and_return('staging'))
+
+      message = Flipper::Echo::Slack::Message.new(nil)
+
+      expect(message.send(:environment)).to eq('staging')
+    end
+  end
+end

data/spec/flipper/echo/slack/notifier_spec.rb

@@ -0,0 +1,126 @@
+# encoding: utf-8
+#
+require 'spec_helper'
+
+describe Flipper::Echo::Slack::Notifier do
+  describe '#channel' do
+    it 'is null if options do not include channel' do
+      notifier = Flipper::Echo::Slack::Notifier.new('https://hook')
+
+      expect(notifier.channel).to eq(nil)
+    end
+
+    it 'returns channel from options without prepending' do
+      notifier = Flipper::Echo::Slack::Notifier.new(
+        'https://hook', channel: '#custom_channel')
+
+      expect(notifier.channel).to eq('#custom_channel')
+    end
+
+    it 'returns channel from options with prepending' do
+      notifier = Flipper::Echo::Slack::Notifier.new(
+        'https://hook', channel: 'custom_channel')
+
+      expect(notifier.channel).to eq('#custom_channel')
+    end
+  end
+
+  describe '#username' do
+    it 'is default if options do not include username' do
+      notifier = Flipper::Echo::Slack::Notifier.new('https://hook')
+
+      expect(notifier.username).to eq('Flipper')
+    end
+
+    it 'returns username from options' do
+      notifier = Flipper::Echo::Slack::Notifier.new(
+        'https://hook', username: 'Custom Username')
+
+      expect(notifier.username).to eq('Custom Username')
+    end
+  end
+
+  describe '#icon_url' do
+    it 'is default if options do not include url' do
+      notifier = Flipper::Echo::Slack::Notifier.new('https://hook')
+
+      expect(notifier.icon_url).to match(/flipper\-echo\-icon/)
+    end
+
+    it 'returns username from options' do
+      notifier = Flipper::Echo::Slack::Notifier.new(
+        'https://hook', icon_url: 'custom-icon')
+
+      expect(notifier.icon_url).to eq('custom-icon')
+    end
+  end
+
+  describe '#notify' do
+    let :feature do
+      double(:feature, name: :experimental)
+    end
+
+    let :event do
+      Flipper::Echo::Event.new(feature, :action)
+    end
+
+    it 'calls notifier with expected options' do
+      notifier = Flipper::Echo::Slack::Notifier.new('https://hook')
+
+      expect(notifier).to receive(:post) do |options|
+        expect(options[:text]).to match(/experimental/)
+      end
+
+      notifier.notify(event)
+    end
+  end
+
+  describe '#post' do
+    it 'posts expected json payload' do
+      notifier = Flipper::Echo::Slack::Notifier.new('https://hook')
+
+      expect(notifier.send(:http)).to receive(:request) do |request|
+        expect(request.body).to eq('{"foo":"bar"}')
+        expect(request.content_type).to eq('application/json')
+
+        double(:response, code: '200')
+      end
+
+      notifier.send(:post, foo: 'bar')
+    end
+
+    it 'prints unexpected api response to stderr' do
+      notifier = Flipper::Echo::Slack::Notifier.new('https://hook')
+
+      expect(notifier.send(:http)).to(
+        receive(:request).and_return(double(:response, code: '404')))
+
+      expect { notifier.send(:post, foo: 'bar') }.to output.to_stderr
+    end
+  end
+
+  describe '.normalize_channel' do
+    it 'is nil for blank name' do
+      expect(Flipper::Echo::Slack::Notifier.normalize_channel(nil)).to eq(nil)
+      expect(Flipper::Echo::Slack::Notifier.normalize_channel('')).to eq(nil)
+    end
+
+    it 'preserves name including number sign' do
+      result = Flipper::Echo::Slack::Notifier.normalize_channel('#channel_name')
+
+      expect(result).to eq('#channel_name')
+    end
+
+    it 'prepends number sign if necessary' do
+      result = Flipper::Echo::Slack::Notifier.normalize_channel('channel_name')
+
+      expect(result).to eq('#channel_name')
+    end
+
+    it 'preserves at sign' do
+      result = Flipper::Echo::Slack::Notifier.normalize_channel('@channel_name')
+
+      expect(result).to eq('@channel_name')
+    end
+  end
+end

data/spec/flipper/echo/stdout/notifier_spec.rb

@@ -0,0 +1,52 @@
+# encoding: utf-8
+#
+require 'spec_helper'
+
+describe Flipper::Echo::Stdout::Notifier do
+  let :feature do
+    double(:feature, name: :experimental)
+  end
+
+  let :event do
+    Flipper::Echo::Event.new(feature, :enabled)
+  end
+
+  let :notifier do
+    Flipper::Echo::Stdout::Notifier.new
+  end
+
+  describe '#notify' do
+    it 'sends message to stdout' do
+      expect { notifier.notify(event) }.to output.to_stdout
+    end
+  end
+
+  describe '#target_message' do
+    it 'builds group message' do
+      allow(event).to receive(:group).and_return(double(:group, name: :admins))
+
+      expect(notifier.send(:target_message, event)).to eq('group admins')
+    end
+
+    it 'builds actor message' do
+      allow(event).to(
+        receive(:actor).and_return(double(:actor, flipper_id: 'id')))
+
+      expect(notifier.send(:target_message, event)).to eq('actor id')
+    end
+
+    it 'builds percentage of actors message' do
+      allow(event).to(
+        receive(:percentage_of_actors).and_return(double(:actors, value: 15)))
+
+      expect(notifier.send(:target_message, event)).to eq('15% of actors')
+    end
+
+    it 'builds percentage of random message' do
+      allow(event).to(
+        receive(:percentage_of_random).and_return(double(:random, value: 15)))
+
+      expect(notifier.send(:target_message, event)).to eq('15% of random')
+    end
+  end
+end

data/spec/flipper/echo_spec.rb

@@ -43,7 +43,7 @@ describe Flipper::Echo do
     describe '#enable' do
       it 'enables feature' do
         expect_any_instance_of(Flipper::Echo::Event).to receive(:notify) do |e|
-          expect(e.action).to eq(:enable)
+          expect(e.action).to eq(:enabled)
         end
 
         adapter.enable(feature, gate, target)
@@ -53,7 +53,7 @@ describe Flipper::Echo do
     describe '#disable' do
       it 'disables feature' do
         expect_any_instance_of(Flipper::Echo::Event).to receive(:notify) do |e|
-          expect(e.action).to eq(:disable)
+          expect(e.action).to eq(:disabled)
         end
 
         adapter.disable(feature, gate, target)
@@ -63,7 +63,7 @@ describe Flipper::Echo do
     describe '#remove' do
       it 'removes feature' do
         expect_any_instance_of(Flipper::Echo::Event).to receive(:notify) do |e|
-          expect(e.action).to eq(:remove)
+          expect(e.action).to eq(:removed)
         end
 
         adapter.remove(feature)
@@ -73,7 +73,7 @@ describe Flipper::Echo do
     describe '#clear' do
       it 'clears feature' do
         expect_any_instance_of(Flipper::Echo::Event).to receive(:notify) do |e|
-          expect(e.action).to eq(:clear)
+          expect(e.action).to eq(:cleared)
         end
 
         adapter.clear(feature)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: flipper-echo
 version: !ruby/object:Gem::Version
-  version: 0.0.1
+  version: 0.0.2
 platform: ruby
 authors:
 - Heather Rivers
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-03-13 00:00:00.000000000 Z
+date: 2015-03-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: flipper
@@ -110,10 +110,17 @@ files:
 - lib/flipper/echo.rb
 - lib/flipper/echo/configuration.rb
 - lib/flipper/echo/event.rb
+- lib/flipper/echo/slack.rb
+- lib/flipper/echo/slack/message.rb
+- lib/flipper/echo/slack/notifier.rb
+- lib/flipper/echo/stdout.rb
+- lib/flipper/echo/stdout/notifier.rb
 - lib/flipper/echo/version.rb
 - spec/flipper/echo/configuration_spec.rb
 - spec/flipper/echo/event_spec.rb
-- spec/flipper/echo/version_spec.rb
+- spec/flipper/echo/slack/message_spec.rb
+- spec/flipper/echo/slack/notifier_spec.rb
+- spec/flipper/echo/stdout/notifier_spec.rb
 - spec/flipper/echo_spec.rb
 - spec/spec_helper.rb
 - spec/support/adapter_support.rb
@@ -144,7 +151,9 @@ summary: Flipper event notifier
 test_files:
 - spec/flipper/echo/configuration_spec.rb
 - spec/flipper/echo/event_spec.rb
-- spec/flipper/echo/version_spec.rb
+- spec/flipper/echo/slack/message_spec.rb
+- spec/flipper/echo/slack/notifier_spec.rb
+- spec/flipper/echo/stdout/notifier_spec.rb
 - spec/flipper/echo_spec.rb
 - spec/spec_helper.rb
 - spec/support/adapter_support.rb