Skip to content

Commit 074b502

Browse files
authored
Merge pull request #714 from Jealrock/main
Convert value to string before saving it in RedisAdapter
2 parents 77d482b + c249153 commit 074b502

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

lib/split/persistence/redis_adapter.rb

+1-1
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ def [](field)
2727
end
2828

2929
def []=(field, value)
30-
Split.redis.hset(redis_key, field, value)
30+
Split.redis.hset(redis_key, field, value.to_s)
3131
expire_seconds = self.class.config[:expire_seconds]
3232
Split.redis.expire(redis_key, expire_seconds) if expire_seconds
3333
end

spec/persistence/redis_adapter_spec.rb

+3-3
Original file line numberDiff line numberDiff line change
@@ -73,9 +73,9 @@
7373
before { Split::Persistence::RedisAdapter.with_config(lookup_by: "lookup") }
7474

7575
describe "#[] and #[]=" do
76-
it "should set and return the value for given key" do
77-
subject["my_key"] = "my_value"
78-
expect(subject["my_key"]).to eq("my_value")
76+
it "should convert to string, set and return the value for given key" do
77+
subject["my_key"] = true
78+
expect(subject["my_key"]).to eq("true")
7979
end
8080
end
8181

0 commit comments

Comments
 (0)