Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Split list unit tests v2 #630

Closed
wants to merge 3 commits into from
Closed

Split list unit tests v2 #630

wants to merge 3 commits into from

Conversation

mniestroj
Copy link
Collaborator

No description provided.

Copy link

github-actions bot commented Sep 26, 2024

Visit the preview URL for this PR (updated for commit b82b42a):

https://golioth-firmware-sdk-doxygen-dev--pr630-split-list-uni-f6i1iz7b.web.app

(expires Tue, 08 Oct 2024 05:16:14 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: a9993e61697a3983f3479e468bcb0b616f9a0578

Copy link

Code Coverage (Linux)

Type Coverage
lines 65.6% (1402 of 2138 lines)
functions 78.1% (139 of 178 functions)

@mniestroj mniestroj force-pushed the split-list-unit-tests-v2 branch 2 times, most recently from e361863 to 78ebb36 Compare September 26, 2024 19:28
Copy link

github-actions bot commented Sep 26, 2024

Code Coverage (Zephyr)

Code Coverage

Package Line Rate Branch Rate Health
include.golioth 75% 50%
port.zephyr 49% 14%
src 67% 27%
Summary 65% (1856 / 2847) 26% (765 / 2915)

@mniestroj mniestroj force-pushed the split-list-unit-tests-v2 branch from 78ebb36 to 96ecd5b Compare September 28, 2024 16:26
Both are not related to each other and they should be split into separate
jobs.

Signed-off-by: Marcin Niestroj <[email protected]>
tests/unit_tests/test.sh is very verbose, while it is possible to do most
of it with single command.

Use Ninja generator to build using multiple threads automatically.

Signed-off-by: Marcin Niestroj <[email protected]>
@mniestroj mniestroj force-pushed the split-list-unit-tests-v2 branch from 96ecd5b to 900439c Compare October 1, 2024 04:45
Install both gcovr and ninja from pypi, so a single tool (pip) is used.

Signed-off-by: Marcin Niestroj <[email protected]>
@mniestroj mniestroj force-pushed the split-list-unit-tests-v2 branch from 900439c to b82b42a Compare October 1, 2024 05:15
@mniestroj mniestroj closed this Oct 4, 2024
@mniestroj mniestroj deleted the split-list-unit-tests-v2 branch October 4, 2024 05:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant