Skip to content

test(ui): pin the humanized provider retry banner copy across locales and scales - #3701

Merged
Astro-Han merged 1 commit into
apache:mainfrom
me2seeks:test/provider-retry-delay-humanized-copy
Aug 25, 2026
Merged

test(ui): pin the humanized provider retry banner copy across locales and scales#3701
Astro-Han merged 1 commit into
apache:mainfrom
me2seeks:test/provider-retry-delay-humanized-copy

Conversation

@me2seeks

Copy link
Copy Markdown
Contributor

Follow-up to #3611 (and the closed #3402): @Astro-Han asked for the unit-test cases from #3402 to be sent standalone, since #3611 shipped the formatter without tests.

This adds one focused test pinning providerRetryScheduled in both locales:

  • second/minute/hour/day scales (45秒, 1分 15秒, 4小时 28分 3秒, 1天 1小时 1分 1秒 — and the en equivalents)
  • the ceil-to-one-second floor, so a zero-length wait reads 1秒后重试 / Retrying in 1s and never 0秒后重试
  • zero-order units are skipped rather than rendered as 0分 / 0m

Also documents (per Astro-Han's note on #3402) that the zh short-delay string intentionally changed shape: 45 秒后重试45秒后重试 (space dropped).

Generated-by: maka

… and scales

Covers the providerRetryScheduled copy introduced by apache#3611: zh/en
outputs across second/minute/hour/day scales, the ceil-to-one-second
floor for zero-length waits, and that zero-order units are skipped
rather than rendered as "0分" / "0m".

Generated-by: maka

@Astro-Han Astro-Han left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I reviewed this head and found no blocking issues.

Single file adds locale/scale focused tests for provider retry display; no production logic.

No P0-P3. Checks on daf66dc70 — code GO.

简体中文该头未发现阻断。

@Astro-Han Astro-Han left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

APPROVE — daf66dc, test-only, no P0-P2.

@Astro-Han
Astro-Han merged commit d2754b1 into apache:main Aug 25, 2026
1 check passed
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