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

Skalkin #6

Closed
antonskalkin73 wants to merge 9 commits from Skalkin into master
pull from: Skalkin
merge into: romanov73:master
Conversation 20 Commits 9 Files Changed 34 +1446
antonskalkin73 commented 2018-05-03 11:58:54 +04:00 (Migrated from gitlab.com)
Copy Link
No description provided.
romanov73 commented 2018-05-03 12:41:11 +04:00 (Migrated from gitlab.com)
Copy Link

много пустых строк

много пустых строк
romanov73 commented 2018-05-03 12:41:27 +04:00 (Migrated from gitlab.com)
Copy Link

комментированный код не коммитим

комментированный код не коммитим
antonskalkin73 commented 2018-05-05 23:02:23 +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/5/diffs?diff_id=14161130&start_sha=32cd4d5f6d27f633ca7531347f3f403f4e4827aa#0315abd11bacb1aecbdd2eb21d65d7a44b36447e_39_35)
antonskalkin73 commented 2018-05-05 23:02:24 +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/5/diffs?diff_id=14161130&start_sha=32cd4d5f6d27f633ca7531347f3f403f4e4827aa#0315abd11bacb1aecbdd2eb21d65d7a44b36447e_41_35)
antonskalkin73 commented 2018-05-05 23:02:25 +04:00 (Migrated from gitlab.com)
Copy Link

added 1 commit

  • 96d2bb76 - Исправлены ошибки:

Compare with previous version

added 1 commit <ul><li>96d2bb76 - Исправлены ошибки:</li></ul> [Compare with previous version](https://gitlab.com/romanov73/tis-2017/merge_requests/5/diffs?diff_id=14161130&start_sha=32cd4d5f6d27f633ca7531347f3f403f4e4827aa)
romanov73 commented 2018-05-07 11:18:24 +04:00 (Migrated from gitlab.com)
Copy Link

Почему везде проверяется isUserPresent?
Даже если во время тестирования будет открываться одна и та же страница, а не информация об актерах, о фильме и т.д. тесты не упадут.
Нужно детектировать информацию, которая специфична для каждой страницы

Почему везде проверяется `isUserPresent`? Даже если во время тестирования будет открываться одна и та же страница, а не информация об актерах, о фильме и т.д. тесты не упадут. Нужно детектировать информацию, которая специфична для каждой страницы
romanov73 commented 2018-05-07 11:20:02 +04:00 (Migrated from gitlab.com)
Copy Link

Вообще не та проверка. Тестируем простой поиск а в качестве ассерта - проверка существования пользователя

Вообще не та проверка. Тестируем простой поиск а в качестве ассерта - проверка существования пользователя
romanov73 commented 2018-05-07 11:21:31 +04:00 (Migrated from gitlab.com)
Copy Link

Везде копипаста. Что за element?

Везде копипаста. Что за element?
antonskalkin73 commented 2018-05-07 15:41:47 +04:00 (Migrated from gitlab.com)
Copy Link

changed this line in version 3 of the diff

changed this line in [version 3 of the diff](https://gitlab.com/romanov73/tis-2017/merge_requests/5/diffs?diff_id=14194492&start_sha=96d2bb769cd12df626974511f7b6987630ebd163#19b3bff50c62cf8b787383e6d64db86d20a3e47b_38_37)
antonskalkin73 commented 2018-05-07 15:41:48 +04:00 (Migrated from gitlab.com)
Copy Link

changed this line in version 3 of the diff

changed this line in [version 3 of the diff](https://gitlab.com/romanov73/tis-2017/merge_requests/5/diffs?diff_id=14194492&start_sha=96d2bb769cd12df626974511f7b6987630ebd163#2870cd23eb9139c538352df2320ce7da559c6122_39_39)
antonskalkin73 commented 2018-05-07 15:41:48 +04:00 (Migrated from gitlab.com)
Copy Link

added 1 commit

  • 315eba8c - Исправлены ошибки второго призыва.

Compare with previous version

added 1 commit <ul><li>315eba8c - Исправлены ошибки второго призыва.</li></ul> [Compare with previous version](https://gitlab.com/romanov73/tis-2017/merge_requests/5/diffs?diff_id=14194492&start_sha=96d2bb769cd12df626974511f7b6987630ebd163)
antonskalkin73 commented 2018-05-07 15:45:18 +04:00 (Migrated from gitlab.com)
Copy Link

added 1 commit

  • a46383c4 - Исправлены ошибки второго призыва.

Compare with previous version

added 1 commit <ul><li>a46383c4 - Исправлены ошибки второго призыва.</li></ul> [Compare with previous version](https://gitlab.com/romanov73/tis-2017/merge_requests/5/diffs?diff_id=14194619&start_sha=315eba8c264cf8bdf316885341443a89a00634a4)
romanov73 commented 2018-05-11 10:07:39 +04:00 (Migrated from gitlab.com)
Copy Link

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

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

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

added 1 commit

  • 22445200 - Улучшено тестирование поиска. Теперь проверяет элементы на странице

Compare with previous version

added 1 commit <ul><li>22445200 - Улучшено тестирование поиска. Теперь проверяет элементы на странице</li></ul> [Compare with previous version](https://gitlab.com/romanov73/tis-2017/merge_requests/5/diffs?diff_id=14403435&start_sha=a46383c4fb104484f7e9925d33dbd8ffc598cc61)
antonskalkin73 commented 2018-05-11 23:09:59 +04:00 (Migrated from gitlab.com)
Copy Link

resolved all discussions

resolved all discussions
romanov73 commented 2018-05-12 12:17:27 +04:00 (Migrated from gitlab.com)
Copy Link

firstResultElement.isDisplayed()
то что элемент найден не означает что содержание результатов поиска корректно

`firstResultElement.isDisplayed()` то что элемент найден не означает что содержание результатов поиска корректно
antonskalkin73 commented 2018-05-13 18:28:53 +04:00 (Migrated from gitlab.com)
Copy Link

changed this line in version 6 of the diff

changed this line in [version 6 of the diff](https://gitlab.com/romanov73/tis-2017/merge_requests/5/diffs?diff_id=14430512&start_sha=224452000a0d227bbe99f019e81764281d996858#2870cd23eb9139c538352df2320ce7da559c6122_53_53)
antonskalkin73 commented 2018-05-13 18:28:53 +04:00 (Migrated from gitlab.com)
Copy Link

added 1 commit

  • 76e1d986 - Улучшено тестирование поиска. Теперь проверяет элементы на странице. Версия 2.0

Compare with previous version

added 1 commit <ul><li>76e1d986 - Улучшено тестирование поиска. Теперь проверяет элементы на странице. Версия 2.0</li></ul> [Compare with previous version](https://gitlab.com/romanov73/tis-2017/merge_requests/5/diffs?diff_id=14430512&start_sha=224452000a0d227bbe99f019e81764281d996858)
antonskalkin73 commented 2018-05-13 18:40:43 +04:00 (Migrated from gitlab.com)
Copy Link

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

Данное название являлось не совсем корректным и было переписано. Также были переделаны проверки поиска на перебор всех значений поиска. В некоторых тест-кейсах такой тип поиска не представляется возможным, поэтому я проверяю нахождение заранее известного элемента, который обязан быть на этой странице
romanov73 commented 2018-05-14 09:17:30 +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#6
No description provided.
Delete Branch "Skalkin"

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?