Use user-data to pass link patterns to Allure-Behave (fixes allure-framework#463 via allure-framework#464 )

kowalcj0 · web-flow · commit 1a704893bba9 · 2020-02-10T13:42:08.000+03:00
diff --git a/allure-behave/src/listener.py b/allure-behave/src/listener.py
@@ -97,7 +97,9 @@ def start_scenario(self, scenario):
         test_case.description = '\n'.join(scenario.description)
         test_case.parameters = scenario_parameters(scenario)
 
-        test_case.links.extend(scenario_links(scenario))
+        issue_pattern = self.behave_config.userdata.get('AllureFormatter.issue_pattern', None)
+        link_pattern = self.behave_config.userdata.get('AllureFormatter.link_pattern', None)
+        test_case.links.extend(scenario_links(scenario, issue_pattern=issue_pattern, link_pattern=link_pattern))
         test_case.labels.extend(scenario_labels(scenario))
         test_case.labels.append(Label(name=LabelType.FEATURE, value=scenario.feature.name))
         test_case.labels.append(Label(name=LabelType.FRAMEWORK, value='behave'))
diff --git a/allure-behave/src/utils.py b/allure-behave/src/utils.py
@@ -38,9 +38,12 @@ def scenario_parameters(scenario):
     return [Parameter(name=name, value=value) for name, value in zip(row.headings, row.cells)] if row else None
 
 
-def scenario_links(scenario):
+def scenario_links(scenario, issue_pattern, link_pattern):
     tags = scenario.feature.tags + scenario.tags
-    parsed = [parse_tag(item) for item in tags]
+    parsed = [
+        parse_tag(item, issue_pattern=issue_pattern, link_pattern=link_pattern)
+        for item in tags
+    ]
     return filter(lambda x: isinstance(x, Link), parsed)
 
 
diff --git a/allure-python-commons/src/mapping.py b/allure-python-commons/src/mapping.py
@@ -13,7 +13,7 @@ def __is(kind, t):
     return kind in [v for k, v in t.__dict__.items() if not k.startswith('__')]
 
 
-def parse_tag(tag):
+def parse_tag(tag, *, issue_pattern=None, link_pattern=None):
     """
     >>> parse_tag("blocker")
     Label(name='severity', value='blocker')
@@ -48,6 +48,10 @@ def parse_tag(tag):
     if prefix == TAG_PREFIX and value is not None:
 
         if __is(kind, LinkType):
+            if issue_pattern and kind == "issue" and not value.startswith("http"):
+                value = issue_pattern.format(value)
+            if link_pattern and kind == "link" and not value.startswith("http"):
+                value = issue_pattern.format(value)
             return Link(type=kind, name=name or value, url=value)
 
         if __is(kind, LabelType):
