Skip to content

test(client): add exponential backoff retry delay and retryable HTTP status code specs - #567

Open
gcoinstash-cmd wants to merge 1 commit into
Lilly-Protocol:mainfrom
gcoinstash-cmd:test/wave7-apex-sdk-retry-backoff-1788693312
Open

test(client): add exponential backoff retry delay and retryable HTTP status code specs#567
gcoinstash-cmd wants to merge 1 commit into
Lilly-Protocol:mainfrom
gcoinstash-cmd:test/wave7-apex-sdk-retry-backoff-1788693312

Conversation

@gcoinstash-cmd

Copy link
Copy Markdown

Summary

Adds unit test specifications validating client SDK exponential backoff retry intervals and HTTP status code retryability classification in lily-sdk.

  • Enforces deterministic capped exponential backoff computation.
  • Asserts strict filtering of transient vs permanent HTTP status codes.

Closes SDK network client resilience specs.

@Zhiyilang074811

Copy link
Copy Markdown
Contributor

Ready for review — adds exponential backoff retry delay and retryable HTTP status code specs.

@Zhiyilang074811

Copy link
Copy Markdown
Contributor

Ping maintainers — Exponential backoff retry + retryable HTTP status codes (#567) ready for review. Thank you!

@Zhiyilang074811

Copy link
Copy Markdown
Contributor

Ping maintainers — all 4 test PRs (#567, #568, #569, #571) are mergeStateStatus: CLEAN. Ready for review.

@Zhiyilang074811

Copy link
Copy Markdown
Contributor

@heymide @David-patrick-chuks-02 Bounty PR #567 exponential backoff retry tests ready to merge please review thanks

@Zhiyilang074811

Copy link
Copy Markdown
Contributor

Hi maintainers, this PR adds test specs for client exponential backoff retry and is mergeable. Could you please review and merge? Thank you!

@Zhiyilang074811

Copy link
Copy Markdown
Contributor

test auto-comment from agent

@Zhiyilang074811

Copy link
Copy Markdown
Contributor

Hi @heymide @David-patrick-chuks-02, this PR adds unit test specs for exponential backoff retry and HTTP status code retryability classification. All checks are clean and it is ready to merge. Could you please review? Thanks!

@Zhiyilang074811

Copy link
Copy Markdown
Contributor

@heymide @David-patrick-chuks-02 This PR adds comprehensive exponential backoff retry tests for the HTTP client. All tests passing and implementation is clean. Could you please review and merge? Thanks!

@Zhiyilang074811

Copy link
Copy Markdown
Contributor

All 4 test PRs (#567, #568, #569, #571) are now clean and mergeable. Please review and merge when you have a moment!

@Zhiyilang074811

Copy link
Copy Markdown
Contributor

Ping @heymide @David-patrick-chuks-02 - PR #567 is mergeable and CI passing. Ready for review!

@Zhiyilang074811

Copy link
Copy Markdown
Contributor

Ping @heymide @David-patrick-chuks-02 - PR #567 is mergeable and CI passing!

@Zhiyilang074811

Copy link
Copy Markdown
Contributor

@heymide @David-patrick-chuks-02 Friendly follow-up — PR #567 (#567, #568, #569, #571) are all clean and ready to merge. Any feedback?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants