-
Notifications
You must be signed in to change notification settings - Fork 118
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
Browserstack test status #306
base: master
Are you sure you want to change the base?
Conversation
remove multiple api calls, use driver to execute js to mark status for better annotation on browserstack dashboard
ignore username/key from config file if those are default values and pick from env variables if available
a35a06c
to
583a4d3
Compare
Thank you for this PR! Much appreciated! 🙏 I will hold off on merging it until the changes we have pending re Selenium4 has been merged. |
Hey, @BeyondEvil do we have an ETA for the changes to be merged? |
ping @BeyondEvil .. |
Thanks for pinging! I've been superbusy, will try to get to this ASAP tho. |
@BeyondEvil bump for this.. |
bump^ any updates on this? |
Hey, sorry for not getting to this yet. 😓 I’m still working on the rewrite and release of the next major release of pytest-html. |
-> Removed extra attempts to mark status
-> Use javascript executor before session quit rather than making a rest api call after session end for visibility on dashboard.