Remove InstallPackageFromPMC_TriggersNuGetCacheUpdatedEventAsync test and revert SolutionManager property

Agent-Logs-Url: https://github.com/NuGet/NuGet.Client/sessions/4ff4fd9e-2c3b-4f98-bd0d-250dd8408f55

Co-authored-by: nkolev92 <[email protected]>

Author: Copilot

test/EndToEnd/tests/NetCoreProjectTest.ps1

@@ -1,3 +1,35 @@
+# VSSolutionManager and ProjectSystemCache event test for .net core
+function Test-NetCoreProjectSystemCacheUpdateEvent {
+
+    # Arrange
+    $projectA = New-NetCoreConsoleApp
+    Build-Solution
+    Assert-NetCoreProjectCreation $projectA
+
+    $componentModel = Get-VSComponentModel
+    $solutionManager = $componentModel.GetService([NuGet.PackageManagement.ISolutionManager])
+
+    $cacheEvent = $null
+
+    Get-Event | Remove-Event
+    Register-ObjectEvent -InputObject $solutionManager -EventName AfterNuGetCacheUpdated -SourceIdentifier SolutionManagerCacheUpdated
+
+    Try
+    {
+        # Act
+        $projectA | Install-Package Newtonsoft.Json -Version '13.0.1'
+
+        $cacheEvent = Wait-Event -SourceIdentifier SolutionManagerCacheUpdated -TimeoutSec 10
+    }
+    Finally
+    {
+        Unregister-Event -SourceIdentifier SolutionManagerCacheUpdated
+    }
+
+    # Assert
+    Assert-NotNull $cacheEvent -Message "Cache update event should've been raised"
+}
+
 function Test-NetCoreTargetFrameworksVSandMSBuildNoOp {
     [SkipTest('https://github.com/NuGet/Home/issues/13003')]
     param ()

test/NuGet.Tests.Apex/NuGet.Tests.Apex/Apex/NuGetApexTestService.cs

@@ -18,7 +18,6 @@
 using Microsoft.VisualStudio.Shell.Interop;
 using Microsoft.VisualStudio.Shell.ServiceBroker;
 using NuGet.Console.TestContract;
-using NuGet.PackageManagement;
 using NuGet.PackageManagement.UI.TestContract;
 using NuGet.SolutionRestoreManager;
 using NuGet.Versioning;
@@ -57,11 +56,6 @@ protected internal IVsSolutionRestoreStatusProvider SolutionRestoreStatusProvide
 
         protected internal IVsPathContextProvider2 PathContextProvider2 => VisualStudioObjectProviders.GetComponentModelService<IVsPathContextProvider2>();
 
-        /// <summary>
-        /// Gets the NuGet ISolutionManager
-        /// </summary>
-        protected internal ISolutionManager SolutionManager => VisualStudioObjectProviders.GetComponentModelService<ISolutionManager>();
-
         /// <summary>
         /// Wait for all nominations and auto restore to complete.
         /// This uses an Action to log since the xunit logger is not fully serializable.

test/NuGet.Tests.Apex/NuGet.Tests.Apex/NuGetEndToEndTests/NetCoreProjectTestCase.cs

@@ -1,12 +1,9 @@
-using System;
 using System.Collections.Generic;
 using System.Diagnostics;
 using System.IO;
-using System.Threading;
 using System.Threading.Tasks;
 using Microsoft.Test.Apex.VisualStudio.Solution;
 using Microsoft.VisualStudio.TestTools.UnitTesting;
-using NuGet.PackageManagement;
 
 namespace NuGet.Tests.Apex
 {
@@ -237,44 +234,6 @@ public async Task WithSourceMappingEnabled_InstallPackageFromPMUIAndNoSourcesFou
             }
         }
 
-        // Migrated from Test-NetCoreProjectSystemCacheUpdateEvent in NetCoreProjectTest.ps1
-        [TestMethod]
-        [Timeout(DefaultTimeout)]
-        public async Task InstallPackageFromPMC_TriggersNuGetCacheUpdatedEventAsync()
-        {
-            // Arrange
-            using var testContext = new ApexTestContext(VisualStudio, ProjectTemplate.NetCoreConsoleApp, Logger, addNetStandardFeeds: true);
-
-            var packageName = "TestPackage";
-            var packageVersion = "1.0.0";
-            await CommonUtility.CreatePackageInSourceAsync(testContext.PackageSource, packageName, packageVersion);
-
-            testContext.SolutionService.Build();
-            testContext.NuGetApexTestService.WaitForAutoRestore();
-
-            // Subscribe to the ISolutionManager.AfterNuGetCacheUpdated event
-            using var cacheUpdatedEvent = new ManualResetEventSlim(false);
-            var solutionManager = testContext.NuGetApexTestService.SolutionManager;
-            void OnAfterNuGetCacheUpdated(object sender, NuGetEventArgs<string> e) => cacheUpdatedEvent.Set();
-            solutionManager.AfterNuGetCacheUpdated += OnAfterNuGetCacheUpdated;
-
-            try
-            {
-                // Act
-                var nugetConsole = GetConsole(testContext.Project);
-                nugetConsole.InstallPackageFromPMC(packageName, packageVersion);
-
-                // Assert
-                Assert.IsTrue(
-                    cacheUpdatedEvent.Wait(TimeSpan.FromSeconds(10)),
-                    "Cache update event should have been raised after package install.");
-            }
-            finally
-            {
-                solutionManager.AfterNuGetCacheUpdated -= OnAfterNuGetCacheUpdated;
-            }
-        }
-
         // Migrated from Test-NetCoreVSandMSBuildNoOp in NetCoreProjectTest.ps1
         [TestMethod]
         [Timeout(DefaultTimeout)]