gom 0.1.0 → 0.2.0

data/spec/lib/gom/object/cached_builder_spec.rb

@@ -0,0 +1,43 @@
+require File.expand_path(File.join(File.dirname(__FILE__), "..", "..", "..", "spec_helper"))
+
+describe GOM::Object::CachedBuilder do
+
+  before :each do
+    @draft = mock GOM::Object::Draft
+    @id = mock GOM::Object::Id
+
+    @object = mock Object
+    GOM::Object::Mapping.stub(:object_by_id).and_return(@object)
+    GOM::Object::Mapping.stub(:put)
+
+    @builder = mock GOM::Object::Builder, :object => @object
+    GOM::Object::Builder.stub(:new).and_return(@builder)
+
+    @cached_builder = described_class.new @draft, @id
+  end
+
+  describe "object" do
+
+    it "should check the mapping for the object" do
+      GOM::Object::Mapping.should_receive(:object_by_id).with(@id).and_return(@object)
+      @cached_builder.object
+    end
+
+    it "should initialize the object builder" do
+      GOM::Object::Builder.should_receive(:new).with(@draft, @object).and_return(@builder)
+      @cached_builder.object
+    end
+
+    it "should set the mapping for the object" do
+      GOM::Object::Mapping.should_receive(:put).with(@object, @id).and_return(@object)
+      @cached_builder.object
+    end
+
+    it "should return the object" do
+      object = @cached_builder.object
+      object.should == @object
+    end
+
+  end
+
+end

data/spec/lib/gom/object/collection_spec.rb

@@ -0,0 +1,158 @@
+require File.expand_path(File.join(File.dirname(__FILE__), "..", "..", "..", "spec_helper"))
+
+describe GOM::Object::Collection do
+
+  before :each do
+    @object = mock Object
+
+    @fetcher = mock Object, :total_count => 1
+
+    @proxy = mock GOM::Object::Proxy
+    GOM::Object::Proxy.stub(:new).and_return(@proxy)
+
+    @builder = mock GOM::Object::CachedBuilder, :object => @object
+    GOM::Object::CachedBuilder.stub(:new).and_return(@builder)
+
+    @collection = described_class.new @fetcher
+  end
+
+  describe "total_count" do
+
+    it "should call the fetcher's total_count" do
+      @fetcher.should_receive(:total_count)
+      @collection.total_count
+    end
+
+  end
+
+  describe "first" do
+
+    context "with no previous fetched objects" do
+
+      context "with a fetcher that provides drafts" do
+
+        before :each do
+          @draft = mock Hash
+          @drafts = [ @draft ]
+
+          @fetcher.stub(:drafts).and_return(@drafts)
+        end
+
+        it "should get the drafts from the fetcher" do
+          @fetcher.should_receive(:drafts).and_return(@drafts)
+          @collection.first
+        end
+
+        it "should initialize the cached object builder with each draft" do
+          GOM::Object::CachedBuilder.should_receive(:new).with(@draft).and_return(@builder)
+          @collection.first
+        end
+
+        it "should initialize an object proxy with the object" do
+          GOM::Object::Proxy.should_receive(:new).with(@object).and_return(@proxy)
+          @collection.first
+        end
+
+        it "should return a object proxy for the first object" do
+          object_proxy = @collection.first
+          object_proxy.should == @proxy
+        end
+
+      end
+
+      context "with a fetcher that provides ids" do
+
+        before :each do
+          @id = mock GOM::Object::Id
+          @ids = [ @id ]
+
+          @fetcher.stub(:ids).and_return(@ids)
+        end
+
+        it "should get the ids from the fetcher" do
+          @fetcher.should_receive(:ids).and_return(@ids)
+          @collection.first
+        end
+
+        it "should not initialize the cached object builder" do
+          GOM::Object::CachedBuilder.should_not_receive(:new)
+          @collection.first
+        end
+
+        it "should initialize an object proxy with the id" do
+          GOM::Object::Proxy.should_receive(:new).with(@id).and_return(@proxy)
+          @collection.first
+        end
+
+        it "should return a object proxy for the first object" do
+          object_proxy = @collection.first
+          object_proxy.should == @proxy
+        end
+
+      end
+
+      context "with a fetcher that provide rows" do
+
+        before :each do
+          @row = mock Object
+          @rows = [ @row ]
+
+          @fetcher.stub(:drafts).and_return(nil)
+          @fetcher.stub(:rows).and_return(@rows)
+        end
+
+        it "should get the rows from the fetcher" do
+          @fetcher.should_receive(:rows).and_return(@rows)
+          @collection.first
+        end
+
+        it "should not initialize the cached object builder" do
+          GOM::Object::CachedBuilder.should_not_receive(:new)
+          @collection.first
+        end
+
+        it "should return the first row" do
+          row = @collection.first
+          row.should == @row
+        end
+
+      end
+
+      context "with a fetcher that doesn't provide drafts, ids nor rows" do
+
+        it "should raise a #{NotImplementedError}" do
+          lambda do
+            @collection.first
+          end.should raise_error(NotImplementedError)
+        end
+
+      end
+
+    end
+
+    context "with previously fetched objects" do
+
+      before :each do
+        @draft = mock Hash
+        @drafts = [ @draft ]
+
+        @fetcher.stub(:drafts).and_return(@drafts)
+
+        @collection.first
+      end
+
+      it "should not call the fetcher" do
+        @fetcher.should_not_receive(:drafts)
+        @collection.first
+      end
+
+      it "should return a object proxy for the first object" do
+        object_proxy = @collection.first
+        object_proxy.should == @proxy
+      end
+
+    end
+
+  end
+
+end

data/spec/lib/gom/object/draft_spec.rb

@@ -0,0 +1,59 @@
+require File.expand_path(File.join(File.dirname(__FILE__), "..", "..", "..", "spec_helper"))
+
+describe GOM::Object::Draft do
+
+  before :each do
+    @draft = described_class.new "object_1", "Object", { "test" => "test value" }, { "test" => "test_storage:object_2" }
+  end
+
+  describe "initialize" do
+
+    it "should set the id" do
+      @draft.id.should == "object_1"
+    end
+
+    it "should set the class name" do
+      @draft.class_name.should == "Object"
+    end
+
+    it "should set the properties" do
+      @draft.properties.should == { "test" => "test value" }
+    end
+
+    it "should set the relations" do
+      @draft.relations.should == { "test" => "test_storage:object_2" }
+    end
+
+  end
+
+  describe "properties" do
+
+    it "should return an empty hash if no properties are given" do
+      @draft.properties = nil
+      @draft.properties.should == { }
+    end
+
+  end
+
+  describe "relations" do
+
+    it "should return an empty hash if no relations are given" do
+      @draft.relations = nil
+      @draft.relations.should == { }
+    end
+
+  end
+
+  describe "==" do
+
+    before :each do
+      @same_draft = described_class.new "object_1", "Object", { "test" => "test value" }, { "test" => "test_storage:object_2" }
+    end
+
+    it "should return true if all members are the same" do
+      @draft.should == @same_draft
+    end
+
+  end
+
+end

data/spec/lib/gom/object/inspector_spec.rb

@@ -9,20 +9,18 @@ describe GOM::Object::Inspector do
     @object.instance_variable_set :@test, "test value"
     @object.instance_variable_set :@related_object, @related_object_proxy
 
-    @object_hash = {
-      :class => "Object",
-      :properties => { :test => "test value" },
-      :relations => { :related_object => @related_object_proxy }
-    }
-
     @inspector = GOM::Object::Inspector.new @object
   end
 
-  describe "perform" do
+  describe "draft" do
+
+    before :each do
+      @draft = GOM::Object::Draft.new nil, "Object", { :test => "test value" }, { :related_object => @related_object_proxy }
+    end
 
-    it "should return the correct object hash" do
-      @inspector.perform
-      @inspector.object_hash.should == @object_hash
+    it "should return the correct draft" do
+      draft = @inspector.draft
+      draft.should == @draft
     end
 
   end

data/spec/lib/gom/storage/adapter_spec.rb

@@ -30,42 +30,16 @@ describe GOM::Storage::Adapter do
 
   end
 
-  describe "setup" do
+  [ :setup, :fetch, :store, :remove, :collection ].each do |method_name|
 
-    it "should raise a NotImplementedError" do
-      lambda do
-        @adapter.setup
-      end.should raise_error(NotImplementedError)
-    end
-
-  end
-
-  describe "fetch" do
-
-    it "should raise a NotImplementedError" do
-      lambda do
-        @adapter.fetch
-      end.should raise_error(NotImplementedError)
-    end
-
-  end
-
-  describe "store" do
-
-    it "should raise a NotImplementedError" do
-      lambda do
-        @adapter.store
-      end.should raise_error(NotImplementedError)
-    end
-
-  end
+    describe "#{method_name}" do
 
-  describe "remove" do
+      it "should raise a NotImplementedError" do
+        lambda do
+          @adapter.send method_name
+        end.should raise_error(NotImplementedError)
+      end
 
-    it "should raise a NotImplementedError" do
-      lambda do
-        @adapter.remove
-      end.should raise_error(NotImplementedError)
     end
 
   end

data/spec/lib/gom/storage/configuration/view/class_spec.rb

@@ -0,0 +1,17 @@
+require File.expand_path(File.join(File.dirname(__FILE__), "..", "..", "..", "..", "..", "spec_helper"))
+
+describe GOM::Storage::Configuration::View::Class do
+
+  before :each do
+    @class = described_class.new "Object"
+  end
+
+  describe "initialize" do
+
+    it "should set the view's class name" do
+      @class.class_name.should == "Object"
+    end
+
+  end
+
+end

data/spec/lib/gom/storage/configuration/view/map_reduce_spec.rb

@@ -0,0 +1,21 @@
+require File.expand_path(File.join(File.dirname(__FILE__), "..", "..", "..", "..", "..", "spec_helper"))
+
+describe GOM::Storage::Configuration::View::MapReduce do
+
+  before :each do
+    @map_reduce = described_class.new "map function", "reduce function"
+  end
+
+  describe "initialize" do
+
+    it "should set the view's map function" do
+      @map_reduce.map.should == "map function"
+    end
+
+    it "should set the view's reduce function" do
+      @map_reduce.reduce.should == "reduce function"
+    end
+
+  end
+
+end

data/spec/lib/gom/storage/configuration_spec.rb

@@ -44,6 +44,35 @@ describe GOM::Storage::Configuration do
 
   end
 
+  describe "views" do
+
+    it "should return a hash including class views" do
+      view = @configuration.views[:test_object_class_view]
+      view.should be_instance_of(described_class::View::Class)
+      view.class_name.should == "Object"
+    end
+
+    it "should return a hash including map reduce views" do
+      view = @configuration.views[:test_map_view]
+      view.should be_instance_of(described_class::View::MapReduce)
+      view.map.should == "function(document) { }"
+      view.reduce.should == "function(key, values) { }"
+    end
+
+    it "should raise a #{NotImplementedError} if the view type is invalid" do
+      @configuration["views"]["test_invalid_view"] = { "type" => "invalid" }
+      lambda do
+        @configuration.views[:test_invalid_view]
+      end.should raise_error(NotImplementedError)
+    end
+
+    it "should return nil if given view name doesn't exists" do
+      view = @configuration.views[:invalid]
+      view.should be_nil
+    end
+
+  end
+
   describe "read" do
 
     it "should read the configuration file" do

data/spec/lib/gom/storage/fetcher_spec.rb

@@ -6,24 +6,14 @@ describe GOM::Storage::Fetcher do
     @id = GOM::Object::Id.new "test_storage", "object_1"
     @object = Object.new
     @object.instance_variable_set :@test, "test value"
-    @object_hash = {
-      :class => "Object",
-      :properties => { :test => "test value" }
-    }
+    @draft = GOM::Object::Draft.new nil, "Object", { :test => "test value" }
 
-    @new_method = mock Method, :arity => 3
-    @klass = mock Class, :new => @object, :method => @new_method
-    Object.stub(:const_get).and_return(@klass)
-
-    @adapter = mock GOM::Storage::Adapter, :fetch => @object_hash
+    @adapter = mock GOM::Storage::Adapter, :fetch => @draft
     @configuration = mock GOM::Storage::Configuration, :adapter => @adapter
     GOM::Storage::Configuration.stub(:[]).and_return(@configuration)
 
-    GOM::Object::Mapping.stub(:object_by_id)
-    GOM::Object::Mapping.stub(:put)
-
-    @injector = mock GOM::Object::Injector, :perform => nil, :object => @object
-    GOM::Object::Injector.stub(:new).and_return(@injector)
+    @builder = mock GOM::Object::CachedBuilder, :object => @object
+    GOM::Object::CachedBuilder.stub(:new).and_return(@builder)
 
     @fetcher = described_class.new @id
     @fetcher.stub(:object_class).and_return(@klass)
@@ -43,33 +33,12 @@ describe GOM::Storage::Fetcher do
     end
 
     it "should fetch the id from the adapter instance" do
-      @adapter.should_receive(:fetch).with("object_1").and_return(@object_hash)
-      @fetcher.object
-    end
-
-    it "should initialize the object with nil-arguments to the constructor" do
-      @klass.should_receive(:new).with(nil, nil, nil).and_return(@object)
-      @fetcher.object
-    end
-
-    it "should initialize the object with no arguments to the constructor if a variable argument count is needed" do
-      @new_method.stub(:arity).and_return(-1)
-      @klass.should_receive(:new).with(no_args).and_return(@object)
-      @fetcher.object
-    end
-
-    it "should check if a mapping exists for the object" do
-      GOM::Object::Mapping.should_receive(:object_by_id).with(@id).and_return(Object.new)
-      @fetcher.object
-    end
-
-    it "should initialize the object if not given" do
-      GOM::Object::Injector.should_receive(:new).with(anything, @object_hash).and_return(@injector)
+      @adapter.should_receive(:fetch).with("object_1").and_return(@draft)
       @fetcher.object
     end
 
-    it "should create mapping between object and id" do
-      GOM::Object::Mapping.should_receive(:put).with(@object, @id)
+    it "should initialize the cached object builder" do
+      GOM::Object::CachedBuilder.should_receive(:new).with(@draft, @id).and_return(@builder)
       @fetcher.object
     end