Skip to content
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

Browser manager support #16

Merged
merged 12 commits into from
Jun 23, 2024
Merged

Browser manager support #16

merged 12 commits into from
Jun 23, 2024

Conversation

quigamdev
Copy link
Contributor

Selenium.WebDriver is distributed with browser manager since version 4.11. Browser manager makes sure that driver and browser are compatible and therefore the tests are not failing due to incompatibility of underlaying testing platform. In addition, browser manager is able to download webdrivers and browsers binaries (CfT).

@quigamdev quigamdev requested review from exyi and cafour June 3, 2024 23:02
@quigamdev
Copy link
Contributor Author

quigamdev commented Jun 3, 2024

NOTE: Firefox does support this feature, but it doesn't work on GitHub Actions (reason unknown).

Copy link
Member

@exyi exyi left a comment

Choose a reason for hiding this comment

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

👍

var elem = browser.Single("#span");
elem.Single("///***-*///@@##@šš+š++++---><<>''", By.XPath);
});
throw new System.Exception("SelectMethod_InvalidXPathSelector_ExpectedException was supposed to fail!!!");
Copy link
Member

Choose a reason for hiding this comment

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

This does not really make sense to me 😅 , did you intend to use Assert.Throws?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Assert.Throw<> checks for specific exception. The mistake is incorrect place of the throw new exc.... I have corrected it.

@quigamdev quigamdev merged commit dd1a748 into main Jun 23, 2024
5 checks passed
@quigamdev quigamdev deleted the browser-manager branch June 23, 2024 13:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants