Add retry option to flaky Throttler test

[DZakh]

Summary

Updated a flaky test in the Throttler test suite to use the itWithOptions function with a retry configuration, improving test reliability.

Key Changes

• Changed Async.it to Async.itWithOptions for the "Does not continuously increase schedule time" test
• Added {retry: 3} option to allow the test to retry up to 3 times on failure
• This addresses intermittent test failures without modifying the test logic itself

Details

The test was experiencing occasional failures, likely due to timing-sensitive assertions in the throttler scheduling behavior. By enabling retries at the test framework level, transient failures are automatically handled while still catching genuine bugs that consistently fail.

https://claude.ai/code/session_019EsgwvJKKEAQ9LYGYhkH78

Summary by CodeRabbit

• Tests
  • Enhanced test reliability with automatic retry configuration for critical test cases.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 70bdcca and 65975fc.

📒 Files selected for processing (1)

• scenarios/test_codegen/test/lib_tests/Throttler_test.res

---

📝 Walkthrough

Walkthrough

Updated a test wrapper in the Throttler test suite from Async.it to Async.itWithOptions, adding a retry configuration of 3 attempts while keeping the test logic unchanged.

Changes

Cohort / File(s)	Summary
Test Configuration Update scenarios/test_codegen/test/lib_tests/Throttler_test.res	Modified test case wrapper from Async.it to Async.itWithOptions and added retry option of 3 for the "Does not continuously increase schedule time" test case.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A test needs patience, not just speed,
With three retries planted like a seed,
The Throttler checks its steady beat,
Now resilience makes it neat! ✨

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly and specifically describes the main change: adding a retry option to a flaky test in the Throttler test suite.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch claude/add-schedule-retries-JuePM

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}