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

Add coverage data to testspace #1088

Merged

Conversation

grasci-arm
Copy link
Collaborator

No description provided.

@github-actions
Copy link

github-actions bot commented Aug 9, 2023

Test Results

    7 files    53 suites   5m 35s ⏱️
186 tests 169 ✔️ 17 💤 0
692 runs  624 ✔️ 68 💤 0

Results for commit ad63297.

♻️ This comment has been updated with latest results.

@codecov
Copy link

codecov bot commented Aug 10, 2023

Codecov Report

Merging #1088 (ad63297) into main (0f66cb4) will increase coverage by 2.83%.
Report is 2 commits behind head on main.
The diff coverage is 81.25%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1088      +/-   ##
==========================================
+ Coverage   55.17%   58.01%   +2.83%     
==========================================
  Files         116      116              
  Lines       22774    22782       +8     
  Branches    12650    12656       +6     
==========================================
+ Hits        12566    13217     +651     
+ Misses       8034     7337     -697     
- Partials     2174     2228      +54     
Flag Coverage Δ
buildmgr-cov 77.95% <ø> (ø)
packchk-cov 64.20% <ø> (ø)
packgen-cov 81.72% <ø> (ø)
projmgr-cov 82.62% <ø> (ø)
svdconv-cov 38.38% <81.25%> (+5.81%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed Coverage Δ
tools/svdconv/SVDModel/src/SvdCpu.cpp 36.97% <40.00%> (+4.20%) ⬆️
tools/svdconv/SVDGenerator/src/PartitionData.cpp 66.10% <100.00%> (+66.10%) ⬆️

... and 13 files with indirect coverage changes

Copy link
Collaborator

@soumeh01 soumeh01 left a comment

Choose a reason for hiding this comment

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

@JonatanAntoni Please review this.
testspace analytics job is expected to fail because of the default threshold set.

@soumeh01 soumeh01 merged commit 5240b4a into Open-CMSIS-Pack:main Aug 10, 2023
106 of 107 checks passed
@soumeh01 soumeh01 deleted the outbound/coverage_testspace branch August 10, 2023 13:25
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