Merge pull request 'Show author info' (#40) from feature/main-38-JenkinsAPIUsage into main

Reviewed-on: #40

Jenkinsfile.cd

@@ -9,7 +9,7 @@ pipeline {
   parameters {
     string(
       name: 'JENKINS_BASE_URL',
-      defaultValue: 'https://openbokeron.org/jenkins',
+      defaultValue: 'https://openbokeron.uma.es/jenkins',
       description: 'Base URL del Jenkins objetivo'
     )
     string(
@@ -36,32 +36,6 @@ pipeline {
 
   stages {
 
-    /* =========================
-       TESTS
-       ========================= */
-
-    stage('Backend: test (main)') {
-      agent {
-        docker {
-          image 'python:3.11-slim'
-          args '-u root'
-        }
-      }
-      steps {
-        dir('backend') {
-          sh '''
-            set -e
-            python -m venv .venv
-            . .venv/bin/activate
-            pip install --upgrade pip
-            pip install -r requirements-dev.txt
-            pytest
-          '''
-        }
-      }
-    }
-
-
     /* =========================
        DOCKER BUILD
        ========================= */
@@ -100,6 +74,22 @@ pipeline {
       }
     }
 
+    /* =========================
+       TRIVY
+       ========================= */
+
+    stage('Security: Trivy job') {
+      agent any
+      steps {
+        build job: 'Trivy Scanner',
+          parameters: [
+            string(name: 'APP_VERSION', value: "${APP_VERSION}")
+          ],
+          propagate: true,
+          wait: true
+        }
+    }
+
     /* =========================
        DEPLOY
        ========================= */

Jenkinsfile.ci

@@ -38,11 +38,12 @@ pipeline {
       agent {
         docker {
           image 'python:3.11-slim'
-          args '-u root'
         }
       }
 
       environment {
+        HOME = "${WORKSPACE}"
+        PIP_CACHE_DIR = "${WORKSPACE}/.cache/pip"
         PYTHONDONTWRITEBYTECODE = 1
       }
 
@@ -50,12 +51,13 @@ pipeline {
         dir('backend') {
           sh '''
             set -e
+            mkdir -p "$PIP_CACHE_DIR" "$WORKSPACE/.cache/pytest"
             python -m venv .venv
             . .venv/bin/activate
             pip install --upgrade pip
             pip install -r requirements-dev.txt
             ruff check app tests
-            pytest
+            pytest -o cache_dir="$WORKSPACE/.cache/pytest" --junitxml=pytest.xml
           '''
         }
       }
@@ -71,23 +73,95 @@ pipeline {
           image 'node:20-slim'
         }
       }
+      environment {
+        HOME = "${WORKSPACE}"
+        NPM_CONFIG_CACHE = "${WORKSPACE}/.cache/npm"
+      }
       steps {
         dir('frontend') {
           sh '''
             set -e
+            mkdir -p "$NPM_CONFIG_CACHE"
             npm install --no-progress --no-audit --prefer-offline
             npm run check
+            mkdir -p test-results
+            npm test
             npm run build
           '''
         }
       }
     }
 
-    stage('Cleanup') {
+  }
-        agent any
+
-        steps {
+post {
-            cleanWs()
+  always {
+    script {
+      node {
+        junit testResults: 'backend/pytest.xml', allowEmptyResults: true
+        junit testResults: 'frontend/test-results/junit.xml', allowEmptyResults: true
+
+        if (env.CHANGE_ID) {
+          def giteaBase = 'https://openbokeron.uma.es'
+          def owner = 'OpenBokeron'
+          def repo  = 'TallerCiCd'
+          def pr    = env.CHANGE_ID
+
+          def result = currentBuild.currentResult
+          def msg = ""
+
+          if (result == "SUCCESS") {
+            msg = """
+            ✅ Todo en orden, camarada.
+
+            - URL: ${env.BUILD_URL}
+            """.stripIndent().trim()
+          }
+          else if (result == "FAILURE") {
+            msg = """
+            ❌ Alto ahí. ¿Qué clase de crímenes de guerra has cometido en esta PR?
+
+            - URL: ${env.BUILD_URL}
+            """.stripIndent().trim()
+            }
+          else {
+            msg = """
+            ⚠️ Algo ha pasado, no sé si es culpa tuyao mía. Por probabilidad, pensaré que la has pifiado tú.
+
+            - URL: ${env.BUILD_URL}
+            """.stripIndent().trim()
+            }
+
+          def commentsUrl = "${giteaBase}/gitea/api/v1/repos/${owner}/${repo}/issues/${pr}/comments"
+
+          withCredentials([usernamePassword(credentialsId: 'jenkins-bot-api',
+                                  usernameVariable: 'GITEA_USER',
+                                  passwordVariable: 'GITEA_TOKEN')]) {
+
+          def body = msg.replace('\\', '\\\\').replace('"','\\"').replace('\n','\\n')
+
+          // Avoid interpolation of secret variables (https://www.jenkins.io/doc/book/pipeline/jenkinsfile/#string-interpolation)
+          withEnv(["GITEA_COMMENTS_URL=${commentsUrl}", "GITEA_BODY=${body}"]) {
+            sh(label: 'Comentar en PR (Gitea)', script: '''
+              curl -sS -X POST \
+              -H "Authorization: token $GITEA_TOKEN" \
+              -H "Content-Type: application/json" \
+              --data-binary @- \
+                "$GITEA_COMMENTS_URL" <<JSON
+                {"body":"$GITEA_BODY"}
+                JSON
+            ''')
+            }
         }
+        } else {
+          echo "No es build de PR (CHANGE_ID vacío); no comento."
+        }
+
+        cleanWs()
+      }
     }
   }
 }
+
+
+}

backend/app/data/build_history.json

@@ -1,44 +0,0 @@
-{
-    "builds": [
-        {
-            "number": 205,
-            "status": "success",
-            "branch": "main",
-            "commit": "9ac3f91",
-            "author": "Miau",
-            "finished_at": "2024-05-04T10:20:00Z",
-            "duration_seconds": 312
-        },
-        {
-            "number": 204,
-            "status": "failed",
-            "branch": "feature/nosetioestoesunmock",
-            "commit": "75c4ba2",
-            "author": "Miau",
-            "finished_at": "2024-05-04T09:50:00Z",
-            "duration_seconds": 188,
-            "failed_stage": "tests",
-            "fun_message": "woops"
-        },
-        {
-            "number": 203,
-            "status": "failed",
-            "branch": "main",
-            "commit": "512ca7e",
-            "author": "Miau",
-            "finished_at": "2024-05-04T09:10:00Z",
-            "duration_seconds": 140,
-            "failed_stage": "lint",
-            "fun_message": "Nadie pasa en local el linter"
-        },
-        {
-            "number": 202,
-            "status": "success",
-            "branch": "hotfix/tehedichoqueestoesunmock?",
-            "commit": "c73d8ab",
-            "author": "Miau",
-            "finished_at": "2024-05-03T18:30:00Z",
-            "duration_seconds": 276
-        }
-    ]
-}

backend/app/main.py

@@ -16,8 +16,10 @@
 
 import time
 
-from fastapi import FastAPI
+import requests
+from fastapi import FastAPI, status
 from fastapi.middleware.cors import CORSMiddleware
+from fastapi.responses import JSONResponse
 
 from app.services.builds import build_history
 from app.services.menu import build_menu
@@ -55,7 +57,7 @@ def health():
         "commit": settings.git_commit,
         "build": settings.build_number,
         "author": settings.commit_author,
-        "uptime_seconds": uptime()
+        "uptime_seconds": uptime(),
     }
 
 
@@ -76,4 +78,10 @@ def price_for_item(item: str):
 
 @app.get("/builds")
 def builds():
-    return build_history()
+    try:
+        return build_history()
+    except (requests.RequestException, ValueError):
+        return JSONResponse(
+            status_code=status.HTTP_503_SERVICE_UNAVAILABLE,
+            content={"builds": [], "error": "jenkins_unavailable"},
+        )

backend/app/services/builds.py

@@ -15,69 +15,116 @@
 # along with this program.  If not, see <https://www.gnu.org/licenses/>.
 
 import base64
-import json
+import html
-from pathlib import Path
+import re
-from typing import Dict, List
+from typing import Dict, List, Tuple
 
 import requests
 
 from app.settings import settings
 
-DATA_DIR = Path(__file__).resolve().parent.parent / "data"
-
-
-def _load_json(filename: str) -> Dict:
-    path = DATA_DIR / filename
-    with open(path, encoding="utf-8") as file:
-        return json.load(file)
-
 
 def _sort_builds(builds: List[Dict]) -> List[Dict]:
     return sorted(builds, key=lambda build: build.get("number", 0), reverse=True)
 
-def normalize_build(build: Dict) -> Dict:
-    changes = build.get("changeSets", [])
-    commits = []
 
-    for cs in changes:
+def _extract_commits(build: Dict) -> List[Dict]:
+    commits = []
+    for cs in build.get("changeSets", []):
         for item in cs.get("items", []):
-            commits.append({
+            commits.append(
-                "commit": item.get("commitId", "")[:7],
+                {
-                "message": item.get("msg", ""),
+                    "commit": item.get("commitId", "")[:7],
-                "author": item.get("author", {}).get("fullName", "unknown"),
+                    "message": item.get("msg", ""),
-            })
+                    "author": item.get("author", {}).get("fullName", "unknown"),
+                }
+            )
+    return commits
+
+
+def _extract_trigger(build: Dict) -> str:
+    for action in build.get("actions", []):
+        for cause in action.get("causes", []):
+            description = cause.get("shortDescription")
+            if description:
+                return html.unescape(description).strip()
+    return ""
+
+
+def _summarize_trigger(trigger: str) -> Tuple[str, str, str]:
+    if not trigger:
+        return "", "", ""
+
+    pr_url = ""
+    for line in trigger.splitlines():
+        if line.strip().lower().startswith("reviewed-on:"):
+            pr_url = line.split(":", 1)[-1].strip()
+            break
+
+    author_match = re.search(r"\(([^)]+)\)\.", trigger)
+    author = author_match.group(1).strip() if author_match else ""
+
+    pr_match = re.search(r"#(\d+)", trigger)
+    title_match = re.search(r"Merge pull request '([^']+)'", trigger)
+
+    if pr_match:
+        pr_number = pr_match.group(1)
+        title = title_match.group(1).strip() if title_match else ""
+        if title:
+            return f"Merge PR #{pr_number} · {title}", pr_url, author
+        return f"Merge PR #{pr_number}", pr_url, author
+
+    return trigger.splitlines()[0].strip(), pr_url, author
+
+
+def normalize_build(build: Dict) -> Dict:
+    commits = _extract_commits(build)
+    trigger = _extract_trigger(build)
+    trigger_label, trigger_url, trigger_author = _summarize_trigger(trigger)
+    status = (build.get("result") or "RUNNING").lower()
+    if build.get("building"):
+        status = "running"
 
     return {
         "number": build.get("number"),
-        "status": (build.get("result") or "RUNNING").lower(),
+        "status": status,
         "finished_at": build.get("timestamp"),
         "duration_seconds": build.get("duration", 0) // 1000,
         "url": build.get("url"),
         "commits": commits,
+        "trigger": trigger,
+        "trigger_label": trigger_label,
+        "trigger_url": trigger_url,
+        "trigger_author": trigger_author,
     }
 
 
 def _auth_header() -> Dict[str, str]:
+    if not settings.jenkins_user or not settings.jenkins_token:
+        return {}
     token = f"{settings.jenkins_user}:{settings.jenkins_token}"
     encoded = base64.b64encode(token.encode()).decode()
     return {"Authorization": f"Basic {encoded}"}
 
+
 def fetch_builds(limit: int = 5) -> List[Dict]:
+    if not settings.jenkins_job_name:
+        raise ValueError("JENKINS_JOB_NAME not configured")
     url = (
         f"{settings.jenkins_base_url}/job/{settings.jenkins_job_name}/api/json"
-        "?tree=builds[number,url,result,timestamp,duration,"
+        "?tree=builds[number,url,result,timestamp,duration,building,"
-        "changesets[items[commitId,msg,author[fullName]]]]"
+        "changeSets[items[commitId,msg,author[fullName]]],"
+        "actions[causes[shortDescription]]]"
     )
 
-    resp = requests.get(url, headers = _auth_header(), timeout=5)
+    resp = requests.get(url, headers=_auth_header(), timeout=5)
     resp.raise_for_status()
 
     builds = resp.json().get("builds", [])
-    return builds[:limit]
+    return _sort_builds(builds)[:limit]
+
 
 def build_history() -> Dict:
     """Return Jenkins build history data."""
     builds = fetch_builds()
-    return {
+    return {"builds": [normalize_build(b) for b in builds]}
-        "builds": [normalize_build(b) for b in builds]
-    }

backend/app/services/menu.py

@@ -54,6 +54,7 @@ def _pick_mains(count: int = ITEMS_PER_SECTION) -> List[str]:
     random.shuffle(mains)
     return mains
 
+
 def _pick_garnish() -> List[str]:
     garnish_options = MENU_SOURCE["mains"]["garnish"]
 
@@ -69,6 +70,7 @@ def _build_alternative() -> Dict:
         "note": alternative.get("note", ""),
     }
 
+
 def build_menu(items_per_section: int = ITEMS_PER_SECTION) -> Dict:
     today = datetime.now()
 

backend/app/settings.py

@@ -24,10 +24,7 @@ class RuntimeConfig:
     git_commit: str = os.getenv("GIT_COMMIT", "local")
     build_number: str = os.getenv("BUILD_NUMBER", "-")
     commit_author: str = os.getenv("COMMIT_AUTHOR", "local")
-    jenkins_base_url: str = os.getenv(
+    jenkins_base_url: str = os.getenv("JENKINS_BASE_URL", "http://localhost:8080").rstrip("/")
-        "JENKINS_BASE_URL",
-        "http://localhost:8080"
-    ).rstrip("/")
     jenkins_job_name: str = os.getenv("JENKINS_JOB_NAME", "")
     jenkins_user: str = os.getenv("JENKINS_USER", "")
     jenkins_token: str = os.getenv("JENKINS_TOKEN", "")

backend/tests/test_api.py

@@ -14,6 +14,7 @@
 # You should have received a copy of the GNU General Public License
 # along with this program.  If not, see <https://www.gnu.org/licenses/>.
 
+import requests
 from fastapi.testclient import TestClient
 
 from app.main import app
@@ -60,6 +61,15 @@ def test_build_history(monkeypatch):
             "timestamp": 1719992400000,
             "duration": 75000,
             "url": "http://jenkins.local/job/demo/205",
+            "actions": [
+                {
+                    "causes": [
+                        {
+                            "shortDescription": "Triggered by Merge pull request #37",
+                        }
+                    ]
+                }
+            ],
             "changeSets": [
                 {
                     "items": [
@@ -78,6 +88,7 @@ def test_build_history(monkeypatch):
             "timestamp": 1719988800000,
             "duration": 1500,
             "url": "http://jenkins.local/job/demo/204",
+            "actions": [],
             "changeSets": [],
         },
     ]
@@ -100,9 +111,30 @@ def test_build_history(monkeypatch):
     assert first["commits"] == [
         {"commit": "9ac3f91", "message": "Anade la API de Jenkins", "author": "Dev One"}
     ]
+    assert first["trigger"] == "Triggered by Merge pull request #37"
+    assert first["trigger_label"] == "Merge PR #37"
+    assert first["trigger_url"] == ""
+    assert first["trigger_author"] == ""
 
     second = builds[1]
     assert second["number"] == 204
     assert second["status"] == "running"
     assert second["duration_seconds"] == 1
     assert second["commits"] == []
+    assert second["trigger"] == ""
+    assert second["trigger_label"] == ""
+    assert second["trigger_url"] == ""
+    assert second["trigger_author"] == ""
+
+
+def test_build_history_error_returns_empty(monkeypatch):
+    def raise_error(limit=5):
+        raise requests.RequestException("boom")
+
+    monkeypatch.setattr("app.services.builds.fetch_builds", raise_error)
+
+    response = client.get("/builds")
+    assert response.status_code == 503
+    body = response.json()
+    assert body["builds"] == []
+    assert body["error"] == "jenkins_unavailable"

frontend/.gitignore

@@ -6,3 +6,4 @@ dist
 *.log
 .svelte-kit
 vite.config.ts.timestamp-*
+TESTS.md

frontend/package.json

@@ -8,7 +8,7 @@
         "build": "vite build",
         "preview": "vite preview",
         "check": "svelte-check",
-        "test": "vitest"
+        "test": "vitest run --reporter=default --reporter=junit --outputFile=./test-results/junit.xml"
     },
     "devDependencies": {
         "@sveltejs/vite-plugin-svelte": "^3.0.1",

frontend/src/App.svelte

@@ -377,12 +377,25 @@ along with this program.  If not, see <https://www.gnu.org/licenses/>.
                                     <span class={`status-dot ${build.status}`}></span>
                                     <span class="summary-title">#{build.number}</span>
 
-                                    {#if build.commits.length === 0}
+                                    {#if build.trigger_label || build.trigger}
-                                        <span class="muted">Ejecución manual</span>
+                                        <div class="summary-trigger">
-                                    {:else}
+                                            {#if build.trigger_url}
+                                                <a href={build.trigger_url} target="_blank" rel="noreferrer">
+                                                    {build.trigger_label || build.trigger}
+                                                </a>
+                                            {:else}
+                                                <span>{build.trigger_label || build.trigger}</span>
+                                            {/if}
+                                            {#if build.trigger_author}
+                                                <span class="trigger-author">{build.trigger_author}</span>
+                                            {/if}
+                                        </div>
+                                    {:else if build.commits.length > 0}
                                         <span class="summary-commit">
                                             {build.commits[0].commit} · {build.commits[0].author}
                                         </span>
+                                    {:else}
+                                        <span class="muted">Ejecución manual</span>
                                     {/if}
                                 </div>
 
@@ -394,7 +407,36 @@ along with this program.  If not, see <https://www.gnu.org/licenses/>.
                             </summary>
 
                             <div class="history-body">
+                                {#if build.trigger_label || build.trigger}
+                                    <p class="history-row">
+                                        <span>Disparo</span>
+                                        <span>
+                                            {#if build.trigger_url}
+                                                <a
+                                                    class="history-link"
+                                                    href={build.trigger_url}
+                                                    target="_blank"
+                                                    rel="noreferrer"
+                                                >
+                                                    {build.trigger_label || build.trigger}
+                                                </a>
+                                            {:else}
+                                                {build.trigger_label || build.trigger}
+                                            {/if}
+                                            {#if build.trigger_author}
+                                                <span class="trigger-author">{build.trigger_author}</span>
+                                            {/if}
+                                        </span>
+                                    </p>
+                                {/if}
+
                                 {#if build.commits.length > 0}
+                                    <p class="history-row">
+                                        <span>Commit</span>
+                                        <span class="mono">
+                                            {build.commits[0].commit} · {build.commits[0].author}
+                                        </span>
+                                    </p>
                                     <p class="history-row">
                                         <span>Mensaje</span>
                                         <span>{build.commits[0].message}</span>
@@ -408,6 +450,8 @@ along with this program.  If not, see <https://www.gnu.org/licenses/>.
 
                                 {#if build.status === 'failure'}
                                     <p class="history-message danger">Build fallida</p>
+                                {:else if build.status === 'running'}
+                                    <p class="history-message running">En curso</p>
                                 {:else}
                                     <p class="history-message success">Correcto</p>
                                 {/if}

frontend/src/App.test.js

@@ -0,0 +1,127 @@
+/*
+CI/CD Workshop
+Copyright (C) 2025 OpenBokeron
+
+This program is free software: you can redistribute it and/or modify
+it under the terms of the GNU General Public License as published by
+the Free Software Foundation, either version 3 of the License, or
+(at your option) any later version.
+
+This program is distributed in the hope that it will be useful,
+but WITHOUT ANY WARRANTY; without even the implied warranty of
+MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+GNU General Public License for more details.
+
+You should have received a copy of the GNU General Public License
+along with this program.  If not, see <https://www.gnu.org/licenses/>.
+*/
+
+import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
+import { render, screen, waitFor } from '@testing-library/svelte';
+import App from './App.svelte';
+
+const mockMenu = {
+    starters: ['Sopa de cocido', 'Ensalada mixta'],
+    mains: ['Pescado al vapor', 'Pollo asado'],
+    garnish: ['Patatas fritas'],
+    desserts: ['Fruta del tiempo', 'Yogur'],
+    notes: ['Pan y bebida incluidos'],
+    menu_price: 5.5,
+    university_deal: {
+        old_price: 4.5,
+        current_price: 5.5,
+        note: 'Gracias al convenio con la Universidad',
+    },
+    alternative: { title: 'Plato alternativo', items: [], price: 5.0 },
+    availability: { last_updated: '12:00' },
+    espetos_tip: 'Hoy toca sardinas',
+};
+
+const mockPrices = {
+    items: [
+        { item: 'cafe', price: 1.2, currency: 'EUR', generated_at: '10:00' },
+        { item: 'tostada', price: 2.0, currency: 'EUR', generated_at: '10:00' },
+    ],
+};
+
+const mockHealth = {
+    status: 'ok',
+    build: 42,
+    commit: 'abc1234def5678',
+    author: 'Test Author',
+    uptime_seconds: 3600,
+};
+
+const mockBuilds = {
+    builds: [
+        {
+            number: 42,
+            status: 'success',
+            finished_at: Date.now(),
+            duration_seconds: 60,
+            commits: [{ commit: 'abc1234', message: 'Fix bug', author: 'Dev' }],
+        },
+    ],
+};
+
+function createMockFetch(url) {
+    if (url.includes('/menu')) {
+        return Promise.resolve({ ok: true, json: () => Promise.resolve(mockMenu) });
+    }
+    if (url.includes('/prices')) {
+        return Promise.resolve({ ok: true, json: () => Promise.resolve(mockPrices) });
+    }
+    if (url.includes('/health')) {
+        return Promise.resolve({ ok: true, json: () => Promise.resolve(mockHealth) });
+    }
+    if (url.includes('/builds')) {
+        return Promise.resolve({ ok: true, json: () => Promise.resolve(mockBuilds) });
+    }
+    return Promise.resolve({ ok: false, status: 404 });
+}
+
+describe('App.svelte', () => {
+    beforeEach(() => {
+        globalThis.fetch = vi.fn(createMockFetch);
+    });
+
+    afterEach(() => {
+        vi.restoreAllMocks();
+    });
+
+    it('renderiza el titulo principal', async () => {
+        render(App);
+
+        await waitFor(() => {
+            expect(screen.getByRole('heading', { level: 1 })).toBeInTheDocument();
+        });
+    });
+
+    it('renderiza la seccion hero con eyebrow', async () => {
+        render(App);
+
+        await waitFor(() => {
+            expect(screen.getByText('Taller CI/CD con Jenkins')).toBeInTheDocument();
+        });
+    });
+
+    it('renderiza los botones de accion', async () => {
+        render(App);
+
+        await waitFor(() => {
+            expect(screen.getByText('Refrescar menú')).toBeInTheDocument();
+            expect(screen.getByText('Recalcular desayunos')).toBeInTheDocument();
+        });
+    });
+
+    it('renderiza la tarjeta de Open Bokeron', async () => {
+        render(App);
+
+        await waitFor(() => {
+            expect(screen.getByText('Open Bokeron')).toBeInTheDocument();
+            expect(
+                screen.getByText('Somos Open Bokeron, la asociación de software libre de la ETSII.')
+            ).toBeInTheDocument();
+        });
+    });
+});

frontend/src/services/api.test.js

@@ -0,0 +1,123 @@
+/*
+CI/CD Workshop
+Copyright (C) 2025 OpenBokeron
+
+This program is free software: you can redistribute it and/or modify
+it under the terms of the GNU General Public License as published by
+the Free Software Foundation, either version 3 of the License, or
+(at your option) any later version.
+
+This program is distributed in the hope that it will be useful,
+but WITHOUT ANY WARRANTY; without even the implied warranty of
+MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+GNU General Public License for more details.
+
+You should have received a copy of the GNU General Public License
+along with this program.  If not, see <https://www.gnu.org/licenses/>.
+*/
+
+import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
+import { getMenu, getPrices, getCiStatus, getBuildHistory } from './api';
+
+describe('API services', () => {
+    beforeEach(() => {
+        vi.stubGlobal('fetch', vi.fn());
+    });
+
+    afterEach(() => {
+        vi.unstubAllGlobals();
+    });
+
+    describe('getMenu', () => {
+        it('devuelve datos del menu cuando la API responde correctamente', async () => {
+            const mockMenu = {
+                starters: ['Sopa'],
+                mains: ['Pescado al vapor'],
+                desserts: ['Fruta'],
+            };
+
+            fetch.mockResolvedValueOnce({
+                ok: true,
+                json: () => Promise.resolve(mockMenu),
+            });
+
+            const result = await getMenu();
+
+            expect(fetch).toHaveBeenCalledWith('/taller/api/menu');
+            expect(result).toEqual(mockMenu);
+        });
+
+        it('lanza error cuando la API falla', async () => {
+            fetch.mockResolvedValueOnce({
+                ok: false,
+                status: 500,
+            });
+
+            await expect(getMenu()).rejects.toThrow('Respuesta no valida del servidor');
+        });
+    });
+
+    describe('getPrices', () => {
+        it('devuelve array de items cuando la API responde', async () => {
+            const mockResponse = {
+                items: [
+                    { item: 'cafe', price: 1.2 },
+                    { item: 'tostada', price: 2.0 },
+                ],
+            };
+
+            fetch.mockResolvedValueOnce({
+                ok: true,
+                json: () => Promise.resolve(mockResponse),
+            });
+
+            const result = await getPrices();
+
+            expect(fetch).toHaveBeenCalledWith('/taller/api/prices');
+            expect(result).toEqual(mockResponse.items);
+        });
+
+        it('devuelve array vacio si no hay items', async () => {
+            fetch.mockResolvedValueOnce({
+                ok: true,
+                json: () => Promise.resolve({}),
+            });
+
+            const result = await getPrices();
+
+            expect(result).toEqual([]);
+        });
+    });
+
+    describe('getCiStatus', () => {
+        it('llama al endpoint /health', async () => {
+            const mockHealth = { status: 'ok', build: 42 };
+
+            fetch.mockResolvedValueOnce({
+                ok: true,
+                json: () => Promise.resolve(mockHealth),
+            });
+
+            const result = await getCiStatus();
+
+            expect(fetch).toHaveBeenCalledWith('/taller/api/health');
+            expect(result).toEqual(mockHealth);
+        });
+    });
+
+    describe('getBuildHistory', () => {
+        it('llama al endpoint /builds', async () => {
+            const mockBuilds = { builds: [{ number: 1 }, { number: 2 }] };
+
+            fetch.mockResolvedValueOnce({
+                ok: true,
+                json: () => Promise.resolve(mockBuilds),
+            });
+
+            const result = await getBuildHistory();
+
+            expect(fetch).toHaveBeenCalledWith('/taller/api/builds');
+            expect(result).toEqual(mockBuilds);
+        });
+    });
+});

frontend/src/setupTests.js

@@ -0,0 +1,19 @@
+/*
+CI/CD Workshop
+Copyright (C) 2025 OpenBokeron
+
+This program is free software: you can redistribute it and/or modify
+it under the terms of the GNU General Public License as published by
+the Free Software Foundation, either version 3 of the License, or
+(at your option) any later version.
+
+This program is distributed in the hope that it will be useful,
+but WITHOUT ANY WARRANTY; without even the implied warranty of
+MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+GNU General Public License for more details.
+
+You should have received a copy of the GNU General Public License
+along with this program.  If not, see <https://www.gnu.org/licenses/>.
+*/
+
+import '@testing-library/jest-dom';

frontend/src/styles/app.css

@@ -697,6 +697,25 @@ li {
     font-size: 0.95rem;
 }
 
+.summary-trigger {
+    display: flex;
+    align-items: center;
+    gap: 0.4rem;
+    flex-wrap: wrap;
+    color: #fbbf24;
+    font-weight: 700;
+    font-size: 0.95rem;
+}
+
+.summary-trigger a {
+    color: inherit;
+    text-decoration: none;
+}
+
+.summary-trigger a:hover {
+    text-decoration: underline;
+}
+
 .summary-meta {
     display: flex;
     gap: 0.6rem;
@@ -723,6 +742,11 @@ li {
     box-shadow: 0 0 8px rgba(248, 113, 113, 0.5);
 }
 
+.status-dot.running {
+    background: #facc15;
+    box-shadow: 0 0 8px rgba(250, 204, 21, 0.5);
+}
+
 .history-body {
     padding: 0.75rem 0.9rem 0.9rem;
     border-top: 1px solid rgba(255, 255, 255, 0.05);
@@ -743,6 +767,21 @@ li {
     color: #cbd5f5;
 }
 
+.history-link {
+    color: #fbbf24;
+    text-decoration: none;
+    font-weight: 700;
+}
+
+.history-link:hover {
+    text-decoration: underline;
+}
+
+.muted {
+    color: #94a3b8;
+    font-size: 0.95rem;
+}
+
 .history-message {
     margin: 0.2rem 0 0;
     background: rgba(248, 113, 113, 0.12);
@@ -758,6 +797,12 @@ li {
     color: #bbf7d0;
 }
 
+.history-message.running {
+    background: rgba(250, 204, 21, 0.16);
+    border-color: rgba(250, 204, 21, 0.5);
+    color: #fef9c3;
+}
+
 .history-item[open] {
     box-shadow: 0 10px 30px rgba(0, 0, 0, 0.28);
 }
@@ -770,6 +815,10 @@ li {
     border-color: rgba(248, 113, 113, 0.35);
 }
 
+.history-item.running {
+    border-color: rgba(250, 204, 21, 0.35);
+}
+
 .chip.danger {
     background: rgba(248, 113, 113, 0.2);
     color: #fecdd3;
@@ -820,3 +869,15 @@ li {
     font-size: 0.8rem;
     opacity: 0.85;
 }
+.trigger-author {
+    padding: 0.1rem 0.4rem;
+    border-radius: 999px;
+    background: rgba(148, 163, 184, 0.16);
+    color: #cbd5f5;
+    font-size: 0.78rem;
+    font-weight: 700;
+}
+
+.history-row .trigger-author {
+    margin-left: 0.35rem;
+}

frontend/src/utils/text.test.js

@@ -0,0 +1,38 @@
+/*
+CI/CD Workshop
+Copyright (C) 2025 OpenBokeron
+
+This program is free software: you can redistribute it and/or modify
+it under the terms of the GNU General Public License as published by
+the Free Software Foundation, either version 3 of the License, or
+(at your option) any later version.
+
+This program is distributed in the hope that it will be useful,
+but WITHOUT ANY WARRANTY; without even the implied warranty of
+MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+GNU General Public License for more details.
+
+You should have received a copy of the GNU General Public License
+along with this program.  If not, see <https://www.gnu.org/licenses/>.
+*/
+
+import { describe, it, expect } from 'vitest';
+import { prettify } from './text';
+
+describe('prettify', () => {
+    it('convierte snake_case a Title Case', () => {
+        expect(prettify('hello_world')).toBe('Hello World');
+    });
+
+    it('maneja palabras multiples con underscore', () => {
+        expect(prettify('cafe_con_leche')).toBe('Cafe Con Leche');
+    });
+
+    it('maneja una sola palabra sin underscore', () => {
+        expect(prettify('bocadillo')).toBe('Bocadillo');
+    });
+
+    it('maneja cadena vacia', () => {
+        expect(prettify('')).toBe('');
+    });
+});

frontend/vite.config.js

@@ -14,6 +14,7 @@ export default defineConfig(({ mode }) => {
         test: {
             environment: 'jsdom',
             globals: true,
+            setupFiles: ['./src/setupTests.js'],
         },
         server: {
             port: 5173,