Azure / Azure/azure-powershell
Remove dependency of "Microsoft.Azure.Management.Storage" from 4 module test.
- Dominant language
- C#
- Stars
- 4.8k
- Forks
- 4.3k
- Avg merge
- 3d 14h
- Merged PRs (30d)
- 54
Description
## Description
Currently there are 4 modules test reference "Microsoft.Azure.Management.Storage" besides Az.Storage.
- src\ApplicationInsights\ApplicationInsights.Test\ApplicationInsights.Test.csproj
- src\EventGrid\EventGrid.Test\EventGrid.Test.csproj
- src\Synapse\Synapse.Test\Synapse.Test.csproj
- src\OperationalInsights\OperationalInsights.Test\OperationalInsights.Test.csproj
Other module (except Az.Storage) should not reference "Microsoft.Azure.Management.Storage", or we will have unnecessary dependency, and it will take trouble when Az.Storage upgrade "Microsoft.Azure.Management.Storage": other module test need upgrade or even re-record.
Please follow Az.Compute Test, to handle storage account management with ` \tools\ScenarioTest.ResourceManager\AzureRM.Storage.ps1`, and remove "Microsoft.Azure.Management.Storage" reference.
In the future, all new added module should not reference "Microsoft.Azure.Management.Storage".
## Cost
NA
Contributor guide
Research direction
Start with the four listed test project files and compare their storage-account handling with the Az.Compute Test approach. Read tools\ScenarioTest.ResourceManager\AzureRM.Storage.ps1, remove the Microsoft.Azure.Management.Storage references from the four projects, and verify that the tests use the shared script instead.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- csharp, powershell
- Domain
- testing-qa
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Clearly specified
- Newbie friendliness
- 45/100