Don't run the new System.Net.ServerSentEvents test with NAOT - #103092
Don't run the new System.Net.ServerSentEvents test with NAOT#103092MichalStrehovsky wants to merge 1 commit into
Conversation
Uses reflection-based JSON so it's unsupportable without someone writing a trimming descriptor.
|
Tagging subscribers to this area: @dotnet/area-infrastructure-libraries |
|
Couldn't we instead change the test to use the source generator? |
If this is something you would be willing to work on, I can file it as a bug and block the test on that. This test doesn't look to have interesting runtime interactions so I didn't plan to file a bug. The other bugs I filed for single file/trim/aot unfriendly testing in the past are in various backlogs so I just stopped filing them for non-essentials to save time. This is just 1 out of 3 new outerloop test failures on native AOT legs in the past ~48 hours and I cannot justify spending too much time on these because it's a never ending stream. |
#103126 merged. Hopefully that fixed it. |
Thank you! |
Uses reflection-based JSON so it's unsupportable without someone writing a trimming descriptor.