firebrew 0.2.0 → 0.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: cfb18b86ed200214495e26c35083760ee06adbfa
-  data.tar.gz: 5d06456d51c4d55ac9cc11355a38b6fe0de60599
+  metadata.gz: 1d7c07821b8298bda0079dd30bdde6dc71cb3f5c
+  data.tar.gz: 1b9f7daf7f567a3748d28b139427f1cbae6a1af7
 SHA512:
-  metadata.gz: b466f350381f97a8ae2e178d3d0a08fa3379ccff2524dfec7f72b14d8945fc33333a787ee839c137b8b6d536f659fff2c452137b1661b3fb4e177085319f0e40
-  data.tar.gz: 54367c7cb733c9a08466ef91a6ef2142783fb7924d0a7e8336a7afea1153e8d5d95915df33b1e7ed0a2fcd87c14b71dae4fe720a1c3c440efe6cb46ae9970e78
+  metadata.gz: 9d81f90568ef3ac06350cf0c82f68f3798a8b49571955f25da2eb221c6ca83bc3bccba1b2377ecc9bb29dcf6e9d06e77bb19a5e03c0b486553e23d86d17579d4
+  data.tar.gz: b78494beacbcb8dd6154964a600fced0b866acb39b217baed1d1c7f92823517d93a151baf06930e3ce66f4f9958282e31a9c9691bc0fdf9ad940280a72e8c703

data/CHANGELOG.md

@@ -1,5 +1,10 @@
 # Changelog
 
+## [0.3.0](https://github.com/mrk21/firebrew/tree/v0.3.0) - 2014-09-30
+
+* Future [#36](https://github.com/mrk21/firebrew/issues/36): Implement the `os` gem
+* Future [#18](https://github.com/mrk21/firebrew/issues/18): Make it possible to display a download progress of the extension
+
 ## [0.2.0](https://github.com/mrk21/firebrew/tree/v0.2.0) - 2014-08-31
 
 * Future [#35](https://github.com/mrk21/firebrew/issues/35): Add the subcommand help

data/firebrew.gemspec

@@ -21,8 +21,11 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency "bundler", "~> 1.5"
   spec.add_development_dependency "rake"
   spec.add_development_dependency "rspec", "~> 3.0"
+  spec.add_development_dependency "webmock", "~> 1.19"
   
   spec.add_dependency "inifile", "~> 2.0"
   spec.add_dependency "rubyzip", "~> 1.1"
   spec.add_dependency "faraday", "~> 0.9"
+  spec.add_dependency "os", "~> 0.9"
+  spec.add_dependency "ruby-progressbar", "~> 1.6"
 end

data/lib/firebrew/command_line.rb

@@ -151,7 +151,7 @@ module Firebrew
     end
     
     def execute
-      runner = Runner.new(self.arguments[:config])
+      runner = Runner.new(self.arguments[:config], true)
       
       case self.arguments[:command]
       when :search, :list then

data/lib/firebrew/downloader.rb

@@ -0,0 +1,106 @@
+require 'uri'
+require 'net/http'
+require 'ruby-progressbar'
+
+module Firebrew
+  class Downloader
+    def self.normalize_uri(uri)
+      begin
+        uri = URI.parse(uri)
+      rescue URI::InvalidURIError
+        uri = URI.parse(URI.encode uri)
+      end
+      
+      uri.normalize!
+      
+      case uri.scheme
+      when 'http','https','file' then
+        # do nothing
+      when nil then
+        uri.scheme = 'file'
+        path = File.expand_path(uri.path)
+        path = "/#{path}" if path =~ /^[a-zA-Z]:/
+        uri.path = path
+      when /^[a-zA-Z]$/ then
+        uri.path = "/#{uri.scheme.upcase}:#{uri.path}"
+        uri.scheme = 'file'
+      else
+        raise Firebrew::NetworkError, "Don't support the scheme: #{uri.scheme}"
+      end
+      
+      uri
+    end
+    
+    def initialize(uri, out, progress_options={})
+      @uri = self.class.normalize_uri(uri)
+      @out = out
+      @progress_options = progress_options
+    end
+    
+    def exec
+      case @uri.scheme
+      when 'http','https' then
+        @size = loop do
+          response = self.http_connection do |http, path|
+            http.head(path)
+          end
+          if response.code.to_i == 302 then
+            @uri = self.class.normalize_uri(response['Location'])
+            next
+          end
+          break response['Content-Length'].to_i
+        end
+        
+        progress_bar = self.create_progress_bar
+        
+        self.http_connection do |http, path|
+          http.get(path) do |chunk|
+            progress_bar.progress += chunk.size
+            @out.write chunk
+          end
+        end
+        
+      when 'file' then
+        @size = self.file_connection do |file|
+          file.size
+        end
+        
+        progress_bar = self.create_progress_bar
+        
+        self.file_connection do |file|
+          loop do
+            chunk = file.read(1000)
+            break if chunk.nil?
+            progress_bar.progress += chunk.size
+            @out.write chunk
+          end
+        end
+      end
+    end
+    
+    protected
+    
+    def http_connection(&block)
+      Net::HTTP.start(@uri.host, @uri.port, use_ssl: @uri.scheme == 'https') do |http|
+        block[http, @uri.request_uri]
+      end
+    end
+    
+    def file_connection(&block)
+      path = @uri.path.gsub(%r{/([a-zA-Z]:)},'\1')
+      open(URI.decode(path), &block)
+    end
+    
+    def create_progress_bar
+      options = {
+        format: "%e |%B| [%j%%]",
+        progress_mark: '=',
+        remainder_mark: '･',
+      }
+      options.merge! @progress_options
+      options.merge! total: @size
+      options[:output] = File.open(File::NULL,'w') unless options[:output]
+      ProgressBar.create(options)
+    end
+  end
+end

data/lib/firebrew/firefox/command.rb

@@ -1,4 +1,5 @@
 require 'shellwords'
+require 'os'
 
 module Firebrew::Firefox
   class Command
@@ -12,10 +13,10 @@ module Firebrew::Firefox
       @config = config
       @executer = executer
       begin
-        result = if ENV['OS'].nil? then
-          @executer.exec('%{firefox} --version' % self.escaped_config)
-        else
+        result = if OS.windows? then
           @executer.exec('"%{firefox}" --version' % @config)
+        else
+          @executer.exec('%{firefox} --version' % self.escaped_config)
         end
         raise Firebrew::FirefoxCommandError, 'Command is not Firefox: %{firefox}' % @config unless result[0] =~ /Mozilla Firefox/
         raise Firebrew::FirefoxCommandError, 'Command is not Firefox: %{firefox}' % @config unless result[1] == 0
@@ -26,10 +27,10 @@ module Firebrew::Firefox
     
     def version
       return @version unless @version.nil?
-      result = if ENV['OS'].nil? then
-        @executer.exec('%{firefox} --version' % self.escaped_config)[0]
-      else
+      result = if OS.windows? then
         @executer.exec('"%{firefox}" --version' % @config)[0]
+      else
+        @executer.exec('%{firefox} --version' % self.escaped_config)[0]
       end
       @version = result.match(/[0-9.]+/)[0]
     end

data/lib/firebrew/firefox/extension.rb

@@ -2,8 +2,10 @@ require 'fileutils'
 require 'open-uri'
 require 'json'
 require 'rexml/document'
+require 'rake/pathmap'
 require 'zip'
 require 'firebrew/firefox/basic_extension'
+require 'firebrew/downloader'
 
 module Firebrew::Firefox
   class Extension < BasicExtension
@@ -39,35 +41,42 @@ module Firebrew::Firefox
         result
       end
       
-      def install(extension)
+      def install(extension, is_displaying_progress = false)
         dir = File.join(self.profile.path, 'extensions')
         FileUtils.mkdir_p dir
+        xpi_path = '%s.xpi' % File.join(dir, extension.guid)
         
-        open([extension.uri].flatten.first, 'rb') do |r|
-          xpi = Zip::File.open(r)
-          install_manifests = xpi.find_entry('install.rdf')
-          install_manifests = install_manifests.get_input_stream.read
-          install_manifests = REXML::Document.new(install_manifests)
-          is_unpacking = REXML::XPath.match(install_manifests, '/RDF/Description/em:unpack/text()').first
-          is_unpacking = is_unpacking.nil? ? false : is_unpacking.value.strip == 'true'
+        File.open(xpi_path,'wb') do |out|
+          uri = [extension.uri].flatten.first
+          pout = is_displaying_progress ? $stdout : nil
+          Firebrew::Downloader.new(uri, out, output: pout).exec
+        end
+        
+        xpi = Zip::File.open(File.open(xpi_path,'rb'))
+        install_manifests = xpi.find_entry('install.rdf')
+        install_manifests = install_manifests.get_input_stream.read
+        install_manifests = REXML::Document.new(install_manifests)
+        is_unpacking = REXML::XPath.match(install_manifests, '/RDF/Description/em:unpack/text()').first
+        is_unpacking = is_unpacking.nil? ? false : is_unpacking.value.strip == 'true'
+        
+        if is_unpacking then
+          extension.uri = xpi_path.pathmap('%X')
+          FileUtils.mkdir_p(extension.uri)
           
-          if is_unpacking then
-            extension.uri = File.join(dir, extension.guid)
-            FileUtils.mkdir_p(extension.uri)
-            xpi.each do |entry|
-              next if entry.ftype == :directory
-              content = entry.get_input_stream.read
-              Dir.chdir(extension.uri) do
-                FileUtils.mkdir_p File.dirname(entry.name)
-                File.write(entry.name, content)
-              end
-            end
-          else
-            extension.uri = '%s.xpi' % File.join(dir, extension.guid)
-            open(extension.uri, 'wb') do |w|
-              w.write r.read
+          xpi.each do |entry|
+            next if entry.ftype == :directory
+            content = entry.get_input_stream.read
+            Dir.chdir(extension.uri) do
+              FileUtils.mkdir_p File.dirname(entry.name)
+              File.write(entry.name, content)
             end
           end
+          
+          xpi.close
+          FileUtils.rm_rf(xpi_path)
+        else
+          extension.uri = xpi_path
+          xpi.close
         end
         
         self.add(extension)
@@ -75,7 +84,7 @@ module Firebrew::Firefox
       end
       
       def uninstall(extension)
-        FileUtils.rm_rf extension.uri
+        FileUtils.rm_r extension.uri
         self.remove(extension)
         self.push
       end

data/lib/firebrew/runner.rb

@@ -7,25 +7,24 @@ module Firebrew
   class Runner
     attr_accessor :config, :profile
     
-    def self.default_config(platform = RUBY_PLATFORM)
+    def self.default_config
       result = {
         base_dir: ENV['FIREBREW_FIREFOX_PROFILE_BASE_DIR'],
         firefox: ENV['FIREBREW_FIREFOX'],
         profile: ENV['FIREBREW_FIREFOX_PROFILE'] || 'default',
       }
       
-      case platform
-      when /darwin/ then
+      if OS.mac? then
         result[:base_dir] ||= '~/Library/Application Support/Firefox'
         result[:firefox] ||= '/Applications/Firefox.app/Contents/MacOS/firefox-bin'
         result[:os] = 'darwin'
         
-      when /linux/ then
+      elsif OS.linux? then
         result[:base_dir] ||= '~/.mozilla/firefox'
         result[:firefox] ||= '/usr/bin/firefox'
         result[:os] = 'linux'
         
-      when /mswin(?!ce)|mingw|cygwin|bccwin/ then
+      elsif OS.windows? then
         appdata = ENV['APPDATA'].to_s.gsub('\\','/')
         programfiles = (ENV['PROGRAMFILES(X86)'] || ENV['PROGRAMFILES']).to_s.gsub('\\','/')
         result[:base_dir] ||= File.join(appdata, 'Mozilla/Firefox')
@@ -36,8 +35,9 @@ module Firebrew
       result
     end
     
-    def initialize(config={})
+    def initialize(config={}, is_displaying_progress = false)
       self.config = self.class.default_config.merge(config)
+      @is_displaying_progress = is_displaying_progress
       
       @profile_manager = Firefox::Profile::Manager.new(
         base_dir: self.config[:base_dir],
@@ -56,7 +56,7 @@ module Firebrew
       extension = self.profile.extensions.find(params[:term])
       raise Firebrew::OperationAlreadyCompletedError, "Already installed: #{params[:term]}" unless extension.nil?
       result = self.fetch_api(term: params[:term], max: 1).first
-      self.profile.extensions.install(result)
+      self.profile.extensions.install(result, @is_displaying_progress)
     end
     
     def uninstall(params={})

data/lib/firebrew/version.rb

@@ -1,3 +1,3 @@
 module Firebrew
-  VERSION = "0.2.0"
+  VERSION = "0.3.0"
 end

data/spec/firebrew/downloader_spec.rb

@@ -0,0 +1,150 @@
+require 'spec_helper'
+require 'stringio'
+require 'digest/md5'
+require 'webmock/rspec'
+require 'os'
+
+module Firebrew
+  describe Downloader do
+    describe '::normalize_uri(uri)' do
+      subject do
+        Downloader.normalize_uri(self.uri)
+      end
+      
+      context 'when the uri contained spaces and unicode characters etc' do
+        let(:uri){'http://www.example.com/a b/あ.html'}
+        let(:expected_uri){URI.parse('http://www.example.com/a%20b/%E3%81%82.html')}
+        
+        it 'should be encoded by percent-encoding' do
+          is_expected.to eq(self.expected_uri)
+        end
+      end
+      
+      context 'when the uri path was empty' do
+        let(:uri){'http://www.example.com'}
+        it 'path should be "/"' do
+          expect(subject.path).to eq('/')
+        end
+      end
+      
+      context 'when the uri scheme was "http"' do
+        let(:uri){'http://www.example.com/a/b.html'}
+        let(:expected_uri){URI.parse('http://www.example.com/a/b.html')}
+        it { is_expected.to eq(self.expected_uri) }
+      end
+      
+      context 'when the uri scheme was "https"' do
+        let(:uri){'https://www.example.com/a/b.html'}
+        let(:expected_uri){URI.parse('https://www.example.com/a/b.html')}
+        it { is_expected.to eq(self.expected_uri) }
+      end
+      
+      context 'when the uri scheme was "file"' do
+        if OS.windows? then
+          let(:uri){'file:///C:/a/b.html'}
+          let(:expected_uri){URI.parse('file:///C:/a/b.html')}
+        else
+          let(:uri){'file:///a/b.html'}
+          let(:expected_uri){URI.parse('file:///a/b.html')}
+        end
+        it { is_expected.to eq(self.expected_uri) }
+      end
+      
+      context 'when the uri scheme was empty' do
+        if OS.windows? then
+          let(:uri){'C:/a/b.html'}
+          let(:expected_uri){URI.parse('file:///C:/a/b.html')}
+        else
+          let(:uri){'/a/b.html'}
+          let(:expected_uri){URI.parse('file:///a/b.html')}
+        end
+        
+        it 'should convert a file scheme' do
+          is_expected.to eq(self.expected_uri)
+        end
+        
+        context 'when the path was relative' do
+          let(:uri){'./a/b.html'}
+          
+          if OS.windows? then
+            let(:expected_uri){URI.parse("file:///#{Dir.pwd}/a/b.html")}
+          else
+            let(:expected_uri){URI.parse("file://#{Dir.pwd}/a/b.html")}
+          end
+          
+          it 'should be an absolute path from the current dir' do
+            is_expected.to eq(self.expected_uri)
+          end
+        end
+      end
+      
+      context 'when the uri scheme was not "http", "https" and "file"' do
+        let(:uri){'hoge:///a/b/c'}
+        
+        it 'should throw Firebrew::NetworkError' do
+          expect{subject}.to raise_error(Firebrew::NetworkError, "Don't support the scheme: hoge")
+        end
+      end
+    end
+    
+    describe '#initialize(uri, out[, progress_options]) -> #exec()' do
+      before do
+        self.before_callback[]
+        Downloader.new(self.uri, self.out).exec
+        self.out.rewind
+      end
+      
+      let(:before_callback){->{}}
+      let(:uri){''}
+      let(:out){StringIO.new}
+      
+      context 'when the uri scheme was "file"' do
+        let(:uri){'./spec/fixtures/firefox/extension/unpack_false.xpi'}
+        
+        it 'should download a content which is referenced by the uri' do
+          md5, path = File.read(self.uri.pathmap('%X.md5')).split(/\s+/)
+          expect(Digest::MD5.hexdigest(self.out.read)).to eq(md5)
+        end
+      end
+      
+      context 'when the uri scheme was "http" or "https"' do
+        let(:before_callback){->{
+          response = File.open(self.response_path, 'rb').read
+          stub_request(:head, 'www.example.com').to_return(headers: {
+            'Content-Length'=> response.size
+          })
+          stub_request(:get, 'www.example.com').to_return(body: response)
+        }}
+        
+        let(:uri){'http://www.example.com'}
+        let(:response_path){'./spec/fixtures/firefox/extension/unpack_false.xpi'}
+        
+        it 'should download a content which is referenced by the uri' do
+          md5, path = File.read(self.response_path.pathmap('%X.md5')).split(/\s+/)
+          expect(Digest::MD5.hexdigest(self.out.read)).to eq(md5)
+        end
+        
+        context 'when the response status code was 302' do
+          let(:before_callback){->{
+            response = File.open(self.response_path, 'rb').read
+            stub_request(:head, 'www.example.com').to_return(status: 302, headers: {
+              'Location'=> 'http://dl.example.com/'
+            })
+            stub_request(:head, 'dl.example.com').to_return(headers: {
+              'Content-Length'=> response.size
+            })
+            stub_request(:get, 'dl.example.com').to_return(body: response)
+          }}
+          
+          let(:uri){'http://www.example.com'}
+          let(:response_path){'./spec/fixtures/firefox/extension/unpack_false.xpi'}
+          
+          it 'should download a content which is referenced by the uri' do
+            md5, path = File.read(self.response_path.pathmap('%X.md5')).split(/\s+/)
+            expect(Digest::MD5.hexdigest(self.out.read)).to eq(md5)
+          end
+        end
+      end
+    end
+  end
+end

data/spec/firebrew/firefox/extension_spec.rb

@@ -7,7 +7,7 @@ module Firebrew::Firefox
     let(:manager) do
       Extension::Manager.new(
         profile: Profile.new(
-          path: File.join(Dir.pwd, 'tmp')
+          path: './tmp'
         )
       )
     end
@@ -17,13 +17,13 @@ module Firebrew::Firefox
         name: 'Japanese Language Pack',
         guid: 'langpack-ja@firefox.mozilla.org',
         version: '30.0',
-        uri: File.join(Dir.pwd, 'tmp/extensions/langpack-ja@firefox.mozilla.org.xpi')
+        uri: './tmp/extensions/langpack-ja@firefox.mozilla.org.xpi'
       }, self.manager),
       Extension.new({
         name: 'Vimperator',
         guid: 'vimperator@mozdev.org',
         version: '3.8.2',
-        uri: File.join(Dir.pwd, 'tmp/extensions/vimperator@mozdev.org.xpi')
+        uri: './tmp/extensions/vimperator@mozdev.org.xpi'
       }, self.manager)
     ]}
     
@@ -88,14 +88,15 @@ module Firebrew::Firefox
         end
       end
       
-      describe '#install(extension)' do
+      describe '#install(extension[, is_displaying_progress])' do
         subject { self.instance.install(self.extension) }
         
         let(:extension) do
-          Extension.new({
-            guid: 'hoge@example.org',
-            uri: File.join(Dir.pwd, 'spec/fixtures/firefox/extension/unpack_false.xpi')
-          }, self.manager)
+          Extension.new({guid: 'hoge@example.org', uri: self.extension_uri}, self.manager)
+        end
+        
+        let(:extension_uri) do
+          './spec/fixtures/firefox/extension/unpack_false.xpi'
         end
         
         let(:expected_path) do
@@ -104,14 +105,15 @@ module Firebrew::Firefox
         
         it 'should copy the `path/to/profile/extensions/<GUID>.xpi`' do
           subject
-          expect(File.exists? self.expected_path).to be_truthy
+          md5, path = File.read(self.extension_uri.pathmap('%X.md5')).split(/\s+/)
+          expect(Digest::MD5.hexdigest(File.open(self.expected_path,'rb').read)).to eq(md5)
         end
         
         it 'should add the `extension` extension' do
           subject
           all = self.instance.all
           extension = self.extension
-          extension.uri = File.join(Dir.pwd, 'tmp/extensions/hoge@example.org.xpi')
+          extension.uri = './tmp/extensions/hoge@example.org.xpi'
           expect(all.size).to eq(3)
           expect(all[0]).to eq(self.extensions[0])
           expect(all[1]).to eq(self.extensions[1])
@@ -120,13 +122,14 @@ module Firebrew::Firefox
         
         context 'when an `uri` of the `extension` was equal or greater than two' do
           let(:extension) do
-            Extension.new({
-              guid: 'hoge@example.org',
-              uri: [
-                File.join(Dir.pwd, 'spec/fixtures/firefox/extension/unpack_false.xpi'),
-                File.join(Dir.pwd, 'spec/fixtures/firefox/extension/not_exists.xpi'),
-              ]
-            }, self.manager)
+            Extension.new({guid: 'hoge@example.org', uri: self.extension_uri}, self.manager)
+          end
+          
+          let(:extension_uri) do
+            [
+              './spec/fixtures/firefox/extension/unpack_false.xpi',
+              './spec/fixtures/firefox/extension/not_exists.xpi'
+            ]
           end
           
           it 'should not throw exceptions' do
@@ -136,22 +139,24 @@ module Firebrew::Firefox
         
         context 'when an `em:unpack` value of the `install.rdf` in the XPI package not exsisted' do
           let(:extension) do
-            Extension.new({
-              guid: 'hoge@example.org',
-              uri: File.join(Dir.pwd, 'spec/fixtures/firefox/extension/unpack_null.xpi')
-            }, self.manager)
+            Extension.new({guid: 'hoge@example.org', uri: self.extension_uri}, self.manager)
+          end
+          
+          let(:extension_uri) do
+            './spec/fixtures/firefox/extension/unpack_null.xpi'
           end
           
           it 'should copy the `path/to/profile/extensions/<GUID>.xpi`' do
             subject
-            expect(File.exists? self.expected_path).to be_truthy
+            md5, path = File.read(self.extension_uri.pathmap('%X.md5')).split(/\s+/)
+            expect(Digest::MD5.hexdigest(File.open(self.expected_path,'rb').read)).to eq(md5)
           end
           
           it 'should add the `extension` extension' do
             subject
             all = self.instance.all
             extension = self.extension
-            extension.uri = File.join(Dir.pwd, 'tmp/extensions/hoge@example.org.xpi')
+            extension.uri = './tmp/extensions/hoge@example.org.xpi'
             expect(all.size).to eq(3)
             expect(all[0]).to eq(self.extensions[0])
             expect(all[1]).to eq(self.extensions[1])
@@ -161,10 +166,11 @@ module Firebrew::Firefox
         
         context 'when an `em:unpack` value of the `install.rdf` in the XPI package was true' do
           let(:extension) do
-            Extension.new({
-              guid: 'hoge@example.org',
-              uri: './spec/fixtures/firefox/extension/unpack_true.xpi'
-            }, self.manager)
+            Extension.new({guid: 'hoge@example.org', uri: self.extension_uri}, self.manager)
+          end
+          
+          let(:extension_uri) do
+            './spec/fixtures/firefox/extension/unpack_true.xpi'
           end
           
           let(:expected_path) do
@@ -178,7 +184,7 @@ module Firebrew::Firefox
           
           it 'should unzip all files in the XPI package' do
             subject
-            md5list = File.read('./spec/fixtures/firefox/extension/unpack_true.md5')
+            md5list = File.read(self.extension_uri.pathmap('%X.md5'))
             Dir.chdir(self.expected_path) do
               md5list.each_line do |item|
                 md5, path = item.split(/\s+/)
@@ -191,7 +197,7 @@ module Firebrew::Firefox
             subject
             all = self.instance.all
             extension = self.extension
-            extension.uri = File.join(Dir.pwd, 'tmp/extensions/hoge@example.org')
+            extension.uri = './tmp/extensions/hoge@example.org'
             expect(all.size).to eq(3)
             expect(all[0]).to eq(self.extensions[0])
             expect(all[1]).to eq(self.extensions[1])
@@ -204,7 +210,7 @@ module Firebrew::Firefox
         let(:extension) do
           Extension.new({
             guid: 'hoge@example.org',
-            uri: File.join(Dir.pwd, 'tmp/extensions/hoge@example.org.xpi')
+            uri: './tmp/extensions/hoge@example.org.xpi'
           }, self.manager)
         end
         
@@ -234,7 +240,7 @@ module Firebrew::Firefox
           let(:extension) do
             Extension.new({
               guid: 'hoge@example.org',
-              uri: File.join(Dir.pwd, 'tmp/extensions/hoge@example.org')
+              uri: './tmp/extensions/hoge@example.org'
             }, self.manager)
           end
           

data/spec/firebrew/runner_spec.rb

@@ -1,14 +1,13 @@
 require 'spec_helper'
+require 'os'
 
 module Firebrew
   describe Firebrew::Runner do
-    describe '::default_config(platform)' do
+    describe '::default_config' do
       subject do
-        Runner.default_config(self.platform)
+        Runner.default_config
       end
       
-      let(:platform){'x86_64-darwin13.0'}
-      
       before do
         ENV['FIREBREW_FIREFOX_PROFILE_BASE_DIR'] = nil
         ENV['FIREBREW_FIREFOX_PROFILE'] = nil
@@ -21,8 +20,7 @@ module Firebrew
         ENV['FIREBREW_FIREFOX'] = nil
       end
       
-      context 'when the `platform` was "MacOS"' do
-        let(:platform){'x86_64-darwin13.0'}
+      if OS.mac? then
         it do
           is_expected.to eq(
             base_dir: '~/Library/Application Support/Firefox',
@@ -31,10 +29,8 @@ module Firebrew
             os: 'darwin'
           )
         end
-      end
-      
-      context 'when the `platform` was "Linux"' do
-        let(:platform){'x86_64-linux'}
+        
+      elsif OS.linux? then
         it do
           is_expected.to eq(
             base_dir: '~/.mozilla/firefox',
@@ -43,11 +39,8 @@ module Firebrew
             os: 'linux'
           )
         end
-      end
-      
-      context 'when the `platform` was "Windows"' do
-        let(:platform){'x64-mingw32'}
         
+      elsif OS.windows? then
         before do
           ENV['APPDATA'] = nil
           ENV['PROGRAMFILES'] = nil
@@ -123,13 +116,10 @@ module Firebrew
           ENV['FIREBREW_FIREFOX'] = 'path/to/firefox'
         end
         
-        it do
-          is_expected.to eq(
-            base_dir: 'path/to/profile_base_directory',
-            firefox: 'path/to/firefox',
-            profile: 'profile-name',
-            os: 'darwin'
-          )
+        it 'should set the default_config' do
+          expect(subject[:base_dir]).to eq('path/to/profile_base_directory')
+          expect(subject[:firefox]).to eq('path/to/firefox')
+          expect(subject[:profile]).to eq('profile-name')
         end
       end
     end
@@ -141,7 +131,7 @@ module Firebrew
         Runner.new(
           base_dir: './tmp',
           data_file: 'profiles.ini',
-          firefox: ENV['OS'].nil? ? './spec/double/firefox.rb' : './spec/double/firefox.bat'
+          firefox: OS.windows? ? './spec/double/firefox.bat' : './spec/double/firefox.rb'
         )
       end
       

data/spec/fixtures/firefox/extension/unpack_false.md5

@@ -0,0 +1 @@
+dea866691da858b819c5850107b40e54  unpack_false.xpi

data/spec/fixtures/firefox/extension/unpack_null.md5

@@ -0,0 +1 @@
+d4546ed68e68f494cba0b0c4fa67ea8b  unpack_null.xpi

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: firebrew
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.3.0
 platform: ruby
 authors:
 - Yuichi Murata
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-08-30 00:00:00.000000000 Z
+date: 2014-09-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -52,6 +52,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '3.0'
+- !ruby/object:Gem::Dependency
+  name: webmock
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.19'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.19'
 - !ruby/object:Gem::Dependency
   name: inifile
   requirement: !ruby/object:Gem::Requirement
@@ -94,6 +108,34 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '0.9'
+- !ruby/object:Gem::Dependency
+  name: os
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.9'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.9'
+- !ruby/object:Gem::Dependency
+  name: ruby-progressbar
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.6'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.6'
 description: Firefox add-ons manager for CUI.
 email:
 - mrk21info+rubygems@gmail.com
@@ -115,6 +157,7 @@ files:
 - lib/firebrew.rb
 - lib/firebrew/amo_api/search.rb
 - lib/firebrew/command_line.rb
+- lib/firebrew/downloader.rb
 - lib/firebrew/entity.rb
 - lib/firebrew/firefox/basic_extension.rb
 - lib/firebrew/firefox/command.rb
@@ -126,6 +169,7 @@ files:
 - spec/double/firefox.rb
 - spec/firebrew/amo_api/search_spec.rb
 - spec/firebrew/command_line_spec.rb
+- spec/firebrew/downloader_spec.rb
 - spec/firebrew/entity_spec.rb
 - spec/firebrew/firefox/command_spec.rb
 - spec/firebrew/firefox/extension_spec.rb
@@ -137,7 +181,9 @@ files:
 - spec/fixtures/amo_api/search/single.xml
 - spec/fixtures/firefox/extension/extensions.v16.json
 - spec/fixtures/firefox/extension/extensions_added_hoge.v16.json
+- spec/fixtures/firefox/extension/unpack_false.md5
 - spec/fixtures/firefox/extension/unpack_false.xpi
+- spec/fixtures/firefox/extension/unpack_null.md5
 - spec/fixtures/firefox/extension/unpack_null.xpi
 - spec/fixtures/firefox/extension/unpack_true.md5
 - spec/fixtures/firefox/extension/unpack_true.xpi
@@ -173,6 +219,7 @@ test_files:
 - spec/double/firefox.rb
 - spec/firebrew/amo_api/search_spec.rb
 - spec/firebrew/command_line_spec.rb
+- spec/firebrew/downloader_spec.rb
 - spec/firebrew/entity_spec.rb
 - spec/firebrew/firefox/command_spec.rb
 - spec/firebrew/firefox/extension_spec.rb
@@ -184,7 +231,9 @@ test_files:
 - spec/fixtures/amo_api/search/single.xml
 - spec/fixtures/firefox/extension/extensions.v16.json
 - spec/fixtures/firefox/extension/extensions_added_hoge.v16.json
+- spec/fixtures/firefox/extension/unpack_false.md5
 - spec/fixtures/firefox/extension/unpack_false.xpi
+- spec/fixtures/firefox/extension/unpack_null.md5
 - spec/fixtures/firefox/extension/unpack_null.xpi
 - spec/fixtures/firefox/extension/unpack_true.md5
 - spec/fixtures/firefox/extension/unpack_true.xpi