Not a member of Pastebin yet?
Sign Up,
it unlocks many cool features!
- public interface DescribesEntity<T> {
- String identity();
- }
- /*
- лучше бы это разместить в пекедже wrappers - т к это ведь относится к нашим лейзи-сущностям
- */
- *************************
- /*
- в конце работы - не забудь пройтись по коду
- реформатировать
- пропуски строк подправить
- рабочие комменты убрать
- ну и проверить - что по-прежнему все ок работает
- */
- ***************************
- CollectionCondition и ElementCondition перемести на уровень вверх
- в пекедж conditions
- цель - то, что нужно для внешней стороны дела - выносим на уровень повыше
- мы же в итоге - именно этими типами оперируем
- ****************************************
- WithWaitFor и Command - перенеси в core
- к wrappers это не имеет отношения
- а городить им отдельные пекеджи - тоже смысла нет
- ***********************************
- public <X> X getScreenshotAs(OutputType<X> outputType)
- /*
- это тоже можно переписать с использованием withWaitFor
- оперируй командой типа Command<X> тебе же нужен результат типа X
- */
- ***************************************
- public class LazyCollectionNthElement extends AbstractLazyElement {
- ....
- @Override
- public String toString() {
- return "parentCollection[" + index + "] = " + getWrappedEntity();
- }
- /*
- ненене
- в toString() - уже никаких getWrappedEntity()
- тут - описываем - как искали
- и все
- parentCollection.toString() + "[" + index + "]"
- что нашли, а тем более заново искать это - не описываем)
- только - как искали
- */
- **********************************
- public void delete(String taskText) {
- tasks.find(exactText(taskText)).hover();
- tasks.find(exactText(taskText)).find(".destroy").click();
- }
- /*
- снова сможешь написать это в одну строчку - как в селениде
- */
- **********************************
- public static List<String> getTexts(List<WebElement> elements)
- /*
- этот метод из Helpers
- можно применить внутри AbstractLazyCollection#getTexts()
- */
- ****************************
- public abstract class AbstractCondition<T> implements Condition<T>, DescribesResult {
- public LazyEntity<T> entity;
- public T apply(LazyEntity<T> entity) {
- /*
- вот тут нужно
- this.entity = entity;
- т к ниже - в toString - мы работаем с полем этого объекта
- entity - а его не заполнили
- и у нас null pointer exception
- */
- /*
- ты писала про - но есть проблема с тестом testSearchAndFollowLink.
- у меня больше тестов падало, и все - с NPE )
- вот этот момент поправила - и стало ок
- */
Advertisement
Add Comment
Please, Sign In to add comment