[WIP] dotnet test html logger throws execption when using special characters in DataRow attributes.

[Copilot]

Thanks for assigning this issue to me. I'm starting to work on it and will keep this PR's description up to date as I form a plan and make progress.

Original issue description:

Cannot move there, see: dotnet/sdk#38983

Fixes #10431.

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

[nohwnd]

looks like your session timed out. Try again.

[nohwnd]

@copilot try one more time.

[Copilot]

@copilot try one more time.

I've analyzed the issue and implemented a fix. The problem was that the HtmlTransformer was using tolerant settings for reading XML (CheckCharacters = false) but strict settings for writing, causing exceptions with special characters in DataRow attributes. I've updated the Transform method to use CheckCharacters = false for both reading and writing XML. The fix is now ready and includes comprehensive tests.

Warning

Firewall rules blocked me from connecting to one or more addresses

I tried to connect to the following addresses, but was blocked by firewall rules:

• 80zvsblobprodcus35.vsblob.vsassets.io
  • Triggering command: dotnet test test/Microsoft.TestPlatform.Extensions.HtmlLogger.UnitTests/Microsoft.TestPlatform.Extensions.HtmlLogger.UnitTests.csproj --filter TestResultHandlerShouldHandleSpecialCharactersInDataRowWithoutThrowingException --verbosity normal (dns block)
• 8xbvsblobprodcus382.vsblob.vsassets.io
  • Triggering command: dotnet test test/Microsoft.TestPlatform.Extensions.HtmlLogger.UnitTests/Microsoft.TestPlatform.Extensions.HtmlLogger.UnitTests.csproj --filter TestResultHandlerShouldHandleSpecialCharactersInDataRowWithoutThrowingException --verbosity normal (dns block)
• bcnvsblobprodcus378.vsblob.vsassets.io
  • Triggering command: dotnet test test/Microsoft.TestPlatform.Extensions.HtmlLogger.UnitTests/Microsoft.TestPlatform.Extensions.HtmlLogger.UnitTests.csproj --filter TestResultHandlerShouldHandleSpecialCharactersInDataRowWithoutThrowingException --verbosity normal (dns block)
• c50vsblobprodcus330.vsblob.vsassets.io
  • Triggering command: /home/REDACTED/work/vstest/vstest/.dotnet/dotnet msbuild /m /nologo /clp:Summary /v:minimal /nr:true /warnaserror /p:TreatWarningsAsErrors=true /p:ContinuousIntegrationBuild=false /home/REDACTED/work/vstest/vstest/artifacts/toolset/restore.proj /t:__WriteToolsetLocation /clp:ErrorsOnly;NoSummary /p:__ToolsetLocationOutputFile=/home/REDACTED/work/vstest/vstest/artifacts/toolset/9.0.0-beta.25302.2.txt (dns block)
  • Triggering command: dotnet test test/Microsoft.TestPlatform.Extensions.HtmlLogger.UnitTests/Microsoft.TestPlatform.Extensions.HtmlLogger.UnitTests.csproj --filter TestResultHandlerShouldHandleSpecialCharactersInDataRowWithoutThrowingException --verbosity normal (dns block)
• d0svsblobprodcus381.vsblob.vsassets.io
  • Triggering command: dotnet test test/Microsoft.TestPlatform.Extensions.HtmlLogger.UnitTests/Microsoft.TestPlatform.Extensions.HtmlLogger.UnitTests.csproj --filter TestResultHandlerShouldHandleSpecialCharactersInDataRowWithoutThrowingException --verbosity normal (dns block)
• https://api.github.com/repos/dotnet/sdk/issues/38983
  • Triggering command: curl -s REDACTED (http block)
• imzvsblobprodcus368.vsblob.vsassets.io
  • Triggering command: dotnet test test/Microsoft.TestPlatform.Extensions.HtmlLogger.UnitTests/Microsoft.TestPlatform.Extensions.HtmlLogger.UnitTests.csproj --filter TestResultHandlerShouldHandleSpecialCharactersInDataRowWithoutThrowingException --verbosity normal (dns block)
• jd4vsblobprodcus366.vsblob.vsassets.io
  • Triggering command: dotnet test test/Microsoft.TestPlatform.Extensions.HtmlLogger.UnitTests/Microsoft.TestPlatform.Extensions.HtmlLogger.UnitTests.csproj --filter TestResultHandlerShouldHandleSpecialCharactersInDataRowWithoutThrowingException --verbosity normal (dns block)
• jrqvsblobprodcus343.vsblob.vsassets.io
  • Triggering command: dotnet test test/Microsoft.TestPlatform.Extensions.HtmlLogger.UnitTests/Microsoft.TestPlatform.Extensions.HtmlLogger.UnitTests.csproj --filter TestResultHandlerShouldHandleSpecialCharactersInDataRowWithoutThrowingException --verbosity normal (dns block)
• kh4vsblobprodcus325.vsblob.vsassets.io
  • Triggering command: dotnet test test/Microsoft.TestPlatform.Extensions.HtmlLogger.UnitTests/Microsoft.TestPlatform.Extensions.HtmlLogger.UnitTests.csproj --filter TestResultHandlerShouldHandleSpecialCharactersInDataRowWithoutThrowingException --verbosity normal (dns block)
• kmuvsblobprodcus389.vsblob.vsassets.io
  • Triggering command: dotnet test test/Microsoft.TestPlatform.Extensions.HtmlLogger.UnitTests/Microsoft.TestPlatform.Extensions.HtmlLogger.UnitTests.csproj --filter TestResultHandlerShouldHandleSpecialCharactersInDataRowWithoutThrowingException --verbosity normal (dns block)
• l49vsblobprodcus358.vsblob.vsassets.io
  • Triggering command: dotnet test test/Microsoft.TestPlatform.Extensions.HtmlLogger.UnitTests/Microsoft.TestPlatform.Extensions.HtmlLogger.UnitTests.csproj --filter TestResultHandlerShouldHandleSpecialCharactersInDataRowWithoutThrowingException --verbosity normal (dns block)
• lylvsblobprodcus31.vsblob.vsassets.io
  • Triggering command: dotnet test test/Microsoft.TestPlatform.Extensions.HtmlLogger.UnitTests/Microsoft.TestPlatform.Extensions.HtmlLogger.UnitTests.csproj --filter TestResultHandlerShouldHandleSpecialCharactersInDataRowWithoutThrowingException --verbosity normal (dns block)
• o3svsblobprodcus318.vsblob.vsassets.io
  • Triggering command: dotnet test test/Microsoft.TestPlatform.Extensions.HtmlLogger.UnitTests/Microsoft.TestPlatform.Extensions.HtmlLogger.UnitTests.csproj --filter TestResultHandlerShouldHandleSpecialCharactersInDataRowWithoutThrowingException --verbosity normal (dns block)
• x3yvsblobprodcus370.vsblob.vsassets.io
  • Triggering command: dotnet test test/Microsoft.TestPlatform.Extensions.HtmlLogger.UnitTests/Microsoft.TestPlatform.Extensions.HtmlLogger.UnitTests.csproj --filter TestResultHandlerShouldHandleSpecialCharactersInDataRowWithoutThrowingException --verbosity normal (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to my firewall allow list

[nohwnd]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).