openstack
/
swift
Code Issues Proposed changes

Remove an useless dependency to swiftclient 

Unit tests in test/unit only have one dependency
to swiftclient in test_direct_client.py. This one
can be easily avoided and this patch removes it.

Change-Id: Ic1c78bc7f7fe426e8f7d8209a783342a0c4f071f
This commit is contained in:
Fabien Boucher 2014-01-16 13:16:26 +01:00
parent 233f539788
commit b6da40899c
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
test/unit/common/test_direct_client.py
View File

@ -21,7 +21,7 @@ from hashlib import md5
from swift.common import direct_client
from swift.common.exceptions import ClientException
from swiftclient import json_loads
from swift.common.utils import json
def mock_http_connect(status, fake_headers=None, body=None):
@ -120,7 +120,7 @@ class TestDirectClient(unittest.TestCase):
fake_headers.update({'user-agent': 'direct-client %s' % os.getpid()})
self.assertEqual(fake_headers, resp_headers)
self.assertEqual(json_loads(body), resp)
self.assertEqual(json.loads(body), resp)
direct_client.http_connect = mock_http_connect(204, fake_headers, body)
@ -167,7 +167,7 @@ class TestDirectClient(unittest.TestCase):
headers.update({'user-agent': 'direct-client %s' % os.getpid()})
self.assertEqual(headers, resp_headers)
self.assertEqual(json_loads(body), resp)
self.assertEqual(json.loads(body), resp)
direct_client.http_connect = mock_http_connect(204, headers, body)