Skip to content

Improve UT Coverage#83

Merged
suryagupta4 merged 5 commits into
mainfrom
usr/suryagupta4/ut-coverage
Feb 27, 2025
Merged

Improve UT Coverage#83
suryagupta4 merged 5 commits into
mainfrom
usr/suryagupta4/ut-coverage

Conversation

@suryagupta4

@suryagupta4 suryagupta4 commented Feb 25, 2025

Copy link
Copy Markdown
Contributor

Description

Improve UT Coverage to 90%

GitHub Issues

List the GitHub issues impacted by this PR:

GitHub Issue #
https://github.com/dell/csm/issues/1707

Checklist:

  • I have performed a self-review of my own code to ensure there are no formatting, vetting, linting, or security issues
  • I have verified that new and existing unit tests pass locally with my changes
  • I have not allowed coverage numbers to degenerate
  • I have maintained at least 90% code coverage
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • Backward compatibility is not broken

Sanity:

Executed cert-csi suite:
image

@suryagupta4 suryagupta4 changed the title Improve UT Coverage [WIP] Improve UT Coverage Feb 25, 2025
@suryagupta4 suryagupta4 marked this pull request as ready for review February 26, 2025 11:17
@suryagupta4 suryagupta4 changed the title [WIP] Improve UT Coverage Improve UT Coverage Feb 26, 2025
@suryagupta4 suryagupta4 self-assigned this Feb 27, 2025
@suryagupta4 suryagupta4 merged commit b336b65 into main Feb 27, 2025
@suryagupta4 suryagupta4 deleted the usr/suryagupta4/ut-coverage branch February 27, 2025 07:13
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