| commit | 53b0123e8c0b2566986c5d8f7969a9ddb039d7c2 | [log] [tgz] |
|---|---|---|
| author | Marc Koderer <marc@koderer.com> | Wed Aug 13 15:57:30 2014 +0200 |
| committer | Marc Koderer <m.koderer@telekom.de> | Tue Aug 19 17:04:49 2014 +0200 |
| tree | ef0e3540979a04f6b57d7b67f74d1f8c6f5e5da6 | |
| parent | b74fb27895ce1808ff0c414b8b4f61b9c51cf697 [diff] |
Add functools.wraps on all function decorators See [1], this enhances logging and keeps the docstrings. [1]: https://docs.python.org/2/library/functools.html#functools.wraps Change-Id: I33beb28abb9a1a955fb3b5a99fd5eec7efecbc06
diff --git a/tempest/common/generator/base_generator.py b/tempest/common/generator/base_generator.py index 57b98f7..0398af1 100644 --- a/tempest/common/generator/base_generator.py +++ b/tempest/common/generator/base_generator.py
@@ -13,6 +13,8 @@ # License for the specific language governing permissions and limitations # under the License. +import functools + import jsonschema from tempest.openstack.common import log as logging @@ -39,6 +41,7 @@ """ Decorator for simple generators that return one value """ + @functools.wraps(fn) def wrapped(self, schema): result = fn(self, schema) if result is not None:
diff --git a/tempest/test.py b/tempest/test.py index 1e67d18..e3971a9 100644 --- a/tempest/test.py +++ b/tempest/test.py
@@ -70,7 +70,7 @@ """A decorator used to wrap the setUpClass for cleaning up resources when setUpClass failed. """ - + @functools.wraps(f) def decorator(cls): try: f(cls)