Data model type hints

- Type hints on common data models
- Standardised internal types to string-based enums
  - Updated `ParameterMode.DEFAULT` from `None` to `"default"`
  - Updated `ALLURE_UNIQUE_LABELS` to use `LabelType` values
- Introduced missing 'Stage' enum
- Corrected `TestResult` to initialise with required UUID

Author: kieran-ryan

allure-python-commons/src/allure_commons/lifecycle.py

@@ -35,8 +35,7 @@ def _last_item_uuid(self, item_type=None):
 
     @contextmanager
     def schedule_test_case(self, uuid=None):
-        test_result = TestResult()
-        test_result.uuid = uuid or uuid4()
+        test_result = TestResult(uuid=uuid or uuid4())
         self._items[test_result.uuid] = test_result
         yield test_result
 

allure-python-commons/src/allure_commons/logger.py

@@ -4,12 +4,20 @@
 import json
 import uuid
 import shutil
+from enum import Enum
 from attr import asdict
 from allure_commons import hookimpl
 
 INDENT = 4
 
 
+def _enum_value_serializer(inst, field, value):
+    """Convert enum values to their string representation for serialization."""
+    if isinstance(value, Enum):
+        return value.value
+    return value
+
+
 class AllureFileLogger:
 
     def __init__(self, report_dir, clean=False):
@@ -21,7 +29,7 @@ def __init__(self, report_dir, clean=False):
     def _report_item(self, item):
         indent = INDENT if os.environ.get("ALLURE_INDENT_OUTPUT") else None
         filename = item.file_pattern.format(prefix=uuid.uuid4())
-        data = asdict(item, filter=lambda _, v: v or v is False)
+        data = asdict(item, filter=lambda _, v: v or v is False, value_serializer=_enum_value_serializer)
         with io.open(self._report_dir / filename, 'w', encoding='utf8') as json_file:
             json.dump(data, json_file, indent=indent, ensure_ascii=False)
 
@@ -57,12 +65,12 @@ def __init__(self):
 
     @hookimpl
     def report_result(self, result):
-        data = asdict(result, filter=lambda _, v: v or v is False)
+        data = asdict(result, filter=lambda _, v: v or v is False, value_serializer=_enum_value_serializer)
         self.test_cases.append(data)
 
     @hookimpl
     def report_container(self, container):
-        data = asdict(container, filter=lambda _, v: v or v is False)
+        data = asdict(container, filter=lambda _, v: v or v is False, value_serializer=_enum_value_serializer)
         self.test_containers.append(data)
 
     @hookimpl

allure-python-commons/src/allure_commons/mapping.py

@@ -26,7 +26,7 @@ def __is(kind, t):
 def parse_tag(tag, issue_pattern=None, link_pattern=None):
     """
     >>> parse_tag("blocker")
-    Label(name='severity', value='blocker')
+    Label(name=<LabelType.SEVERITY: 'severity'>, value='blocker')
 
     >>> parse_tag("allure.issue:http://example.com/BUG-42")
     Link(type='issue', url='http://example.com/BUG-42', name='http://example.com/BUG-42')
@@ -44,10 +44,10 @@ def parse_tag(tag, issue_pattern=None, link_pattern=None):
     Label(name='owner', value='me')
 
     >>> parse_tag("foo.label:1")
-    Label(name='tag', value='foo.label:1')
+    Label(name=<LabelType.TAG: 'tag'>, value='foo.label:1')
 
     >>> parse_tag("allure.foo:1")
-    Label(name='tag', value='allure.foo:1')
+    Label(name=<LabelType.TAG: 'tag'>, value='allure.foo:1')
     """
     sep = allure_tag_sep(tag)
     schema, value = islice(chain(tag.split(sep, 1), [None]), 2)
@@ -82,7 +82,7 @@ def labels_set(labels):
     >>> labels_set([Label(name=LabelType.SEVERITY, value=Severity.NORMAL),
     ...             Label(name=LabelType.SEVERITY, value=Severity.BLOCKER)
     ... ])
-    [Label(name='severity', value=<Severity.BLOCKER: 'blocker'>)]
+    [Label(name=<LabelType.SEVERITY: 'severity'>, value=<Severity.BLOCKER: 'blocker'>)]
 
     >>> labels_set([Label(name=LabelType.SEVERITY, value=Severity.NORMAL),
     ...             Label(name='severity', value='minor')
@@ -92,12 +92,12 @@ def labels_set(labels):
     >>> labels_set([Label(name=LabelType.EPIC, value="Epic"),
     ...             Label(name=LabelType.EPIC, value="Epic")
     ... ])
-    [Label(name='epic', value='Epic')]
+    [Label(name=<LabelType.EPIC: 'epic'>, value='Epic')]
 
     >>> labels_set([Label(name=LabelType.EPIC, value="Epic1"),
     ...             Label(name=LabelType.EPIC, value="Epic2")
     ... ])
-    [Label(name='epic', value='Epic1'), Label(name='epic', value='Epic2')]
+    [Label(name=<LabelType.EPIC: 'epic'>, value='Epic1'), Label(name=<LabelType.EPIC: 'epic'>, value='Epic2')]
     """
     class Wl:
         def __init__(self, label):

allure-python-commons/src/allure_commons/model2.py

@@ -1,6 +1,11 @@
+from __future__ import annotations
+
+from enum import Enum
+
 from attr import attrs, attrib
 from attr import Factory
 
+from allure_commons.types import AttachmentType, LabelType, LinkType, ParameterMode
 
 TEST_GROUP_PATTERN = "{prefix}-container.json"
 TEST_CASE_PATTERN = "{prefix}-result.json"
@@ -12,49 +17,49 @@
 class TestResultContainer:
     file_pattern = TEST_GROUP_PATTERN
 
-    uuid = attrib(default=None)
-    name = attrib(default=None)
-    children = attrib(default=Factory(list))
-    description = attrib(default=None)
-    descriptionHtml = attrib(default=None)
-    befores = attrib(default=Factory(list))
-    afters = attrib(default=Factory(list))
-    links = attrib(default=Factory(list))
-    start = attrib(default=None)
-    stop = attrib(default=None)
+    uuid: str = attrib(default=None)
+    name: str | None = attrib(default=None)
+    children: list[str] = attrib(default=Factory(list))
+    description: str | None = attrib(default=None)
+    descriptionHtml: str | None = attrib(default=None)
+    befores: list[TestBeforeResult] = attrib(default=Factory(list))
+    afters: list[TestAfterResult] = attrib(default=Factory(list))
+    links: list[Link] = attrib(default=Factory(list))
+    start: int | None = attrib(default=None)
+    stop: int | None = attrib(default=None)
 
 
 @attrs
 class ExecutableItem:
-    name = attrib(default=None)
-    status = attrib(default=None)
-    statusDetails = attrib(default=None)
-    stage = attrib(default=None)
-    description = attrib(default=None)
-    descriptionHtml = attrib(default=None)
-    steps = attrib(default=Factory(list))
-    attachments = attrib(default=Factory(list))
-    parameters = attrib(default=Factory(list))
-    start = attrib(default=None)
-    stop = attrib(default=None)
+    name: str | None = attrib(default=None)
+    status: Status | None = attrib(default=None)
+    statusDetails: StatusDetails | None = attrib(default=None)
+    stage: Stage | None = attrib(default=None)
+    description: str | None = attrib(default=None)
+    descriptionHtml: str | None = attrib(default=None)
+    steps: list[TestStepResult] = attrib(default=Factory(list))
+    attachments: list[Attachment] = attrib(default=Factory(list))
+    parameters: list[Parameter] = attrib(default=Factory(list))
+    start: int | None = attrib(default=None)
+    stop: int | None = attrib(default=None)
 
 
 @attrs
 class TestResult(ExecutableItem):
     file_pattern = TEST_CASE_PATTERN
 
-    uuid = attrib(default=None)
-    historyId = attrib(default=None)
-    testCaseId = attrib(default=None)
-    fullName = attrib(default=None)
-    labels = attrib(default=Factory(list))
-    links = attrib(default=Factory(list))
-    titlePath = attrib(default=Factory(list))
+    uuid: str = attrib(default=None)
+    historyId: str | None = attrib(default=None)
+    testCaseId: str | None = attrib(default=None)
+    fullName: str | None = attrib(default=None)
+    labels: list[Label] = attrib(default=Factory(list))
+    links: list[Link] = attrib(default=Factory(list))
+    titlePath: list[str] = attrib(default=Factory(list))
 
 
 @attrs
 class TestStepResult(ExecutableItem):
-    id = attrib(default=None)  # noqa: A003
+    id: str | None = attrib(default=None)  # noqa: A003
 
 
 @attrs
@@ -69,43 +74,51 @@ class TestAfterResult(ExecutableItem):
 
 @attrs
 class Parameter:
-    name = attrib(default=None)
-    value = attrib(default=None)
-    excluded = attrib(default=None)
-    mode = attrib(default=None)
+    name: str = attrib(default=None)
+    value: str = attrib(default=None)
+    excluded: bool | None = attrib(default=None)
+    mode: ParameterMode | None = attrib(default=None)
 
 
 @attrs
 class Label:
-    name = attrib(default=None)
-    value = attrib(default=None)
+    name: LabelType | str = attrib(default=None)
+    value: str = attrib(default=None)
 
 
 @attrs
 class Link:
-    type = attrib(default=None)  # noqa: A003
-    url = attrib(default=None)
-    name = attrib(default=None)
+    type: LinkType | str | None = attrib(default=None)  # noqa: A003
+    url: str = attrib(default=None)
+    name: str | None = attrib(default=None)
 
 
 @attrs
 class StatusDetails:
-    known = attrib(default=None)
-    flaky = attrib(default=None)
-    message = attrib(default=None)
-    trace = attrib(default=None)
+    known: bool | None = attrib(default=None)
+    flaky: bool | None = attrib(default=None)
+    message: str | None = attrib(default=None)
+    trace: str | None = attrib(default=None)
 
 
 @attrs
 class Attachment:
-    name = attrib(default=None)
-    source = attrib(default=None)
-    type = attrib(default=None)  # noqa: A003
+    name: str = attrib(default=None)
+    source: str = attrib(default=None)
+    type: AttachmentType | str | None = attrib(default=None)  # noqa: A003
 
 
-class Status:
+class Status(str, Enum):
     FAILED = 'failed'
     BROKEN = 'broken'
     PASSED = 'passed'
     SKIPPED = 'skipped'
     UNKNOWN = 'unknown'
+
+
+class Stage(str, Enum):
+    SCHEDULED = "scheduled"
+    RUNNING = "running"
+    FINISHED = "finished"
+    PENDING = "pending"
+    INTERRUPTED = "interrupted"

allure-python-commons/src/allure_commons/types.py

@@ -1,6 +1,6 @@
-from enum import Enum
+from __future__ import annotations
 
-ALLURE_UNIQUE_LABELS = ['severity', 'thread', 'host']
+from enum import Enum
 
 
 class Severity(str, Enum):
@@ -11,13 +11,13 @@ class Severity(str, Enum):
     TRIVIAL = 'trivial'
 
 
-class LinkType:
+class LinkType(str, Enum):
     LINK = 'link'
     ISSUE = 'issue'
     TEST_CASE = 'tms'
 
 
-class LabelType(str):
+class LabelType(str, Enum):
     EPIC = 'epic'
     FEATURE = 'feature'
     STORY = 'story'
@@ -34,9 +34,12 @@ class LabelType(str):
     MANUAL = 'ALLURE_MANUAL'
 
 
+ALLURE_UNIQUE_LABELS = [LabelType.SEVERITY, LabelType.THREAD, LabelType.HOST]
+
+
 class AttachmentType(Enum):
 
-    def __init__(self, mime_type, extension):
+    def __init__(self, mime_type: str, extension: str) -> None:
         self.mime_type = mime_type
         self.extension = extension
 
@@ -66,7 +69,7 @@ def __init__(self, mime_type, extension):
     PDF = ("application/pdf", "pdf")
 
 
-class ParameterMode(Enum):
+class ParameterMode(str, Enum):
     HIDDEN = 'hidden'
     MASKED = 'masked'
-    DEFAULT = None
+    DEFAULT = 'default'

allure-robotframework/src/listener/utils.py

@@ -26,13 +26,13 @@ def get_allure_parameters(parameters):
 def get_allure_suites(longname):
     """
     >>> get_allure_suites('Suite1.Test')
-    [Label(name='suite', value='Suite1')]
+    [Label(name=<LabelType.SUITE: 'suite'>, value='Suite1')]
     >>> get_allure_suites('Suite1.Suite2.Test') # doctest: +NORMALIZE_WHITESPACE
-    [Label(name='suite', value='Suite1'), Label(name='subSuite', value='Suite2')]
+    [Label(name=<LabelType.SUITE: 'suite'>, value='Suite1'), Label(name=<LabelType.SUB_SUITE: 'subSuite'>, value='Suite2')]
     >>> get_allure_suites('Suite1.Suite2.Suite3.Test') # doctest: +NORMALIZE_WHITESPACE
-    [Label(name='parentSuite', value='Suite1'),
-    Label(name='suite', value='Suite2'),
-    Label(name='subSuite', value='Suite3')]
+    [Label(name=<LabelType.PARENT_SUITE: 'parentSuite'>, value='Suite1'),
+    Label(name=<LabelType.SUITE: 'suite'>, value='Suite2'),
+    Label(name=<LabelType.SUB_SUITE: 'subSuite'>, value='Suite3')]
     """
     labels = []
     suites = longname.split('.')