julia_v_iluhina

Untitled

Nov 30th, 2016
81
0
Never
Not a member of Pastebin yet? Sign Up, it unlocks many cool features!
Java 2.81 KB | None | 0 0
  1.  public Task(int id, String title, String description, boolean done) {
  2.         this.title = title;
  3.         this.description = description;
  4.         this.done = done;
  5.         this.uri = TasksRest.uri + "/" + id;
  6.     }
  7. /*
  8.     вот это я пропустила)
  9.  
  10.     лучше бы - чтоб классы-контейнеры ничего не знали про ресурс
  11.  
  12.     передавай как параметр не ид, а сам uri
  13.  
  14.     цель -  чтоб классы-контейнеры были автономны от логики ресурса
  15.  
  16.     примерно также  - на логику ресурса не влияет логика тест-классов, например
  17.  
  18.     получим - инструменты более простые, не знают ничего про инструменты более сложные
  19.     с другой стороны - более сложные  инструменты используют эти простые инструметы
  20.     такие простые правила - дают нам как раз такую реализацию инструментов, которые легко использовать
  21.     нету никаких сложных взаимовлияний и сложных связей
  22. */
  23. ************************************************************
  24.     @Test
  25.     public void testUnauthorizedReadTasks() {
  26.         Response response = requestTo(uri).get();
  27.  
  28.         ErrorContainer expected = new ErrorContainer("Unauthorized access");
  29.         assertEquals(403, response.getStatus());
  30.         assertEquals(expected, response.readEntity(ErrorContainer.class));
  31.     }
  32. /*
  33.     можно не делать ErrorContainer expected = new ErrorContainer("Unauthorized access");
  34.     тогда проверяем текст ошибки - assertEquals("Unauthorized access", response.readEntity(ErrorContainer.class).getError());
  35.     будет чуть лаконичнее
  36. */
  37. ***********************************************
  38.  public void testCreate() {
  39.         TasksRest.create(new Task("task title"));
  40. ...
  41.   public void testCreateWithTitle() {
  42.         Task newTask = new Task(3, "task title", "task descr", false);
  43. /*
  44.     я бы первый метод назвала как testCreateWithTitle()
  45.  
  46.     второй - testCreateWithFullInformation() или testCreate()
  47. */
  48. ************************************************
  49.     @Test
  50.     public void testDelete() {
  51.         TasksRest.delete(2);
  52.  
  53.         TasksRest.assertTasks(DEFAULT_TASKS[1]);
  54.     }
  55. /*
  56.     уверена, что все ок с методом?
  57.  
  58.     удаляем таску с ид = 2 (осталас  - с ид=1 = первая из списка)
  59.  
  60.     сверяем - со второй (индекс=1, нумерация с нуля)
  61. */
Advertisement
Add Comment
Please, Sign In to add comment