julia_v_iluhina

Untitled

Nov 13th, 2016
77
0
Never
Not a member of Pastebin yet? Sign Up, it unlocks many cool features!
Java 5.85 KB | None | 0 0
  1.     @Before
  2.     public void reset() {
  3.         resetBase();
  4.     }
  5. /*
  6.     мне кажется, было бы точнее использовать вызов  с указанием имени класса-ресурса
  7.     касается всего кода
  8.  
  9.     не resetBase();
  10.     а TasksRest.resetBase();
  11.     или даже TasksRest.reset(); (можно сократить имя метода в таком случае - как мы и для других методов сделали)
  12. */
  13. ******************************************
  14.     @Test
  15.     public void testUnauthorizedReadTasks() {
  16.  
  17.         response = requestTo(uri).get();
  18.  
  19.         assertEquals(403, getResponseStatus());
  20.         assertEquals("Unauthorized access", getErrorAnswer());
  21.     }
  22. /*
  23.     у нас тест-класс называется - RESTfulTasksTest
  24.     т е - про таски мы тут уже написали
  25.  
  26.     и потому - в именах тест-методов уже про таски можно не писать
  27.  
  28.     еще момент по пути - согласно правилам CamelCase - не RESTfulTasksTest, а RestFulTasksTest
  29.     https://google.github.io/styleguide/javaguide.html#s5.3-camel-case
  30.  
  31.     раз мы уже тут оперируем переменной response, то и методы getResponseStatus() и getErrorAnswer() - лишнее
  32.     сравни с вариантом ниже
  33.     соответствующие методы из класса-ресурса - убирай
  34. */
  35.    @Test
  36.     public void testUnauthorizedRead() {
  37.         Response response = requestTo(TasksRest.uri).get();
  38.  
  39.         assertEquals(403, response.getStatus());
  40.         assertEquals("Unauthorized access", response.readEntity(ErrorContainer.class).getError());
  41.     }
  42. **********************************************
  43.     @Test
  44.     public void testReadTasks() {
  45.  
  46.         get();
  47.  
  48.         assertEquals(200, getResponseStatus());
  49.         assertEquals(2, getTasksSize());
  50.         assertTasks(DEFAULT_TASKS[0], DEFAULT_TASKS[1]);
  51.     }
  52. /*
  53.     Тут - хитрее)
  54.  
  55.     как и для остальных операций, для get - тоже проверяй статус в методе сервиса
  56.  
  57.     метод get() - возвращает список тасок
  58.     и именно эти таски и надо сверять с DEFAULT_TASKS
  59.  
  60.     но - нужно ли использовать assertTasks?
  61.     ведь там - мы заново вызываем get
  62.     а вот вторая строчка из assertTasks - как раз то, что нужно и здесь
  63.     грамотнее вместо assertTasks - тут вызвать именно ее
  64.  
  65.     да и размер списка - проверять незачем
  66.     при сравнении двух списков с помощью assertEquals - и это проверим
  67.  
  68.     вот и метод getTasksSize() - не нужен
  69. */
  70. ***********************************************
  71.    @Test
  72.     public void testCreate() {
  73.         Task myTask = new Task("my task", false, "created task", uri + "/3");
  74. ...
  75. /*
  76.     Кстати, есть смысл создавать таску не только по полной информации,
  77.     но и только по title - чтобы полнее проверить создание таски
  78.     Стоит реализовать 2 тест-метода -
  79.     testCreateWithTitle()  (такой надо реализовать)
  80.     и testCreateWithFullInformation()(собственно, такой у тебя уже есть)
  81.  
  82.     В testCreateWithTitle() - тоже проверку делай через assertTasks (на этот момент будут известны все свойства добавленной таски -
  83.     так что это не проблема)
  84. */
  85. **********************
  86.     public void testDelete() {
  87. /*
  88.     В целом - все ок
  89.     единственное - не надо размер списка тасок проверять
  90.     ведь assertTasks и это выполнит
  91. */
  92. *******************************
  93.     public void testCreateUpdateDelete() {
  94.  
  95.         Task createCRUDTask = new Task("my CRUD task");
  96.  
  97.         create(createCRUDTask);
  98.         /*
  99.             делай проверки - после каждого действия
  100.             тут уже нужен assertTasks
  101.  
  102.             а следующее действие мы не можем посчитать косвенной проверкой
  103.             во-первых - потому что тасок уже 3 и надо проверить состояние каждой
  104.             во-вторых - даже если бы таска была единственной - следующая операция не проверит состояние всех свойств таски
  105.         */
  106.  
  107.         Task updatedTask = new Task("updated description", true, "updated task", uri + "/3");
  108.  
  109.         update(updatedTask);
  110.         /*
  111.             и тут проверку встрой
  112.         */
  113.  
  114.         delete(3);
  115.  
  116.         assertEquals(2, getTasksSize());
  117.         /*
  118.             этого не нужно
  119.         */
  120.         assertTasks(DEFAULT_TASKS[0], DEFAULT_TASKS[1]);
  121.     }
  122. /*
  123.     согласна - что в е2е тесте все операции выполняем уже над свежедобавленной таской, а не над теми, что были по умолчанию
  124.     это улучшит покрытие
  125.     хороший ход)
  126. */
  127. ********************************
  128. /*
  129.     Итого - остались мелочи
  130.    
  131.     подчистить код, убрать лишнее, добавить недостающее
  132.     но - уже ничего особенно нового или сложного
  133. */
Advertisement
Add Comment
Please, Sign In to add comment