julia_v_iluhina

Untitled

Oct 2nd, 2016
72
0
Never
Not a member of Pastebin yet? Sign Up, it unlocks many cool features!
Java 1.88 KB | None | 0 0
  1. Element not found {... .za}
  2.  
  3. /*
  4.     http://joxi.ru/V2VBQLqf06P0d2
  5.     я в прошлый раз ошиблась в регистре одной буквы)
  6.  
  7.     лучше бы поправить)
  8.     будет чуть лаконичнее селектор
  9. */
  10. *************************
  11.     public static void assertNthMail(int index, String text){
  12.         mails.get(index).shouldHave(text(text));
  13.     }
  14.  
  15.     public static void assertMails(String... text){
  16.         mails.shouldHave(texts(text));
  17.     }
  18.  
  19. /*
  20.     почти идеально)
  21.  
  22.     в метод assertMails - мы передаем не text, а  texts
  23.     это поправь обязательно
  24.  
  25.     и варианты shouldHave(text(text)) и shouldHave(texts(texts)) -
  26.     несколько смущают)
  27.     на первый взгляд)
  28.     чтоб и на первый взгляд не смущали - я бы параметры методов назвала
  29.     mailText & mailTexts
  30.     or
  31.     mailHeaderText & mailHeaderTexts
  32.     это - не настаиваю, но рекомендую)
  33.  
  34. */
  35. ***********************************
  36.  //com.codeborne.selenide.Selectors.withText - его нельзя написать как By.withText
  37.  /*
  38.     да, тут я была не права
  39.     )
  40.     оставляй так, как у тебя
  41.     Selectors.withText - ... - я бы не стала так тоже писать)
  42.  */
  43. ************************************
  44.  public static String generateUniqueText(String prefix){
  45.         DateFormat dateFormat = new SimpleDateFormat("yyyy/MM/dd HH:mm:ss");
  46.         Date date = new Date();
  47.         return (prefix + dateFormat.format(date));
  48.  }
  49.  /*
  50.     можно в одну строку
  51.  
  52.     return (prefix + (new SimpleDateFormat("yyyy/MM/dd HH:mm:ss")).format(new Date()));
  53.  
  54.     мне кажется - ок
  55.     не настаиваю
  56.  */
Advertisement
Add Comment
Please, Sign In to add comment