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

Kochkaleva #12

Closed
Ekaterina_Kochkaleva wants to merge 5 commits from Kochkaleva into master
pull from: Kochkaleva
merge into: romanov73:master
Conversation 34 Commits 5 Files Changed 13 +792
Ekaterina_Kochkaleva commented 2018-11-16 15:03:46 +04:00 (Migrated from gitlab.com)
Copy Link
No description provided.
Ekaterina_Kochkaleva commented 2018-11-20 15:10:55 +04:00 (Migrated from gitlab.com)
Copy Link

assigned to @Ekaterina_Kochkaleva

assigned to @Ekaterina_Kochkaleva
Ekaterina_Kochkaleva commented 2018-11-20 15:11:01 +04:00 (Migrated from gitlab.com)
Copy Link

assigned to @romanov73

assigned to @romanov73
Ekaterina_Kochkaleva commented 2018-11-20 15:13:12 +04:00 (Migrated from gitlab.com)
Copy Link

changed title from {-InitialCommit-} to {+Kochkaleva+}

changed title from **{-InitialCommit-}** to **{+Kochkaleva+}**
romanov73 commented 2018-11-22 22:07:11 +04:00 (Migrated from gitlab.com)
Copy Link

драйвер уже есть в проекте, смотрите где подключается

драйвер уже есть в проекте, смотрите где подключается
romanov73 commented 2018-11-22 22:09:36 +04:00 (Migrated from gitlab.com)
Copy Link

здесь должны быть только шаги из документации и проверка.

например:

...

email.submitButtonClick();

Assert.assertTrue(email.isSuccsessAlert());

здесь должны быть только шаги из документации и проверка. например: `...` `email.submitButtonClick();` `Assert.assertTrue(email.isSuccsessAlert());`
romanov73 commented 2018-11-22 22:11:03 +04:00 (Migrated from gitlab.com)
Copy Link

если данные по TC есть их нужно указывать, тест проверет только что-то одно!

если данные по TC есть их нужно указывать, тест проверет только что-то одно!
romanov73 commented 2018-11-22 22:11:15 +04:00 (Migrated from gitlab.com)
Copy Link

где финальная проверка?

где финальная проверка?
romanov73 commented 2018-11-22 22:11:40 +04:00 (Migrated from gitlab.com)
Copy Link

тоже не понятно, чего ожидаем от теста

тоже не понятно, чего ожидаем от теста
romanov73 commented 2018-11-22 22:12:31 +04:00 (Migrated from gitlab.com)
Copy Link

зачем здесь отчеты?

зачем здесь отчеты?
romanov73 commented 2018-11-22 22:13:34 +04:00 (Migrated from gitlab.com)
Copy Link

такие дествия выносятся в классы-хелперы, в данных методах все точно должно соответствовать кейсам!

такие дествия выносятся в классы-хелперы, в данных методах все точно должно соответствовать кейсам!
romanov73 commented 2018-11-22 22:18:18 +04:00 (Migrated from gitlab.com)
Copy Link

здесь вообще не понятно что за проверка

здесь вообще не понятно что за проверка
romanov73 commented 2018-11-22 22:21:23 +04:00 (Migrated from gitlab.com)
Copy Link

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

тесты выполняются в тестовом окружении, они не должны запускаться из классов программы
romanov73 commented 2018-11-22 22:21:49 +04:00 (Migrated from gitlab.com)
Copy Link

здесь вообще ни одного теста нет

здесь вообще ни одного теста нет
Ekaterina_Kochkaleva commented 2018-11-27 13:46:44 +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/11/diffs?diff_id=28179685&start_sha=978bf399a8aa96775458c16af2463d7dc0253cf7#40d2fd0ed08784524750209726d40689708bbb62_20_20)
Ekaterina_Kochkaleva commented 2018-11-27 13:46:44 +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/11/diffs?diff_id=28179685&start_sha=978bf399a8aa96775458c16af2463d7dc0253cf7#40d2fd0ed08784524750209726d40689708bbb62_49_51)
Ekaterina_Kochkaleva commented 2018-11-27 13:46:45 +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/11/diffs?diff_id=28179685&start_sha=978bf399a8aa96775458c16af2463d7dc0253cf7#40d2fd0ed08784524750209726d40689708bbb62_58_67)
Ekaterina_Kochkaleva commented 2018-11-27 13:46:45 +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/11/diffs?diff_id=28179685&start_sha=978bf399a8aa96775458c16af2463d7dc0253cf7#568c6df027d3e18dc7c4768820a24732944bacca_23_0)
Ekaterina_Kochkaleva commented 2018-11-27 13:46:45 +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/11/diffs?diff_id=28179685&start_sha=978bf399a8aa96775458c16af2463d7dc0253cf7#30f0c856f8cb344ccd6db899dee931bacb39fb76_38_39)
Ekaterina_Kochkaleva commented 2018-11-27 13:46:46 +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/11/diffs?diff_id=28179685&start_sha=978bf399a8aa96775458c16af2463d7dc0253cf7#356a30956ee65ee11bf5a4cd9152f162905f14ab_38_42)
Ekaterina_Kochkaleva commented 2018-11-27 13:46:46 +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/11/diffs?diff_id=28179685&start_sha=978bf399a8aa96775458c16af2463d7dc0253cf7#04587f02a6b29deb2b88a1e9196012c280aab2d7_11_0)
Ekaterina_Kochkaleva commented 2018-11-27 13:46:46 +04:00 (Migrated from gitlab.com)
Copy Link

added 1 commit

Compare with previous version

added 1 commit <ul><li>e8b00cbe - Bugfix</li></ul> [Compare with previous version](https://gitlab.com/romanov73/tis-2017/merge_requests/11/diffs?diff_id=28179685&start_sha=978bf399a8aa96775458c16af2463d7dc0253cf7)
romanov73 commented 2018-11-28 15:13:29 +04:00 (Migrated from gitlab.com)
Copy Link

тесты не строятся на проверках url, нужно проверять содержимое

тесты не строятся на проверках url, нужно проверять содержимое
Ekaterina_Kochkaleva commented 2018-11-29 11:28:19 +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/11/diffs?diff_id=28348562&start_sha=e8b00cbe09ecfbbd4773d083ba133e4ee4cd9cda#40d2fd0ed08784524750209726d40689708bbb62_69_69)
Ekaterina_Kochkaleva commented 2018-11-29 11:28:19 +04:00 (Migrated from gitlab.com)
Copy Link

added 1 commit

Compare with previous version

added 1 commit <ul><li>6039438f - Edit tests</li></ul> [Compare with previous version](https://gitlab.com/romanov73/tis-2017/merge_requests/11/diffs?diff_id=28348562&start_sha=e8b00cbe09ecfbbd4773d083ba133e4ee4cd9cda)
romanov73 commented 2018-11-29 17:53:04 +04:00 (Migrated from gitlab.com)
Copy Link

На исключениях логику нельзя строить, элемент либо есть либо его нет. К чему тут исключительная ситуация?
К тому же методы внутри Page не должны содержать логику проверки (это для проверок ниже по коду, таких как сравнение в хардкодным текстом)

Прочитайте еще раз про назначение PageObject. Это интерфейс к странице!!!

На исключениях логику нельзя строить, элемент либо есть либо его нет. К чему тут исключительная ситуация? К тому же методы внутри Page не должны содержать логику проверки (это для проверок ниже по коду, таких как сравнение в хардкодным текстом) Прочитайте еще раз про назначение PageObject. Это интерфейс к странице!!!
romanov73 commented 2018-11-29 17:55:43 +04:00 (Migrated from gitlab.com)
Copy Link

Еще раз, в тестах находятся строки, которые повторяют шаги из тестовой документации (тест-кейсов)

Все другие инструкции (циклы, методы, обходы, вызовы методов) выносятся в классы-хелперы

Еще раз, в тестах находятся строки, которые повторяют шаги из тестовой документации (тест-кейсов) Все другие инструкции (циклы, методы, обходы, вызовы методов) выносятся в классы-хелперы
romanov73 commented 2018-11-29 17:57:06 +04:00 (Migrated from gitlab.com)
Copy Link

Если в коде одинаковые строки встречаются более 3-х раз это сигал что вы что-то делаете не правильно.

Не нужно в каждый метод втыкать try ... catch

Прекрасно можно обойтись и без них, просто возвращая нужный объект или данные из метода

Если в коде одинаковые строки встречаются более 3-х раз это сигал что вы что-то делаете не правильно. Не нужно в каждый метод втыкать try ... catch Прекрасно можно обойтись и без них, просто возвращая нужный объект или данные из метода
Ekaterina_Kochkaleva commented 2018-12-11 11:04:48 +04:00 (Migrated from gitlab.com)
Copy Link

changed this line in version 4 of the diff

changed this line in [version 4 of the diff](https://gitlab.com/romanov73/tis-2017/merge_requests/11/diffs?diff_id=29235095&start_sha=6039438f6e8176ccf6c6af9a11d18e536d583b96#a7776fade97dbb57589a2cd5fa86188975bfc8c7_76_71)
Ekaterina_Kochkaleva commented 2018-12-11 11:04:49 +04:00 (Migrated from gitlab.com)
Copy Link

added 1 commit

Compare with previous version

added 1 commit <ul><li>d458c53a - Edit tests2</li></ul> [Compare with previous version](https://gitlab.com/romanov73/tis-2017/merge_requests/11/diffs?diff_id=29235095&start_sha=6039438f6e8176ccf6c6af9a11d18e536d583b96)
romanov73 commented 2018-12-11 13:23:47 +04:00 (Migrated from gitlab.com)
Copy Link

Смысл проверять если у вас тест никогда не упадет?
Assert.assertTrue("сообщение", условие для проверки);

У вас в качестве второго аргумента почти всегда передается true

Смысл проверять если у вас тест никогда не упадет? `Assert.assertTrue("сообщение", условие для проверки);` У вас в качестве второго аргумента почти всегда передается true
Ekaterina_Kochkaleva commented 2018-12-12 17:54:52 +04:00 (Migrated from gitlab.com)
Copy Link

changed this line in version 5 of the diff

changed this line in [version 5 of the diff](https://gitlab.com/romanov73/tis-2017/merge_requests/11/diffs?diff_id=29378716&start_sha=d458c53a175af37ef7776ce41034abc76960e4a9#40d2fd0ed08784524750209726d40689708bbb62_38_38)
Ekaterina_Kochkaleva commented 2018-12-12 17:54:52 +04:00 (Migrated from gitlab.com)
Copy Link

added 1 commit

Compare with previous version

added 1 commit <ul><li>50dac2dd - Edit tests3</li></ul> [Compare with previous version](https://gitlab.com/romanov73/tis-2017/merge_requests/11/diffs?diff_id=29378716&start_sha=d458c53a175af37ef7776ce41034abc76960e4a9)
romanov73 commented 2018-12-13 16:07:27 +04:00 (Migrated from gitlab.com)
Copy Link

resolved all discussions

resolved all discussions
romanov73 commented 2018-12-13 16:08:33 +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#12
No description provided.
Delete Branch "Kochkaleva"

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?