Following changes are made to the browserstack provider:
- On Every request from provider there was an explicit sleep introduced in the API call, which degraded the performance. Refactored code around that to improve the performance of provider.
- Bug around stopping testcafe session run in JS moded to send STOP two times.
- Provider is spawning BrowserStack Tunnel on its own at the startup of session, which add a significant time in starting of session and also gives less control to user around how it's tunnel is configured. Added a flag to provide capability to use same tunnel for different sessions so that significant time can be saved and more control for logging and configuration of binary.
- Currently for every capability, there is an associated environment variable. This is not a scalable approach as for every new capability we have to open a PR to add an environment variable related to that. So instead of using environment variable user can provider a path to JSON file to use for capability declaration. Prefrence would be given to the values of file over environment variable.
Related PR regarding above points:
| Points | PR Link |
|---|---|
| 1, 2 | #84 |
| 3 | #97 |
| 4 | #102 |