couch_potato 0.6.0 → 0.7.0.pre.1

data/spec/property_spec.rb

@@ -4,16 +4,19 @@ require 'fixtures/person'
 
 class Watch
   include CouchPotato::Persistence
-  
+
   property :time, :type => Time
   property :date, :type => Date
+  property :custom, :type => [String]
+  property :custom_date, :type => [Date]
+  property :custom_address, :type => [Address]
   property :overwritten_read
   property :overwritten_write
-  
+
   def overwritten_read
     super.to_s
   end
-  
+
   def overwritten_write=(value)
     super value.to_s
   end
@@ -27,40 +30,40 @@ describe 'properties' do
   before(:all) do
     recreate_db
   end
-  
+
   it "should allow me to overwrite read accessor and call super" do
     Watch.new(:overwritten_read => 1).overwritten_read.should == '1'
   end
-  
+
   it "should allow me to overwrite write accessor and call super" do
     Watch.new(:overwritten_write => 1).overwritten_write.should == '1'
   end
-  
+
   it "should return the property names" do
     Comment.property_names.should == [:created_at, :updated_at, :title]
   end
-  
+
   it "should persist a string" do
     c = Comment.new :title => 'my title'
     CouchPotato.database.save_document! c
     c = CouchPotato.database.load_document c.id
     c.title.should == 'my title'
   end
-  
+
   it "should persist a number" do
     c = Comment.new :title => 3
     CouchPotato.database.save_document! c
     c = CouchPotato.database.load_document c.id
     c.title.should == 3
   end
-  
+
   it "should persist a hash" do
     c = Comment.new :title => {'key' => 'value'}
     CouchPotato.database.save_document! c
     c = CouchPotato.database.load_document c.id
     c.title.should == {'key' => 'value'}
   end
-  
+
   def it_should_persist value
     c = Comment.new :title => value
     CouchPotato.database.save_document! c
@@ -99,7 +102,7 @@ describe 'properties' do
     ]
     it_should_persist something_very_complex
   end
-  
+
   it "should persist an object" do
     p = Person.new
     a = Address.new :city => 'Denver'
@@ -108,7 +111,7 @@ describe 'properties' do
     p = CouchPotato.database.load_document p.id
     p.ship_address.should === a
   end
-  
+
   it "should persist null for a null " do
     p = Person.new
     p.ship_address = nil
@@ -116,7 +119,7 @@ describe 'properties' do
     p = CouchPotato.database.load_document p.id
     p.ship_address.should be_nil
   end
-  
+
   it "should actually pass the null value down in the JSON document " do
     p = Person.new
     p.ship_address = nil
@@ -135,7 +138,7 @@ describe 'properties' do
     p = CouchPotato.database.load_document p.id
     p.ship_address.should be_false
   end
-  
+
   describe "time properties" do
     it "should persist a Time as utc" do
       time = Time.now
@@ -144,7 +147,7 @@ describe 'properties' do
       w = CouchPotato.database.load_document w.id
       w.time.to_s.should == time.utc.to_s
     end
-    
+
     it "should parse a string and persist it as utc time" do
       w = Watch.new :time => '2009-01-01 13:25 +0100'
       CouchPotato.database.save_document! w
@@ -152,14 +155,14 @@ describe 'properties' do
       w.time.should be_a(Time)
       w.time.should == Time.parse('2009-01-01 12:25 +0000')
     end
-    
+
     it "should store nil" do
       w = Watch.new :time => nil
       CouchPotato.database.save_document! w
       w = CouchPotato.database.load_document w.id
       w.time.should be_nil
     end
-    
+
     it "should store an empty string as nil" do
       w = Watch.new :time => ''
       CouchPotato.database.save_document! w
@@ -167,7 +170,7 @@ describe 'properties' do
       w.time.should be_nil
     end
   end
-  
+
   describe "date properties" do
     it "should persist a date" do
       date = Date.today
@@ -176,21 +179,21 @@ describe 'properties' do
       w = CouchPotato.database.load_document w.id
       w.date.should == date
     end
-    
+
     it "should parse a string and persist it as a date" do
       w = Watch.new :date => '2009-01-10'
       CouchPotato.database.save_document! w
       w = CouchPotato.database.load_document w.id
       w.date.should == Date.parse('2009-01-10')
     end
-    
+
     it "should store nil" do
       w = Watch.new :date => nil
       CouchPotato.database.save_document! w
       w = CouchPotato.database.load_document w.id
       w.date.should be_nil
     end
-    
+
     it "should store an empty string as nil" do
       w = Watch.new :date => ''
       CouchPotato.database.save_document! w
@@ -198,7 +201,40 @@ describe 'properties' do
       w.date.should be_nil
     end
   end
-  
+
+  describe "array properties" do
+    it "should persist an array of strings" do
+      w = Watch.new :custom => ["moin"]
+      CouchPotato.database.save_document! w
+      w = CouchPotato.database.load_document w.id
+      w.custom.should eql(["moin"])
+    end
+
+    it "should persist an array of dates" do
+      date = Date.today
+      w = Watch.new :custom_date => [date]
+      CouchPotato.database.save_document! w
+      w = CouchPotato.database.load_document w.id
+      w.custom_date.should eql([date])
+    end
+
+    it "should persist an array of nested documents" do
+      address = Address.new(:verified => 1)
+      w = Watch.new :custom_address => [address]
+      CouchPotato.database.save_document! w
+      w = CouchPotato.database.load_document w.id
+      w.custom_address.should eql([address])
+    end
+
+    it "should handle nil values" do
+      address = Address.new(:verified => 1)
+      w = Watch.new :custom_address => nil
+      CouchPotato.database.save_document! w
+      w = CouchPotato.database.load_document w.id
+      w.custom_address.should eql(nil)
+    end
+  end
+
   describe "boolean properties" do
     it "should persist '0' as false" do
       a = Address.new
@@ -207,7 +243,7 @@ describe 'properties' do
       a = CouchPotato.database.load_document a.id
       a.verified.should be_false
     end
-    
+
     it "should persist 0 as false" do
       a = Address.new
       a.verified = 0
@@ -215,7 +251,7 @@ describe 'properties' do
       a = CouchPotato.database.load_document a.id
       a.verified.should be_false
     end
-    
+
     it "should persist 'false' as false" do
       a = Address.new
       a.verified = 'false'
@@ -223,7 +259,7 @@ describe 'properties' do
       a = CouchPotato.database.load_document a.id
       a.verified.should be_false
     end
-    
+
     it "should persist '1' as true" do
       a = Address.new
       a.verified = '1'
@@ -231,7 +267,7 @@ describe 'properties' do
       a = CouchPotato.database.load_document a.id
       a.verified.should be_true
     end
-    
+
     it "should persist 1 as true" do
       a = Address.new
       a.verified = 1
@@ -239,7 +275,7 @@ describe 'properties' do
       a = CouchPotato.database.load_document a.id
       a.verified.should be_true
     end
-    
+
     it "should leave nil as nil" do
       a = Address.new
       a.verified = nil
@@ -253,33 +289,33 @@ describe 'properties' do
     it "should return true if property set" do
       Comment.new(:title => 'title').title?.should be_true
     end
-    
+
     it "should return false if property nil" do
       Comment.new.title?.should be_false
     end
-    
+
     it "should return false if property false" do
       Comment.new(:title => false).title?.should be_false
     end
-    
+
     it "should return false if property blank" do
       Comment.new(:title => '').title?.should be_false
     end
   end
-  
+
   describe "with subclasses" do
     it "should include properties of superclasses" do
       CuckooClock.properties.map(&:name).should include(:time)
       CuckooClock.properties.map(&:name).should include(:cuckoo)
     end
-    
+
     it "should return attributes of superclasses" do
       clock = CuckooClock.new(:time => Time.now, :cuckoo => 'bavarian')
-      clock.attributes[:time].should_not == nil
-      clock.attributes[:cuckoo].should == 'bavarian'
+      clock.attributes[:time].should_not be_nil
+      clock.attributes[:cuckoo].should eql('bavarian')
     end
   end
-  
+
   describe "inspecting an object" do
     let(:comment) do
       comment = Comment.new(:title => 'title')
@@ -289,35 +325,36 @@ describe 'properties' do
       end
       comment
     end
-    
+
     it "should not include change-tracking variables" do
       comment.inspect.should_not include('title_was')
     end
-    
+
     it "should include the normal persistent variables" do
       comment.inspect.should include('title: "title"')
     end
-    
+
     it "should include the id" do
       comment.inspect.should include(%Q{_id: "123456abcdef",})
     end
-    
+
     it "should include the revision" do
       comment.inspect.should include(%Q{_rev: "1-654321fedcba",})
     end
-    
+
     it "should return a complete string" do
       # stub to work around (un)sorted hash on different rubies
       comment.stub!(:attributes).and_return([['created_at', ''], ['updated_at', ''], ['title', 'title']])
-      comment.inspect.should == %Q{#<Comment _id: "123456abcdef", _rev: "1-654321fedcba", created_at: "", updated_at: "", title: "title">}
+      doc = '#<Comment _id: "123456abcdef", _rev: "1-654321fedcba", created_at: "", updated_at: "", title: "title">'
+      comment.inspect.should eql(doc)
     end
-    
+
     it "should include complex datatypes fully inspected" do
       comment.title = {'en' => 'Blog post'}
       comment.inspect.should include('title: {"en"=>"Blog post"}')
-      
+
       comment.title = nil
       comment.inspect.should include('title: nil')
     end
   end
-end
+end

data/spec/railtie_spec.rb

@@ -10,7 +10,7 @@ module Rails
     def self.initializer(*args)
     end
   end
-  
+
   def self.root
     RSpec::Mocks::Mock.new :join => ''
   end
@@ -20,46 +20,48 @@ require 'couch_potato/railtie'
 
 describe "railtie" do
   before(:all) do
-    @validation_framework = CouchPotato::Config.validation_framework
+    @database_name = CouchPotato::Config.database_name
+    @default_language = CouchPotato::Config.default_language
   end
-  
+
   after(:all) do
-    CouchPotato::Config.validation_framework = @validation_framework
+    CouchPotato::Config.database_name = @database_name
+    CouchPotato::Config.default_language = @default_language
   end
-  
+
   context 'yaml file contains only database names' do
     it "should set the database name from the yaml file" do
       File.stub(:read => "test: test_db")
-      
+
       CouchPotato::Config.should_receive(:database_name=).with('test_db')
-      
+
       CouchPotato.rails_init
     end
   end
-  
+
   context 'yaml file contains more configuration' do
     before(:each) do
-      File.stub(:read => "test: \n  database: test_db\n  validation_framework: :active_model")
+      File.stub(:read => "test: \n  database: test_db\n  default_language: :erlang")
     end
-    
-    it "should set the database name from the yaml file" do
+
+    it "set the database name from the yaml file" do
       CouchPotato::Config.should_receive(:database_name=).with('test_db')
-      
+
       CouchPotato.rails_init
     end
-    
-    it "should set the validation framework from the yaml file" do
-      CouchPotato::Config.should_receive(:validation_framework=).with(:active_model)
-      
+
+    it 'sets the default language from the yaml file' do
+      CouchPotato::Config.should_receive(:default_language=).with(:erlang)
+
       CouchPotato.rails_init
     end
   end
-  
+
   it "should process the yml file with erb" do
     File.stub(:read => "test: \n  database: <%= 'db' %>")
 
     CouchPotato::Config.should_receive(:database_name=).with('db')
-    
+
     CouchPotato.rails_init
   end
 end

data/spec/spec_helper.rb

@@ -9,7 +9,6 @@ $:.unshift(File.dirname(__FILE__) + '/../lib')
 require 'couch_potato'
 
 CouchPotato::Config.database_name = ENV['DATABASE'] || 'couch_potato_test'
-CouchPotato::Config.validation_framework = ENV['VALIDATION_FRAMEWORK'].to_sym unless ENV['VALIDATION_FRAMEWORK'].blank?
 
 # silence deprecation warnings from ActiveModel as the Spec uses Errors#on
 begin
@@ -42,3 +41,22 @@ RSpec::Matchers.define :string_matching do |regex|
     string =~ regex
   end
 end
+
+RSpec::Matchers.define :eql_ignoring_indentation do |expected|
+  match do |string|
+    strip_indentation(string) == strip_indentation(expected)
+  end
+
+  failure_message_for_should do |actual|
+    "expected\n#{strip_indentation(actual).inspect} to == \n#{strip_indentation(expected).inspect} but wasn't."
+  end
+
+  failure_message_for_should_not do |actual|
+    "expected\n#{strip_indentation(actual).inspect} to not == \n#{strip_indentation(expected).inspect} but wasn."
+  end
+
+  def strip_indentation(string)
+    string.gsub(/^\s+/m, '')
+  end
+
+end

data/spec/unit/active_model_compliance_spec.rb

@@ -2,7 +2,7 @@ require 'spec_helper'
 
 begin
   require 'active_model'
-  
+
   describe 'ActiveModel conformance of couch potato objects' do
     include ActiveModel::Lint::Tests
 
@@ -23,47 +23,53 @@ begin
     before(:each) do
       @model = ActiveComment.new
     end
-    
+
+    describe '#to_partial_path' do
+      it 'returns a path based on the class name' do
+        @model.to_partial_path.should == 'active_comments/active_comment'
+      end
+    end
+
     describe "#persisted?" do
       it "should return false if it is a new document " do
         @model.should_not be_persisted
       end
-      
+
       it "should be true if it was saved" do
         @comment = ActiveComment.new(:name => 'Thilo', :email => 'test@local.host')
         CouchPotato.database.save_document! @comment
         @comment.should be_persisted
       end
     end
-    
+
     describe "#to_key" do
       it "should return nil if the document was not persisted" do
         @model.to_key.should be_nil
       end
-      
+
       it "should return the id of the document if it was persisted" do
         @comment = ActiveComment.new(:name => 'Thilo', :email => 'test@local.host')
         CouchPotato.database.save_document! @comment
         @comment.to_key.should == [@comment.id]
       end
     end
-    
-    
+
+
     describe "#errors" do
       it "should return a single error as array" do
         @model.valid?
         @model.errors[:name].should be_kind_of(Array)
       end
-      
+
       it "should return multiple errors as array" do
         @model.valid?
         @model.errors[:email].size.should == 2
       end
-      
+
       it "should return no error as an empty array" do
         @model.errors[:name].should == []
       end
-      
+
       it "should be able to be Marshal.dump'ed" do
         lambda { Marshal.dump(@model.errors) }.should_not raise_error
       end
@@ -88,7 +94,7 @@ begin
       object.should be_a(klass)
     end
   end
-  
+
 rescue LoadError
   STDERR.puts "WARNING: active_model gem not installed. Not running ActiveModel specs."
 end

data/spec/unit/attributes_spec.rb

@@ -12,7 +12,7 @@ class Plant
   property :leaf_count
   property :typed_leaf_count, :type => Fixnum
   property :typed_leaf_size,  :type => Float
-  property :branch, type: Branch
+  property :branch, :type => Branch
 end
 
 
@@ -23,13 +23,32 @@ describe "attributes" do
       plant.attributes = {:leaf_count => 1}
       plant.leaf_count.should == 1
     end
+
+    it "should assign the attributes via []=" do
+      plant = Plant.new
+      plant[:leaf_count] = 1
+      plant.leaf_count.should == 1
+    end
   end
 
   context "attributes" do
     it "should return the attributes" do
       plant = Plant.new(:leaf_count => 1)
       plant.attributes.should == {:leaf_count => 1, :created_at => nil, :updated_at => nil,
-        :typed_leaf_count => nil, :typed_leaf_size => nil, :branch => nil}
+                                  :typed_leaf_count => nil, :typed_leaf_size => nil, :branch => nil}
+    end
+
+    it "should return the attributes via []" do
+      plant = Plant.new(:leaf_count => 1)
+      plant.attributes[:leaf_count].should eql(plant[:leaf_count])
+      plant.attributes[:leaf_count].should eql(1)
+    end
+  end
+
+  context "has_key?" do
+    it "should respond to has_key?" do
+      plant = Plant.new(:leaf_count => 1)
+      plant.has_key?(:leaf_count).should be_true
     end
   end
 
@@ -67,37 +86,42 @@ describe "attributes" do
     end
 
     describe "fixnum" do
-      it "should convert a string into a fixnum" do
+      it 'rounds a float to a fixnum' do
+        @plant.typed_leaf_count = 4.5
+        @plant.typed_leaf_count.should == 5
+      end
+
+      it "converts a string into a fixnum" do
         @plant.typed_leaf_count = '4'
         @plant.typed_leaf_count.should == 4
       end
 
-      it "should convert a string into a negative fixnum" do
+      it "converts a string into a negative fixnum" do
         @plant.typed_leaf_count = '-4'
         @plant.typed_leaf_count.should == -4
       end
 
-      it "should leave a fixnum as is" do
+      it "leaves a fixnum as is" do
         @plant.typed_leaf_count = 4
         @plant.typed_leaf_count.should == 4
       end
 
-      it "should leave nil as is" do
+      it "leaves nil as is" do
         @plant.typed_leaf_count = nil
         @plant.typed_leaf_count.should be_nil
       end
 
-      it "should set the attributes to zero if a string given" do
+      it "sets the attributes to zero if a string given" do
         @plant.typed_leaf_count = 'x'
         @plant.typed_leaf_count.should == 0
       end
 
-      it "should parse numbers out of a string" do
+      it "parses numbers out of a string" do
         @plant.typed_leaf_count = 'x123'
         @plant.typed_leaf_count.should == 123
       end
 
-      it "should set the attributes to nil if given a blank string" do
+      it "set the attributes to nil if given a blank string" do
         @plant.typed_leaf_count = ''
         @plant.typed_leaf_count.should be_nil
       end
@@ -146,4 +170,3 @@ describe "attributes" do
     end
   end
 end
-

data/spec/unit/base_view_spec_spec.rb

@@ -4,6 +4,11 @@ describe CouchPotato::View::BaseViewSpec, 'initialize' do
   describe "view parameters" do
     before(:each) do
       CouchPotato::Config.split_design_documents_per_view = false
+      @default_language = CouchPotato::Config.default_language
+    end
+
+    after(:each) do
+      CouchPotato::Config.default_language = @default_language
     end
 
     it "should raise an error when passing invalid view parameters" do
@@ -32,7 +37,7 @@ describe CouchPotato::View::BaseViewSpec, 'initialize' do
         }
       }.should_not raise_error
     end
-    
+
     it "should remove stale when it's nil" do
       spec = CouchPotato::View::BaseViewSpec.new Object, 'all', {}, {:stale => nil}
       spec.view_parameters.should == {}
@@ -81,17 +86,17 @@ describe CouchPotato::View::BaseViewSpec, 'initialize' do
       spec = CouchPotato::View::BaseViewSpec.new stub(:lists => nil), 'all', {:list => :test_list}, {}
       spec.list_name.should == :test_list
     end
-    
+
     it "should extract the list from the view parameters" do
       spec = CouchPotato::View::BaseViewSpec.new stub(:lists => nil), 'all', {}, {:list => :test_list}
       spec.list_name.should == :test_list
     end
-    
+
     it "should prefer the list name from the view parameters over the one from the options" do
       spec = CouchPotato::View::BaseViewSpec.new stub(:lists => nil), 'all', {:list => 'my_list'}, {:list => :test_list}
       spec.list_name.should == :test_list
     end
-    
+
     it "should return the list function" do
       klass = stub 'class'
       klass.stub(:lists).with('test_list').and_return('<list_code>')
@@ -99,8 +104,17 @@ describe CouchPotato::View::BaseViewSpec, 'initialize' do
       spec.list_function.should == '<list_code>'
     end
 
-  end
+    it 'reads the language from the couch potato config by default' do
+      CouchPotato::Config.default_language = :ruby
+      spec = CouchPotato::View::BaseViewSpec.new Object, 'all', {}, {}
+      spec.language.should == :ruby
+    end
 
+    it 'sets the language to the given language' do
+      spec = CouchPotato::View::BaseViewSpec.new Object, 'all', {:language => :erlang}, {}
+      spec.language.should == :erlang
+    end
+  end
 end