ero_getter 0.1.2 → 0.1.3

data/VERSION

@@ -1 +1 @@
-0.1.2
+0.1.3

data/ero_getter.gemspec

@@ -5,7 +5,7 @@
 
 Gem::Specification.new do |s|
   s.name = "ero_getter"
-  s.version = "0.1.2"
+  s.version = "0.1.3"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["masarakki"]

data/lib/ero_getter.rb

@@ -14,7 +14,12 @@ class EroGetter
     klazz.new(url).run
   end
 
+
   class << self
+    def directory
+      File.join ENV['HOME'], 'ero_getter'
+    end
+
     def url_mapping
       @url_mapping ||= {}
     end

data/lib/ero_getter/downloader.rb

@@ -12,9 +12,11 @@ class EroGetter::Downloader
     end
 
     def mkdir(path)
-      basedir = File.dirname(path)
-      mkdir(basedir) unless File.exists?(basedir)
-      Dir.mkdir(path)
+      unless File.exists?(path)
+        basedir = File.dirname(path)
+        mkdir(basedir) unless File.exists?(basedir)
+        Dir.mkdir(path)
+      end
     end
 
     def unzip(zip_data)

data/lib/ero_getter/downloader/base.rb

@@ -13,6 +13,14 @@ class EroGetter::Downloader::Base
     self.class.to_s.underscore
   end
 
+  def directory
+    unless @dir
+      @dir = File.join(EroGetter.directory, base_dir, sub_directory)
+      EroGetter::Downloader.mkdir(@dir)
+    end
+    @dir
+  end
+
   def http_client
     @http_client ||= HTTPClient.new
   end
@@ -58,9 +66,7 @@ class EroGetter::Downloader::Base
     def sub_directory(&block)
       define_method(:sub_directory) do
         unless instance_variable_defined?(:@sub_directory)
-          dir = self.instance_eval(&block)
-          EroGetter::Downloader.mkdir(dir)
-          instance_variable_set(:@sub_directory, dir)
+          instance_variable_set(:@sub_directory, self.instance_eval(&block))
         end
         instance_variable_get(:@sub_directory)
       end

data/spec/downloader/base_spec.rb

@@ -31,6 +31,7 @@ describe EroGetter::Downloader::Base do
         @dl = @klazz.new('http://example.net/10101010.html')
         @dl.stub(:open).and_return(File.open(sample_path('sample.html')))
         EroGetter::Downloader.stub(:mkdir).and_return(true)
+        EroGetter.stub('directory').and_return('/tmp')
       end
       its(:name) { should == 'NijiEro BBS' }
       its(:url_regex) { should == regex }
@@ -41,14 +42,8 @@ describe EroGetter::Downloader::Base do
       its(:url) { should == 'http://example.net/10101010.html' }
       its(:targets) { should == ['https://github.com/masarakki/ero_getter_server',
           'https://github.com/masarakki/ero_getter_chrome_extension'] }
-      describe :sub_directory do
-        its(:sub_directory) { should == 'ero_getter_server/ero_getter_chrome_extension' }
-        it "mkdir only once" do
-          EroGetter::Downloader.should_receive(:mkdir).with('ero_getter_server/ero_getter_chrome_extension').exactly(1)
-          @dl.sub_directory
-          @dl.sub_directory
-        end
-      end
+      its(:sub_directory) { should == 'ero_getter_server/ero_getter_chrome_extension' }
+      its(:directory) { should == '/tmp/test_class/ero_getter_server/ero_getter_chrome_extension' }
     end
     context :url_mismatch do
       it {

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: ero_getter
 version: !ruby/object:Gem::Version
-  version: 0.1.2
+  version: 0.1.3
   prerelease: 
 platform: ruby
 authors:
@@ -180,7 +180,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 1155287302627282076
+      hash: 4601793703788942628
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements: