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

Kyrnaev #9

Closed
Lisyra wants to merge 7 commits from Kyrnaev into master
pull from: Kyrnaev
merge into: romanov73:master
Conversation 32 Commits 7 Files Changed 19 +1240
Lisyra commented 2018-09-20 16:02:38 +04:00 (Migrated from gitlab.com)
Copy Link
No description provided.
romanov73 commented 2018-09-20 16:07:37 +04:00 (Migrated from gitlab.com)
Copy Link

Если класс не используется он должет быть удален

Если класс не используется он должет быть удален
romanov73 commented 2018-09-20 16:08:40 +04:00 (Migrated from gitlab.com)
Copy Link

В тесте должна быть проверка. Почему она закомменчена?

В тесте должна быть проверка. Почему она закомменчена?
romanov73 commented 2018-09-20 16:09:50 +04:00 (Migrated from gitlab.com)
Copy Link

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

Комментированный код должен быть удален. На то мы и используем репозиторий, чтобы версионность сохранялась в нем. Возможность отката также будет. Комментированный код только засоряет
romanov73 commented 2018-09-20 16:10:21 +04:00 (Migrated from gitlab.com)
Copy Link

по Java Code Conversions название метода с маленькой буквы

по Java Code Conversions название метода с маленькой буквы
romanov73 commented 2018-09-20 16:12:47 +04:00 (Migrated from gitlab.com)
Copy Link

Это конфигурирование уже выполнили (см. пример) В тесте не должно быть ничего не относящегося к выполнению тест-кейсов. Содержание тестов = переведенные в код шаги тест-кейсов из документации. Все остальные технические детали скрываются либо во вспомогательные классы (хелперы) либо в один из классов по методологии POM

Это конфигурирование уже выполнили (см. пример) В тесте не должно быть ничего не относящегося к выполнению тест-кейсов. Содержание тестов = переведенные в код шаги тест-кейсов из документации. Все остальные технические детали скрываются либо во вспомогательные классы (хелперы) либо в один из классов по методологии POM
romanov73 commented 2018-09-20 16:13:19 +04:00 (Migrated from gitlab.com)
Copy Link

Эта строка так же не должна здесь быть. На результат тестов локаторы никак не влияют

Эта строка так же не должна здесь быть. На результат тестов локаторы никак не влияют
romanov73 commented 2018-09-20 16:14:35 +04:00 (Migrated from gitlab.com)
Copy Link

Тесты запускаются из тестового окружения, main - главная точка входа в программу.

Тесты запускаются из тестового окружения, main - главная точка входа в программу.
romanov73 commented 2018-09-20 16:15:14 +04:00 (Migrated from gitlab.com)
Copy Link

Тесты, разработанные здесь выполняются все и сразу. Иначе смысла в них нет

Тесты, разработанные здесь выполняются все и сразу. Иначе смысла в них нет
romanov73 commented 2018-09-20 16:17:08 +04:00 (Migrated from gitlab.com)
Copy Link

метод длиной больше экрана должен быть разбит

метод длиной больше экрана должен быть разбит
romanov73 commented 2018-09-20 16:17:26 +04:00 (Migrated from gitlab.com)
Copy Link

название методов не говорит о содержании

название методов не говорит о содержании
Lisyra commented 2018-09-22 00:21:51 +04:00 (Migrated from gitlab.com)
Copy Link

извините здравствйте я немного не понял

извините здравствйте я немного не понял
Lisyra commented 2018-09-22 00:24:05 +04:00 (Migrated from gitlab.com)
Copy Link

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

мне получается убрать меню и просто чтобы при запуске выполнялись все тесты подряд?
Lisyra commented 2018-09-22 00:54:19 +04:00 (Migrated from gitlab.com)
Copy Link

Прошу уточнить: просто всё лишнее из тестов (кроме ожидания) перенести в другой класс, а переменные сделать глобальными, чтобы их не было в тесте? как то так или я не правильно понял? все же не умею этим сайтом пользоваться

Прошу уточнить: просто всё лишнее из тестов (кроме ожидания) перенести в другой класс, а переменные сделать глобальными, чтобы их не было в тесте? как то так или я не правильно понял? все же не умею этим сайтом пользоваться
romanov73 commented 2018-09-22 09:42:19 +04:00 (Migrated from gitlab.com)
Copy Link

К сайту тут никакого отношения нет. Вот эта инструкция System.setProperty("webdriver.chrome.driver", file.getAbsolutePath()); выполняется только в одном месте, в специальном классе. Далее мы просто получаем его инстанс. Глобальные переменные так же должны быть минимизированы, это плохая практика. По другим классам разносится по методологии POM

К сайту тут никакого отношения нет. Вот эта инструкция `System.setProperty("webdriver.chrome.driver", file.getAbsolutePath());` выполняется только в одном месте, в специальном классе. Далее мы просто получаем его инстанс. Глобальные переменные так же должны быть минимизированы, это плохая практика. По другим классам разносится по методологии POM
romanov73 commented 2018-09-22 09:42:45 +04:00 (Migrated from gitlab.com)
Copy Link

Строка должна быть в Page Object

Строка должна быть в Page Object
Lisyra commented 2018-09-23 17:19: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/8/diffs?diff_id=23738193&start_sha=1e5f6aa0b955bb37f16820a94e3cd275c28554d9#c5e3ea740ebeff4ed33866e12e499fa2d9a77b39_9_0)
Lisyra commented 2018-09-23 17:19: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/8/diffs?diff_id=23738193&start_sha=1e5f6aa0b955bb37f16820a94e3cd275c28554d9#bfc6552d3ccb197e05a0b95656efacf2990cb020_39_0)
Lisyra commented 2018-09-23 17:19: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/8/diffs?diff_id=23738193&start_sha=1e5f6aa0b955bb37f16820a94e3cd275c28554d9#913e813663eff5241e3c308dcd3de348eebce293_23_25)
Lisyra commented 2018-09-23 17:19: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/8/diffs?diff_id=23738193&start_sha=1e5f6aa0b955bb37f16820a94e3cd275c28554d9#94a4f26c0654a5515f3d0da2e1464020db55ff63_22_0)
Lisyra commented 2018-09-23 17:19:25 +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/8/diffs?diff_id=23738193&start_sha=1e5f6aa0b955bb37f16820a94e3cd275c28554d9#94a4f26c0654a5515f3d0da2e1464020db55ff63_33_0)
Lisyra commented 2018-09-23 17:19:25 +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/8/diffs?diff_id=23738193&start_sha=1e5f6aa0b955bb37f16820a94e3cd275c28554d9#94a4f26c0654a5515f3d0da2e1464020db55ff63_42_0)
Lisyra commented 2018-09-23 17:19:26 +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/8/diffs?diff_id=23738193&start_sha=1e5f6aa0b955bb37f16820a94e3cd275c28554d9#94a4f26c0654a5515f3d0da2e1464020db55ff63_902_0)
Lisyra commented 2018-09-23 17:19:26 +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/8/diffs?diff_id=23738193&start_sha=1e5f6aa0b955bb37f16820a94e3cd275c28554d9#94a4f26c0654a5515f3d0da2e1464020db55ff63_908_0)
Lisyra commented 2018-09-23 17:19:26 +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/8/diffs?diff_id=23738193&start_sha=1e5f6aa0b955bb37f16820a94e3cd275c28554d9#94a4f26c0654a5515f3d0da2e1464020db55ff63_909_0)
Lisyra commented 2018-09-23 17:19:27 +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/8/diffs?diff_id=23738193&start_sha=1e5f6aa0b955bb37f16820a94e3cd275c28554d9#94a4f26c0654a5515f3d0da2e1464020db55ff63_843_0)
Lisyra commented 2018-09-23 17:19:27 +04:00 (Migrated from gitlab.com)
Copy Link

added 1 commit

  • d06aeccf - заливаю 2 теста 1) тест навигации открытие страницы "Правила" 2) проверка работы…

Compare with previous version

added 1 commit <ul><li>d06aeccf - заливаю 2 теста 1) тест навигации открытие страницы &quot;Правила&quot; 2) проверка работы…</li></ul> [Compare with previous version](https://gitlab.com/romanov73/tis-2017/merge_requests/8/diffs?diff_id=23738193&start_sha=1e5f6aa0b955bb37f16820a94e3cd275c28554d9)
Lisyra commented 2018-09-24 21:17:28 +04:00 (Migrated from gitlab.com)
Copy Link

added 1 commit

Compare with previous version

added 1 commit <ul><li>ad1ff0d7 - первые 5 тестов</li></ul> [Compare with previous version](https://gitlab.com/romanov73/tis-2017/merge_requests/8/diffs?diff_id=23806785&start_sha=d06aeccf8da1ad31ad12d2d875b611528430ebd3)
Lisyra commented 2018-09-25 20:07:08 +04:00 (Migrated from gitlab.com)
Copy Link

added 1 commit

Compare with previous version

added 1 commit <ul><li>eb549dab - первые 7 тестов</li></ul> [Compare with previous version](https://gitlab.com/romanov73/tis-2017/merge_requests/8/diffs?diff_id=23885635&start_sha=ad1ff0d77d57a1eda5ea7ab6ee59453d51525bc6)
Lisyra commented 2018-09-26 22:16:56 +04:00 (Migrated from gitlab.com)
Copy Link

added 1 commit

  • 375b49ee - 11 тестов, посмотрите нормально теперь выглядит ожидание, так можно оставить?

Compare with previous version

added 1 commit <ul><li>375b49ee - 11 тестов, посмотрите нормально теперь выглядит ожидание, так можно оставить?</li></ul> [Compare with previous version](https://gitlab.com/romanov73/tis-2017/merge_requests/8/diffs?diff_id=23977918&start_sha=eb549dabf48eb392b87cb1a2c7029b8e4ebb6f53)
Lisyra commented 2018-09-28 20:26:20 +04:00 (Migrated from gitlab.com)
Copy Link

added 1 commit

Compare with previous version

added 1 commit <ul><li>964ead17 - все 20 тестов</li></ul> [Compare with previous version](https://gitlab.com/romanov73/tis-2017/merge_requests/8/diffs?diff_id=24133192&start_sha=375b49eeae78fa2cda6ba6909d6aadafe033bdce)
Lisyra commented 2018-10-01 12:55:55 +04:00 (Migrated from gitlab.com)
Copy Link

resolved all discussions

resolved all discussions
romanov73 commented 2018-10-18 20:45:14 +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#9
No description provided.
Delete Branch "Kyrnaev"

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?