From 5aadf8c52efd84abe5195c09e5d13d3fad278ba5 Mon Sep 17 00:00:00 2001
From: AlexButeau <alexbuteau1@gmail.com>
Date: Tue, 18 Mar 2025 11:07:11 +0100
Subject: [PATCH] fix: remove unsupported report variables argument

---
 tofupilot/client.py | 6 ------
 tofupilot/plugin.py | 5 -----
 2 files changed, 11 deletions(-)

diff --git a/tofupilot/client.py b/tofupilot/client.py
index 56e7427..151c19b 100644
--- a/tofupilot/client.py
+++ b/tofupilot/client.py
@@ -74,7 +74,6 @@ def create_run(  # pylint: disable=too-many-arguments,too-many-locals
         started_at: Optional[datetime] = None,
         duration: Optional[timedelta] = None,
         sub_units: Optional[List[SubUnit]] = None,
-        report_variables: Optional[Dict[str, str]] = None,
         attachments: Optional[List[str]] = None,
     ) -> dict:
         """
@@ -99,8 +98,6 @@ def create_run(  # pylint: disable=too-many-arguments,too-many-locals
                 [A list of steps included in the test run](https://tofupilot.com/docs/steps). Default is None.
             sub_units (Optional[List[SubUnit]], optional):
                 [A list of sub-units included in the test run](https://tofupilot.com/docs/sub-units). Default is None.
-            report_variables (Optional[Dict[str, str]], optional):
-                [A dictionary of key values that will replace the procedure's {{report_variables}}](https://tofupilot.com/docs/report). Default is None.
             attachments (Optional[List[str]], optional):
                 [A list of file paths for attachments to include with the test run](https://tofupilot.com/docs/attachments). Default is None.
 
@@ -148,9 +145,6 @@ def create_run(  # pylint: disable=too-many-arguments,too-many-locals
         if sub_units is not None:
             payload["sub_units"] = sub_units
 
-        if report_variables is not None:
-            payload["report_variables"] = report_variables
-
         self._log_request("POST", "/runs", payload)
 
         try:
diff --git a/tofupilot/plugin.py b/tofupilot/plugin.py
index 86c6a53..5fb930b 100644
--- a/tofupilot/plugin.py
+++ b/tofupilot/plugin.py
@@ -22,7 +22,6 @@ def __init__(self):
         self.procedure_id: Optional[str] = None
         self.unit_under_test: Dict[str, Any] = {}
         self.sub_units: Optional[List[SubUnit]] = None
-        self.report_variables: Optional[Dict[str, str]] = None
         self.attachments: Optional[List[str]] = None
 
     def set(
@@ -33,7 +32,6 @@ def set(
         revision: Optional[str] = None,
         batch_number: Optional[str] = None,
         sub_units: Optional[List[SubUnit]] = None,
-        report_variables: Optional[Dict[str, str]] = None,
         attachments: Optional[List[str]] = None,
     ) -> None:
         if procedure_id is not None:
@@ -48,8 +46,6 @@ def set(
             self.unit_under_test["batch_number"] = batch_number
         if sub_units is not None:
             self.sub_units = sub_units
-        if report_variables is not None:
-            self.report_variables = report_variables
         if attachments is not None:
             self.attachments = attachments
         return self
@@ -184,7 +180,6 @@ def pytest_sessionfinish(self) -> None:
                 duration=duration_td,
                 steps=steps,
                 sub_units=conf.sub_units,
-                report_variables=conf.report_variables,
                 attachments=conf.attachments,
             )
         except Exception as e: