garner 0.5.0 → 0.5.1
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- checksums.yaml +4 -4
- data/.rubocop.yml +1 -30
- data/.rubocop_todo.yml +49 -0
- data/.travis.yml +2 -0
- data/CHANGELOG.md +6 -0
- data/Gemfile +1 -1
- data/README.md +2 -2
- data/garner.gemspec +2 -2
- data/lib/garner/cache.rb +1 -1
- data/lib/garner/strategies/binding/key/binding_index.rb +1 -1
- data/lib/garner/version.rb +1 -1
- data/spec/garner/cache/context_spec.rb +4 -4
- data/spec/garner/cache/identity_spec.rb +6 -6
- data/spec/garner/cache_spec.rb +14 -6
- data/spec/garner/config_spec.rb +1 -1
- data/spec/garner/mixins/mongoid/document_spec.rb +12 -12
- data/spec/garner/mixins/mongoid/identity_spec.rb +25 -25
- data/spec/garner/mixins/rack_spec.rb +6 -6
- data/spec/garner/strategies/binding/invalidation/binding_index_spec.rb +1 -1
- data/spec/garner/strategies/binding/invalidation/touch_spec.rb +3 -3
- data/spec/garner/strategies/binding/key/binding_index_spec.rb +44 -44
- data/spec/garner/strategies/binding/key/cache_key_spec.rb +2 -2
- data/spec/garner/strategies/binding/key/safe_cache_key_spec.rb +12 -12
- data/spec/garner/strategies/context/key/caller_spec.rb +17 -17
- data/spec/garner/strategies/context/key/jsonp_spec.rb +2 -2
- data/spec/garner/strategies/context/key/request_get_spec.rb +3 -3
- data/spec/garner/strategies/context/key/request_path_spec.rb +3 -3
- data/spec/garner/strategies/context/key/request_post_spec.rb +3 -3
- data/spec/garner/version_spec.rb +2 -2
- data/spec/integration/active_record_spec.rb +4 -4
- data/spec/integration/grape_spec.rb +1 -1
- data/spec/integration/mongoid_spec.rb +57 -57
- data/spec/integration/rack_spec.rb +7 -7
- data/spec/integration/sinatra_spec.rb +1 -1
- data/spec/shared/binding_invalidation_strategy.rb +1 -1
- data/spec/shared/binding_key_strategy.rb +4 -4
- data/spec/shared/conditional_get.rb +13 -13
- data/spec/shared/context_key_strategy.rb +2 -2
- data/spec/spec_helper.rb +1 -0
- metadata +8 -7
@@ -22,17 +22,17 @@ describe Garner::Mixins::Rack do
|
|
22
22
|
end
|
23
23
|
|
24
24
|
it 'returns a Garner::Cache::Identity' do
|
25
|
-
subject.call.
|
25
|
+
expect(subject.call).to be_a(Garner::Cache::Identity)
|
26
26
|
end
|
27
27
|
|
28
28
|
it "sets the identity's ruby_binding to self" do
|
29
|
-
subject.call.ruby_context.
|
29
|
+
expect(subject.call.ruby_context).to eq @mock_app
|
30
30
|
end
|
31
31
|
|
32
32
|
it 'applies each of Garner.config.rack_context_key_strategies' do
|
33
33
|
# Default :context_key_strategies
|
34
|
-
subject.call.key_hash[:caller].
|
35
|
-
subject.call.key_hash[:request_params].
|
34
|
+
expect(subject.call.key_hash[:caller]).not_to be_nil
|
35
|
+
expect(subject.call.key_hash[:request_params]).to eq('foo' => 'bar')
|
36
36
|
|
37
37
|
# Custom :context_key_strategies
|
38
38
|
Garner.configure do |config|
|
@@ -40,8 +40,8 @@ describe Garner::Mixins::Rack do
|
|
40
40
|
Garner::Strategies::Context::Key::RequestGet
|
41
41
|
]
|
42
42
|
end
|
43
|
-
subject.call.key_hash[:caller].
|
44
|
-
subject.call.key_hash[:request_params].
|
43
|
+
expect(subject.call.key_hash[:caller]).to be_nil
|
44
|
+
expect(subject.call.key_hash[:request_params]).to eq('foo' => 'bar')
|
45
45
|
end
|
46
46
|
|
47
47
|
end
|
@@ -4,7 +4,7 @@ describe Garner::Strategies::Binding::Invalidation::Touch do
|
|
4
4
|
|
5
5
|
before(:each) do
|
6
6
|
@mock = double('model')
|
7
|
-
@mock.
|
7
|
+
allow(@mock).to receive(:touch)
|
8
8
|
end
|
9
9
|
|
10
10
|
subject { Garner::Strategies::Binding::Invalidation::Touch }
|
@@ -12,12 +12,12 @@ describe Garner::Strategies::Binding::Invalidation::Touch do
|
|
12
12
|
it_behaves_like 'Garner::Strategies::Binding::Invalidation strategy'
|
13
13
|
|
14
14
|
it 'calls :touch on the object' do
|
15
|
-
@mock.
|
15
|
+
expect(@mock).to receive(:touch)
|
16
16
|
subject.apply(@mock)
|
17
17
|
end
|
18
18
|
|
19
19
|
it 'does not raise error if :touch is undefined' do
|
20
|
-
@mock.
|
20
|
+
allow(@mock).to receive(:touch)
|
21
21
|
expect { subject.apply(@mock) }.to_not raise_error
|
22
22
|
end
|
23
23
|
end
|
@@ -6,21 +6,21 @@ describe Garner::Strategies::Binding::Key::BindingIndex do
|
|
6
6
|
@new_mock = double('new_mock')
|
7
7
|
|
8
8
|
@persisted_mock = double('persisted_mock')
|
9
|
-
@persisted_mock.
|
10
|
-
@persisted_mock.
|
9
|
+
allow(@persisted_mock).to receive(:identity_string) { 'Mocker/id=4' }
|
10
|
+
allow(@persisted_mock).to receive(:updated_at) { @time_dot_now }
|
11
11
|
|
12
12
|
@persisted_mock_alias = double('persisted_mock_alias')
|
13
|
-
@persisted_mock_alias.
|
14
|
-
@persisted_mock_alias.
|
13
|
+
allow(@persisted_mock_alias).to receive(:identity_string) { 'MockerAlias/id=alias-4' }
|
14
|
+
allow(@persisted_mock_alias).to receive(:proxy_binding) { @persisted_mock }
|
15
15
|
|
16
|
-
subject.
|
16
|
+
allow(subject).to receive(:canonical?) do |binding|
|
17
17
|
binding == @persisted_mock
|
18
18
|
end
|
19
19
|
|
20
20
|
# Marshal.load will return a new mock object, breaking equivalence tests
|
21
21
|
# when fetching from cache.
|
22
22
|
load_method = Marshal.method(:load)
|
23
|
-
Marshal.
|
23
|
+
allow(Marshal).to receive(:load) do |dump|
|
24
24
|
default = load_method.call(dump)
|
25
25
|
if default.is_a?(RSpec::Mocks::Double) &&
|
26
26
|
default.instance_variable_get(:@name) == 'persisted_mock'
|
@@ -34,7 +34,7 @@ describe Garner::Strategies::Binding::Key::BindingIndex do
|
|
34
34
|
@mock_key = 'cc318d04bac07d5d91f06f8c'
|
35
35
|
@mock_alias_key = 'f254b853d7b32406b5749410'
|
36
36
|
@random_key = 'b1d44bb6b369903b28549271'
|
37
|
-
subject.
|
37
|
+
allow(subject).to receive(:new_cache_key_for) do |binding|
|
38
38
|
if binding == @persisted_mock
|
39
39
|
@mock_key
|
40
40
|
elsif binding == @persisted_mock_alias
|
@@ -54,7 +54,7 @@ describe Garner::Strategies::Binding::Key::BindingIndex do
|
|
54
54
|
|
55
55
|
describe 'apply' do
|
56
56
|
it 'calls fetch_cache_key_for' do
|
57
|
-
subject.
|
57
|
+
expect(subject).to receive(:fetch_cache_key_for).with(@persisted_mock)
|
58
58
|
subject.apply(@persisted_mock)
|
59
59
|
end
|
60
60
|
end
|
@@ -62,54 +62,54 @@ describe Garner::Strategies::Binding::Key::BindingIndex do
|
|
62
62
|
describe 'fetch_cache_key_for' do
|
63
63
|
context 'with a canonical binding' do
|
64
64
|
it 'returns a cache key string' do
|
65
|
-
subject.fetch_cache_key_for(@persisted_mock).
|
65
|
+
expect(subject.fetch_cache_key_for(@persisted_mock)).to eq @mock_key
|
66
66
|
end
|
67
67
|
|
68
68
|
it 'stores the cache key to cache' do
|
69
69
|
subject.fetch_cache_key_for(@persisted_mock)
|
70
|
-
Garner.config.cache.read(
|
70
|
+
expect(Garner.config.cache.read(
|
71
71
|
strategy: subject,
|
72
72
|
proxied_binding: 'Mocker/id=4'
|
73
|
-
).
|
73
|
+
)).to eq @mock_key
|
74
74
|
end
|
75
75
|
end
|
76
76
|
|
77
77
|
context 'with a non-canonical binding' do
|
78
78
|
it 'returns a cache key string' do
|
79
|
-
subject.fetch_cache_key_for(@persisted_mock_alias).
|
79
|
+
expect(subject.fetch_cache_key_for(@persisted_mock_alias)).to eq @mock_key
|
80
80
|
end
|
81
81
|
|
82
82
|
it 'stores the canonical binding to cache' do
|
83
83
|
subject.fetch_cache_key_for(@persisted_mock_alias)
|
84
|
-
Garner.config.cache.read(
|
84
|
+
expect(Garner.config.cache.read(
|
85
85
|
strategy: subject,
|
86
86
|
proxied_binding: 'MockerAlias/id=alias-4'
|
87
|
-
).
|
87
|
+
)).to eq @persisted_mock
|
88
88
|
end
|
89
89
|
|
90
90
|
it 'stores the cache key to cache' do
|
91
91
|
subject.fetch_cache_key_for(@persisted_mock_alias)
|
92
|
-
Garner.config.cache.read(
|
92
|
+
expect(Garner.config.cache.read(
|
93
93
|
strategy: subject,
|
94
94
|
proxied_binding: 'Mocker/id=4'
|
95
|
-
).
|
95
|
+
)).to eq @mock_key
|
96
96
|
end
|
97
97
|
|
98
98
|
context 'whose canonical binding is nil' do
|
99
99
|
before(:each) do
|
100
|
-
@persisted_mock_alias.
|
100
|
+
allow(@persisted_mock_alias).to receive(:proxy_binding) { nil }
|
101
101
|
end
|
102
102
|
|
103
103
|
it 'returns a nil cache key' do
|
104
|
-
subject.fetch_cache_key_for(@persisted_mock_alias).
|
104
|
+
expect(subject.fetch_cache_key_for(@persisted_mock_alias)).to be_nil
|
105
105
|
end
|
106
106
|
|
107
107
|
it 'does not store the cache key to cache' do
|
108
108
|
subject.fetch_cache_key_for(@persisted_mock_alias)
|
109
|
-
Garner.config.cache.read(
|
109
|
+
expect(Garner.config.cache.read(
|
110
110
|
strategy: subject,
|
111
111
|
proxied_binding: ''
|
112
|
-
).
|
112
|
+
)).to be_nil
|
113
113
|
end
|
114
114
|
end
|
115
115
|
end
|
@@ -118,22 +118,22 @@ describe Garner::Strategies::Binding::Key::BindingIndex do
|
|
118
118
|
describe 'write_cache_key_for' do
|
119
119
|
context 'with a canonical binding' do
|
120
120
|
it 'returns a cache key string' do
|
121
|
-
subject.write_cache_key_for(@persisted_mock).
|
121
|
+
expect(subject.write_cache_key_for(@persisted_mock)).to eq @mock_key
|
122
122
|
end
|
123
123
|
end
|
124
124
|
|
125
125
|
context 'with a non-canonical binding' do
|
126
126
|
it 'returns a cache key string' do
|
127
|
-
subject.write_cache_key_for(@persisted_mock_alias).
|
127
|
+
expect(subject.write_cache_key_for(@persisted_mock_alias)).to eq @mock_key
|
128
128
|
end
|
129
129
|
|
130
130
|
context 'whose canonical binding is nil' do
|
131
131
|
before(:each) do
|
132
|
-
@persisted_mock_alias.
|
132
|
+
allow(@persisted_mock_alias).to receive(:proxy_binding) { nil }
|
133
133
|
end
|
134
134
|
|
135
135
|
it 'returns a nil cache key' do
|
136
|
-
subject.write_cache_key_for(@persisted_mock_alias).
|
136
|
+
expect(subject.write_cache_key_for(@persisted_mock_alias)).to be_nil
|
137
137
|
end
|
138
138
|
end
|
139
139
|
end
|
@@ -142,27 +142,27 @@ describe Garner::Strategies::Binding::Key::BindingIndex do
|
|
142
142
|
describe 'fetch_canonical_binding_for' do
|
143
143
|
context 'with a canonical binding' do
|
144
144
|
it 'returns the canonical binding' do
|
145
|
-
subject.fetch_canonical_binding_for(@persisted_mock).
|
145
|
+
expect(subject.fetch_canonical_binding_for(@persisted_mock)).to eq @persisted_mock
|
146
146
|
end
|
147
147
|
end
|
148
148
|
|
149
149
|
context 'with a non-canonical binding' do
|
150
150
|
it 'returns the canonical binding' do
|
151
|
-
subject.fetch_canonical_binding_for(@persisted_mock_alias).
|
151
|
+
expect(subject.fetch_canonical_binding_for(@persisted_mock_alias)).to eq @persisted_mock
|
152
152
|
end
|
153
153
|
|
154
154
|
it 'stores the canonical binding to cache' do
|
155
155
|
subject.fetch_canonical_binding_for(@persisted_mock_alias)
|
156
|
-
Garner.config.cache.read(
|
156
|
+
expect(Garner.config.cache.read(
|
157
157
|
strategy: subject,
|
158
158
|
proxied_binding: 'MockerAlias/id=alias-4'
|
159
|
-
).
|
159
|
+
)).to eq @persisted_mock
|
160
160
|
end
|
161
161
|
end
|
162
162
|
|
163
163
|
context 'with a proxyless binding' do
|
164
164
|
it 'returns nil' do
|
165
|
-
subject.fetch_canonical_binding_for(@new_mock).
|
165
|
+
expect(subject.fetch_canonical_binding_for(@new_mock)).to be nil
|
166
166
|
end
|
167
167
|
end
|
168
168
|
end
|
@@ -170,27 +170,27 @@ describe Garner::Strategies::Binding::Key::BindingIndex do
|
|
170
170
|
describe 'write_canonical_binding_for' do
|
171
171
|
context 'with a canonical binding' do
|
172
172
|
it 'returns the canonical binding' do
|
173
|
-
subject.write_canonical_binding_for(@persisted_mock).
|
173
|
+
expect(subject.write_canonical_binding_for(@persisted_mock)).to eq @persisted_mock
|
174
174
|
end
|
175
175
|
end
|
176
176
|
|
177
177
|
context 'with a non-canonical binding' do
|
178
178
|
it 'returns the canonical binding' do
|
179
|
-
subject.write_canonical_binding_for(@persisted_mock_alias).
|
179
|
+
expect(subject.write_canonical_binding_for(@persisted_mock_alias)).to eq @persisted_mock
|
180
180
|
end
|
181
181
|
end
|
182
182
|
|
183
183
|
context 'with a proxyless binding' do
|
184
184
|
it 'returns nil' do
|
185
|
-
subject.write_canonical_binding_for(@new_mock).
|
185
|
+
expect(subject.write_canonical_binding_for(@new_mock)).to be nil
|
186
186
|
end
|
187
187
|
end
|
188
188
|
end
|
189
189
|
|
190
190
|
context 'with real objects' do
|
191
191
|
before(:each) do
|
192
|
-
subject.
|
193
|
-
subject.
|
192
|
+
allow(subject).to receive(:canonical?).and_call_original
|
193
|
+
allow(subject).to receive(:new_cache_key_for).and_call_original
|
194
194
|
Garner.configure do |config|
|
195
195
|
config.mongoid_identity_fields = [:_id, :_slugs]
|
196
196
|
end
|
@@ -212,33 +212,33 @@ describe Garner::Strategies::Binding::Key::BindingIndex do
|
|
212
212
|
describe 'apply' do
|
213
213
|
it 'retrieves the correct key' do
|
214
214
|
key = subject.apply(Cheese.find('m1'))
|
215
|
-
subject.apply(Cheese.identify('m1')).
|
215
|
+
expect(subject.apply(Cheese.identify('m1'))).to eq key
|
216
216
|
end
|
217
217
|
|
218
218
|
it 'stores the appropriate values to cache' do
|
219
219
|
key1 = subject.apply(Food.identify(@cheese.id))
|
220
220
|
key2 = subject.apply(Cheese.identify('m1'))
|
221
|
-
key1.
|
221
|
+
expect(key1).to eq key2
|
222
222
|
|
223
|
-
Garner.config.cache.read(
|
223
|
+
expect(Garner.config.cache.read(
|
224
224
|
strategy: subject,
|
225
225
|
proxied_binding: "Garner::Mixins::Mongoid::Identity/klass=Food,handle=#{@cheese.id}"
|
226
|
-
).
|
226
|
+
)).to eq @cheese
|
227
227
|
|
228
|
-
Garner.config.cache.read(
|
228
|
+
expect(Garner.config.cache.read(
|
229
229
|
strategy: subject,
|
230
230
|
proxied_binding: 'Garner::Mixins::Mongoid::Identity/klass=Cheese,handle=m1'
|
231
|
-
).
|
231
|
+
)).to eq @cheese
|
232
232
|
|
233
|
-
Garner.config.cache.read(
|
233
|
+
expect(Garner.config.cache.read(
|
234
234
|
strategy: subject,
|
235
235
|
proxied_binding: "Cheese/id=#{@cheese.id}"
|
236
|
-
).
|
236
|
+
)).to eq key1
|
237
237
|
|
238
|
-
Garner.config.cache.read(
|
238
|
+
expect(Garner.config.cache.read(
|
239
239
|
strategy: subject,
|
240
240
|
proxied_binding: "Food/id=#{@cheese.id}"
|
241
|
-
).
|
241
|
+
)).to be_nil
|
242
242
|
end
|
243
243
|
end
|
244
244
|
end
|
@@ -4,7 +4,7 @@ describe Garner::Strategies::Binding::Key::CacheKey do
|
|
4
4
|
|
5
5
|
before(:each) do
|
6
6
|
@mock = double('model')
|
7
|
-
@mock.
|
7
|
+
allow(@mock).to receive(:cache_key) { 'mocks/4' }
|
8
8
|
end
|
9
9
|
|
10
10
|
subject { Garner::Strategies::Binding::Key::CacheKey }
|
@@ -16,7 +16,7 @@ describe Garner::Strategies::Binding::Key::CacheKey do
|
|
16
16
|
|
17
17
|
describe 'apply' do
|
18
18
|
it "returns the object's cache key, or nil" do
|
19
|
-
subject.apply(@mock).
|
19
|
+
expect(subject.apply(@mock)).to eq 'mocks/4'
|
20
20
|
end
|
21
21
|
end
|
22
22
|
|
@@ -4,11 +4,11 @@ describe Garner::Strategies::Binding::Key::SafeCacheKey do
|
|
4
4
|
|
5
5
|
before(:each) do
|
6
6
|
@new_mock = double('model')
|
7
|
-
@new_mock.
|
7
|
+
allow(@new_mock).to receive(:cache_key) { 'mocks/4' }
|
8
8
|
@persisted_mock = double('model')
|
9
9
|
@time_dot_now = Time.now
|
10
|
-
@persisted_mock.
|
11
|
-
@persisted_mock.
|
10
|
+
allow(@persisted_mock).to receive(:cache_key) { "mocks/4-#{@time_dot_now.utc.to_s(:number)}" }
|
11
|
+
allow(@persisted_mock).to receive(:updated_at) { @time_dot_now }
|
12
12
|
end
|
13
13
|
|
14
14
|
subject { Garner::Strategies::Binding::Key::SafeCacheKey }
|
@@ -21,21 +21,21 @@ describe Garner::Strategies::Binding::Key::SafeCacheKey do
|
|
21
21
|
describe 'apply' do
|
22
22
|
it "returns the object's cache key + milliseconds if defined" do
|
23
23
|
timestamp = @time_dot_now.utc.to_s(:number)
|
24
|
-
subject.apply(@persisted_mock).
|
24
|
+
expect(subject.apply(@persisted_mock)).to match(/^mocks\/4-#{timestamp}.[0-9]{10}$/)
|
25
25
|
end
|
26
26
|
|
27
27
|
it 'returns nil if :cache_key is undefined or nil' do
|
28
|
-
@persisted_mock.
|
29
|
-
subject.apply(@persisted_mock).
|
30
|
-
@persisted_mock.
|
31
|
-
subject.apply(@persisted_mock).
|
28
|
+
allow(@persisted_mock).to receive(:cache_key)
|
29
|
+
expect(subject.apply(@persisted_mock)).to be_nil
|
30
|
+
allow(@persisted_mock).to receive(:cache_key) { nil }
|
31
|
+
expect(subject.apply(@persisted_mock)).to be_nil
|
32
32
|
end
|
33
33
|
|
34
34
|
it 'returns nil if :updated_at is undefined or nil' do
|
35
|
-
@persisted_mock.
|
36
|
-
subject.apply(@persisted_mock).
|
37
|
-
@persisted_mock.
|
38
|
-
subject.apply(@persisted_mock).
|
35
|
+
allow(@persisted_mock).to receive(:updated_at)
|
36
|
+
expect(subject.apply(@persisted_mock)).to be_nil
|
37
|
+
allow(@persisted_mock).to receive(:updated_at) { nil }
|
38
|
+
expect(subject.apply(@persisted_mock)).to be_nil
|
39
39
|
end
|
40
40
|
end
|
41
41
|
|
@@ -11,21 +11,21 @@ describe Garner::Strategies::Context::Key::Caller do
|
|
11
11
|
it_behaves_like 'Garner::Strategies::Context::Key strategy'
|
12
12
|
|
13
13
|
it 'ignores nil caller' do
|
14
|
-
@mock_context.
|
14
|
+
allow(@mock_context).to receive(:caller) { nil }
|
15
15
|
subject.apply(@cache_identity, @mock_context)
|
16
|
-
@cache_identity.key_hash[:caller].
|
16
|
+
expect(@cache_identity.key_hash[:caller]).to be_nil
|
17
17
|
end
|
18
18
|
|
19
19
|
it 'ignores nil caller location' do
|
20
|
-
@mock_context.
|
20
|
+
allow(@mock_context).to receive(:caller) { [nil] }
|
21
21
|
subject.apply(@cache_identity, @mock_context)
|
22
|
-
@cache_identity.key_hash[:caller].
|
22
|
+
expect(@cache_identity.key_hash[:caller]).to be_nil
|
23
23
|
end
|
24
24
|
|
25
25
|
it 'ignores blank caller location' do
|
26
|
-
@mock_context.
|
26
|
+
allow(@mock_context).to receive(:caller) { [''] }
|
27
27
|
subject.apply(@cache_identity, @mock_context)
|
28
|
-
@cache_identity.key_hash[:caller].
|
28
|
+
expect(@cache_identity.key_hash[:caller]).to be_nil
|
29
29
|
end
|
30
30
|
|
31
31
|
context 'with default Garner.config.caller_root' do
|
@@ -35,17 +35,17 @@ describe Garner::Strategies::Context::Key::Caller do
|
|
35
35
|
end
|
36
36
|
|
37
37
|
it 'sets default_root to the nearest ancestor with a Gemfile' do
|
38
|
-
subject.default_root.
|
38
|
+
expect(subject.default_root).to eq @gemfile_root
|
39
39
|
end
|
40
40
|
|
41
41
|
it 'sets Garner.config.caller_root to the nearest ancestor with a Gemfile' do
|
42
|
-
Garner.config.caller_root.
|
42
|
+
expect(Garner.config.caller_root).to eq @gemfile_root
|
43
43
|
end
|
44
44
|
|
45
45
|
it 'sets an appropriate value for :caller' do
|
46
46
|
truncated = __FILE__.gsub(@gemfile_root + File::SEPARATOR, '')
|
47
47
|
subject.apply(@cache_identity, self)
|
48
|
-
@cache_identity.key_hash[:caller].
|
48
|
+
expect(@cache_identity.key_hash[:caller]).to eq "#{truncated}:#{__LINE__ - 1}"
|
49
49
|
end
|
50
50
|
end
|
51
51
|
|
@@ -53,21 +53,21 @@ describe Garner::Strategies::Context::Key::Caller do
|
|
53
53
|
before(:each) do
|
54
54
|
class Rails
|
55
55
|
end
|
56
|
-
Rails.
|
56
|
+
allow(Rails).to receive(:root) { Pathname.new(File.dirname(__FILE__)) }
|
57
57
|
end
|
58
58
|
|
59
59
|
it 'sets default_root to Rails.root' do
|
60
|
-
subject.default_root.
|
60
|
+
expect(subject.default_root).to eq ::Rails.root.realpath.to_s
|
61
61
|
end
|
62
62
|
|
63
63
|
it 'sets Garner.config.caller_root to Rails.root' do
|
64
|
-
Garner.config.caller_root.
|
64
|
+
expect(Garner.config.caller_root).to eq ::Rails.root.realpath.to_s
|
65
65
|
end
|
66
66
|
|
67
67
|
it 'sets an appropriate value for :caller' do
|
68
68
|
truncated = File.basename(__FILE__)
|
69
69
|
subject.apply(@cache_identity, self)
|
70
|
-
@cache_identity.key_hash[:caller].
|
70
|
+
expect(@cache_identity.key_hash[:caller]).to eq "#{truncated}:#{__LINE__ - 1}"
|
71
71
|
end
|
72
72
|
end
|
73
73
|
|
@@ -81,7 +81,7 @@ describe Garner::Strategies::Context::Key::Caller do
|
|
81
81
|
it 'sets an appropriate value for :caller' do
|
82
82
|
truncated = File.basename(__FILE__)
|
83
83
|
subject.apply(@cache_identity, self)
|
84
|
-
@cache_identity.key_hash[:caller].
|
84
|
+
expect(@cache_identity.key_hash[:caller]).to eq "#{truncated}:#{__LINE__ - 1}"
|
85
85
|
end
|
86
86
|
end
|
87
87
|
|
@@ -94,13 +94,13 @@ describe Garner::Strategies::Context::Key::Caller do
|
|
94
94
|
|
95
95
|
it 'sets an appropriate value for :caller' do
|
96
96
|
subject.apply(@cache_identity, self)
|
97
|
-
@cache_identity.key_hash[:caller].
|
97
|
+
expect(@cache_identity.key_hash[:caller]).to eq "#{__FILE__}:#{__LINE__ - 1}"
|
98
98
|
end
|
99
99
|
|
100
100
|
it "doesn't require ActiveSupport" do
|
101
|
-
String.
|
101
|
+
allow_any_instance_of(String).to receive(:blank?) { fail NoMethodError.new }
|
102
102
|
subject.apply(@cache_identity, self)
|
103
|
-
@cache_identity.key_hash[:caller].
|
103
|
+
expect(@cache_identity.key_hash[:caller]).to eq "#{__FILE__}:#{__LINE__ - 1}"
|
104
104
|
end
|
105
105
|
end
|
106
106
|
end
|