hide self parameter (fixes #222 via #235)

Author: Sup3rGeo

allure-pytest/test/steps/parameters/step_parameters_test.py

@@ -60,4 +60,27 @@ def test_text_type_parameters():
     ...             )
     ... )
     """
-    step_with_parameters(u'первый', kwarg_parameter='второй')
+    step_with_parameters(u'первый', kwarg_parameter='второй')
+
+
+class StepClass(object):
+
+    @allure.step('First step')
+    def first_step_method(self, arg1):
+        pass
+
+
+def test_self_not_in_parameters():
+    """
+    >>> from allure_commons.utils import represent
+    >>> allure_report = getfixture('allure_report_with_params')('-k test_self_not_in_parameters')
+    >>> assert_that(allure_report,
+    ...             has_test_case('test_self_not_in_parameters',
+    ...                           has_step('First step',
+    ...                                    has_parameter('arg1', represent('argument one')),
+    ...                                    doesnt_have_parameter('self'),
+    ...                           )
+    ...             )
+    ... )
+    """
+    StepClass().first_step_method('argument one')

allure-python-commons-test/src/result.py

@@ -62,7 +62,7 @@
 
 """
 
-from hamcrest import all_of, anything
+from hamcrest import all_of, anything, not_
 from hamcrest import equal_to, not_none
 from hamcrest import has_entry, has_item
 from hamcrest import contains_string, starts_with
@@ -100,6 +100,14 @@ def has_parameter(name, value):
                      ))
 
 
+def doesnt_have_parameter(name):
+    return has_entry('parameters',
+                     not_(
+                         has_item(
+                             has_entry('name', equal_to(name)),
+                     )))
+
+
 def has_link(url, link_type=None, name=None):
     return has_entry('links',
                      has_item(

allure-python-commons/src/utils.py

@@ -109,7 +109,15 @@ def represent(item):
 def func_parameters(func, *a, **kw):
     bowels = inspect.getargspec(func) if sys.version_info.major < 3 else inspect.getfullargspec(func)
     args_dict = dict(zip(bowels.args, map(represent, a)))
+    try:
+        del args_dict["self"]
+    except KeyError:
+        pass
     kwargs_dict = dict(zip(kw, list(map(lambda i: represent(kw[i]), kw))))
+    try:
+        del kwargs_dict["self"]
+    except KeyError:
+        pass
     kwarg_defaults = dict(zip(reversed(bowels.args), reversed(list(map(represent, bowels.defaults or ())))))
     kwarg_defaults.update(kwargs_dict)
     return args_dict, kwarg_defaults