-
Notifications
You must be signed in to change notification settings - Fork 4.5k
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.
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
Make all credential types mockable #43959
Comments
@scottaddie: Can you help us understand for what the scenarios are where mocking |
@jsquire We have customers who want to test code that didn't accept/use the My bigger concern is that we're inconsistent across credential types. |
To be clear, I'm not opposed to the requested change, just trying to understand the scenario. |
While reading through #20911, I noticed that only some credential types in the Azure Identity library expose protected, parameterless constructors and are therefore mockable. This is a tracking issue to update the affected credential types accordingly, to adhere to Azure SDK for .NET mocking guidelines.
The text was updated successfully, but these errors were encountered: