Skip to content

Add methods to get devices and boards (#1277)#2164

Merged
edriouk merged 2 commits into
Open-CMSIS-Pack:mainfrom
ARM-software:outbound/DevicesBoards
Jul 29, 2025
Merged

Add methods to get devices and boards (#1277)#2164
edriouk merged 2 commits into
Open-CMSIS-Pack:mainfrom
ARM-software:outbound/DevicesBoards

Conversation

@grasci-arm

Copy link
Copy Markdown
Collaborator
  • Added GetDeviceList and GetDeviceData methods and corresponding structures
  • Unit Tests and corrections
  • Use id string
  • Board data
  • Tests and fixes
  • Update csolution-rpc version
  • add #include

Co-authored-by: Daniel Brondani daniel.brondani@arm.com

* Added GetDeviceList and GetDeviceData methods and corresponding structures
* Unit Tests and corrections
* Use id string
* Board data
* Tests and fixes
* Update `csolution-rpc` version
* add #include <list>

---------

Co-authored-by: Daniel Brondani <daniel.brondani@arm.com>
edriouk
edriouk previously approved these changes Jul 29, 2025
@github-actions

github-actions Bot commented Jul 29, 2025

Copy link
Copy Markdown

Test Results

 33 files   66 suites   10s ⏱️
166 tests 166 ✅ 0 💤 0 ❌
498 runs  498 ✅ 0 💤 0 ❌

Results for commit 3c51303.

♻️ This comment has been updated with latest results.

Comment thread tools/projmgr/src/ProjMgrRpcServerData.cpp Fixed
Remove redudant line
@edriouk edriouk merged commit 963c817 into Open-CMSIS-Pack:main Jul 29, 2025
81 of 82 checks passed
@edriouk edriouk deleted the outbound/DevicesBoards branch July 29, 2025 14:26
@codecov

codecov Bot commented Jul 29, 2025

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 62.61468% with 163 lines in your changes missing coverage. Please review.
✅ Project coverage is 66.58%. Comparing base (8b49f0e) to head (3c51303).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
tools/projmgr/test/src/ProjMgrRpcTests.cpp 49.33% 0 Missing and 114 partials ⚠️
tools/projmgr/src/ProjMgrRpcServer.cpp 55.55% 32 Missing ⚠️
tools/projmgr/src/ProjMgrRpcServerData.cpp 88.97% 9 Missing and 6 partials ⚠️
tools/projmgr/src/ProjMgrRunDebug.cpp 33.33% 0 Missing and 2 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2164      +/-   ##
==========================================
- Coverage   66.59%   66.58%   -0.02%     
==========================================
  Files         181      181              
  Lines       37867    38243     +376     
  Branches    22079    22375     +296     
==========================================
+ Hits        25219    25464     +245     
- Misses       8064     8076      +12     
- Partials     4584     4703     +119     
Flag Coverage Δ
buildmgr-cov 73.95% <ø> (ø)
packchk-cov 70.18% <ø> (ø)
projmgr-cov 80.89% <62.61%> (-0.38%) ⬇️

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

Files with missing lines Coverage Δ
tools/projmgr/include/ProjMgrRpcServerData.h 100.00% <ø> (ø)
tools/projmgr/include/ProjMgrRunDebug.h 100.00% <ø> (ø)
tools/projmgr/src/ProjMgrRunDebug.cpp 86.91% <33.33%> (+0.21%) ⬆️
tools/projmgr/src/ProjMgrRpcServerData.cpp 83.58% <88.97%> (+3.48%) ⬆️
tools/projmgr/src/ProjMgrRpcServer.cpp 62.22% <55.55%> (+3.21%) ⬆️
tools/projmgr/test/src/ProjMgrRpcTests.cpp 56.71% <49.33%> (-8.66%) ⬇️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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.

4 participants