1

Тема: Coverage на github

Хтось мені може пояснити, що це за хрінь така на гітхабі - coverage?

Хотів добре діло зробити, додати фічу до одної ліби, дуже корисну, зробив форк-пул-реквест, а воно ругається, що я зменшив коверейдж оцей собачий... Що воно в біса таке?

2

Re: Coverage на github

Від модератора: виділив у нову тему.

Ви не могли б навести точне повідомлення? Загалом зазвичай це слово означає покриття тестами, але я не зовсім розумію, чому треба його зменшувати.

Подякували: leofun011

3

Re: Coverage на github

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

4

Re: Coverage на github

Його якраз не треба зменшувати. Я так розумію, основна проблема, це оце:

Overall coverage decreased (-1.5%) to 93.333%

5

Re: Coverage на github

Це отаке на сторінці гітгаба показує:

Прихований текст

Some checks were not successful
1 failing and 2 successful checks
coverage/coveralls — Coverage decreased (-1.5%) to 93.333%
Required
Details
@travis-ci
Travis CI - Pull Request Successful in 1m — Build Passed
Required
Details
security/snyk (lahsivjar) — No manifest changes detected in 1 project
Details

6

Re: Coverage на github

Ну то додайте тести для нових функцій.

7

Re: Coverage на github

koala написав:

Ну то додайте тести для нових функцій.

:(
По-перше, я не створював нових функцій, лише додав одни пропс і кондішн для нього.
По-друге, я повний баран у оцих юніт-тестах і ніколи їх в житті не використовував :(

8

Re: Coverage на github

Ну от і маєте можливість нарешті дізнатися, як пропси і кондішни тестуються. Я, наприклад, навіть не знаю, що то за пропси.

Подякували: leofun011

9

Re: Coverage на github

koala написав:

Ну от і маєте можливість нарешті дізнатися, як пропси і кондішни тестуються. Я, наприклад, навіть не знаю, що то за пропси.

підозрюю то модний фронтенд на ректі якомусь

Подякували: flatliner, leofun012

10 Востаннє редагувалося Vo_Vik (09.11.2021 16:28:26)

Re: Coverage на github

То означає, що в юніт тесті немає випадку який заходить в ваш кондішин. Відповідно чатина коду, яку ви добавили не покрита тестами.
Знайдіть в тому репозиторії папочку з тестами і там пошукайте де згадується ваша функція. І подивіться як покриваються інші іфи. Скопіюйте і вставте і трошки поміняйте, щоб покривало ваш випадок

Подякували: flatliner, koala, leofun013

11

Re: Coverage на github

Q-bart написав:
koala написав:

Ну от і маєте можливість нарешті дізнатися, як пропси і кондішни тестуються. Я, наприклад, навіть не знаю, що то за пропси.

підозрюю то модний фронтенд на ректі якомусь

Так, це реакт компонент react-stomp. Просто вони якогось біса реалізували таку гарну фічу, як безумовне декодування json-повідомлень, а я саме зіткнувся з випадком, коли мені ця фіча не потрібна ну аж ніяк. Хотів додати опцію, яка б дозволяла вимикати поведінку за замовчуванням. Здавалося б простий випадок, всього один проп додати і завернути в умовний оператор... а от не так все просто виявилось. І я трохи побоююся лізти в оті юніт-тести зі своїми експериментами, коли я їх не до кінця розумію. Вже навіть виникло бажання, здатися і закрити до біса свій пул-реквест.

12

Re: Coverage на github

bvn написав:

І я трохи побоююся лізти в оті юніт-тести зі своїми експериментами, коли я їх не до кінця розумію

Так усе ж навпаки. Тестування якраз і потрібне, щоб НЕ боятися. Зараз ви боїтеся, що щось зламається. Тест покаже вам, що саме зламалося, і ви зможете це спокійно пофіксити, якщо зламали - і бути певним, що не зламали, якщо все запрацює.

Подякували: leofun01, Q-bart, bebyk3

13

Re: Coverage на github

Все, спробував - не вийшло. Я здався. Пул-реквест закрив і тему можна закривати.

14

Re: Coverage на github

йой, я аж захотів глянути на той PR

15

Re: Coverage на github

Q-bart написав:

йой, я аж захотів глянути на той PR

https://github.com/lahsivjar/react-stomp/pull/202

Подякували: leofun011