handle steps outside tests/fixtures without breaking test (fixes #232 via #233)

Author: Sup3rGeo

allure-pytest/test/steps/outside/run_step_outside_test.py

@@ -0,0 +1,12 @@
+from . import init_step
+
+init_step()
+
+def test_pass_with_step_outside():
+    """
+    >>> allure_report = getfixture('allure_report')
+    >>> assert_that(allure_report,
+    ...             has_test_case('test_pass_with_step_outside')
+    ... )
+    """
+    assert True

allure-python-commons/src/reporter.py

@@ -11,6 +11,7 @@
 class AllureReporter(object):
     def __init__(self):
         self._items = OrderedDict()
+        self._orphan_items = []
 
     def _update_item(self, uuid, **kwargs):
         item = self._items[uuid] if uuid else self._items[next(reversed(self._items))]
@@ -74,12 +75,18 @@ def close_test(self, uuid):
 
     def start_step(self, parent_uuid, uuid, step):
         parent_uuid = parent_uuid if parent_uuid else self._last_executable()
-        self._items[parent_uuid].steps.append(step)
-        self._items[uuid] = step
+        if parent_uuid is None:
+            self._orphan_items.append(uuid)
+        else:
+            self._items[parent_uuid].steps.append(step)
+            self._items[uuid] = step
 
     def stop_step(self, uuid, **kwargs):
-        self._update_item(uuid, **kwargs)
-        self._items.pop(uuid)
+        if uuid in self._orphan_items:
+            self._orphan_items.remove(uuid)
+        else:
+            self._update_item(uuid, **kwargs)
+            self._items.pop(uuid)
 
     def _attach(self, uuid, name=None, attachment_type=None, extension=None):
         mime_type = attachment_type