openstack/swift
Code Issues Proposed changes

Fix pep8 issues in test_memcached.py.

Browse Source 
Change-Id: Ieeee2411f84489ea106909e18e758fd170aa9bca
This commit is contained in:
Alex Yang 2012-09-04 23:01:02 +08:00
parent 6e8284ecbd
commit d56772a898
1 changed files with 24 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

37
test/unit/common/test_memcached.py
View File

@ -17,7 +17,6 @@
""" Tests for swift.common.utils """ """ Tests for swift.common.utils """
from __future__ import with_statement from __future__ import with_statement
import hashlib
import logging import logging
import socket import socket
import time import time
@ -30,17 +29,22 @@ from test.unit import NullLoggingHandler
class ExplodingMockMemcached(object): class ExplodingMockMemcached(object):
exploded = False exploded = False
def sendall(self, string): def sendall(self, string):
self.exploded = True self.exploded = True
raise socket.error() raise socket.error()
def readline(self): def readline(self):
self.exploded = True self.exploded = True
raise socket.error() raise socket.error()
def read(self, size): def read(self, size):
self.exploded = True self.exploded = True
raise socket.error() raise socket.error()
class MockMemcached(object): class MockMemcached(object):
def __init__(self): def __init__(self):
self.inbuf = '' self.inbuf = ''
self.outbuf = '' self.outbuf = ''
@ -58,13 +62,13 @@ class MockMemcached(object):
parts = cmd.split() parts = cmd.split()
if parts[0].lower() == 'set': if parts[0].lower() == 'set':
self.cache[parts[1]] = parts[2], parts[3], \ self.cache[parts[1]] = parts[2], parts[3], \
self.inbuf[:int(parts[4])] self.inbuf[:int(parts[4])]
self.inbuf = self.inbuf[int(parts[4])+2:] self.inbuf = self.inbuf[int(parts[4]) + 2:]
if len(parts) < 6 or parts[5] != 'noreply': if len(parts) < 6 or parts[5] != 'noreply':
self.outbuf += 'STORED\r\n' self.outbuf += 'STORED\r\n'
elif parts[0].lower() == 'add': elif parts[0].lower() == 'add':
value = self.inbuf[:int(parts[4])] value = self.inbuf[:int(parts[4])]
self.inbuf = self.inbuf[int(parts[4])+2:] self.inbuf = self.inbuf[int(parts[4]) + 2:]
if parts[1] in self.cache: if parts[1] in self.cache:
if len(parts) < 6 or parts[5] != 'noreply': if len(parts) < 6 or parts[5] != 'noreply':
self.outbuf += 'NOT_STORED\r\n' self.outbuf += 'NOT_STORED\r\n'
@ -85,7 +89,8 @@ class MockMemcached(object):
for key in parts[1:]: for key in parts[1:]:
if key in self.cache: if key in self.cache:
val = self.cache[key] val = self.cache[key]
self.outbuf += 'VALUE %s %s %s\r\n' % (key, val[0], len(val[2])) self.outbuf += 'VALUE %s %s %s\r\n' % (
key, val[0], len(val[2]))
self.outbuf += val[2] + '\r\n' self.outbuf += val[2] + '\r\n'
self.outbuf += 'END\r\n' self.outbuf += 'END\r\n'
elif parts[0].lower() == 'incr': elif parts[0].lower() == 'incr':
@ -107,6 +112,7 @@ class MockMemcached(object):
self.outbuf += str(val[2]) + '\r\n' self.outbuf += str(val[2]) + '\r\n'
else: else:
self.outbuf += 'NOT_FOUND\r\n' self.outbuf += 'NOT_FOUND\r\n'
def readline(self): def readline(self):
if self.read_return_none: if self.read_return_none:
return None return None
@ -114,7 +120,8 @@ class MockMemcached(object):
raise Exception('mock is down') raise Exception('mock is down')
if '\n' in self.outbuf: if '\n' in self.outbuf:
response, self.outbuf = self.outbuf.split('\n', 1) response, self.outbuf = self.outbuf.split('\n', 1)
return response+'\n' return response + '\n'
def read(self, size): def read(self, size):
if self.down: if self.down:
raise Exception('mock is down') raise Exception('mock is down')
@ -123,6 +130,7 @@ class MockMemcached(object):
self.outbuf = self.outbuf[size:] self.outbuf = self.outbuf[size:]
return response return response
class TestMemcached(unittest.TestCase): class TestMemcached(unittest.TestCase):
""" Tests for swift.common.memcached""" """ Tests for swift.common.memcached"""
@ -163,8 +171,10 @@ class TestMemcached(unittest.TestCase):
memcache_client.set('some_key', [4, 5, 6]) memcache_client.set('some_key', [4, 5, 6])
self.assertEquals(memcache_client.get('some_key'), [4, 5, 6]) self.assertEquals(memcache_client.get('some_key'), [4, 5, 6])
memcache_client.set('some_key', ['simple str', 'utf8 str éà']) memcache_client.set('some_key', ['simple str', 'utf8 str éà'])
# As per http://wiki.openstack.org/encoding, we should expect to have unicode # As per http://wiki.openstack.org/encoding,
self.assertEquals(memcache_client.get('some_key'), ['simple str', u'utf8 str éà']) # we should expect to have unicode
self.assertEquals(
memcache_client.get('some_key'), ['simple str', u'utf8 str éà'])
self.assert_(float(mock.cache.values()[0][1]) == 0) self.assert_(float(mock.cache.values()[0][1]) == 0)
esttimeout = time.time() + 10 esttimeout = time.time() + 10
memcache_client.set('some_key', [1, 2, 3], timeout=10) memcache_client.set('some_key', [1, 2, 3], timeout=10)
@ -204,10 +214,10 @@ class TestMemcached(unittest.TestCase):
self.assertRaises(memcached.MemcacheConnectionError, self.assertRaises(memcached.MemcacheConnectionError,
memcache_client.decr, 'some_key', delta=15) memcache_client.decr, 'some_key', delta=15)
def test_retry(self): def test_retry(self):
logging.getLogger().addHandler(NullLoggingHandler()) logging.getLogger().addHandler(NullLoggingHandler())
memcache_client = memcached.MemcacheRing(['1.2.3.4:11211', '1.2.3.5:11211']) memcache_client = memcached.MemcacheRing(
['1.2.3.4:11211', '1.2.3.5:11211'])
mock1 = ExplodingMockMemcached() mock1 = ExplodingMockMemcached()
mock2 = MockMemcached() mock2 = MockMemcached()
memcache_client._client_cache['1.2.3.4:11211'] = [(mock2, mock2)] memcache_client._client_cache['1.2.3.4:11211'] = [(mock2, mock2)]
@ -240,8 +250,9 @@ class TestMemcached(unittest.TestCase):
timeout=10) timeout=10)
self.assert_(-1 <= float(mock.cache.values()[0][1]) - esttimeout <= 1) self.assert_(-1 <= float(mock.cache.values()[0][1]) - esttimeout <= 1)
self.assert_(-1 <= float(mock.cache.values()[1][1]) - esttimeout <= 1) self.assert_(-1 <= float(mock.cache.values()[1][1]) - esttimeout <= 1)
self.assertEquals(memcache_client.get_multi(('some_key2', 'some_key1', self.assertEquals(memcache_client.get_multi(
'not_exists'), 'multi_key'), [[4, 5, 6], [1, 2, 3], None]) ('some_key2', 'some_key1', 'not_exists'), 'multi_key'),
[[4, 5, 6], [1, 2, 3], None])
def test_serialization(self): def test_serialization(self):
memcache_client = memcached.MemcacheRing(['1.2.3.4:11211'], memcache_client = memcached.MemcacheRing(['1.2.3.4:11211'],
@ -262,6 +273,6 @@ class TestMemcached(unittest.TestCase):
memcache_client._allow_pickle = True memcache_client._allow_pickle = True
self.assertEquals(memcache_client.get('some_key'), [1, 2, 3]) self.assertEquals(memcache_client.get('some_key'), [1, 2, 3])
if __name__ == '__main__': if __name__ == '__main__':
unittest.main() unittest.main()