From 1aa60187c380875f714c61931c3600d90e5f753f Mon Sep 17 00:00:00 2001
From: Dale W <dale.wells@satellitevu.com>
Date: Tue, 3 Jun 2025 09:59:36 +0100
Subject: [PATCH] chore: use suite based architecture for tests

---
 satellitevu/apis/base_test.py    | 4 ++--
 satellitevu/apis/catalog_test.py | 4 ++--
 satellitevu/apis/id_test.py      | 4 ++--
 satellitevu/apis/orders_test.py  | 4 ++--
 satellitevu/apis/otm_test.py     | 4 ++--
 satellitevu/auth/auth_test.py    | 4 ++--
 satellitevu/auth/cache_test.py   | 4 ++--
 satellitevu/conftest.py          | 2 +-
 satellitevu/http/http_test.py    | 4 ++--
 9 files changed, 17 insertions(+), 17 deletions(-)

diff --git a/satellitevu/apis/base_test.py b/satellitevu/apis/base_test.py
index 004d4c3..98e9d3f 100644
--- a/satellitevu/apis/base_test.py
+++ b/satellitevu/apis/base_test.py
@@ -1,4 +1,4 @@
-from allure import description, feature, title
+from allure import description, title, suite
 from json import dumps
 from typing import List
 from urllib.parse import parse_qs
@@ -19,7 +19,7 @@ class TestApi(AbstractApi):
     scopes = ["test"]
 
 
-@feature("Base")
+@suite("Base")
 @title("Scopes")
 @description("Test that the correct scopes are sent in the request")
 @mark.parametrize("kwargs", ({}, {"scopes": ["foo"]}))
diff --git a/satellitevu/apis/catalog_test.py b/satellitevu/apis/catalog_test.py
index b4c3393..970a582 100644
--- a/satellitevu/apis/catalog_test.py
+++ b/satellitevu/apis/catalog_test.py
@@ -1,4 +1,4 @@
-from allure import description, feature, title
+from allure import description, title, suite
 from datetime import datetime, timezone
 from json import dumps
 from urllib.parse import urlparse
@@ -14,7 +14,7 @@
 API_PATH = "catalog/v1/contract-id/"
 
 
-@feature("Catalog")
+@suite("Catalog")
 class TestCatalog:
     @mocketize(strict_mode=True)
     @mark.parametrize(
diff --git a/satellitevu/apis/id_test.py b/satellitevu/apis/id_test.py
index 8f85860..c3927ea 100644
--- a/satellitevu/apis/id_test.py
+++ b/satellitevu/apis/id_test.py
@@ -1,4 +1,4 @@
-from allure import description, feature, title
+from allure import description, title, suite
 import json
 import re
 from urllib.parse import urlparse
@@ -13,7 +13,7 @@
 
 @mark.parametrize("pact", ["id-service"], indirect=True)
 @mark.usefixtures("mocketize_fixture")
-@feature("ID")
+@suite("ID")
 class TestID:
     @title("Get user details")
     @description("Retrieve the details of a user.")
diff --git a/satellitevu/apis/orders_test.py b/satellitevu/apis/orders_test.py
index 370d705..5abd4e9 100644
--- a/satellitevu/apis/orders_test.py
+++ b/satellitevu/apis/orders_test.py
@@ -1,4 +1,4 @@
-from allure import description, feature, title
+from allure import description, title, suite
 import tempfile
 from io import BytesIO
 from json import dumps
@@ -18,7 +18,7 @@
 
 
 @mark.usefixtures("mocketize_fixture")
-@feature("Orders")
+@suite("Orders")
 class TestOrders:
     @title("Submit order (single item)")
     @description("Submit an order for a single item")
diff --git a/satellitevu/apis/otm_test.py b/satellitevu/apis/otm_test.py
index 137ba45..36434c6 100644
--- a/satellitevu/apis/otm_test.py
+++ b/satellitevu/apis/otm_test.py
@@ -1,4 +1,4 @@
-from allure import description, feature, title
+from allure import description, title, suite
 from json import dumps, loads
 from itertools import product
 from secrets import token_urlsafe
@@ -16,7 +16,7 @@
 API_PATH_ORDERS = "otm/v2/contract-id/tasking/orders/"
 
 
-@feature("Tasking")
+@suite("Tasking")
 @mark.usefixtures("mocketize_fixture")
 class TestTasking:
     @title("Cannot use OTM v2 without contract ID")
diff --git a/satellitevu/auth/auth_test.py b/satellitevu/auth/auth_test.py
index 2195842..226c092 100644
--- a/satellitevu/auth/auth_test.py
+++ b/satellitevu/auth/auth_test.py
@@ -1,4 +1,4 @@
-from allure import feature, title
+from allure import title, suite
 from base64 import urlsafe_b64encode
 from datetime import datetime
 from json import dumps
@@ -14,7 +14,7 @@
 from .exc import AuthError
 
 
-@feature("Auth")
+@suite("Auth")
 @mark.usefixtures("mocketize_fixture")
 class TestAuth:
     def _encode(self, claims: Dict[str, any]) -> str:
diff --git a/satellitevu/auth/cache_test.py b/satellitevu/auth/cache_test.py
index 4d7e3cb..f5ffd4c 100644
--- a/satellitevu/auth/cache_test.py
+++ b/satellitevu/auth/cache_test.py
@@ -1,4 +1,4 @@
-from allure import feature, title
+from allure import title, suite
 from configparser import ConfigParser
 from pathlib import Path
 
@@ -9,7 +9,7 @@
 TEST_DIR = Path("/test")
 
 
-@feature("Cache")
+@suite("Cache")
 class TestCache:
     @title("Empty cache save")
     def test_empty_cache_save(self, fs: FakeFilesystem):
diff --git a/satellitevu/conftest.py b/satellitevu/conftest.py
index 2b359de..20ca84d 100644
--- a/satellitevu/conftest.py
+++ b/satellitevu/conftest.py
@@ -712,4 +712,4 @@ def pact(request) -> Generator[Pact, None, None]:
 def allure_metadata():
     """Allure labels to be added at runtime"""
     dynamic.label("layer", "unit")
-    dynamic.epic("Python SDK")
+    dynamic.parent_suite("Python SDK")
diff --git a/satellitevu/http/http_test.py b/satellitevu/http/http_test.py
index 59c342f..4b15076 100644
--- a/satellitevu/http/http_test.py
+++ b/satellitevu/http/http_test.py
@@ -1,4 +1,4 @@
-from allure import feature, title
+from allure import title, suite
 from importlib.metadata import version
 from json import dumps
 from unittest.mock import Mock
@@ -12,7 +12,7 @@
 from . import ResponseWrapper, UrllibClient
 
 
-@feature("HTTP")
+@suite("HTTP")
 class TestHttp:
     @title("HTTP client")
     @mark.parametrize("method", ("GET", "POST"))