Azure / Azure/azure-powershell

Remove dependency of "Microsoft.Azure.Management.Storage" from 4 module test.

Open
#11,891 8 comments 0 reactions 0 assignees View on GitHub
Azure PS Team Engineering
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

Open the contributing 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.