seomoz-ripple 1.0.0.pre

data/spec/ripple/associations/one_embedded_proxy_spec.rb

@@ -0,0 +1,125 @@
+require 'spec_helper'
+
+describe Ripple::Associations::OneEmbeddedProxy do
+  require 'support/models/family'
+  require 'support/models/user'
+  require 'support/models/address'
+  
+  before :each do
+    @parent = Parent.new
+    @child  = Child.new
+    @gchild = Grandchild.new
+  end
+  
+  it "should not have a child before one is set" do
+    @parent.child.should be_nil
+  end
+
+  it "should raise NoMethodError when an undefined method is called on the unset child" do
+    expect { @parent.child.some_undefined_method }.to raise_error(NoMethodError)
+  end
+  
+  it "should be able to set and get its child" do
+    @parent.child = @child
+    @parent.child.should equal(@child)
+  end
+  
+  it "should set the parent document on the child when assigning" do
+    @parent.child = @child
+    @child._parent_document.should == @parent
+  end
+  
+  it "should return the assignment when assigning" do
+    rtn = @parent.child = @child
+    rtn.should == @child
+  end
+  
+  it "should set the parent document on the child when accessing" do
+    @parent.child = @child
+    @parent.child._parent_document.should == @parent
+  end
+  
+  it "should be able to replace its child with a different child" do
+    @son = Child.new(:name => 'Son')
+    @parent.child = @child
+    @parent.child.name.should be_blank
+    @parent.child = @son
+    @parent.child.name.should == 'Son'
+  end
+  
+  it "should be able to build a new child" do
+    Child.stub!(:new).and_return(@child)
+    @parent.child.build.should == @child
+  end
+  
+  it "should assign a parent to the child created with instantiate_target" do
+    Child.stub!(:new).and_return(@child)
+    @child._parent_document.should be_nil
+    @parent.child.build._parent_document.should == @parent
+  end
+  
+  it "should validate the child when saving the parent" do
+    @parent.valid?.should be_true
+    @child.name = ''
+    @parent.child = @child
+    @child.valid?.should be_false
+    @parent.valid?.should be_false
+  end
+  
+  it "should not save the root document when a child is invalid" do
+    @parent.child = @child
+    @parent.save.should be_false
+  end
+  
+  it "should allow embedding documents in embedded documents" do
+    @parent.child = @child
+    @child.gchild = @gchild
+    @gchild._root_document.should   == @parent
+    @gchild._parent_document.should == @child
+  end
+
+  it "should refuse assigning a document of the wrong type" do
+    lambda { @parent.child = @gchild }.should raise_error
+    lambda { @child.gchild = [] }.should raise_error
+  end
+  
+  describe "callbacks" do
+    before :each do
+      $pinger = mock("callback verifier")
+    end
+    
+    it "should run callbacks for the child and documents" do
+      $pinger.should_receive(:ping).once
+      Child.before_validation { $pinger.ping }
+      @child = Child.new
+      @child.valid?
+    end 
+
+    # this will work using parent and child classes, but only run by itself
+    # it also works using different classes, but only run in this file
+    # IDK why that is, but my Yakshaver 2000 just ran out of juice
+    
+    # does this even matter? we call valid? all over the place and that
+    # will trigger the callback anyway. 
+    # you probably shouldn't use validation callbacks and expect them to 
+    # *only* run once
+    
+    # it "should run callbacks for the parent and child and documents respectivly" do
+    #   $pinger = mock("callback verifier")
+    #   $pinger.should_receive(:ping).once
+    #   $pinger.should_receive(:pong).once
+    #   Child.before_validation  { $pinger.ping }
+    #   Parent.before_validation { $pinger.pong }
+    #   @child  = Child.new
+    #   @parent = Parent.new
+    #   @parent.child = @child
+    #   @parent.valid?
+    # end
+    
+    after :each do
+      Child.reset_callbacks(:validation)
+      Parent.reset_callbacks(:validation)
+    end
+  end
+
+end

data/spec/ripple/associations/one_key_proxy_spec.rb

@@ -0,0 +1,82 @@
+require 'spec_helper'
+
+describe Ripple::Associations::OneKeyProxy do
+
+  before :each do
+    @user    = User.new
+    @profile = Profile.new(:color => 'blue')
+  end
+
+  describe "User with a key corresponded profile" do
+    it "should not have a profile before it is set" do
+      @user.profile.should be_nil
+    end
+
+    it "should be able to get and set its profile" do
+      @user.profile = @profile
+      @user.profile.should eq(@profile)
+    end
+
+    it "should return the assignment when assigning" do
+      rtn = @user.profile = @profile
+      rtn.should eq(@profile)
+    end
+
+    it "should be able to replace its profile with a new one" do
+      @user.profile = @profile
+      @user.profile = Profile.new(:color => 'red')
+      @user.profile.color.should eq('red')
+    end
+
+    it "should be able to build a new profile" do
+      Profile.stub(:new).and_return(@profile)
+      @user.profile.build.should eq(@profile)
+    end
+
+    it "should assign its key to the associated profile when assigning" do
+      @user.key     = "foo"
+      @user.profile = @profile
+      @profile.key.should eq("foo")
+    end
+
+    it "should assign its key to the built profile" do
+      @user.key = "foo"
+      @user.profile.build.key.should eq("foo")
+    end
+
+    it "should update the key on the profile when updating it on the user" do
+      @user.profile = @profile
+      @user.key = "foo"
+      @profile.key.should eq("foo")
+    end
+
+    it "should not update the key of a previous profile" do
+      @profile2 = Profile.new
+      @user.profile = @profile
+      @user.profile = @profile2
+      @user.key = "foo"
+      @profile.key.should_not eq("foo")
+    end
+
+    it "should not infinitely loop when assigning a user to the profile" do
+      @user2 = User.new
+      @user.profile = @profile
+      @profile.user = @user2
+      @profile.key = "foo"
+      @user2.key.should eq("foo")
+    end
+
+    it "should update the user's key when being updated on the profile" do
+      @user.profile = @profile
+      @profile.key = "foo"
+      @user.key.should eq("foo")
+    end
+
+    it "should work properly with custom key methods" do
+      @user = Ninja.new(:name => 'Naruto')
+      @user.profile = @profile
+      @profile.key.should eq("ninja-naruto")
+    end
+  end
+
+end

data/spec/ripple/associations/one_linked_proxy_spec.rb

@@ -0,0 +1,91 @@
+require 'spec_helper'
+
+describe Ripple::Associations::OneLinkedProxy do
+  require 'support/models/tasks'
+  require 'support/models/family'
+
+  before :each do
+    @person = Person.new {|p| p.key = "riak-user" }
+    @profile = Profile.new {|t| t.key = "one" }
+    @other_profile = Profile.new {|t| t.key = "two" }
+    [@person, @profile, @other_profile].each do |doc|
+      doc.stub!(:new?).and_return(false)
+    end
+  end
+
+  it "should be blank before the associated document is set" do
+    @person.profile.should_not be_present
+  end
+
+  it "should accept a single document" do
+    lambda { @person.profile = @profile }.should_not raise_error
+  end
+
+  it "should set the link on the RObject when assigning" do
+    @person.profile = @profile
+    @person.robject.links.should include(@profile.to_link("profile"))
+  end
+
+  it "should return the assigned document when assigning" do
+    ret = (@person.profile = @profile)
+    ret.should == @profile
+  end
+
+  it "should link-walk to the associated document when accessing" do
+    @person.robject.links << @profile.robject.to_link("profile")
+    @person.robject.should_receive(:walk).with(Riak::WalkSpec.new(:bucket => "profiles", :tag => "profile")).and_return([[@profile.robject]])
+    @person.profile.should be_present
+  end
+
+  it "handles conflict appropriately by selecting the linked-walk robject that matches the link" do
+    @person.robject.links << @profile.robject.to_link("profile")
+    @person.robject.
+      should_receive(:walk).
+      with(Riak::WalkSpec.new(:bucket => "profiles", :tag => "profile")).
+      and_return([[@other_profile.robject, @profile.robject]])
+
+    @person.profile.should == @profile
+  end
+
+  it "allows the links to be replaced directly" do
+    @person.profile = @profile
+    @person.profile.__send__(:should_receive, :reset)
+    @person.profile.__send__(:links).should == [@profile.robject.to_link("profile")]
+    @person.profile.replace_links(@other_profile.robject.to_link("profile"))
+    @person.profile.__send__(:links).should == [@other_profile.robject.to_link("profile")]
+  end
+
+  it "should return nil immediately if the association link is missing" do
+    @person.robject.links.should be_empty
+    @person.profile.should be_nil
+  end
+
+  it "should replace associated document with a new one" do
+    @person.profile = @profile
+    @person.profile = @other_profile
+    @person.profile.should == @other_profile
+  end
+
+  it "replaces the associated document with the target of the proxy" do
+    @other_person = Person.new {|p| p.key = "another-riak-user" }
+    @other_person.profile = @other_profile
+
+    @person.profile = @other_person.profile
+    @person.profile.should == @other_profile
+  end
+
+  it "refuses assigning a proxy if its target is the wrong type" do
+    parent = Parent.new
+    parent.child = Child.new
+
+    lambda { @person.profile = parent.child }.should raise_error
+  end
+
+  # it "should be able to build a new associated document" do
+  #   @person.profile.build
+  # end
+
+  it "should refuse assigning a document of the wrong type" do
+    lambda { @person.profile = @person }.should raise_error
+  end
+end

data/spec/ripple/associations/one_stored_key_proxy_spec.rb

@@ -0,0 +1,72 @@
+require 'spec_helper'
+
+describe Ripple::Associations::OneStoredKeyProxy do
+  require 'support/models/transactions'
+  require 'support/models/family'
+
+  before :each do
+    @account = Account.new {|e| e.key = "accounty" }
+    @other_account = Account.new{|e| e.key = "ickycount" }
+    @transaction = Transaction.new{|e| e.key = "transacty" }
+  end
+
+  it "should be blank before the associated document is set" do
+    @transaction.account.should_not be_present
+  end
+
+  it "should accept a single document" do
+    lambda { @transaction.account = @account }.should_not raise_error
+  end
+
+  it "should set the key when assigning" do
+    @transaction.account = @account
+    @transaction.account_key.should == "accounty"
+  end
+
+  it "should return the assigned document when assigning" do
+    ret = (@transaction.account = @account)
+    ret.should == @account
+  end
+
+  it "should find the associated document when accessing" do
+    @transaction.account_key = "accounty"
+    Account.should_receive(:find).with("accounty").and_return(@account)
+    @transaction.account.should be_present
+  end
+
+  it "should return nil immediately if the association link is missing" do
+    @transaction.account_key.should be_nil
+    @transaction.account.should be_nil
+  end
+
+  it "should replace associated document with a new one" do
+    @transaction.account = @account
+    @transaction.account = @other_account
+    @transaction.account.should == @other_account
+    @transaction.account_key.should == "ickycount"
+  end
+
+  it "should replace the associated document with the target of the proxy" do
+    other_transaction = Transaction.new {|e| e.key = "ickytrans" }
+    other_transaction.account = @other_account
+
+    @transaction.account = other_transaction.account
+    @transaction.account.should == @other_account
+  end
+
+  it "refuses assigning a proxy if its target is the wrong type" do
+    parent = Parent.new{|e| e.child = Child.new}
+    lambda { @transaction.account = parent.child }.should raise_error
+  end
+
+  it "should refuse assigning a document of the wrong type" do
+    lambda { @transaction.account = @transaction }.should raise_error
+  end
+
+  it "should nil out the association if nil is assigned" do
+    @transaction.account = @account
+    @transaction.account = nil
+    @transaction.account.should be_nil
+    @transaction.account_key.should be_nil
+  end
+end

data/spec/ripple/associations/proxy_spec.rb

@@ -0,0 +1,84 @@
+require 'spec_helper'
+
+describe Ripple::Associations::Proxy do
+  require 'support/associations/proxies'
+
+  before :each do
+    @owner = mock('owner')
+    @owner.stub!(:new?).and_return(false)
+    @association = mock('association')
+    @association.stub!(:options).and_return({:extend => []})
+
+    @proxy = FakeProxy.new(@owner, @association)
+    @nil_proxy = FakeNilProxy.new(@owner, @association)
+    @blank_proxy = FakeBlankProxy.new(@owner, @association)
+  end
+
+  it "should pretend to be the target class" do
+    @proxy.should be_kind_of(Array)
+  end
+
+  it "should set the target to nil when reset" do
+    @proxy.reset
+    @proxy.target.should be_nil
+  end
+
+  describe "delegation" do
+    it "should inspect the target" do
+      @proxy.inspect.should == "[1, 2]"
+    end
+
+    it "should respond to the same methods as the target" do
+      [:each, :size].each do |m|
+        @proxy.should respond_to(m)
+      end
+      @proxy.should_not respond_to(:gsub)
+    end
+
+    it "should send to the proxy if it responds to the method" do
+      @proxy.send(:reset)
+      @proxy.target.should be_nil
+    end
+
+    it "should send to the target if target responds to the method" do
+      @proxy.send(:size).should == 2
+    end
+
+    it "should send resulting in a method missing if neither the proxy nor the target respond to the method" do
+      lambda { @proxy.send(:explode) }.should raise_error(NoMethodError)
+    end
+  end
+
+  describe "when target is nil" do
+    subject { @nil_proxy }
+    it { should be_nil }
+    it { should be_blank }
+    it { should_not be_present }
+  end
+
+  describe "when the target is blank" do
+    subject { @blank_proxy }
+    it { should_not be_nil }
+    it { should be_blank }
+    it { should_not be_present }
+  end
+
+  describe "#has_changed_documents?" do
+    before(:each) { @proxy.respond_to?(:loaded_documents).should be_true }
+
+    it "returns true if any of the loaded documents return true from #changed?" do
+      @proxy.stub(:loaded_documents => [stub(:changed? => false), stub(:changed? => true)])
+      @proxy.has_changed_documents?.should be_true
+    end
+
+    it "returns false if none of the loaded documents return true from #changed?" do
+      @proxy.stub(:loaded_documents => [stub(:changed? => false), stub(:changed? => false)])
+      @proxy.has_changed_documents?.should be_false
+    end
+
+    it "returns false if it has no loaded documents" do
+      @proxy.stub(:loaded_documents => [])
+      @proxy.has_changed_documents?.should be_false
+    end
+  end
+end