fix missing logic

Author: ben-c-at-moz

l10n_CM/run_l10n.py

@@ -39,7 +39,7 @@
     "yellowkorner",
     "wish",
     "bijoubrigitte",
-    "whittard"
+    "whittard",
 }
 
 loaded_valid_sites = valid_l10n_mappings().keys()

modules/testrail.py

@@ -350,6 +350,12 @@ def create_new_plan_entry(
             payload["runs"] = runs
         return self.client.send_post(f"/add_plan_entry/{plan_id}", payload)
 
+    def update_plan(self, plan_id, **kwargs):
+        """Given a plan id, update the plan per kwargs"""
+        if not kwargs:
+            return False
+        return self.client.send_post(f"/update_plan/{plan_id}", kwargs)
+
     def update_plan_entry(self, plan_id, entry_id, **kwargs):
         """Given a plan id and entry id, update the entry per kwargs"""
         if not kwargs:

modules/testrail_integration.py

@@ -28,7 +28,7 @@ def get_execution_link() -> str:
     if "TASKCLUSTER_PROXY_URL" in os.environ:
         link = TC_EXECUTION_TEMPLATE
         for item in ["RUN_ID", "TASK_ID"]:
-            link = link.replace(f"%{item}", os.environ.get(item))
+            link = link.replace(f"%{item}%", os.environ.get(item))
         return link
 
 
@@ -355,7 +355,7 @@ def organize_l10n_entries(
 
     os_name = config.split(" ")[0]
     description = replace_link_in_description(expected_plan["description"], os_name)
-    tr.update_plan_entry(plan_id, entry["id"], description)
+    tr.update_plan(plan_id, description=description)
 
     # There should only be one entry per site per plan
     # Check that this entry has a run with the correct config