Merge pull request Azure#5095 from darshanhs90/preview2

Fix ServicePrincipalName misnaming in new-azurermroleassignment command

Author: cormacpayne

src/ResourceManager/Resources/ChangeLog.md

@@ -29,6 +29,8 @@
 * Allow users to create RoleAssignment using delegation flag
     - Users can now use New-AzureRMRoleAssignment with an option of adding the delegation flag
 * Fix RoleAssignment get to respect the scope parameter
+* Add an alias for ServicePrincipalName in the New-AzureRmRoleAssignment Commandlet
+    - Users can now use the ApplicationId instead of the ServicePrincipalName when using the New-AzureRmRoleAssignment commandlet
 
 ## Version 5.0.0
 * Add support for online help

src/ResourceManager/Resources/Commands.Resources.Test/ScenarioTests/RoleAssignmentTests.ps1

@@ -255,10 +255,19 @@ function Test-RaByServicePrincipal
                         -RoleDefinitionName $definitionName `
                         -Scope $scope 
 
-    
     # cleanup 
     DeleteRoleAssignment $newAssignment
 
+    # Test
+    [Microsoft.Azure.Commands.Resources.Models.Authorization.AuthorizationClient]::RoleAssignmentNames.Enqueue("0b018870-59ba-49ca-9405-9ba5dce77311")
+    $newAssignment = New-AzureRmRoleAssignment `
+                        -ApplicationId $servicePrincipals[0].ServicePrincipalNames[0] `
+                        -RoleDefinitionName $definitionName `
+                        -Scope $scope 
+                        
+    # cleanup 
+    DeleteRoleAssignment $newAssignment
+
     # Assert
     Assert-NotNull $newAssignment
     Assert-AreEqual $definitionName $newAssignment.RoleDefinitionName