Allow assemblies without extensions to be excluded (#6834) (#6839)

* Restored previous behavior of only stripping .exe / .dll

* Allow assemblies without extensions to be excluded

---------

Co-authored-by: Ramon Smits <[email protected]>

Author: Daniel Marbach

src/NServiceBus.Core.Tests/AssemblyScanner/When_exclusion_predicate_is_used.cs

@@ -29,5 +29,22 @@ public void No_files_explicitly_excluded_are_returned()
             Assert.That(explicitlySkippedDll, Is.Not.Null);
             Assert.That(explicitlySkippedDll.SkipReason, Contains.Substring("File was explicitly excluded from scanning"));
         }
+
+        [Test]
+        public void Assemblies_that_have_no_extension_can_be_excluded()
+        {
+            var results = new AssemblyScanner(Path.Combine(TestContext.CurrentContext.TestDirectory, "TestDlls"))
+            {
+                AssembliesToSkip = new List<string> { "some.random" },
+                ScanAppDomainAssemblies = false
+            }
+            .GetScannableAssemblies();
+
+            var skippedFiles = results.SkippedFiles;
+            var explicitlySkippedDll = skippedFiles.FirstOrDefault(s => s.FilePath.Contains("some.random.dll"));
+
+            Assert.That(explicitlySkippedDll, Is.Not.Null);
+            Assert.That(explicitlySkippedDll.SkipReason, Contains.Substring("File was explicitly excluded from scanning"));
+        }
     }
 }

src/NServiceBus.Core.Tests/TestDlls/some.random.dll

@@ -0,0 +1 @@
+this is not a DLL

src/NServiceBus.Core/Hosting/Helpers/AssemblyScanner.cs

@@ -54,9 +54,14 @@ internal AssemblyScanner(Assembly assemblyToScan)
 
         internal IReadOnlyCollection<string> AssembliesToSkip
         {
-            set => assembliesToSkip = new HashSet<string>(value.Select(Path.GetFileNameWithoutExtension), StringComparer.OrdinalIgnoreCase);
+            set => assembliesToSkip = new HashSet<string>(value.Select(RemoveExtension), StringComparer.OrdinalIgnoreCase);
         }
 
+        static string RemoveExtension(string assemblyName) =>
+            assemblyName.EndsWith(".dll", StringComparison.OrdinalIgnoreCase) || assemblyName.EndsWith(".exe", StringComparison.OrdinalIgnoreCase)
+                ? Path.GetFileNameWithoutExtension(assemblyName)
+                : assemblyName;
+
         internal IReadOnlyCollection<Type> TypesToSkip
         {
             set => typesToSkip = new HashSet<Type>(value);
@@ -342,9 +347,7 @@ static List<FileInfo> ScanDirectoryForAssemblyFiles(string directoryToScan, bool
             return fileInfo;
         }
 
-        bool IsExcluded(string assemblyNameOrFileNameWithoutExtension) =>
-            assembliesToSkip.Contains(assemblyNameOrFileNameWithoutExtension) ||
-            DefaultAssemblyExclusions.Contains(assemblyNameOrFileNameWithoutExtension);
+        bool IsExcluded(string assemblyName) => assembliesToSkip.Contains(assemblyName) || DefaultAssemblyExclusions.Contains(assemblyName);
 
         // The parameter and return types of this method are deliberately using the most concrete types
         // to avoid unnecessary allocations