julia_v_iluhina

Untitled

Nov 1st, 2016
78
0
Never
Not a member of Pastebin yet? Sign Up, it unlocks many cool features!
Java 2.78 KB | None | 0 0
  1.  public static Actions action(){
  2.         return new Actions(getDriver());
  3.  }
  4. /*
  5.     Метод корректнее назвать actions()
  6.     мы же создаем объект типа Actions
  7. */
  8. **********************
  9. public static WebElement $(ExpectedCondition<WebElement> conditionToWaitParentElement){
  10. /*
  11.     я бы этот метод первым из методов $ привела
  12.     и в $(By locator) - использовала уже его
  13.  
  14.     имя параметра conditionToWaitParentElement - упрости
  15.     про аналогичное писала в прошлом ревью, строки 15-22
  16. */
  17. ******************************
  18. public String toString() {
  19.     return String.format(index + " element in list should have text: " + expectedText + ". While actual text is " + elementText);
  20. }
  21. /*
  22.     не забывай и локатор выводить
  23.     да и я бы разделяла сообщение на несколько строк
  24.  
  25.         что мы проверяем
  26.         для какого элемента/элементов мы проверяем
  27.         что ожидаем
  28.         что есть по факту
  29.  
  30.         тут было бы ок
  31.         for list found by locator ...
  32.         text of list element [index]
  33.         should be :
  34.         while actual text is:
  35.  
  36.         просмотри и другие сообщения
  37. */
  38. ***************************
  39. actualTexts.clear();
  40. actualTexts.addAll(Helpers.getTexts(actualElements));
  41.  
  42. /*
  43.     поскольку actualTexts - не final-переменная теперь
  44.     то можно проще
  45.     actualTexts = getTexts(actualElements);
  46.  
  47.     здесь и далее - используй import static
  48.     для методов класса Helpers
  49.     и так будет достаточно понятно и точно
  50.  
  51.     ну и усти по поводу работы со списками
  52.     и другие места есть с аналогичным кодом
  53. */
  54. ******************************************
  55. public WebElement apply(WebDriver driver) {
  56.     List<WebElement> webElementList = ConciseAPI.getDriver().findElements(elementsLocator);
  57. /*
  58.     не нужно тебе в методе кондишена apply
  59.     использовать вебдрайвер из ConciseAPI
  60.     т к в apply - вебдрайвер передан как параметр
  61. */
  62. **********************************
  63. public class TodoMvcPage
  64. /*
  65.     пейдж можно реализовать как пейдж-модуль
  66.  
  67.     это на твое усмотрение
  68. */
  69. ************************************
  70. $(byCss("#toggle-all"))
  71. //можно проще
  72. $("#toggle-all")
  73.  
  74. //и для $(byCss("#clear-completed")) - аналогично
Advertisement
Add Comment
Please, Sign In to add comment