allowing workers to handle .dll file extensions (#6420)

Author: brettsam

src/WebJobs.Script/Host/FunctionMetadataProvider.cs

@@ -172,6 +172,13 @@ internal static string ParseLanguage(string scriptFilePath, IEnumerable<RpcWorke
 
             // determine the script type based on the primary script file extension
             string extension = Path.GetExtension(scriptFilePath).ToLowerInvariant().TrimStart('.');
+            var workerConfig = workerConfigs.FirstOrDefault(config => config.Description.Extensions.Contains("." + extension));
+            if (workerConfig != null)
+            {
+                return workerConfig.Description.Language;
+            }
+
+            // If no worker claimed these extensions, use in-proc.
             switch (extension)
             {
                 case "csx":
@@ -180,11 +187,7 @@ internal static string ParseLanguage(string scriptFilePath, IEnumerable<RpcWorke
                 case "dll":
                     return DotNetScriptTypes.DotNetAssembly;
             }
-            var workerConfig = workerConfigs.FirstOrDefault(config => config.Description.Extensions.Contains("." + extension));
-            if (workerConfig != null)
-            {
-                return workerConfig.Description.Language;
-            }
+
             return null;
         }
 

test/WebJobs.Script.Tests.Shared/TestHelpers.cs

@@ -250,16 +250,21 @@ public static async Task<string> ReadStreamToEnd(Stream stream)
             }
         }
 
-        public static IList<RpcWorkerConfig> GetTestWorkerConfigs()
+        public static IList<RpcWorkerConfig> GetTestWorkerConfigs(bool includeDllWorker = false)
         {
-            var nodeWorkerDesc = GetTestWorkerDescription("node", ".js");
-            var javaWorkerDesc = GetTestWorkerDescription("java", ".jar");
-
-            return new List<RpcWorkerConfig>()
+            var workerConfigs = new List<RpcWorkerConfig>
             {
-                new RpcWorkerConfig() { Description = nodeWorkerDesc },
-                new RpcWorkerConfig() { Description = javaWorkerDesc },
+                new RpcWorkerConfig() { Description = GetTestWorkerDescription("node", ".js") },
+                new RpcWorkerConfig() { Description = GetTestWorkerDescription("java", ".jar") }
             };
+
+            // Allow tests to have a worker that claims the .dll extension.
+            if (includeDllWorker)
+            {
+                workerConfigs.Add(new RpcWorkerConfig() { Description = GetTestWorkerDescription("dllWorker", ".dll") });
+            }
+
+            return workerConfigs;
         }
 
         public static IList<RpcWorkerConfig> GetTestWorkerConfigsNoLanguage()

test/WebJobs.Script.Tests/FunctionMetadataProviderTests.cs

@@ -115,15 +115,16 @@ public void ValidateFunctionName_DoesNotThrowOnValidName(string functionName)
         }
 
         [Theory]
-        [InlineData("node", "test.js")]
-        [InlineData("java", "test.jar")]
-        [InlineData("CSharp", "test.cs")]
-        [InlineData("CSharp", "test.csx")]
-        [InlineData("DotNetAssembly", "test.dll")]
-        [InlineData(null, "test.x")]
-        public void ParseLanguage_Returns_ExpectedLanguage(string language, string scriptFile)
+        [InlineData("node", "test.js", false)]
+        [InlineData("java", "test.jar", false)]
+        [InlineData("CSharp", "test.cs", false)]
+        [InlineData("CSharp", "test.csx", false)]
+        [InlineData("dllWorker", "test.dll", true)] // The test "dllWorker" will claim ".dll" extensions before falling back to DotNetAssembly
+        [InlineData("DotNetAssembly", "test.dll", false)]
+        [InlineData(null, "test.x", false)]
+        public void ParseLanguage_Returns_ExpectedLanguage(string language, string scriptFile, bool includeDllWorker)
         {
-            Assert.Equal(language, FunctionMetadataProvider.ParseLanguage(scriptFile, TestHelpers.GetTestWorkerConfigs()));
+            Assert.Equal(language, FunctionMetadataProvider.ParseLanguage(scriptFile, TestHelpers.GetTestWorkerConfigs(includeDllWorker: includeDllWorker)));
         }
 
         [Theory]