feat(einstellungen): Betriebswerte im Dashboard einstellbar (#025)
Vier Bereiche unter Einstellungen -> System: Adresse und Agent, Kaufmaennisches, Sicherheit sowie Firma und Darstellung. Alles liegt als eine JSON-Zeile in der Tabelle Einstellung; fehlende Felder bekommen ihren Vorgabewert, eine kaputte Zeile legt den Betrieb nicht lahm. Die Werte wirken auch wirklich: Die Dashboard-Adresse und der Melderhythmus landen im ausgelieferten Einzeiler und im systemd-Timer (live geprueft), der Steuersatz in neuen Rechnungspositionen, die Sitzungsdauer im Token, die Firmenangaben im Kopf der PDF-Dateien. Nebenbei geschlossen: die OIDC-Luecke aus #024. Erlaubte Domaenen und ein Schalter gegen automatisch angelegte Konten. Dazu raeumt der Worker nachts nach den eingestellten Fristen auf - der juengste Report je Host bleibt. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This commit is contained in:
@@ -0,0 +1,92 @@
|
||||
import { afterAll, beforeEach, describe, expect, it } from "vitest";
|
||||
import { altdatenLoeschen } from "@/server/aufraeumen";
|
||||
import { einstellungenSpeichern, VORGABEN, zwischenspeicherLeeren } from "@/server/einstellungen";
|
||||
import { datenbankLeeren, datenbankVerfuegbar, testPrisma } from "./helpers/db";
|
||||
|
||||
const JETZT = new Date("2026-09-18T12:00:00Z");
|
||||
const tageVorher = (tage: number) => new Date(JETZT.getTime() - tage * 86_400_000);
|
||||
|
||||
async function hostMitReports(tage: number[]) {
|
||||
const host = await testPrisma.host.create({
|
||||
data: { agentId: "agent-1", hostname: "pve-01", produkt: "PVE", letzterReportAm: JETZT },
|
||||
});
|
||||
|
||||
for (const alter of tage) {
|
||||
await testPrisma.report.create({
|
||||
data: {
|
||||
hostId: host.id,
|
||||
reportId: `report-${alter}`,
|
||||
erhobenAm: tageVorher(alter),
|
||||
empfangenAm: tageVorher(alter),
|
||||
subStatus: "ACTIVE",
|
||||
rohdaten: {},
|
||||
},
|
||||
});
|
||||
}
|
||||
|
||||
return host;
|
||||
}
|
||||
|
||||
describe.runIf(datenbankVerfuegbar)("Altdaten löschen", () => {
|
||||
beforeEach(async () => {
|
||||
await datenbankLeeren();
|
||||
zwischenspeicherLeeren();
|
||||
});
|
||||
|
||||
afterAll(async () => {
|
||||
await testPrisma.$disconnect();
|
||||
});
|
||||
|
||||
it("löscht Reports, die älter sind als die Frist", async () => {
|
||||
await einstellungenSpeichern({ ...VORGABEN, aufbewahrungReportsTage: 30 });
|
||||
await hostMitReports([1, 10, 45, 400]);
|
||||
|
||||
const ergebnis = await altdatenLoeschen(JETZT);
|
||||
|
||||
expect(ergebnis.reports).toBe(2);
|
||||
expect(await testPrisma.report.count()).toBe(2);
|
||||
});
|
||||
|
||||
it("behält den jüngsten Report jedes Hosts, auch wenn er zu alt ist", async () => {
|
||||
await einstellungenSpeichern({ ...VORGABEN, aufbewahrungReportsTage: 30 });
|
||||
await hostMitReports([200, 400]);
|
||||
|
||||
await altdatenLoeschen(JETZT);
|
||||
|
||||
const uebrig = await testPrisma.report.findMany();
|
||||
expect(uebrig).toHaveLength(1);
|
||||
expect(uebrig[0].erhobenAm).toEqual(tageVorher(200));
|
||||
});
|
||||
|
||||
it("löscht nichts, wenn die Frist auf 0 steht", async () => {
|
||||
await einstellungenSpeichern({
|
||||
...VORGABEN,
|
||||
aufbewahrungReportsTage: 0,
|
||||
aufbewahrungAuditTage: 0,
|
||||
});
|
||||
await hostMitReports([400, 800]);
|
||||
await testPrisma.auditEintrag.create({
|
||||
data: { aktion: "test", entitaet: "Host", erstelltAm: tageVorher(3000) },
|
||||
});
|
||||
|
||||
const ergebnis = await altdatenLoeschen(JETZT);
|
||||
|
||||
expect(ergebnis).toEqual({ reports: 0, protokoll: 0 });
|
||||
expect(await testPrisma.report.count()).toBe(2);
|
||||
expect(await testPrisma.auditEintrag.count()).toBe(1);
|
||||
});
|
||||
|
||||
it("räumt auch das Protokoll auf", async () => {
|
||||
await einstellungenSpeichern({ ...VORGABEN, aufbewahrungAuditTage: 365 });
|
||||
for (const alter of [10, 400]) {
|
||||
await testPrisma.auditEintrag.create({
|
||||
data: { aktion: "test", entitaet: "Host", erstelltAm: tageVorher(alter) },
|
||||
});
|
||||
}
|
||||
|
||||
const ergebnis = await altdatenLoeschen(JETZT);
|
||||
|
||||
expect(ergebnis.protokoll).toBe(1);
|
||||
expect(await testPrisma.auditEintrag.count()).toBe(1);
|
||||
});
|
||||
});
|
||||
@@ -0,0 +1,58 @@
|
||||
import { describe, expect, it } from "vitest";
|
||||
import { domaeneErlaubt, einstellungenSchema, onCalendar, VORGABEN } from "@/server/einstellungen";
|
||||
|
||||
describe("Einstellungen", () => {
|
||||
it("füllt fehlende Werte mit den Vorgaben", () => {
|
||||
const werte = einstellungenSchema.parse({});
|
||||
expect(werte.steuersatz).toBe(19);
|
||||
expect(werte.sitzungsdauerStunden).toBe(12);
|
||||
expect(werte.agentRhythmus).toBe("taeglich");
|
||||
expect(werte.oidcKontenAnlegen).toBe(true);
|
||||
});
|
||||
|
||||
it("übernimmt gesetzte Werte und lässt den Rest auf Vorgabe", () => {
|
||||
const werte = einstellungenSchema.parse({ steuersatz: 7.7, waehrung: "CHF" });
|
||||
expect(werte.steuersatz).toBe(7.7);
|
||||
expect(werte.waehrung).toBe("CHF");
|
||||
expect(werte.zahlungszielTage).toBe(VORGABEN.zahlungszielTage);
|
||||
});
|
||||
|
||||
it("weist unsinnige Werte ab", () => {
|
||||
expect(einstellungenSchema.safeParse({ steuersatz: 250 }).success).toBe(false);
|
||||
expect(einstellungenSchema.safeParse({ sitzungsdauerStunden: 0 }).success).toBe(false);
|
||||
expect(einstellungenSchema.safeParse({ dashboardUrl: "kein-url" }).success).toBe(false);
|
||||
expect(einstellungenSchema.safeParse({ anmeldeversuche: 1 }).success).toBe(false);
|
||||
});
|
||||
|
||||
it("lässt eine leere Dashboard-Adresse zu (dann gilt die Umgebung)", () => {
|
||||
expect(einstellungenSchema.parse({ dashboardUrl: "" }).dashboardUrl).toBe("");
|
||||
});
|
||||
|
||||
it("übersetzt den Rhythmus nach systemd", () => {
|
||||
expect(onCalendar("taeglich")).toBe("daily");
|
||||
expect(onCalendar("stuendlich")).toBe("hourly");
|
||||
expect(onCalendar("zwoelfstuendlich")).toBe("0/12:00:00");
|
||||
});
|
||||
});
|
||||
|
||||
describe("Erlaubte OIDC-Domänen", () => {
|
||||
it("lässt ohne Angabe jede Domäne zu", () => {
|
||||
expect(domaeneErlaubt("wer@auch-immer.de", "")).toBe(true);
|
||||
});
|
||||
|
||||
it("lässt nur die genannten Domänen zu", () => {
|
||||
expect(domaeneErlaubt("fw@wilde-server.de", "wilde-server.de")).toBe(true);
|
||||
expect(domaeneErlaubt("fremd@example.com", "wilde-server.de")).toBe(false);
|
||||
});
|
||||
|
||||
it("versteht mehrere Domänen, Leerzeichen und führendes @", () => {
|
||||
const liste = " @wilde-server.de, kunde.de ";
|
||||
expect(domaeneErlaubt("a@kunde.de", liste)).toBe(true);
|
||||
expect(domaeneErlaubt("b@WILDE-SERVER.DE", liste)).toBe(true);
|
||||
expect(domaeneErlaubt("c@fremd.de", liste)).toBe(false);
|
||||
});
|
||||
|
||||
it("weist eine Adresse ohne Domäne ab", () => {
|
||||
expect(domaeneErlaubt("kaputt", "kunde.de")).toBe(false);
|
||||
});
|
||||
});
|
||||
Reference in New Issue
Block a user