Not a member of Pastebin yet?
Sign Up,
it unlocks many cool features!
- @Step
- public NewEmail withRecipients(String... recipients) {
- for (String recipient : recipients) {
- $$(".GS .eV").filter(Condition.visible).first().find(".oj .vO")
- .setValue(recipient)
- .pressEnter();
- } return this;
- }
- ретурн на новую строку лучше же перенести ) я так понимаю очепятка
- ***
- public class LoginPage {
- @Step
- public void ensureLoggedAs(String email, String password) {
- $("#Email").setValue(email).pressEnter();
- $("#Passwd").setValue(password).pressEnter();
- $(".cmsg").waitUntil(hidden, 15_000);
- }
- }
- поднимая снова тему о "эншурности"
- я бы тут убрал еншуре вместе с вейтом...
- а еншуре версию метода встроил бы таки в Application
- и там бы и добавил еще и проверку не залогинены ли мы уже, ведь если да, то незачем и логин делать еще раз...
- у тебя кстати этой проверку тут и нет:
- public void ensureLoggedAs(String email, String password) {
- new LoginPage().ensureLoggedAs(email, password);
- }
- а это уже не сильно хорошо... потому что он на то и еншуре - что должен все быть "приводить к цели как можно более эффективно..."
- ***
- плюс почему не:
- public void ensureLoggedAs(String email, String password) {
- this.atLoginPage().ensureLoggedAs(email, password);
- }
- ?
- (ну this писать/не писать - то уже такое, дело вкуса)
- ***
- @Step
- public Emails emails() {
- return Emails.all();
- }
- @Step
- public Emails unreadedEmails() {
- return Emails.unreaded();
- }
- @Step
- public Emails readedEmails() {
- return Emails.readed();
- }
- все ок, но просто как идея еще одного варианта апи:
- просто
- @Step
- public Emails emails() {
- return new Emails();
- }
- но потом можно:
- emails().unreaded()
- emails().readed()
- ну эт такое... я еще сам не уверен что так было бы лучше.... нужно пробовать...
Advertisement
Add Comment
Please, Sign In to add comment