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

tests: hil: run HIL tests on 'native_sim' and generate coverage data #590

Merged
merged 3 commits into from
Sep 19, 2024

Conversation

mniestroj
Copy link
Collaborator

@mniestroj mniestroj commented Sep 13, 2024

Contains changes (1 commit) from: #581

Copy link

github-actions bot commented Sep 13, 2024

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

https://golioth-firmware-sdk-doxygen-dev--pr590-native-sim-cov-z6t6hcd6.web.app

(expires Tue, 24 Sep 2024 17:03:27 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: a9993e61697a3983f3479e468bcb0b616f9a0578

@mniestroj mniestroj force-pushed the native-sim-coverage-hil branch 4 times, most recently from 9de8cb9 to 06b6dd7 Compare September 13, 2024 15:43
Copy link

github-actions bot commented Sep 13, 2024

Code Coverage (Linux)

Type Coverage
lines 65.7% (1404 of 2138 lines)
functions 78.1% (139 of 178 functions)

@mniestroj mniestroj force-pushed the native-sim-coverage-hil branch 8 times, most recently from fb8b3cb to b09fe26 Compare September 15, 2024 09:30
Copy link

github-actions bot commented Sep 15, 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 native-sim-coverage-hil branch 3 times, most recently from 9865dea to 1b6d7d2 Compare September 17, 2024 13:22
@mniestroj mniestroj marked this pull request as ready for review September 17, 2024 13:55
@mniestroj mniestroj requested review from sam-golioth and szczys and removed request for sam-golioth September 17, 2024 13:56
Implement 'send_all()' async method for sending data to (real or virtual)
board. This will allow to communicate with virtual boards in the same
way (with the same API) as with real boards.

Signed-off-by: Marcin Niestroj <m.niestroj@emb.dev>
Add configuration overlays for 'hil' application, so that networking is
configured on native_sim Zephyr platform.

Add support for 'native_sim' board with NativeSimBoard class, which is a
hybrid of LinuxBoard and ZephyrBoard. Override 'started()' method, so
native_sim process is executed as part of it automatically.

Signed-off-by: Marcin Niestroj <m.niestroj@emb.dev>
Execute HIL tests on 'native_sim' platform. Enable coverage data
generation, collect it and merge it together with Twister generated sample
coverage.

Signed-off-by: Marcin Niestroj <m.niestroj@emb.dev>
Copy link
Contributor

@sam-golioth sam-golioth left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome work @mniestroj! I'm stoked to get this in.

@mniestroj mniestroj merged commit 2b6e77d into main Sep 19, 2024
43 of 47 checks passed
@mniestroj mniestroj deleted the native-sim-coverage-hil branch September 19, 2024 03:58
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.

3 participants