Register off-by-default analyzer: configuration binding generator

src/Tasks/Microsoft.NET.Build.Tasks/targets/Microsoft.NET.Sdk.FrameworkReferenceResolution.targets

@@ -423,6 +423,8 @@ Copyright (c) .NET Foundation. All rights reserved.
     <ItemGroup>
       <OffByDefaultAnalyzer Include="Microsoft.AspNetCore.Http.RequestDelegateGenerator"
                             IsEnabled="$(EnableRequestDelegateGenerator)"/>
+      <OffByDefaultAnalyzer Include="Microsoft.Extensions.Configuration.Binder.SourceGeneration"
+                            IsEnabled="$(EnableConfigurationBindingGenerator)"/>
     </ItemGroup>
 
     <ItemGroup>

src/Tests/Microsoft.NET.Build.Tests/GivenThatWeWantToUseAnalyzers.cs

@@ -24,42 +24,51 @@ public GivenThatWeWantToUseAnalyzers(ITestOutputHelper log) : base(log)
         }
 
         [Theory]
-        [InlineData("WebApp", false)]
-        [InlineData("WebApp", true)]
-        [InlineData("WebApp", null)]
-        public void It_resolves_requestdelegategenerator_correctly(string testAssetName, bool? isEnabled)
+        [MemberData(nameof(OffByDefaultGeneratorTestData))]
+        public void It_resolves_offbydefaultgenerator_correctly(GeneratorSpec generator, bool? isEnabled)
         {
             var asset = _testAssetsManager
-                .CopyTestAsset(testAssetName, identifier: isEnabled.ToString())
+                .CopyTestAsset("WebApp", identifier: isEnabled.ToString())
                 .WithSource()
                 .WithProjectChanges(project =>
                 {
                     if (isEnabled != null)
                     {
                         var ns = project.Root.Name.Namespace;
-                        project.Root.Add(new XElement(ns + "PropertyGroup", new XElement("EnableRequestDelegateGenerator", isEnabled)));
+                        // Delete when we have a new targeting pack that contains Microsoft.Extensions.Configuration.Binder.SourceGeneration.
+                        if (generator == GeneratorSpec.ConfigurationBinding)
+                        {
+                            project.Root.Add(new XElement(ns + "ItemGroup", new XElement("Analyzer", new XAttribute("Include", "does_not_yet_exist\\Microsoft.Extensions.Configuration.Binder.SourceGeneration.dll"))));
+                        }
+                        project.Root.Add(new XElement(ns + "PropertyGroup", new XElement(generator.EnablingPropertyName, isEnabled)));
                     }
                 });
 
-            VerifyRequestDelegateGeneratorIsUsed(asset, isEnabled);
+            VerifyOffByDefaultGeneratorIsUsed(asset, generator.DllFileName, isEnabled);
         }
 
-        [Fact]
-        public void It_enables_requestdelegategenerator_for_PublishAot()
+        [Theory]
+        [MemberData(nameof(OffByDefaultGenerators))]
+        public void It_enables_offbydefaultgenerator_for_PublishAot(GeneratorSpec generator)
         {
             var asset = _testAssetsManager
                 .CopyTestAsset("WebApp")
                 .WithSource()
                 .WithProjectChanges(project =>
                 {
                     var ns = project.Root.Name.Namespace;
+                    // Delete when we have a new targeting pack that contains Microsoft.Extensions.Configuration.Binder.SourceGeneration.
+                    if (generator == GeneratorSpec.ConfigurationBinding)
+                    {
+                        project.Root.Add(new XElement(ns + "ItemGroup", new XElement("Analyzer", new XAttribute("Include", "does_not_yet_exist\\Microsoft.Extensions.Configuration.Binder.SourceGeneration.dll"))));
+                    }
                     project.Root.Add(new XElement(ns + "PropertyGroup", new XElement("PublishAot", "true")));
                 });
 
-            VerifyRequestDelegateGeneratorIsUsed(asset, expectEnabled: true);
+            VerifyOffByDefaultGeneratorIsUsed(asset, generator.DllFileName, expectEnabled: true);
         }
 
-        private void VerifyRequestDelegateGeneratorIsUsed(TestAsset asset, bool? expectEnabled)
+        private void VerifyOffByDefaultGeneratorIsUsed(TestAsset asset, string dllFileName, bool? expectEnabled)
         {
             var command = new GetValuesCommand(
                 Log,
@@ -75,7 +84,31 @@ private void VerifyRequestDelegateGeneratorIsUsed(TestAsset asset, bool? expectE
 
             var analyzers = command.GetValues();
 
-            Assert.Equal(expectEnabled ?? false, analyzers.Any(analyzer => analyzer.Contains("Microsoft.AspNetCore.Http.RequestDelegateGenerator.dll")));
+            Assert.Equal(expectEnabled ?? false, analyzers.Any(analyzer => analyzer.Contains(dllFileName)));
+        }
+
+        public record GeneratorSpec(string EnablingPropertyName, string DllFileName)
+        {
+            public static GeneratorSpec RequestDelegate { get; } = new GeneratorSpec("EnableRequestDelegateGenerator", "Microsoft.AspNetCore.Http.RequestDelegateGenerator.dll");
+            public static GeneratorSpec ConfigurationBinding { get; } = new GeneratorSpec("EnableConfigurationBindingGenerator", "Microsoft.Extensions.Configuration.Binder.SourceGeneration.dll");
+        }
+
+        public static IEnumerable<object[]> OffByDefaultGenerators()
+        {
+            yield return new object[] { GeneratorSpec.RequestDelegate };
+            yield return new object[] { GeneratorSpec.ConfigurationBinding };
+        }
+
+        public static IEnumerable<object[]> OffByDefaultGeneratorTestData()
+        {
+            foreach (object[] generator in OffByDefaultGenerators())
+            {
+                // Test asset name, generator to test, whether generator is enabled.
+                var spec = (GeneratorSpec)generator[0];
+                yield return new object[] { spec, false };
+                yield return new object[] { spec, true };
+                yield return new object[] { spec, null };
+            }
         }
 
         [Theory]

src/WebSdk/ProjectSystem/Targets/Microsoft.NET.Sdk.Web.ProjectSystem.targets

@@ -57,6 +57,7 @@ Copyright (c) .NET Foundation. All rights reserved.
    -->
   <PropertyGroup Condition="'$(PublishAot)' == 'true'">
     <EnableRequestDelegateGenerator Condition="'$(EnableRequestDelegateGenerator)' == ''">true</EnableRequestDelegateGenerator>
+    <EnableConfigurationBindingGenerator Condition="'$(EnableConfigurationBindingGenerator)' == ''">true</EnableConfigurationBindingGenerator>
   </PropertyGroup>
 
   <!--