fix switch-window with PhantomJS #264

Uunnamed · 2020-08-04T11:54:09Z

#fixies #146

igrishaev · 2020-08-04T13:15:06Z

test/etaoin/api_test.clj

@@ -354,6 +354,15 @@
        (is (not= width width'))
        (is (not= height height'))))))



я как-то не пойму логику этого теста. Ожидал, что будет так:

открыть первую страницу, получить заголовок или урл

создать новое окно

открыть вторую странцу, получить ее заголовок или урл

переключиться на первое окно

убедиться, что его заголовк или урл равны первому.

чтобы не ходить в сеть, можно создать локальные HTML-файлы test1.html и test2.html

логика такая:

браузер открывается с одним единственным окном

получаем его id

открываем следующее окно

получаем id всех окон, и выбираем id нового окна

свитчимся и проверяем что действительно переключились

страницы и хождение по урлу это дополнительная проверка скорее, если нужно добавлю

Uunnamed added 2 commits August 4, 2020 14:52

fix switch-window with PhantomJS

c2a0c84

update test

37e94e9

igrishaev reviewed Aug 4, 2020

View reviewed changes

update

fd0a752

Uunnamed merged commit e236ed1 into master Aug 5, 2020

Uunnamed deleted the fix-switch-window-with-phantomjs branch August 5, 2020 09:09

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

fix switch-window with PhantomJS #264

fix switch-window with PhantomJS #264

Uunnamed commented Aug 4, 2020

igrishaev Aug 4, 2020 •

edited

Loading

Uunnamed Aug 4, 2020 •

edited

Loading

		@@ -354,6 +354,15 @@
		(is (not= width width'))
		(is (not= height height'))))))

fix switch-window with PhantomJS #264

fix switch-window with PhantomJS #264

Conversation

Uunnamed commented Aug 4, 2020

igrishaev Aug 4, 2020 • edited Loading

Choose a reason for hiding this comment

Uunnamed Aug 4, 2020 • edited Loading

Choose a reason for hiding this comment

igrishaev Aug 4, 2020 •

edited

Loading

Uunnamed Aug 4, 2020 •

edited

Loading