gem-search 0.1.1 → 0.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: d6ff3a0f935a8163dae5b0184b2244d681de1b1b
-  data.tar.gz: e3ae44d76970cfabf800fbc3fe3db8d0fb69764c
+  metadata.gz: 4a1e1368ab020ee422a5df16ebed3581a5d1d295
+  data.tar.gz: bf0313e4e6bbc42fdbff3b5f51f781048531ceb2
 SHA512:
-  metadata.gz: 01886f5d4122f7e45c08ad2042a0540079f821ef86c7363af4440b8fcf39d6c3e9e2f4e00ddbb980aa3d1b60948c268cf33e7ffa2d0f6ee7266f30d4c5f20a56
-  data.tar.gz: 40d0eb2af445c882777d04aefe62aa03b65fd42718594a4acd3f1a1fc570510c75bd25ded9ea399ff5447dbf29aab3558498ccb302b1be50338aba014bb7cf2b
+  metadata.gz: d95848032e753ecb45d4757ccb2a951e4da0a3dc2983822b533a8bc6a824a8d847dfa1f30e9221190c8a799e4c415a08775614b9733df301ce3e994d6f4cd421
+  data.tar.gz: 0e6b871fcc56fb661ae43db82a2812c01f506cc59b480d124778743ca8e48025008c0098e0406e3450ab6f915ff9dc04472285dbc64088f687e42080331d692a

data/.gitignore

@@ -16,3 +16,4 @@ spec/reports
 test/tmp
 test/version_tmp
 tmp
+TODO.txt

data/CHANGELOG.md

@@ -1,3 +1,8 @@
+## v0.1.2
+
+* Correspond to linux with browse option [80bb881]
+* Implement a feature that is to open homepage url by a default browser. [49d2924]
+
 ## v0.1.1
 
 * Change default sort option from n[ame] to do a[ll] (download) [7041464]

data/README.md

@@ -19,8 +19,32 @@ install it yourself as:
 
 Use the gem-search as follows:
 
-### Sorted by gem name
+
+### Sorted by DL(all)
+
 	gem-search `keyword'
+	or
+	gem-search `keyword' -s a
+
+e.g.:
+
+```
+$ gem-search webkit
+NAME                                                DL(ver)   DL(all)
+-------------------------------------------------- -------- ---------
+capybara-webkit (0.12.1)                              24863    185214
+jasmine-headless-webkit (0.8.4)                       19444     46147
+guard-jasmine-headless-webkit (0.3.2)                 11964     17872
+rspec-formatter-webkit (2.2.0)                          438      3549
+webkit-rspec-formatter (2.0.2)                          613      1573
+intentmedia-capybara-webkit (0.7.2.4)                   370      1362
+capybara-webkit-remote (0.0.4)                          209       838
+otherinbox-capybara-webkit (0.12.0.1)                   594       740
+gtk-webkit-ruby (0.0.3)                                 289       289
+```
+
+### Sorted by gem name
+	gem-search `keyword' -s n
 
 e.g.:
 
@@ -59,26 +83,12 @@ gtk-webkit-ruby (0.0.3)                                 289       289
 capybara-webkit-remote (0.0.4)                          209       838
 ```
 
-### Sorted by DL(all)
+### Browse
+It open rubygem's homepage in the system's default web browser.
 
-	gem-search `keyword' -s a
+	gem-search browse 'gem_name'
 
-e.g.:
 
-```
-$ gem-search webkit -s a
-NAME                                                DL(ver)   DL(all)
--------------------------------------------------- -------- ---------
-capybara-webkit (0.12.1)                              24863    185214
-jasmine-headless-webkit (0.8.4)                       19444     46147
-guard-jasmine-headless-webkit (0.3.2)                 11964     17872
-rspec-formatter-webkit (2.2.0)                          438      3549
-webkit-rspec-formatter (2.0.2)                          613      1573
-intentmedia-capybara-webkit (0.7.2.4)                   370      1362
-capybara-webkit-remote (0.0.4)                          209       838
-otherinbox-capybara-webkit (0.12.0.1)                   594       740
-gtk-webkit-ruby (0.0.3)                                 289       289
-```
 
 ## Contributing
 

data/bin/gem-search

@@ -1,6 +1,4 @@
 #!/usr/bin/env ruby
-$LOAD_PATH.unshift File.expand_path(File.dirname(__FILE__) + '/../lib')
-require 'gem-search'
+require 'gem_search'
 
 Gem::Search::Command.new.run
-

data/gem-search.gemspec

@@ -1,9 +1,9 @@
-require File.expand_path('../lib/gem-search/version', __FILE__)
+require File.expand_path('../lib/gem_search/version', __FILE__)
 
 def install_message
   s = ''
   s << "\xf0\x9f\x8d\xba  " if or_over_mac_os_lion?
-  s << "Thanks for installing!"
+  s << 'Thanks for installing!'
 end
 
 def or_over_mac_os_lion?
@@ -11,31 +11,31 @@ def or_over_mac_os_lion?
 
   macos_full_version = `/usr/bin/sw_vers -productVersion`.chomp
   macos_version = macos_full_version[/10\.\d+/]
-  return macos_version >= '10.7'  # 10.7 is lion
+  macos_version >= '10.7'  # 10.7 is lion
 end
 
 Gem::Specification.new do |gem|
-  gem.authors       = ["rochefort"]
-  gem.email         = ["terasawan@gmail.com"]
-  gem.homepage      = "https://github.com/rochefort/gem-search"
-  gem.summary       = "search gems with using rubygems.org API"
+  gem.authors       = ['rochefort']
+  gem.email         = ['terasawan@gmail.com']
+  gem.homepage      = 'https://github.com/rochefort/gem-search'
+  gem.summary       = 'search gems with using rubygems.org API'
   gem.description   = gem.summary
 
   gem.files         = `git ls-files`.split($\)
   gem.executables   = gem.files.grep(%r{^bin/}).map{ |f| File.basename(f) }
   gem.test_files    = gem.files.grep(%r{^(test|spec|features)/})
-  gem.name          = "gem-search"
-  gem.require_paths = ["lib"]
+  gem.name          = 'gem-search'
+  gem.require_paths = ['lib']
   gem.version       = Gem::Search::VERSION
 
   gem.post_install_message = install_message
 
-  gem.add_dependency 'slop', '~>3.5.0'
+  gem.add_dependency 'slop', '~>3.6.0'
   gem.add_dependency 'json', '~>1.8.1'
 
-  gem.add_development_dependency 'webmock', '~>1.17.4'
+  gem.add_development_dependency 'webmock', '~>1.18.0'
   gem.add_development_dependency 'rake',    '~>10.3.1'
 
-  gem.add_development_dependency 'rspec',       '~> 2.14.1'
-  gem.add_development_dependency 'simplecov',   '~> 0.8.2'
+  gem.add_development_dependency 'rspec',       '~> 3.0.0'
+  gem.add_development_dependency 'simplecov',   '~> 0.9.0'
 end

data/lib/gem_search.rb

@@ -0,0 +1,9 @@
+module Gem
+  module Search
+    class LibraryNotFound    < StandardError; end
+    autoload :Command,   'gem_search/command'
+    autoload :Executor,  'gem_search/executor'
+    autoload :Rendering, 'gem_search/rendering'
+    autoload :VERSION,   'gem_search/version'
+  end
+end

data/lib/{gem-search → gem_search}/command.rb

@@ -18,13 +18,32 @@ module Gem::Search
     OPTS = Slop.parse(help: true) do
       banner "Usage: gem-search gem_name [options]\n"
       on :s, :sort, opt_description([
-          'Sort by the item.',
-          '  default [a]ll',
-          '  [a]ll  :DL(all)  eg. gem-search webkit -s a',
-          '  [v]er  :DL(ver)  eg. gem-search webkit -s v',
-          '  [n]ame :         eg. gem-search webkit -s n',
-        ]), :argument => :optional
+        'Sort by the item.',
+        '  default [a]ll',
+        '  [a]ll  :DL(all)  eg. gem-search webkit -s a',
+        '  [v]er  :DL(ver)  eg. gem-search webkit -s v',
+        '  [n]ame :         eg. gem-search webkit -s n',
+      ]), argument: :optional
       on :v, :version, 'Display the version.'
+
+      command 'browse' do
+        description "Open rubygem's homepage in the system's default web browser"
+        run do |_opts, args|
+          gem = args.first
+          executor = Executor.new
+          begin
+            executor.browse(gem)
+          rescue OpenURI::HTTPError
+            puts 'No such gem.'
+            abort
+          rescue => e
+            puts e.message
+            puts e.stacktrace
+            abort
+          end
+          exit
+        end
+      end
     end
 
     def run
@@ -47,21 +66,22 @@ module Gem::Search
     end
 
     private
-      def version
-        puts "gem-search #{Gem::Search::VERSION}"
-        exit
-      end
 
-      def validate
-        if (ARGV.size == 0)
-          puts OPTS
-          abort
-        end
+    def version
+      puts "gem-search #{Gem::Search::VERSION}"
+      exit
+    end
 
-        if ARGV.any? { |arg| arg.match(/\A-/) }
-          puts OPTS
-          exit
-        end
+    def validate
+      if (ARGV.size == 0)
+        puts OPTS
+        abort
       end
+
+      if ARGV.any? { |arg| arg.match(/\A-/) }
+        puts OPTS
+        exit
+      end
+    end
   end
 end

data/lib/gem_search/executor.rb

@@ -0,0 +1,79 @@
+require 'json'
+require 'open-uri'
+
+module Gem::Search
+  class Executor
+    include Gem::Search::Rendering
+    BASE_URL   = 'https://rubygems.org'
+    SEARCH_API = "#{BASE_URL}/api/v1/search.json?query=%s&page=%d"
+    GEM_API    = "#{BASE_URL}/api/v1/gems/%s.json"
+    GEM_URL    = "#{BASE_URL}/%s"
+
+    MAX_REQUEST_COUNT = 20
+
+    def search(query, opt_sort = 'downloads')
+      return unless query
+      print 'Searching '
+      gems = []
+      (1..MAX_REQUEST_COUNT).each do |n|
+        print '.'
+        url = SEARCH_API % [query, n]
+        gems_by_page = open_rubygems_api(url)
+        break if gems_by_page.size.zero?
+        gems += gems_by_page
+      end
+      puts
+
+      fail Gem::Search::LibraryNotFound, 'We did not find results.' if gems.size.zero?
+      gems_sort!(gems, opt_sort)
+      Executor.render(gems)
+    end
+
+    def browse(gem)
+      return unless gem
+      api_url = GEM_API % gem
+      result = open_rubygems_api(api_url)
+      url = result['homepage_uri']
+      if url.nil? || url.empty?
+        url = GEM_URL % gem
+      end
+      browser_open(url)
+    end
+
+    private
+
+    # https://github.com/github/hub/blob/9c589396ae38f7b9f98319065ad491149954c152/lib/hub/context.rb#L517
+    def browser_open(url)
+      cmd = osx? ? 'open' :
+        %w[xdg-open cygstart x-www-browser firefox opera mozilla netscape].find { |comm| which comm }
+      system(cmd, url)
+    end
+
+    # refer to: https://github.com/github/hub/blob/9c589396ae38f7b9f98319065ad491149954c152/lib/hub/context.rb#L527
+    def osx?
+      require 'rbconfig'
+      RbConfig::CONFIG['host_os'].to_s.include?('darwin')
+    end
+
+    def open_rubygems_api(url)
+      option = {}
+      proxy = URI.parse(url).find_proxy
+      if proxy
+        if proxy.user && proxy.password
+          option[:proxy_http_basic_authentication] = [proxy, proxy.user, proxy.password]
+        else
+          option[:proxy] = proxy
+        end
+      end
+      JSON.parse(open(url, option).read)
+    end
+
+    def gems_sort!(gems, opt_sort)
+      if opt_sort == 'name'
+        gems.sort! { |x, y| x[opt_sort] <=> y[opt_sort] }
+      else
+        gems.sort! { |x, y| y[opt_sort] <=> x[opt_sort] }
+      end
+    end
+  end
+end

data/lib/gem_search/rendering.rb

@@ -0,0 +1,41 @@
+module Gem::Search
+  module Rendering
+    DEFAULT_RULED_LINE_SIZE = [50, 8, 9]
+
+    def self.included(base)
+      base.extend(self)
+    end
+
+    def render(gems)
+      ruled_line_size(gems)
+      render_to_header
+      render_to_body(gems)
+    end
+
+    private
+
+    def ruled_line_size(gems)
+      max_name_size = gems.map { |gem| "#{gem['name'] } (#{gem['version']})".size }.max
+      DEFAULT_RULED_LINE_SIZE[0] = max_name_size if max_name_size > DEFAULT_RULED_LINE_SIZE[0]
+    end
+
+    def render_to_header
+      f = DEFAULT_RULED_LINE_SIZE
+      fmt = "%-#{f[0]}s %#{f[1]}s %#{f[2]}s"
+      puts fmt % ['NAME', 'DL(ver)', 'DL(all)']
+      puts fmt % ['-'*f[0], '-'*f[1], '-'*f[2]]
+    end
+
+    def render_to_body(gems)
+      f = DEFAULT_RULED_LINE_SIZE
+      fmt = "%-#{f[0]}s %#{f[1]}d %#{f[2]}d"
+      gems.each do |gem|
+        puts fmt % [
+          "#{gem['name']} (#{gem['version']})",
+          gem['version_downloads'],
+          gem['downloads']
+        ]
+      end
+    end
+  end
+end

data/lib/{gem-search → gem_search}/version.rb

@@ -1,3 +1,3 @@
 module Gem::Search
-  VERSION = '0.1.1'
+  VERSION = '0.1.2'
 end

data/spec/bin/{gem-search_spec.rb → gem_search_spec.rb}

@@ -11,16 +11,21 @@ Usage: gem-search gem_name [options]
                         [n]ame :         eg. gem-search webkit -s n
     -v, --version      Display the version.
     -h, --help         Display this help message.
-EOS
 
+Available commands:
+
+  browse   Open rubygem's homepage in the system's default web browser
+
+See `<command> --help` for more information on a specific command.
+EOS
 
-shared_examples 'display an usage' do
-  it 'should display an usage' do
+RSpec.shared_examples 'display an usage' do
+  it 'display an usage' do
     should == USAGE
   end
 end
 
-describe 'bin/gem-search' do
+RSpec.describe 'bin/gem-search' do
   context 'with no argument' do
     subject { `#{BIN}` }
     it_behaves_like 'display an usage'
@@ -38,7 +43,7 @@ describe 'bin/gem-search' do
 
   context 'with -v' do
     subject { `#{BIN} -v` }
-    it 'should display an usage' do
+    it 'display an usage' do
       should == "gem-search #{Gem::Search::VERSION}\n"
     end
   end

data/spec/command_spec.rb

@@ -2,34 +2,34 @@ require 'spec_helper'
 
 include Gem::Search
 
-describe Command do
+RSpec.describe Command do
   describe '#run' do
     before do
       @executor = Executor.new
-      Executor.should_receive(:new).and_return(@executor)
-      ARGV.stub(:[]).with(0).and_return(query)
-      stub_request_search(1, dummy_search_result)
-      stub_request_no_result_with_page(2)
+      allow(Executor).to receive(:new).and_return(@executor)
+      allow(ARGV).to receive(:[]).with(0).and_return(query)
+      stub_request_search(query, 1, dummy_search_result)
+      stub_request_search_no_result_with_page(query, 2)
     end
-    let(:query) {'factory-girl'}
+    let(:query) { 'factory_girl' }
 
     context 'with no sort option' do
       before do
-        Command::OPTS.stub(:[]).and_return(nil)
+        allow(Command::OPTS).to receive(:[]).and_return(nil)
       end
       it 'called with no sort option' do
-        @executor.should_receive(:search).with(query).once
+        expect(@executor).to receive(:search).with(query).once
         Command.new.run
       end
     end
 
     context 'with sort option' do
       before do
-        Command::OPTS.stub(:[]).and_return(nil)
-        Command::OPTS.stub(:[]).with('sort').and_return('a')
+        allow(Command::OPTS).to receive(:[]).and_return(nil)
+        allow(Command::OPTS).to receive(:[]).with('sort').and_return('a')
       end
       it 'called with sort option' do
-        @executor.should_receive(:search).with(query, 'downloads').once
+        expect(@executor).to receive(:search).with(query, 'downloads').once
         Command.new.run
       end
     end

data/spec/executor_spec.rb

@@ -2,37 +2,37 @@ require 'spec_helper'
 
 include Gem::Search
 
-describe Executor do
-  describe '#search' do
-    before do
-      @executor = Gem::Search::Executor.new
-    end
+RSpec.describe Executor do
+  before do
+    @executor = Executor.new
+  end
 
+  describe '#search' do
     context 'when a network error occurred' do
       before do
-        stub_request(:get, build_search_uri(query, 1)).
-          to_return(:status => 500, :body => '[]')
+        stub_request(:get, build_search_uri(query, 1))
+          .to_return(status: 500, body: '[]')
       end
-      let(:query) {'network_error_orccurred'}
-      it { expect{ @executor.search(query) }.to raise_error(Exception) }
+      let(:query) { 'network_error_orccurred' }
+      it { expect { @executor.search(query) }.to raise_error(Exception) }
     end
 
     context 'when no matching gem' do
-      before { stub_request_no_result_with_page(1) }
-      let(:query) {'no_match_gem_name'}
-      it { expect{ @executor.search(query) }.to raise_error(LibraryNotFound) }
+      before { stub_request_search_no_result_with_page(query, 1) }
+      let(:query) { 'no_match_gem_name' }
+      it { expect { @executor.search(query) }.to raise_error(LibraryNotFound) }
     end
 
     describe 'sorting' do
       before do
-        stub_request_search(1, dummy_search_result)
-        stub_request_no_result_with_page(2)
+        stub_request_search(query, 1, dummy_search_result)
+        stub_request_search_no_result_with_page(query, 2)
       end
-      let(:query) {'factory_girl'}
+      let(:query) { 'factory_girl' }
 
       context 'with no sort option' do
         it 'display rubygems ordering by name' do
-          capture(:stdout) { @executor.search(query) }.should == <<-'EOS'.unindent
+          res = <<-'EOS'.unindent
             |Searching ..
             |NAME                                                DL(ver)   DL(all)
             |-------------------------------------------------- -------- ---------
@@ -40,12 +40,13 @@ describe Executor do
             |factory_girl_rails (3.5.0)                            39724   1238780
             |factory_girl_generator (0.0.3)                         8015     15547
           EOS
+          expect { @executor.search(query) }.to output(res).to_stdout
         end
       end
 
       context 'with sort option: [a]download' do
         it 'display rubygems ordering by name' do
-          capture(:stdout) { @executor.search(query, 'download') }.should == <<-'EOS'.unindent
+          res = <<-'EOS'.unindent
             |Searching ..
             |NAME                                                DL(ver)   DL(all)
             |-------------------------------------------------- -------- ---------
@@ -53,12 +54,13 @@ describe Executor do
             |factory_girl_rails (3.5.0)                            39724   1238780
             |factory_girl_generator (0.0.3)                         8015     15547
           EOS
+          expect { @executor.search(query, 'download') }.to output(res).to_stdout
         end
       end
 
       context 'with sort option: [v]version_downloads' do
         it 'display rubygems ordering by name' do
-          capture(:stdout) { @executor.search(query, 'version_downloads') }.should == <<-'EOS'.unindent
+          res = <<-'EOS'.unindent
             |Searching ..
             |NAME                                                DL(ver)   DL(all)
             |-------------------------------------------------- -------- ---------
@@ -66,12 +68,13 @@ describe Executor do
             |factory_girl_generator (0.0.3)                         8015     15547
             |factory_girl (3.6.0)                                    541   2042859
           EOS
+          expect { @executor.search(query, 'version_downloads') }.to output(res).to_stdout
         end
       end
 
       context 'with sort option: [n]ame' do
         it 'display rubygems ordering by name' do
-          capture(:stdout) { @executor.search(query, 'name') }.should == <<-'EOS'.unindent
+          res = <<-'EOS'.unindent
             |Searching ..
             |NAME                                                DL(ver)   DL(all)
             |-------------------------------------------------- -------- ---------
@@ -79,20 +82,21 @@ describe Executor do
             |factory_girl_generator (0.0.3)                         8015     15547
             |factory_girl_rails (3.5.0)                            39724   1238780
           EOS
+          expect { @executor.search(query, 'name') }.to output(res).to_stdout
         end
       end
     end
 
     describe 'multiple page' do
       before do
-        stub_request_search(1, load_http_stubs('cucumber-_1.json'))
-        stub_request_search(2, load_http_stubs('cucumber-_2.json'))
-        stub_request_search(3, load_http_stubs('cucumber-_3.json'))
-        stub_request_no_result_with_page(4)
+        stub_request_search(query, 1, load_http_stubs('search/cucumber-_1.json'))
+        stub_request_search(query, 2, load_http_stubs('search/cucumber-_2.json'))
+        stub_request_search(query, 3, load_http_stubs('search/cucumber-_3.json'))
+        stub_request_search_no_result_with_page(query, 4)
       end
       let(:query) { 'cucumber-' }
       it 'display rubygems ordering by name' do
-        capture(:stdout) { @executor.search(query, 'name') }.should == <<-'EOS'.unindent
+        res = <<-'EOS'.unindent
           |Searching ....
           |NAME                                                DL(ver)   DL(all)
           |-------------------------------------------------- -------- ---------
@@ -160,58 +164,106 @@ describe Executor do
           |tasty-cucumber-client (0.1.10)                         1504     11518
           |vagrant-cucumber-host (0.1.14)                          163       163
         EOS
+        expect { @executor.search(query, 'name') }.to output(res).to_stdout
       end
     end
 
     describe 'ruled NAME line' do
       context 'NAME size is 42' do
         before do
-          stub_request_search(1, dummy_search_result_name_size_is_42)
-          stub_request_no_result_with_page(2)
+          stub_request_search(query, 1, dummy_search_result_name_size_is_42)
+          stub_request_search_no_result_with_page(query, 2)
         end
-        let(:query) {'size_is_42_2345678901234567890123456789012'}
+        let(:query) { 'size_is_42_2345678901234567890123456789012' }
         it 'is 50 characters' do
-          capture(:stdout) { @executor.search(query) }.should == <<-'EOS'.unindent
+          res = <<-'EOS'.unindent
             |Searching ..
             |NAME                                                DL(ver)   DL(all)
             |-------------------------------------------------- -------- ---------
             |size_is_42_2345678901234567890123456789012 (0.0.1)      100      1000
           EOS
+          expect { @executor.search(query) }.to output(res).to_stdout
         end
       end
 
       context 'NAME size is 43' do
         before do
-          stub_request_search(1, dummy_search_result_name_size_is_43)
-          stub_request_no_result_with_page(2)
+          stub_request_search(query, 1, dummy_search_result_name_size_is_43)
+          stub_request_search_no_result_with_page(query, 2)
         end
-        let(:query) {'size_is_43_23456789012345678901234567890123'}
+        let(:query) { 'size_is_43_23456789012345678901234567890123' }
         it 'is 51 characters' do
-          capture(:stdout) { @executor.search(query) }.should == <<-'EOS'.unindent
+          res = <<-'EOS'.unindent
             |Searching ..
             |NAME                                                 DL(ver)   DL(all)
             |--------------------------------------------------- -------- ---------
             |size_is_43_23456789012345678901234567890123 (0.0.2)      200      2000
           EOS
+          expect { @executor.search(query) }.to output(res).to_stdout
         end
       end
     end
   end
 
-  private
-    def dummy_search_result_name_size_is_42
-      [{"name"=>"size_is_42_2345678901234567890123456789012",
-        "downloads"=>1000,
-        "version"=>"0.0.1",
-        "version_downloads"=>100}
-      ].to_json
+  describe '#browse' do
+    context 'when a network error occurred' do
+      before do
+        stub_request(:get, build_gems_uri(query))
+          .to_return(status: 500, body: '[]')
+      end
+      let(:query) { 'network_error_orccurred' }
+      it { expect { @executor.browse(query) }.to raise_error(Exception) }
+    end
+
+    context 'when no matching gem' do
+      before { stub_request_gems_no_result(query) }
+      let(:query) { 'no_match_gem_name' }
+      it { expect { @executor.browse(query) }.to raise_error(OpenURI::HTTPError) }
+    end
+
+    context 'when no homepage_uri' do
+      before do
+        url = Executor::GEM_URL % query
+        allow(@executor).to receive(:system).with('open', url)
+        stub_request_gems(query, load_http_stubs("gems/#{query}.json"))
+      end
+      let(:query) { 'git-trend_no_homepage' }
+
+      it 'open a rubygems url' do
+        @executor.browse(query)
+      end
     end
 
-    def dummy_search_result_name_size_is_43
-      [{"name"=>"size_is_43_23456789012345678901234567890123",
-        "downloads"=>2000,
-        "version"=>"0.0.2",
-        "version_downloads"=>200}
-      ].to_json
+    context 'when homepage_uri is existed' do
+      before do
+        http_stub = load_http_stubs("gems/#{query}.json")
+        url = JSON.parse(http_stub)['homepage_uri']
+        allow(@executor).to receive(:system).with('open', url)
+        stub_request_gems(query, http_stub)
+      end
+      let(:query) { 'git-trend' }
+
+      it 'open a homepage url' do
+        @executor.browse(query)
+      end
     end
+  end
+
+  private
+
+  def dummy_search_result_name_size_is_42
+    [{"name"=>"size_is_42_2345678901234567890123456789012",
+      "downloads"=>1000,
+      "version"=>"0.0.1",
+      "version_downloads"=>100}
+    ].to_json
+  end
+
+  def dummy_search_result_name_size_is_43
+    [{"name"=>"size_is_43_23456789012345678901234567890123",
+      "downloads"=>2000,
+      "version"=>"0.0.2",
+      "version_downloads"=>200}
+    ].to_json
+  end
 end

data/spec/http_stubs/gems/git-trend.json

@@ -0,0 +1,66 @@
+{
+  "name": "git-trend",
+  "downloads": 1106,
+  "version": "0.0.8",
+  "version_downloads": 36,
+  "platform": "ruby",
+  "authors": "rochefort",
+  "info": "cli based; show Trending repository on github",
+  "licenses": [
+    "MIT"
+  ],
+  "project_uri": "http://rubygems.org/gems/git-trend",
+  "gem_uri": "http://rubygems.org/gems/git-trend-0.0.8.gem",
+  "homepage_uri": "https://github.com/rochefort/git-trend",
+  "wiki_uri": null,
+  "documentation_uri": null,
+  "mailing_list_uri": null,
+  "source_code_uri": null,
+  "bug_tracker_uri": null,
+  "dependencies": {
+    "development": [
+      {
+        "name": "bundler",
+        "requirements": "~> 1.6"
+      },
+      {
+        "name": "coveralls",
+        "requirements": ">= 0"
+      },
+      {
+        "name": "rake",
+        "requirements": ">= 0"
+      },
+      {
+        "name": "rspec",
+        "requirements": "~> 3.0.0"
+      },
+      {
+        "name": "rspec-its",
+        "requirements": "~> 1.0.1"
+      },
+      {
+        "name": "simplecov",
+        "requirements": "~> 0.9.0"
+      },
+      {
+        "name": "webmock",
+        "requirements": "~> 1.18.0"
+      }
+    ],
+    "runtime": [
+      {
+        "name": "addressable",
+        "requirements": "~> 2.3.6"
+      },
+      {
+        "name": "mechanize",
+        "requirements": "~> 2.7.3"
+      },
+      {
+        "name": "thor",
+        "requirements": "~> 0.19.1"
+      }
+    ]
+  }
+}

data/spec/http_stubs/gems/git-trend_no_homepage.json

@@ -0,0 +1,65 @@
+{
+  "name": "git-trend",
+  "downloads": 1106,
+  "version": "0.0.8",
+  "version_downloads": 36,
+  "platform": "ruby",
+  "authors": "rochefort",
+  "info": "cli based; show Trending repository on github",
+  "licenses": [
+    "MIT"
+  ],
+  "project_uri": "http://rubygems.org/gems/git-trend",
+  "gem_uri": "http://rubygems.org/gems/git-trend-0.0.8.gem",
+  "wiki_uri": null,
+  "documentation_uri": null,
+  "mailing_list_uri": null,
+  "source_code_uri": null,
+  "bug_tracker_uri": null,
+  "dependencies": {
+    "development": [
+      {
+        "name": "bundler",
+        "requirements": "~> 1.6"
+      },
+      {
+        "name": "coveralls",
+        "requirements": ">= 0"
+      },
+      {
+        "name": "rake",
+        "requirements": ">= 0"
+      },
+      {
+        "name": "rspec",
+        "requirements": "~> 3.0.0"
+      },
+      {
+        "name": "rspec-its",
+        "requirements": "~> 1.0.1"
+      },
+      {
+        "name": "simplecov",
+        "requirements": "~> 0.9.0"
+      },
+      {
+        "name": "webmock",
+        "requirements": "~> 1.18.0"
+      }
+    ],
+    "runtime": [
+      {
+        "name": "addressable",
+        "requirements": "~> 2.3.6"
+      },
+      {
+        "name": "mechanize",
+        "requirements": "~> 2.7.3"
+      },
+      {
+        "name": "thor",
+        "requirements": "~> 0.19.1"
+      }
+    ]
+  }
+}

data/spec/spec_helper.rb

@@ -4,37 +4,40 @@ require 'simplecov'
 SimpleCov.start
 
 require 'webmock/rspec'
-require 'gem-search'
+require 'gem_search'
 
-# http://d.hatena.ne.jp/POCHI_BLACK/20100324
-# this method is written by wycats
-def capture(stream)
-  begin
-    stream = stream.to_s
-    eval "$#{stream} = StringIO.new"
-    yield
-    result = eval("$#{stream}").string
-  ensure
-    eval("$#{stream} = #{stream.upcase}")
-  end
-
-  result
+RSpec.configure do |config|
+  config.expose_dsl_globally = false
 end
 
 def load_http_stubs(file_name)
   open(File.join(File.dirname(__FILE__), 'http_stubs', file_name)).read
 end
 
-def stub_request_search(page, body)
-  stub_request(:get, build_search_uri(query, page)).to_return(:status => 200, :body => body)
+# stubs for search API
+def stub_request_search(query, page, body)
+  stub_request(:get, build_search_uri(query, page)).to_return(status: 200, body: body)
 end
 
-def stub_request_no_result_with_page(page)
-  stub_request(:get, build_search_uri(query, page)).to_return(:status => 200, :body => '[]')
+def stub_request_search_no_result_with_page(query, page)
+  stub_request(:get, build_search_uri(query, page)).to_return(status: 200, body: '[]')
 end
 
 def build_search_uri(query, page)
-  Executor::SEARCH_URL % [query, page]
+  Executor::SEARCH_API % [query, page]
+end
+
+# stubs for gems API
+def stub_request_gems(query, body)
+  stub_request(:get, build_gems_uri(query)).to_return(status: 200, body: body)
+end
+
+def stub_request_gems_no_result(query)
+  stub_request(:get, build_gems_uri(query)).to_return(status: 404, body: '[]')
+end
+
+def build_gems_uri(query)
+  Executor::GEM_API % query
 end
 
 def dummy_search_result

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: gem-search
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.1.2
 platform: ruby
 authors:
 - rochefort
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-06-05 00:00:00.000000000 Z
+date: 2014-08-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: slop
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 3.5.0
+        version: 3.6.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 3.5.0
+        version: 3.6.0
 - !ruby/object:Gem::Dependency
   name: json
   requirement: !ruby/object:Gem::Requirement
@@ -44,14 +44,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.17.4
+        version: 1.18.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.17.4
+        version: 1.18.0
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -72,28 +72,28 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.14.1
+        version: 3.0.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.14.1
+        version: 3.0.0
 - !ruby/object:Gem::Dependency
   name: simplecov
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.8.2
+        version: 0.9.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.8.2
+        version: 0.9.0
 description: search gems with using rubygems.org API
 email:
 - terasawan@gmail.com
@@ -111,17 +111,19 @@ files:
 - Rakefile
 - bin/gem-search
 - gem-search.gemspec
-- lib/gem-search.rb
-- lib/gem-search/command.rb
-- lib/gem-search/executor.rb
-- lib/gem-search/rendering.rb
-- lib/gem-search/version.rb
-- spec/bin/gem-search_spec.rb
+- lib/gem_search.rb
+- lib/gem_search/command.rb
+- lib/gem_search/executor.rb
+- lib/gem_search/rendering.rb
+- lib/gem_search/version.rb
+- spec/bin/gem_search_spec.rb
 - spec/command_spec.rb
 - spec/executor_spec.rb
-- spec/http_stubs/cucumber-_1.json
-- spec/http_stubs/cucumber-_2.json
-- spec/http_stubs/cucumber-_3.json
+- spec/http_stubs/gems/git-trend.json
+- spec/http_stubs/gems/git-trend_no_homepage.json
+- spec/http_stubs/search/cucumber-_1.json
+- spec/http_stubs/search/cucumber-_2.json
+- spec/http_stubs/search/cucumber-_3.json
 - spec/spec_helper.rb
 homepage: https://github.com/rochefort/gem-search
 licenses: []
@@ -147,10 +149,12 @@ signing_key:
 specification_version: 4
 summary: search gems with using rubygems.org API
 test_files:
-- spec/bin/gem-search_spec.rb
+- spec/bin/gem_search_spec.rb
 - spec/command_spec.rb
 - spec/executor_spec.rb
-- spec/http_stubs/cucumber-_1.json
-- spec/http_stubs/cucumber-_2.json
-- spec/http_stubs/cucumber-_3.json
+- spec/http_stubs/gems/git-trend.json
+- spec/http_stubs/gems/git-trend_no_homepage.json
+- spec/http_stubs/search/cucumber-_1.json
+- spec/http_stubs/search/cucumber-_2.json
+- spec/http_stubs/search/cucumber-_3.json
 - spec/spec_helper.rb

data/lib/gem-search.rb

@@ -1,9 +0,0 @@
-module Gem
-  module Search
-    class LibraryNotFound    < StandardError; end
-    autoload :Command ,  'gem-search/command'
-    autoload :Executor , 'gem-search/executor'
-    autoload :Rendering, 'gem-search/rendering'
-    autoload :VERSION  , 'gem-search/version'
-  end
-end

data/lib/gem-search/executor.rb

@@ -1,52 +0,0 @@
-require 'json'
-require 'open-uri'
-
-module Gem::Search
-  class Executor
-    include Gem::Search::Rendering
-    BASE_URL   = 'https://rubygems.org'
-    SEARCH_URL = "#{BASE_URL}/api/v1/search.json?query=%s&page=%d"
-    MAX_REQUEST_COUNT = 20
-
-    def search(query, opt_sort='downloads')
-      return unless query
-      print 'Searching '
-      gems = []
-      (1..MAX_REQUEST_COUNT).each do |n|
-        print '.'
-        url = SEARCH_URL % [query, n]
-        gems_by_page = search_rubygems(url)
-        break if gems_by_page.size.zero?
-        gems += gems_by_page
-      end
-      puts
-
-      raise Gem::Search::LibraryNotFound, 'We did not find results.' if gems.size.zero?
-      gems_sort!(gems, opt_sort)
-      Executor.render(gems)
-    end
-
-    private
-      def search_rubygems(url)
-        option = {}
-        proxy = URI.parse(url).find_proxy
-        if proxy
-          if proxy.user && proxy.password
-            option[:proxy_http_basic_authentication] = [proxy, proxy.user, proxy.password]
-          else
-            option[:proxy] = proxy
-          end
-        end
-        JSON.parse(open(url, option).read)
-      end
-
-      def gems_sort!(gems, opt_sort)
-        if opt_sort == 'name'
-          gems.sort!{ |x,y| x[opt_sort] <=> y[opt_sort] }
-        else
-          gems.sort!{ |x,y| y[opt_sort] <=> x[opt_sort] }
-        end
-      end
-
-  end
-end

data/lib/gem-search/rendering.rb

@@ -1,42 +0,0 @@
-module Gem::Search
-  module Rendering
-    DEFAULT_RULED_LINE_SIZE = [50, 8, 9]
-
-    def self.included(base)
-      base.extend(self)
-    end
-
-    def render(gems)
-      set_ruled_line_size(gems)
-      render_to_header
-      render_to_body(gems)
-    end
-
-    private
-      def set_ruled_line_size(gems)
-        max_name_size = gems.map { |gem| "#{gem['name'] } (#{gem['version']})".size }.max
-        if max_name_size > DEFAULT_RULED_LINE_SIZE[0]
-          DEFAULT_RULED_LINE_SIZE[0] = max_name_size
-        end
-      end
-
-      def render_to_header
-        f=DEFAULT_RULED_LINE_SIZE
-        fmt = "%-#{f[0]}s %#{f[1]}s %#{f[2]}s"
-        puts fmt % ['NAME', 'DL(ver)', 'DL(all)']
-        puts fmt % ['-'*f[0], '-'*f[1], '-'*f[2]]
-      end
-
-      def render_to_body(gems)
-        f=DEFAULT_RULED_LINE_SIZE
-        fmt = "%-#{f[0]}s %#{f[1]}d %#{f[2]}d"
-        gems.each do |gem|
-          puts fmt % [
-            "#{gem['name']} (#{gem['version']})",
-            gem['version_downloads'],
-            gem['downloads']
-          ]
-        end
-      end
-  end
-end