util/retry: remove condition to ensure timing checks always run

sanki92 · sanki92 · commit 41ed3c6cd572 · 2025-10-04T04:50:35.000+05:30
Release note: None
diff --git a/pkg/util/retry/retry_test.go b/pkg/util/retry/retry_test.go
@@ -529,11 +529,9 @@ func TestRetryWithMaxDuration(t *testing.T) {
 				require.NoError(t, err)
 			}
 
-			if tc.expectedTimeSpent != 0 {
-				require.Equal(
-					t, tc.expectedTimeSpent, timeSource.Since(start), "expected time does not match actual spent time",
-				)
-			}
+			require.Equal(
+				t, tc.expectedTimeSpent, timeSource.Since(start), "expected time does not match actual spent time",
+			)
 		})
 	}
 }

Original file line number	Diff line number	Diff line change
`@@ -529,11 +529,9 @@ func TestRetryWithMaxDuration(t *testing.T) {`
`529`	`529`	`require.NoError(t, err)`
`530`	`530`	`}`
`531`	`531`
`532`		`- if tc.expectedTimeSpent != 0 {`
`533`		`- require.Equal(`
`534`		`- t, tc.expectedTimeSpent, timeSource.Since(start), "expected time does not match actual spent time",`
`535`		`- )`
`536`		`- }`
	`532`	`+ require.Equal(`
	`533`	`+ t, tc.expectedTimeSpent, timeSource.Since(start), "expected time does not match actual spent time",`
	`534`	`+ )`
`537`	`535`	`})`
`538`	`536`	`}`
`539`	`537`	`}`