TEST: Adjusting unit tests

[joao-voltarelli]

Main changes

Edge

There was a change in the webdriver download URL:

• [🐛 Bug]: error sending request for url (https://msedgedriver.azureedge.net/LATEST_RELEASE_138_WINDOWS) SeleniumHQ/selenium#16073
• [rust] Update base URL for Edge web driver SeleniumHQ/selenium#16056

A fix has been made to the Selenium package, but a new release has not been made yet.

On the webdriver-manager side, it still uses the old URL, so for now, I think the solution would be to specify the new URL via parameters in the EdgeChromiumDriverManager class.

Undetected Chrome

Most of the failures in Windows were throwing:

<selenium.webdriver.remote.errorhandler.ErrorHandler object at 0x00000162CF375D50> response = {'status': 500, 'value': '{"value":{"error":"timeout","message":"timeout\\nfrom aborted by navigation: Not attached to...[0x0x75bb67f9+25]\\n\\tRtlGetFullPathName_UEx [0x0x771b7f4d+173]\\n\\tRtlGetFullPathName_UEx [0x0x771b7f1b+123]\\n"}}'}

From my investigation, it was always when executing the CDP command: self.driver.execute_cdp_cmd("Page.setDownloadBehavior", params) when starting the browser. It seems that at the time Selenium was sending the command, the tab was no longer attached.

Since the tests worked normally locally, I suspect that because execution via GitHub Actions was slower, the behavior of Undetected Chrome of opening/reopening tabs at the beginning could be causing this, where it tried to send the CDP command to a tab that had already been replaced.

Possible solution

As an initial solution, I tried forcing a navigation to about:blank before running the CDP command to ensure that the command would be executed in a valid tab.

At first, it seems to have resolved the issue, and the tests are running normally again on Windows.

@hhslepicka @kayqueGovetri , Do you think this is a valid solution? Do you have any other thoughts on this point?

[kayqueGovetri]

LGTM!

[hhslepicka]

The proposed solution is fine. Let's open a new issue and add the reference to monitor the new Selenium release so we can later on remove our local patch!
Great work @joao-voltarelli