kotovalexarian-likes-gitlab/gitlab-org--gitlab-foss
1
0
Fork 0
Code Releases Activity

Expect calls to read_attribute depending on cache presence

Browse source
This commit is contained in:
Matija Čupić 2018-05-14 15:54:23 +02:00
parent f4810647a0
commit 13f68f55b3
No known key found for this signature in database
GPG key ID: 4BAF84FFACD2E5DE
1 changed files with 8 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
spec/models/concerns/redis_cacheable_spec.rb
View file

@ -51,12 +51,16 @@ describe RedisCacheable do
context 'when there is no cached value' do context 'when there is no cached value' do
it 'reads the attribute' do it 'reads the attribute' do
expect(instance).to receive(:read_attribute).and_call_original
expect(subject).to eq(payload[:name]) expect(subject).to eq(payload[:name])
end end
end end
context 'when there is a cached value' do context 'when there is a cached value' do
it 'reads the cached value' do it 'reads the cached value' do
expect(instance).not_to receive(:read_attribute)
instance.cache_attributes(payload) instance.cache_attributes(payload)
expect(subject).to eq(payload[:name]) expect(subject).to eq(payload[:name])
@ -81,6 +85,8 @@ describe RedisCacheable do
context 'when there is no cached value' do context 'when there is no cached value' do
it 'reads the attribute' do it 'reads the attribute' do
expect(instance).to receive(:read_attribute).and_call_original
expect(subject).to be_instance_of(ActiveSupport::TimeWithZone) expect(subject).to be_instance_of(ActiveSupport::TimeWithZone)
expect(subject).to be_within(1.minute).of(Time.zone.now) expect(subject).to be_within(1.minute).of(Time.zone.now)
end end
@ -88,6 +94,8 @@ describe RedisCacheable do
context 'when there is a cached value' do context 'when there is a cached value' do
it 'reads the cached value' do it 'reads the cached value' do
expect(instance).not_to receive(:read_attribute)
instance.cache_attributes(time: Time.zone.now) instance.cache_attributes(time: Time.zone.now)
expect(subject).to be_instance_of(ActiveSupport::TimeWithZone) expect(subject).to be_instance_of(ActiveSupport::TimeWithZone)