Skip to content

Conversation

@lewijw
Copy link
Contributor

@lewijw lewijw commented Apr 24, 2017

Does anyone have a problem with this change? Note there are some special circumstances. First, the Selenium method is declared @beta. See:

https://seleniumhq.github.io/selenium/docs/api/java/org/openqa/selenium/Alert.html

Second, I have a test that is working with the new method, but it only works if I use the explorer driver. Any advice or information on why that might be or what I might do to resolve this is appreciated. Details: with the chrome or firefox drivers, Selenium is having problems finding the alert even though it is clearly appearing. The timeout is occurring. I set it to 10 seconds. I am also making sure I set the browserID after the timeout.

@leiwang008
Copy link
Member

I know that there are 3 kinds of Alert provided by HTML DOM, "alert", "confirm" and "prompt". Please refer to https://www.w3schools.com/js/js_popup.asp. They are supported by Selenium API, we have regression test running with them every day, please refer to DriverMiscCommandTests.testAPI_Misc_Alert().

We need to know what type of alert is being tested and if it is supported by Selenium. Maybe Some javascript window looks like an Alert, but it is not considered as an Alert by Selenium.
If it is supported by IE as you said, then we might need to check the version of chromedriver.exe and selenium-driver-standalone.xxx.jar to get it supported in chrome or firefox.

@leiwang008 leiwang008 force-pushed the Add_AlertAuthenticateUsingUserAndPassword branch 2 times, most recently from 83ed1c8 to ff9c55d Compare July 7, 2017 02:11
@leiwang008 leiwang008 force-pushed the Add_AlertAuthenticateUsingUserAndPassword branch from ff9c55d to 7431faf Compare July 10, 2017 07:46
@leiwang008 leiwang008 force-pushed the Add_AlertAuthenticateUsingUserAndPassword branch from 7431faf to 13aa529 Compare July 13, 2017 05:28
@leiwang008 leiwang008 force-pushed the Add_AlertAuthenticateUsingUserAndPassword branch 3 times, most recently from b08b3c9 to cc87e58 Compare July 13, 2017 06:25
@leiwang008 leiwang008 force-pushed the Add_AlertAuthenticateUsingUserAndPassword branch from cc87e58 to 5ab2545 Compare July 17, 2017 03:04
@leiwang008 leiwang008 force-pushed the Add_AlertAuthenticateUsingUserAndPassword branch 2 times, most recently from 45b1807 to 51224b9 Compare July 17, 2017 07:10
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.

3 participants