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

Cut off WebElement screenshot size when it doesn't fit in full page screenshot #378

Merged
merged 2 commits into from Dec 11, 2016

Conversation

Projects
None yet
2 participants
@BorisOsipov
Collaborator

BorisOsipov commented Aug 21, 2016

#373

  • cut off WebElement screenshot size when it doesn't fit in full page screenshot
  • add warning message for cases when element isn’t displayed on current screenshot
#373
- cut off WebElement screenshot size when it doesn't fit in full page screenshot
- add warning message for cases when element isn’t displayed on current screenshot
@asolntsev

This comment has been minimized.

Show comment
Hide comment
@asolntsev

asolntsev Aug 21, 2016

Member

@BorisOsipov Can you add some tests? Like a page that has an element that needs to be cut.

Member

asolntsev commented Aug 21, 2016

@BorisOsipov Can you add some tests? Like a page that has an element that needs to be cut.

@asolntsev

This comment has been minimized.

Show comment
Hide comment
@asolntsev
Member

asolntsev commented Sep 19, 2016

@BorisOsipov ping!

@asolntsev

This comment has been minimized.

Show comment
Hide comment
@asolntsev
Member

asolntsev commented Sep 25, 2016

@BorisOsipov ping?

@BorisOsipov

This comment has been minimized.

Show comment
Hide comment
@BorisOsipov

BorisOsipov Nov 24, 2016

Collaborator

@asolntsev pong!

Can you add some tests?

Can you look this changes. That's good?
BorisOsipov@e3c9509

p.s. а это ок, что я не могу запустить нормально локально тесты для селенида, которые запускают реальный браузер? для тестирования, перед стартом тестов запускает локальный http server с https и http. Https не заработал т.к. браузер(firefox) ругался на сертификат, а http я так и не понял почему не отдавал странички. В итоге после получаса разбирательств потестил все на поднятом MS IIS отключив лишнее(

Collaborator

BorisOsipov commented Nov 24, 2016

@asolntsev pong!

Can you add some tests?

Can you look this changes. That's good?
BorisOsipov@e3c9509

p.s. а это ок, что я не могу запустить нормально локально тесты для селенида, которые запускают реальный браузер? для тестирования, перед стартом тестов запускает локальный http server с https и http. Https не заработал т.к. браузер(firefox) ругался на сертификат, а http я так и не понял почему не отдавал странички. В итоге после получаса разбирательств потестил все на поднятом MS IIS отключив лишнее(

@asolntsev

This comment has been minimized.

Show comment
Hide comment
@asolntsev

asolntsev Nov 30, 2016

Member

@BorisOsipov Нет, это ненормально. Тесты селенида запускаются, кажется, только на https. Насчёт firefox не знаю - он уже несколько месяцев не работает нифига, а вот на хроме, phantomjs и htmlunit все тесты прекрасно бегают.

Member

asolntsev commented Nov 30, 2016

@BorisOsipov Нет, это ненормально. Тесты селенида запускаются, кажется, только на https. Насчёт firefox не знаю - он уже несколько месяцев не работает нифига, а вот на хроме, phantomjs и htmlunit все тесты прекрасно бегают.

@asolntsev

This comment has been minimized.

Show comment
Hide comment
@asolntsev

asolntsev Nov 30, 2016

Member

@BorisOsipov Да, тест ок. Со старым кодом красный, с новым кодом - зелёный. Значит, тест доказывает необходимость изменения. Коммить! Я смержу pull request. Один падающий тест в Travis CI, похоже, не связан с твоими изменениями.

Member

asolntsev commented Nov 30, 2016

@BorisOsipov Да, тест ок. Со старым кодом красный, с новым кодом - зелёный. Значит, тест доказывает необходимость изменения. Коммить! Я смержу pull request. Один падающий тест в Travis CI, похоже, не связан с твоими изменениями.

@BorisOsipov

This comment has been minimized.

Show comment
Hide comment
@BorisOsipov

BorisOsipov Dec 1, 2016

Collaborator

@asolntsev Done.

По поводу проблем с https и прочим. Там 2 проблемы:

  1. Какая-то проблема с запуском на Windows. Мб firewall или еще что-то. На ubuntu заработало отлично.
  2. Файрфокс блочит в некоторых случаех открытие страницы если SSL сертификат невалидный.
    В общем мои локальные проблемы.
Collaborator

BorisOsipov commented Dec 1, 2016

@asolntsev Done.

По поводу проблем с https и прочим. Там 2 проблемы:

  1. Какая-то проблема с запуском на Windows. Мб firewall или еще что-то. На ubuntu заработало отлично.
  2. Файрфокс блочит в некоторых случаех открытие страницы если SSL сертификат невалидный.
    В общем мои локальные проблемы.

@asolntsev asolntsev merged commit f884a87 into codeborne:master Dec 11, 2016

1 check failed

continuous-integration/travis-ci/pr The Travis CI build could not complete due to an error
Details

asolntsev added a commit that referenced this pull request Dec 11, 2016

asolntsev added a commit that referenced this pull request Dec 11, 2016

@asolntsev

This comment has been minimized.

Show comment
Hide comment
@asolntsev

asolntsev Dec 13, 2016

Member

@BorisOsipov Помоги пожалуйста!
На нашем корпоративном дженкинсе падают три теста (локально и на ci.selenide.org они зелёные). Никак не могу понять, почему они падают, причём не везде.

Test Result (3 failures / ±0)
* integration.ScreenshotTest.resizeBigImageWidth
* integration.ScreenshotTest.resizeBigImageHeight
* integration.ScreenshotTest.resizeBigImage

И стэк трейс одного из них:

Error Message

java.lang.AssertionError: Screenshot doesn't fit width - element.location: (8, 80), element.size: (3000, 3000), screen.size: (3008,3089)
Expected: is a value less than <3000>
     but: <3000> was equal to <3000>
Stacktrace

java.lang.AssertionError: Screenshot doesn't fit width - element.location: (8, 80), element.size: (3000, 3000), screen.size: (3008,3089)
Expected: is a value less than <3000>
     but: <3000> was equal to <3000>
	at org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:20)
	at org.junit.Assert.assertThat(Assert.java:956)
	at integration.ScreenshotTest.resizeBigImage(ScreenshotTest.java:73)

Браузер, кажется, Chrome.

Member

asolntsev commented Dec 13, 2016

@BorisOsipov Помоги пожалуйста!
На нашем корпоративном дженкинсе падают три теста (локально и на ci.selenide.org они зелёные). Никак не могу понять, почему они падают, причём не везде.

Test Result (3 failures / ±0)
* integration.ScreenshotTest.resizeBigImageWidth
* integration.ScreenshotTest.resizeBigImageHeight
* integration.ScreenshotTest.resizeBigImage

И стэк трейс одного из них:

Error Message

java.lang.AssertionError: Screenshot doesn't fit width - element.location: (8, 80), element.size: (3000, 3000), screen.size: (3008,3089)
Expected: is a value less than <3000>
     but: <3000> was equal to <3000>
Stacktrace

java.lang.AssertionError: Screenshot doesn't fit width - element.location: (8, 80), element.size: (3000, 3000), screen.size: (3008,3089)
Expected: is a value less than <3000>
     but: <3000> was equal to <3000>
	at org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:20)
	at org.junit.Assert.assertThat(Assert.java:956)
	at integration.ScreenshotTest.resizeBigImage(ScreenshotTest.java:73)

Браузер, кажется, Chrome.

@BorisOsipov

This comment has been minimized.

Show comment
Hide comment
@BorisOsipov

BorisOsipov Dec 13, 2016

Collaborator

@asolntsev выглядит так, что на вашем CI каким-то образом скриншотится все окно браузера, а не только его видимая часть - тогда обрезать нечего и тесты падают.
Думаю имеет смысл повторить исходную проблему на вашем CI из ишью
#373
(она не будет у вас повторятся видимо)

я сделал пример:
BorisOsipov@98de953
добавишь временно тест?

Collaborator

BorisOsipov commented Dec 13, 2016

@asolntsev выглядит так, что на вашем CI каким-то образом скриншотится все окно браузера, а не только его видимая часть - тогда обрезать нечего и тесты падают.
Думаю имеет смысл повторить исходную проблему на вашем CI из ишью
#373
(она не будет у вас повторятся видимо)

я сделал пример:
BorisOsipov@98de953
добавишь временно тест?

@asolntsev

This comment has been minimized.

Show comment
Hide comment
@asolntsev

asolntsev Feb 24, 2017

Member
Member

asolntsev commented Feb 24, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment