Not a member of Pastebin yet?
Sign Up,
it unlocks many cool features!
- public Task(int id, String title, String description, boolean done) {
- this.title = title;
- this.description = description;
- this.done = done;
- this.uri = TasksRest.uri + "/" + id;
- }
- /*
- вот это я пропустила)
- лучше бы - чтоб классы-контейнеры ничего не знали про ресурс
- передавай как параметр не ид, а сам uri
- цель - чтоб классы-контейнеры были автономны от логики ресурса
- примерно также - на логику ресурса не влияет логика тест-классов, например
- получим - инструменты более простые, не знают ничего про инструменты более сложные
- с другой стороны - более сложные инструменты используют эти простые инструметы
- такие простые правила - дают нам как раз такую реализацию инструментов, которые легко использовать
- нету никаких сложных взаимовлияний и сложных связей
- */
- ************************************************************
- @Test
- public void testUnauthorizedReadTasks() {
- Response response = requestTo(uri).get();
- ErrorContainer expected = new ErrorContainer("Unauthorized access");
- assertEquals(403, response.getStatus());
- assertEquals(expected, response.readEntity(ErrorContainer.class));
- }
- /*
- можно не делать ErrorContainer expected = new ErrorContainer("Unauthorized access");
- тогда проверяем текст ошибки - assertEquals("Unauthorized access", response.readEntity(ErrorContainer.class).getError());
- будет чуть лаконичнее
- */
- ***********************************************
- public void testCreate() {
- TasksRest.create(new Task("task title"));
- ...
- public void testCreateWithTitle() {
- Task newTask = new Task(3, "task title", "task descr", false);
- /*
- я бы первый метод назвала как testCreateWithTitle()
- второй - testCreateWithFullInformation() или testCreate()
- */
- ************************************************
- @Test
- public void testDelete() {
- TasksRest.delete(2);
- TasksRest.assertTasks(DEFAULT_TASKS[1]);
- }
- /*
- уверена, что все ок с методом?
- удаляем таску с ид = 2 (осталас - с ид=1 = первая из списка)
- сверяем - со второй (индекс=1, нумерация с нуля)
- */
Advertisement
Add Comment
Please, Sign In to add comment