Commit 89a47cf1 authored by Stavros Sachtouris's avatar Stavros Sachtouris Committed by Giorgos Korfiatis
Browse files

Fix "No LOG_PID" bug in LoggedAstakosClient

The class is now a subclass of "kamaki.cliets.Logged", through
multiple inheritance.
parent 64a40e14
......@@ -39,7 +39,7 @@ from astakosclient import AstakosClientException, parse_endpoints
import astakosclient
from kamaki.clients import (
Client, ClientError, KamakiSSLError, RequestManager, recvlog)
Client, Logged, ClientError, KamakiSSLError, RequestManager, recvlog)
from kamaki.clients.utils import https
......@@ -120,19 +120,16 @@ for m in inspect.getmembers(AstakosClient):
setattr(AstakosClient, m[0], _astakos_error(m[1]))
class LoggedAstakosClient(AstakosClient):
class LoggedAstakosClient(AstakosClient, Logged):
"""An AstakosClient wrapper with modified logging
Logs are adjusted to appear similar to the ones of kamaki clients.
No other changes are made to the parent class.
LOG_DATA = False
def __init__(self, *args, **kwargs):
kwargs.setdefault('logger', log)
super(LoggedAstakosClient, self).__init__(*args, **kwargs)
AstakosClient.__init__(self, *args, **kwargs)
def _dump_response(self, request, status, message, data):'%d %s' % (status, message))
......@@ -145,7 +142,7 @@ class LoggedAstakosClient(AstakosClient):
def _call_astakos(self, *args, **kwargs):
r = super(LoggedAstakosClient, self)._call_astakos(*args, **kwargs)
r = AstakosClient._call_astakos(self, *args, **kwargs)
log_request = getattr(self, 'log_request', None)
if log_request:
......@@ -170,7 +170,7 @@ class LoggedAstakosClient(TestCase):
@patch('%s.AstakosClient._call_astakos' % astakos_pkg, return_value='ret')
def test__call_astakos(self, super_call):
self.assertEqual(self.client._call_astakos('x', y='y'), 'ret')
super_call.assert_called_once_with('x', y='y')
super_call.assert_called_once_with(self.client, 'x', y='y')
class CachedAstakosClient(TestCase):
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment