Not a member of Pastebin yet?
Sign Up,
it unlocks many cool features!
- /*
- встречается не реформатированный код
- просмотри и подправь это
- */
- public static void searchBySubject(String subject) {
- /*
- да, мы в метод передаем subject, и даже заявляем - searchBySubject
- если мы так говорим - так надо это и делать
- надо в тексте запроса для поиска писать строку subject: ...и тут текст из переданного subject....
- или - при той реализации как сейчас - название метода и параметра надо подправить до
- search(String queryText)
- когда ты даешь имя методу или параметру метода
- надо перестать думать - о контексте вызова метода или о том - что за данные ты передаешь
- и надо начать думать о том - что делает метод и какими данными он оперирует
- */
- **************************
- public static void assertMails(String... emailSubjects) {
- public static void assertMail(int n, String subject) {
- /*
- даже если бы мы на уровне методов и правда рабтали именно с subject-ами
- то хорошо бы - чтоб одинаковый термин был использован для параметров - subjects, subject
- но - на уровне методов - мы лишь проверяем вхождение нужного нам текста в текст заголовка мейла,
- мы ведь в проверке - не уточняемся до элемента-subject-а и не проверяем именно его
- потому - правильнее параметр метод называть не subject, а text/texts or mailText/mailTexts or mailHeaderText/mailHeaderTexts
- да, передадим мы subject или subject-ы
- но - технически - такой ассерт можно не только для проверки subject-ов писем использовать
- но и для поверки любых кусков текста для мейлов
- насчет параметра int n
- традиционно для номера в коллекции - используют термин index
- тут - та же история - мы обращаемся к мейлу из коллекции мейлов - по его индексу - лучше используй термин index
- это большему количеству людей будет со старта понятно
- это уже для общего развития в этом вопросе
- https://en.wikipedia.org/wiki/Zero-based_numbering
- http://c2.com/cgi/wiki?ZeroAndOneBasedIndexes
- http://stackoverflow.com/questions/24841172/why-is-array-indexing-in-java-start-w
- */
- ***********************************
- $(byText("Inbox")).click();
- /*
- по тексту - можно много другого наловить
- http://joxi.ru/n2YkKaGUjY8Y9r
- тут надо по-другому это делать
- или оттолкнуться от элемента-контейнера для вариантов папок http://joxi.ru/GrqLOX3SN17NzA
- и затем уже уточняться до Inbox
- или - постороить css Selector - title начинается с ...
- http://joxi.ru/5md7jYwtvx7v9r
- */
- *********************************
- System.out.println();
- /*
- уже не нужно
- */
Advertisement
Add Comment
Please, Sign In to add comment