Skip to content

Add install, update gate tests #7265

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

Open
Nigusu-Allehu wants to merge 9 commits into
base: dev
Choose a base branch
from
Open

Add install, update gate tests #7265

Show file tree
Hide file tree
Changes from 7 commits
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
dec65a3
Add search, install, uninstall gate tests
Nigusu-Allehu Apr 8, 2026
09ef498
Ad packagereference tests
Nigusu-Allehu Apr 8, 2026
77bb359
bring back
Nigusu-Allehu Apr 8, 2026
b098fbd
test
Nigusu-Allehu Apr 8, 2026
0995a8f
net core
Nigusu-Allehu Apr 9, 2026
d7afcf1
remove
Nigusu-Allehu Apr 22, 2026
255c4a4
use daily
Nigusu-Allehu Apr 22, 2026
2fb4bc0
Move tests to daily
Nigusu-Allehu Apr 22, 2026
8cdde95
remove
Nigusu-Allehu Apr 22, 2026
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
2 changes: 1 addition & 1 deletion build/runsettings.proj
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@
<MSBuild
Projects="template.runsettingsproj"
Properties="RunName=NuGet.Tests.Apex.Gate;
FileName=NuGet.Tests.Apex.dll;
FileName=NuGet.Tests.Apex.dll;NuGet.Tests.Apex.Daily.dll;
TestCaseFilter=TestCategory=Gate;" />

<!-- NuGet.Tests.Apex.Daily.runsettings -->
Expand Down
2 changes: 2 additions & 0 deletions test/NuGet.Tests.Apex/NuGet.Tests.Apex.Daily/NuGetUITestCase.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -568,6 +568,7 @@ public async Task SearchTopLevelAndTransitivePackagePublicFieldInInstalledTabFro

[TestMethod]
[Timeout(DefaultTimeout)]
[TestCategory("Gate")]
public async Task InstallTopLevelPackageFromUI()
Copy link
Copy Markdown
Member

@nkolev92 nkolev92 Apr 22, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How did you pick this one?

There's an overlap in the scenario in this method and the one from the regular apex tests.

Copy link
Copy Markdown
Member Author

@Nigusu-Allehu Nigusu-Allehu Apr 22, 2026 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This one is Net core, and the other ones are packages config.

Copy link
Copy Markdown
Member

@nkolev92 nkolev92 Apr 22, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So this test case is from the apex daily pipeline, which is not even something we see in our CI right now. It's supposed to be additional tests.

I'm assuming there's probably a net core version of this scenario in the regular apex tests.

Copy link
Copy Markdown
Member Author

@Nigusu-Allehu Nigusu-Allehu Apr 22, 2026 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe I missed it, but I didn't find any focused(simple install/restore) net core tests in the per commit tests. However there are PSM net core tests. Maybe we could make use of them. I will take a look.

Copy link
Copy Markdown
Member

@nkolev92 nkolev92 Apr 22, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wow, that's surprising.
We can keep this one then for simplicity. We can always move things around I guess.
Maybe we move that test into the regular pipeline since you've already determined it's stable?

{
// Arrange
Expand Down Expand Up @@ -598,6 +599,7 @@ public async Task InstallTopLevelPackageFromUI()

[TestMethod]
[Timeout(DefaultTimeout)]
[TestCategory("Gate")]
Copy link
Copy Markdown
Member

@nkolev92 nkolev92 Apr 22, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Similarly here, this tests the PM UI Top Level and Transitive Package differentiation.

I don't think that's something likely to be broken by partner teams, so I don't think this is a relevant test.

public async Task UninstallTopLevelPackageFromUI()
{
// Arrange
Expand Down
1 change: 0 additions & 1 deletion test/NuGet.Tests.Apex/NuGet.Tests.Apex/NuGetEndToEndTests/IVsServicesTestCase.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,6 @@ public class IVsServicesTestCase : SharedVisualStudioHostTestClass

[TestMethod]
[Timeout(LongerTimeout)]
[TestCategory("Gate")]
public void SimpleInstallFromIVsInstaller()
{
// Arrange
Expand Down
3 changes: 3 additions & 0 deletions test/NuGet.Tests.Apex/NuGet.Tests.Apex/NuGetEndToEndTests/NuGetUITestCase.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -49,6 +49,7 @@ public async Task SearchPackageFromUI()

[TestMethod]
[Timeout(DefaultTimeout)]
[TestCategory("Gate")]
public async Task InstallPackageFromUI()
{
// Arrange
Expand Down Expand Up @@ -107,6 +108,7 @@ public async Task InstallPackageToProjectsFromUI()

[TestMethod]
[Timeout(DefaultTimeout)]
[TestCategory("Gate")]
public async Task UninstallPackageFromUI()
{
// Arrange
Expand Down Expand Up @@ -140,6 +142,7 @@ public async Task UninstallPackageFromUI()

[TestMethod]
[Timeout(DefaultTimeout)]
[TestCategory("Gate")]
public async Task UpdatePackageFromUI()
{
// Arrange
Expand Down