NuGet: Fix binding redirect XML parse error to report unparseable file

When updating binding redirects, a failure to parse the XML config file
was throwing an InvalidOperationException which resulted in an unknown
error being reported. This change catches XmlException specifically and
throws UnparseableFileException instead, which is properly mapped to the
dependency_file_not_parseable error type.

Also fixes the typo 'binging' -> 'binding' in the error message.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: brettfo

nuget/helpers/lib/NuGetUpdater/NuGetUpdater.Core.Test/Update/BindingRedirectsTests.cs

@@ -205,6 +205,58 @@ await VerifyBindingRedirectsAsync(
         );
     }
 
+    [Fact]
+    public async Task MalformedConfigFileThrowsUnparseableFileException()
+    {
+        var projectContents = """
+            <Project ToolsVersion="15.0" DefaultTargets="Build" xmlns="http://schemas.microsoft.com/developer/msbuild/2003">
+              <Import Project="$(MSBuildExtensionsPath)\$(MSBuildToolsVersion)\Microsoft.Common.props" Condition="Exists('$(MSBuildExtensionsPath)\$(MSBuildToolsVersion)\Microsoft.Common.props')" />
+              <PropertyGroup>
+                <TargetFrameworkVersion>v4.5</TargetFrameworkVersion>
+              </PropertyGroup>
+              <ItemGroup>
+                <None Include="app.config" />
+              </ItemGroup>
+              <ItemGroup>
+                <Reference Include="Some.Package, Version=2.0.0.0, Culture=neutral, PublicKeyToken=null">
+                  <HintPath>packages\Some.Package.2.0.0\lib\net45\Some.Package.dll</HintPath>
+                  <Private>True</Private>
+                </Reference>
+              </ItemGroup>
+              <Import Project="$(MSBuildToolsPath)\Microsoft.CSharp.targets" />
+            </Project>
+            """;
+        var configContents = """
+            <configuration>
+              <appSettings>
+                <add key="SomeKey" value="SomeValue" />
+              </appSettings
+              <runtime>
+                <assemblyBinding xmlns="urn:schemas-microsoft-com:asm.v1">
+                  <dependentAssembly>
+                    <assemblyIdentity name="Some.Package" publicKeyToken="null" culture="neutral" />
+                    <bindingRedirect oldVersion="0.0.0.0-1.0.0.0" newVersion="1.0.0.0" />
+                  </dependentAssembly>
+                </assemblyBinding>
+              </runtime>
+            </configuration>
+            """;
+
+        using var tempDir = new TemporaryDirectory();
+        var projectFileName = "project.csproj";
+        var projectFilePath = Path.Combine(tempDir.DirectoryPath, projectFileName);
+        var configFilePath = Path.Combine(tempDir.DirectoryPath, "app.config");
+
+        File.WriteAllText(projectFilePath, projectContents);
+        File.WriteAllText(configFilePath, configContents);
+
+        var projectBuildFile = ProjectBuildFile.Open(tempDir.DirectoryPath, projectFilePath);
+        var exception = await Assert.ThrowsAsync<UnparseableFileException>(
+            () => BindingRedirectManager.UpdateBindingRedirectsAsync(tempDir.DirectoryPath, projectBuildFile, "Some.Package", "2.0.0").AsTask());
+        Assert.Equal(configFilePath.NormalizePathToUnix(), exception.FilePath);
+        Assert.Contains("Error loading binding redirect configuration", exception.Message);
+    }
+
     private static async Task VerifyBindingRedirectsAsync(string projectContents, string configContents, string expectedConfigContents, string updatedPackageName, string updatedPackageVersion, string configFileName = "app.config")
     {
         using var tempDir = new TemporaryDirectory();

nuget/helpers/lib/NuGetUpdater/NuGetUpdater.Core/Updater/BindingRedirectManager.cs

@@ -1,4 +1,5 @@
 using System.Collections.Immutable;
+using System.Xml;
 using System.Xml.Linq;
 
 using Microsoft.Language.Xml;
@@ -154,7 +155,7 @@ private static string AddBindingRedirects(ConfigurationFile configFile, IEnumera
         }
 
         // Get the configuration file
-        var document = GetConfiguration(configFile.Content);
+        var document = GetConfiguration(configFile.Content, configFile.Path);
 
         // Get the runtime element
         var runtime = document.Root?.Element("runtime");
@@ -211,15 +212,19 @@ private static string AddBindingRedirects(ConfigurationFile configFile, IEnumera
             document.ToString()
         );
 
-        static XDocument GetConfiguration(string configFileContent)
+        static XDocument GetConfiguration(string configFileContent, string configFilePath)
         {
             try
             {
                 return XDocument.Parse(configFileContent, LoadOptions.PreserveWhitespace);
             }
+            catch (XmlException ex)
+            {
+                throw new UnparseableFileException($"Error loading binding redirect configuration: {ex.Message}", configFilePath);
+            }
             catch (Exception ex)
             {
-                throw new InvalidOperationException("Error loading binging redirect configuration", ex);
+                throw new InvalidOperationException("Error loading binding redirect configuration", ex);
             }
         }