Fix serializing special characters in Jsonite

[nohwnd]

Fix #5120

[Evangelink]

LGTM! It might be interesting to use a DynamicData source instead of the foreach so we can replay easily but not blocking on it.

[Youssef1313]

LGTM

[nohwnd]

LGTM! It might be interesting to use a DynamicData source instead of the foreach so we can replay easily but not blocking on it.

This is addressed in comments. I did not want to emit the data, because then too many serializers need to handle them correctly to get the results. If any of them is broken (dynamic data serializer or the protocol serializer) we get no info about the test in UI, leaving us blind.

// This could be converted to Data source, but this way we have more control about where in the result message the
// special characters will be (hopefully nowhere) so in case of failure, we can still serialize the message to IDE
// even if the serializer does not support special characters.

[nohwnd]

Tests passed, new commit is adding just comment.

[Evangelink]

LGTM! It might be interesting to use a DynamicData source instead of the foreach so we can replay easily but not blocking on it.

This is addressed in comments. I did not want to emit the data, because then too many serializers need to handle them correctly to get the results. If any of them is broken (dynamic data serializer or the protocol serializer) we get no info about the test in UI, leaving us blind.

// This could be converted to Data source, but this way we have more control about where in the result message the // special characters will be (hopefully nowhere) so in case of failure, we can still serialize the message to IDE // even if the serializer does not support special characters.

Didn't think about it but makes total sense! Thanks!

[Evangelink]

/backport to rel/3.8

[github-actions]

Started backporting to rel/3.8: https://github.com/microsoft/testfx/actions/runs/13521711902