julia_v_iluhina

Untitled

Nov 28th, 2016
73
0
Never
Not a member of Pastebin yet? Sign Up, it unlocks many cool features!
Java 4.30 KB | None | 0 0
  1. static ElementsCollection mails = $$(".aDP tbody tr")
  2.  
  3. /*
  4.     http://joxi.ru/zANDEYxulbdMRm
  5.     твой вариант селектора - находит не один мейл в режиме поиска
  6.     а гораздо больше
  7.  
  8.     в прошлом ревью были иллюстрации, очень подсказывающие  - как селектор для списка мейлов получить
  9.  
  10.     проверяй его и в таком режиме
  11. */
  12. ***************************************************
  13.     public  static void assertMailInInbox(int index, String mailHeaderText){
  14.     public static void assertMailInSent(int index, String mailHeaderText) {
  15. /*
  16.     посмотри на реализацию этих методов
  17.     есть хоть какая-то разница?
  18.  
  19.     а если в InBox - применить assertMailInSent?
  20.  
  21.     вопрос - все ли ок с названиями методов?
  22.     вопрос - а нужно ли 2 метода?
  23.     перечитай про это прошлое ревью
  24.     строки 91-94
  25. */
  26. **************************************************
  27. public static void searchMail(String subject) {
  28. /*
  29.     если параметр subject - то поиск должен идти четко по subject
  30.     для этого - в строке поиска нужен запрос subject:....
  31.  
  32.     а лучше и универсальнее искать некий queryText - что соответствует твоей реализации
  33.     какое выражение передадут - так и ищем
  34. */
  35. ************************************************
  36.     public static void assertMailInSearchList(int mailsCount, String mailSubjectTexts) {
  37.             $$(mails).shouldHave(texts(mailSubjectTexts)).shouldHaveSize(mailsCount).shouldHave(texts(mailSubjectTexts));
  38.     }
  39. /*
  40.     странное имя, странный набор параметров, и странная реализация )
  41.  
  42.     буду мучать вопросами
  43.     первое
  44.     вспомни todoMVC
  45.     как ты проверял -
  46.     в списке тасок 2 таски и их тексты = "a", "b"
  47.     в списке тасок 1 таска и ее текст  = "a"
  48.     ?
  49.     нужны были для этих ситуаций разные методы?
  50.     нужно было передавать ожидаемое количество тасок или достаточно было собственно текстов тасок ?
  51.  
  52.     второе
  53.     что мы проверяем тут
  54.     мы проверяем - что в списке мейлов = столько-то мейлов и их текст содержит то-то
  55.     вопрос - мы эту проверку можем применить только для отфильтрованного списка мейлов?
  56.     а почему мы так какую-то другую папку с письмами проверить не можем?
  57.     ведь селектор для списка мейлов - тот же самый
  58.  
  59.     $$(mails) или mails
  60.     что есть mails?
  61.     нужно нам еще раз его передавать в метод $$ ?
  62.  
  63.     имя параметра mailSubjectTexts - текстЫ
  64.     а тип String - подразумевает получение нескольких текстОВ?
  65.     какой ты для этих целей тип использовал?
  66.  
  67.     что делает кондишен  texts при проверке?
  68.     посмотри прошлое ревью
  69.     зажми ctrl+кликни на texts
  70.     посмотри - как реализован кондишен, что он проверяет
  71.  
  72.     зачем в одной строке - дважды проверять shouldHave(texts(mailSubjectTexts)) ?
  73.     и нужно ли в дополнение к проверке shouldHave(texts(mailSubjectTexts)) делать проверку shouldHaveSize(mailsCount)?
  74.  
  75.     часть проблем связана с тем, что селектор для mails - подобран неточно
  76.     в самом начале ревью написала про это
  77. */
  78. ***************************************************
Advertisement
Add Comment
Please, Sign In to add comment