github-mirror/discourse
github-mirror/discourse
2
0
Fork 0
mirror of https://github.com/discourse/discourse.git synced 2025-04-29 10:14:35 +08:00
Code Issues Actions 1 Packages Projects Releases Wiki Activity

moving redis to readonly mode destablizes the test suite

Browse Source
This commit is contained in:
Sam 2017-10-25 14:39:49 +11:00
parent 0d816302e2
commit 1db4c2d1a0
1 changed files with 4 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
spec/models/application_request_spec.rb
View File

@ -20,23 +20,21 @@ describe ApplicationRequest do
end end
context "readonly test" do context "readonly test" do
after do
$redis.slaveof("no", "one")
end
it 'works even if redis is in readonly' do it 'works even if redis is in readonly' do
disable_date_flush! disable_date_flush!
inc(:http_total) inc(:http_total)
inc(:http_total) inc(:http_total)
$redis.slaveof("127.0.0.1", 666) $redis.without_namespace.stubs(:incr).raises(Redis::CommandError.new("READONLY"))
$redis.without_namespace.stubs(:eval).raises(Redis::CommandError.new("READONLY"))
# flush will be deferred no error raised # flush will be deferred no error raised
inc(:http_total, autoflush: 3) inc(:http_total, autoflush: 3)
ApplicationRequest.write_cache! ApplicationRequest.write_cache!
$redis.slaveof("no", "one") $redis.without_namespace.unstub(:incr)
$redis.without_namespace.unstub(:eval)
inc(:http_total, autoflush: 3) inc(:http_total, autoflush: 3)
expect(ApplicationRequest.http_total.first.count).to eq(3) expect(ApplicationRequest.http_total.first.count).to eq(3)