Резюме зустрічі 20.01.16

Відео

Робота над задачами [00:00 - 5:35]

• необхідно дотримуватись наступного циклу роботи над задачами:
  • виконання задачі;
  • code-review;
  • демо (в коментарях до задачі на GitHub необхідно додати коментар на відео з демо)
  • назначити її на Віктора і перенести її в колонку Ready на Waffle.io, при цьому на GitHub у задачі з'явиться тег ready;

Code-review таска №53 [05:35 - 1:11:13]

[12:30 - 29:00]

• треба створити дефолтні .propperties, які б використовувались за замовченням;
• треба створити профілі;
• треба налаштувти наступний порядок завантаження: default.properties > local.properties

[29:00 - 1:11:13]

• не дотримується TDD;
• погана назва класу MailReminderAboutEventService.java;
• в коміти треба додавати "тег" (наприлад, #53: commit message);
• гілки треба називати, наприклад, так: #53 або issue53;
• приклад написання тесту; [56:50]
• виправлення тесту MailReminderAboutEventService.executeTest() [43:10]
• створення template для Intellij IDEA: Editor > File and Code Templates, Code > JUnit 4 Test Method :

   @org.junit.Test
   public void should${NAME}() throws Exception {
     ${BODY}
   }

• треба дотримуватись єдиного стилю в назві змінних в .properties-файлах; [55:00]
• треба розбити applicationContext.xml на декілька модулів і в тестах використовувати тільки потрібні модулі [57:20]
• треба кидати виключення в блоці catch; [1:01:35]
• треба створити пакет integration, де б знаходились класи, що відповідають за інтеграцію зі сторонніми сервісами; [1:07:00]