Removed feature to send BrowserStacks screenshot url to TestRail and made Webapp and Mobileapp helper free from BrowserStack flag/code

qxf2 / qxf2-page-object-model

Write Selenium and Appium tests in Python using the Page Object pattern. This Pythonic GUI and API test automation framework will help you get started with QA automation quickly. It comes with many useful integrations like - email, BrowserStack, Slack, TestRail, etc. This repository is developed and maintained by Qxf2 Services.

https://qxf2.com

MIT License

256 stars 191 forks source link

Removed feature to send BrowserStacks screenshot url to TestRail and made Webapp and Mobileapp helper free from BrowserStack flag/code #433

Closed rohandudam closed 1 week ago

rohandudam commented 3 weeks ago

Removed BrowserStack feature to send screenshot urls to TestRail because BrowserStack no more allowing to access session log api endpoint during the run.
Made Web app and mobile app helpers (Base pages) free from BrowserStack flag/code

Note: I didn't delete complete BrowserStack Library as we need the file during BrowserStack's enhancement feature https://github.com/qxf2/qxf2-page-object-model/issues/432. In this PR I removed the outdated method related to screenshot from library.

rohandudam commented 3 weeks ago

What is the purpose of BS_ENABLE_SCREENSHOTS? I don't notice any changes whether if it is set to true or false

It is updating the debug capability of BrowserStack driver. Look at https://github.com/qxf2/qxf2-page-object-model/blob/master/integrations/cross_browsers/browserstack_runner.py#L41

rohandudam commented 3 weeks ago

However i noticed that for mobile tests, no matter what value we set for remote_browser_platform , it still runs on Browserstack.

I am able to run mobile test on SauceLab, I set REMOTE_BROWSER_PLATFORM = "SL" to run it. Can you please try it. Your observation is true, if you set REMOTE_BROWSER_PLATFORM value other than 'SL'. Look at the code here: https://github.com/qxf2/qxf2-page-object-model/blob/master/core_helpers/drivers/driverfactory.py#L157#L166 @akkuldn this is not a really issue and unrelated to this PR.