aspnet/Tooling

Remove the Json.net dependency from Unit Test Test adapter.

Closed this issue · 5 comments

Currently, VS unit test adapter depends on Json.Net to serialize and de-serialize messages. Need to remove this dependency so that json.net changes does not impact the unit test discovery/execution.

Issue description is available here - aspnet/Testing#134

I can try to dig up code for this if needed. It'll be similar to the code we use to read project.json.

No changes to a publicly released version of Json.NET will ever break the unit test tools.

The error I got was from a bug I accidentally introduced during development which took down the unit test tools.

I think this is pretty low priority. The only person it would ever help would be me 😄

(and if you want to help me, there are other things I'd rather you look at... aspnet/dnx#1894)

@vijayrkn @Eilon can we close this?

Eilon commented

I don't think there's any action for my team here. This is tooling only, right?

Yes, This is a tooling only item.