Ви не увійшли. Будь ласка, увійдіть або зареєструйтесь.
Ласкаво просимо вас на україномовний форум з програмування, веб-дизайну, SEO та всього пов'язаного з інтернетом та комп'ютерами.
Будемо вдячні, якщо ви поділитись посиланням на Replace.org.ua на інших ресурсах.
Для того щоб створювати теми та надсилати повідомлення вам потрібно Зареєструватись.
Український форум програмістів → Системи контролю версій (SCM, VCS) → Coverage на github
Сторінки 1
Для відправлення відповіді ви повинні увійти або зареєструватися
Хтось мені може пояснити, що це за хрінь така на гітхабі - coverage?
Хотів добре діло зробити, додати фічу до одної ліби, дуже корисну, зробив форк-пул-реквест, а воно ругається, що я зменшив коверейдж оцей собачий... Що воно в біса таке?
Від модератора: виділив у нову тему.
Ви не могли б навести точне повідомлення? Загалом зазвичай це слово означає покриття тестами, але я не зовсім розумію, чому треба його зменшувати.
Pull Request Test Coverage Report for Build 653
2 of 3 (66.67%) changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-1.5%) to 93.333%
Changes Missing Coverage Covered Lines Changed/Added Lines %
src/client.jsx 2 3 66.67%
Totals Coverage Status
Change from base Build 640: -1.5%
Covered Lines: 82
Relevant Lines: 84
Його якраз не треба зменшувати. Я так розумію, основна проблема, це оце:
Overall coverage decreased (-1.5%) to 93.333%
Це отаке на сторінці гітгаба показує:
Ну то додайте тести для нових функцій.
Ну то додайте тести для нових функцій.
По-перше, я не створював нових функцій, лише додав одни пропс і кондішн для нього.
По-друге, я повний баран у оцих юніт-тестах і ніколи їх в житті не використовував
Ну от і маєте можливість нарешті дізнатися, як пропси і кондішни тестуються. Я, наприклад, навіть не знаю, що то за пропси.
То означає, що в юніт тесті немає випадку який заходить в ваш кондішин. Відповідно чатина коду, яку ви добавили не покрита тестами.
Знайдіть в тому репозиторії папочку з тестами і там пошукайте де згадується ваша функція. І подивіться як покриваються інші іфи. Скопіюйте і вставте і трошки поміняйте, щоб покривало ваш випадок
koala написав:Ну от і маєте можливість нарешті дізнатися, як пропси і кондішни тестуються. Я, наприклад, навіть не знаю, що то за пропси.
підозрюю то модний фронтенд на ректі якомусь
Так, це реакт компонент react-stomp. Просто вони якогось біса реалізували таку гарну фічу, як безумовне декодування json-повідомлень, а я саме зіткнувся з випадком, коли мені ця фіча не потрібна ну аж ніяк. Хотів додати опцію, яка б дозволяла вимикати поведінку за замовчуванням. Здавалося б простий випадок, всього один проп додати і завернути в умовний оператор... а от не так все просто виявилось. І я трохи побоююся лізти в оті юніт-тести зі своїми експериментами, коли я їх не до кінця розумію. Вже навіть виникло бажання, здатися і закрити до біса свій пул-реквест.
І я трохи побоююся лізти в оті юніт-тести зі своїми експериментами, коли я їх не до кінця розумію
Так усе ж навпаки. Тестування якраз і потрібне, щоб НЕ боятися. Зараз ви боїтеся, що щось зламається. Тест покаже вам, що саме зламалося, і ви зможете це спокійно пофіксити, якщо зламали - і бути певним, що не зламали, якщо все запрацює.
Все, спробував - не вийшло. Я здався. Пул-реквест закрив і тему можна закривати.
йой, я аж захотів глянути на той PR
Сторінки 1
Для відправлення відповіді ви повинні увійти або зареєструватися
