New driver instance starts when trying to switch to alert

[maxim-zaitsev]

Hi all!
I am using pre-release version of selene: 1.0.0.a10
I got an error when i was trying to switch to alert.
Here is a script to reproduce:

`
from selene.browser import driver, visit
from selene.support.jquery_style_selectors import s
from selene import config
from selene.browsers import Browser

config.browser_name = Browser.CHROME

visit('http://javascript.ru/alert')
s('input[value="Запустить"]').click()
alert = driver().switch_to.alert # Current driver instance has been closed and new driver instance is starting
alert.accept() # so we get an exception at the last step: selenium.common.exceptions.NoAlertPresentException: Message: no alert open
`

[yashaka]

Hi, can you please check the same case for raw selenium? I am curious where the real problem lies...

…

On Sat, Mar 25, 2017, 4:46 AM zaitsevmaxim80 ***@***.***> wrote: Hi all! I am using pre-release version of selene: 1.0.0.a10 I got an error when i was trying to switch to alert. Here is a script to reproduce: ` from selene.browser import driver, visit from selene.support.jquery_style_selectors import s from selene import config from selene.browsers import Browser config.browser_name = Browser.CHROME visit('http://javascript.ru/alert') s('input[value="Запустить"]').click() alert = driver().switch_to.alert # Current driver instance has been closed and new driver instance is starting alert.accept() so we get an exception at the last step selenium.common.exceptions.NoAlertPresentException: Message: no alert open ` — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#117>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AB3sntOznYMpncUu5EsXcQDv4Mq1JVXjks5rpMZDgaJpZM4MpAAg> .

[SergeyPirogov]

Hmmm, it's veeery interesting, probably we should add unit test for this.

[yashaka]

We should:) But I am interested whether the raw selenium will work in this case:) It may also fail... :) сб, 25 бер. 2017 о 17:06 Serhii Pirohov <notifications@github.com> пише:

…

Hmmm, it's veeery interesting, probably we should add unit test for this. — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#117 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AB3snkLlwSmUMW3UcJOH-9iIbljjAeZwks5rpS1ggaJpZM4MpAAg> .

[maxim-zaitsev]

The same case for raw selenium is OK, no errors:

`
from selenium import webdriver

driver = webdriver.Chrome(executable_path=r"C:\_Tools\chromedriver.exe")
driver.get('http://javascript.ru/alert')
driver.find_element_by_css_selector('input[value="Запустить"]').click()
alert = driver.switch_to.alert
alert.accept()
`

[yashaka]

ok, great:-) Let's fix Selene then:-)

…

On Sat, Mar 25, 2017, 1:06 PM zaitsevmaxim80 ***@***.***> wrote: The same case for raw selenium is OK, no errors: ` from selenium import webdriver driver = webdriver.Chrome(executable_path=r"C:\_Tools\chromedriver.exe") driver.get('http://javascript.ru/alert') driver.find_element_by_css_selector('input[value="Запустить"]').click() alert = driver.switch_to.alert alert.accept() ` — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#117 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AB3snpyIq6m0hsMk7OYzl9J0XwFsdHpcks5rpTuTgaJpZM4MpAAg> .

[SergeyPirogov]

@zaitsevmaxim80 could you please clarify which python version do you use? Seems this problem relates to v2 vs v3 string unicode =(

[maxim-zaitsev]

I am using
Python 3.5.3
32-bit

[SergeyPirogov]

Thank you. I already identified the problem.

[SergeyPirogov]

Fixed by #118