romanov73/tis-2017
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Stroeva #7

Closed
StroevaJ wants to merge 6 commits from Stroeva into master
pull from: Stroeva
merge into: romanov73:master
Conversation 15 Commits 6 Files Changed 24 +1183
StroevaJ commented 2018-05-07 00:34:00 +04:00 (Migrated from gitlab.com)
Copy Link
No description provided.
romanov73 commented 2018-05-07 11:23:31 +04:00 (Migrated from gitlab.com)
Copy Link

testFavoritePage = assertTrue(pageAn.isFavoritePage())

testFuturePage = assertTrue(pageAn.isFuturePage())
?

`testFavoritePage` = `assertTrue(pageAn.isFavoritePage())` `testFuturePage` = `assertTrue(pageAn.isFuturePage())` ?
romanov73 commented 2018-05-07 11:24:17 +04:00 (Migrated from gitlab.com)
Copy Link

тестируем поиск, а валидируем пользователя?

тестируем поиск, а валидируем пользователя?
romanov73 commented 2018-05-07 11:25:22 +04:00 (Migrated from gitlab.com)
Copy Link

процерять нужно не пользователя, а специфичную информацию со страницы

процерять нужно не пользователя, а специфичную информацию со страницы
StroevaJ commented 2018-05-07 14:13:08 +04:00 (Migrated from gitlab.com)
Copy Link

changed this line in version 2 of the diff

changed this line in [version 2 of the diff](https://gitlab.com/romanov73/tis-2017/merge_requests/6/diffs?diff_id=14190402&start_sha=54217223b63ba197bb5749848e420132c1d9ead1#2dbf693a68659bc8ca8eb31e8a4edd3ec4d17d7e_90_90)
StroevaJ commented 2018-05-07 14:13:08 +04:00 (Migrated from gitlab.com)
Copy Link

changed this line in version 2 of the diff

changed this line in [version 2 of the diff](https://gitlab.com/romanov73/tis-2017/merge_requests/6/diffs?diff_id=14190402&start_sha=54217223b63ba197bb5749848e420132c1d9ead1#76eab88c3c46b7ae29749635fb1afb11424e44cd_37_37)
StroevaJ commented 2018-05-07 14:13:09 +04:00 (Migrated from gitlab.com)
Copy Link

changed this line in version 2 of the diff

changed this line in [version 2 of the diff](https://gitlab.com/romanov73/tis-2017/merge_requests/6/diffs?diff_id=14190402&start_sha=54217223b63ba197bb5749848e420132c1d9ead1#47bb0794861b79ac28d7da0c96df2cdaa48f2ced_42_42)
StroevaJ commented 2018-05-07 14:13:09 +04:00 (Migrated from gitlab.com)
Copy Link

added 1 commit

  • f7ab019a - Исправлены замечания

Compare with previous version

added 1 commit <ul><li>f7ab019a - Исправлены замечания</li></ul> [Compare with previous version](https://gitlab.com/romanov73/tis-2017/merge_requests/6/diffs?diff_id=14190402&start_sha=54217223b63ba197bb5749848e420132c1d9ead1)
romanov73 commented 2018-05-11 10:11:56 +04:00 (Migrated from gitlab.com)
Copy Link

Комментарий ко всем методам:

проверяется нахождение на станице элементов, т.е. только переход на страницу с результатами. В то же время не проверятся, что искомый результат найден

Комментарий ко всем методам: проверяется нахождение на станице элементов, т.е. только переход на страницу с результатами. В то же время не проверятся, что искомый результат найден
StroevaJ commented 2018-05-11 22:52:57 +04:00 (Migrated from gitlab.com)
Copy Link

added 1 commit

  • 594b41c4 - Исправлены замечания

Compare with previous version

added 1 commit <ul><li>594b41c4 - Исправлены замечания</li></ul> [Compare with previous version](https://gitlab.com/romanov73/tis-2017/merge_requests/6/diffs?diff_id=14403038&start_sha=f7ab019a6e3c0c2f72ab906d80e02fed8d87fa98)
StroevaJ commented 2018-05-11 22:58:26 +04:00 (Migrated from gitlab.com)
Copy Link

Фильтры по стране и по жанру разве не такие?
Выбирается пункт фильтра, выводится список фильмов и потом в первом попавшемся проверяется в описании фильма есть ли там искомые результаты

Фильтры по стране и по жанру разве не такие? Выбирается пункт фильтра, выводится список фильмов и потом в первом попавшемся проверяется в описании фильма есть ли там искомые результаты
romanov73 commented 2018-05-12 12:15:08 +04:00 (Migrated from gitlab.com)
Copy Link

genreAnySerialElement.click();
означает что выбираем произвольный жанр. А как узнать что найден именной корректный? а не так что нажали боевик, а вышли комедии

`genreAnySerialElement.click();` означает что выбираем произвольный жанр. А как узнать что найден именной корректный? а не так что нажали `боевик`, а вышли `комедии`
StroevaJ commented 2018-05-13 18:26:09 +04:00 (Migrated from gitlab.com)
Copy Link

added 1 commit

  • 658e827b - Исправлен поиск

Compare with previous version

added 1 commit <ul><li>658e827b - Исправлен поиск</li></ul> [Compare with previous version](https://gitlab.com/romanov73/tis-2017/merge_requests/6/diffs?diff_id=14430478&start_sha=594b41c40213ace317d4aa6b789b85cd59d61649)
StroevaJ commented 2018-05-13 18:30:26 +04:00 (Migrated from gitlab.com)
Copy Link

page.clickGenre(); - означает, что выбираем произвольный жанр
genreAnySerialElement.click(); - означает, что после того, как мы выбрали жанр, мы открываем первый попавшийся результат и смотрим в нем действительно ли это боевик

Когда мы выбираем жанр, то на странице результатов не написаны жанры фильмов, они описаны только на самих страницах фильмов, поэтому и приходится открывать результат и смотреть то ли это
Или мне надо циклом каждый фильм на странице результатов открывать и проверять соответствует ли он запросу?

page.clickGenre(); - означает, что выбираем произвольный жанр genreAnySerialElement.click(); - означает, что после того, как мы выбрали жанр, мы открываем первый попавшийся результат и смотрим в нем действительно ли это боевик Когда мы выбираем жанр, то на странице результатов не написаны жанры фильмов, они описаны только на самих страницах фильмов, поэтому и приходится открывать результат и смотреть то ли это Или мне надо циклом каждый фильм на странице результатов открывать и проверять соответствует ли он запросу?
romanov73 commented 2018-05-14 09:15:43 +04:00 (Migrated from gitlab.com)
Copy Link

resolved all discussions

resolved all discussions
romanov73 commented 2018-05-14 09:15:56 +04:00 (Migrated from gitlab.com)
Copy Link

closed

closed

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: romanov73/tis-2017#7
No description provided.
Delete Branch "Stroeva"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?