identity_cache 0.4.1 → 0.5.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 1fa182ed254be40433e84f2bdef8faf341581e5f
-  data.tar.gz: b795a7e03b4cf94a9d5ff3fd77e04186cde84d5c
+  metadata.gz: 11bd2d6f14e2b77b00c8bd58dd0731fa2976a81a
+  data.tar.gz: 4d547e1b739ce13b443dfb9ea44416a81936ed95
 SHA512:
-  metadata.gz: 51c4645eb6a548c720444d6964412576bf9c84a399ad1a100251953d24f7e114654170eb4ae3111c906fedd1d9f0555c1c3c2734904675383f84f9ee62fe3362
-  data.tar.gz: fabf846eb3ac2d9ccd9eda7f11dc3e42ed64ac5b678863fd1755e066bc04fc77cc46c4cf9eade1149da4a7fa244c28a45a62fdd4c798501a0b3764ded56d697f
+  metadata.gz: 63aa5192013b0d99b426e0b1cf70ea6962924a57da29beaac507b4d5dae458693822a55cd24bc743768826d7b725592b456c0ca68cc9655df5509100a0ab68b3
+  data.tar.gz: 99cab0e7c1b4d8a4a5378b408582486404182e86529d4e62926d29ab3c021a2ff9af02a4af05d0a5eae4cd6996a76f4588a4e03584645025d7c7131a122407fd

data/CHANGELOG.md

@@ -1,5 +1,10 @@
 # IdentityCache changelog
 
+#### 0.5.0
+
+- `never_set_inverse_association` and `fetch_read_only_records` are now `true` by default (#315)
+- Store the class name instead of the class itself (#311)
+
 #### 0.4.1
 
 - Deprecated embedded associations on models that don't use IDC (#305)

data/lib/identity_cache.rb

@@ -53,19 +53,17 @@ module IdentityCache
     mattr_accessor :cache_namespace
     self.cache_namespace = "IDC:#{CACHE_VERSION}:".freeze
 
-    version = Gem::Version.new(IdentityCache::VERSION)
-
     # Inverse active record associations are set when loading embedded
     # cache_has_many associations from the cache when never_set_inverse_association
     # is false. When set to true, it will only set the inverse cached association.
     mattr_accessor :never_set_inverse_association
-    self.never_set_inverse_association = version >= Gem::Version.new("0.5")
+    self.never_set_inverse_association = true
 
     # Fetched records are not read-only and this could sometimes prevent IDC from
     # reflecting what's truly in the database when fetch_read_only_records is false.
     # When set to true, it will only return read-only records when cache is used.
     mattr_accessor :fetch_read_only_records
-    self.fetch_read_only_records = version >= Gem::Version.new("0.5")
+    self.fetch_read_only_records = true
 
     def included(base) #:nodoc:
       raise AlreadyIncludedError if base.respond_to?(:cached_model)

data/lib/identity_cache/query_api.rb

@@ -122,7 +122,7 @@ module IdentityCache
 
       def record_from_coder(coder) #:nodoc:
         if coder
-          klass = coder[:class]
+          klass = coder[:class].constantize
           record = klass.instantiate(coder[:attributes].dup)
 
           coder[:associations].each {|name, value| set_embedded_association(record, name, value) } if coder.has_key?(:associations)
@@ -179,7 +179,7 @@ module IdentityCache
         unless record.nil?
           coder = {
             attributes: record.attributes_before_type_cast.dup,
-            class: record.class,
+            class: record.class.name,
           }
           add_cached_associations_to_coder(record, coder)
           coder

data/lib/identity_cache/version.rb

@@ -1,4 +1,4 @@
 module IdentityCache
-  VERSION = "0.4.1"
-  CACHE_VERSION = 6
+  VERSION = "0.5.0"
+  CACHE_VERSION = 7
 end

data/test/denormalized_has_many_test.rb

@@ -130,12 +130,15 @@ class DenormalizedHasManyTest < IdentityCache::TestCase
   end
 
   def test_deprecated_set_inverse_association_on_cache_hit
+    IdentityCache.never_set_inverse_association = false
     Item.fetch(@record.id) # warm cache
 
     item = Item.fetch(@record.id)
 
     associated_record = item.fetch_associated_records.to_a.first
     assert_equal item.object_id, associated_record.item.object_id
+  ensure
+    IdentityCache.never_set_inverse_association = true
   end
 
   def test_returned_records_should_be_readonly_on_cache_hit

data/test/fetch_multi_test.rb

@@ -286,7 +286,7 @@ class FetchMultiTest < IdentityCache::TestCase
   end
 
   def cache_response_for(record)
-    {class: record.class, attributes: record.attributes_before_type_cast}
+    { class: record.class.name, attributes: record.attributes_before_type_cast }
   end
 
   def with_batch_size(size)

data/test/fetch_test.rb

@@ -11,7 +11,7 @@ class FetchTest < IdentityCache::TestCase
     @record = Item.new
     @record.id = 1
     @record.title = 'bob'
-    @cached_value = {class: @record.class, attributes: @record.attributes_before_type_cast}
+    @cached_value = { class: @record.class.name, attributes: @record.attributes_before_type_cast }
     @blob_key = "#{NAMESPACE}blob:Item:#{cache_hash("created_at:datetime,id:integer,item_id:integer,title:string,updated_at:datetime")}:1"
     @index_key = "#{NAMESPACE}attr:Item:id:title:#{cache_hash('bob')}"
   end

data/test/helpers/serialization_format.rb

@@ -20,7 +20,16 @@ module SerializationFormat
     end
     record.reload
     Item.fetch(record.id)
-    IdentityCache.fetch(record.primary_cache_index_key)
+
+    IdentityCache.fetch(record.primary_cache_index_key) do
+      STDERR.puts(
+        "\e[31m" \
+          "The record could not be retrieved from the cache." \
+          "Did you configure MEMCACHED_HOST?" \
+          "\e[0m",
+      )
+      exit(1)
+    end
   end
 
   def serialized_record_file

data/test/helpers/update_serialization_format.rb

@@ -16,7 +16,10 @@ DatabaseConnection.setup
 DatabaseConnection.drop_tables
 DatabaseConnection.create_tables
 IdentityCache.logger = Logger.new(nil)
-IdentityCache.cache_backend = ActiveSupport::Cache::MemcachedStore.new("localhost:11211", :support_cas => true)
+IdentityCache.cache_backend = ActiveSupport::Cache::MemcachedStore.new(
+  "#{ENV["MEMCACHED_HOST"] || "localhost"}:11211",
+  :support_cas => true,
+)
 setup_models
 File.open(serialized_record_file, 'w') {|file| serialize(serialized_record, file) }
 puts "Serialized record to #{serialized_record_file}"

data/test/memoized_attributes_test.rb

@@ -1,6 +1,16 @@
 require "test_helper"
 
 class MemoizedAttributesTest < IdentityCache::TestCase
+  def setup
+    IdentityCache.fetch_read_only_records = false
+    super
+  end
+
+  def teardown
+    super
+    IdentityCache.fetch_read_only_records = true
+  end
+
   def test_memoization_should_not_break_dirty_tracking_with_empty_cache
     item = Item.create!
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: identity_cache
 version: !ruby/object:Gem::Version
-  version: 0.4.1
+  version: 0.5.0
 platform: ruby
 authors:
 - Camilo Lopez
@@ -14,7 +14,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-02-02 00:00:00.000000000 Z
+date: 2017-02-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: ar_transaction_changes
@@ -270,7 +270,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.5.1
+rubygems_version: 2.5.2
 signing_key: 
 specification_version: 4
 summary: IdentityCache lets you specify how you want to cache your model objects,