X-Git-Url: https://gerrit.opnfv.org/gerrit/gitweb?a=blobdiff_plain;f=functest%2Futils%2Fdecorators.py;h=73e0a35209e5386fbc39c6f10470fc176e3858fc;hb=efbe74c4cc1b6b6982e9e13950319f690541d6a3;hp=276235d96091e274a510ed586f14602eec3ebe74;hpb=18bdde48e866f198b5d08b62fc9e26e3dfe4e818;p=functest.git

diff --git a/functest/utils/decorators.py b/functest/utils/decorators.py
index 276235d96..73e0a3520 100644
--- a/functest/utils/decorators.py
+++ b/functest/utils/decorators.py
@@ -1,41 +1,48 @@
 #!/usr/bin/env python
 
+# pylint: disable=missing-docstring
+
 import errno
-import mock
+import functools
 import os
+
+import mock
 import requests.sessions
-import urlparse
+from six.moves import urllib
 
 
 def can_dump_request_to_file(method):
 
     def dump_preparedrequest(request, **kwargs):
-        parseresult = urlparse.urlparse(request.url)
+        # pylint: disable=unused-argument
+        parseresult = urllib.parse.urlparse(request.url)
         if parseresult.scheme == "file":
             try:
-                os.makedirs(parseresult.path)
-            except OSError as e:
-                if e.errno != errno.EEXIST:
+                dirname = os.path.dirname(parseresult.path)
+                os.makedirs(dirname)
+            except OSError as ex:
+                if ex.errno != errno.EEXIST:
                     raise
-            with open(os.path.join(parseresult.path, 'dump.txt'), 'a') as f:
+            with open(parseresult.path, 'a') as dumpfile:
                 headers = ""
                 for key in request.headers:
                     headers += key + " " + request.headers[key] + "\n"
                 message = "{} {}\n{}\n{}\n\n\n".format(
                     request.method, request.url, headers, request.body)
-                f.write(message)
+                dumpfile.write(message)
         return mock.Mock()
 
     def patch_request(method, url, **kwargs):
         with requests.sessions.Session() as session:
-            parseresult = urlparse.urlparse(url)
+            parseresult = urllib.parse.urlparse(url)
             if parseresult.scheme == "file":
-                with mock.patch.object(
-                        session, 'send', side_effect=dump_preparedrequest):
+                with mock.patch.object(session, 'send',
+                                       side_effect=dump_preparedrequest):
                     return session.request(method=method, url=url, **kwargs)
             else:
                 return session.request(method=method, url=url, **kwargs)
 
+    @functools.wraps(method)
     def hook(*args, **kwargs):
         with mock.patch('requests.api.request', side_effect=patch_request):
             return method(*args, **kwargs)