From 68b8acbcfa148904b70394a99ae94d864a3f8366 Mon Sep 17 00:00:00 2001 From: Guo Xiang Tan Date: Thu, 24 Aug 2017 10:00:15 +0900 Subject: [PATCH] Make Redis test less brittle. --- spec/components/discourse_redis_spec.rb | 13 +++++++------ 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/spec/components/discourse_redis_spec.rb b/spec/components/discourse_redis_spec.rb index dd6c85409fc..374ee3c1619 100644 --- a/spec/components/discourse_redis_spec.rb +++ b/spec/components/discourse_redis_spec.rb @@ -25,10 +25,11 @@ describe DiscourseRedis do let(:redis) { DiscourseRedis.new } it 'should append namespace to the keys' do - redis.set('key', 1) + raw_redis.set('default:key', 1) + raw_redis.set('test:key2', 1) - expect(raw_redis.get('default:key')).to eq('1') - expect(redis.keys).to eq(['key']) + expect(redis.keys).to include('key') + expect(redis.keys).to_not include('key2') redis.del('key') @@ -45,10 +46,10 @@ describe DiscourseRedis do let(:redis) { DiscourseRedis.new(nil, namespace: false) } it 'should not append any namespace to the keys' do - redis.set('key', 1) + raw_redis.set('default:key', 1) + raw_redis.set('test:key2', 1) - expect(raw_redis.get('key')).to eq('1') - expect(redis.keys).to eq(['key']) + expect(redis.keys).to include('default:key', 'test:key2') redis.del('key')