julia_v_iluhina

Untitled

Sep 9th, 2016
78
0
Never
Not a member of Pastebin yet? Sign Up, it unlocks many cool features!
Java 4.21 KB | None | 0 0
  1. public class Configuration {
  2.  
  3.     public static int timeout = 10;
  4. }
  5.  
  6. /*
  7.     правильнее тут выставить 4 секунды
  8.  
  9.     не всегда нужен такой большой таймаут - 10, 20 секунд
  10.  
  11.     пусть по умолчанию - будет более востребованное значение
  12.     (не зря в Selenide таймаут по умолчанию - 4 секунды - часто его достаточно)
  13.  
  14.     а уже в самом тест-методе - переназначишь Configuration.timeout
  15.  
  16.     посмотри - как ты это делала в этом же задании - когда на Selenide это реализовывала
  17. */
  18. ****************************
  19. public abstract class ConsizeAPI {
  20. /*
  21.     поискала разные написания concise
  22.  
  23.     consize - не вижу такого варианта
  24.  
  25.     кинь мне линку на почитать - если есть у тебя полезный для проверки написаний
  26.  
  27.     я проверила во всех своих словариках и погуглила
  28.     не вижу варианта consize )
  29.  
  30.     допускаю, что чего-то не знаю )
  31. */
  32. ************************************
  33.             actualTexts = new ArrayList<String>();
  34.             actualTexts = getTexts(elements);
  35. /*
  36.     в getTexts - есть и инициализация списка, и его заполнение
  37.     в коде кондишена - достаточно actualTexts = getTexts(elements);
  38. */
  39. ***********************************************
  40.     public static ExpectedCondition<List<WebElement>> text(final int indexOfVerifiedElement, final By elementLocator, final String expectedText) {
  41. /*
  42.     про имя кондишена - писала в прошлый раз
  43.     лучше -  listNthElementHasText
  44.  
  45.     порядок и имена параметров
  46.     final int indexOfVerifiedElement,
  47.         будет достаточно index - имя кондишена будет подробным - будет достаточно
  48.         параметр должен быть вторым
  49.  
  50.         все же надо бы идти от главного к второстепенному
  51.         сначала - что проверяем - локатор / лейзи-прокси элемент/список
  52.         потом - уточнения
  53.  
  54.     final By elementLocator,
  55.         будет правильнее - elementsLocator
  56.         мы же получаем список элементов
  57.         и в нем ищем подходящий элемент
  58.         это  - локатор списка = elementsLocator
  59.  
  60.         параметр должен быть первым
  61.  
  62.     final String expectedText
  63.         тут все ок - и название, и то, что это последний параметр
  64.  
  65. */
  66. **************************************
  67.  
  68. /*
  69.     см прошлое ревью - строки 52-69
  70.     это надо сделать
  71. */
  72. *****************************************
  73.  
  74.     @BeforeClass
  75.     public static void driverOpen() {
  76.         driver.get("http://gmail.com/");
  77.     }
  78.  
  79. /*
  80.     открывать урл - единожды перед запуском всех тестов - перебор...
  81.  
  82.     лучше бы - пока у тебя один тест-метод
  83.     это перенести в начало самого тест-метода
  84.  
  85.     а когда поймешь - что во всех тест-методах нужно открыть этот урл
  86.     то тогда такой код разумно вынести в @Before-метод
  87.  
  88.     но - однозначно - не надо располагать код такой в предке тест-класса
  89.     т к - остальное в BaseTest - универсально подходит в качестве предков для любіх тест-классов
  90.     вот и стоит его оставить универсальным
  91.  
  92.     для driver.get(...) - в ConciseAPI организуй метод open(...) и его уже используй
  93. */
Advertisement
Add Comment
Please, Sign In to add comment