Merge pull request #7801 from MabOneSdk/afs_dev

[RecoveryServices.Backup] Fixed modify policy for protected file share

Author: Maddie Clayton

src/ResourceManager/RecoveryServices/Commands.RecoveryServices.Backup.Providers/Providers/AzureFilesPsBackupProvider.cs

@@ -929,11 +929,6 @@ private void ValidateAzureFilesModifyProtectionRequest(ItemBase itemBase,
                 throw new ArgumentException(string.Format(Resources.InvalidProtectionPolicyException,
                                             typeof(AzureFileShareItem).ToString()));
             }
-
-            if (string.IsNullOrEmpty(((AzureFileShareItem)itemBase).ParentContainerFabricId))
-            {
-                throw new ArgumentException(Resources.ParentContainerFabricIdIsEmptyOrNull);
-            }
         }
     }
 }

src/ResourceManager/RecoveryServices/Commands.RecoveryServices.Backup.Test/ScenarioTests/AzureFiles/ItemTests.ps1

@@ -26,6 +26,7 @@ $folderPath = "pstestfolder1bca8f8e"
 $filePath = "pstestfolder1bca8f8e/pstestfile1bca8f8e.txt"
 $skuName="Standard_LRS"
 $policyName = "AFSBackupPolicy"
+$newPolicyName = "NewAFSBackupPolicy"
 
 # Setup Instructions:
 # 1. Create a resource group
@@ -171,7 +172,7 @@ function Test-AzureFSProtection
 
 		$policy = Get-AzureRmRecoveryServicesBackupProtectionPolicy `
 			-VaultId $vault.ID `
-			-WorkloadType AzureFiles
+			-Name $policyName
 
 		$enableJob = Enable-AzureRmRecoveryServicesBackupProtection `
 			-VaultId $vault.ID `
@@ -191,6 +192,26 @@ function Test-AzureFSProtection
 			-WorkloadType AzureFiles
 		Assert-True { $item.Name -contains $fileShareName }
 		Assert-True { $item.LastBackupStatus -eq "IRPending" }
+		Assert-True { $item.ProtectionPolicyName -eq $policyName }
+		
+		# Modify Policy
+		$newPolicy = Get-AzureRmRecoveryServicesBackupProtectionPolicy `
+		-VaultId $vault.ID `
+		-Name $newPolicyName
+
+		$enableJob =  Enable-AzureRmRecoveryServicesBackupProtection `
+			-VaultId $vault.ID `
+			-Policy $newPolicy `
+			-Item $item
+		
+		$item = Get-AzureRmRecoveryServicesBackupItem `
+		-VaultId $vault.ID `
+		-Container $container `
+		-WorkloadType AzureFiles
+
+		Assert-True { $item.Name -contains $fileShareName }
+		Assert-True { $item.LastBackupStatus -eq "IRPending" }
+		Assert-True { $item.ProtectionPolicyName -eq $newPolicyName }
 	}
 	finally
 	{