julia_v_iluhina

Untitled

Dec 5th, 2016
89
0
Never
Not a member of Pastebin yet? Sign Up, it unlocks many cool features!
Java 2.39 KB | None | 0 0
  1. public class ExactTexts extends Texts {
  2.  
  3.     public ExactTexts(String... expectedTexts) {
  4.         this.expectedTexts = Arrays.asList(expectedTexts);
  5.     }
  6. /*
  7.     вызывай super(...)
  8. */
  9. *************************
  10. public class Configuration {
  11.     public static long timeout = 4000;
  12.     public static long timeoutInSeconds = timeout / 1000;
  13. /*
  14.     не, вариант не очень хороший
  15.  
  16.     пример - поменяли таймаут в миллисекундах
  17.     и не поменяли в секундах
  18.  
  19.     а по идее - это одна величина должна быть
  20.  
  21.     да и просто - смотрим на настройки и не со старту можем догадаться - что зачем нужно
  22.  
  23.     так что - лучше оставь только timeout
  24.     а в new WebDriverWait - передавай timeout/1000
  25.  
  26.     давай еще и комментариев добавим тут
  27.     что-то типа такого
  28. */
  29. public class Configuration {
  30.     //default interval for WaitFor.Until - smart waiting check one condition, in milliseconds
  31.     public static int timeout = 4000;
  32.     //interval between checks inside WaitFor.Until, in milliseconds
  33.     public static int pollingInterval = 100;
  34. }
  35. **************************
  36. public static void prepareTasks(Task... tasks) {
  37.         ensureAppOpened();
  38.    ...
  39.         executeScript(jsCode.concat("]\")"));
  40.         refresh();
  41.         $$("#todo-list>li").shouldHave(size(tasks.length));
  42. }
  43. /*
  44.     что-то я отстаю  с материалом по ожиданиям загрузки джаваскрипта
  45.  
  46.     давай уже сейчас улучшим текущую версию
  47.  
  48.     пусть prepareTasks - будет prepareGivenJavaScript, к примеру
  49.     и пусть метод только готовит и возвращает строку
  50.  
  51.     а       ensureAppOpened();
  52.             и
  53.             executeScript(jsCode.concat("]\")"));
  54.             refresh();
  55.             $$("#todo-list>li").shouldHave(size(tasks.length));
  56.     уже делай в пейдже - в prepareTasks, к примеру
  57.     или в гивен-методе, уже как решишь
  58.  
  59.     цель - избавиться от дублирования селектора для $$("#todo-list>li")
  60.  
  61.     неаккуратненько)
  62. */
Advertisement
Add Comment
Please, Sign In to add comment