Advertisement
Not a member of Pastebin yet?
Sign Up,
it unlocks many cool features!
- loginPage.assertUserLoggedIn();
- этот шаг ИМХо не нужен
- зачем он? мы ведь конкретно логин не тестируем
- более того имя метода говорит что какого то юзера мы проверяем... что типа он залогинен. .. но что то я не вижу что бы ты указывал какого...
- если уж проверять то проверять...
- но если сильно хочешь можешь оставить этот... мне уже лень возится:-) это типа ошибка но не сильно большая...
- ***
- loginPage.openGmail();
- loginPage.signIn(TestData.login, TestData.password);
- loginPage.assertUserLoggedIn();
- gmailPage.writeAndSendNewEmail(emailSubject, contactEmail);
- gmailPage.assertEmailSentMessage();
- gmailPage.goToInbox();
- последний шаг выглядит тупо... ты же уже в инбоксе. ..
- ты что, не нашёл кнопки рефреш в жмейле?
- ***
- gmailPage.verifyThatEmailHasArrived(emailSubject);
- не используй никакие слова для ассертов кроме assert should expect
- потому что эти слова все знают и они сразу бросаются в глаза
- verify, check - не стоит использовать они реже встречаются
- и если используешь одно слово какое то то используй его везде.
- для консистентности
- ***
- gmailPage.searchForEmailBySubject(emailSubject);
- gmailPage.verifyResultsSize(1);
- gmailPage.goToSentEmailFolder();
- gmailPage.assertFirstEmailWithThisSubject(emailSubject);
- }
- разве в сценарии эти шаги шли в таком порядке?
- ***
- gmailPage.verifyResultsSize(1);
- разве этой проверки достаточно?
- ты же тестировщик типа как:-)
- ***
- запусти свой тест два раза подряд, ничего не делая вручную между запусками. разве второй раз он пройдет?
- ***
- public class LoginPage {
- public void openGmail() {
- open("http://gmail.com");
- }
- public void signIn(String login, String password) {
- loginAndPasswordSet(login, password);
- $(byText("COMPOSE")).shouldBe(visible);
- }
- public void loginAndPasswordSet(String login, String password) {
- $("#Email").setValue(login).pressEnter();
- $("#Passwd").setValue(password).pressEnter();
- }
- public void assertUserLoggedIn(){
- $("#gb_71").find(byText("Sign out")).shouldBe(hidden);
- }
- }
- что тут делает метод signin с ассертом ? я тебе что говорил раньше?
- assertUserLoggedIn
- а этот метод что тут делает? разве он отображает работу со страницей Login?
- понимаешь что значит "сортировать вещи по нужным ящикам"?
- OOP в том числе и об этом.
- ***
- public void assertFirstEmailWithThisSubject(String emailSubject) {
- $$("[role='main'] .zA .y6").first().shouldHave(text(emailSubject));
- }
- public void verifyResultsSize(int expectedSize) {
- $$("[role='main'] .zA").shouldHave(size(expectedSize));
- }
- обрати внимание на
- ("[role='main'] .zA
- вот это называется duplication
- о правиле DRY уже забыл?
- в фак об этом есть.
Advertisement
Add Comment
Please, Sign In to add comment
Advertisement