Remove inner class for divide by zero test as requested

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

Author: Copilot

test/NuGet.Core.Tests/NuGet.CommandLine.Xplat.Tests/ListPackageCommandRunnerTests.cs

@@ -369,58 +369,55 @@ public void FiltersFrameworkPackagesCollectionWithVulnerableMetadata(
             }
         }
 
-        public class GetPackageMetadataAsyncWithEmptyPackageSources
+        [Fact]
+        public async Task GetPackageMetadataAsync_WithEmptyPackageSources_DoesNotThrowDivideByZero()
         {
-            [Fact]
-            public async Task GetPackageMetadataAsync_WithEmptyPackageSources_DoesNotThrowDivideByZero()
+            // Arrange
+            var packages = new FrameworkPackages("net40");
+            var topLevelPackages = new List<InstalledPackageReference>
             {
-                // Arrange
-                var packages = new FrameworkPackages("net40");
-                var topLevelPackages = new List<InstalledPackageReference>
-                {
-                    ListPackageTestHelper.CreateInstalledPackageReference(name: "TestPackage")
-                };
-                packages.TopLevelPackages = topLevelPackages;
-                var allPackages = new List<FrameworkPackages> { packages };
-
-                var output = new StringBuilder();
-                var error = new StringBuilder();
-                using TextWriter consoleOut = new StringWriter(output);
-                using TextWriter consoleError = new StringWriter(error);
-
-                // Create ListPackageArgs with empty packageSources list to trigger the divide by zero scenario
-                var listPackageArgs = new ListPackageArgs(
-                    path: "",
-                    packageSources: new List<PackageSource>(), // Empty package sources - this would cause divide by zero
-                    frameworks: new List<string>(),
-                    ReportType.Outdated, // This will trigger the code path that calls GetPackageMetadataAsync
-                    new ListPackageConsoleRenderer(consoleOut, consoleError),
-                    includeTransitive: false,
-                    prerelease: false,
-                    highestPatch: false,
-                    highestMinor: false,
-                    auditSources: null,
-                    logger: new Mock<ILogger>().Object,
-                    CancellationToken.None);
-
-                var listPackageRunner = new ListPackageCommandRunner();
-
-                // Act & Assert - Test the private method using reflection
-                var getPackageMetadataAsyncMethod = typeof(ListPackageCommandRunner)
-                    .GetMethod("GetPackageMetadataAsync", System.Reflection.BindingFlags.NonPublic | System.Reflection.BindingFlags.Instance);
-
-                Assert.NotNull(getPackageMetadataAsyncMethod);
-
-                // This should not throw DivideByZeroException
-                Exception exception = await Record.ExceptionAsync(async () =>
-                {
-                    var task = (Task<Dictionary<string, List<IPackageSearchMetadata>>>)getPackageMetadataAsyncMethod.Invoke(
-                        listPackageRunner, new object[] { allPackages, listPackageArgs });
-                    await task;
-                });
+                ListPackageTestHelper.CreateInstalledPackageReference(name: "TestPackage")
+            };
+            packages.TopLevelPackages = topLevelPackages;
+            var allPackages = new List<FrameworkPackages> { packages };
+
+            var output = new StringBuilder();
+            var error = new StringBuilder();
+            using TextWriter consoleOut = new StringWriter(output);
+            using TextWriter consoleError = new StringWriter(error);
+
+            // Create ListPackageArgs with empty packageSources list to trigger the divide by zero scenario
+            var listPackageArgs = new ListPackageArgs(
+                path: "",
+                packageSources: new List<PackageSource>(), // Empty package sources - this would cause divide by zero
+                frameworks: new List<string>(),
+                ReportType.Outdated, // This will trigger the code path that calls GetPackageMetadataAsync
+                new ListPackageConsoleRenderer(consoleOut, consoleError),
+                includeTransitive: false,
+                prerelease: false,
+                highestPatch: false,
+                highestMinor: false,
+                auditSources: null,
+                logger: new Mock<ILogger>().Object,
+                CancellationToken.None);
+
+            var listPackageRunner = new ListPackageCommandRunner();
+
+            // Act & Assert - Test the private method using reflection
+            var getPackageMetadataAsyncMethod = typeof(ListPackageCommandRunner)
+                .GetMethod("GetPackageMetadataAsync", System.Reflection.BindingFlags.NonPublic | System.Reflection.BindingFlags.Instance);
+
+            Assert.NotNull(getPackageMetadataAsyncMethod);
+
+            // This should not throw DivideByZeroException
+            Exception exception = await Record.ExceptionAsync(async () =>
+            {
+                var task = (Task<Dictionary<string, List<IPackageSearchMetadata>>>)getPackageMetadataAsyncMethod.Invoke(
+                    listPackageRunner, new object[] { allPackages, listPackageArgs });
+                await task;
+            });
 
-                Assert.Null(exception);
-            }
+            Assert.Null(exception);
         }
     }
 }