mongoid_cached_document 0.1.1 → 0.1.2

data/README.rdoc

@@ -25,26 +25,26 @@ Assuming <tt>@user</tt> is a valid <tt>User</tt> with an <tt>id</tt> of 42 and <
 
 This will create a new <tt>Post</tt> document, with an <tt>author</tt> field, the content of which is a hash:
 
-    { '_type' => 'User', '_id' = 1 }
+    { '_type' => 'User', '_id' = 42 }
     
     @post.author._type
     => 'User'
     @post.author._id
-    => 1
+    => 42
     
 Attempting to get the user's <tt>login</tt> will cause the real user document to be fetched from the collection, which replaces the cached values.
 
   class User
     #...
     
-    def cached_attributes
+    def cachable_attributes
       [ :login ]
     end
   end
   
 With the above class definition, the cached attributes will be:
 
-    { '_type' => 'User', '_id' = 1, 'login' => 'jsmith' }
+    { '_type' => 'User', '_id' = 42, 'login' => 'jsmith' }
 
 To use the above in a criteria:
 

data/Rakefile

@@ -28,6 +28,7 @@ Spec::Rake::SpecTask.new(:rcov) do |spec|
   spec.libs << 'lib' << 'spec'
   spec.pattern = 'spec/**/*_spec.rb'
   spec.rcov = true
+  spec.rcov_opts = [ '--exclude', 'gem', '--exclude', 'spec' ]
 end
 
 task :spec => :check_dependencies

data/VERSION

@@ -1 +1 @@
-0.1.1
+0.1.2

data/lib/mongoid/cached_document.rb

@@ -35,7 +35,9 @@ module Mongoid
 
     class << self
       def set(value)
-        if value.respond_to? :cachable_attributes
+        if value.nil?
+          nil
+        elsif value.respond_to? :cachable_attributes
           value.cachable_attributes.merge({ '_type' => value.class.to_s, '_id' => value.id })
         else
           { '_type' => value.class.to_s, '_id' => value.id }
@@ -57,20 +59,23 @@ module Mongoid
     def method_missing(name, *args, &block)
       if @document
         _document.send name, *args, &block
-      elsif @cached_attributes.has_key? name.to_s
+      elsif @cached_attributes && @cached_attributes.has_key? name.to_s
         @cached_attributes[name.to_s]
-      else
+      elsif _document
         if defined? Rails
           Rails.logger.debug("#{@cached_attributes['_type']}[:#{name}] is not cached (called from #{caller(1).first})")
         end
         
         _document.send name, *args, &block
+      else
+        super
       end
     end
 
     private
       def _document
-        @document ||= @cached_attributes['_type'].constantize.find(@cached_attributes['_id'])
+        @document ||= @cached_attributes && @cached_attributes['_type'].constantize.find(@cached_attributes['_id'])
+        end
       end
   end
 end

data/mongoid_cached_document.gemspec

@@ -5,11 +5,11 @@
 
 Gem::Specification.new do |s|
   s.name = %q{mongoid_cached_document}
-  s.version = "0.1.1"
+  s.version = "0.1.2"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Matthew Gibbons"]
-  s.date = %q{2010-04-07}
+  s.date = %q{2010-04-08}
   s.description = %q{Adds support for caching Mongoid documents as a field within other Mongoid documents, with complete control over which fields to cache.}
   s.email = %q{mhgibbons@me.com}
   s.extra_rdoc_files = [

data/spec/unit/mongoid/cached_document_spec.rb

@@ -54,4 +54,30 @@ describe Mongoid::CachedDocument do
       @cached_document.title == 'Title'
     end
   end
+  
+  describe "tesing for equality" do
+    before :each do
+      @cachable_document = mock(CachableDocument)
+      
+      @cachable_document.stub(:id).and_return 42
+      @cachable_document.stub(:class).and_return CachableDocument
+      
+      @cached_document = Mongoid::CachedDocument.get('_type' => 'CachableDocument', '_id' => 42)
+    end
+        
+    it "#== delegates to the document" do
+      CachableDocument.should_receive(:find).with(42).and_return(@cachable_document)    
+      @cached_document == @cachable_document
+    end
+
+    it "#eql? delegates to the document" do
+      CachableDocument.should_receive(:find).with(42).and_return(@cachable_document)    
+      @cached_document.eql? @cachable_document
+    end
+
+    it "#equal? delegates to the document" do
+      CachableDocument.should_receive(:find).with(42).and_return(@cachable_document)    
+      @cached_document.equal? @cachable_document
+    end
+  end
 end

metadata

@@ -5,8 +5,8 @@ version: !ruby/object:Gem::Version
   segments: 
   - 0
   - 1
-  - 1
-  version: 0.1.1
+  - 2
+  version: 0.1.2
 platform: ruby
 authors: 
 - Matthew Gibbons
@@ -14,7 +14,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2010-04-07 00:00:00 +01:00
+date: 2010-04-08 00:00:00 +01:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency