mongoid 1.0.6 → 1.1.0

data/spec/integration/mongoid/associations_spec.rb

@@ -3,9 +3,39 @@ require "spec_helper"
 describe Mongoid::Associations do
 
   before do
-    Mongoid.database.collection(:people).drop
-    Mongoid.database.collection(:games).drop
-    Mongoid.database.collection(:posts).drop
+    Person.delete_all
+    Game.delete_all
+    Post.delete_all
+  end
+
+  context "criteria on has many embedded associations" do
+
+    before do
+      @person = Person.new(:title => "Sir")
+      @sf_apartment = Address.new(:street => "Genoa Pl", :state => "CA", :address_type => "Apartment")
+      @la_home = Address.new(:street => "Rodeo Dr", :state => "CA", :address_type => "Home")
+      @sf_home = Address.new(:street => "Pacific", :state => "CA", :address_type => "Home")
+      @person.addresses << [ @sf_apartment, @la_home, @sf_home ]
+    end
+
+    it "handles a single criteria" do
+      cas = @person.addresses.california
+      cas.size.should == 3
+      cas.should == [ @sf_apartment, @la_home, @sf_home ]
+    end
+
+    it "handles chained criteria" do
+      ca_homes = @person.addresses.california.homes
+      ca_homes.size.should == 2
+      ca_homes.should == [ @la_home, @sf_home ]
+    end
+
+    it "handles chained criteria with named scopes" do
+      ca_homes = @person.addresses.california.homes.rodeo
+      ca_homes.size.should == 1
+      ca_homes.should == [ @la_home ]
+    end
+
   end
 
   context "one-to-one relational associations" do
@@ -43,8 +73,8 @@ describe Mongoid::Associations do
     end
 
     it "sets the association on save" do
-      @from_db = Person.find(@person.id)
-      @from_db.posts.should == [@post]
+      from_db = Person.find(@person.id)
+      from_db.posts.should == [@post]
     end
 
     context "when building" do
@@ -60,6 +90,20 @@ describe Mongoid::Associations do
 
     end
 
+    context "finding associated objects" do
+
+      before do
+        @extra_post = Post.create(:title => "Orphan")
+      end
+
+      it "returns only those objects scoped to the parent" do
+        from_db = Person.find(@person.id)
+        Post.all.size.should == 2
+        from_db.posts.all.size.should == 1
+      end
+
+    end
+
   end
 
   context "nested embedded associations" do
@@ -99,6 +143,24 @@ describe Mongoid::Associations do
 
       end
 
+      context "when a has-one to has-many" do
+
+        before do
+          @person = Person.new(:title => "Sir", :ssn => "1")
+          @name = Name.new(:first_name => "Syd")
+          @person.name = @name
+          @person.save
+        end
+
+        it "persists all the associations properly" do
+          from_db = Person.find(@person.id)
+          translation = Translation.new(:language => "fr")
+          from_db.name.translations << translation
+          from_db.attributes[:name][:translations].should_not be_nil
+        end
+
+      end
+
       context "when a has-many to has-many" do
 
         before do

data/spec/integration/mongoid/attributes_spec.rb

@@ -0,0 +1,22 @@
+require "spec_helper"
+
+describe Mongoid::Attributes do
+
+  context "when persisting nil attributes" do
+
+    before do
+      @person = Person.create(:score => nil, :ssn => "555-66-7777")
+    end
+
+    after do
+      Person.delete_all
+    end
+
+    it "the field should not exist" do
+      from_db = Person.find(@person.id)
+      from_db.attributes.has_key?(:score).should be_false
+    end
+
+  end
+
+end

data/spec/integration/mongoid/commands_spec.rb

@@ -10,6 +10,37 @@ describe Mongoid::Commands do
     @person.delete
   end
 
+  describe ".create" do
+
+    it "saves and returns the document" do
+      person = Person.create(:title => "Sensei")
+      person.should be_a_kind_of(Person)
+      person.should_not be_a_new_record
+    end
+
+  end
+
+  describe ".create!" do
+
+    context "inserting with a field that is not unique" do
+
+      context "when a unique index exists" do
+
+        after do
+          Person.delete_all
+        end
+
+        it "raises an error" do
+          Person.create!(:ssn => "555-55-9999")
+          lambda { Person.create!(:ssn => "555-55-9999") }.should raise_error
+        end
+
+      end
+
+    end
+
+  end
+
   describe "#delete" do
 
     before do
@@ -56,16 +87,34 @@ describe Mongoid::Commands do
 
   end
 
+  describe "save!" do
+
+    context "inserting with a field that is not unique" do
+
+      context "when a unique index exists" do
+
+        it "raises an error" do
+          Person.create!(:ssn => "555-55-9999")
+          person = Person.new(:ssn => "555-55-9999")
+          lambda { person.save!(:ssn => "555-55-9999") }.should raise_error
+        end
+
+      end
+
+    end
+
+  end
+
   describe "#update_attributes" do
 
     context "when validation passes" do
 
       it "returns true" do
-        @person.update_attributes(:title => "Blah").should be_true
+        @person.update_attributes(:ssn => "555-55-1234").should be_true
       end
 
       it "saves the attributes" do
-        @person.update_attributes(:title => "Blah")
+        @person.update_attributes(:ssn => "555-55-1235", :title => "Blah")
         @from_db = Person.find(@person.id)
         @from_db.title.should == "Blah"
       end
@@ -74,16 +123,6 @@ describe Mongoid::Commands do
 
   end
 
-  describe ".create" do
-
-    it "saves and returns the document" do
-      person = Person.create(:title => "Sensei")
-      person.should be_a_kind_of(Person)
-      person.should_not be_a_new_record
-    end
-
-  end
-
   describe ".delete_all" do
 
     it "returns true" do

data/spec/integration/mongoid/criteria_spec.rb

@@ -6,7 +6,7 @@ describe Mongoid::Criteria do
 
     before do
       10.times do |n|
-        Person.create(:title => "Sir", :age => (n * 10), :aliases => ["D", "Durran"])
+        Person.create(:title => "Sir", :age => (n * 10), :aliases => ["D", "Durran"], :ssn => "#{n}")
       end
     end
 
@@ -24,7 +24,7 @@ describe Mongoid::Criteria do
 
     before do
       10.times do |n|
-        Person.create(:title => "Sir", :age => ((n + 1) * 10), :aliases => ["D", "Durran"])
+        Person.create(:title => "Sir", :age => ((n + 1) * 10), :aliases => ["D", "Durran"], :ssn => "#{n}")
       end
     end
 
@@ -42,7 +42,7 @@ describe Mongoid::Criteria do
 
     before do
       10.times do |n|
-        Person.create(:title => "Sir", :age => 5, :aliases => ["D", "Durran"])
+        Person.create(:title => "Sir", :age => 5, :aliases => ["D", "Durran"], :ssn => "#{n}")
       end
     end
 

data/spec/integration/mongoid/document_spec.rb

@@ -3,7 +3,7 @@ require "spec_helper"
 describe Mongoid::Document do
 
   before do
-    Mongoid.database.collection(:people).drop
+    Person.delete_all
   end
 
   context "when document contains a hash field" do
@@ -50,7 +50,7 @@ describe Mongoid::Document do
 
     before do
       5.times do |n|
-        Person.create(:title => "Sir")
+        Person.create(:title => "Sir", :ssn => "#{n}")
       end
     end
 
@@ -87,10 +87,10 @@ describe Mongoid::Document do
   context "chaining criteria scopes" do
 
     before do
-      @one = Person.create(:title => "Mr", :age => 55, :terms => true)
-      @two = Person.create(:title => "Sir", :age => 55, :terms => true)
-      @three = Person.create(:title => "Sir", :age => 35, :terms => true)
-      @four = Person.create(:title => "Sir", :age => 55, :terms => false)
+      @one = Person.create(:title => "Mr", :age => 55, :terms => true, :ssn => "q")
+      @two = Person.create(:title => "Sir", :age => 55, :terms => true, :ssn => "w")
+      @three = Person.create(:title => "Sir", :age => 35, :terms => true, :ssn => "e")
+      @four = Person.create(:title => "Sir", :age => 55, :terms => false, :ssn => "r")
     end
 
     it "finds by the merged criteria" do
@@ -120,7 +120,7 @@ describe Mongoid::Document do
 
       before do
         @person = Person.create(:title => "Lead")
-        @person.addresses.create(:street => "1st Street")
+        address = @person.addresses.create(:street => "1st Street")
         @person.create_name(:first_name => "Emmanuel")
         @person.save
       end
@@ -288,7 +288,7 @@ describe Mongoid::Document do
 
     before do
       10.times do |num|
-        Person.create(:title => "Test-#{num}")
+        Person.create(:title => "Test-#{num}", :ssn => "55#{num}")
       end
     end
 

data/spec/integration/mongoid/finders_spec.rb

@@ -5,7 +5,7 @@ describe Mongoid::Finders do
   describe "#find" do
 
     before do
-      @document = Person.create(:title => "Mrs.")
+      @document = Person.create(:title => "Mrs.", :ssn => "another")
     end
 
     after do

data/spec/integration/mongoid/inheritance_spec.rb

@@ -2,6 +2,12 @@ require "spec_helper"
 
 describe Mongoid::Document do
 
+  before do
+    Browser.delete_all
+    Firefox.delete_all
+    Canvas.delete_all
+  end
+
   context "when document is a subclass of a root class" do
 
     before do

data/spec/integration/mongoid/named_scope_spec.rb

@@ -4,7 +4,7 @@ describe Mongoid::NamedScope do
 
   describe ".named_scope" do
 
-    class Person
+    class ::Person
       named_scope :doctors, {:where => {:title => 'Dr.'} }
       named_scope :old, criteria.where(:age.gt => 50)
     end

data/spec/spec_helper.rb

@@ -38,6 +38,14 @@ class Person
   field :map, :type => Hash
   field :score, :type => Integer
   field :blood_alcohol_content, :type => Float
+  field :ssn
+
+  index :age
+  index :addresses
+  index :dob
+  index :name
+  index :title
+  index :ssn, :unique => true
 
   attr_reader :rescored
 
@@ -50,12 +58,6 @@ class Person
   accepts_nested_attributes_for :addresses, :reject_if => lambda { |attrs| attrs["street"].blank? }
   accepts_nested_attributes_for :name
 
-  index :age
-  index :addresses
-  index :dob
-  index :name
-  index :title
-
   has_one_related :game
   has_many_related :posts
 
@@ -145,18 +147,34 @@ end
 
 class Address
   include Mongoid::Document
+  field :address_type
+  field :number, :type => Integer
   field :street
   field :city
   field :state
   field :post_code
   field :parent_title
+  field :services, :type => Array
   key :street
   has_many :locations
   belongs_to :addressable, :inverse_of => :addresses
 
+  named_scope :rodeo, where(:street => "Rodeo Dr")
+
   def set_parent=(set = false)
     self.parent_title = addressable.title if set
   end
+
+  class << self
+    def california
+      where(:state => "CA")
+    end
+
+    def homes
+      where(:address_type => "Home")
+    end
+
+  end
 end
 
 class Location
@@ -171,6 +189,7 @@ class Name
   field :last_name
   field :parent_title
   key :first_name, :last_name
+  has_many :translations
   belongs_to :person, :inverse_of => :name
 
   def set_parent=(set = false)
@@ -178,6 +197,12 @@ class Name
   end
 end
 
+class Translation
+  include Mongoid::Document
+  field :language
+  belongs_to :name, :inverse_of => :translations
+end
+
 class Comment
   include Mongoid::Document
   include Mongoid::Versioning
@@ -292,3 +317,19 @@ module Medical
     field :name
   end
 end
+
+###################################
+
+class Author
+  include Mongoid::Document
+  field :name
+  has_many :books
+end
+
+class Book
+  include Mongoid::Document
+  field :title
+  belongs_to :author, :inverse_of => :books
+end
+
+

data/spec/unit/mongoid/associations/has_many_related_spec.rb

@@ -193,6 +193,48 @@ describe Mongoid::Associations::HasManyRelated do
 
     end
 
+    context "when finding all with conditions" do
+
+      before do
+        @post = stub
+      end
+
+      it "passes the conditions to the association class" do
+        Post.expects(:find).with(:all, :conditions => { :title => "Testing", :person_id => @parent.id }).returns([@post])
+        posts = @association.find(:all, :conditions => { :title => "Testing" })
+        posts.should == [@post]
+      end
+
+    end
+
+    context "when finding first with conditions" do
+
+      before do
+        @post = stub
+      end
+
+      it "passes the conditions to the association class" do
+        Post.expects(:find).with(:first, :conditions => { :title => "Testing", :person_id => @parent.id }).returns(@post)
+        post = @association.find(:first, :conditions => { :title => "Testing" })
+        post.should == @post
+      end
+
+    end
+
+    context "when finding last with conditions" do
+
+      before do
+        @post = stub
+      end
+
+      it "passes the conditions to the association class" do
+        Post.expects(:find).with(:last, :conditions => { :title => "Testing", :person_id => @parent.id }).returns(@post)
+        post = @association.find(:last, :conditions => { :title => "Testing" })
+        post.should == @post
+      end
+
+    end
+
   end
 
   describe ".initialize" do