summaryrefslogtreecommitdiff
path: root/tests/test_pipeline.py
diff options
context:
space:
mode:
authorandy <andy@whiskeymedia.com>2013-06-04 23:01:11 -0400
committerandy <andy@whiskeymedia.com>2013-06-04 23:01:11 -0400
commit97de031fb67d691b10a15b8c176823b01c1299fe (patch)
treebae742e1eba69d3efafeb278b29835c9cd487d29 /tests/test_pipeline.py
parent5e30cd66fabe06b67298c4946b412a0f08fdab5c (diff)
downloadredis-py-97de031fb67d691b10a15b8c176823b01c1299fe.tar.gz
pipeline tests converted to pytest
Diffstat (limited to 'tests/test_pipeline.py')
-rw-r--r--tests/test_pipeline.py202
1 files changed, 202 insertions, 0 deletions
diff --git a/tests/test_pipeline.py b/tests/test_pipeline.py
new file mode 100644
index 0000000..ee860e7
--- /dev/null
+++ b/tests/test_pipeline.py
@@ -0,0 +1,202 @@
+from __future__ import with_statement
+import pytest
+
+import redis
+from redis._compat import b
+
+
+class TestPipeline(object):
+ def test_pipeline(self, r):
+ with r.pipeline() as pipe:
+ pipe.set('a', 'a1').get('a').zadd('z', z1=1).zadd('z', z2=4)
+ pipe.zincrby('z', 'z1').zrange('z', 0, 5, withscores=True)
+ assert pipe.execute() == \
+ [
+ True,
+ b('a1'),
+ True,
+ True,
+ 2.0,
+ [(b('z1'), 2.0), (b('z2'), 4)],
+ ]
+
+ def test_pipeline_length(self, r):
+ with r.pipeline() as pipe:
+ # Initially empty.
+ assert len(pipe) == 0
+ assert not pipe
+
+ # Fill 'er up!
+ pipe.set('a', 'a1').set('b', 'b1').set('c', 'c1')
+ assert len(pipe) == 3
+ assert pipe
+
+ # Execute calls reset(), so empty once again.
+ pipe.execute()
+ assert len(pipe) == 0
+ assert not pipe
+
+ def test_pipeline_no_transaction(self, r):
+ with r.pipeline(transaction=False) as pipe:
+ pipe.set('a', 'a1').set('b', 'b1').set('c', 'c1')
+ assert pipe.execute() == [True, True, True]
+ assert r['a'] == b('a1')
+ assert r['b'] == b('b1')
+ assert r['c'] == b('c1')
+
+ def test_pipeline_no_transaction_watch(self, r):
+ r['a'] = 0
+
+ with r.pipeline(transaction=False) as pipe:
+ pipe.watch('a')
+ a = pipe.get('a')
+
+ pipe.multi()
+ pipe.set('a', int(a) + 1)
+ assert pipe.execute() == [True]
+
+ def test_pipeline_no_transaction_watch_failure(self, r):
+ r['a'] = 0
+
+ with r.pipeline(transaction=False) as pipe:
+ pipe.watch('a')
+ a = pipe.get('a')
+
+ r['a'] = 'bad'
+
+ pipe.multi()
+ pipe.set('a', int(a) + 1)
+
+ with pytest.raises(redis.WatchError):
+ pipe.execute()
+
+ assert r['a'] == b('bad')
+
+ def test_exec_error_in_response(self, r):
+ """
+ an invalid pipeline command at exec time adds the exception instance
+ to the list of returned values
+ """
+ r['c'] = 'a'
+ with r.pipeline() as pipe:
+ pipe.set('a', 1).set('b', 2).lpush('c', 3).set('d', 4)
+ result = pipe.execute(raise_on_error=False)
+
+ assert result[0]
+ assert r['a'] == b('1')
+ assert result[1]
+ assert r['b'] == b('2')
+
+ # we can't lpush to a key that's a string value, so this should
+ # be a ResponseError exception
+ assert isinstance(result[2], redis.ResponseError)
+ assert r['c'] == b('a')
+
+ # since this isn't a transaction, the other commands after the
+ # error are still executed
+ assert result[3]
+ assert r['d'] == b('4')
+
+ # make sure the pipe was restored to a working state
+ assert pipe.set('z', 'zzz').execute() == [True]
+ assert r['z'] == b('zzz')
+
+ def test_exec_error_raised(self, r):
+ r['c'] = 'a'
+ with r.pipeline() as pipe:
+ pipe.set('a', 1).set('b', 2).lpush('c', 3).set('d', 4)
+ with pytest.raises(redis.ResponseError):
+ pipe.execute()
+
+ # make sure the pipe was restored to a working state
+ assert pipe.set('z', 'zzz').execute() == [True]
+ assert r['z'] == b('zzz')
+
+ def test_parse_error_raised(self, r):
+ with r.pipeline() as pipe:
+ # the zrem is invalid because we don't pass any keys to it
+ pipe.set('a', 1).zrem('b').set('b', 2)
+ with pytest.raises(redis.ResponseError):
+ pipe.execute()
+
+ # make sure the pipe was restored to a working state
+ assert pipe.set('z', 'zzz').execute() == [True]
+ assert r['z'] == b('zzz')
+
+ def test_watch_succeed(self, r):
+ r['a'] = 1
+ r['b'] = 2
+
+ with r.pipeline() as pipe:
+ pipe.watch('a', 'b')
+ assert pipe.watching
+ a_value = pipe.get('a')
+ b_value = pipe.get('b')
+ assert a_value == b('1')
+ assert b_value == b('2')
+ pipe.multi()
+
+ pipe.set('c', 3)
+ assert pipe.execute() == [True]
+ assert not pipe.watching
+
+ def test_watch_failure(self, r):
+ r['a'] = 1
+ r['b'] = 2
+
+ with r.pipeline() as pipe:
+ pipe.watch('a', 'b')
+ r['b'] = 3
+ pipe.multi()
+ pipe.get('a')
+ with pytest.raises(redis.WatchError):
+ pipe.execute()
+
+ assert not pipe.watching
+
+ def test_unwatch(self, r):
+ r['a'] = 1
+ r['b'] = 2
+
+ with r.pipeline() as pipe:
+ pipe.watch('a', 'b')
+ r['b'] = 3
+ pipe.unwatch()
+ assert not pipe.watching
+ pipe.get('a')
+ assert pipe.execute() == [b('1')]
+
+ def test_transaction_callable(self, r):
+ r['a'] = 1
+ r['b'] = 2
+ has_run = []
+
+ def my_transaction(pipe):
+ a_value = pipe.get('a')
+ assert a_value in (b('1'), b('2'))
+ b_value = pipe.get('b')
+ assert b_value == b('2')
+
+ # silly run-once code... incr's "a" so WatchError should be raised
+ # forcing this all to run again. this should incr "a" once to "2"
+ if not has_run:
+ r.incr('a')
+ has_run.append('it has')
+
+ pipe.multi()
+ pipe.set('c', int(a_value) + int(b_value))
+
+ result = r.transaction(my_transaction, 'a', 'b')
+ assert result == [True]
+ assert r['c'] == b('4')
+
+ def test_exec_error_in_no_transaction_pipeline(self, r):
+ r['a'] = 1
+ with r.pipeline(transaction=False) as pipe:
+ pipe.llen('a')
+ pipe.expire('a', 100)
+
+ with pytest.raises(redis.ResponseError):
+ pipe.execute()
+
+ assert r['a'] == b('1')