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

ci: increase twister verbosity with single -v flag #752

Merged
merged 1 commit into from
Mar 1, 2025
Merged

Conversation

mniestroj
Copy link
Collaborator

This changes following report:

INFO    - Total complete:    1/   4   25%  built (not run):    1, filtered:    0, failed:    0, error:    0
INFO    - Total complete:    2/   4   50%  built (not run):    1, filtered:    0, failed:    0, error:    0
INFO    - Total complete:    3/   4   75%  built (not run):    1, filtered:    0, failed:    0, error:    0
INFO    - Total complete:    4/   4  100%  built (not run):    1, filtered:    0, failed:    0, error:    0
INFO    - 4 test scenarios (4 configurations) selected, 0 configurations filtered (0 by static filter, 0 at runtime).
INFO    - 3 of 4 executed test configurations passed (75.00%), 1 built (not run), 0 failed, 0 errored, with no warnings in 49.12 seconds.
INFO    - 3 of 3 executed test cases passed (100.00%) on 1 out of total 860 platforms (0.12%).
INFO    - 1 selected test cases not executed: 1 not run (built only).
INFO    - 3 test configurations executed on platforms, 1 test configurations were only built.

to:

INFO    - 1/4 native_sim/native/64      sample.golioth.location                            NOT RUN (build)
INFO    - 2/4 native_sim/native/64      sample.golioth.location.playback.wifi              PASSED (native 12.659s)
INFO    - 3/4 native_sim/native/64      sample.golioth.location.playback.cellular          PASSED (native 27.586s)
INFO    - 4/4 native_sim/native/64      sample.golioth.location.playback                   PASSED (native 37.606s)

INFO    - 4 test scenarios (4 configurations) selected, 0 configurations filtered (0 by static filter, 0 at runtime).
INFO    - 3 of 4 executed test configurations passed (75.00%), 1 built (not run), 0 failed, 0 errored, with no warnings in 49.04 seconds.
INFO    - 3 of 3 executed test cases passed (100.00%) on 1 out of total 860 platforms (0.12%).
INFO    - 1 selected test cases not executed: 1 not run (built only).
INFO    - 3 test configurations executed on platforms, 1 test configurations were only built.

Copy link

github-actions bot commented Feb 28, 2025

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

https://golioth-firmware-sdk-doxygen-dev--pr752-twister-v-wgo29idg.web.app

(expires Fri, 07 Mar 2025 17:06:53 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: a9993e61697a3983f3479e468bcb0b616f9a0578

@mniestroj mniestroj force-pushed the twister-v branch 2 times, most recently from bf8e521 to fd9c06f Compare February 28, 2025 13:53
Copy link

github-actions bot commented Feb 28, 2025

Code Coverage

Code Coverage

Package Line Rate Branch Rate Health
include.golioth 75% 50%
port.linux 62% 34%
port.utils 58% 46%
port.zephyr 53% 22%
src 70% 32%
Summary 69% (2850 / 4159) 31% (1183 / 3767)

Copy link

codecov bot commented Feb 28, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

see 8 files with indirect coverage changes

This prints more details in the GitHub Actions job log, specifically
per-sample status:

  INFO    - 1/4 native_sim/native/64      \
    sample.golioth.location                        NOT RUN (build)
  INFO    - 2/4 native_sim/native/64      \
    sample.golioth.location.playback.wifi          PASSED (native 12.659s)
  INFO    - 3/4 native_sim/native/64      \
    sample.golioth.location.playback.cellular      PASSED (native 27.586s)
  INFO    - 4/4 native_sim/native/64      \
    sample.golioth.location.playback               PASSED (native 37.606s)

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

Just removed second -v flag from one of the invocations, so that we have always the same verbosity.

@mniestroj mniestroj merged commit 5746568 into main Mar 1, 2025
132 of 140 checks passed
@mniestroj mniestroj deleted the twister-v branch March 1, 2025 12:57
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.

2 participants