julia_v_iluhina

Untitled

Sep 9th, 2016
78
0
Never
Not a member of Pastebin yet? Sign Up, it unlocks many cool features!
Java 2.79 KB | None | 0 0
  1. core.Configuration.timeout или Configuration.timeout
  2. /*
  3.     Configuration.timeout - достаточно подробно
  4. */
  5. **************************
  6. public class Configuration {
  7.  
  8.     public static int timeout = 15;
  9. }
  10. /*
  11.     правильнее тут выставить 4 секунды
  12.  
  13.     не всегда нужен такой большой таймаут - 15 секунд
  14.  
  15.     пусть по умолчанию - будет более востребованное значение
  16.     (не зря в Selenide таймаут по умолчанию - 4 секунды - часто его достаточно)
  17.  
  18.     а уже в самом тест-методе - переназначишь Configuration.timeout
  19.  
  20.     посмотри - как ты это делал в этом же задании - когда на Selenide это реализовывала
  21. */
  22. ***********************************
  23.  return Boolean.valueOf(actualText.contains(expectedText));
  24.  
  25.  /*
  26.     тут можно проще
  27.  
  28.     return actualText.contains(expectedText);
  29.  */
  30. ********************************************
  31. public String toString() {
  32.     return String.format("text (\'%s\') isn't present in element. Actual text is '%s'", expectedText, actualText);
  33. }
  34. ...
  35. public String toString() {
  36.     return String.format("Text in element isn't contains actual text '%s'. " +
  37.         "Expected text: '%s'", elementsTexts.toArray().toString(), expectedTexts.toString());
  38. }
  39. /*
  40.      во всех кондишенах
  41.         стоит строить фразу
  42.  
  43.         element of list ...
  44.         т е  - указать  - про что речь - что проверяем (что тоже важно)
  45.  
  46.         should ...
  47.         описываем expected
  48.  
  49.         while actual ...
  50.         описываем actual
  51.  
  52.      не хватает описания проверяемой сущности
  53.  
  54.      ну и если совсем занудствовать - то корректнее  - не утверждать - что проверка не выполнена
  55.      а описать - проверяли тото, ждали тото, получили тото
  56.  
  57.      так корректнее - т к строка, возвращаемая toString() - описывает объект
  58.  
  59. */
  60. ************************************
  61.  
  62. /*
  63.     тут этого не нужно
  64.     но в общем - полезнее реализовать кондишены как
  65.  
  66.     ExpectedCondition<WebElement> listNthElementHasText(final List<WebElement> elements, final int index, final String expectedText)
  67.     ExpectedCondition<List<WebElement>> texts(final List<WebElement> elements, final String... expectedTexts)
  68.  
  69.     советую попробовать)
  70.     настаивать не буду
  71. */
Advertisement
Add Comment
Please, Sign In to add comment