Skip to content

test(smc): add unit tests for SMC-R networking functionality#891

Merged
l1b0k merged 1 commit intoAliyunContainerService:mainfrom
l1b0k:feat/ut
Sep 9, 2025
Merged

test(smc): add unit tests for SMC-R networking functionality#891
l1b0k merged 1 commit intoAliyunContainerService:mainfrom
l1b0k:feat/ut

Conversation

@l1b0k
Copy link
Collaborator

@l1b0k l1b0k commented Sep 8, 2025

  • Implement comprehensive unit tests for SMC-R networking features
  • Cover various scenarios including success and failure cases
  • Mock system commands and file operations for isolated testing
  • Use testify/assert for test assertions
  • Include tests for ensuring SMC-R support, configuring devices, and handling errors

- Implement comprehensive unit tests for SMC-R networking features
- Cover various scenarios including success and failure cases
- Mock system commands and file operations for isolated testing
- Use testify/assert for test assertions
- Include tests for ensuring SMC-R support, configuring devices, and handling errors

Signed-off-by: l1b0k <libokang.lbk@alibaba-inc.com>
@codecov
Copy link

codecov bot commented Sep 9, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 53.75%. Comparing base (c146cfb) to head (7f78358).
⚠️ Report is 7 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #891      +/-   ##
==========================================
+ Coverage   53.48%   53.75%   +0.27%     
==========================================
  Files         132      132              
  Lines       19523    19523              
==========================================
+ Hits        10441    10495      +54     
+ Misses       8024     7970      -54     
  Partials     1058     1058              
Flag Coverage Δ
unittests 53.75% <ø> (+0.27%) ⬆️

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@l1b0k l1b0k merged commit 84ebc2b into AliyunContainerService:main Sep 9, 2025
9 checks passed
@l1b0k l1b0k deleted the feat/ut branch September 9, 2025 12:42
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