Function TestQueryDetailFansOutAcrossCPs

controlplane/admin/live_test.go:64–94

TestQueryDetailFansOutAcrossCPs proves /queries/by-worker/:wid scatter-gathers: when the serving replica doesn't own the worker, it fetches peers and returns the one owner's body; and a scope=local (peer) call never fans out.

(t *testing.T)

// TestQueryDetailFansOutAcrossCPs proves /queries/by-worker/:wid scatter-gathers:
// when the serving replica doesn't own the worker, it fetches peers and returns
// the one owner's body; and a scope=local (peer) call never fans out.
func TestQueryDetailFansOutAcrossCPs(t *testing.T) {
	local := &fakeLiveInfo{} // owns nothing
	peerBody, _ := json.Marshal(QueryDetail{Org: "b", PID: 77, WorkerID: 20, Query: "SELECT 2"})
	fetcher := &fakePeerFetcher{byPath: map[string][][]byte{"/api/v1/queries/by-worker/20": {peerBody}}}
	r := liveTestRouter(local, fetcher)

	// Default scope: not owned locally → fan out → peer's detail returned.
	w := httptest.NewRecorder()
	r.ServeHTTP(w, httptest.NewRequest(http.MethodGet, "/api/v1/queries/by-worker/20", nil))
	if w.Code != http.StatusOK {
		t.Fatalf("expected 200 from peer fan-out, got %d (%s)", w.Code, w.Body.String())
	}
	var got QueryDetail
	if err := json.Unmarshal(w.Body.Bytes(), &got); err != nil {
		t.Fatalf("decode: %v", err)
	}
	if got.WorkerID != 20 || got.Query != "SELECT 2" || got.Org != "b" {
		t.Fatalf("unexpected fanned-out body: %+v", got)
	}

	// scope=local: recursion guard — no fan-out even with a fetcher present.
	before := fetcher.callCount()
	w = httptest.NewRecorder()
	r.ServeHTTP(w, httptest.NewRequest(http.MethodGet, "/api/v1/queries/by-worker/20?scope=local", nil))
	if w.Code != http.StatusNotFound {
		t.Fatalf("expected 404 under scope=local, got %d", w.Code)
	}
	if fetcher.callCount() != before {
		t.Fatalf("scope=local must not fan out, but FetchPeers was called (%d → %d)", before, fetcher.callCount())
	}
}

// TestQueryDetailWorkerIDDistinguishesCollidingPIDs is the regression for the