use a different way of removing the access for the temporary test account

[adamsitnik]

@danmosemsft Based on what I can see in Kusto my previous PR (#46213) has not solved the problem and this test has failed 140 times since the 25th of December.

I've changed the logic to use RemoveAccessRule instead of adding a new AccessControlType.Deny rule

fixes #46208

[ghost]

Tagging subscribers to this area: @eiriktsarpalis
See info in area-owners.md if you want to be subscribed.

Issue Details

---

@danmosemsft Based on what I can see in Kusto my previous PR (#46213) has not solved the problem and this test has failed 140 times since the 25th of December.

I've changed the logic to use RemoveAccessRule instead of adding a new AccessControlType.Deny rule

fixes #46208

Author:	adamsitnik
Assignees:	-
Labels:	area-System.Diagnostics.Process, test enhancement
Milestone:	6.0.0

[adamsitnik]

/azp run runtime-libraries-coreclr outerloop-windows

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[adamsitnik]

FWIW this is the only remaining unstable System.Diagnostics.Process test

TestResults 
| where Type startswith "System.Diagnostics.Tests"
| join kind=inner WorkItems on WorkItemId 
| where Finished >= datetime(2020-12-25 0:00:00)
| join kind=inner Jobs on JobId
| where Branch == "refs/heads/master"
| project Started, Duration, Method, QueueName, Message, StackTrace, Arguments, Type, QueueAlias, DockerTag, Type1, Result
| summarize count() by Message, Type, Method, Result, QueueName, StackTrace