Changes as per review comments

Author: Gautam Sheth

documentation/Remove-PnPFlow.md

@@ -20,7 +20,7 @@ Removes the specified flow.
 
 ```powershell
 Remove-PnPFlow -Environment <PowerAutomateEnvironmentPipeBind> -Identity <PowerAutomateFlowPipeBind> [-AsAdmin]
- [-Force] [-Connection <PnPConnection>] 
+ [-Force] [-ThrowExceptionIfPowerAutomateNotFound] [-Connection <PnPConnection>] 
 ```
 
 ## DESCRIPTION
@@ -36,6 +36,14 @@ Remove-PnPFlow -Environment $environment -Identity fba63225-baf9-4d76-86a1-1b42c
 
 This removes the specified flow.
 
+### Example 1
+```powershell
+$environment = Get-PnPFlowEnvironment
+Remove-PnPFlow -Environment $environment -Identity fba63225-baf9-4d76-86a1-1b42c917a182 -ThrowExceptionIfPowerAutomateNotFound
+```
+
+This removes the specified flow and throws an exception if the specified flow is not present.
+
 ## PARAMETERS
 
 ### -AsAdmin
@@ -99,6 +107,20 @@ Accept pipeline input: False
 Accept wildcard characters: False
 ```
 
+### -ThrowExceptionIfPowerAutomateNotFound
+Switch parameter if an exception should be thrown if the requested flow does not exist (true) or if omitted, nothing will be returned in case the flow does not exist
+
+```yaml
+Type: SwitchParameter
+Parameter Sets: (All)
+
+Required: False
+Position: Named
+Default value: None
+Accept pipeline input: False
+Accept wildcard characters: False
+```
+
 ### -Force
 If specified, no confirmation question will be asked.
 

src/Commands/PowerPlatform/PowerAutomate/RemoveFlow.cs

@@ -18,6 +18,9 @@ public class RemoveFlow : PnPAzureManagementApiCmdlet
         [Parameter(Mandatory = false)]
         public SwitchParameter AsAdmin;
 
+        [Parameter(Mandatory = false)]
+        public SwitchParameter ThrowExceptionIfPowerAutomateNotFound;
+
         [Parameter(Mandatory = false)]
         public SwitchParameter Force;
 
@@ -28,21 +31,29 @@ protected override void ExecuteCmdlet()
 
             if (Force || ShouldContinue($"Remove flow with name '{flowName}'?", "Remove flow"))
             {
-                try
+                WriteVerbose($"Attempting to delete Flow with name {flowName}");
+                if (ThrowExceptionIfPowerAutomateNotFound)
                 {
-                    // Had to add this because DELETE doesn't throw error if invalid Flow Id or Name is provided
-                    WriteVerbose($"Retrieving Flow with name {flowName} in environment ${environmentName}");
-                    var result = GraphHelper.GetAsync<Model.PowerPlatform.PowerAutomate.Flow>(Connection, $"https://management.azure.com/providers/Microsoft.ProcessSimple{(AsAdmin ? "/scopes/admin" : "")}/environments/{environmentName}/flows/{flowName}?api-version=2016-11-01", AccessToken).GetAwaiter().GetResult();
-                    if (result != null)
+                    try
+                    {
+                        // Had to add this because DELETE doesn't throw error if invalid Flow Id or Name is provided
+                        WriteVerbose($"Retrieving Flow with name {flowName} in environment ${environmentName}");
+                        var result = GraphHelper.GetAsync<Model.PowerPlatform.PowerAutomate.Flow>(Connection, $"https://management.azure.com/providers/Microsoft.ProcessSimple{(AsAdmin ? "/scopes/admin" : "")}/environments/{environmentName}/flows/{flowName}?api-version=2016-11-01", AccessToken).GetAwaiter().GetResult();
+                        if (result != null)
+                        {
+                            RestHelper.DeleteAsync(Connection.HttpClient, $"https://management.azure.com/providers/Microsoft.ProcessSimple{(AsAdmin ? "/scopes/admin" : "")}/environments/{environmentName}/flows/{flowName}?api-version=2016-11-01", AccessToken).GetAwaiter().GetResult();
+                            WriteVerbose($"Flow with name {flowName} deleted");
+                        }
+                    }
+                    catch
                     {
-                        WriteVerbose($"Attempting to delete Flow with name {flowName}");
-                        RestHelper.DeleteAsync(Connection.HttpClient, $"https://management.azure.com/providers/Microsoft.ProcessSimple{(AsAdmin ? "/scopes/admin" : "")}/environments/{environmentName}/flows/{flowName}?api-version=2016-11-01", AccessToken).GetAwaiter().GetResult();
-                        WriteVerbose($"Flow with name {flowName} deleted");
+                        throw new Exception($"Cannot find flow with Identity '{flowName}'");
                     }
                 }
-                catch
+                else
                 {
-                    throw new Exception($"Cannot find flow with Identity '{flowName}'");
+                    RestHelper.DeleteAsync(Connection.HttpClient, $"https://management.azure.com/providers/Microsoft.ProcessSimple{(AsAdmin ? "/scopes/admin" : "")}/environments/{environmentName}/flows/{flowName}?api-version=2016-11-01", AccessToken).GetAwaiter().GetResult();
+                    WriteVerbose($"Flow with name {flowName} deleted");
                 }
             }
         }