[AppService]: fix #5833 Edit-AzureRmWebAppBackupConfiguration always sets Enabled to True (#21221)

* [AppService]: fix #5833 Edit-AzureRmWebAppBackupConfiguration always sets Enabled to True

* Fixed build issues

* Updated Synopsis

---------

Co-authored-by: Yabo Hu <[email protected]>

Author: Kotasudhakarreddy

src/Websites/Websites.Test/ScenarioTests/WebAppBackupRestoreTests.ps1

@@ -223,7 +223,7 @@ function Test-EditAndGetWebAppBackupConfiguration
             -ResourceGroupName $rgName -Name $wName -StorageAccountUrl $sasUri `
             -FrequencyInterval $frequencyInterval -FrequencyUnit $frequencyUnit `
             -RetentionPeriodInDays $retentionPeriod -StartTime $startTime `
-            -KeepAtLeastOneBackup 
+            -KeepAtLeastOneBackup -Enabled
 
         # Assert
         Assert-True { $config.Enabled }

src/Websites/Websites.Test/SessionRecords/Microsoft.Azure.Commands.Websites.Test.ScenarioTests.WebAppBackupRestoreTests/TestEditAndGetWebAppBackupConfiguration.json

@@ -18,6 +18,7 @@
         - Additional information about change #1
 -->
 ## Upcoming Release
+* Fixed `Edit-AzWebAppBackupConfiguration` to pass backup configuration enabled or not
 * Added a new parametre `-SoftRestart` for `Restart-AzWebApp` and `Restart-AzWebApp` to perform a soft restart
 * Updated `Get-AzWebApp` and `Get-AzWebAppSlot` to expose `VirtualNetwork Integration Info` [#10665]
 * Added a new parameter `-CopyIdentity` for `New-AzWebAppSlot` to copy the identity from the parent app to the slot.

src/Websites/Websites.Test/SessionRecords/Microsoft.Azure.Commands.Websites.Test.ScenarioTests.WebAppBackupRestoreTests/TestEditAndGetWebAppBackupConfigurationPiping.json

@@ -49,6 +49,9 @@ public class EditAzureWebAppBackupConfiguration : WebAppOptionalSlotBaseCmdlet
         [Parameter(Position = 9, Mandatory = false, HelpMessage = "True if one backup should always be kept in the storage account, regardless of how old it is.", ValueFromPipelineByPropertyName = true)]
         public SwitchParameter KeepAtLeastOneBackup { get; set; }
 
+        [Parameter(Position = 10, Mandatory = false, HelpMessage = "To enable backup configuration.", ValueFromPipelineByPropertyName = true)]
+        public SwitchParameter Enabled { get; set; }
+
         public override void ExecuteCmdlet()
         {
             base.ExecuteCmdlet();
@@ -59,7 +62,7 @@ public override void ExecuteCmdlet()
                 RetentionPeriodInDays, StartTime);
             BackupRequest request = new BackupRequest()
             {
-                Enabled = true,
+                Enabled = this.Enabled.IsPresent,
                 StorageAccountUrl = this.StorageAccountUrl,
                 BackupSchedule = schedule,
                 Databases = this.Databases,
@@ -76,7 +79,7 @@ public override void ExecuteCmdlet()
                 StartTime = this.StartTime,
                 KeepAtLeastOneBackup = this.KeepAtLeastOneBackup.IsPresent,
                 Databases = this.Databases,
-                Enabled = true
+                Enabled = this.Enabled.IsPresent
             };
             WriteObject(config);
         }

src/Websites/Websites/ChangeLog.md

@@ -9,21 +9,22 @@ schema: 2.0.0
 # Edit-AzWebAppBackupConfiguration
 
 ## SYNOPSIS
+Edits the current configuration backup for an Azure Web App
 
 ## SYNTAX
 
 ### FromResourceName
 ```
 Edit-AzWebAppBackupConfiguration [-FrequencyInterval] <Int32> [-FrequencyUnit] <String>
- [-RetentionPeriodInDays] <Int32> [[-StartTime] <DateTime>] [-KeepAtLeastOneBackup]
+ [-RetentionPeriodInDays] <Int32> [[-StartTime] <DateTime>] [-KeepAtLeastOneBackup] [Enabled]
  [-ResourceGroupName] <String> [-Name] <String> [[-Slot] <String>] [-DefaultProfile <IAzureContextContainer>]
  [-StorageAccountUrl] <String> [[-Databases] <DatabaseBackupSetting[]>] [<CommonParameters>]
 ```
 
 ### FromWebApp
 ```
 Edit-AzWebAppBackupConfiguration [-FrequencyInterval] <Int32> [-FrequencyUnit] <String>
- [-RetentionPeriodInDays] <Int32> [[-StartTime] <DateTime>] [-KeepAtLeastOneBackup] [-WebApp] <PSSite>
+ [-RetentionPeriodInDays] <Int32> [[-StartTime] <DateTime>] [-KeepAtLeastOneBackup] [Enabled] [-WebApp] <PSSite>
  [-DefaultProfile <IAzureContextContainer>] [-StorageAccountUrl] <String>
  [[-Databases] <DatabaseBackupSetting[]>] [<CommonParameters>]
 ```
@@ -74,6 +75,21 @@ Accept pipeline input: False
 Accept wildcard characters: False
 ```
 
+### -Enabled
+Keep At Least One Backup Option
+
+```yaml
+Type: System.Management.Automation.SwitchParameter
+Parameter Sets: (All)
+Aliases:
+
+Required: False
+Position: 10
+Default value: None
+Accept pipeline input: True (ByPropertyName)
+Accept wildcard characters: False
+```
+
 ### -FrequencyInterval
 Frequency Interval