chrono_model 0.5.1 → 0.5.2

data/lib/chrono_model/time_machine.rb

@@ -147,11 +147,28 @@ module ChronoModel
     end
 
     # Returns a read-only representation of this record as it was +time+ ago.
+    # Returns nil if no record is found.
     #
     def as_of(time)
-      self.class.as_of(time).where(:id => self.id).first!
+      _as_of(time).first
     end
 
+    # Returns a read-only representation of this record as it was +time+ ago.
+    # Raises ActiveRecord::RecordNotFound if no record is found.
+    #
+    def as_of!(time)
+      _as_of(time).first!
+    end
+
+    # Delegates to +HistoryMethods.as_of+ to fetch this instance as it was on
+    # +time+. Used both by +as_of+ and +as_of!+ for performance reasons, to
+    # avoid a `rescue` (@lleirborras).
+    #
+    def _as_of(time)
+      self.class.as_of(time).where(:id => self.id)
+    end
+    protected :_as_of
+
     # Return the complete read-only history of this instance.
     #
     def history

data/lib/chrono_model/version.rb

@@ -1,3 +1,3 @@
 module ChronoModel
-  VERSION = "0.5.1"
+  VERSION = "0.5.2"
 end

data/spec/time_machine_spec.rb

@@ -94,8 +94,14 @@ describe ChronoModel::TimeMachine do
       it { bar.as_of(bar.ts[3]).foo.name.should == 'new foo' }
     end
 
-    it 'raises RecordNotFound when no history records are found' do
-      expect { foo.as_of(1.minute.ago) }.to raise_error
+    it 'doesn\'t raise RecordNotFound when no history records are found' do
+      expect { foo.as_of(1.minute.ago) }.to_not raise_error
+      foo.as_of(1.minute.ago).should be_nil
+    end
+
+
+    it 'raises ActiveRecord::RecordNotFound in the bang variant' do
+      expect { foo.as_of!(1.minute.ago) }.to raise_error(ActiveRecord::RecordNotFound)
     end
 
     describe 'it honors default_scopes' do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: chrono_model
 version: !ruby/object:Gem::Version
-  version: 0.5.1
+  version: 0.5.2
   prerelease: 
 platform: ruby
 authors:
@@ -9,11 +9,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-05-30 00:00:00.000000000 Z
+date: 2013-08-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
-  requirement: &70273330 !ruby/object:Gem::Requirement
+  requirement: &84655320 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -21,10 +21,10 @@ dependencies:
         version: '3.2'
   type: :runtime
   prerelease: false
-  version_requirements: *70273330
+  version_requirements: *84655320
 - !ruby/object:Gem::Dependency
   name: pg
-  requirement: &70273120 !ruby/object:Gem::Requirement
+  requirement: &84655110 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -32,7 +32,7 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70273120
+  version_requirements: *84655110
 description: Give your models as-of date temporal extensions. Built entirely for PostgreSQL
   >= 9.0
 email: