Skip to content

Commit 29790ab

Browse files
author
Jesse Myers
committed
Add return values
1 parent a82757d commit 29790ab

File tree

2 files changed

+8
-5
lines changed

2 files changed

+8
-5
lines changed

mockredis/client.py

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -231,14 +231,16 @@ def flushdb(self):
231231
self.timeouts.clear()
232232

233233
def rename(self, old_key, new_key):
234-
self._rename(old_key, new_key)
234+
return self._rename(old_key, new_key)
235235

236236
def renamenx(self, old_key, new_key):
237-
self._rename(old_key, new_key, True)
237+
return 1 if self._rename(old_key, new_key, True) else 0
238238

239239
def _rename(self, old_key, new_key, nx=False):
240240
if old_key in self.redis and (not nx or new_key not in self.redis):
241241
self.redis[new_key] = self.redis.pop(old_key)
242+
return True
243+
return False
242244

243245
#### String Functions ####
244246

@@ -1380,6 +1382,7 @@ def _score_inclusive(self, score):
13801382
return False, float(score[1:])
13811383
return True, float(score)
13821384

1385+
13831386
def get_total_seconds(td):
13841387
"""
13851388
For python 2.6 support

mockredis/tests/test_redis.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -211,13 +211,13 @@ def test_delitem(self):
211211

212212
def test_rename(self):
213213
self.redis["foo"] = "bar"
214-
self.redis.rename("foo", "new_foo")
214+
ok_(self.redis.rename("foo", "new_foo"))
215215
eq_("bar", self.redis.get("new_foo"))
216216

217217
def test_renamenx(self):
218218
self.redis["foo"] = "bar"
219219
self.redis["foo2"] = "bar2"
220-
self.redis.renamenx("foo", "foo2")
220+
eq_(self.redis.renamenx("foo", "foo2"), 0)
221221
eq_("bar2", self.redis.get("foo2"))
222-
self.redis.renamenx("foo", "foo3")
222+
eq_(self.redis.renamenx("foo", "foo3"), 1)
223223
eq_("bar", self.redis.get("foo3"))

0 commit comments

Comments
 (0)