Not a member of Pastebin yet?
Sign Up,
it unlocks many cool features!
- public Task (String description, boolean done, String title, int id, String uri) {
- ...
- this.uri = uri + "/" + id;
- }
- /*
- я бы все равно не стала внутри класса Task - зашивать логику работы с uri
- Не надо классу-контейнеру знать - как этот uri получается
- У этого класса - одна функция - быть контейнером
- И нечего ему знать как генерить uri для таски
- Что ему передали - то и будет uri для таски
- Single Responsibility Principle
- считаю вот такой вариант - лучше
- public Task (String description, boolean done, String title, String uri)
- и соответственно - передавать будешь в качестве uri - TasksApi.uri + "/" + id
- */
- ************************
- @Before
- public void resetRestfulBase() {
- reset();
- }
- //или
- @Before
- public void reset() {
- TasksApi.reset();
- }
- /*
- в целом - букв примерно столько же)
- но - такой вариант мне кажется - буквы более полезные ) - от них больше полезной уточняющей информации
- я бы вызывала методы ресурса-модуля - указывая его имя
- и сэкономила
- на именах методов тест-класса(не testReadTasks(), а testRead())
- и самого ресурса(тут ты уже это сделал)
- но вызывая методы ресурса - указывала бы имя класса для ресурса
- */
- *************************
- public void testCreateTask()
- public void testCreateNotFilledTask()
- //сравни
- public void testCreateWithFullInformation()
- public void testCreateWithTitle()
- /*
- второй вариант - однозначнее, понятнее
- */
Advertisement
Add Comment
Please, Sign In to add comment