Fix link name.

Add More information to build info section in the report. Center and split the top report headers

Author: GilbN

ci/ci.py

@@ -375,7 +375,8 @@ def _endtest(self, container:Container, tag:str, build_info:dict[str,str], packa
             "test_results": self.tag_report_tests[tag]["test"],
             "test_success": test_success,
             "runtime": runtime,
-            "build_url": self.make_build_url(tag)
+            "build_url": self.get_build_url(tag),
+            "platform": self.get_platform(tag).upper()
             }
         self.report_containers[tag]["has_warnings"] = any(warning[1] for warning in self.report_containers[tag]["warnings"].items())
 
@@ -511,6 +512,39 @@ def get_build_version(self,container:Container,tag:str) -> str:
             self.report_status = "FAIL"
         return build_version
 
+    def get_image_name(self) -> str:
+        """Get the image name from the IMAGE env.
+
+        Returns:
+            str: The container name
+        """
+        _, container_name = self.image.split("/")
+        match self.image:
+            case _ if "lspipepr" in self.image:
+                return f"linuxserver/lspipepr-{container_name}"
+            case _ if "lsiodev" in self.image:
+                return f"linuxserver/lsiodev-{container_name}"
+            case _:
+                return self.image
+
+    def get_build_url(self, tag) -> str:
+        """Get the build url from the IMAGE env.
+
+        Args:
+            tag (str): The tag we are testing
+            
+        Returns:
+            dict: Returns a dictionary with the build url and container name
+        """
+        _, container_name = self.image.split("/")
+        match self.image:
+            case _ if "lspipepr" in self.image:
+                return f"https://ghcr.io/linuxserver/lspipepr-{container_name}:{tag}"
+            case _ if "lsiodev" in self.image:
+                return f"https://ghcr.io/linuxserver/lsiodev-{container_name}:{tag}"
+            case _:
+                return f"https://ghcr.io/{self.image}:{tag}"
+    
     def get_build_info(self,container:Container,tag:str) -> dict[str,str]:
         """Get the build information from the container object.
 
@@ -527,7 +561,9 @@ def get_build_info(self,container:Container,tag:str) -> dict[str,str]:
                 "created": "xxxx-xx-xx",
                 "size": "100MB",
                 "maintainer": "user"
-                "builder": "node"
+                "builder": "node",
+                "tag": "latest",
+                "image": "linuxserver/xxx"
             }
             ```
         """
@@ -540,7 +576,9 @@ def get_build_info(self,container:Container,tag:str) -> dict[str,str]:
                 "created": container.attrs["Config"]["Labels"]["org.opencontainers.image.created"],
                 "size": "%.2f" % float(int(container.image.attrs["Size"])/1000000) + "MB",
                 "maintainer": container.attrs["Config"]["Labels"]["maintainer"],
-                "builder": self.builder
+                "builder": self.builder,
+                "tag": tag,
+                "image": self.get_image_name()
             }
             self._add_test_result(tag, test, "PASS", "-", start_time)
             self.logger.success("Get build info on tag '%s': PASS", tag)
@@ -600,7 +638,7 @@ def report_render(self) -> None:
             report_containers=self.report_containers,
             report_status=self.report_status,
             meta_tag=self.meta_tag,
-            image=self.image,
+            image=self.get_image_name(),
             bucket=self.bucket,
             region=self.region,
             screenshot=self.screenshot,
@@ -894,23 +932,6 @@ def create_docker_client(self) -> DockerClient|None:
         except Exception:
             self.logger.error("Failed to create Docker client!")
 
-    def make_build_url(self, tag) -> str:
-        """Create a build url for the image
 
-        Args:
-            tag (str): The tag we are testing
-            
-        Returns:
-            str: Returns a build url
-        """
-        _, container_name = self.image.split("/")
-        match self.image:
-            case _ if "lspipepr" in self.image:
-                return f"https://ghcr.io/linuxserver/lspipepr-{container_name}:{tag}"
-            case _ if "lsiodev" in self.image:
-                return f"https://ghcr.io/linuxserver/lsiodev-{container_name}:{tag}"
-            case _:
-                return f"https://ghcr.io/{self.image}:{tag}"
-        
 class CIError(Exception):
     pass

ci/template.html

@@ -562,21 +562,23 @@
       <h1>Linux<span>Server</span>.io</h1>
     </header>
     <div id="results">
-      <h1>Test Results <strong>{{ image }}:{{ meta_tag }}</strong></h1>
+      <h1 style="margin-bottom: 0; text-align: center;">Test Results</h1>
+      <h2 style="margin-bottom: 0; text-align: center;"><strong>{{ image }}</strong></span></h2>
+      <h3 style="margin-top:0; margin-bottom: 0; text-align: center;"><strong>{{ meta_tag }}</strong></span></h2>
       <h2 style="margin-bottom: 0">Cumulative: <span class="report-status-{{ report_status.lower() }}">{{ report_status }}</span></h2>
       <span>Total Runtime: {{ total_runtime }}</span>
       <main>
         {% for tag in report_containers %}
         <section>
           <div class="section-header">
             {% if report_containers[tag]["test_success"] %}
-            <h3 class="section-header-status"><span class="report-status-pass">PASS</span></h3>
+            <h3 class="section-header-status"><span class="report-status-pass">{{ report_containers[tag]["platform"] }} PASS</span></h3>
             {% else %}
-            <h3 class="section-header-status"><span class="report-status-fail">FAIL</span></h3>
+            <h3 class="section-header-status"><span class="report-status-fail">{{ report_containers[tag]["platform"] }} FAIL</span></h3>
             {% endif %}
             <h2 class="section-header-h2">
               {% if report_status.lower() == "pass" %}
-                <a target="_blank" href="{{ report_containers[tag]['build_url'] }}">{{ image }}:{{ tag }}</a>
+                <a target="_blank" href="{{ report_containers[tag]['build_url'] }}">{{ image }}</a>
               {% else %}
                 {{ image }}:{{ tag }}
               {% endif %}

tests/test_ci.py

@@ -118,7 +118,9 @@ def test_get_build_info(ci: CI, mock_container: Mock):
                 "created": "2024-08-21T02:17:44+00:00",
                 "size": '275.93MB',
                 "maintainer": "Roxedus,thespad",
-                "builder": "test-node"
+                "builder": "test-node",
+                "tag": "amd64-nightly-5.10.1.9109-ls85",
+                "image": "linuxserver/test",
             }
     assert info == mock_info
 
@@ -175,11 +177,19 @@ def test_upload_file(ci: CI) -> None:
         # Upload a file to the bucket
         ci.upload_file("tests/log_blob.log", "log_blob.log", {"ContentType": "text/plain", "ACL": "public-read"})
 
-def test_make_build_url(ci: CI) -> None:
+def test_get_build_url(ci: CI) -> None:
     ci.image = "linuxserver/plex"
     tag = "amd64-nightly-5.10.1.9109-ls85"
-    assert ci.make_build_url(tag) == f"https://ghcr.io/{ci.image}:{tag}"
+    assert ci.get_build_url(tag) == f"https://ghcr.io/{ci.image}:{tag}"
     ci.image = "lsiodev/plex"
-    assert ci.make_build_url(tag) == f"https://ghcr.io/linuxserver/lsiodev-plex:{tag}"
+    assert ci.get_build_url(tag) == f"https://ghcr.io/linuxserver/lsiodev-plex:{tag}"
     ci.image = "lspipepr/plex"
-    assert ci.make_build_url(tag) == f"https://ghcr.io/linuxserver/lspipepr-plex:{tag}"
+    assert ci.get_build_url(tag) == f"https://ghcr.io/linuxserver/lspipepr-plex:{tag}"
+
+def test_get_image_name(ci: CI) -> None:
+    ci.image = "linuxserver/plex"
+    assert ci.get_image_name() == "linuxserver/plex"
+    ci.image = "lsiodev/plex"
+    assert ci.get_image_name() == "linuxserver/lsiodev-plex"
+    ci.image = "lspipepr/plex"
+    assert ci.get_image_name() == "linuxserver/lspipepr-plex"