Not a member of Pastebin yet?
Sign Up,
it unlocks many cool features!
- private void complete(String taskText) {
- tasks.filterBy(cssClass("active")).findBy(exactText(taskText)).$(".toggle").click();
- }
- private void activate(String taskText){
- tasks.filterBy(cssClass("completed")).findBy(exactText(taskText)).$(".toggle").click();
- }
- /*
- посмотри - код этих методов очень похож
- и уже - не сказать, что он очень прост
- потому - чтобы код был более DRY лучше резализовать метод
- toggle(String taskClass, String taskText)
- и уже - переиспользовать этот метод внутри complete и activate
- впоследствии - когда мы вынесем из тест-класса вспомогательные методы в другой класс
- такой метод - toggle(String taskClass, String taskText)
- можно будет объявить как private
- чтобы он не был видим со стороны тест-класса
- такой - внутренний вспомогательный метод, чтоб код более DRY сделать
- */
- ************************************************
- private void completeAll() {
- $("#toggle-all").click();
- }
- /*
- А вот тут уже вряд ли такой номер пройдет
- вызови такой метод - для всех закомпличеных тасок
- и выполнится не complete all
- а - activate all
- следовательно - имя метода не отражает в точности то, что он делает
- при такой реализации метода - это вариант переключить, а не закомптитить/активировать
- так что - в данном случае - имя метода = toggleAll
- */
- ******************************************************
- private SelenideElement setNewTaskText(String oldTaskText, String newTaskText){
- /*
- и твое имя метода - точное и использует уже применяемые термины
- значит - хорошее)
- можно чуть лаконичнее - startEdit
- в данном случае - меньше подробностей, зато есть отсыл - какую операцию начали делать
- на изменениях не настаиваю
- */
Advertisement
Add Comment
Please, Sign In to add comment