julia_v_iluhina

Untitled

Nov 24th, 2016
82
0
Never
Not a member of Pastebin yet? Sign Up, it unlocks many cool features!
Java 2.77 KB | None | 0 0
  1. http://joxi.ru/DmBNWL6FNXWELm
  2. /*
  3.     видишь повторения?
  4.     нужна переменная)
  5. */
  6. ***************************
  7. private void followLink(int linkIndex)
  8. /*
  9.     сравни
  10. */
  11. private void followLink(int index)
  12. /*
  13.      в имени параметра - link - уже лишнее
  14.      в имени метода - уже написали что и для какого объекта мы делаем
  15.      в имени параметра - уже повторяться не надо
  16. */
  17. ***********************************
  18. public static ExpectedCondition<Boolean> sizeOf(final By elementsLocator, final int expectedSize) {
  19.         ....
  20.             public String toString() {
  21.                 return String.format("\nSize of list: %s\nExpected to be: %s\nActual size is: %s\n"
  22.                         , elements
  23.                         , expectedSize
  24.                         , listSize);
  25.             }
  26. /*
  27.     Сравни - как будет выглядеть сообщение об ошибке -
  28.     если выводить туда elements или elementsLocator
  29.  
  30.     проверить легко
  31.     вызови проверку этого кондишена - которая заведомо упадет
  32. */
  33. ***************************************
  34.     public static ExpectedCondition<Boolean> textInElements(final By elementsLicator
  35.             , final int linkIndex
  36.             , final String expectedText)
  37. //сравни
  38.     public static ExpectedCondition<Boolean> nthListElementHasText(final By elementsLocator
  39.             , final int index
  40.             , final String expectedText)
  41. /*
  42.     название кондишена - чуть уточнила
  43.     пару параметров - тоже предлагаю другое имя    
  44. */            
  45.             public Boolean apply(WebDriver driver) {
  46. /*
  47.     в целом реализация ОК
  48.     молодец, замахнулся на что-то более серьезное
  49.     и практически все верно реализовал)
  50.    
  51.     единственный момент - если ты вызовешь такую проверку
  52.     для элемента по индексу за границей списка
  53.     то - вызовется исключение IndexOfOutException
  54.     добавь try-catch в этом методе
  55. */
  56. **********************************
  57.             public String toString() {
  58.                 return String.format("\nCurrent text of element with index %s is: %s\nExpected text: %s"
  59.                         , linkIndex
  60.                         , currentText
  61.                         , expectedText);
  62.             }
  63. /*
  64.     Я бы еще локатор elementsLocator уточнила
  65. */
Advertisement
Add Comment
Please, Sign In to add comment