Tests: refreshAllRepos im Health-Loop #70

Open
opened 2026-07-20 18:44:26 +02:00 by frank · 2 comments
Owner

Teil von #66

Problem

ExpandAllRepos ist getestet; der Health-Pfad refreshAllRepos (Diff → Build → Ensure → ScheduleReload) hat keine eigenen Unit-Tests.

Vorschlag

  • Tests: neue Repos werden erkannt und Reload geplant
  • Tests: keine Änderung → kein Reload
  • Tests: Expand-/Build-Fehler loggen und geben 0 zurück (Watcher bleibt stabil)
  • Optional: EnsureRepos wird für neu hinzugekommene Targets aufgerufen

Akzeptanz

  • Abdeckung in internal/agent (z. B. health_test.go)
  • make test grün
Teil von #66 ## Problem `ExpandAllRepos` ist getestet; der Health-Pfad `refreshAllRepos` (Diff → Build → Ensure → ScheduleReload) hat keine eigenen Unit-Tests. ## Vorschlag - Tests: neue Repos werden erkannt und Reload geplant - Tests: keine Änderung → kein Reload - Tests: Expand-/Build-Fehler loggen und geben 0 zurück (Watcher bleibt stabil) - Optional: EnsureRepos wird für neu hinzugekommene Targets aufgerufen ## Akzeptanz - [ ] Abdeckung in `internal/agent` (z. B. `health_test.go`) - [ ] `make test` grün
Collaborator

🤖 forge agent started

  • Agent: cursor-agent
  • Model: auto
  • Mode: pr
  • Trigger: assignee=agent|+4 mapped
  • Open ToDos: 2

Tests: refreshAllRepos im Health-Loop — Teil von #66

🤖 **forge agent started** - Agent: `cursor-agent` - Model: `auto` - Mode: `pr` - Trigger: `assignee=agent|+4 mapped` - Open ToDos: 2 Tests: refreshAllRepos im Health-Loop — Teil von #66
Collaborator

forge agent failed

agent branch has no commits ahead of main (empty PR); dirty work must be committed before finish
❌ **forge agent failed** ``` agent branch has no commits ahead of main (empty PR); dirty work must be committed before finish ```
Sign in to join this conversation.
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Reference
frank/forgecli#70
No description provided.