refactor(ci-insights): rename /ci/traces route with new format (#167)

This route is not used anymore

Fixes MRGFY-5715

Author: Syffe

pytest_mergify/ci_insights.py

@@ -103,9 +103,13 @@ def __post_init__(self) -> None:
             self.exporter = InMemorySpanExporter()
             span_processor = export.SimpleSpanProcessor(self.exporter)
         elif self.token and self.repo_name:
+            try:
+                owner, repo = utils.split_full_repo_name(self.repo_name)
+            except utils.InvalidRepositoryFullNameError:
+                return
             self.exporter = OTLPSpanExporter(
                 session=SessionHardRaiser(),
-                endpoint=f"{self.api_url}/v1/repos/{self.repo_name}/ci/traces",
+                endpoint=f"{self.api_url}/v1/ci/{owner}/repositories/{repo}/traces",
                 headers={"Authorization": f"Bearer {self.token}"},
                 compression=Compression.Gzip,
             )

pytest_mergify/utils.py

@@ -65,6 +65,20 @@ def get_repository_name_from_env_url(env: str) -> typing.Optional[str]:
     return None
 
 
+class InvalidRepositoryFullNameError(Exception):
+    pass
+
+
+def split_full_repo_name(
+    full_repo_name: str,
+) -> typing.Tuple[str, str]:
+    split_name = full_repo_name.split("/")
+    if len(split_name) == 2:
+        return split_name[0], split_name[1]
+
+    raise InvalidRepositoryFullNameError(f"Invalid repository name: {full_repo_name}")
+
+
 def get_repository_name() -> typing.Optional[str]:
     provider = get_ci_provider()
 

tests/conftest.py

@@ -77,12 +77,18 @@ def _run(
         )
 
         full_repository = utils.get_repository_name()
-        passthrough = responses.Response(
-            responses.POST,
-            f"{api_url}/v1/repos/{full_repository}/ci/traces",
-            passthrough=True,
-        )
-        responses.add(passthrough)
+        if full_repository is not None:
+            try:
+                owner, repo = utils.split_full_repo_name(full_repository)
+            except utils.InvalidRepositoryFullNameError:
+                pass
+            else:
+                passthrough = responses.Response(
+                    responses.POST,
+                    f"{api_url}/v1/ci/{owner}/repositories/{repo}/traces",
+                    passthrough=True,
+                )
+                responses.add(passthrough)
 
         plugin = pytest_mergify.PytestMergify()
         pytester.makepyfile(code)
@@ -111,7 +117,7 @@ class TestHTTPRequestHandler(http.server.BaseHTTPRequestHandler):
     def do_POST(self) -> None:
         path = self.path[1:].split("/")
         # loozy match, who cares
-        if path[0] == "v1" and path[-1] == "traces" and path[-2] == "ci":
+        if path[0] == "v1" and path[-1] == "traces":
             self.send_response(self.__class__.response_code)
             self.send_header("Content-Type", "text/plain")
             self.end_headers()