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>
98 lines
3.3 KiB
C#
98 lines
3.3 KiB
C#
using System.Net;
|
|
using ClawdDotNet.Core.Deploymentcenter;
|
|
using Microsoft.Extensions.Logging.Abstractions;
|
|
using Shouldly;
|
|
|
|
namespace ClawdDotNet.Core.Tests.Deploymentcenter;
|
|
|
|
public sealed class ErrorReporterTests
|
|
{
|
|
private static (ErrorReporter Reporter, CapturingHandler Handler) Build(
|
|
Func<DateTimeOffset>? now = null,
|
|
HttpStatusCode status = HttpStatusCode.Created)
|
|
{
|
|
var handler = new CapturingHandler(status,
|
|
"""{"status":"success","item_id":7,"is_new":true,"occurrence_count":1}""");
|
|
|
|
var api = new DeploymentcenterApi("https://dc.example", "tok", new HttpClient(handler));
|
|
var reporter = new ErrorReporter(
|
|
api, "clawddotnet", "production", "0.1.0", NullLogger.Instance, ownsApi: true, now: now);
|
|
|
|
return (reporter, handler);
|
|
}
|
|
|
|
private static Exception Thrown(string message)
|
|
{
|
|
try { throw new InvalidOperationException(message); }
|
|
catch (Exception ex) { return ex; }
|
|
}
|
|
|
|
[Fact]
|
|
public async Task Meldung_geht_an_den_Fehler_Endpunkt()
|
|
{
|
|
var (reporter, handler) = Build();
|
|
using var _ = reporter;
|
|
|
|
var sent = await reporter.ReportAsync(Thrown("Kaputt"), fatal: true);
|
|
|
|
sent.ShouldBeTrue();
|
|
handler.LastRequest!.RequestUri!.ToString().ShouldBe("https://dc.example/api/errors/v1/report");
|
|
handler.LastBody!.ShouldContain("\"exception\":\"System.InvalidOperationException\"");
|
|
handler.LastBody.ShouldNotBeNull().ShouldContain("\"message\":\"Kaputt\"");
|
|
handler.LastBody.ShouldNotBeNull().ShouldContain("\"level\":\"fatal\"");
|
|
handler.LastBody.ShouldNotBeNull().ShouldContain("\"project_slug\":\"clawddotnet\"");
|
|
}
|
|
|
|
/// <summary>
|
|
/// Eine Fehlerschleife darf nicht in tausend Anfragen münden. Derselbe Fehler geht
|
|
/// höchstens einmal je Zeitfenster raus — der Server drosselt zwar auch, aber erst,
|
|
/// nachdem die Anfragen schon über die Leitung waren.
|
|
/// </summary>
|
|
[Fact]
|
|
public async Task Derselbe_Fehler_geht_nur_einmal_je_Zeitfenster_raus()
|
|
{
|
|
var now = DateTimeOffset.UtcNow;
|
|
var (reporter, handler) = Build(() => now);
|
|
using var _ = reporter;
|
|
|
|
var error = Thrown("Immer wieder");
|
|
|
|
(await reporter.ReportAsync(error)).ShouldBeTrue();
|
|
(await reporter.ReportAsync(error)).ShouldBeFalse();
|
|
(await reporter.ReportAsync(error)).ShouldBeFalse();
|
|
|
|
handler.Calls.ShouldBe(1);
|
|
}
|
|
|
|
[Fact]
|
|
public async Task Nach_dem_Zeitfenster_wird_wieder_gemeldet()
|
|
{
|
|
var now = DateTimeOffset.UtcNow;
|
|
var (reporter, handler) = Build(() => now);
|
|
using var _ = reporter;
|
|
|
|
var error = Thrown("Immer wieder");
|
|
|
|
await reporter.ReportAsync(error);
|
|
now = now.AddMinutes(6);
|
|
await reporter.ReportAsync(error);
|
|
|
|
handler.Calls.ShouldBe(2);
|
|
}
|
|
|
|
/// <summary>
|
|
/// Ein Meldeweg, der selbst wirft, wäre die schlechteste aller Welten — der
|
|
/// ursprüngliche Fehler ginge dabei verloren.
|
|
/// </summary>
|
|
[Fact]
|
|
public async Task Ein_abgelehnter_Aufruf_wirft_nicht_nach_aussen()
|
|
{
|
|
var (reporter, _) = Build(status: HttpStatusCode.Unauthorized);
|
|
using var __ = reporter;
|
|
|
|
var sent = await reporter.ReportAsync(Thrown("Egal"));
|
|
|
|
sent.ShouldBeFalse();
|
|
}
|
|
}
|