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>
108 lines
3.9 KiB
C#
108 lines
3.9 KiB
C#
using ClawdDotNet.Core.Accounting;
|
|
using ClawdDotNet.Core.Config;
|
|
using ClawdDotNet.Core.Deploymentcenter.Watchdog;
|
|
using Shouldly;
|
|
|
|
namespace ClawdDotNet.Core.Tests.Deploymentcenter;
|
|
|
|
public sealed class InstanceHealthProviderTests
|
|
{
|
|
/// <summary>Liefert einen festen Tagesverbrauch, ohne DB.</summary>
|
|
private sealed class FakeUsage(DailyUsage daily) : IUsageRepository
|
|
{
|
|
public Task RecordAsync(RunUsage usage, CancellationToken ct) => Task.CompletedTask;
|
|
|
|
public Task<DailyUsage> GetDailyAsync(DateOnly date, string agentId, CancellationToken ct)
|
|
=> Task.FromResult(daily);
|
|
|
|
public Task<IReadOnlyList<RunUsage>> GetRecentAsync(int limit, CancellationToken ct)
|
|
=> Task.FromResult((IReadOnlyList<RunUsage>)Array.Empty<RunUsage>());
|
|
|
|
public Task<int> PurgeBeforeAsync(DateOnly date, CancellationToken ct) => Task.FromResult(0);
|
|
}
|
|
|
|
private static DailyUsage Usage(decimal cost, int tokens = 0) =>
|
|
new(DateOnly.FromDateTime(DateTime.Now), tokens, 0, cost, true, 1);
|
|
|
|
[Fact]
|
|
public async Task Ohne_API_Key_meldet_error()
|
|
{
|
|
var provider = new InstanceHealthProvider(
|
|
"Test", agentsEnabled: false, InstanceBudget.Unlimited,
|
|
usage: null, () => 2, () => 0);
|
|
|
|
var health = await provider.GetAsync(default);
|
|
|
|
health.Status.ShouldBe(WatchdogStatus.Error);
|
|
health.Metrics["agentCount"].ShouldBe(2);
|
|
health.Checks["agents"].Ok.ShouldBeFalse();
|
|
}
|
|
|
|
[Fact]
|
|
public async Task Erschoepftes_Kostenbudget_meldet_warning()
|
|
{
|
|
var budget = new InstanceBudget { DailyCostUsd = 5m };
|
|
var provider = new InstanceHealthProvider(
|
|
"Test", agentsEnabled: true, budget,
|
|
new FakeUsage(Usage(cost: 5m)), () => 1, () => 0);
|
|
|
|
var health = await provider.GetAsync(default);
|
|
|
|
health.Status.ShouldBe(WatchdogStatus.Warning);
|
|
health.Checks["budget"].Ok.ShouldBeFalse();
|
|
health.Message.ShouldNotBeNull();
|
|
}
|
|
|
|
/// <summary>
|
|
/// Ein Heartbeat beweist nur, dass ein Faden läuft. Steht der Scanner, arbeitet die
|
|
/// Instanz nichts mehr ab — und genau das soll die Prüfung sichtbar machen.
|
|
/// </summary>
|
|
[Fact]
|
|
public async Task Stehender_Scanner_meldet_warning()
|
|
{
|
|
var provider = new InstanceHealthProvider(
|
|
"Test", agentsEnabled: true, InstanceBudget.Unlimited,
|
|
new FakeUsage(Usage(cost: 0m)), () => 1, () => 0,
|
|
schedulerRunning: () => false);
|
|
|
|
var health = await provider.GetAsync(default);
|
|
|
|
health.Status.ShouldBe(WatchdogStatus.Warning);
|
|
health.Checks["scheduler"].Ok.ShouldBeFalse();
|
|
}
|
|
|
|
[Fact]
|
|
public async Task Normalbetrieb_meldet_ok_mit_Metriken()
|
|
{
|
|
var provider = new InstanceHealthProvider(
|
|
"Test", agentsEnabled: true, InstanceBudget.Unlimited,
|
|
new FakeUsage(Usage(cost: 0.5m, tokens: 1200)), () => 3, () => 1,
|
|
schedulerRunning: () => true);
|
|
|
|
var health = await provider.GetAsync(default);
|
|
|
|
health.Status.ShouldBe(WatchdogStatus.Ok);
|
|
health.Metrics["runningChats"].ShouldBe(1);
|
|
health.Metrics.Keys.ShouldContain("todayCostUsd");
|
|
health.Metrics.Keys.ShouldContain("todayTokens");
|
|
health.Checks.Values.ShouldAllBe(c => c.Ok);
|
|
}
|
|
|
|
/// <summary>
|
|
/// Metriken werden serverseitig nur ausgewertet, wenn sie Zahlen sind — alles andere
|
|
/// verwirft der Verlauf stillschweigend. Beschreibendes gehört in die Meldung.
|
|
/// </summary>
|
|
[Fact]
|
|
public async Task Instanzname_steht_in_der_Meldung_nicht_in_den_Metriken()
|
|
{
|
|
var provider = new InstanceHealthProvider(
|
|
"Produktion", agentsEnabled: true, InstanceBudget.Unlimited,
|
|
usage: null, () => 1, () => 0);
|
|
|
|
var health = await provider.GetAsync(default);
|
|
|
|
health.Message.ShouldNotBeNull().ShouldContain("Produktion");
|
|
health.Metrics.Keys.ShouldNotContain("instanceName");
|
|
}
|
|
}
|