Add-DbaDbRoleMember - Add support for adding roles to roles

[andreasjordan]

Type of Change

• Bug fix (non-breaking change, fixes Add-DbaDbRoleMember - Unable to Add Role #9302 )
• New feature (non-breaking change, adds functionality, fixes # )
• Breaking change (affects multiple commands or functionality, fixes # )
• Ran manual Pester test and has passed (.\tests\manual.pester.ps1)
• Adding code coverage to existing functionality
• Pester test is included
• If new file reference added for test, has is been added to github.com/dataplat/appveyor-lab ?
• Unit test is included
• Documentation
• Build system

See issue for details.

[andreasjordan]

We have to change Get-DbaDbRoleMember as well. Will work on this in the next days.

[andreasjordan]

I added them member roles to Get-DbaDbRoleMember in a way that this is a non-breaking change. But I would prefer changing the output in a way that we don't use "user" or "role" but "member". But that would be a breaking change. Maybe planning that for the next major version?

[mattcargile]

The code looks good. Thanks!

Would love to see DbaServerRoleMember cmdlets get the same treatment on the parameters. Might involve more breaking changes.

[andreasjordan]

Let's first have @potatoqualitee have a look at this PR. Then we can talk about breaking changes.

[potatoqualitee]

thank you for the alias! @ClaudioESSilva can you pls take a look and approve?

[wsmelton]

Just rewording suggesting...but LGTM

[potatoqualitee]

oh, good. thanks everyone 😊