mattetti-couchrest 0.14.2 → 0.15

data/spec/couchrest/more/extended_doc_spec.rb

@@ -1,4 +1,7 @@
 require File.dirname(__FILE__) + '/../../spec_helper'
+require File.join(FIXTURE_PATH, 'more', 'article')
+require File.join(FIXTURE_PATH, 'more', 'course')
+
 
 describe "ExtendedDocument" do
   
@@ -41,10 +44,86 @@ describe "ExtendedDocument" do
     end
   end
   
+  class WithTemplateAndUniqueID < CouchRest::ExtendedDocument
+    use_database TEST_SERVER.default_database
+    unique_id do |model|
+      model['important-field']
+    end
+    property :preset, :default => 'value'
+    property :has_no_default
+  end
+  
   before(:each) do
     @obj = WithDefaultValues.new
   end
   
+  describe "instance database connection" do
+    it "should use the default database" do
+      @obj.database.name.should == 'couchrest-test'
+    end
+    
+    it "should override the default db" do
+      @obj.database = TEST_SERVER.database!('couchrest-extendedmodel-test')
+      @obj.database.name.should == 'couchrest-extendedmodel-test'
+      @obj.database.delete!
+    end
+  end
+  
+  describe "a new model" do
+    it "should be a new_record" do
+      @obj = Basic.new
+      @obj.rev.should be_nil
+      @obj.should be_a_new_record
+    end
+    it "should be a new_document" do
+      @obj = Basic.new
+      @obj.rev.should be_nil
+      @obj.should be_a_new_document
+    end
+  end
+  
+  describe "update attributes without saving" do
+    before(:each) do
+      a = Article.get "big-bad-danger" rescue nil
+      a.destroy if a
+      @art = Article.new(:title => "big bad danger")
+      @art.save
+    end
+    it "should work for attribute= methods" do
+      @art['title'].should == "big bad danger"
+      @art.update_attributes_without_saving('date' => Time.now, :title => "super danger")
+      @art['title'].should == "super danger"
+    end
+    
+    it "should flip out if an attribute= method is missing" do
+      lambda {
+        @art.update_attributes_without_saving('slug' => "new-slug", :title => "super danger")        
+      }.should raise_error
+    end
+    
+    it "should not change other attributes if there is an error" do
+      lambda {
+        @art.update_attributes_without_saving('slug' => "new-slug", :title => "super danger")        
+      }.should raise_error
+      @art['title'].should == "big bad danger"
+    end
+  end
+  
+  describe "update attributes" do
+    before(:each) do
+      a = Article.get "big-bad-danger" rescue nil
+      a.destroy if a
+      @art = Article.new(:title => "big bad danger")
+      @art.save
+    end
+    it "should save" do
+      @art['title'].should == "big bad danger"
+      @art.update_attributes('date' => Time.now, :title => "super danger")
+      loaded = Article.get(@art.id)
+      loaded['title'].should == "super danger"
+    end
+  end
+  
   describe "with default" do
     it "should have the default value set at initalization" do
       @obj.preset.should == {:right => 10, :top_align => false}
@@ -67,7 +146,137 @@ describe "ExtendedDocument" do
     end
   end
   
+  describe "a doc with template values (CR::Model spec)" do
+    before(:all) do
+      WithTemplateAndUniqueID.all.map{|o| o.destroy(true)}
+      WithTemplateAndUniqueID.database.bulk_delete
+      @tmpl = WithTemplateAndUniqueID.new
+      @tmpl2 = WithTemplateAndUniqueID.new(:preset => 'not_value', 'important-field' => '1')
+    end
+    it "should have fields set when new" do
+      @tmpl.preset.should == 'value'
+    end
+    it "shouldn't override explicitly set values" do
+      @tmpl2.preset.should == 'not_value'
+    end
+    it "shouldn't override existing documents" do
+      @tmpl2.save
+      tmpl2_reloaded = WithTemplateAndUniqueID.get(@tmpl2.id)
+      @tmpl2.preset.should == 'not_value'
+      tmpl2_reloaded.preset.should == 'not_value'
+    end
+  end
+  
+  describe "getting a model" do
+    before(:all) do
+      @art = Article.new(:title => 'All About Getting')
+      @art.save
+    end
+    it "should load and instantiate it" do
+      foundart = Article.get @art.id
+      foundart.title.should == "All About Getting"
+    end
+  end
+
+  describe "getting a model with a subobjects array" do
+    before(:all) do
+      course_doc = {
+        "title" => "Metaphysics 200",
+        "questions" => [
+          {
+            "q" => "Carve the ___ of reality at the ___.",
+            "a" => ["beast","joints"]
+          },{
+            "q" => "Who layed the smack down on Leibniz's Law?",
+            "a" => "Willard Van Orman Quine"
+          }
+        ]
+      }
+      r = Course.database.save_doc course_doc
+      @course = Course.get r['id']
+    end
+    it "should load the course" do
+      @course.title.should == "Metaphysics 200"
+    end
+    it "should instantiate them as such" do
+      @course["questions"][0].a[0].should == "beast"
+    end
+  end
+  
+  describe "finding all instances of a model" do
+    before(:all) do
+      WithTemplateAndUniqueID.all.map{|o| o.destroy(true)}
+      WithTemplateAndUniqueID.database.bulk_delete
+      WithTemplateAndUniqueID.new('important-field' => '1').save
+      WithTemplateAndUniqueID.new('important-field' => '2').save
+      WithTemplateAndUniqueID.new('important-field' => '3').save
+      WithTemplateAndUniqueID.new('important-field' => '4').save
+    end
+    it "should make the design doc" do
+      WithTemplateAndUniqueID.all
+      d = WithTemplateAndUniqueID.design_doc
+      d['views']['all']['map'].should include('WithTemplateAndUniqueID')
+    end
+    it "should find all" do
+      rs = WithTemplateAndUniqueID.all 
+      rs.length.should == 4
+    end
+  end
+
+  describe "finding the first instance of a model" do
+    before(:each) do      
+      @db = reset_test_db!
+      WithTemplateAndUniqueID.new('important-field' => '1').save
+      WithTemplateAndUniqueID.new('important-field' => '2').save
+      WithTemplateAndUniqueID.new('important-field' => '3').save
+      WithTemplateAndUniqueID.new('important-field' => '4').save
+    end
+    it "should make the design doc" do
+      WithTemplateAndUniqueID.all
+      d = WithTemplateAndUniqueID.design_doc
+      d['views']['all']['map'].should include('WithTemplateAndUniqueID')
+    end
+    it "should find first" do
+      rs = WithTemplateAndUniqueID.first
+      rs['important-field'].should == "1"
+    end
+    it "should return nil if no instances are found" do
+      WithTemplateAndUniqueID.all.each {|obj| obj.destroy }
+      WithTemplateAndUniqueID.first.should be_nil
+    end
+  end
+  
+  describe "getting a model with a subobject field" do
+    before(:all) do
+      course_doc = {
+        "title" => "Metaphysics 410",
+        "professor" => {
+          "name" => ["Mark", "Hinchliff"]
+        },
+        "final_test_at" => "2008/12/19 13:00:00 +0800"
+      }
+      r = Course.database.save_doc course_doc
+      @course = Course.get r['id']
+    end
+    it "should load the course" do
+      @course["professor"]["name"][1].should == "Hinchliff"
+    end
+    it "should instantiate the professor as a person" do
+      @course['professor'].last_name.should == "Hinchliff"
+    end
+    it "should instantiate the final_test_at as a Time" do
+      @course['final_test_at'].should == Time.parse("2008/12/19 13:00:00 +0800")
+    end
+  end
+  
   describe "timestamping" do
+    before(:each) do
+      oldart = Article.get "saving-this" rescue nil
+      oldart.destroy if oldart
+      @art = Article.new(:title => "Saving this")
+      @art.save
+    end
+    
     it "should define the updated_at and created_at getters and set the values" do
       @obj.save
       obj = WithDefaultValues.get(@obj.id)
@@ -76,9 +285,18 @@ describe "ExtendedDocument" do
       obj.updated_at.should be_an_instance_of(Time)
       obj.created_at.to_s.should == @obj.updated_at.to_s
     end 
+    it "should set the time on create" do
+      (Time.now - @art.created_at).should < 2
+      foundart = Article.get @art.id
+      foundart.created_at.should == foundart.updated_at
+    end
+    it "should set the time on update" do
+      @art.save
+      @art.created_at.should < @art.updated_at
+    end
   end
   
-  describe "saving and retrieving" do
+  describe "basic saving and retrieving" do
     it "should work fine" do
       @obj.name = "should be easily saved and retrieved"
       @obj.save
@@ -96,6 +314,145 @@ describe "ExtendedDocument" do
     end
   end
   
+  describe "saving a model" do
+    before(:all) do
+      @sobj = Basic.new
+      @sobj.save.should == true
+    end
+    
+    it "should save the doc" do
+      doc = Basic.get(@sobj.id)
+      doc['_id'].should == @sobj.id
+    end
+    
+    it "should be set for resaving" do
+      rev = @obj.rev
+      @sobj['another-key'] = "some value"
+      @sobj.save
+      @sobj.rev.should_not == rev
+    end
+    
+    it "should set the id" do
+      @sobj.id.should be_an_instance_of(String)
+    end
+    
+    it "should set the type" do
+      @sobj['couchrest-type'].should == 'Basic'
+    end
+  end
+  
+  describe "saving a model with a unique_id configured" do
+    before(:each) do
+      @art = Article.new
+      @old = Article.database.get('this-is-the-title') rescue nil
+      Article.database.delete_doc(@old) if @old
+    end
+    
+    it "should be a new document" do
+      @art.should be_a_new_document
+      @art.title.should be_nil
+    end
+    
+    it "should require the title" do
+      lambda{@art.save}.should raise_error
+      @art.title = 'This is the title'
+      @art.save.should == true
+    end
+    
+    it "should not change the slug on update" do
+      @art.title = 'This is the title'
+      @art.save.should == true
+      @art.title = 'new title'
+      @art.save.should == true
+      @art.slug.should == 'this-is-the-title'
+    end
+    
+    it "should raise an error when the slug is taken" do
+      @art.title = 'This is the title'
+      @art.save.should == true
+      @art2 = Article.new(:title => 'This is the title!')
+      lambda{@art2.save}.should raise_error
+    end
+    
+    it "should set the slug" do
+      @art.title = 'This is the title'
+      @art.save.should == true
+      @art.slug.should == 'this-is-the-title'
+    end
+    
+    it "should set the id" do
+      @art.title = 'This is the title'
+      @art.save.should == true
+      @art.id.should == 'this-is-the-title'
+    end
+  end
+
+  describe "saving a model with a unique_id lambda" do
+    before(:each) do
+      @templated = WithTemplateAndUniqueID.new
+      @old = WithTemplateAndUniqueID.get('very-important') rescue nil
+      @old.destroy if @old
+    end
+    
+    it "should require the field" do
+      lambda{@templated.save}.should raise_error
+      @templated['important-field'] = 'very-important'
+      @templated.save.should == true
+    end
+    
+    it "should save with the id" do
+      @templated['important-field'] = 'very-important'
+      @templated.save.should == true
+      t = WithTemplateAndUniqueID.get('very-important')
+      t.should == @templated
+    end
+    
+    it "should not change the id on update" do
+      @templated['important-field'] = 'very-important'
+      @templated.save.should == true
+      @templated['important-field'] = 'not-important'
+      @templated.save.should == true
+      t = WithTemplateAndUniqueID.get('very-important')
+      t.should == @templated
+    end
+    
+    it "should raise an error when the id is taken" do
+      @templated['important-field'] = 'very-important'
+      @templated.save.should == true
+      lambda{WithTemplateAndUniqueID.new('important-field' => 'very-important').save}.should raise_error
+    end
+    
+    it "should set the id" do
+      @templated['important-field'] = 'very-important'
+      @templated.save.should == true
+      @templated.id.should == 'very-important'
+    end
+  end
+  
+  describe "destroying an instance" do
+    before(:each) do
+      @dobj = Basic.new
+      @dobj.save.should == true
+    end
+    it "should return true" do
+      result = @dobj.destroy
+      result.should == true
+    end
+    it "should be resavable" do
+      pending "TO FIX" do
+        @dobj.destroy
+        @dobj.rev.should be_nil
+        @dobj.id.should be_nil
+        @dobj.save.should == true
+      end
+    end
+    it "should make it go away" do
+      @dobj.destroy
+      lambda{Basic.get(@dobj.id)}.should raise_error
+    end
+  end
+  
+  
   describe "callbacks" do
     
     before(:each) do
@@ -103,28 +460,11 @@ describe "ExtendedDocument" do
     end
     
     describe "save" do
-      it "should not run the before filter before saving if the save failed" do
-        @doc.run_before_save.should be_nil
-        @doc.save.should be_true
-        @doc.run_before_save.should be_true
-      end
-      it "should not run the before filter before saving if the save failed" do
-        @doc.should_receive(:save).and_return(false)
-        @doc.run_before_save.should be_nil
-        @doc.save.should be_false
-        @doc.run_before_save.should be_nil
-      end
       it "should run the after filter after saving" do
         @doc.run_after_save.should be_nil
         @doc.save.should be_true
         @doc.run_after_save.should be_true
       end
-      it "should not run the after filter before saving if the save failed" do
-        @doc.should_receive(:save).and_return(false)
-        @doc.run_after_save.should be_nil
-        @doc.save.should be_false
-        @doc.run_after_save.should be_nil
-      end
     end
     describe "create" do
       it "should run the before save filter when creating" do
@@ -132,14 +472,6 @@ describe "ExtendedDocument" do
         @doc.create.should_not be_nil
         @doc.run_before_save.should be_true
       end
-      it "should not run the before save filter when the object creation fails" do
-        pending "need to ask wycats about chainable callbacks" do
-          @doc.should_receive(:create_without_callbacks).and_return(false)
-          @doc.run_before_save.should be_nil
-          @doc.save
-          @doc.run_before_save.should be_nil
-        end
-      end
       it "should run the before create filter" do
         @doc.run_before_create.should be_nil
         @doc.create.should_not be_nil

data/spec/couchrest/more/extended_doc_view_spec.rb

@@ -0,0 +1,204 @@
+require File.dirname(__FILE__) + '/../../spec_helper'
+require File.join(FIXTURE_PATH, 'more', 'article')
+require File.join(FIXTURE_PATH, 'more', 'course')
+
+describe "ExtendedDocument views" do
+  
+  describe "a model with simple views and a default param" do
+    before(:all) do
+      Article.all.map{|a| a.destroy(true)}
+      Article.database.bulk_delete
+      written_at = Time.now - 24 * 3600 * 7
+      @titles = ["this and that", "also interesting", "more fun", "some junk"]
+      @titles.each do |title|
+        a = Article.new(:title => title)
+        a.date = written_at
+        a.save
+        written_at += 24 * 3600
+      end
+    end
+
+    it "should have a design doc" do
+      Article.design_doc["views"]["by_date"].should_not be_nil
+    end
+  
+    it "should save the design doc" do
+      Article.by_date #rescue nil
+      doc = Article.database.get Article.design_doc.id
+      doc['views']['by_date'].should_not be_nil
+    end
+  
+    it "should return the matching raw view result" do
+      view = Article.by_date :raw => true
+      view['rows'].length.should == 4
+    end
+  
+    it "should not include non-Articles" do
+      Article.database.save_doc({"date" => 1})
+      view = Article.by_date :raw => true
+      view['rows'].length.should == 4
+    end
+  
+    it "should return the matching objects (with default argument :descending => true)" do
+      articles = Article.by_date
+      articles.collect{|a|a.title}.should == @titles.reverse
+    end
+  
+    it "should allow you to override default args" do
+      articles = Article.by_date :descending => false
+      articles.collect{|a|a.title}.should == @titles
+    end
+  end
+
+  describe "another model with a simple view" do
+    before(:all) do
+      reset_test_db!
+      %w{aaa bbb ddd eee}.each do |title|
+        Course.new(:title => title).save
+      end
+    end
+    it "should make the design doc upon first query" do
+      Course.by_title 
+      doc = Course.design_doc
+      doc['views']['all']['map'].should include('Course')
+    end
+    it "should can query via view" do
+      # register methods with method-missing, for local dispatch. method
+      # missing lookup table, no heuristics.
+      view = Course.view :by_title
+      designed = Course.by_title
+      view.should == designed
+    end
+    it "should get them" do
+      rs = Course.by_title 
+      rs.length.should == 4
+    end
+    it "should yield" do
+      courses = []
+      rs = Course.by_title # remove me
+      Course.view(:by_title) do |course|
+        courses << course
+      end
+      courses[0]["doc"]["title"].should =='aaa'
+    end
+  end
+
+
+  describe "a ducktype view" do
+    before(:all) do
+      @id = TEST_SERVER.default_database.save_doc({:dept => true})['id']
+    end
+    it "should setup" do
+      duck = Course.get(@id) # from a different db
+      duck["dept"].should == true
+    end
+    it "should make the design doc" do
+      @as = Course.by_dept
+      @doc = Course.design_doc
+      @doc["views"]["by_dept"]["map"].should_not include("couchrest")
+    end
+    it "should not look for class" do |variable|
+      @as = Course.by_dept
+      @as[0]['_id'].should == @id
+    end
+  end
+
+  describe "a model with a compound key view" do
+    before(:all) do
+      written_at = Time.now - 24 * 3600 * 7
+      @titles = ["uniq one", "even more interesting", "less fun", "not junk"]
+      @user_ids = ["quentin", "aaron"]
+      @titles.each_with_index do |title,i|
+        u = i % 2
+        a = Article.new(:title => title, :user_id => @user_ids[u])
+        a.date = written_at
+        a.save
+        written_at += 24 * 3600
+      end
+    end
+    it "should create the design doc" do
+      Article.by_user_id_and_date rescue nil
+      doc = Article.design_doc
+      doc['views']['by_date'].should_not be_nil
+    end
+    it "should sort correctly" do
+      articles = Article.by_user_id_and_date
+      articles.collect{|a|a['user_id']}.should == ['aaron', 'aaron', 'quentin', 
+        'quentin']
+      articles[1].title.should == 'not junk'
+    end
+    it "should be queryable with couchrest options" do
+      articles = Article.by_user_id_and_date :limit => 1, :startkey => 'quentin'
+      articles.length.should == 1
+      articles[0].title.should == "even more interesting"
+    end
+  end
+
+  describe "with a custom view" do
+    before(:all) do
+      @titles = ["very uniq one", "even less interesting", "some fun", 
+        "really junk", "crazy bob"]
+      @tags = ["cool", "lame"]
+      @titles.each_with_index do |title,i|
+        u = i % 2
+        a = Article.new(:title => title, :tags => [@tags[u]])
+        a.save
+      end
+    end
+    it "should be available raw" do
+      view = Article.by_tags :raw => true
+      view['rows'].length.should == 5
+    end
+
+    it "should be default to :reduce => false" do
+      ars = Article.by_tags
+      ars.first.tags.first.should == 'cool'
+    end
+  
+    it "should be raw when reduce is true" do
+      view = Article.by_tags :reduce => true, :group => true
+      view['rows'].find{|r|r['key'] == 'cool'}['value'].should == 3
+    end
+  end
+
+  # TODO: moved to Design, delete
+  describe "adding a view" do
+    before(:each) do
+      Article.by_date
+      @design_docs = Article.database.documents :startkey => "_design/", 
+        :endkey => "_design/\u9999"
+    end
+    it "should not create a design doc on view definition" do
+      Article.view_by :created_at
+      newdocs = Article.database.documents :startkey => "_design/", 
+        :endkey => "_design/\u9999"
+      newdocs["rows"].length.should == @design_docs["rows"].length
+    end
+    it "should create a new design document on view access" do
+      Article.view_by :updated_at
+      Article.by_updated_at
+      newdocs = Article.database.documents :startkey => "_design/", 
+        :endkey => "_design/\u9999"
+      # puts @design_docs.inspect
+      # puts newdocs.inspect
+      newdocs["rows"].length.should == @design_docs["rows"].length + 1
+    end
+  end
+
+  describe "with a lot of designs left around" do
+    before(:each) do
+      Article.by_date
+      Article.view_by :field
+      Article.by_field
+    end
+    it "should clean them up" do
+      Article.view_by :stream
+      Article.by_stream
+      ddocs = Article.all_design_doc_versions
+      ddocs["rows"].length.should > 1
+      Article.cleanup_design_docs!
+      ddocs = Article.all_design_doc_versions
+      ddocs["rows"].length.should == 1
+    end
+  end
+end

data/spec/couchrest/more/property_spec.rb

@@ -1,7 +1,8 @@
 require File.join(File.dirname(__FILE__), '..', '..', 'spec_helper')
 require File.join(FIXTURE_PATH, 'more', 'card')
 require File.join(FIXTURE_PATH, 'more', 'invoice')
-require File.join(FIXTURE_PATH, 'more', 'service.rb')
+require File.join(FIXTURE_PATH, 'more', 'service')
+require File.join(FIXTURE_PATH, 'more', 'event')
 
 
 describe "ExtendedDocument properties" do
@@ -43,7 +44,6 @@ describe "ExtendedDocument properties" do
   end
   
   describe "validation" do
-    
     before(:each) do
       @invoice = Invoice.new(:client_name => "matt", :employee_name => "Chris", :location => "San Diego, CA")
     end
@@ -79,11 +79,9 @@ describe "ExtendedDocument properties" do
       @invoice.save.should be_false
       @invoice.should be_new_document
     end
-  
   end
   
   describe "autovalidation" do
-    
     before(:each) do
       @service = Service.new(:name => "Coumpound analysis", :price => 3_000)
     end
@@ -112,7 +110,20 @@ describe "ExtendedDocument properties" do
         @service.errors.on(:name).first.should == "Name must be between 4 and 19 characters long"
       end
     end
-    
+  end
+  
+  describe "casting" do
+    describe "cast keys to any type" do
+      before(:all) do
+        event_doc = { :subject => "Some event", :occurs_at => Time.now }
+        e = Event.database.save_doc event_doc
+
+        @event = Event.get e['id']
+      end
+      it "should cast created_at to Time" do
+        @event['occurs_at'].should be_an_instance_of(Time)
+      end
+    end
   end
   
 end

data/spec/spec_helper.rb

@@ -14,6 +14,10 @@ unless defined?(FIXTURE_PATH)
   TEST_SERVER.default_database = TESTDB
 end
 
+class Basic < CouchRest::ExtendedDocument
+  use_database TEST_SERVER.default_database
+end
+
 def reset_test_db!
   cr = TEST_SERVER
   db = cr.database(TESTDB)