Alle Behebungen auf der Linux-Test-VM gegengeprueft (Ubuntu, .NET 10.0.111): Build 0 Warnungen / 0 Fehler, 637 Tests gruen, 1 uebersprungen (WindowsFact). Sicherheit und Zielplattform #373 PathBoundary.Canonicalize fiel offen aus. File.ResolveLinkTarget wirft fuer einen fehlenden Abschnitt FileNotFoundException; der catch(IOException) verliess daraufhin die GANZE Aufloesung mit dem unaufgeloesten Pfad - und der besteht jede Praefixpruefung. Der Symlink-Schutz war damit bei jedem Schreibvorgang wirkungslos. Jetzt ueber FileSystemInfo.LinkTarget, und TryCanonicalize meldet "nicht bestimmbar", statt fail-open durchzureichen. #374 WorkspacePath weist den Backslash rundheraus ab, wie den Doppelpunkt. Eine Schreibweise statt einer Sonderregel je Windows-Form. #375 Log-Verzeichnis: leer heisst "Anwendungseinstellung"; ein relativer Wert zaehlt gegen das Instanzverzeichnis, nicht gegen das Arbeitsverzeichnis des Prozesses. Ein systemd-Dienst landet nicht mehr auf /Logs. Agentenkommunikation #206 Delegationskette gegen A->B->A (Test A13). Zyklusfrei per Bauart, dazu MaxDelegationDepth gegen die Staffel, die niemand beauftragt hat. #461 Zwei unabhaengige Ketten ueber Kreuz - beim Beheben von #206 gefunden und reproduziert. Ein delegierter Lauf wartet nur noch begrenzt auf ein fremdes Gate; ein Chat vom Menschen reiht sich weiter unbegrenzt ein. #234 Instanz-Id wird durchgereicht statt aus einem nur beilaeufig gesetzten Feld gelesen. Betrieb und Oberflaeche #376 Log-Ansicht chronologisch statt gruppiert je Moduldatei, mit Stacktraces an ihrer Meldung - auch ueber Taktgrenzen hinweg. #377 Info-Tabelle: feste Spaltenbreiten, damit ein langer Aenderungstext den anderen Spalten nicht den Platz nimmt. #378 Sicherung: Mindestabstand zwischen Ueberschrift und Schaltflaechenleiste. #379 Hellmodus: drei von vier gedaempften Textpinseln lagen unter 4,5:1. Die helle Palette ist als Ganzes verschoben, damit die Staffelung sichtbar bleibt statt auf einen Wert zusammenzufallen. Hygiene #233 stock_add: der Indexstand wird innerhalb der Sperre festgehalten. #235 Zeitumstellung: eine nicht existierende Ortszeit wird auf den ersten Zeitpunkt nach der Luecke verschoben statt uebersprungen (R4, R5). #236 45 Nullable-Warnungen und eine CA1416 - nicht unterdrueckt, sondern die Stellen so umgebaut, dass die Zusage nachvollziehbar wird. Mail meldet jetzt, WELCHER Zugangsschluessel fehlt, statt an einer KeyNotFoundException zu scheitern. #203 Kompaktiert wird vor dem API-Aufruf statt danach; die gemessenen Prompt-Groessen kalibrieren die Schaetzung je Modell. Neu: tests/ClawdDotNet.App.Tests - fuer #375 und #376 gab es keinen Ort fuer Tests. Dazu Regressionstests fuer jede Behebung; jeder wurde gegen den alten Stand geprueft, damit belegt ist, dass er den Fehler faengt. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
138 lines
5.3 KiB
C#
138 lines
5.3 KiB
C#
using System.Net;
|
|
using ClawdDotNet.Core.Deploymentcenter;
|
|
using ClawdDotNet.Core.Deploymentcenter.Watchdog;
|
|
using Shouldly;
|
|
|
|
namespace ClawdDotNet.Core.Tests.Deploymentcenter;
|
|
|
|
public sealed class WatchdogClientTests
|
|
{
|
|
private static WatchdogClient Client(HttpMessageHandler handler, string instance = "inst-1") =>
|
|
WatchdogClient.Create("https://dc.example", "tok", "clawddotnet", instance,
|
|
"ClawdDotNet", "Windows / .NET 10", "0.1.0", new HttpClient(handler));
|
|
|
|
private static InstanceHealth Health(
|
|
string status = WatchdogStatus.Ok,
|
|
string? message = null,
|
|
Dictionary<string, double>? metrics = null,
|
|
Dictionary<string, HealthCheck>? checks = null)
|
|
=> new(status, message, metrics ?? [], checks ?? []);
|
|
|
|
[Fact]
|
|
public async Task Heartbeat_geht_an_den_v1_Pfad_mit_Bearer_Token()
|
|
{
|
|
var handler = new CapturingHandler();
|
|
|
|
await Client(handler).SendHeartbeatAsync(Health(message: "Betrieb normal."), 60, default);
|
|
|
|
handler.LastRequest!.RequestUri!.ToString().ShouldBe("https://dc.example/api/watchdog/v1/ping");
|
|
handler.LastRequest.Method.ShouldBe(HttpMethod.Post);
|
|
handler.LastRequest.Headers.Authorization!.ToString().ShouldBe("Bearer tok");
|
|
|
|
handler.LastBody.ShouldNotBeNull();
|
|
handler.LastBody.ShouldNotBeNull().ShouldContain("\"source\":\"clawddotnet\"");
|
|
handler.LastBody.ShouldNotBeNull().ShouldContain("\"instance\":\"inst-1\"");
|
|
handler.LastBody.ShouldNotBeNull().ShouldContain("\"type\":\"heartbeat\"");
|
|
handler.LastBody.ShouldNotBeNull().ShouldContain("\"interval\":60");
|
|
}
|
|
|
|
/// <summary>
|
|
/// Der Server legt die Version in <c>watchdog_monitors.app_version</c> ab. Bei
|
|
/// mehreren Instanzen ist das der Unterschied zwischen „läuft" und „läuft noch auf
|
|
/// der alten Fassung".
|
|
/// </summary>
|
|
[Fact]
|
|
public async Task Heartbeat_traegt_die_Anwendungsversion()
|
|
{
|
|
var handler = new CapturingHandler();
|
|
|
|
await Client(handler).SendHeartbeatAsync(Health(), 60, default);
|
|
|
|
handler.LastBody!.ShouldContain("\"version\":\"0.1.0\"");
|
|
}
|
|
|
|
/// <summary>
|
|
/// Der Monitor wird serverseitig über source + instance geführt. Zwei Instanzen
|
|
/// derselben Anwendung müssen deshalb unterschiedliche instance-Werte melden —
|
|
/// sonst überschreiben sie sich gegenseitig, und der Ausfall einer bliebe unsichtbar.
|
|
/// </summary>
|
|
[Fact]
|
|
public async Task Jede_Instanz_meldet_unter_eigenem_Instanznamen()
|
|
{
|
|
var first = new CapturingHandler();
|
|
var second = new CapturingHandler();
|
|
|
|
await Client(first, "inst-a").SendHeartbeatAsync(Health(), 60, default);
|
|
await Client(second, "inst-b").SendHeartbeatAsync(Health(), 60, default);
|
|
|
|
first.LastBody!.ShouldContain("\"instance\":\"inst-a\"");
|
|
second.LastBody!.ShouldContain("\"instance\":\"inst-b\"");
|
|
}
|
|
|
|
[Fact]
|
|
public async Task Checks_und_Metriken_gehen_mit()
|
|
{
|
|
var handler = new CapturingHandler();
|
|
|
|
await Client(handler).SendHeartbeatAsync(
|
|
Health(
|
|
metrics: new Dictionary<string, double> { ["agentCount"] = 3 },
|
|
checks: new Dictionary<string, HealthCheck>
|
|
{
|
|
["scheduler"] = new(false, "Steht.")
|
|
}),
|
|
60, default);
|
|
|
|
handler.LastBody!.ShouldContain("\"agentCount\":3");
|
|
handler.LastBody.ShouldNotBeNull().ShouldContain("\"scheduler\":{\"ok\":false,\"message\":\"Steht.\"}");
|
|
}
|
|
|
|
/// <summary>Leere Angaben bleiben weg, statt als leeres Objekt zu erscheinen.</summary>
|
|
[Fact]
|
|
public async Task Ohne_Checks_und_Metriken_bleiben_die_Felder_weg()
|
|
{
|
|
var handler = new CapturingHandler();
|
|
|
|
await Client(handler).SendHeartbeatAsync(Health(), 60, default);
|
|
|
|
handler.LastBody!.ShouldNotContain("\"checks\"");
|
|
handler.LastBody.ShouldNotBeNull().ShouldNotContain("\"metrics\"");
|
|
}
|
|
|
|
[Fact]
|
|
public async Task Antwort_liefert_Zustand_und_fehlgeschlagene_Pruefungen()
|
|
{
|
|
var handler = new CapturingHandler(HttpStatusCode.OK,
|
|
"""{"status":"success","monitor":{"state":"warning","failing_checks":["scheduler"]}}""");
|
|
|
|
var result = await Client(handler).SendHeartbeatAsync(Health(), 60, default);
|
|
|
|
result.State.ShouldBe("warning");
|
|
result.FailingChecks.ShouldHaveSingleItem().ShouldBe("scheduler");
|
|
}
|
|
|
|
[Fact]
|
|
public async Task Ereignis_geht_an_den_Event_Endpunkt()
|
|
{
|
|
var handler = new CapturingHandler();
|
|
|
|
await Client(handler).SendEventAsync(
|
|
WatchdogEventKind.StoppedGraceful, "info", "Instanz beendet.", null, default);
|
|
|
|
handler.LastRequest!.RequestUri!.ToString().ShouldBe("https://dc.example/api/watchdog/v1/event");
|
|
handler.LastBody!.ShouldContain("\"kind\":\"stopped_graceful\"");
|
|
}
|
|
|
|
[Fact]
|
|
public async Task Abgelehntes_Token_wirft_mit_Code()
|
|
{
|
|
var handler = new CapturingHandler(HttpStatusCode.Unauthorized,
|
|
"""{"status":"error","error":{"code":"unauthorized","message":"Kein gueltiges Token."}}""");
|
|
|
|
var ex = await Should.ThrowAsync<DeploymentcenterException>(
|
|
() => Client(handler).SendHeartbeatAsync(Health(), 60, default));
|
|
|
|
ex.Code.ShouldBe("unauthorized");
|
|
}
|
|
}
|