couch_potato-rails2 0.5.6

data/spec/rails_spec.rb

@@ -0,0 +1,50 @@
+require 'spec_helper'
+
+module LoadConst
+  def const_missing(name)
+    name = "#{self.name}::#{name}"
+    eval("#{name} = Class.new; #{name}.send(:include, CouchPotato::Persistence); #{name}.extend(LoadConst)")
+  end
+end
+
+class Autoloader
+  extend LoadConst
+end
+
+class WithUnloadedEmbedded
+  include CouchPotato::Persistence
+  extend LoadConst
+  
+  property :embedded
+  
+  view :all, :type => :custom, :map => 'function(doc) {emit(doc._id, null)}', :include_docs => true
+end
+
+
+describe CouchPotato::Database, 'rails specific behavior' do
+  
+  before(:each) do
+    recreate_db
+  end
+  
+  context 'load a document' do
+    it "should load models whose constants are currently uninitialized (like with rails in development mode)" do
+      CouchPotato.couchrest_database.save_doc(JSON.create_id => 'Autoloader::Uninitialized', '_id' => '1')
+      CouchPotato.database.load('1').class.name.should == 'Autoloader::Uninitialized'
+    end
+    
+    it "should load nested models" do
+      CouchPotato.couchrest_database.save_doc(JSON.create_id => 'Autoloader::Nested::Nested2', '_id' => '1')
+      CouchPotato.database.load('1').class.name.should == 'Autoloader::Nested::Nested2'
+    end
+  end  
+  
+  context 'load documents using a view' do
+    it "should load models from a view whose constants are currently uninitialized" do
+      doc = {JSON.create_id => 'WithUnloadedEmbedded', '_id' => '1', 'embedded' => {JSON.create_id => 'WithUnloadedEmbedded::Uninitialized'}}
+      CouchPotato.couchrest_database.save_doc(doc)
+      CouchPotato.database.view(WithUnloadedEmbedded.all).first.embedded.class.name.should == 'WithUnloadedEmbedded::Uninitialized'
+    end
+  end
+end
+

data/spec/railtie_spec.rb

@@ -0,0 +1,65 @@
+require 'spec_helper'
+require 'yaml'
+require 'spec/mocks'
+
+module Rails
+  def self.env
+    'test'
+  end
+  class Railtie
+    def self.initializer(*args)
+    end
+  end
+  
+  def self.root
+    RSpec::Mocks::Mock.new :join => ''
+  end
+end
+
+require 'couch_potato/railtie'
+
+describe "railtie" do
+  before(:all) do
+    @validation_framework = CouchPotato::Config.validation_framework
+  end
+  
+  after(:all) do
+    CouchPotato::Config.validation_framework = @validation_framework
+  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")
+    end
+    
+    it "should 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)
+      
+      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.opts

@@ -0,0 +1,2 @@
+--colour
+--format progress

data/spec/spec_helper.rb

@@ -0,0 +1,44 @@
+require 'rubygems'
+require 'rspec'
+require 'time'
+require 'active_support'
+require 'timecop'
+
+$:.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
+  ActiveSupport::Deprecation.silenced = true
+rescue
+  # ignore errors, ActiveSupport is probably not installed
+end
+
+class Child
+  include CouchPotato::Persistence
+
+  property :text
+end
+
+class Comment
+  include CouchPotato::Persistence
+
+  validates_presence_of :title
+
+  property :title
+end
+
+def recreate_db
+  CouchPotato.couchrest_database.recreate!
+end
+recreate_db
+
+RSpec::Matchers.define :string_matching do |regex|
+  match do |string|
+    string =~ regex
+  end
+end

data/spec/unit/active_model_compliance_spec.rb

@@ -0,0 +1,98 @@
+require 'spec_helper'
+
+begin
+  require 'active_model'
+  
+  describe 'ActiveModel conformance of couch potato objects' do
+    include ActiveModel::Lint::Tests
+
+    instance_methods.sort.select{|method| method.to_s.scan(/^test_/).first}.each do |method|
+      it "should #{method.to_s.sub(/^test_/, '').gsub('_', ' ')}" do
+        send method
+      end
+    end
+
+    class ActiveComment
+      include CouchPotato::Persistence
+      property :name
+      property :email
+      validates_presence_of :name, :email
+      validates_format_of :email, :with => /.+@.+/
+    end
+
+    before(:each) do
+      @model = ActiveComment.new
+    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
+    end
+
+    describe "#destroyed" do
+      it "should return destroyed if the object is deleted" do
+        @model._deleted = true
+        @model.should be_destroyed
+      end
+
+      it "should not return destroyed if it's not deleted" do
+        @model.should_not be_destroyed
+      end
+    end
+
+    def assert(boolean, message = '')
+      boolean || raise(message)
+    end
+
+    def assert_kind_of(klass, object)
+      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

@@ -0,0 +1,135 @@
+require 'spec_helper'
+
+class Plant
+  include CouchPotato::Persistence
+  property :leaf_count
+  property :typed_leaf_count, :type => Fixnum
+  property :typed_leaf_size,  :type => Float
+end
+
+describe "attributes" do
+  
+  describe 'attributes=' do
+    it "should assign the attributes" do
+      plant = Plant.new 
+      plant.attributes = {:leaf_count => 1}
+      plant.leaf_count.should == 1
+    end
+  end
+
+  describe "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}
+    end
+  end
+  
+  # useful when loading models from custom views
+  describe "accessing ghost attributes" do
+    it "should allow me to access attributes that are in the couchdb document but not defined as a property" do
+      plant = Plant.json_create({JSON.create_id => "Plant", "color" => "red", "leaf_count" => 1})
+      plant.color.should == 'red'
+    end
+    
+    it "should raise a no method error when trying to read attributes that are not in the document" do
+      plant = Plant.json_create({JSON.create_id => "Plant", "leaf_count" => 1})
+      lambda do
+        plant.length
+      end.should raise_error(NoMethodError)
+    end
+    
+    it "should raise a no method error if the document hasn't been loaded from the database" do
+      plant = Plant.new
+      lambda do
+        plant.length
+      end.should raise_error(NoMethodError, /undefined method `length'/)
+    end
+  end
+
+  describe 'typed attributes' do
+    
+    before(:each) do
+      @plant = Plant.new
+    end
+    
+    describe "fixnum" do  
+      it "should convert 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
+        @plant.typed_leaf_count = '-4'
+        @plant.typed_leaf_count.should == -4
+      end
+      
+      it "should leave a fixnum as is" do
+        @plant.typed_leaf_count = 4
+        @plant.typed_leaf_count.should == 4
+      end
+      
+      it "should leave 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
+        @plant.typed_leaf_count = 'x'
+        @plant.typed_leaf_count.should == 0
+      end
+      
+      it "should parse 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
+        @plant.typed_leaf_count = ''
+        @plant.typed_leaf_count.should be_nil
+      end
+    end
+    
+    describe "float" do  
+      it "should convert a number in a string with a decimal place" do
+        @plant.typed_leaf_size = '0.5001'
+        @plant.typed_leaf_size.should == 0.5001
+      end
+
+      it "should convert a number in a string without a decimal place" do
+        @plant.typed_leaf_size = '5'
+        @plant.typed_leaf_size.should == 5.0
+      end
+      
+      it "should convert a negative number in a string" do
+        @plant.typed_leaf_size = '-5.0'
+        @plant.typed_leaf_size.should == -5.0
+      end
+
+      it "should leave a float as it is" do
+        @plant.typed_leaf_size = 0.5
+        @plant.typed_leaf_size.should == 0.5
+      end
+      
+      it "should leave nil as is" do
+        @plant.typed_leaf_size = nil
+        @plant.typed_leaf_size.should be_nil
+      end
+      
+      it "should set the attributes to zero if a string given" do
+        @plant.typed_leaf_size = 'x'
+        @plant.typed_leaf_size.should == 0
+      end
+      
+      it "should parse numbers out of a string" do
+        @plant.typed_leaf_size = 'x00.123'
+        @plant.typed_leaf_size.should == 0.123
+      end
+      
+      it "should set the attributes to nil if given a blank string" do
+        @plant.typed_leaf_size = ''
+        @plant.typed_leaf_size.should be_nil
+      end
+    end
+  end
+end
+

data/spec/unit/base_view_spec_spec.rb

@@ -0,0 +1,106 @@
+require 'spec_helper'
+
+describe CouchPotato::View::BaseViewSpec, 'initialize' do
+  describe "view parameters" do
+    before(:each) do
+      CouchPotato::Config.split_design_documents_per_view = false
+    end
+
+    it "should raise an error when passing invalid view parameters" do
+      lambda {
+        CouchPotato::View::BaseViewSpec.new Object, 'all', {}, {:start_key => '1'}
+      }.should raise_error(ArgumentError, "invalid view parameter: start_key")
+    end
+
+    it "should not raise an error when passing valid view parameters" do
+      lambda {
+        CouchPotato::View::BaseViewSpec.new Object, 'all', {}, {
+          :key => 'keyvalue',
+          :startkey => 'keyvalue',
+          :startkey_docid => 'docid',
+          :endkey => 'keyvalue',
+          :endkey_docid => 'docid',
+          :limit => 3,
+          :stale => 'ok',
+          :descending => true,
+          :skip => 1,
+          :group => true,
+          :group_level => 1,
+          :reduce => false,
+          :include_docs => true,
+          :inclusive_end => true
+        }
+      }.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 == {}
+    end
+
+    it "should convert a range passed as key into startkey and endkey" do
+      spec = CouchPotato::View::BaseViewSpec.new Object, 'all', {}, {:key => '1'..'2'}
+      spec.view_parameters.should == {:startkey => '1', :endkey => '2'}
+    end
+
+    it "should convert a plain value to a hash with a key" do
+      spec = CouchPotato::View::BaseViewSpec.new Object, 'all', {}, '2'
+      spec.view_parameters.should == {:key => '2'}
+    end
+
+    it "should generate the design document path by snake_casing the class name but keeping double colons" do
+      spec = CouchPotato::View::BaseViewSpec.new 'Foo::BarBaz', '', {}, ''
+      spec.design_document.should == 'foo::bar_baz'
+    end
+
+    it "should generate the design document independent of the view name by default" do
+      CouchPotato::Config.split_design_documents_per_view = false
+      spec = CouchPotato::View::BaseViewSpec.new 'User', 'by_login_and_email', {}, ''
+      spec.design_document.should == 'user'
+    end
+
+    it "should generate the design document per view if configured to" do
+      CouchPotato::Config.split_design_documents_per_view = true
+      spec = CouchPotato::View::BaseViewSpec.new 'User', 'by_login_and_email', {}, ''
+      spec.design_document.should == 'user_view_by_login_and_email'
+    end
+
+    it "should generate the design document independent of the list name by default" do
+      CouchPotato::Config.split_design_documents_per_view = false
+      spec = CouchPotato::View::BaseViewSpec.new stub(:lists => nil, :to_s => 'User'), '', {:list => 'test_list'}, {}
+      spec.design_document.should == 'user'
+    end
+
+    it "should generate the design document per view if configured to" do
+      CouchPotato::Config.split_design_documents_per_view = true
+      spec = CouchPotato::View::BaseViewSpec.new stub(:lists => nil, :to_s => 'User'), '', {:list => :test_list}, {}
+      spec.design_document.should == 'user_list_test_list'
+    end
+
+    it "should extract the list name from the options" 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>')
+      spec = CouchPotato::View::BaseViewSpec.new klass, 'all', {:list => 'test_list'}, {}
+      spec.list_function.should == '<list_code>'
+    end
+
+  end
+
+end
+
+