octospy 0.0.8 → 0.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 1a73c640fd360e1e16615586c052e08e38c88303
-  data.tar.gz: f7b6ea37bb938458d8222293e2866b252155d689
+  metadata.gz: f7923492e57e7a32c5d3c16e4eeeecdecd1888d8
+  data.tar.gz: 6b7ed648db808d49a433732875d47a3478419976
 SHA512:
-  metadata.gz: 7fc62aa57019b25d910d998669a21ac23fdbaf745eec989ff167d2f68fa81639a86986081403760a3adfb5668fab17be10c36c1734e62f35048930983bbe115c
-  data.tar.gz: b23ca9be2d3ce6ca52a2eea1e41fe9a1cd6d64e250b1db8218ef92d081da66140a1cd07a3efb2cf51294d84cce04060c302af97ce49721b84abc191419ea9570
+  metadata.gz: c811098f2c8a11f3f04246cede91a5e2dc14383a17b8834ce92a967d1c7c0a3c226e0b83cfca48b029f0a2b027684ed3f5b9a10e3caab7533eb02b690542d54c
+  data.tar.gz: 031ff920915e73b8053690b94d0ef02bc3c3d8ab42d907cb688d4b16b117405a907b8e689e2df60813bde9e0a1ec5e78f381651ce77418de3073f8639abcc6c4

data/.travis.yml

@@ -1,21 +1,12 @@
 language: ruby
-
 rvm:
-  - 1.9.3
   - 2.0.0
-
+  - 2.1.1
 before_install:
   - gem update bundler
-
 script:
-  - bundle exec rake spec
-
+  - bundle exec rspec
 notifications:
-  email:
-    recipients:
-    - linyows@gmail.com
-    on_success: change
-    on_failure: always
   irc:
     on_success: change
     on_failure: always

data/README.md

@@ -13,7 +13,7 @@ Octospy
 [codeclimate]: https://codeclimate.com/github/linyows/octospy
 [coveralls]: https://coveralls.io/r/linyows/octospy
 
-Octospy notifies eventsf github repositories to IRC channels.
+Octospy notifies events of github repositories to IRC channels.
 
 <div align="center">
 <img src="http://octodex.github.com/images/daftpunktocat-thomas.gif" width="300">
@@ -77,6 +77,12 @@ GITHUB_WEB_ENDPOINT=http://your.enterprise.domain
 env DOTENV=".env.myserver" octospy
 ```
 
+### Debug
+
+```sh
+env DEBUG=true octospy
+```
+
 Commands
 --------
 
@@ -87,12 +93,14 @@ Command                          | Description
 `watch <repository>`             | add repository to watch list (ex: watch rails/rails)
 `unwatch <repository>`           | remove repository to watch list
 `watch <user or organization>`   | add user's repositories to watch list (ex: watch dotcloud)
-`unwatch <user or organization>` | remove user's repositories to watch list
+`unwatch <user or organization>` | remove user's repositories from watch list
+`clear watched`                  | remove all repositories from watch list
 `start`                          | start the repository event monitoring
 `stop`                           | stop the repository event monitoring
 `join <channel>`                 | invite octospy to another channel
 `show watched`                   | display the watch list
 `show status`                    | display the joined channels
+`help`                           | display the commands
 
 Contributing
 ------------

data/lib/cinch/plugins/octospy.rb

@@ -8,7 +8,7 @@ module Cinch
       include Octospy::Recording
       include Octospy::Job
 
-      set :prefix, ->(m) { %r(^#{Regexp.escape "#{m.bot.nick}: "}) }
+      set :prefix, ->(m) { %r(^ ?#{Regexp.escape "#{m.bot.nick}"}:? ) }
 
       match(/hello|hi|hey/, method: :greet)
       match('ping', method: :pong)
@@ -16,7 +16,7 @@ module Cinch
       match(/join (.+)/, method: :join)
       match(/part(?: (.+))?/, method: :part)
       match(/show status/, method: :show_status)
-      match(/show commands/, method: :show_commands)
+      match(/show commands|help/, method: :show_commands)
 
       listen_to :invite, method: :join_on_invite
 
@@ -50,14 +50,20 @@ module Cinch
       def show_status(m)
         @bot.channels.each.with_index(1) do |channel, i|
           number = ::Octospy::Recordable.channel(channel).repos.count
-          m.reply "#{"%02d" % i} #{channel}: #{number} repo"
+          m.reply "#{"%02d" % i} #{channel}: #{number} repo#{'s' unless number.zero?}"
         end
       end
 
       def show_commands(m)
-        # @matchers.each.with_index(1) do |matcher, i|
-          # m.reply "#{"%02d" % i} #{matcher}"
-        # end
+        m.reply "#{m.bot.name}:"
+        @handlers.each do |handler|
+          pattern = handler.pattern.pattern
+          command = case pattern.class.name
+            when 'Regexp' then pattern.source unless pattern.source == ''
+            when 'String' then pattern unless pattern.empty?
+            end
+          m.reply " #{command}" if command
+        end
       end
 
       def join_on_invite(m)

data/lib/cinch/plugins/octospy/recording.rb

@@ -8,6 +8,7 @@ module Cinch
             match(/unwatch ([\w\-\.]+)\/([\w\-\.]+)$/, method: :unwatch_repository)
             match(/watch ([\w\-\.]+)$/, method: :watch_repositories)
             match(/unwatch ([\w\-\.]+)$/, method: :unwatch_repositories)
+            match(/clear watched( repos(itories)?)?/, method: :clear_watched_repositories)
             match(/show watched( repos(itories)?)?/, method: :show_watched_repositories)
           end
         end
@@ -37,11 +38,12 @@ module Cinch
 
           ::Octospy::Recordable.add_channel m.channel.name
           method = "#{'org_' if user.type == 'Organization'}repos".to_sym
-          repos = ::Octokit.send(method, owner).map { |repo|
-            ::Octospy::Recordable.channel(m.channel.name).add_repo(repo.full_name)
+          repos = ::Octokit.send(method, owner, per_page: 100).map { |repo|
             repo.full_name
           }
 
+          ::Octospy::Recordable.channel(m.channel.name).add_repos(repos)
+
           if repos.count > 0
             m.reply "started to watch events of #{repos.count} repositories"
             restart(m)
@@ -64,11 +66,11 @@ module Cinch
 
         def unwatch_repositories(m, owner)
           repos = ::Octospy::Recordable.channel(m.channel.name).repos.each_with_object([]) do |repo, obj|
-            next unless repo.split('/').first == owner
-            ::Octospy::Recordable.channel(m.channel.name).remove_repo(repo)
-            opj << repo
+            obj << repo if repo.to_s.split('/').first == owner
           end
 
+          ::Octospy::Recordable.channel(m.channel.name).remove_repos(repos)
+
           if repos.count > 0
             if ::Octospy::Recordable.channel(m.channel.name).repos.count > 0
               m.reply "stopped to watch events of #{repos.count} repositories"
@@ -80,8 +82,9 @@ module Cinch
           end
         end
 
-        def unwatch_all(m)
+        def clear_watched_repositories(m)
           ::Octospy::Recordable.remove_channel(m.channel.name)
+          m.reply 'cleared'
         end
 
         def show_watched_repositories(m)

data/lib/octospy/configurable.rb

@@ -7,6 +7,7 @@ module Octospy
         ssl
         password
         nick
+        debug
         worker_interval
         api_request_interval
         cinch_config_block
@@ -45,6 +46,7 @@ module Octospy
       @server               = ENV['SERVER']
       @port                 = ENV['PORT']
       @ssl                  = !!ENV['SSL']
+      @debug                = !!ENV['DEBUG']
       @password             = ENV['PASSWORD']
       @worker_interval      = ENV['WORKER_INTERVAL'] ? ENV['WORKER_INTERVAL'].to_i : 30 #sec
       # you can make up to 20 requests per minute.

data/lib/octospy/extensions/string.rb

@@ -17,6 +17,10 @@ module Octospy
       end
       alias_method :split_lfbl, :split_by_linefeed_except_blankline
 
+      def compact
+        self.gsub(/\s+/, ' ').strip
+      end
+
       def colorize_for_irc
         StringIrc.new(self)
       end

data/lib/octospy/recordable.rb

@@ -27,7 +27,8 @@ module Octospy
         if channels_include? name
           find_channel name
         else
-          Channel.new(name)
+          add_channel name
+          find_channel name
         end
       end
     end

data/lib/octospy/recordable/channel.rb

@@ -12,11 +12,22 @@ module Octospy
       end
 
       def add_repo(name)
-        @repos << name if repos.empty? || !repos.include?(name)
+        repos << name.to_sym if !repos.include?(name.to_sym)
+      end
+
+      def add_repos(names = nil)
+        return if names.nil? || names.empty?
+        repos.concat(names.map { |repo|
+          repo.to_sym unless repos.include?(repo.to_sym) }.compact)
       end
 
       def remove_repo(name)
-        @repos.delete(name.to_sym) if !repos.empty? && repos.include?(name)
+        repos.delete(name.to_sym) if repos.include?(name.to_sym)
+      end
+
+      def remove_repos(names = nil)
+        return if names.nil? || names.empty?
+        repos.delete_if { |repo| names.include? repo.to_s }
       end
     end
   end

data/lib/octospy/version.rb

@@ -1,3 +1,3 @@
 module Octospy
-  VERSION = '0.0.8'
+  VERSION = '0.1.0'
 end

data/lib/octospy/worker.rb

@@ -9,52 +9,133 @@ module Octospy
       thread_loop
     end
 
+    def work_interval
+      (Octospy.api_request_interval * @repositories.count) + Octospy.worker_interval
+    end
+
     def thread_loop
-      @thread = Thread.start do
-        loop do
-          begin
-            watch_repositories
-            sleep Octospy.worker_interval
-          rescue => e
-            @block.call "Octospy Error: #{e.message}"
-            sleep Octospy.worker_interval
-          end
-        end
+      debug 'thread_start', <<-MSG.compact
+          api_request_interval(A): #{Octospy.api_request_interval},
+          repoisitory_count(R): #{@repositories.count},
+          worker_interval(W): #{Octospy.worker_interval},
+          work_interval(A*R+W): #{work_interval}
+        MSG
+
+      @thread = Thread.start { loop { work } }
+    end
+
+    def work
+      notify_recent_envets
+      debug 'sleep', work_interval
+      sleep work_interval
+    rescue => e
+      error e.message
+      debug 'sleep', work_interval
+      sleep worker_interval
+    end
+
+    def api_requestable?
+      limit = Octokit.rate_limit
+      if limit.remaining.zero?
+        notify "ヾ(;´Д`)ﾉ #{limit}"
+        false
+      else
+        true
       end
+    # No rate limit for white listed users
+    rescue Octokit::NotFound
+      true
     end
 
-    def events
+    def repository_events
       @repositories.each_with_object([]) do |repo, arr|
-        if Octokit.rate_limit.remaining.zero?
-          @block.call "ヾ(;´Д`)ﾉ #{::Octokit.rate_limit}"
-          break
-        end
+        break unless api_requestable?
 
         sleep Octospy.api_request_interval
-        arr.concat ::Octokit.repository_events(repo)
+        events = ::Octokit.repository_events(repo.to_s)
+        arr.concat events
+
+        debug_attrs = <<-MSG.compact
+          repo: #{repo},
+          limit: #{Octokit.rate_limit.remaining}/#{Octokit.rate_limit.limit},
+          reset: #{Octokit.rate_limit.resets_at.strftime('%H:%M:%S')} *after #{Octokit.rate_limit.resets_in}sec,
+        MSG
+
+        if !events.nil? && !events.empty?
+          debug_attrs << ' ' + <<-MSG.compact
+            first: #{events.first.type},
+            last: #{events.last.type}
+          MSG
+        end
+
+        debug 'get_event', debug_attrs
       end
     end
 
-    def while_ago
-      Time.now.utc - (60 * 30)
+    def skipping?(event)
+      case
+      when event.nil?,
+           @last_event_id.nil? && while_ago >= event.created_at,
+          !@last_event_id.nil? && @last_event_id >= event.id.to_i
+        true
+      else
+        false
+      end
     end
 
-    def watch_repositories
+    def notify_recent_envets
+      events = repository_events
+      return if events.count.zero?
+
+      last_event = events.sort_by(&:id).last
+      debug 'last_event', <<-MSG.compact
+          repo: #{last_event.repo.name},
+          event_type: #{last_event.type},
+          #{@last_event_id.nil? ?
+            "while_ago: #{while_ago}, created_at: #{last_event.created_at}" :
+            "last_id: #{@last_event_id}, current_id: #{last_event.id}"}
+        MSG
+
       # ascending by event.id
       events.sort_by(&:id).each { |event|
-        case
-        when @last_event_id.nil? && while_ago >= event.created_at
-          next
-        when !@last_event_id.nil? && @last_event_id >= event.id.to_i
-          next
-        end
+        next if skipping?(event)
 
         parsed_event = Octospy.parse(event)
-        next unless parsed_event
+
+        unless parsed_event
+          debug 'could_not_parse', <<-MSG.compact
+              repo: #{event.repo.name},
+              event: #{event.type}
+            MSG
+          next
+        end
 
         @last_event_id = event.id.to_i
-        parsed_event.each { |p| @block.call p[:message] }
+        parsed_event.each { |p| notify p[:message] }
       }
     end
+
+    private
+
+    def while_ago
+      Time.now.utc - (60 * 30)
+    end
+
+    def notify(message)
+      @block.call message
+    end
+
+    def debug(name, message = nil)
+      return unless Octospy.debug
+
+      prefix = '[DEBUG]'.colorize_for_irc.orange
+      info = name.colorize_for_irc.bold
+      @block.call "#{prefix} #{info} #{message}"
+    end
+
+    def error(message)
+      prefix = '[ERROR]'.colorize_for_irc.red
+      @block.call "#{prefix} #{message}"
+    end
   end
 end

data/octospy.gemspec

@@ -8,8 +8,8 @@ Gem::Specification.new do |spec|
   spec.version       = Octospy::VERSION
   spec.authors       = ['linyows']
   spec.email         = ['linyows@gmail.com']
-  spec.description   = %q{Octospy notifies eventsf github repositories to IRC channels.}
-  spec.summary       = %q{Octospy notifies eventsf github repositories to IRC channels.}
+  spec.description   = %q{Octospy notifies events of github repositories to IRC channels.}
+  spec.summary       = %q{Octospy notifies events of github repositories to IRC channels.}
   spec.homepage      = 'https://github.com/linyows/octospy'
   spec.license       = 'MIT'
 

data/spec/helper.rb

@@ -4,11 +4,9 @@ require 'simplecov'
 require 'coveralls'
 require 'octospy'
 require 'rspec'
-require 'json'
-require 'hashie'
-require 'awesome_print'
-require 'webmock/rspec'
+require 'ap'
 require 'vcr'
+require 'webmock/rspec'
 
 SimpleCov.formatter = SimpleCov::Formatter::MultiFormatter[
   SimpleCov::Formatter::HTMLFormatter,
@@ -20,8 +18,15 @@ WebMock.disable_net_connect!(allow: 'coveralls.io')
 RSpec.configure { |c| c.include WebMock::API }
 
 VCR.configure do |c|
+  c.configure_rspec_metadata!
   c.cassette_library_dir = 'spec/cassettes'
   c.hook_into :webmock
+  c.default_cassette_options = {
+    serialize_with: :json,
+    preserve_exact_body_bytes: true,
+    decode_compressed_response: true,
+    record: :once
+  }
 end
 
 class String
@@ -71,9 +76,3 @@ def method_missing(method, *args, &block)
     super
   end
 end
-
-def support_path
-  File.expand_path('../support', __FILE__)
-end
-
-Dir["#{support_path}/**/*.rb"].each { |f| require f }

data/spec/octospy/extensions/string_spec.rb

@@ -24,7 +24,6 @@ describe Octospy::Extensions::String do
       }
 
       it { expect(subject).to be_instance_of Array }
-      it { expect(subject).to have(4).items }
       it { expect(subject[1]).to include 'Blue sky' }
     end
 
@@ -32,18 +31,13 @@ describe Octospy::Extensions::String do
       let(:sentence) { 'hi, hello world.' }
 
       it { expect(subject).to be_instance_of Array }
-      it { expect(subject).to have(1).items }
-    end
-
-    it 'have alias_method' do
-      expect(''.respond_to? :split_lfbl).to be_true
     end
   end
 
   describe '#colorize_for_irc' do
     let(:word) { 'hello world' }
 
-    it 'call StringIrc#new' do
+    it 'calls StringIrc#new' do
       expect(StringIrc).to receive(:new).with(word).once.and_call_original
       expect(word.colorize_for_irc).to be_instance_of StringIrc
     end
@@ -54,8 +48,8 @@ describe Octospy::Extensions::String do
       let(:url) { 'https://github.com/linyows/octospy' }
       let(:result) { 'http://git.io/aaaaa' }
 
-      it 'call Octospy::Shortener.shorten_by_github' do
-        expect(Octospy::Shortener).to receive(:shorten_by_github).and_return result
+      it 'calls Octospy::Url.shorten_by_github' do
+        expect(Octospy::Url).to receive(:shorten_by_github).and_return result
         expect(url.shorten_url).to eq result
       end
     end
@@ -64,8 +58,8 @@ describe Octospy::Extensions::String do
       let(:url) { 'http://www.google.com/search?q=octospy' }
       let(:result) { 'http://goo.gl/aaaaa' }
 
-      it 'call Octospy::Shortener.shorten_by_google' do
-        expect(Octospy::Shortener).to receive(:shorten_by_google).and_return result
+      it 'calls Octospy::Url.shorten_by_google' do
+        expect(Octospy::Url).to receive(:shorten_by_google).and_return result
         expect(url.shorten_url).to eq result
       end
     end
@@ -74,9 +68,9 @@ describe Octospy::Extensions::String do
       let(:url) { 'hello world' }
       let(:result) { url }
 
-      it 'not call Octospy::Shortener' do
-        expect(Octospy::Shortener).to_not receive(:shorten_by_github)
-        expect(Octospy::Shortener).to_not receive(:shorten_by_google)
+      it 'not calls Octospy::Url' do
+        expect(Octospy::Url).to_not receive(:shorten_by_github)
+        expect(Octospy::Url).to_not receive(:shorten_by_google)
         expect(url.shorten_url).to eq result
       end
     end

data/spec/octospy/parser/download_spec.rb

@@ -1,9 +1,6 @@
 require 'helper'
 
 describe Octospy::Parser do
-  include_context :parser_params
-
   describe '#parse_download_event' do
-    it {}
   end
 end

data/spec/octospy/parser/gist_spec.rb

@@ -1,17 +1,6 @@
 require 'helper'
 
 describe Octospy::Parser do
-  include_context :parser_params
-
   describe '#parse_gist_event' do
-    let(:event_name) { 'gist_event' }
-    before { parser.instance_variable_set(:@event, event) }
-
-    subject { parser.parse_gist_event }
-
-    it { expect(subject[:status]).to eq 'created gist' }
-    it { expect(subject[:title]).to eq 'testing activegist' }
-    it { expect(subject[:body]).to be_nil }
-    it { expect(subject[:link]).to eq 'https://gist.github.com/5993603' }
   end
 end

data/spec/octospy/parser/issue_spec.rb

@@ -1,33 +1,9 @@
 require 'helper'
 
 describe Octospy::Parser do
-  include_context :parser_params
-
   describe '#parse_issues_event' do
-    let(:event_name) { 'issues_event' }
-    before { parser.instance_variable_set(:@event, event) }
-
-    subject { parser.parse_issues_event }
-
-    it { expect(subject[:status]).to eq 'opened issue #430' }
-    it { expect(subject[:title]).to eq 'mismatching hashes for css' }
-    it { expect(subject[:body]).to be_an_instance_of Array }
-    it { expect(subject[:body]).to include 'Server: nginx/1.4.1' }
-    it { expect(subject[:body]).to have(40).items }
-    it { expect(subject[:link]).to eq 'https://github.com/pagespeed/ngx_pagespeed/issues/430' }
   end
 
   describe '#parse_issue_comment_event' do
-    let(:event_name) { 'issue_comment_event' }
-    before { parser.instance_variable_set(:@event, event) }
-
-    subject { parser.parse_issue_comment_event }
-
-    it { expect(subject[:status]).to eq 'commented on issue #582' }
-    it { expect(subject[:title]).to eq 'Remove `engines` from package.json.' }
-    it { expect(subject[:body]).to be_an_instance_of Array }
-    it { expect(subject[:body][0]).to include 'anything?' }
-    it { expect(subject[:body]).to have(1).items }
-    it { expect(subject[:link]).to eq 'https://github.com/bower/bower/issues/582#issuecomment-20416296' }
   end
 end

data/spec/octospy/parser/organization_spec.rb

@@ -1,33 +1,9 @@
 require 'helper'
 
 describe Octospy::Parser do
-  include_context :parser_params
-
   describe '#parse_team_add_event' do
-    let(:event_name) { 'team_add_event' }
-    before { parser.instance_variable_set(:@event, event) }
-
-    subject { parser.parse_team_add_event }
-
-    it { expect(subject[:status]).to eq 'add team' }
-    it { expect(subject[:title]).to eq 'dev' }
-    it { expect(subject[:body]).to be_nil }
-    it { expect(subject[:link]).to be_nil }
   end
 
   describe '#parse_member_event' do
-    let(:event_name) { 'member_event' }
-
-    before do
-      client.stub(:web_endpoint).and_return Octokit.web_endpoint
-      parser.instance_variable_set(:@event, event)
-    end
-
-    subject { parser.parse_member_event }
-
-    it { expect(subject[:status]).to eq 'added member' }
-    it { expect(subject[:title]).to eq 'jamiesarahg' }
-    it { expect(subject[:body]).to be_nil }
-    it { expect(subject[:link]).to eq 'https://github.com//jamiesarahg' }
   end
 end

data/spec/octospy/parser/pull_request_spec.rb

@@ -1,33 +1,9 @@
 require 'helper'
 
 describe Octospy::Parser do
-  include_context :parser_params
-
   describe '#parse_pull_request_event' do
-    let(:event_name) { 'pull_request_event' }
-    before { parser.instance_variable_set(:@event, event) }
-
-    subject { parser.parse_pull_request_event }
-
-    it { expect(subject[:status]).to eq 'closed pull request #11363' }
-    it { expect(subject[:title]).to eq 'Fix 2 grammatical errors/typos in Active Record Basics guide.' }
-    it { expect(subject[:body]).to eq [] }
-    it { expect(subject[:link]).to eq 'https://github.com/rails/rails/pull/11363' }
   end
 
   describe '#parse_pull_request_review_comment_event' do
-    let(:pull) { double(Octokit::Client, title: 'Mocking title') }
-    let(:client) { double(Octokit::Client, pull: pull) }
-    let(:event_name) { 'pull_request_review_comment_event' }
-    before { parser.instance_variable_set(:@event, event) }
-
-    subject { parser.parse_pull_request_review_comment_event }
-
-    it { expect(subject[:status]).to eq 'commented on pull request' }
-    it { expect(subject[:title]).to eq 'Mocking title: packages/ember-handlebars/lib/helpers/collection.js' }
-    it { expect(subject[:body]).to be_an_instance_of Array }
-    it { expect(subject[:body][0]).to include 'Of course.' }
-    it { expect(subject[:body]).to have(1).items }
-    it { expect(subject[:link]).to eq 'https://github.com/emberjs/ember.js/pull/2930#discussion_r5064430' }
   end
 end

data/spec/octospy/parser/repository_spec.rb

@@ -1,101 +1,24 @@
 require 'helper'
 
 describe Octospy::Parser do
-  include_context :parser_params
-
   describe '#parse_push_event' do
-    let(:shorten_url) { 'http://git.io/Qq_ufw' }
-    let(:client) {
-      client = double(Octokit::Client)
-      client.stub_chain(:commit, :author, :login).and_return('mockingname')
-      client.stub(:web_endpoint).and_return(Octokit.web_endpoint)
-      String.any_instance.stub(:shorten).and_return shorten_url
-      client
-    }
-    let(:event_name) { 'push_event' }
-
-    before { parser.instance_variable_set(:@event, event) }
-
-    subject { parser.parse_push_event }
-
-    it { expect(subject[:status]).to eq 'pushed to master' }
-    it { expect(subject[:title]).to be_nil }
-    it { expect(subject[:body]).to be_an_instance_of Array }
-    it {
-      expect(subject[:body][0]).to eq <<-BODY.pretty_heredoc
-        \u000315mockingname\u000F:
-        update send and connect - \u000302#{shorten_url}\u000F
-      BODY
-    }
-    it { expect(subject[:body]).to have(2).items }
-    it { expect(subject[:link]).to eq 'https://github.com//visionmedia/express' }
   end
 
   describe '#parse_create_event' do
-    let(:client) { double(Octokit::Client, web_endpoint: Octokit.web_endpoint) }
-    let(:event_name) { 'create_event' }
-    before { parser.instance_variable_set(:@event, event) }
-
-    subject { parser.parse_create_event }
-
-    it { expect(subject[:status]).to eq 'created tag:3.3.3' }
-    it {
-      expect(subject[:title]).to eq <<-TITLE.pretty_heredoc
-        Sinatra inspired web development framework for node.js -- insanely fast,
-        flexible, and simple
-      TITLE
-    }
-    it { expect(subject[:body]).to be_nil }
-    it { expect(subject[:link]).to eq 'https://github.com//visionmedia/express' }
   end
 
   describe '#parse_commit_comment_event' do
-    let(:event_name) { 'commit_comment_event' }
-    before { parser.instance_variable_set(:@event, event) }
-
-    subject { parser.parse_commit_comment_event }
-
-    it { expect(subject[:status]).to eq 'commented on commit' }
-    it { expect(subject[:title]).to eq '' }
-    it { expect(subject[:body]).to be_an_instance_of Array }
-    it { expect(subject[:body][0]).to include 'AFAIK' }
-    it { expect(subject[:body]).to have(1).items }
-    it { expect(subject[:link]).to eq 'https://github.com/boxen/our-boxen/commit/08009e9b0718869d269d9b1c48383e6e145950db#commitcomment-3583654' }
   end
 
   describe '#parse_delete_event' do
-    let(:client) { double(Octokit::Client, web_endpoint: Octokit.web_endpoint) }
-    let(:event_name) { 'delete_event' }
-    before { parser.instance_variable_set(:@event, event) }
-
-    subject { parser.parse_delete_event }
-
-    it { expect(subject[:status]).to eq 'deleted branch:jefftk-fix-beacon' }
-    it { expect(subject[:title]).to be_nil }
-    it { expect(subject[:body]).to be_nil }
-    it { expect(subject[:link]).to eq 'https://github.com//pagespeed/ngx_pagespeed' }
   end
 
   describe '#parse_fork_event' do
   end
 
   describe '#parse_fork_apply_event' do
-    it { expect(parser.parse_fork_apply_event).to eq({}) }
   end
 
   describe '#parse_public_event' do
-    let(:event_name) { 'public_event' }
-
-    before do
-      client.stub(:web_endpoint).and_return Octokit.web_endpoint
-      parser.instance_variable_set(:@event, event)
-    end
-
-    subject { parser.parse_public_event }
-
-    it { expect(subject[:status]).to eq 'published JustinBeaudry/brudniakbook' }
-    it { expect(subject[:title]).to be_nil }
-    it { expect(subject[:body]).to be_nil }
-    it { expect(subject[:link]).to eq 'https://github.com//JustinBeaudry/brudniakbook' }
   end
 end

data/spec/octospy/parser/user_spec.rb

@@ -1,44 +1,9 @@
 require 'helper'
 
 describe Octospy::Parser do
-  include_context :parser_params
-
   describe '#parse_watch_event' do
-    let(:client) { double(Octokit::Client, web_endpoint: Octokit.web_endpoint) }
-    let(:event_name) { 'watch_event' }
-    before { parser.instance_variable_set(:@event, event) }
-
-    subject { parser.parse_watch_event }
-
-    it { expect(subject[:status]).to eq 'started repository' }
-    it { expect(subject[:title]).to eq 'intridea/hashie' }
-    it { expect(subject[:body]).to be_nil }
-    it { expect(subject[:link]).to eq 'https://github.com//intridea/hashie' }
-    it { expect(subject[:repository]).to be_nil }
   end
 
   describe '#parse_follow_event' do
-    let(:client) { double(Octokit::Client, web_endpoint: Octokit.web_endpoint) }
-    let(:event_name) { 'follow_event' }
-    before { parser.instance_variable_set(:@event, event) }
-
-    subject { parser.parse_follow_event }
-
-    it { expect(subject[:status]).to eq 'followed' }
-    it { expect(subject[:title]).to eq 'Watson1978 (Watson)' }
-    it {
-      expect(subject[:body]).to eq <<-BODY.pretty_heredoc
-        \u000315repos\u000F: 66,
-        \u000315followers\u000F: 101,
-        \u000315following\u000F: 15,
-        \u000315location\u000F: Japan,
-        \u000315company\u000F: -,
-        \u000315bio\u000F: -,
-        \u000315blog\u000F: http://watson1978.github.io/
-      BODY
-    }
-    it { expect(subject[:link]).to eq 'https://github.com//Watson1978' }
-    it { expect(subject[:repository]).to be_nil }
-    it { expect(subject[:notice]).to be_true }
   end
 end

data/spec/octospy/parser/wiki_spec.rb

@@ -1,17 +1,6 @@
 require 'helper'
 
 describe Octospy::Parser do
-  include_context :parser_params
-
   describe '#parse_gollum_event' do
-    let(:event_name) { 'gollum_event' }
-    before { parser.instance_variable_set(:@event, event) }
-
-    subject { parser.parse_gollum_event }
-
-    it { expect(subject[:status]).to eq 'edited the component/component wiki' }
-    it { expect(subject[:title]).to eq 'Components' }
-    it { expect(subject[:body]).to be_nil }
-    it { expect(subject[:link]).to eq 'https://github.com/component/component/wiki/Components/_compare/3df147%5E...3df147' }
   end
 end

data/spec/octospy/parser_spec.rb

@@ -1,63 +1,21 @@
 require 'helper'
 
 describe Octospy::Parser do
-  include_context :parser_params
-
   describe '#initialize' do
-    it { expect(parser).to be_an_instance_of Octospy::Parser }
   end
 
   describe '#default_params' do
-    it { expect(parser.default_params).to eq default_params }
   end
 
   describe '#parse' do
-    it {
-      expect(parser).to receive(:parsing_method).once.and_return(parsing_method)
-      expect(parser).to receive(parsing_method).once.and_return(parsed_params)
-      expect(parser).to receive(:build).once.with(merged_params).and_return(built_params)
-      expect(parser.parse event).to eq built_params
-    }
   end
 
   describe '#build' do
-    it {
-      expect_any_instance_of(String).to receive(:shorten).once.and_return('http://git.io/A0ARbg')
-      expect(parser.build merged_params).to eq built_params
-    }
   end
 
   describe '#parsing_method' do
-    let(:event) { double(Octokit::Client, type: 'FooBarBaz') }
-    before { parser.instance_variable_set(:@event, event) }
-    it { expect(parser.parsing_method).to eq :parse_foo_bar_baz }
-  end
-
-  describe '#behavior_color' do
-    {
-      pink: 'created',
-      yellow: 'commented',
-      lime: 'pushed',
-      orange: 'forked',
-      brown: 'closed',
-      red: 'deleted',
-      green: 'edited',
-      blue: 'published',
-      rainbow: 'started',
-      seven_eleven: 'followed',
-      aqua: 'foobar'
-    }.each { |color, word|
-      it { expect(parser.behavior_color word).to eq color }
-    }
   end
 
   describe '#colorize_to' do
-    let(:sentence) { 'created issue' }
-
-    it {
-      expect_any_instance_of(StringIrc).to receive(:pink).
-        once.with(no_args).and_call_original
-      expect(parser.colorize_to sentence).to eq "\u000313#{sentence}\u000F"
-    }
   end
 end

data/spec/octospy/url_spec.rb

@@ -0,0 +1,19 @@
+require 'helper'
+
+describe Octospy::Url do
+  let(:url) { 'https://github.com/linyows/octospy' }
+
+  describe '.shorten_by_github', :vcr do
+    subject { described_class.shorten_by_github url }
+
+    it { expect(subject).to be_an_instance_of String }
+    it { expect(subject).to eq 'http://git.io/Pd8gXg' }
+  end
+
+  describe '.shorten_by_google', :vcr do
+    subject { described_class.shorten_by_google url }
+
+    it { expect(subject).to be_an_instance_of String }
+    it { expect(subject).to eq 'http://goo.gl/8vrLj' }
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: octospy
 version: !ruby/object:Gem::Version
-  version: 0.0.8
+  version: 0.1.0
 platform: ruby
 authors:
 - linyows
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-03-01 00:00:00.000000000 Z
+date: 2014-03-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -108,7 +108,7 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '0.10'
-description: Octospy notifies eventsf github repositories to IRC channels.
+description: Octospy notifies events of github repositories to IRC channels.
 email:
 - linyows@gmail.com
 executables:
@@ -176,8 +176,7 @@ files:
 - spec/octospy/parser/user_spec.rb
 - spec/octospy/parser/wiki_spec.rb
 - spec/octospy/parser_spec.rb
-- spec/octospy/shortener_spec.rb
-- spec/support/shared_context.rb
+- spec/octospy/url_spec.rb
 homepage: https://github.com/linyows/octospy
 licenses:
 - MIT
@@ -201,7 +200,7 @@ rubyforge_project:
 rubygems_version: 2.2.2
 signing_key: 
 specification_version: 4
-summary: Octospy notifies eventsf github repositories to IRC channels.
+summary: Octospy notifies events of github repositories to IRC channels.
 test_files:
 - spec/fixtures/commit_comment_event.json
 - spec/fixtures/create_event.json
@@ -230,5 +229,4 @@ test_files:
 - spec/octospy/parser/user_spec.rb
 - spec/octospy/parser/wiki_spec.rb
 - spec/octospy/parser_spec.rb
-- spec/octospy/shortener_spec.rb
-- spec/support/shared_context.rb
+- spec/octospy/url_spec.rb

data/spec/octospy/shortener_spec.rb

@@ -1,46 +0,0 @@
-require 'helper'
-
-describe Octospy::Shortener do
-  let(:url) { 'https://github.com/linyows/octospy' }
-
-  describe '.shorten_by_github' do
-    subject {
-      VCR.use_cassette "git.io/#{url.escaping}" do
-        described_class.shorten_by_github url
-      end
-    }
-
-    it { expect(subject).to be_an_instance_of String }
-    it { expect(subject).to eq 'http://git.io/Pd8gXg' }
-
-    context 'when it is not in the url of github' do
-      let(:url) { 'https://www.google.co.jp/search?q=octospy' }
-      it { expect(subject).to eq url }
-    end
-
-    context 'when raise error' do
-      it 'return url of argument' do
-        expect(Faraday).to receive(:new).and_raise Faraday::Error
-        expect(described_class.shorten_by_github url).to eq url
-      end
-    end
-  end
-
-  describe '.shorten_by_google' do
-    subject {
-      VCR.use_cassette "googleapis.com/urlshortener/#{url.escaping}" do
-        described_class.shorten_by_google url
-      end
-    }
-
-    it { expect(subject).to be_an_instance_of String }
-    it { expect(subject).to eq 'http://goo.gl/8vrLj' }
-
-    context 'when raise error' do
-      it 'return url of argument' do
-        expect(Faraday).to receive(:new).and_raise Faraday::Error
-        expect(described_class.shorten_by_google url).to eq url
-      end
-    end
-  end
-end

data/spec/support/shared_context.rb

@@ -1,60 +0,0 @@
-shared_context :parser_params do
-  let(:client) { double Octokit::Client }
-  let(:parser) { described_class.new client }
-  let(:event_name) { 'issue_comment_event' }
-  let(:parsing_method) { :"parse_#{event_name}" }
-  let(:file) { "#{event_name}.json" }
-  let(:event) { Hashie::Mash.new.deep_merge(decode file) }
-
-  let(:default_params) do
-    {
-      notice_body: false,
-      none_repository: false,
-      nick: '',
-      repository: '',
-      status: '',
-      link: '',
-      title: '',
-      body: []
-    }
-  end
-
-  let(:parsed_params) do
-    {
-      status: 'commented on issue #582',
-      title: 'Remove `engines` from package.json.',
-      body: [
-        <<-BODY.pretty_heredoc
-          Has is ever done anything? It still has a purpose though;
-          To warn users that they're using antdated version.
-        BODY
-      ],
-      link: 'https://github.com/bower/bower/issues/582#issuecomment-20416296'
-    }
-  end
-
-  let(:merged_params) do
-    default_params.merge(parsed_params).merge(
-      nick: 'sindresorhus',
-      repository: 'bower/bower'
-    )
-  end
-
-  let(:built_params) do
-    [
-      {
-        nick: merged_params[:nick],
-        type: :notice,
-        message: <<-MSG.pretty_heredoc
-          (bower/bower) \u0002sindresorhus\u000F \u000308commented on issue #582\u000F
-          Remove `engines` from package.json. - \u000302http://git.io/A0ARbg\u000F
-        MSG
-      },
-      {
-        nick: merged_params[:nick],
-        type: :private,
-        message: parsed_params[:body]
-      }
-    ]
-  end
-end