cache-object 0.0.5 → 0.0.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: c2a98fe0b987a204106bb9e8f273ae36b7af394e
-  data.tar.gz: 32890adb0048d8f08c61bad3e3041c58841b2fdd
+  metadata.gz: ccaaecfb055eeb74feef3c2cd4447b19789f128a
+  data.tar.gz: 2ab0deb7a2a7d6f5ddd529d2fcd7a5d6e6f1edcb
 SHA512:
-  metadata.gz: 84721447edb9c798e46b9b8e094153c8472dfef2ad52803027cdafdefc698d2a7978c437650406f331450767dcc1dbbd1786e78e9f2dda062440ebaf3a49ed84
-  data.tar.gz: 4d492ccb088d4620d9167bf18a15598b10f9b4d1ab988779a0747aef8f44f9a43d27df0db195f2e7c6261a2f51c76327ec6bc3f0f00bff724f5715efc17ab5b3
+  metadata.gz: ac61615c7da8577d9c9be7a4eb8677caea2484b641cd327a700c27c3cebe2849662c946eb4e6f93802e88077387612c7a08940657e0398a22f2767d830a8fc48
+  data.tar.gz: 7aa792ad602a7b75f53545d98b3bb1704b0910830caf9908de5217e513c011d9adc3183b84b727d74312eb076d2760d13bb27da5889a2484667d1a464d988010

data/lib/cache/object/active_record.rb

@@ -37,10 +37,12 @@ module Cache
       end
 
       def write_cache!
+        return unless self.send(self.class.primary_key)
         Cache::Object.adapter.write(_cache_object_decorator)
       end
 
       def expire_cache!
+        return unless self.send(self.class.primary_key)
         Cache::Object.adapter.delete(_cache_object_decorator)
       end
 

data/lib/cache/object/version.rb

@@ -1,5 +1,5 @@
 module Cache
   module Object
-    VERSION = "0.0.5"
+    VERSION = "0.0.6"
   end
 end

data/spec/cache/object/active_record_spec.rb

@@ -25,6 +25,14 @@ RSpec.describe Cache::Object::ActiveRecord do
   class Model < FakeActiveRecord
     include Cache::Object::ActiveRecord
 
+    def self.primary_key
+      :id
+    end
+
+    def id
+      Random.rand * 100
+    end
+
     object_cache_on :name, :age
   end
 
@@ -99,8 +107,7 @@ RSpec.describe Cache::Object::ActiveRecord do
     describe ".find_by_id" do
       describe "caching interactions" do
         it "yields to super with cache" do
-          expect(super_clazz).to receive(:primary_key) { "primary_key" }
-          expect(super_clazz).to receive(:where).with("primary_key" => 12).once { double(first: true) }
+          expect(super_clazz).to receive(:where).with(:id => 12).once { double(first: true) }
           clazz.find_by_id(12)
         end
       end

data/spec/features/cache_object_spec.rb

@@ -101,5 +101,12 @@ RSpec.describe "Caching" do
     end
   end
 
+  describe "when object is not persisted" do
+    it "does not call the adapter" do
+      expect(Cache::Object.adapter).to receive(:write).never
+      User.new(name: "blah").write_cache!
+    end
+  end
+
 
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: cache-object
 version: !ruby/object:Gem::Version
-  version: 0.0.5
+  version: 0.0.6
 platform: ruby
 authors:
 - Matt Camuto
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-06-24 00:00:00.000000000 Z
+date: 2014-06-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord