Advertisement
Guest User

Untitled

a guest
Feb 20th, 2017
65
0
Never
Not a member of Pastebin yet? Sign Up, it unlocks many cool features!
text 3.72 KB | None | 0 0
  1. loginPage.assertUserLoggedIn();
  2.  
  3. этот шаг ИМХо не нужен
  4. зачем он? мы ведь конкретно логин не тестируем
  5.  
  6. более того имя метода говорит что какого то юзера мы проверяем... что типа он залогинен. .. но что то я не вижу что бы ты указывал какого...
  7. если уж проверять то проверять...
  8.  
  9.  
  10.  
  11. но если сильно хочешь можешь оставить этот... мне уже лень возится:-) это типа ошибка но не сильно большая...
  12.  
  13. ***
  14. loginPage.openGmail();
  15. loginPage.signIn(TestData.login, TestData.password);
  16. loginPage.assertUserLoggedIn();
  17. gmailPage.writeAndSendNewEmail(emailSubject, contactEmail);
  18. gmailPage.assertEmailSentMessage();
  19. gmailPage.goToInbox();
  20.  
  21. последний шаг выглядит тупо... ты же уже в инбоксе. ..
  22.  
  23. ты что, не нашёл кнопки рефреш в жмейле?
  24.  
  25.  
  26. ***
  27. gmailPage.verifyThatEmailHasArrived(emailSubject);
  28.  
  29. не используй никакие слова для ассертов кроме assert should expect
  30. потому что эти слова все знают и они сразу бросаются в глаза
  31.  
  32. verify, check - не стоит использовать они реже встречаются
  33.  
  34. и если используешь одно слово какое то то используй его везде.
  35.  
  36. для консистентности
  37.  
  38. ***
  39.  
  40. gmailPage.searchForEmailBySubject(emailSubject);
  41. gmailPage.verifyResultsSize(1);
  42. gmailPage.goToSentEmailFolder();
  43. gmailPage.assertFirstEmailWithThisSubject(emailSubject);
  44. }
  45.  
  46. разве в сценарии эти шаги шли в таком порядке?
  47.  
  48. ***
  49.  
  50. gmailPage.verifyResultsSize(1);
  51.  
  52. разве этой проверки достаточно?
  53.  
  54. ты же тестировщик типа как:-)
  55.  
  56. ***
  57. запусти свой тест два раза подряд, ничего не делая вручную между запусками. разве второй раз он пройдет?
  58.  
  59. ***
  60.  
  61. public class LoginPage {
  62.  
  63. public void openGmail() {
  64. open("http://gmail.com");
  65. }
  66.  
  67. public void signIn(String login, String password) {
  68. loginAndPasswordSet(login, password);
  69. $(byText("COMPOSE")).shouldBe(visible);
  70. }
  71.  
  72. public void loginAndPasswordSet(String login, String password) {
  73. $("#Email").setValue(login).pressEnter();
  74. $("#Passwd").setValue(password).pressEnter();
  75. }
  76.  
  77. public void assertUserLoggedIn(){
  78. $("#gb_71").find(byText("Sign out")).shouldBe(hidden);
  79. }
  80.  
  81. }
  82.  
  83. что тут делает метод signin с ассертом ? я тебе что говорил раньше?
  84.  
  85. assertUserLoggedIn
  86.  
  87. а этот метод что тут делает? разве он отображает работу со страницей Login?
  88.  
  89. понимаешь что значит "сортировать вещи по нужным ящикам"?
  90. OOP в том числе и об этом.
  91.  
  92. ***
  93. public void assertFirstEmailWithThisSubject(String emailSubject) {
  94. $$("[role='main'] .zA .y6").first().shouldHave(text(emailSubject));
  95. }
  96.  
  97. public void verifyResultsSize(int expectedSize) {
  98. $$("[role='main'] .zA").shouldHave(size(expectedSize));
  99. }
  100.  
  101. обрати внимание на
  102. ("[role='main'] .zA
  103.  
  104. вот это называется duplication
  105.  
  106. о правиле DRY уже забыл?
  107.  
  108. в фак об этом есть.
Advertisement
Add Comment
Please, Sign In to add comment
Advertisement