slack-notifier 0.1.0 → 0.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 83db950c51929ccead9dbf89c58da3b9541962ce
-  data.tar.gz: 5ee3e1251538ebae0fbdfc4424a9f9896e25dd91
+  metadata.gz: 343978628e7a8085818befad991f0c3629527237
+  data.tar.gz: c05543a0c0385f954bd79bd64def1743b2d8e346
 SHA512:
-  metadata.gz: bc71aa9876f7243a9c62f2aa3538e225e8c987b9df937712c67587917f26a6fdcf6fa275864efd162612d3c056e63a2c1b3fc64452b3a00f06ce39d3705f1c2e
-  data.tar.gz: 5003ed1c65c2c6f6555a1e5b41d9f3c436d4ec453e59c548699249adac6d77cd9bac59ebb631c0a9959f50911324c7a4d1cae97ca977f3382f74174824c825c3
+  metadata.gz: cb4ab2e7781936a0d9829e5426848a89832063f2e9791ab5531f89df244acfc29d01b814ac8e2f6d7d36797d7229f8d1acdf2e8b27992a13255980f29bf66d06
+  data.tar.gz: c21154be148aa4e44d432f3006229989e29a889b4302de2f7b5170a15e481150fe733809ea98ec5971eca428ce70fe3a2599e88fb490bd1e8ecb39de1d17dfbb

data/lib/slack-notifier.rb

@@ -1,6 +1,8 @@
 require 'httparty'
 require 'json'
 
+require_relative 'slack-notifier/link_formatter'
+
 module Slack
   class Notifier
 
@@ -39,47 +41,5 @@ module Slack
         "https://#{team}.slack.com/services/hooks/incoming-webhook?token=#{token}"
       end
 
-      class LinkFormatter
-        class << self
-          def format string
-            LinkFormatter.new(string).formatted
-          end
-        end
-
-        def initialize string
-          @orig = string
-        end
-
-        def formatted
-          @orig.gsub( html_pattern ) do |match|
-            link = Regexp.last_match[1]
-            text = Regexp.last_match[2]
-            slack_link link, text
-          end.gsub( markdown_pattern ) do |match|
-            link = Regexp.last_match[2]
-            text = Regexp.last_match[1]
-            slack_link link, text
-          end
-        end
-
-        private
-
-          def slack_link link, text=nil
-            out = "<#{link}"
-            out << "|#{text}" if text && !text.empty?
-            out << ">"
-
-            return out
-          end
-
-          def html_pattern
-            / <a (?:.*?) href=['"](.+)['"] (?:.*)> (.+?) <\/a> /x
-          end
-
-          def markdown_pattern
-            /\[(.*?)\]\((.+?)\)/
-          end
-      end
-
   end
 end

data/lib/slack-notifier/link_formatter.rb

@@ -0,0 +1,49 @@
+module Slack
+  class Notifier
+    class LinkFormatter
+
+      class << self
+
+        def format string
+          LinkFormatter.new(string).formatted
+        end
+
+      end
+
+      def initialize string
+        @orig = string
+      end
+
+      def formatted
+        @orig.gsub( html_pattern ) do |match|
+          link = Regexp.last_match[1]
+          text = Regexp.last_match[2]
+          slack_link link, text
+        end.gsub( markdown_pattern ) do |match|
+          link = Regexp.last_match[2]
+          text = Regexp.last_match[1]
+          slack_link link, text
+        end
+      end
+
+      private
+
+        def slack_link link, text=nil
+          out = "<#{link}"
+          out << "|#{text}" if text && !text.empty?
+          out << ">"
+
+          return out
+        end
+
+        def html_pattern
+          / <a (?:.*?) href=['"](.+)['"] (?:.*)> (.+?) <\/a> /x
+        end
+
+        def markdown_pattern
+          /\[(.*?)\]\((.+?)\)/
+        end
+
+    end
+  end
+end

data/lib/slack-notifier/version.rb

@@ -1,5 +1,5 @@
 module Slack
   class Notifier
-    VERSION = "0.1.0"
+    VERSION = "0.1.1"
   end
 end

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

@@ -0,0 +1,24 @@
+require 'spec_helper'
+
+describe Slack::Notifier::LinkFormatter do
+
+  describe "::format" do
+
+    it "formats html links" do
+      formatted = described_class.format("Hello World, enjoy <a href='http://example.com'>this</a>.")
+      expect( formatted ).to include("<http://example.com|this>")
+    end
+
+    it "formats markdown links" do
+      formatted = described_class.format("Hello World, enjoy [this](http://example.com).")
+      expect( formatted ).to include("<http://example.com|this>")
+    end
+
+    it "formats markdown links with no title" do
+      formatted = described_class.format("Hello World, enjoy [](http://example.com).")
+      expect( formatted ).to include("<http://example.com>")
+    end
+
+  end
+
+end

data/spec/lib/slack-notifier_spec.rb

@@ -0,0 +1,75 @@
+require 'spec_helper'
+
+describe Slack::Notifier do
+
+  describe "#initialize" do
+    it "sets the given team" do
+      subject = described_class.new 'team', 'token'
+      expect( subject.team ).to eq 'team'
+    end
+
+    it "sets the given token" do
+      subject = described_class.new 'team', 'token'
+      expect( subject.token ).to eq 'token'
+    end
+  end
+
+  describe "#ping" do
+    it "passes the message through LinkFormatter" do
+      allow( HTTParty ).to receive(:post)
+      expect( Slack::Notifier::LinkFormatter ).to receive(:format)
+                                              .with("the message")
+
+      described_class.new('team','token').ping "the message", channel: 'foo'
+    end
+
+    it "requires a channel to be set" do
+      allow( HTTParty ).to receive(:post)
+
+      expect{
+        described_class.new('team','token').ping "the message"
+      }.to raise_error
+    end
+
+    context "with a default channel set" do
+
+      before :each do
+        allow( HTTParty ).to receive(:post)
+        @subject = described_class.new('team','token')
+        @subject.channel = 'default'
+      end
+
+      it "does not require a channel to ping" do
+        expect{
+          @subject.ping "the message"
+        }.not_to raise_error
+      end
+
+      it "uses default channel" do
+        expect( HTTParty ).to receive(:post)
+                          .with "https://team.slack.com/services/hooks/incoming-webhook?token=token",
+                                body: 'payload={"text":"the message","channel":"default"}'
+
+        @subject.ping "the message"
+      end
+
+      it "allows override channel to be set" do
+        expect( HTTParty ).to receive(:post)
+                          .with "https://team.slack.com/services/hooks/incoming-webhook?token=token",
+                                body: 'payload={"text":"the message","channel":"new"}'
+
+        @subject.ping "the message", channel: "new"
+      end
+
+    end
+
+    it "posts with the correct endpoint & data" do
+        expect( HTTParty ).to receive(:post)
+                          .with "https://team.slack.com/services/hooks/incoming-webhook?token=token",
+                                body: 'payload={"text":"the message","channel":"channel"}'
+
+        described_class.new("team","token").ping "the message", channel: "channel"
+    end
+  end
+
+end

data/spec/spec_helper.rb

@@ -1,2 +1,13 @@
+require 'rspec'
 require 'slack-notifier'
-require 'pry-debugger'
+
+if ENV['DEBUG']
+  require 'pry'
+end
+
+RSpec.configure do |config|
+  config.mock_with :rspec do |mocks|
+    mocks.verify_doubled_constant_names = true
+    mocks.verify_partial_doubles        = true
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: slack-notifier
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.1.1
 platform: ruby
 authors:
 - Steven Sloan
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-09-07 00:00:00.000000000 Z
+date: 2014-03-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: httparty
@@ -16,24 +16,26 @@ dependencies:
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: 0.11.0
+        version: '0.11'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: 0.11.0
-description: ' A simple wrapper for posting to slack channels '
+        version: '0.11'
+description: ' A slim ruby wrapper for posting to slack channels '
 email:
 - stevenosloan@gmail.com
 executables: []
 extensions: []
 extra_rdoc_files: []
 files:
-- lib/slack-notifier/version.rb
 - lib/slack-notifier.rb
-- spec/slack-notifier_spec.rb
+- lib/slack-notifier/link_formatter.rb
+- lib/slack-notifier/version.rb
+- spec/lib/slack-notifier/link_formatter_spec.rb
+- spec/lib/slack-notifier_spec.rb
 - spec/spec_helper.rb
 homepage: http://github.com/stevenosloan/slack-notifier
 licenses:
@@ -55,10 +57,11 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.0.5
+rubygems_version: 2.2.2
 signing_key: 
 specification_version: 4
-summary: A simple wrapper for posting to slack channels
+summary: A slim ruby wrapper for posting to slack channels
 test_files:
-- spec/slack-notifier_spec.rb
+- spec/lib/slack-notifier/link_formatter_spec.rb
+- spec/lib/slack-notifier_spec.rb
 - spec/spec_helper.rb

data/spec/slack-notifier_spec.rb

@@ -1,22 +0,0 @@
-require 'spec_helper'
-
-describe Slack::Notifier do
-
-  describe Slack::Notifier::LinkFormatter do
-    it "formats html links" do
-      formatted = Slack::Notifier::LinkFormatter.format("Hello World, enjoy <a href='http://example.com'>this</a>.")
-      expect( formatted ).to include("<http://example.com|this>")
-    end
-
-    it "formats markdown links" do
-      formatted = Slack::Notifier::LinkFormatter.format("Hello World, enjoy [this](http://example.com).")
-      expect( formatted ).to include("<http://example.com|this>")
-    end
-
-    it "formats markdown links with no title" do
-      formatted = Slack::Notifier::LinkFormatter.format("Hello World, enjoy [](http://example.com).")
-      expect( formatted ).to include("<http://example.com>")
-    end
-  end
-
-end