`FabricRecoveryPoint` functions need alignments to module's pattern

[NowinskiK]

Problem description

It's about a few functions:

• Get-FabricRecoveryPoint
• New-FabricRecoveryPoint
• Restore-FabricRecoveryPoint
• Remove-FabricRecoveryPoint

I understand they were moved into the project as-is, so now is a good time to start aligning them with the patterns we use in other functions.
It doesn’t have to be a complete refactor right away, but even partial alignment would be a good step forward.

Verbose logs

n/a

Module Version

develop branch

Suggested solution

1. Recovery Points are related to the Warehouse in Fabric? If so, (sorry, I'm not sure) maybe we should consider renaming the functions by adding Datawarehouse: Get-FabricDatawarehouseRecoveryPoint or Get-FabricWarehouseRecoveryPoint
2. There are no mandatory/non-mandatory attributes for input parameters
3. Delete Get-FabricUri and use Invoke-FabricRestMethod instead
4. Formatting: sometimes no spaces after if
5. Input parameters should be *Id rather than *GUID
6. WorkspaceId & DataWarehouseId should be mandatory
7. Switch param Wait should be replaced by NoWait
8. .SYNOPSIS & .DESCRIPTION are the same. The latter could be more descriptive.
9. Use Confirm-TokenState at the beginning of each function

[jpomfret]

Lots of good comments on here @NowinskiK - agreed they need some alignment - I'm going to unassign myself for now as I have 0 bandwidth for the next couple of weeks. Cheers

[NowinskiK]

Sure, not a problem at all. It was an obvious choice for me, as you were the author and know it the best.
Enjoy the time outside of the project and any conference you'll join soon!