identity_cache 0.5.1 → 1.0.0

data/test/denormalized_has_one_test.rb

@@ -1,160 +0,0 @@
-require "test_helper"
-
-class DenormalizedHasOneTest < IdentityCache::TestCase
-  def setup
-    super
-    PolymorphicRecord.include(IdentityCache::WithoutPrimaryIndex)
-    Item.cache_has_one :associated
-    Item.cache_index :title, :unique => true
-    @record = Item.new(:title => 'foo')
-    @record.associated = AssociatedRecord.new(:name => 'bar')
-    @record.save
-
-    @record.reload
-  end
-
-  def test_on_cache_miss_record_should_embed_associated_object
-    fetch = Spy.on(IdentityCache.cache, :fetch).and_call_through
-
-    record_from_cache_miss = Item.fetch_by_title('foo')
-
-    assert_equal @record, record_from_cache_miss
-    assert_not_nil @record.fetch_associated
-    assert_equal @record.associated, record_from_cache_miss.fetch_associated
-    assert fetch.has_been_called_with?(@record.attribute_cache_key_for_attribute_and_current_values(:id, [:title], true))
-    assert fetch.has_been_called_with?(@record.primary_cache_index_key)
-  end
-
-  def test_on_cache_miss_record_should_embed_nil_object
-
-    @record.associated = nil
-    @record.save!
-    @record.reload
-    Item.expects(:resolve_cache_miss).with(@record.id).once.returns(@record)
-
-    fetch = Spy.on(IdentityCache.cache, :fetch).and_call_through
-
-    record_from_cache_miss = Item.fetch_by_title('foo')
-    record_from_cache_miss.expects(:associated).never
-
-    assert_equal @record, record_from_cache_miss
-    5.times do
-      assert_nil record_from_cache_miss.fetch_associated
-    end
-    assert fetch.has_been_called_with?(@record.attribute_cache_key_for_attribute_and_current_values(:id, [:title], true))
-    assert fetch.has_been_called_with?(@record.primary_cache_index_key)
-  end
-
-  def test_on_record_from_the_db_will_use_normal_association
-    record_from_db = Item.find_by_title('foo')
-
-    assert_equal @record, record_from_db
-    assert_not_nil record_from_db.fetch_associated
-  end
-
-  def test_on_cache_hit_record_should_come_back_with_cached_association
-    Item.expects(:resolve_cache_miss).with(1).once.returns(@record)
-    Item.fetch_by_title('foo')
-
-    record_from_cache_hit = Item.fetch_by_title('foo')
-    expected = @record.associated
-
-    assert_equal @record, record_from_cache_hit
-    assert_equal expected, record_from_cache_hit.fetch_associated
-  end
-
-  def test_on_cache_hit_record_should_come_back_with_cached_nil_association
-    @record.associated = nil
-    @record.save!
-    @record.reload
-
-    Item.expects(:resolve_cache_miss).with(1).once.returns(@record)
-    Item.fetch_by_title('foo')
-
-    record_from_cache_hit = Item.fetch_by_title('foo')
-    record_from_cache_hit.expects(:associated).never
-
-    assert_equal @record, record_from_cache_hit
-    5.times do
-      assert_nil record_from_cache_hit.fetch_associated
-    end
-  end
-
-  def test_changes_in_associated_record_should_expire_the_parents_cache
-    Item.fetch_by_title('foo')
-    key = @record.primary_cache_index_key
-    assert_not_nil IdentityCache.cache.fetch(key)
-
-    IdentityCache.cache.expects(:delete).at_least(1).with(key)
-    IdentityCache.cache.expects(:delete).with(@record.associated.primary_cache_index_key)
-
-    @record.associated.save
-  end
-
-  def test_cached_associations_after_commit_hook_will_not_fail_on_undefined_parent_association
-    ar = AssociatedRecord.new
-    ar.save
-    assert_nothing_raised { ar.expire_parent_caches }
-  end
-
-  def test_cache_without_guessable_inverse_name_raises
-    assert_raises IdentityCache::InverseAssociationError do
-      Item.cache_has_one :polymorphic_record, :embed => true
-    end
-  end
-
-  def test_cache_without_guessable_inverse_name_does_not_raise_when_inverse_name_specified
-    assert_nothing_raised do
-      Item.cache_has_one :polymorphic_record, :inverse_name => :owner, :embed => true
-    end
-  end
-
-  def test_unsupported_through_assocation
-    assert_raises IdentityCache::UnsupportedAssociationError, "caching through associations isn't supported" do
-      Item.has_one :deeply_associated, :through => :associated, :class_name => 'DeeplyAssociatedRecord'
-      Item.cache_has_one :deeply_associated, :embed => true
-    end
-  end
-
-  def test_cache_has_one_on_derived_model_raises
-    assert_raises(IdentityCache::DerivedModelError) do
-      StiRecordTypeA.cache_has_one :polymorphic_record, :inverse_name => :owner, :embed => true
-    end
-  end
-
-  def test_returned_record_should_be_readonly_on_cache_hit
-    IdentityCache.with_fetch_read_only_records do
-      Item.fetch_by_title('foo')
-      record_from_cache_hit = Item.fetch_by_title('foo')
-      assert record_from_cache_hit.fetch_associated.readonly?
-      refute record_from_cache_hit.associated.readonly?
-    end
-  end
-
-  def test_returned_record_should_be_readonly_on_cache_miss
-    IdentityCache.with_fetch_read_only_records do
-      assert IdentityCache.should_use_cache?
-      record_from_cache_miss = Item.fetch_by_title('foo')
-      assert record_from_cache_miss.fetch_associated.readonly?
-    end
-  end
-
-  def test_db_returned_record_should_never_be_readonly
-    IdentityCache.with_fetch_read_only_records do
-      record_from_db = Item.find_by_title('foo')
-      uncached_record = record_from_db.associated
-      refute uncached_record.readonly?
-      record_from_db.fetch_associated
-      refute uncached_record.readonly?
-    end
-  end
-
-  def test_returned_record_with_open_transactions_should_not_be_readonly
-    IdentityCache.with_fetch_read_only_records do
-      Item.transaction do
-        refute IdentityCache.should_use_cache?
-        refute Item.fetch_by_title('foo').fetch_associated.readonly?
-      end
-    end
-  end
-end

data/test/fetch_multi_test.rb

@@ -1,308 +0,0 @@
-require "test_helper"
-
-class FetchMultiTest < IdentityCache::TestCase
-  NAMESPACE = IdentityCache.cache_namespace
-
-  def setup
-    super
-    @bob = Item.create!(:title => 'bob')
-    @joe = Item.create!(:title => 'joe')
-    @fred = Item.create!(:title => 'fred')
-    @bob_blob_key = "#{NAMESPACE}blob:Item:#{cache_hash("created_at:datetime,id:integer,item_id:integer,title:string,updated_at:datetime")}:1"
-    @joe_blob_key = "#{NAMESPACE}blob:Item:#{cache_hash("created_at:datetime,id:integer,item_id:integer,title:string,updated_at:datetime")}:2"
-    @fred_blob_key = "#{NAMESPACE}blob:Item:#{cache_hash("created_at:datetime,id:integer,item_id:integer,title:string,updated_at:datetime")}:3"
-    @tenth_blob_key = "#{NAMESPACE}blob:Item:#{cache_hash("created_at:datetime,id:integer,item_id:integer,title:string,updated_at:datetime")}:10"
-  end
-
-  def test_fetch_multi_with_no_records
-    assert_equal [], Item.fetch_multi
-  end
-
-  def test_fetch_multi_namespace
-    Item.send(:include, SwitchNamespace)
-    bob_blob_key, joe_blob_key, fred_blob_key = [@bob_blob_key, @joe_blob_key, @fred_blob_key].map { |k| "ns:#{k}" }
-    cache_response = {}
-    cache_response[bob_blob_key] = cache_response_for(@bob)
-    cache_response[joe_blob_key] = cache_response_for(@joe)
-    cache_response[fred_blob_key] = cache_response_for(@fred)
-    IdentityCache.cache.expects(:fetch_multi).with(bob_blob_key, joe_blob_key, fred_blob_key).returns(cache_response)
-    assert_equal [@bob, @joe, @fred], Item.fetch_multi(@bob.id, @joe.id, @fred.id)
-  end
-
-  def test_fetch_multi_with_all_hits
-    cache_response = {}
-    cache_response[@bob_blob_key] = cache_response_for(@bob)
-    cache_response[@joe_blob_key] = cache_response_for(@joe)
-    cache_response[@fred_blob_key] = cache_response_for(@fred)
-    IdentityCache.cache.expects(:fetch_multi).with(@bob_blob_key, @joe_blob_key, @fred_blob_key).returns(cache_response)
-    assert_equal [@bob, @joe, @fred], Item.fetch_multi(@bob.id, @joe.id, @fred.id)
-  end
-
-  def test_fetch_multi_with_all_misses
-    cache_response = {}
-    cache_response[@bob_blob_key] = nil
-    cache_response[@joe_blob_key] = nil
-    cache_response[@fred_blob_key] = nil
-    fetch_multi = fetch_multi_stub(cache_response)
-    assert_equal [@bob, @joe, @fred], Item.fetch_multi(@bob.id, @joe.id, @fred.id)
-    assert fetch_multi.has_been_called_with?(@bob_blob_key, @joe_blob_key, @fred_blob_key)
-  end
-
-  def test_fetch_multi_with_mixed_hits_and_misses
-    cache_response = {}
-    cache_response[@bob_blob_key] = cache_response_for(@bob)
-    cache_response[@joe_blob_key] = nil
-    cache_response[@fred_blob_key] = cache_response_for(@fred)
-    fetch_multi = fetch_multi_stub(cache_response)
-    assert_equal [@bob, @joe, @fred], Item.fetch_multi(@bob.id, @joe.id, @fred.id)
-    assert fetch_multi.has_been_called_with?(@bob_blob_key, @joe_blob_key, @fred_blob_key)
-  end
-
-  def test_fetch_multi_with_mixed_hits_and_misses_and_responses_in_the_wrong_order
-    cache_response = {}
-    cache_response[@bob_blob_key] = nil
-    cache_response[@joe_blob_key] = nil
-    cache_response[@fred_blob_key] = cache_response_for(@fred)
-    fetch_multi = fetch_multi_stub(cache_response)
-    assert_equal [@bob, @fred, @joe], Item.fetch_multi(@bob.id, @fred.id, @joe.id)
-    assert fetch_multi.has_been_called_with?(@bob_blob_key, @fred_blob_key, @joe_blob_key)
-  end
-
-  def test_fetch_multi_with_mixed_hits_and_misses_and_non_existant_keys_1
-    populate_only_fred
-
-    fetch_multi = fetch_multi_stub(@cache_response)
-    assert_equal [@bob, @joe, @fred], Item.fetch_multi(10, @bob.id, @joe.id, @fred.id)
-    assert fetch_multi.has_been_called_with?(@tenth_blob_key, @bob_blob_key, @joe_blob_key, @fred_blob_key)
-  end
-
-  def test_fetch_multi_with_mixed_hits_and_misses_and_non_existant_keys_2
-    populate_only_fred
-
-    fetch_multi = fetch_multi_stub(@cache_response)
-    assert_equal [@fred, @bob, @joe], Item.fetch_multi(@fred.id, @bob.id, 10, @joe.id)
-    assert fetch_multi.has_been_called_with?(@fred_blob_key, @bob_blob_key, @tenth_blob_key, @joe_blob_key)
-  end
-
-
-  def test_fetch_multi_works_with_nils
-    cache_result = {1 => IdentityCache::CACHED_NIL, 2 => IdentityCache::CACHED_NIL}
-    fetch_result = {1 => nil, 2 => nil}
-
-    fetcher.expects(:cas_multi).with([1, 2]).twice.returns(nil, cache_result)
-    fetcher.expects(:add).with(1, IdentityCache::CACHED_NIL).once
-    fetcher.expects(:add).with(2, IdentityCache::CACHED_NIL).once
-
-    results = IdentityCache.fetch_multi(1,2) do |keys|
-      [nil, nil]
-    end
-    assert_equal fetch_result, results
-
-    results = IdentityCache.fetch_multi(1,2) do |keys|
-      flunk "Contents should have been fetched from cache successfully"
-    end
-
-    assert_equal fetch_result, results
-  end
-
-  def test_fetch_multi_works_with_blanks
-    cache_result = {1 => false, 2 => '   '}
-
-    fetcher.expects(:fetch_multi).with([1,2]).returns(cache_result)
-
-    results = IdentityCache.fetch_multi(1,2) do |keys|
-      flunk "Contents should have been fetched from cache successfully"
-    end
-
-    assert_equal cache_result, results
-  end
-
-  def test_fetch_multi_duplicate_ids
-    assert_equal [@joe, @bob, @joe], Item.fetch_multi(@joe.id, @bob.id, @joe.id)
-  end
-
-  def test_fetch_multi_with_duplicate_ids_hits_backend_once_per_id
-    cache_response = {
-      @joe_blob_key => cache_response_for(@joe),
-      @bob_blob_key => cache_response_for(@bob),
-    }
-
-    fetcher.expects(:fetch_multi).with([@joe_blob_key, @bob_blob_key]).returns(cache_response)
-    result = Item.fetch_multi(@joe.id, @bob.id, @joe.id)
-
-    assert_equal [@joe, @bob, @joe], result
-  end
-
-  def test_fetch_multi_with_open_transactions_hits_the_database
-    Item.connection.expects(:open_transactions).at_least_once.returns(1)
-    fetcher.expects(:fetch_multi).never
-    assert_equal [@bob, @joe, @fred], Item.fetch_multi(@bob.id, @joe.id, @fred.id)
-  end
-
-  def test_fetch_multi_with_open_transactions_returns_results_in_the_order_of_the_passed_ids
-    Item.connection.expects(:open_transactions).at_least_once.returns(1)
-    assert_equal [@joe, @bob, @fred], Item.fetch_multi(@joe.id, @bob.id, @fred.id)
-  end
-
-  def test_fetch_multi_with_open_transactions_should_compacts_returned_array
-    Item.connection.expects(:open_transactions).at_least_once.returns(1)
-    assert_equal [@joe, @fred], Item.fetch_multi(@joe.id, 0, @fred.id)
-  end
-
-  def test_fetch_multi_with_duplicate_ids_in_transaction_returns_results_in_the_order_of_the_passed_ids
-    Item.connection.expects(:open_transactions).at_least_once.returns(1)
-    assert_equal [@joe, @bob, @joe], Item.fetch_multi(@joe.id, @bob.id, @joe.id)
-  end
-
-  def test_find_batch_coerces_ids_to_primary_key_type
-    mock_relation = mock("ActiveRecord::Relation")
-    Item.expects(:where).returns(mock_relation)
-    mock_relation.expects(:includes).returns(stub(:to_a => [@bob, @joe, @fred]))
-
-    Item.send(:find_batch, [@bob, @joe, @fred].map(&:id).map(&:to_s))
-  end
-
-  def test_fetch_multi_doesnt_freeze_keys
-    cache_response = {}
-    cache_response[@bob_blob_key] = cache_response_for(@bob)
-    cache_response[@joe_blob_key] = cache_response_for(@fred)
-
-    IdentityCache.expects(:fetch_multi).with{ |*args| args.none?(&:frozen?) }.returns(cache_response)
-
-    Item.fetch_multi(@bob.id, @joe.id)
-  end
-
-  def test_fetch_multi_array
-    assert_equal [@joe, @bob], Item.fetch_multi([@joe.id, @bob.id])
-  end
-
-  def test_fetch_multi_reads_in_batches
-    cache_response = {}
-    cache_response[@bob_blob_key] = cache_response_for(@bob)
-    cache_response[@joe_blob_key] = cache_response_for(@joe)
-
-    with_batch_size 1 do
-      fetcher.expects(:fetch_multi).with([@bob_blob_key]).returns(cache_response).once
-      fetcher.expects(:fetch_multi).with([@joe_blob_key]).returns(cache_response).once
-      assert_equal [@bob, @joe], Item.fetch_multi(@bob.id, @joe.id)
-    end
-  end
-
-  def test_fetch_multi_max_stack_level
-    cache_response = { @fred_blob_key => cache_response_for(@fred) }
-    fetcher.stubs(:fetch_multi).returns(cache_response)
-    assert_nothing_raised { Item.fetch_multi([@fred.id] * 200_000) }
-  end
-
-  def test_fetch_multi_with_non_id_primary_key
-    fixture = KeyedRecord.create!(:value => "a") { |r| r.hashed_key = 123 }
-    assert_equal [fixture], KeyedRecord.fetch_multi(123, 456)
-  end
-
-  def test_fetch_multi_after_expiring_a_record
-    Item.fetch_multi(@joe.id, @fred.id)
-    @bob.send(:expire_cache)
-    assert_equal IdentityCache::DELETED, backend.read(@bob.primary_cache_index_key)
-
-    add = Spy.on(IdentityCache.cache.cache_fetcher, :add).and_call_through
-
-    assert_equal [@bob, @joe, @fred], Item.fetch_multi(@bob.id, @joe.id, @fred.id)
-    refute add.has_been_called?
-    assert_equal cache_response_for(Item.find(@bob.id)), backend.read(@bob.primary_cache_index_key)
-  end
-
-  def test_fetch_multi_raises_when_called_on_a_scope
-    assert_raises(IdentityCache::UnsupportedScopeError) do
-      Item.where(updated_at: nil).fetch_multi(@bob.id, @joe.id, @fred.id)
-    end
-  end
-
-  def test_fetch_multi_on_derived_model_raises
-    assert_raises(IdentityCache::DerivedModelError) do
-      StiRecordTypeA.fetch_multi(1, 2)
-    end
-  end
-
-  def test_fetch_multi_with_mixed_hits_and_misses_returns_only_readonly_records
-    IdentityCache.with_fetch_read_only_records do
-      cache_response = {}
-      cache_response[@bob_blob_key] = cache_response_for(@bob)
-      cache_response[@joe_blob_key] = nil
-      cache_response[@fred_blob_key] = cache_response_for(@fred)
-      fetch_multi = fetch_multi_stub(cache_response)
-
-      response = Item.fetch_multi(@bob.id, @joe.id, @fred.id)
-      assert fetch_multi.has_been_called_with?(@bob_blob_key, @joe_blob_key, @fred_blob_key)
-      assert_equal [@bob, @joe, @fred], response
-
-      assert response.all?(&:readonly?)
-    end
-  end
-
-  def test_fetch_multi_with_mixed_hits_and_misses_and_responses_in_the_wrong_order_returns_readonly
-    IdentityCache.with_fetch_read_only_records do
-      cache_response = {}
-      cache_response[@bob_blob_key] = nil
-      cache_response[@joe_blob_key] = nil
-      cache_response[@fred_blob_key] = cache_response_for(@fred)
-      fetch_multi = fetch_multi_stub(cache_response)
-
-      response = Item.fetch_multi(@bob.id, @joe.id, @fred.id)
-      assert fetch_multi.has_been_called_with?(@bob_blob_key, @joe_blob_key, @fred_blob_key)
-      assert_equal [@bob, @joe, @fred], response
-
-      assert response.all?(&:readonly?)
-    end
-  end
-
-  def test_fetch_multi_with_open_transactions_returns_non_readonly_records
-    IdentityCache.with_fetch_read_only_records do
-      Item.transaction do
-        assert_equal IdentityCache.should_use_cache?, false
-        IdentityCache.cache.expects(:fetch_multi).never
-        assert Item.fetch_multi(@bob.id, @joe.id, @fred.id).none?(&:readonly?)
-      end
-    end
-  end
-
-  def test_fetch_multi_with_no_keys_does_not_query_when_cache_is_disabled
-    Item.stubs(:should_use_cache?).returns(false)
-
-    assert_queries(0) do
-      assert_memcache_operations(0) do
-        Item.fetch_multi
-      end
-    end
-  end
-
-  private
-
-  def populate_only_fred
-    @cache_response = {}
-    @cache_response[@bob_blob_key] = nil
-    @cache_response[@joe_blob_key] = nil
-    @cache_response[@tenth_blob_key] = nil
-    @cache_response[@fred_blob_key] = cache_response_for(@fred)
-  end
-
-  def cache_response_for(record)
-    { class: record.class.name, attributes: record.attributes_before_type_cast }
-  end
-
-  def with_batch_size(size)
-    previous_batch_size = IdentityCache::BATCH_SIZE
-    IdentityCache.send(:remove_const, :BATCH_SIZE)
-    IdentityCache.const_set(:BATCH_SIZE, size)
-    yield
-  ensure
-    IdentityCache.send(:remove_const, :BATCH_SIZE)
-    IdentityCache.const_set(:BATCH_SIZE, previous_batch_size)
-  end
-
-  def fetch_multi_stub(cache_response)
-    Spy.on(IdentityCache.cache, :fetch_multi).and_return do |*args, &block|
-      nil_keys = cache_response.select {|_, v| v.nil? }.keys
-      cache_response.merge(Hash[nil_keys.zip(block.call(nil_keys))])
-    end
-  end
-end