Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Resources: Tests using static RoleDefinitionNames, cannot run in parallelization #6067

Closed
1 task
MiYanni opened this issue Apr 28, 2018 · 1 comment
Closed
1 task

Resources: Tests using static RoleDefinitionNames, cannot run in parallelization #6067

MiYanni opened this issue Apr 28, 2018 · 1 comment
Assignees
@ravbhatnagar
Labels
ARM NetCore Service Attention This issue is responsible by Azure service team.

Comments

@MiYanni
Copy link
Contributor

MiYanni commented Apr 28, 2018 •
edited
Loading

In .NET Core, the Xunit flag [assembly: CollectionBehavior(DisableTestParallelization = true)] does not function (referenced here and here). So, tests always run in parallel. All RoleDefinition tests and 1 RoleAssignment test (RaPropertiesValidation) use RoleDefinitionNames, a static queue, at the same time. So, these tests fail intermittently during runs on .NET Core.

  • Refactor the RoleDefinition and RoleAssignment->RaPropertiesValidation scenario tests so they do not use the static RoleDefinitionNames queue
This was referenced Aug 1, 2018
Closed
Merged
@cormacpayne
Copy link
Member

This has been fixed.

@bsiegel bsiegel added the Service Attention This issue is responsible by Azure service team. label Sep 26, 2018
@mozehgir mozehgir added the ARM label Aug 14, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ravbhatnagar ravbhatnagar

Labels
ARM NetCore Service Attention This issue is responsible by Azure service team.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
6 participants
@bsiegel @ravbhatnagar @vivsriaus @cormacpayne @MiYanni @mozehgir