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

Fix the issue when running the unit tests #18300

Merged
merged 2 commits into from
May 31, 2022
Merged

Fix the issue when running the unit tests #18300

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
ba91517
Fix the issue in unit tests.
kceiw May 25, 2022
4f492bd
Test
kceiw May 28, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions tools/Az.Tools.Predictor/Az.Tools.Predictor.Test/Az.Tools.Predictor.Test.csproj
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,11 @@
<PrivateAssets>all</PrivateAssets>
</PackageReference>
<ProjectReference Include="..\Az.Tools.Predictor\Az.Tools.Predictor.csproj" />
<!-- The test project doesn't reference to it directly. Az.Tools.Predictor has a reference to Microsoft.PowerShell.SDK.
- But it's excluding content files to reduce the size of the module. The content files are not needed since they're in the runtime (PowerShell)
- The content files are needed to set up the runspace to parse the command line. So we need to include them when we run the unit tests.
-->
<PackageReference Include="Microsoft.PowerShell.SDK" Version="7.2.0" />
</ItemGroup>

<ItemGroup>
Expand Down
1 change: 0 additions & 1 deletion tools/Az.Tools.Predictor/Az.Tools.Predictor/Az.Tools.Predictor.csproj
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,6 @@ For more information on Az Predictor, please visit the following: https://aka.ms
<PackageReference Include="Microsoft.ApplicationInsights" Version="2.18.0" />
<PackageReference Include="Microsoft.Azure.PowerShell.Common.Share" Version="1.3.45-preview" />
<PackageReference Include="Microsoft.PowerShell.SDK" Version="7.2.0">
<ExcludeAssets>contentFiles</ExcludeAssets>
</PackageReference>
<PackageReference Include="System.Management.Automation" Version="7.2.0">
<ExcludeAssets>contentFiles</ExcludeAssets>
Expand Down