X Tutup
Skip to content

Rf v5 v6#1807

Merged
emanlove merged 10 commits intorobotframework:masterfrom
emanlove:rf-v5-v6
Dec 28, 2022
Merged

Rf v5 v6#1807
emanlove merged 10 commits intorobotframework:masterfrom
emanlove:rf-v5-v6

Conversation

@emanlove
Copy link
Member

Should come in after the "Remove remote driver server exception" #1806 pull request

emanlove and others added 10 commits October 10, 2022 09:22
In a few unit test cases we were using a generic exception for testing
the webdriver cache. That exception was depreciated and I replaced it
with another.
There is a failing test, `multiple_browsers_executable_path.Chrome
Browser With executable_path Argument`, that is failing due to a recent
change/addition by the Selenium project to have a driver manager which,
apparently, is correcting for this incorrect path to driver test. As
there are several factors in play at this time I am kicking this test
down the road.
Latest builds are failing in somewhat unusual places. I want to see what
is the consitancy of runs for version 4.x, 5.x, and 6.x.
@emanlove emanlove requested a review from aaltat December 22, 2022 03:06
@@ -10,7 +10,7 @@ Documentation Creating test which would work on all browser is not possible.

*** Test Cases ***
Chrome Browser With executable_path Argument
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

#1806 (comment) applies here too. Doesn't really matter which PR it's removed in though.

@emanlove emanlove merged commit e304f85 into robotframework:master Dec 28, 2022
@emanlove emanlove added this to the v6.1.0 milestone Apr 29, 2023
@emanlove emanlove added enhancement priority: low acknowledge To be acknowledged in release notes labels Apr 29, 2023
@emanlove emanlove deleted the rf-v5-v6 branch December 19, 2023 23:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

acknowledge To be acknowledged in release notes enhancement priority: low

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

X Tutup