Function TestFetchAllPages_StopsOnEmptyItems

cmd/ui_test.go:88–133

--------------------------------------------------------------------------- FetchAllPages ---------------------------------------------------------------------------

(t *testing.T)

// ---------------------------------------------------------------------------

func TestFetchAllPages_StopsOnEmptyItems(t *testing.T) {
	viper.Set("CRONITOR_API_KEY", "test-key")
	defer viper.Set("CRONITOR_API_KEY", "")

	var requestCount atomic.Int32

	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
		requestCount.Add(1)
		page := r.URL.Query().Get("page")
		var body string
		switch page {
		case "", "1":
			body = `{"items":[{"id":1}]}`
		case "2":
			body = `{"items":[{"id":2}]}`
		default:
			body = `{"items":[]}`
		}
		w.Header().Set("Content-Type", "application/json")
		w.WriteHeader(200)
		w.Write([]byte(body))
	}))
	defer server.Close()

	client := &lib.APIClient{
		BaseURL:   server.URL,
		ApiKey:    "test-key",
		UserAgent: "test",
	}

	pages, err := FetchAllPages(client, "/monitors", nil, "items")
	if err != nil {
		t.Fatalf("unexpected error: %v", err)
	}

	// Pages 1, 2, and 3 are fetched; page 3 is the empty one that stops iteration.
	// The empty page body is still included in the returned slice.
	if len(pages) != 3 {
		t.Fatalf("expected 3 pages, got %d", len(pages))
	}

	// Verify that request count matches: pages 1, 2, 3
	if int(requestCount.Load()) != 3 {
		t.Errorf("expected 3 requests, got %d", requestCount.Load())
	}
}

func TestFetchAllPages_PageQueryParamsIncrement(t *testing.T) {
	viper.Set("CRONITOR_API_KEY", "test-key")