rajshekarp87 commented 7 months ago

Description

A few sentences describing the overall goals of the pull request's commits.

GitHub Issues

List the GitHub issues impacted by this PR:

GitHub Issue #

ISSUE TYPE

Bugfix Pull Request
Docs Pull Request
Feature Pull Request
Test Pull Request

COMPONENT NAME

OUTPUT

ADDITIONAL INFORMATION

Checklist:

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

ansible-collections-svc commented 7 months ago

Can one of the admins verify this patch?

codecov-commenter commented 7 months ago

Codecov Report

Attention: Patch coverage is 62.86765% with 101 lines in your changes are missing coverage. Please review.

Project coverage is 93.16%. Comparing base (abde3b5) to head (05ae11e). Report is 1 commits behind head on collections.

Files	Patch %	Lines
plugins/module_utils/session_utils.py	33.33%	64 Missing :warning:
plugins/modules/idrac_session.py	58.88%	34 Missing and 3 partials :warning:

Additional details and impacted files

```diff @@ Coverage Diff @@ ## collections #640 +/- ## =============================================== - Coverage 93.40% 93.16% -0.25% =============================================== Files 197 200 +3 Lines 33421 33693 +272 Branches 5139 5170 +31 =============================================== + Hits 31216 31389 +173 - Misses 1337 1432 +95 - Partials 868 872 +4 ``` | [Flag](https://app.codecov.io/gh/dell/dellemc-openmanage-ansible-modules/pull/640/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dell) | Coverage Δ | | |---|---|---| | [units](https://app.codecov.io/gh/dell/dellemc-openmanage-ansible-modules/pull/640/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dell) | `93.16% <62.86%> (-0.25%)` | :arrow_down: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dell#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.