Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Extend modal close wait with custom timeout #3783

Merged
merged 4 commits into from
May 24, 2024

Conversation

leboff
Copy link
Contributor

@leboff leboff commented May 10, 2024

Enhanced flexibility in wait_until_modal_is_closed method by adding an optional timeout parameter. This allows custom waiting periods for slower environments or particularly time-consuming operations, improving test reliability. Default timeout remains unchanged at 15 seconds.

Enhanced flexibility in `wait_until_modal_is_closed` method by adding an optional timeout parameter. This allows custom waiting periods for slower environments or particularly time-consuming operations, improving test reliability. Default timeout remains unchanged at 15 seconds.
@leboff leboff requested a review from a team as a code owner May 10, 2024 16:40
@vsbharath
Copy link
Contributor

@leboff Can this custom timeout be considered for wait_until_modal_is_open and select_app_launcher_app as well to make this consistent?

Refactored two methods to allow custom timeout values
@leboff
Copy link
Contributor Author

leboff commented May 20, 2024

@leboff Can this custom timeout be considered for wait_until_modal_is_open and select_app_launcher_app as well to make this consistent?

@vsbharath I've updated the branch with those changes.

@vsbharath vsbharath merged commit 706c71b into SFDO-Tooling:main May 24, 2024
54 of 63 checks passed
@jstvz jstvz removed the in review label May 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants