update removeroleassignment comparison to case insensitive

darshanhs90 · darshanhs90 · commit 9007c2de591c · 2017-07-20T16:07:25.000-07:00
diff --git a/src/ResourceManager/Resources/Commands.Resources/Models.Authorization/AuthorizationClient.cs b/src/ResourceManager/Resources/Commands.Resources/Models.Authorization/AuthorizationClient.cs
@@ -476,7 +476,7 @@ public IEnumerable<PSRoleAssignment> RemoveRoleAssignment(FilterRoleAssignmentsO
             // but an edge case can have multiple role assignments to the same role or multiple role assignments to different roles, with same name.
             // The FilterRoleAssignments takes care of paging internally
             IEnumerable<PSRoleAssignment> roleAssignments = FilterRoleAssignments(options, currentSubscription: subscriptionId)
-                                                .Where(ra => ra.Scope == options.Scope.TrimEnd('/'));
+                                                .Where(ra => ra.Scope.Equals(options.Scope.TrimEnd('/'), StringComparison.OrdinalIgnoreCase));
 
             if (roleAssignments == null || !roleAssignments.Any())
             {

Original file line number	Diff line number	Diff line change
`@@ -476,7 +476,7 @@ public IEnumerable<PSRoleAssignment> RemoveRoleAssignment(FilterRoleAssignmentsO`
`476`	`476`	`// but an edge case can have multiple role assignments to the same role or multiple role assignments to different roles, with same name.`
`477`	`477`	`// The FilterRoleAssignments takes care of paging internally`
`478`	`478`	`IEnumerable<PSRoleAssignment> roleAssignments = FilterRoleAssignments(options, currentSubscription: subscriptionId)`
`479`		`- .Where(ra => ra.Scope == options.Scope.TrimEnd('/'));`
	`479`	`+ .Where(ra => ra.Scope.Equals(options.Scope.TrimEnd('/'), StringComparison.OrdinalIgnoreCase));`
`480`	`480`
`481`	`481`	`if (roleAssignments == null \|\| !roleAssignments.Any())`
`482`	`482`	`{`