Added help message.

Author: pikumarmsft16

src/ResourceManager/AzureBackup/Commands.AzureBackup/AzureBackupCmdletHelpMessage.cs

@@ -33,5 +33,9 @@ internal static class AzureBackupCmdletHelpMessage
         public const string Type = "Type of Azure Backup Item.";
         public const string Status = "Status of Azure Backup Item";
         public const string AzureBackupContainer = "Azure Backup Container for Azure Backup Item.";
+        public const string AzureBackupItem = "Azure Backup Item for enabling protection";
+        public const string RemoveProtectionOption = "Remove Protection Option";
+        public const string Reason = "Reson for removing protection";
+        public const string Comments = "Comments for for removing protection";
     }
 }

src/ResourceManager/AzureBackup/Commands.AzureBackup/AzureBackupItemCmdletBase.cs

@@ -30,14 +30,14 @@ public abstract class AzureBackupItemCmdletBase : AzureBackupCmdletBase
     {
         // ToDO:
         // Correct Help message and other attributes related to paameters
-        [Parameter(Position = 0, Mandatory = true, HelpMessage = AzureBackupCmdletHelpMessage.ResourceGroupName, ValueFromPipelineByPropertyName = true)]
+        [Parameter(Position = 0, Mandatory = true, HelpMessage = AzureBackupCmdletHelpMessage.AzureBackupItem, ValueFromPipelineByPropertyName = true)]
         [ValidateNotNullOrEmpty]
         public AzureBackupContainerContextObject Item { get; set; }
 
         public override void ExecuteCmdlet()
         {
             base.ExecuteCmdlet();
-            WriteDebug(String.Format("Cmdlet called for ResourceGroupName: {0}, ResourceName: {1}", Item.ResourceGroupName, Item.ResourceName));
+            WriteDebug(String.Format("Cmdlet called for ResourceGroupName: {0}, ResourceName: {1}, Location: {2}", Item.ResourceGroupName, Item.ResourceName, Item.Location));
             InitializeAzureBackupCmdlet(Item.ResourceGroupName, Item.ResourceName, Item.Location);
         }
     }

src/ResourceManager/AzureBackup/Commands.AzureBackup/Cmdlets/DataSource/Disable-AzureBackupProtection .cs

@@ -22,18 +22,30 @@ namespace Microsoft.Azure.Commands.AzureBackup.Cmdlets.DataSource
     [Cmdlet(VerbsLifecycle.Disable, "AzureBackupProtection"), OutputType(typeof(OperationResponse))]
     public class DisableAzureBackupProtection : AzureBackupDSCmdletBase
     {
-        [Parameter(Mandatory = false, HelpMessage = AzureBackupCmdletHelpMessage.PolicyName)]
+        [Parameter(Position = 1, Mandatory = false, HelpMessage = AzureBackupCmdletHelpMessage.RemoveProtectionOption)]
         [ValidateSet("Invalid", "RetainBackupData", "DeleteBackupData")] 
         public string RemoveProtectionOption { get; set; }
+
+        [Parameter(Position = 2, Mandatory = false, HelpMessage = AzureBackupCmdletHelpMessage.Reason)]
+        public string Reason { get; set; }
+
+        [Parameter(Position = 3, Mandatory = false, HelpMessage = AzureBackupCmdletHelpMessage.Comments)]
+        public string Comments { get; set; }
+
         public override void ExecuteCmdlet()
         {
             base.ExecuteCmdlet();
 
             ExecutionBlock(() =>
             {
                 WriteVerbose("Making client call");
-                RemoveProtectionRequestInput input = new RemoveProtectionRequestInput();
-                input.RemoveProtectionOption = this.RemoveProtectionOption == null ? "RetainBackupData" : this.RemoveProtectionOption;
+                RemoveProtectionRequestInput input = new RemoveProtectionRequestInput()
+                {
+                    RemoveProtectionOption = this.RemoveProtectionOption == null ? "RetainBackupData" : this.RemoveProtectionOption,
+                    Reason = this.Reason,
+                    Comments = this.Comments,
+                };
+
                 WriteVerbose("RemoveProtectionOption = " + input.RemoveProtectionOption);
                 var disbaleAzureBackupProtection = AzureBackupClient.DataSource.DisableProtectionAsync(GetCustomRequestHeaders(), item.ContainerUniqueName, item.Type, item.DataSourceId, input, CmdletCancellationToken).Result;
 

src/ResourceManager/AzureBackup/Commands.AzureBackup/Cmdlets/DataSource/Enable-AzureBackupProtection .cs

@@ -63,7 +63,7 @@ public override void ExecuteCmdlet()
                 }
                 else
                 {
-
+                    throw new Exception("Uknown item type");
                 }
 
                 var enableAzureBackupProtection = AzureBackupClient.DataSource.EnableProtectionAsync(GetCustomRequestHeaders(), input, CmdletCancellationToken).Result;

src/ResourceManager/AzureBackup/Commands.AzureBackup/Cmdlets/DataSource/GetAzureBackupItem.cs

@@ -92,15 +92,13 @@ public void WriteAzureBackupProtectionPolicy(List<DataSourceInfo> sourceDataSour
 
             foreach (var item in sourceDataSourceList)
             {
-                WriteVerbose("Type of DS is = " + item.Type);
                 targetList.Add(new AzureBackupItem(item, azureBackupContainer));
             }
 
             if (sourcePOList != null)
             {
                 foreach (var item in sourcePOList)
                 {
-                    WriteVerbose("Type of PO is = " + item.Type);
                     //If user has stopped protection for some datasoure then we will have duplicate items(po and ds).
                     //So in this case removing  poitems from the list.
                     if (targetList.FindIndex(element => element.Name == item.Name) < 0)