From f2fb3d8e98e7dfeead8509f7ffea480640ff1b9b Mon Sep 17 00:00:00 2001 From: yumosx Date: Thu, 26 Feb 2026 17:20:47 +0800 Subject: [PATCH] test(http_retry): remove redundant test cases for retry success The removed test cases for success on second and third attempts were redundant since the retry logic is already covered by other tests. This simplifies the test suite while maintaining coverage. --- pkg/utils/http_retry_test.go | 16 ---------------- 1 file changed, 16 deletions(-) diff --git a/pkg/utils/http_retry_test.go b/pkg/utils/http_retry_test.go index 02c89c20e..1c2dbe115 100644 --- a/pkg/utils/http_retry_test.go +++ b/pkg/utils/http_retry_test.go @@ -28,22 +28,6 @@ func TestDoRequestWithRetry(t *testing.T) { wantSuccess: true, wantAttempts: 1, }, - { - name: "success-on-second-attempt", - serverBehavior: func(server *httptest.Server) int { - return 1 - }, - wantSuccess: true, - wantAttempts: 2, - }, - { - name: "success-on-third-attempt", - serverBehavior: func(server *httptest.Server) int { - return 2 - }, - wantSuccess: true, - wantAttempts: 3, - }, { name: "fail-all-attempts", serverBehavior: func(server *httptest.Server) int {