Issue 487 - Expose volume letter in "Get-RubrikVolumeGroup"

[jaapbrasser]

Description

Add the associated protected volume letter to a VolumeGroup ID in the Get-RubrikVolumeGroup CMDlet

Related Issue

Resolve #487

Motivation and Context

The default display does not display any associated volumes, this is the default in the GUI interface and would be interesting data to display. In order to resolve this I have done the following:

• Added -DetailedObject parameter, as is present in other Get-Rubrik* functions
• Updated documentation
• Updated default display to in the 'includes' field

How Has This Been Tested?

Existing unit tests run, new unit tests have been created. Ran the code against the TME test clusters.

Screenshots (if appropriate):

Types of changes

What types of changes does your code introduce? Put an x in all the boxes that apply:

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

Go over all the following points, and put an x in all the boxes that apply. If you're unsure about any of these, don't hesitate to ask. We're here to help!

• My code follows the code style of this project.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have read the CONTRIBUTION document.
• I have updated the CHANGELOG file accordingly for the version that this merge modifies.
• I have added tests to cover my changes.
• All new and existing tests passed.

[jaapbrasser]

No review required yet, working out some issues with this at the moment

[jaapbrasser]

Ready for review

[shamsway]

Tested against CDM 5.0.2 - working as expected

There are some failing unit tests for Get-RubrikVolumeGroup.Tests.ps1, but this may be a false positive

[jaapbrasser]

Interesting, yea seems the Windows PowerShell tests behave differently from the PowerShell unit tests, I'll resolve this before we can merge

[jaapbrasser]

@shamsway resolved issues and resubmitted for review

[mwpreston]

Looks good @jaapbrasser .

Submit-Requests tests are failing when run in core - but that's happening with my current branch as well so unrelated :)