julia_v_iluhina

Untitled

Jan 13th, 2017
121
0
Never
Not a member of Pastebin yet? Sign Up, it unlocks many cool features!
Java 4.11 KB | None | 0 0
  1.      @Test
  2.     public void testTasksLifeCycle() {
  3.  
  4.         given(new Task(TaskType.ACTIVE, "a"));
  5. /*
  6.     можно так
  7.  
  8.     а можно и вот так
  9. */
  10.     @Test
  11.     public void testTasksLifeCycle() {
  12.  
  13.         given();
  14.  
  15.         add("a");
  16. /*
  17.     получаем разницу - покрыли мы операцию добавления таски на all фильтре в е2е тесте или нет
  18.  
  19.     не забывай о форматировании кода
  20.     пропускай строку после вызова гивен-метода
  21.     касается всех без исключения тест-методов
  22.  
  23.     еще не учел мои прошлые помментарии про разнообразие тестовых ситуаций
  24.     если не учтешь это - поставить по работе смогу - только accepted
  25.  
  26.     ну и вообще - еще раз прошлое ревью проработай
  27.     это не единственное что осталось
  28. */
  29. ***************
  30.     @Step
  31.     private void confirmEditByClickOutside(String oldTaskText, String newTaskText) {
  32.         startEdit(oldTaskText, newTaskText);
  33.         $("#new-todo").click();
  34.     }
  35. /*
  36.     теперь $("#new-todo") - используется дважды
  37.     значит - пора выносить это в переменную
  38.  
  39.     https://docs.google.com/document/d/13dNyFGbI7mV22UUhH8E0LJ7SzabAmX7Bw7VCHScYfiU/edit#bookmark=id.4i6i27d7lwn4
  40. */
  41. ***********************************
  42. public class AtTodoMVCPageWithClearedDataAfterEachTest extends BaseTest {
  43.  
  44.     @Before
  45.     public void openPage() {
  46.         open("https://todomvc4tasj.herokuapp.com/");
  47.     }
  48.     public void emptyTask(){
  49.  
  50.  
  51.     }
  52.  
  53.     @After
  54.     public void clearData() {
  55.         executeJavaScript("localStorage.clear()");
  56.     }
  57.  
  58.     @Step
  59.     public void ensureCorrectURL(){
  60.         String url = url();
  61.         String todomvc = "https://todomvc4tasj.herokuapp.com/";
  62.         if ( url == todomvc )
  63.             System.out.println("correct url");
  64.         else
  65.             System.out.println("wrong url");
  66.     }
  67. }
  68. /*
  69.     странный класс получился)
  70.  
  71.     ensureCorrectURL() - обеспечить корректный урл
  72.     а он - просто что-то рассказывает)
  73.     причем - не правду )
  74.  
  75.     потому что строки нужна сравнивать вот так - url().equals("https://todomvc4tasj.herokuapp.com/")
  76.     http://stackoverflow.com/questions/513832/how-do-i-compare-strings-in-java
  77.     http://www.javatpoint.com/string-comparison-in-java
  78.     http://alvinalexander.com/java/edu/qanda/pjqa00001.shtml
  79.  
  80.     получишь примерно такой метод
  81. */
  82.     @Step
  83.     public void ensureCorrectURL(){
  84.         String todoMvcUrl = "https://todomvc4tasj.herokuapp.com/";
  85.         if ( ! todoMvcUrl.equals(url()) )
  86.             open(todoMvcUrl);
  87.         }
  88.     }
  89. /*
  90.     обрати внимание - не надо создавать лишних переменных
  91.     да и на название переменных обрати внимание
  92.  
  93.     и на вот это - правила для CamelCase написания
  94.     https://google.github.io/styleguide/javaguide.html#s5.3-camel-case
  95.  
  96.     этот метод - перенеси к гивенам, в тест-класс
  97.     и вызови - в начале гивен-метода public void given(Task... tasks)
  98.     далее - в остальных гивен-методах - мы его переиспользуем
  99.     так что - все гивен-методы будут сначала обеспечивать урл
  100.     а потом делать все остальное
  101.  
  102.     а этот класс - AtTodoMVCPageWithClearedDataAfterEachTest - полностью удаляй из проекта
  103.     в прошлом ревью я писала подробнее - почему это можно сделать
  104.    
  105.     тестовый класс будешь наследовать от BaseTest
  106. */
Advertisement
Add Comment
Please, Sign In to add comment