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

feat: allow to slow down browser actions #788

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

bagrat
Copy link

@bagrat bagrat commented Aug 22, 2024

For cases mentioned in #337 and in order to enable the developer to watch their test execute in a comprehensible way while developing, I've added the :slow_down config option to implement that feature.

I'm not sure how to test this automatically, but open for suggestions and recommendations. I've verified this manually.

@mhanberg
Copy link
Member

Hi! I have a couple questions/comments

Q: Are there any similar features in similar libraries for other languages?

C: this is just implemented in the chrome driver adapter, it would need to be useable for all adapters

C: features like this should be implemented in a way that you can pass the option to start_session, and fall back to app config, then fall back to a default. And in this case, potentially even pass it to individual functions

@bagrat
Copy link
Author

bagrat commented Aug 22, 2024

Hey, thanks for a quick response!

Q: Are there any similar features in similar libraries for other languages?

Yes, I've used them both in Puppeteer and Playwright before, it's called slowMo.

C: this is just implemented in the chrome driver adapter, it would need to be useable for all adapters

Could you possible point me in the right direction where this delay should also be put in?

C: features like this should be implemented in a way that you can pass the option to start_session, and fall back to app config, then fall back to a default. And in this case, potentially even pass it to individual functions

In chrome.ex and selenium.ex?

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