From e5864d941e31a0e4ce59e31bee13e9c7c43909f4 Mon Sep 17 00:00:00 2001 From: Claudomator Agent Date: Sat, 14 Mar 2026 07:03:55 +0000 Subject: test --- internal/api/server_test.go | 20 ++++++++++++++++++++ internal/executor/executor_test.go | 32 ++++++++++++++++++++++++++++++++ 2 files changed, 52 insertions(+) (limited to 'internal') diff --git a/internal/api/server_test.go b/internal/api/server_test.go index 2209a69..ec927c0 100644 --- a/internal/api/server_test.go +++ b/internal/api/server_test.go @@ -896,6 +896,26 @@ func TestServer_CancelTask_Queued_TransitionsToCancelled(t *testing.T) { } } +func TestServer_CancelTask_Blocked_TransitionsToCancelled(t *testing.T) { + srv, store := testServer(t) + createTaskWithState(t, store, "cancel-blocked-1", task.StateBlocked) + + req := httptest.NewRequest("POST", "/api/tasks/cancel-blocked-1/cancel", nil) + w := httptest.NewRecorder() + srv.Handler().ServeHTTP(w, req) + + if w.Code != http.StatusOK { + t.Fatalf("status: want 200, got %d; body: %s", w.Code, w.Body.String()) + } + updated, err := store.GetTask("cancel-blocked-1") + if err != nil { + t.Fatal(err) + } + if updated.State != task.StateCancelled { + t.Errorf("state: want CANCELLED, got %s", updated.State) + } +} + func TestServer_CancelTask_Completed_Returns409(t *testing.T) { srv, store := testServer(t) createTaskWithState(t, store, "cancel-completed-1", task.StateCompleted) diff --git a/internal/executor/executor_test.go b/internal/executor/executor_test.go index 17982f8..f6d0179 100644 --- a/internal/executor/executor_test.go +++ b/internal/executor/executor_test.go @@ -964,6 +964,7 @@ func (m *minimalMockStore) UpdateTaskSummary(taskID, summary string) error func (m *minimalMockStore) AppendTaskInteraction(taskID string, _ task.Interaction) error { return nil } +func (m *minimalMockStore) UpdateTaskAgent(id string, agent task.AgentConfig) error { return nil } func (m *minimalMockStore) lastStateUpdate() (string, task.State, bool) { m.mu.Lock() @@ -1159,3 +1160,34 @@ func TestPool_SpecificAgent_SkipsLoadBalancing(t *testing.T) { t.Errorf("expected claude runner to NOT be called, got %d", claudeRunner.callCount()) } } + +// TestPool_SpecificAgent_PersistsToDB verifies that if a specific agent +// is requested, it is persisted to the database before the task runs. +func TestPool_SpecificAgent_PersistsToDB(t *testing.T) { + store := testStore(t) + geminiRunner := &mockRunner{} + runners := map[string]Runner{ + "gemini": geminiRunner, + } + logger := slog.New(slog.NewTextHandler(os.Stderr, &slog.HandlerOptions{Level: slog.LevelError})) + pool := NewPool(4, runners, store, logger) + + tk := makeTask("persist-gemini") + tk.Agent.Type = "gemini" + store.CreateTask(tk) + + if err := pool.Submit(context.Background(), tk); err != nil { + t.Fatalf("submit: %v", err) + } + + <-pool.Results() + + // Check the task in the database. + reloaded, err := store.GetTask(tk.ID) + if err != nil { + t.Fatalf("get task: %v", err) + } + if reloaded.Agent.Type != "gemini" { + t.Errorf("expected agent type gemini in DB, got %q", reloaded.Agent.Type) + } +} -- cgit v1.2.3