added requested changes, after first code review

Author: chan18

src/BenchmarkDotNet/ConsoleArguments/ListBenchmarks/BenchmarkCasesPrinter.cs

@@ -1,5 +1,9 @@
-using System.Collections.Generic;
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using BenchmarkDotNet.Configs;
 using BenchmarkDotNet.Loggers;
+using BenchmarkDotNet.Running;
 
 namespace BenchmarkDotNet.ConsoleArguments.ListBenchmarks
 {
@@ -14,6 +18,18 @@ public BenchmarkCasesPrinter(ListBenchmarkCaseMode listBenchmarkCaseMode)
                 : new FlatBenchmarkCasesPrinter();
         }
 
+        public static void PrintList(ILogger nonNullLogger, IConfig effectiveConfig, IReadOnlyList<Type> allAvailableTypesWithRunnableBenchmarks, CommandLineOptions options)
+        {
+            var printer = new BenchmarkCasesPrinter(options.ListBenchmarkCaseMode);
+
+            var testNames = TypeFilter.Filter(effectiveConfig, allAvailableTypesWithRunnableBenchmarks)
+                .SelectMany(p => p.BenchmarksCases)
+                .Select(p => p.Descriptor.GetFilterName())
+                .Distinct();
+
+            printer.Print(testNames, nonNullLogger);
+        }
+
         public void Print(IEnumerable<string> testNames, ILogger logger) => printer.Print(testNames, logger);
     }
 }

src/BenchmarkDotNet/Running/BenchmarkRunnerDirty.cs

@@ -20,10 +20,6 @@ namespace BenchmarkDotNet.Running
     // so it can ignore the version mismatch ;)
     public static class BenchmarkRunner
     {
-        private static readonly IUserInteraction userInteraction = new UserInteraction();
-        private static readonly List<Type> types = new List<Type>();
-        private static readonly List<Assembly> assemblies = new List<Assembly>();
-
         [PublicAPI]
         public static Summary Run<T>(IConfig config = null, string[] args = null)
         {
@@ -84,6 +80,7 @@ public static Summary RunSource(string source, IConfig config = null)
         private static Summary RunWithDirtyAssemblyResolveHelper(Type type, IConfig config, string[] args)
         {
             var logger = config.GetNonNullCompositeLogger();
+            var userInteraction = new UserInteraction();
             var (isParsingSuccess, parsedConfig, options) = ConfigParser.Parse(args, logger, config);
             if (!isParsingSuccess) // invalid console args, the ConfigParser printed the error
                 return null;
@@ -96,8 +93,9 @@ private static Summary RunWithDirtyAssemblyResolveHelper(Type type, IConfig conf
 
             var effectiveConfig = ManualConfig.Union(config, parsedConfig);
 
+            var types = new List<Type>();
             types.Add(type);
-            var (allTypesValid, allAvailableTypesWithRunnableBenchmarks) = TypeFilter.GetTypesWithRunnableBenchmarks(types, assemblies, logger);
+            var (allTypesValid, allAvailableTypesWithRunnableBenchmarks) = TypeFilter.GetTypesWithRunnableBenchmarks(types, new List<Assembly>(), logger);
             if (!allTypesValid) // there were some invalid and TypeFilter printed errors
                 return null;
 
@@ -109,22 +107,15 @@ private static Summary RunWithDirtyAssemblyResolveHelper(Type type, IConfig conf
 
             if (options.ListBenchmarkCaseMode != ListBenchmarkCaseMode.Disabled)
             {
-                PrintList(logger, effectiveConfig, allAvailableTypesWithRunnableBenchmarks, options);
+                BenchmarkCasesPrinter.PrintList(logger, effectiveConfig, allAvailableTypesWithRunnableBenchmarks, options);
                 return null;
             }
 
-            var benchmarksToFilter = options.UserProvidedFilters
-                ? allAvailableTypesWithRunnableBenchmarks
-                : userInteraction.AskUser(allAvailableTypesWithRunnableBenchmarks, logger);
-
-            var filteredBenchmarks = TypeFilter.Filter(effectiveConfig, benchmarksToFilter);
-
-            if (filteredBenchmarks.IsEmpty())
-            {
-                return BenchmarkRunnerClean.Run(new[] { BenchmarkConverter.TypeToBenchmarks(type, config) }).Single();
-            }
+            var benchmarks = options.UserProvidedFilters
+                           ? TypeFilter.Filter(effectiveConfig, allAvailableTypesWithRunnableBenchmarks)
+                           : new BenchmarkRunInfo[] { BenchmarkConverter.TypeToBenchmarks(type, config) };
 
-            return BenchmarkRunnerClean.Run(filteredBenchmarks).Single();
+            return BenchmarkRunnerClean.Run(benchmarks).Single();
         }
 
 
@@ -177,16 +168,6 @@ private static Summary[] RunWithExceptionHandling(Func<Summary[]> run)
                 return new[] { Summary.NothingToRun(e.Message, string.Empty, string.Empty) };
             }
         }
-        private static void PrintList(ILogger nonNullLogger, IConfig effectiveConfig, IReadOnlyList<Type> allAvailableTypesWithRunnableBenchmarks, CommandLineOptions options)
-        {
-            var printer = new BenchmarkCasesPrinter(options.ListBenchmarkCaseMode);
 
-            var testNames = TypeFilter.Filter(effectiveConfig, allAvailableTypesWithRunnableBenchmarks)
-                .SelectMany(p => p.BenchmarksCases)
-                .Select(p => p.Descriptor.GetFilterName())
-                .Distinct();
-
-            printer.Print(testNames, nonNullLogger);
-        }
     }
 }

src/BenchmarkDotNet/Running/BenchmarkSwitcher.cs

@@ -13,8 +13,7 @@
 using JetBrains.Annotations;
 
 namespace BenchmarkDotNet.Running
-{
-    public class BenchmarkSwitcher
+{    public class BenchmarkSwitcher
     {
         private readonly IUserInteraction userInteraction = new UserInteraction();
         private readonly List<Type> types = new List<Type>();
@@ -65,7 +64,7 @@ public IEnumerable<Summary> Run(string[] args = null, IConfig config = null)
         [MethodImpl(MethodImplOptions.NoInlining)]
         private IEnumerable<Summary> RunWithDirtyAssemblyResolveHelper(string[] args, IConfig config)
         {
-            var logger = config.GetNonNullCompositeLogger();
+            var logger = config.GetNonNullCompositeLogger();            
             var (isParsingSuccess, parsedConfig, options) = ConfigParser.Parse(args, logger, config);
             if (!isParsingSuccess) // invalid console args, the ConfigParser printed the error
                 return Array.Empty<Summary>();
@@ -90,7 +89,7 @@ private IEnumerable<Summary> RunWithDirtyAssemblyResolveHelper(string[] args, IC
 
             if (options.ListBenchmarkCaseMode != ListBenchmarkCaseMode.Disabled)
             {
-                PrintList(logger, effectiveConfig, allAvailableTypesWithRunnableBenchmarks, options);
+                BenchmarkCasesPrinter.PrintList(logger, effectiveConfig, allAvailableTypesWithRunnableBenchmarks, options);
                 return Array.Empty<Summary>();
             }
 
@@ -108,17 +107,6 @@ private IEnumerable<Summary> RunWithDirtyAssemblyResolveHelper(string[] args, IC
 
             return BenchmarkRunnerClean.Run(filteredBenchmarks);
         }
-
-        private static void PrintList(ILogger nonNullLogger, IConfig effectiveConfig, IReadOnlyList<Type> allAvailableTypesWithRunnableBenchmarks, CommandLineOptions options)
-        {
-            var printer = new BenchmarkCasesPrinter(options.ListBenchmarkCaseMode);
-
-            var testNames = TypeFilter.Filter(effectiveConfig, allAvailableTypesWithRunnableBenchmarks)
-                .SelectMany(p => p.BenchmarksCases)
-                .Select(p => p.Descriptor.GetFilterName())
-                .Distinct();
-
-            printer.Print(testNames, nonNullLogger);
-        }
+               
     }
 }