Closed passsy closed 7 months ago
[!WARNING]
Rate Limit Exceeded
@passsy has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 14 minutes and 1 seconds before requesting another review.
How to resolve this issue?
After the wait time has elapsed, a review can be triggered using the `@coderabbitai review` command as a PR comment. Alternatively, push new commits to this PR. We recommend that you space out your commits to avoid hitting the rate limit.How do rate limits work?
CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our [FAQ](https://coderabbit.ai/docs/faq) for further information.Commits
Files that changed from the base of the PR and between ba533664d8daf283af07c95e4ec41874eafe2aa1 and 2a6c89a0a5283355bcc4e6987fb209e1937464d6.
The modifications streamline the framework's widget selection process by consolidating previous parameters into a singular stages
parameter, introducing a new class for text filtering, and adjusting test assertions accordingly. This reflects a shift towards a more unified and simplified approach in handling widget properties and their relationships, enhancing the clarity and efficiency of the codebase.
File Path | Change Summary |
---|---|
lib/src/spot/.../any_text.dart |
Replaced props , children , parents with stages ; added MatchTextFilter ; commented out createElementFilters . |
test/.../selector_test.dart |
Commented out old assertions; added new ones for quantityConstraint ; updated API tests. |
test/spot/.../exists_once_test.dart |
Modified assertions and comments; updated element locating logic. |
🐇✨
In the realm of code, where widgets play,
A rabbit hopped, refining the way.
With a leap and a bound,stages
took flight,
Simplifying the dance, from morning till night.
🌟🐾
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Refactor
WidgetSelector
to have a chain elementFilters, calledstages
.These changes allow
.atIndex(n)
to work correctly. Beforespot<Text>().atIndex(1).withText('b').existsOnce();
would not work as expected, because theatIndex
filter was always executed last, which is different than executing it beforewithText
.Todo
Summary by CodeRabbit
stages
parameter, enhancing code readability and maintenance.MatchTextFilter
for more intuitive text matching in widgets.