Jennifer-John commented 6 months ago

Description

Add support for resource group provisioning to create, validate and delete a resource group deployment

GitHub Issues

List the GitHub issues impacted by this PR:

GitHub Issue #

Checklist:

[X] I have performed a self-review of my own code to ensure there are no formatting, pep8, linting, or security issues
[X] I have performed Ansible Sanity test using --docker default
[] 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
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have made corresponding changes to the documentation
[] I have added tests that prove my fix is effective or that my feature works
[X] Backward compatibility is not broken

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Please also list any relevant details for your test configuration

[X] Functional test

codecov-commenter commented 6 months ago

Codecov Report

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

Project coverage is 91.71%. Comparing base (ce855f1) to head (7ed44a5). Report is 12 commits behind head on main.

Files	Patch %	Lines
plugins/modules/resource_group.py	82.29%	15 Missing and 22 partials :warning:

Additional details and impacted files

```diff @@ Coverage Diff @@ ## main #42 +/- ## ========================================== - Coverage 91.85% 91.71% -0.14% ========================================== Files 46 49 +3 Lines 6481 6870 +389 Branches 810 855 +45 ========================================== + Hits 5953 6301 +348 - Misses 297 316 +19 - Partials 231 253 +22 ``` | [Flag](https://app.codecov.io/gh/dell/ansible-powerflex/pull/42/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/ansible-powerflex/pull/42/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dell) | `91.71% <89.86%> (-0.14%)` | :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.