Not a member of Pastebin yet?
Sign Up,
it unlocks many cool features!
- static ElementsCollection mails = $$(".aDP tbody tr")
- /*
- http://joxi.ru/zANDEYxulbdMRm
- твой вариант селектора - находит не один мейл в режиме поиска
- а гораздо больше
- в прошлом ревью были иллюстрации, очень подсказывающие - как селектор для списка мейлов получить
- проверяй его и в таком режиме
- */
- ***************************************************
- public static void assertMailInInbox(int index, String mailHeaderText){
- public static void assertMailInSent(int index, String mailHeaderText) {
- /*
- посмотри на реализацию этих методов
- есть хоть какая-то разница?
- а если в InBox - применить assertMailInSent?
- вопрос - все ли ок с названиями методов?
- вопрос - а нужно ли 2 метода?
- перечитай про это прошлое ревью
- строки 91-94
- */
- **************************************************
- public static void searchMail(String subject) {
- /*
- если параметр subject - то поиск должен идти четко по subject
- для этого - в строке поиска нужен запрос subject:....
- а лучше и универсальнее искать некий queryText - что соответствует твоей реализации
- какое выражение передадут - так и ищем
- */
- ************************************************
- public static void assertMailInSearchList(int mailsCount, String mailSubjectTexts) {
- $$(mails).shouldHave(texts(mailSubjectTexts)).shouldHaveSize(mailsCount).shouldHave(texts(mailSubjectTexts));
- }
- /*
- странное имя, странный набор параметров, и странная реализация )
- буду мучать вопросами
- первое
- вспомни todoMVC
- как ты проверял -
- в списке тасок 2 таски и их тексты = "a", "b"
- в списке тасок 1 таска и ее текст = "a"
- ?
- нужны были для этих ситуаций разные методы?
- нужно было передавать ожидаемое количество тасок или достаточно было собственно текстов тасок ?
- второе
- что мы проверяем тут
- мы проверяем - что в списке мейлов = столько-то мейлов и их текст содержит то-то
- вопрос - мы эту проверку можем применить только для отфильтрованного списка мейлов?
- а почему мы так какую-то другую папку с письмами проверить не можем?
- ведь селектор для списка мейлов - тот же самый
- $$(mails) или mails
- что есть mails?
- нужно нам еще раз его передавать в метод $$ ?
- имя параметра mailSubjectTexts - текстЫ
- а тип String - подразумевает получение нескольких текстОВ?
- какой ты для этих целей тип использовал?
- что делает кондишен texts при проверке?
- посмотри прошлое ревью
- зажми ctrl+кликни на texts
- посмотри - как реализован кондишен, что он проверяет
- зачем в одной строке - дважды проверять shouldHave(texts(mailSubjectTexts)) ?
- и нужно ли в дополнение к проверке shouldHave(texts(mailSubjectTexts)) делать проверку shouldHaveSize(mailsCount)?
- часть проблем связана с тем, что селектор для mails - подобран неточно
- в самом начале ревью написала про это
- */
- ***************************************************
Advertisement
Add Comment
Please, Sign In to add comment