picky 0.9.0 → 0.9.1

data/spec/lib/index/files_spec.rb

@@ -0,0 +1,186 @@
+require 'spec_helper'
+
+describe Index::Files do
+
+  before(:each) do
+    @files_class = Index::Files
+    @files       = @files_class.new :some_name, :some_category, :some_type
+    
+    @prepared    = @files.prepared
+    
+    @index       = @files.index
+    @similarity  = @files.similarity
+    @weights     = @files.weights
+  end
+
+  describe 'create_directory' do
+    it 'should use makedirs to create the necessary directory structure' do
+      FileUtils.should_receive(:mkdir_p).once.with 'some/search/root/index/test/some_type'
+
+      @files.create_directory
+    end
+  end
+
+  describe 'cache_directory' do
+    it 'should be correct' do
+      @files.cache_directory.should == 'some/search/root/index/test/some_type'
+    end
+  end
+  
+  describe "retrieve" do
+    it "delegates to the prepared" do
+      @prepared.should_receive(:retrieve).once.with
+      
+      @files.retrieve
+    end
+  end
+  
+  describe "dump indexes" do
+    describe "dump_index" do
+      it "uses the right file" do
+        @index.should_receive(:dump).once.with :some_hash
+        
+        @files.dump_index :some_hash
+      end
+    end
+    describe "dump_similarity" do
+      it "uses the right file" do
+        @similarity.should_receive(:dump).once.with :some_hash
+        
+        @files.dump_similarity :some_hash
+      end
+    end
+    describe "dump_weights" do
+      it "uses the right file" do
+        @weights.should_receive(:dump).once.with :some_hash
+        
+        @files.dump_weights :some_hash
+      end
+    end
+  end
+  
+  describe "loading indexes" do
+    before(:each) do
+      @files.stub! :timed_exclaim
+    end
+    describe "load_index" do
+      it "uses the right file" do
+        Yajl::Parser.stub! :parse
+        
+        File.should_receive(:open).once.with 'some/search/root/index/test/some_type/some_name_some_category_index.json', 'r'
+        
+        @files.load_index
+      end
+    end
+    describe "load_similarity" do
+      it "uses the right file" do
+        Marshal.stub! :load
+        
+        File.should_receive(:open).once.with 'some/search/root/index/test/some_type/some_name_some_category_similarity.dump', 'r:binary'
+        
+        @files.load_similarity
+      end
+    end
+    describe "load_weights" do
+      it "uses the right file" do
+        Yajl::Parser.stub! :parse
+        
+        File.should_receive(:open).once.with 'some/search/root/index/test/some_type/some_name_some_category_weights.json', 'r'
+        
+        @files.load_weights
+      end
+    end
+  end
+  
+  describe "dump indexes" do
+    describe "index_cache_ok?" do
+      it "uses the right method" do
+        @index.should_receive(:cache_ok?).once.with
+        
+        @files.index_cache_ok?
+      end
+    end
+    describe "similarity_cache_ok?" do
+      it "uses the right method" do
+        @similarity.should_receive(:cache_ok?).once.with
+        
+        @files.similarity_cache_ok?
+      end
+    end
+    describe "weights_cache_ok?" do
+      it "uses the right method" do
+        @weights.should_receive(:cache_ok?).once.with
+        
+        @files.weights_cache_ok?
+      end
+    end
+  end
+  
+  describe "dump indexes" do
+    describe "index_cache_small?" do
+      it "uses the right method" do
+        @index.should_receive(:cache_small?).once.with
+        
+        @files.index_cache_small?
+      end
+    end
+    describe "similarity_cache_small?" do
+      it "uses the right method" do
+        @similarity.should_receive(:cache_small?).once.with
+        
+        @files.similarity_cache_small?
+      end
+    end
+    describe "weights_cache_small?" do
+      it "uses the right method" do
+        @weights.should_receive(:cache_small?).once.with
+        
+        @files.weights_cache_small?
+      end
+    end
+  end
+  
+  describe 'backup' do
+    it 'should call backup on all' do
+      @index.should_receive(:backup).once.with
+      @similarity.should_receive(:backup).once.with
+      @weights.should_receive(:backup).once.with
+      
+      @files.backup
+    end
+  end
+  describe 'restore' do
+    it 'should call delete on all' do
+      @index.should_receive(:restore).once.with
+      @similarity.should_receive(:restore).once.with
+      @weights.should_receive(:restore).once.with
+      
+      @files.restore
+    end
+  end
+  describe 'delete' do
+    it 'should call delete on all' do
+      @index.should_receive(:delete).once.with
+      @similarity.should_receive(:delete).once.with
+      @weights.should_receive(:delete).once.with
+      
+      @files.delete
+    end
+  end
+  
+  describe 'initialization' do
+    before(:each) do
+      @files = @files_class.new :some_name, :some_category, :some_type
+    end
+    it 'should initialize the name correctly' do
+      @files.bundle_name.should == :some_name
+    end
+    it 'should initialize the name correctly' do
+      @files.category_name.should == :some_category
+    end
+    it 'should initialize the name correctly' do
+      @files.type_name.should == :some_type
+    end
+  end
+
+end

data/spec/lib/index/type_spec.rb

@@ -4,8 +4,11 @@ describe Index::Type do
   
   context "with categories" do
     before(:each) do
-      @category1 = Index::Category.new :some_category_name1, :some_type1
-      @category2 = Index::Category.new :some_category_name2, :some_type2
+      @some_type1 = stub :some_type1, :name => :some_type1
+      @some_type2 = stub :some_type2, :name => :some_type2
+      
+      @category1 = Index::Category.new :some_category_name1, @some_type1
+      @category2 = Index::Category.new :some_category_name2, @some_type2
       
       @index = Index::Type.new :some_name, :some_result_type, false, @category1, @category2
     end

data/spec/lib/query/combinator_spec.rb

@@ -29,9 +29,11 @@ describe Query::Combinator do
   
   context "with real categories" do
     before(:each) do
-      @category1 = Index::Category.new :some_name, :some_type
-      @category2 = Index::Category.new :some_name, :some_type
-      @category3 = Index::Category.new :some_name, :some_type
+      @type1     = stub :type1, :name => :some_type
+      
+      @category1 = Index::Category.new :some_name, @type1
+      @category2 = Index::Category.new :some_name, @type1
+      @category3 = Index::Category.new :some_name, @type1
       @categories = [@category1, @category2, @category3]
       
       @combinator = Query::Combinator.new @categories

data/spec/lib/sources/delicious_spec.rb

@@ -3,7 +3,15 @@ require 'spec_helper'
 describe Sources::Delicious do
   
   context "with file" do
-
+    
+    describe "check_gem" do
+      # TODO How to actually test this?
+      #
+      it "checks if the gem is there" do
+        lambda { Sources::Delicious.new(:username, :password) }.should_not raise_error
+      end
+    end
+    
     describe "harvest" do
       before(:each) do
         @source = Sources::Delicious.new(:username, :password)

metadata

@@ -5,8 +5,8 @@ version: !ruby/object:Gem::Version
   segments: 
   - 0
   - 9
-  - 0
-  version: 0.9.0
+  - 1
+  version: 0.9.1
 platform: ruby
 authors: 
 - Florian Hanke
@@ -14,7 +14,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2010-10-26 00:00:00 +02:00
+date: 2010-10-27 00:00:00 +02:00
 default_executable: picky
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -75,8 +75,12 @@ files:
 - lib/picky/helpers/gc.rb
 - lib/picky/helpers/measuring.rb
 - lib/picky/index/bundle.rb
-- lib/picky/index/bundle_checker.rb
 - lib/picky/index/category.rb
+- lib/picky/index/file/basic.rb
+- lib/picky/index/file/json.rb
+- lib/picky/index/file/marshal.rb
+- lib/picky/index/file/text.rb
+- lib/picky/index/files.rb
 - lib/picky/index/type.rb
 - lib/picky/index/wrappers/exact_first.rb
 - lib/picky/indexers/base.rb
@@ -169,10 +173,14 @@ files:
 - spec/lib/helpers/cache_spec.rb
 - spec/lib/helpers/gc_spec.rb
 - spec/lib/helpers/measuring_spec.rb
-- spec/lib/index/bundle_checker_spec.rb
 - spec/lib/index/bundle_partial_generation_speed_spec.rb
 - spec/lib/index/bundle_spec.rb
 - spec/lib/index/category_spec.rb
+- spec/lib/index/file/basic_spec.rb
+- spec/lib/index/file/json_spec.rb
+- spec/lib/index/file/marshal_spec.rb
+- spec/lib/index/file/text_spec.rb
+- spec/lib/index/files_spec.rb
 - spec/lib/index/type_spec.rb
 - spec/lib/index/wrappers/exact_first_spec.rb
 - spec/lib/indexers/base_spec.rb
@@ -265,10 +273,14 @@ test_files:
 - spec/lib/helpers/cache_spec.rb
 - spec/lib/helpers/gc_spec.rb
 - spec/lib/helpers/measuring_spec.rb
-- spec/lib/index/bundle_checker_spec.rb
 - spec/lib/index/bundle_partial_generation_speed_spec.rb
 - spec/lib/index/bundle_spec.rb
 - spec/lib/index/category_spec.rb
+- spec/lib/index/file/basic_spec.rb
+- spec/lib/index/file/json_spec.rb
+- spec/lib/index/file/marshal_spec.rb
+- spec/lib/index/file/text_spec.rb
+- spec/lib/index/files_spec.rb
 - spec/lib/index/type_spec.rb
 - spec/lib/index/wrappers/exact_first_spec.rb
 - spec/lib/indexers/base_spec.rb

data/lib/picky/index/bundle_checker.rb

@@ -1,58 +0,0 @@
-# encoding: utf-8
-#
-module Index
-
-  # Checks bundle indexes.
-  #
-  class BundleChecker
-    
-    attr_reader :bundle
-    
-    def initialize bundle
-      @bundle = bundle
-    end
-    
-    # Check all index files and raise if necessary.
-    #
-    def raise_unless_cache_exists
-      warn_cache_small :index      if cache_small?(bundle.index_cache_path)
-      # warn_cache_small :similarity if cache_small?(similarity_cache_path)
-      warn_cache_small :weights    if cache_small?(bundle.weights_cache_path)
-
-      raise_cache_missing :index      unless cache_ok?(bundle.index_cache_path)
-      raise_cache_missing :similarity unless cache_ok?(bundle.similarity_cache_path)
-      raise_cache_missing :weights    unless cache_ok?(bundle.weights_cache_path)
-    end
-    
-    def size_of path
-      `ls -l #{path} | awk '{print $5}'`.to_i
-    end
-    # Check if the cache files are there and do not have size 0.
-    #
-    def caches_ok?
-      cache_ok?(bundle.index_cache_path) &&
-      cache_ok?(bundle.similarity_cache_path) &&
-      cache_ok?(bundle.weights_cache_path)
-    end
-    # Is the cache ok? I.e. larger than four in size.
-    #
-    def cache_ok? path
-      size_of(path) > 0
-    end
-    # Raises an appropriate error message.
-    #
-    def raise_cache_missing what
-      raise "#{what} cache for #{bundle.identifier} missing."
-    end
-    # Is the cache small?
-    #
-    def cache_small? path
-      size_of(path) < 16
-    end
-    def warn_cache_small what
-      puts "#{what} cache for #{bundle.identifier} smaller than 16 bytes."
-    end
-    
-  end
-  
-end

data/spec/lib/index/bundle_checker_spec.rb

@@ -1,67 +0,0 @@
-require 'spec_helper'
-
-describe Index::BundleChecker do
-  
-  before(:each) do
-    @category    = stub :category, :name => :some_category
-    @type        = stub :type, :name => :some_type
-    @partial     = stub :partial
-    @weights     = stub :weights
-    @similarity  = stub :similarity
-    @index_class = Index::Bundle
-    @index       = @index_class.new :some_name, @category, @type, @partial, @weights, @similarity
-    
-    @checker = @index.checker
-  end
-  
-  describe 'raise_unless_cache_exists' do
-    before(:each) do
-      @checker.stub! :cache_small? => false
-    end
-    context 'weights cache missing' do
-      before(:each) do
-        @checker.stub! :cache_ok? => true
-        @index.stub! :weights_cache_path => 'weights_cache_path'
-        @checker.should_receive(:cache_ok?).any_number_of_times.with('weights_cache_path').and_return false
-      end
-      it 'should raise' do
-        lambda do
-          @checker.raise_unless_cache_exists
-        end.should raise_error("weights cache for some_name: some_type some_category missing.")
-      end
-    end
-    context 'similarity cache missing' do
-      before(:each) do
-        @checker.stub! :cache_ok? => true
-        @index.stub! :similarity_cache_path => 'similarity_cache_path'
-        @checker.should_receive(:cache_ok?).any_number_of_times.with('similarity_cache_path').and_return false
-      end
-      it 'should raise' do
-        lambda do
-          @checker.raise_unless_cache_exists
-        end.should raise_error("similarity cache for some_name: some_type some_category missing.")
-      end
-    end
-    context 'index cache missing' do
-      before(:each) do
-        @checker.stub! :cache_ok? => true
-        @index.stub! :index_cache_path => 'index_cache_path'
-        @checker.should_receive(:cache_ok?).any_number_of_times.with('index_cache_path').and_return false
-      end
-      it 'should raise' do
-        lambda do
-          @checker.raise_unless_cache_exists
-        end.should raise_error("index cache for some_name: some_type some_category missing.")
-      end
-    end
-    context 'all ok' do
-      before(:each) do
-        @checker.stub! :cache_ok? => true
-      end
-      it 'should not raise' do
-        lambda { @checker.raise_unless_cache_exists }.should_not raise_error
-      end
-    end
-  end
-  
-end