Not a member of Pastebin yet?
Sign Up,
it unlocks many cool features!
- @Test
- public void testTasksLifeCycle() {
- given(new Task(TaskType.ACTIVE, "a"));
- /*
- можно так
- а можно и вот так
- */
- @Test
- public void testTasksLifeCycle() {
- given();
- add("a");
- /*
- получаем разницу - покрыли мы операцию добавления таски на all фильтре в е2е тесте или нет
- не забывай о форматировании кода
- пропускай строку после вызова гивен-метода
- касается всех без исключения тест-методов
- еще не учел мои прошлые помментарии про разнообразие тестовых ситуаций
- если не учтешь это - поставить по работе смогу - только accepted
- ну и вообще - еще раз прошлое ревью проработай
- это не единственное что осталось
- */
- ***************
- @Step
- private void confirmEditByClickOutside(String oldTaskText, String newTaskText) {
- startEdit(oldTaskText, newTaskText);
- $("#new-todo").click();
- }
- /*
- теперь $("#new-todo") - используется дважды
- значит - пора выносить это в переменную
- https://docs.google.com/document/d/13dNyFGbI7mV22UUhH8E0LJ7SzabAmX7Bw7VCHScYfiU/edit#bookmark=id.4i6i27d7lwn4
- */
- ***********************************
- public class AtTodoMVCPageWithClearedDataAfterEachTest extends BaseTest {
- @Before
- public void openPage() {
- open("https://todomvc4tasj.herokuapp.com/");
- }
- public void emptyTask(){
- }
- @After
- public void clearData() {
- executeJavaScript("localStorage.clear()");
- }
- @Step
- public void ensureCorrectURL(){
- String url = url();
- String todomvc = "https://todomvc4tasj.herokuapp.com/";
- if ( url == todomvc )
- System.out.println("correct url");
- else
- System.out.println("wrong url");
- }
- }
- /*
- странный класс получился)
- ensureCorrectURL() - обеспечить корректный урл
- а он - просто что-то рассказывает)
- причем - не правду )
- потому что строки нужна сравнивать вот так - url().equals("https://todomvc4tasj.herokuapp.com/")
- http://stackoverflow.com/questions/513832/how-do-i-compare-strings-in-java
- http://www.javatpoint.com/string-comparison-in-java
- http://alvinalexander.com/java/edu/qanda/pjqa00001.shtml
- получишь примерно такой метод
- */
- @Step
- public void ensureCorrectURL(){
- String todoMvcUrl = "https://todomvc4tasj.herokuapp.com/";
- if ( ! todoMvcUrl.equals(url()) )
- open(todoMvcUrl);
- }
- }
- /*
- обрати внимание - не надо создавать лишних переменных
- да и на название переменных обрати внимание
- и на вот это - правила для CamelCase написания
- https://google.github.io/styleguide/javaguide.html#s5.3-camel-case
- этот метод - перенеси к гивенам, в тест-класс
- и вызови - в начале гивен-метода public void given(Task... tasks)
- далее - в остальных гивен-методах - мы его переиспользуем
- так что - все гивен-методы будут сначала обеспечивать урл
- а потом делать все остальное
- а этот класс - AtTodoMVCPageWithClearedDataAfterEachTest - полностью удаляй из проекта
- в прошлом ревью я писала подробнее - почему это можно сделать
- тестовый класс будешь наследовать от BaseTest
- */
Advertisement
Add Comment
Please, Sign In to add comment