Freegle / Iznik / 4474

64%

master: 70%

Pull #167

circleci

test(handler): cover RetryGroup Post/Put/Patch/Delete and chain retry scoping

Adds 6 tests extending retry_test.go's RetryGroup coverage:

- Post/Put/Patch/Delete: each verb dispatches wrapAll-wrapped handlers and
  retries on a distinct retryable error message.
- TestRetryGroup_MultipleHandlersRetriesAreScoped: documents that each
  handler position is independently retried — a middleware that called
  c.Next() is NOT re-invoked when the next handler recovers on retry.
- TestRetryGroup_NonRetryableStopsImmediately: 4xx errors returned from a
  grouped route must short-circuit immediately (regression guard against
  silently upgrading validation errors to retry behaviour).

Handler package coverage rises from 73.7% to 80.3%.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

8617 of 9983 branches covered (86.32%)

Branch coverage included in aggregate %.

60525 of 98114 relevant lines covered (61.69%)

10.38 hits per line