Not a member of Pastebin yet?
Sign Up,
it unlocks many cool features!
- public static void assertThat(ExpectedCondition condition) {
- new WebDriverWait(getDriver(), 4).until(condition);
- }
- /*
- ну уже б Configuration.timeout применила)
- или просто параметром вынесла
- */
- *****************************
- http://joxi.ru/vAW36KgskbnezA
- /*
- зверинец прибивай)
- */
- *******************************
- public class MinimumSizeOf extends SizeOf {
- /*
- из имен обоих классов убирай Of
- писала в прошлый раз - почему стоит так сделать
- */
- *************************
- public abstract class AbstractLazyElement implements LazyElement {
- ...
- public LazyElement lazyElement;
- ...
- @Override
- public boolean is(Condition<WebElement> condition) {
- try {
- return condition.check(lazyElement.getWrappedEntity());
- } catch (WebDriverException e) {
- return false;
- }
- }
- /*
- напоминаю - мы в контексте лейзи элемента
- this - это тот лейзи-элемент, у которого мы вызываем этот метод is
- проверять при этом выполнение кондишена не для this, а для чего-то не инициализированного lazyElement
- не стоит
- не нужно тебе такое поле lazyElement
- и в этом методе используй this
- */
- ***********************************
- public class LazyWebDriverCollection extends AbstractLazyCollection {
- private static By locator;
- public class LazyWebDriverCollection extends AbstractLazyCollection {
- private static By locator;
- /*
- оу
- какая ошибка)
- раз поле locator статическое - то оно одно на весь класс
- не для каждого объекта - свое значение
- а для класса - одно на всех
- так можно таких феерических глюков наловить)))
- делай поле статическим - если тебе что-то единственное нужно
- у нас получилось
- не - у каждого человека есть рука
- а - у человечества есть рука. Одна на всех
- нам так точно не надо)
- проверь и остальные классы
- */
- *********************************
- public class SizeOf extends CollectionCondition {
- ...
- public boolean checkListSize() {
- return listSize == expectedSize;
- }
- @Override
- public boolean check(List<WebElement> elements) {
- listSize = elements.size();
- //return listSize == expectedSize;
- return checkListSize();
- }
- /*
- ну... много ты тут не наэкономишь)
- ради одной строчки - я б не стала checkListSize() реализовывать
- для texts & exactTexts - оно оправдано
- а тут - когда сам check в 2 строчки ...
- сомнительная польза
- */
Advertisement
Add Comment
Please, Sign In to add comment