search

shri-2015-org / 666

666 team's chat =>
http://anonym.club
4 stars 3 forks source link

fix tests 😡 #60

Closed pofigizm closed 9 years ago

pofigizm commented 9 years ago
  1. Мы пишем тесты!
  2. Тот кто считает что это не нужно -> см. п 1.
  3. Все тесты должны проходить
  4. Если были изменения в структуре тестируемого модуля -> меняем тесты
  5. Добавили новые методы / функционал -> добавляем в тесты

p.s. тесты на server/index.js пока убрал т.к. в рамках настройки инфраструктуры я планирую его изменить

tempname11 commented 9 years ago

+1

А тесты не стоит автоматически запускать а-ля линтер?

pofigizm commented 9 years ago

Добавил в хуки, но у меня они не работают, даже если не добавлять. Не понимаю почему. Все обновил несколько раз с удалением и отчисткой кэша. Один фиг не срабатывает ни линт ни тест при коммите. Сами по себе - работают.

pofigizm commented 9 years ago

Я еще зависимости обновил на последние стабильные, после мержа обновиться не забудте.

pofigizm commented 9 years ago

Ок.