couchrest_model 1.0.0.beta7

data/spec/couchrest/subclass_spec.rb

@@ -0,0 +1,99 @@
+require File.expand_path("../../spec_helper", __FILE__)
+require File.join(FIXTURE_PATH, 'more', 'cat')
+require File.join(FIXTURE_PATH, 'more', 'person')
+require File.join(FIXTURE_PATH, 'more', 'card')
+require File.join(FIXTURE_PATH, 'more', 'course')
+
+# add a default value
+Card.property :bg_color, :default => '#ccc'
+
+class BusinessCard < Card
+  property :extension_code
+  property :job_title
+
+  validates_presence_of :extension_code
+  validates_presence_of :job_title
+end
+
+class DesignBusinessCard < BusinessCard
+  property :bg_color, :default => '#eee'
+end
+
+class OnlineCourse < Course
+  property :url
+  view_by :url
+end
+
+class Animal < CouchRest::Model::Base
+  use_database TEST_SERVER.default_database
+  property :name
+  view_by :name
+end
+
+class Dog < Animal; end
+
+describe "Subclassing a Model" do
+  
+  before(:each) do
+    @card = BusinessCard.new
+  end
+  
+  it "shouldn't messup the parent's properties" do
+    Card.properties.should_not == BusinessCard.properties
+  end
+  
+  it "should share the same db default" do
+    @card.database.uri.should == Card.database.uri
+  end
+  
+  it "should have kept the validation details" do
+    @card.should_not be_valid
+  end
+  
+  it "should have added the new validation details" do
+    validated_fields = @card.class.validators.map{|v| v.attributes}.flatten
+    validated_fields.should include(:extension_code)
+    validated_fields.should include(:job_title)
+  end
+  
+  it "should not add to the parent's validations" do
+    validated_fields = Card.validators.map{|v| v.attributes}.flatten
+    validated_fields.should_not include(:extension_code)
+    validated_fields.should_not include(:job_title) 
+  end
+  
+  it "should inherit default property values" do
+    @card.bg_color.should == '#ccc'
+  end
+  
+  it "should be able to overwrite a default property" do
+    DesignBusinessCard.new.bg_color.should == '#eee'
+  end
+  
+  it "should have a design doc slug based on the subclass name" do
+    Course.refresh_design_doc
+    OnlineCourse.design_doc_slug.should =~ /^OnlineCourse/
+  end
+  
+  it "should have its own design_doc_fresh" do
+    Animal.refresh_design_doc
+    Dog.send(:design_doc_fresh, Dog.database).should_not == true
+    Dog.refresh_design_doc
+    Dog.send(:design_doc_fresh, Dog.database).should == true
+  end
+  
+  it "should not add views to the parent's design_doc" do
+    Course.design_doc['views'].keys.should_not include('by_url')
+  end
+  
+  it "should not add the parent's views to its design doc" do
+    Course.refresh_design_doc
+    OnlineCourse.refresh_design_doc
+    OnlineCourse.design_doc['views'].keys.should_not include('by_title')
+  end
+  
+  it "should have an all view with a guard clause for couchrest-type == subclass name in the map function" do
+    OnlineCourse.design_doc['views']['all']['map'].should =~ /if \(doc\['couchrest-type'\] == 'OnlineCourse'\)/
+  end
+end
+

data/spec/couchrest/validations.rb

@@ -0,0 +1,73 @@
+require File.expand_path("../../spec_helper", __FILE__)
+
+require File.join(FIXTURE_PATH, 'more', 'cat')
+require File.join(FIXTURE_PATH, 'more', 'article')
+require File.join(FIXTURE_PATH, 'more', 'course')
+require File.join(FIXTURE_PATH, 'more', 'card')
+require File.join(FIXTURE_PATH, 'base')
+
+# TODO Move validations from other specs to here
+
+describe "Validations" do
+
+  describe "Uniqueness" do
+
+    before(:all) do
+      @objs = ['title 1', 'title 2', 'title 3'].map{|t| WithUniqueValidation.create(:title => t)}
+    end
+    
+    it "should validate a new unique document" do
+      @obj = WithUniqueValidation.create(:title => 'title 4')
+      @obj.new?.should_not be_true
+      @obj.should be_valid
+    end
+
+    it "should not validate a non-unique document" do
+      @obj = WithUniqueValidation.create(:title => 'title 1')
+      @obj.should_not be_valid
+      @obj.errors[:title].should eql(['is already taken'])
+    end
+
+    it "should save already created document" do
+      @obj = @objs.first
+      @obj.save.should_not be_false
+      @obj.should be_valid
+    end
+
+    it "should allow own view to be specified" do
+      # validates_uniqueness_of :code, :view => 'all'
+      WithUniqueValidationView.create(:title => 'title 1', :code => '1234')
+      @obj = WithUniqueValidationView.new(:title => 'title 5', :code => '1234')
+      @obj.should_not be_valid
+    end
+
+    it "should raise an error if specified view does not exist" do
+      WithUniqueValidationView.validates_uniqueness_of :title, :view => 'fooobar'
+      @obj = WithUniqueValidationView.new(:title => 'title 2', :code => '12345')
+      lambda {
+        @obj.valid?
+      }.should raise_error
+    end
+
+    context "with a pre-defined view" do
+      it "should not try to create new view" do
+        @obj = @objs[1]
+        @obj.class.should_not_receive('view_by')
+        @obj.class.should_receive('has_view?').and_return(true)
+        @obj.class.should_receive('view').and_return({'rows' => [ ]})
+        @obj.valid?
+      end
+    end
+ 
+    context "with a proxy parameter" do
+      it "should be used" do
+        @obj = WithUniqueValidationProxy.new(:title => 'test 6')
+        proxy = @obj.should_receive('proxy').and_return(@obj.class)
+        @obj.valid?.should be_true
+      end
+    end
+
+ 
+  end
+
+end

data/spec/couchrest/view_spec.rb

@@ -0,0 +1,463 @@
+require File.expand_path("../../spec_helper", __FILE__)
+require File.join(FIXTURE_PATH, 'more', 'cat')
+require File.join(FIXTURE_PATH, 'more', 'person')
+require File.join(FIXTURE_PATH, 'more', 'article')
+require File.join(FIXTURE_PATH, 'more', 'course')
+
+describe "Model views" do
+
+  class Unattached < CouchRest::Model::Base
+    # Note: no use_database here
+    property :title
+    property :questions
+    property :professor
+    view_by :title
+  end
+ 
+
+  describe "ClassMethods" do
+    # NOTE! Add more unit tests!
+
+    describe "#view" do
+    
+      it "should not alter original query" do
+        options = { :database => DB }
+        view = Article.view('by_date', options)
+        options[:database].should_not be_nil
+      end
+
+    end
+  end
+
+  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 save design doc if a view changed" do
+      Article.by_date
+      orig = Article.stored_design_doc
+      orig['views']['by_date']['map'] = "function() { }"
+      Article.database.save_doc(orig)
+      rev = Article.stored_design_doc['_rev']
+      Article.req_design_doc_refresh # prepare for re-load
+      Article.by_date
+      orig = Article.stored_design_doc
+      orig['views']['by_date']['map'].should eql(Article.design_doc['views']['by_date']['map'])
+      orig['_rev'].should_not eql(rev)
+    end
+    it "should not save design doc if not changed" do
+      Article.by_date
+      orig = Article.stored_design_doc['_rev']
+      Article.req_design_doc_refresh
+      Article.by_date
+      Article.stored_design_doc['_rev'].should eql(orig)
+    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
+    it "should allow you to create a new view on the fly" do
+      lambda{Article.by_title}.should raise_error 
+      Article.view_by :title
+      lambda{Article.by_title}.should_not raise_error 
+    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 = []
+      Course.view(:by_title) do |course|
+        courses << course
+      end
+      courses[0]["doc"]["title"].should =='aaa'
+    end
+    it "should yield with by_key method" do
+      courses = []
+      Course.by_title do |course|
+        courses << course
+      end
+      courses[0]["doc"]["title"].should =='aaa'
+    end
+  end
+
+  describe "find a single item using a view" do
+    before(:all) do
+      reset_test_db!
+      %w{aaa bbb ddd eee}.each do |title|
+        Course.new(:title => title, :active => (title == 'bbb')).save
+      end
+    end
+
+    it "should return single matched record with find helper" do
+      course = Course.find_by_title('bbb')
+      course.should_not be_nil
+      course.title.should eql('bbb') # Ensure really is a Course!
+    end
+
+    it "should return nil if not found" do
+      course = Course.find_by_title('fff')
+      course.should be_nil
+    end
+
+    it "should peform search on view with two properties" do
+      course = Course.find_by_title_and_active(['bbb', true])
+      course.should_not be_nil
+      course.title.should eql('bbb') # Ensure really is a Course!
+    end
+
+    it "should return nil if not found" do
+      course = Course.find_by_title_and_active(['bbb', false])
+      course.should be_nil
+    end
+
+    it "should raise exception if view not present" do
+      lambda { Course.find_by_foobar('123') }.should raise_error(NoMethodError)
+    end
+
+    it "should perform a search directly with specific key" do
+      course = Course.first_from_view('by_title', 'bbb')
+      course.title.should eql('bbb')
+    end
+
+    it "should perform a search directly with specific key with options" do
+      course = Course.first_from_view('by_title', 'bbb', :reverse => true)
+      course.title.should eql('bbb')
+    end
+
+    it "should perform a search directly with range" do
+      course = Course.first_from_view('by_title', :startkey => 'bbb', :endkey => 'eee')
+      course.title.should eql('bbb')
+    end
+
+  end
+  
+  describe "a ducktype view" do
+    before(:all) do
+      reset_test_db!
+      @id = DB.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
+      @as = Course.by_dept
+      @as[0]['_id'].should == @id
+    end
+  end
+  
+  describe "a model class not tied to a database" do
+    before(:all) do
+      reset_test_db!
+      @db = DB 
+      %w{aaa bbb ddd eee}.each do |title|
+        u = Unattached.new(:title => title)
+        u.database = @db
+        u.save
+        @first_id ||= u.id
+      end
+    end
+    it "should barf on all if no database given" do
+      lambda{Unattached.all}.should raise_error
+    end
+    it "should query all" do
+      # Unattached.cleanup_design_docs!(@db)
+      rs = Unattached.all :database => @db
+      rs.length.should == 4
+    end
+    it "should barf on query if no database given" do
+      lambda{Unattached.view :by_title}.should raise_error
+    end
+    it "should make the design doc upon first query" do
+      Unattached.by_title :database => @db
+      doc = Unattached.design_doc
+      doc['views']['all']['map'].should include('Unattached')
+    end
+    it "should merge query params" do
+      rs = Unattached.by_title :database=>@db, :startkey=>"bbb", :endkey=>"eee"
+      rs.length.should == 3
+    end
+    it "should query via view" do
+      view = Unattached.view :by_title, :database=>@db
+      designed = Unattached.by_title :database=>@db
+      view.should == designed
+    end
+    it "should yield" do
+      things = []
+      Unattached.view(:by_title, :database=>@db) do |thing|
+        things << thing
+      end 
+      things[0]["doc"]["title"].should =='aaa'
+    end
+    it "should yield with by_key method" do
+      things = []
+      Unattached.by_title(:database=>@db) do |thing|
+        things << thing
+      end
+      things[0]["doc"]["title"].should =='aaa'
+    end
+    it "should return nil on get if no database given" do
+      Unattached.get("aaa").should be_nil
+    end
+    it "should barf on get! if no database given" do
+      lambda{Unattached.get!("aaa")}.should raise_error
+    end
+    it "should get from specific database" do
+      u = Unattached.get(@first_id, @db)
+      u.title.should == "aaa"
+    end
+    it "should barf on first if no database given" do
+      lambda{Unattached.first}.should raise_error
+    end
+    it "should get first" do
+      u = Unattached.first :database=>@db
+      u.title.should =~ /\A...\z/
+    end
+    it "should barf on all_design_doc_versions if no database given" do
+      lambda{Unattached.all_design_doc_versions}.should raise_error
+    end
+    it "should be able to cleanup the db/bump the revision number" do
+      # if the previous specs were not run, the model_design_doc will be blank
+      Unattached.use_database DB
+      Unattached.view_by :questions
+      Unattached.by_questions(:database => @db)
+      original_revision = Unattached.model_design_doc(@db)['_rev']
+      Unattached.save_design_doc!(@db)
+      Unattached.model_design_doc(@db)['_rev'].should_not == original_revision
+    end
+  end
+  
+  describe "a model with a compound key view" do
+    before(:all) do
+      Article.by_user_id_and_date.each{|a| a.destroy(true)}
+      Article.database.bulk_delete
+      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
+      reset_test_db!
+      Article.by_date
+      @original_doc_rev = Article.model_design_doc['_rev']
+      @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 version of the design document on view access" do
+      ddocs = Article.all_design_doc_versions["rows"].length
+      Article.view_by :updated_at
+      Article.by_updated_at
+      @original_doc_rev.should_not == Article.model_design_doc['_rev']
+      Article.design_doc["views"].keys.should include("by_updated_at")
+    end
+  end
+  
+  describe "with a collection" do
+    before(:all) do
+      reset_test_db!
+      titles = ["very uniq one", "really interesting", "some fun",
+        "really awesome", "crazy bob", "this rocks", "super rad"]
+      titles.each_with_index do |title,i|
+        a = Article.new(:title => title, :date => Date.today)
+        a.save
+      end
+      
+      titles = ["yesterday very uniq one", "yesterday really interesting", "yesterday some fun",
+        "yesterday really awesome", "yesterday crazy bob", "yesterday this rocks"]
+      titles.each_with_index do |title,i|
+        a = Article.new(:title => title, :date => Date.today - 1)
+        a.save
+      end
+    end 
+    require 'date'
+    it "should return a proxy that looks like an array of 7 Article objects" do
+      articles = Article.by_date :key => Date.today
+      articles.class.should == Array
+      articles.size.should == 7
+    end
+    it "should get a subset of articles using paginate" do
+      articles = Article.by_date :key => Date.today
+      articles.paginate(:page => 1, :per_page => 3).size.should == 3
+      articles.paginate(:page => 2, :per_page => 3).size.should == 3
+      articles.paginate(:page => 3, :per_page => 3).size.should == 1
+    end
+    it "should get all articles, a few at a time, using paginated each" do
+      articles = Article.by_date :key => Date.today
+      articles.paginated_each(:per_page => 3) do |a|
+        a.should_not be_nil
+      end
+    end 
+    it "should provide a class method to access the collection directly" do
+      articles = Article.collection_proxy_for('Article', 'by_date', :descending => true,
+        :key => Date.today, :include_docs => true)
+      articles.class.should == Array
+      articles.size.should == 7
+    end
+    it "should provide a class method for paginate" do
+      articles = Article.paginate(:design_doc => 'Article', :view_name => 'by_date',
+        :per_page => 3, :descending => true, :key => Date.today, :include_docs => true)
+      articles.size.should == 3
+      
+      articles = Article.paginate(:design_doc => 'Article', :view_name => 'by_date',
+        :per_page => 3, :page => 2, :descending => true, :key => Date.today, :include_docs => true)
+      articles.size.should == 3
+      
+      articles = Article.paginate(:design_doc => 'Article', :view_name => 'by_date',
+        :per_page => 3, :page => 3, :descending => true, :key => Date.today, :include_docs => true)
+      articles.size.should == 1
+    end
+    it "should provide a class method for paginated_each" do
+      options = { :design_doc => 'Article', :view_name => 'by_date',
+        :per_page => 3, :page => 1, :descending => true, :key => Date.today,
+        :include_docs => true }
+      Article.paginated_each(options) do |a|
+        a.should_not be_nil
+      end
+    end
+    it "should provide a class method to get a collection for a view" do
+      articles = Article.find_all_article_details(:key => Date.today)
+      articles.class.should == Array
+      articles.size.should == 7
+    end
+    it "should raise an exception if design_doc is not provided" do
+      lambda{Article.collection_proxy_for(nil, 'by_date')}.should raise_error
+      lambda{Article.paginate(:view_name => 'by_date')}.should raise_error
+    end
+    it "should raise an exception if view_name is not provided" do
+      lambda{Article.collection_proxy_for('Article', nil)}.should raise_error
+      lambda{Article.paginate(:design_doc => 'Article')}.should raise_error
+    end
+    it "should be able to span multiple keys" do
+      articles = Article.by_date :startkey => Date.today, :endkey => Date.today - 1
+      articles.paginate(:page => 1, :per_page => 3).size.should == 3
+      articles.paginate(:page => 2, :per_page => 3).size.should == 3
+      articles.paginate(:page => 3, :per_page => 3).size.should == 3
+      articles.paginate(:page => 4, :per_page => 3).size.should == 3
+      articles.paginate(:page => 5, :per_page => 3).size.should == 1
+    end
+    it "should pass database parameter to pager" do
+      proxy = mock(:proxy)
+      proxy.stub!(:paginate)
+      ::CouchRest::Model::Collection::CollectionProxy.should_receive(:new).with('database', anything(), anything(), anything(), anything()).and_return(proxy)
+      Article.paginate(:design_doc => 'Article', :view_name => 'by_date', :database => 'database')
+    end
+  end
+
+end