Nicolas
ba131bd8c8
Un rafraîchissement (F5) sur /admin/monitoring affichait le JSON brut de
l'API au lieu de la page — même chose à craindre pour tout futur /admin/*
GET sans segment supplémentaire.
Cause : `server.mountRouter("/admin", adminRouter)` (app.ts) montait
l'API admin exactement sous /admin, la même racine que les pages
/admin/* d'apps/web, servies depuis la même origine/le même process en
prod (image Docker unique). Express matche par chemin, sans égard au
Accept — `GET /admin/monitoring` (route API) et la page du même nom
étaient en conflit direct ; l'API gagnait systématiquement (montée
avant serveStaticFrontend), donc chaque F5 sur cette page recevait le
JSON de l'API plutôt que index.html.
Seule /admin/monitoring collidait réellement aujourd'hui (seul
sous-routeur admin à répondre GET sur sa racine bare) mais le
mécanisme touche toute future route admin nommée comme sa page — d'où
un correctif structurel plutôt qu'un renommage ponctuel.
Correctif : l'API admin est montée sous /admin/api (app.ts), jamais
bare /admin. Toute la surface admin bouge en conséquence
(auth/metrics/monitoring/tech-steps/catalog) ; les pages front
(/admin/monitoring, /admin/catalogue, /admin/corrections...) sont
inchangées, seul admin-client.ts change de base URL. Mis à jour en
cohérence : commentaires JSDoc citant les routes (apps/api + apps/web +
packages/shared), specs/backend-architecture.md,
specs/batch-cooking-modele.md, les intercepts Cypress (globs + un
matcher regex qui avait échappé au premier passage) et les chemins
supertest du suite Mocha admin-*.test.ts.
Simplifie au passage admin-monitoring.cy.ts : le contournement
resourceType ajouté en #32 pour cette même collision n'a plus lieu
d'être une fois les chemins distincts (retour à un intercept déclaratif
simple) — le bug Cypress/Chrome qu'il avait mis au jour reste noté en
commentaire pour référence future.
Vérifié : pnpm lint clean ; build api+web clean ; suite e2e complète
28/28 fichiers, 116/116 tests verts (dont les 16 tests admin-*).
Suite Mocha apps/api non exécutable cette session (Postgres/Docker
injoignable) — chemins mis à jour mécaniquement en cohérence avec le
nouveau mount point, à faire tourner en CI/à la prochaine session avec
accès DB.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
95 lines
3 KiB
TypeScript
95 lines
3 KiB
TypeScript
// Mocks the admin API via cy.intercept — no live backend.
|
|
|
|
const adminBody = {
|
|
id: 1,
|
|
email: "ops@example.com",
|
|
name: "Ops",
|
|
createdAt: "2026-08-01T00:00:00.000Z",
|
|
lastLoginAt: "2026-08-28T09:00:00.000Z",
|
|
};
|
|
|
|
/** A 3-day series helper for the fixture. */
|
|
function series(counts: number[]) {
|
|
return counts.map((count, i) => ({
|
|
date: `2026-08-${String(10 + i).padStart(2, "0")}`,
|
|
count,
|
|
}));
|
|
}
|
|
|
|
function metricsFixture() {
|
|
return {
|
|
generatedAt: "2026-08-28T09:00:00.000Z",
|
|
rangeDays: 30,
|
|
snapshot: {
|
|
admins: 2,
|
|
users: 42,
|
|
households: 15,
|
|
activeHouseholds: 9,
|
|
recipes: 120,
|
|
recipesManual: 30,
|
|
recipesImported: 90,
|
|
recipesBySource: [
|
|
{ key: "themealdb", label: "TheMealDB", count: 60 },
|
|
{ key: "marmiton", label: "Marmiton", count: 30 },
|
|
],
|
|
plannings: 18,
|
|
planningItems: 210,
|
|
steps: 640,
|
|
detectedTechniques: 900,
|
|
favorites: 55,
|
|
corrections: 12,
|
|
correctionsUnconsumed: 4,
|
|
correctionsRemoval: 2,
|
|
trainingSuggestions: 8,
|
|
trainingSuggestionsByStatus: [{ key: "pending", label: "pending", count: 8 }],
|
|
trainingSuggestionsBySourceType: [{ key: "correction", label: "correction", count: 8 }],
|
|
},
|
|
series: {
|
|
signups: series([1, 3, 2]),
|
|
recipesCreated: series([0, 2, 1]),
|
|
planningItemsAdded: series([4, 1, 5]),
|
|
correctionsSubmitted: series([0, 0, 1]),
|
|
trainingSuggestions: series([0, 1, 0]),
|
|
},
|
|
events: [{ type: "user.signup", buckets: series([1, 3, 2]) }],
|
|
};
|
|
}
|
|
|
|
describe("Admin dashboard", () => {
|
|
beforeEach(() => {
|
|
cy.viewport(1400, 900);
|
|
cy.intercept("GET", "**/admin/api/auth/me", { statusCode: 200, body: adminBody });
|
|
});
|
|
|
|
it("renders KPI tiles, a chart per series, and the breakdown lists", () => {
|
|
cy.intercept("GET", "**/admin/api/metrics*", { statusCode: 200, body: metricsFixture() }).as(
|
|
"getMetrics",
|
|
);
|
|
cy.visit("/admin");
|
|
cy.wait("@getMetrics").its("request.url").should("include", "days=30");
|
|
|
|
// KPI tiles — value + label.
|
|
cy.contains(".kpi-tile", "Utilisateurs").should("contain.text", "42");
|
|
cy.contains(".kpi-tile", "Recettes importées").should("contain.text", "90");
|
|
cy.contains(".kpi-tile", "Corrections à traiter").should("contain.text", "4");
|
|
|
|
// One chart card per instrumented series.
|
|
cy.get(".chart-card").should("have.length", 5);
|
|
cy.contains(".chart-card", "Inscriptions").should("contain.text", "6 sur 30 j");
|
|
|
|
// Breakdown lists.
|
|
cy.contains(".breakdown", "Recettes importées par source")
|
|
.should("contain.text", "TheMealDB")
|
|
.and("contain.text", "Marmiton");
|
|
cy.contains(".breakdown", "Évènements enregistrés").should("contain.text", "user.signup");
|
|
});
|
|
|
|
it("shows an error state when the metrics request fails", () => {
|
|
cy.intercept("GET", "**/admin/api/metrics*", {
|
|
statusCode: 500,
|
|
body: { code: 5000, message: "x" },
|
|
});
|
|
cy.visit("/admin");
|
|
cy.contains("Impossible de charger").should("be.visible");
|
|
});
|
|
});
|