Open Eilon opened 9 years ago
@rynowak @vijayrkn I think you two guys might know something about this?
From @JamesNK on August 2, 2015 6:42
I tracked this down to a bug I added to Json.NET while making changes. The test host uses Json.NET so my bug took down the host when it encountered the bug.
So no bug in the host but I think you should add more error handling so it doesn't crash so badly and pass the error message back to VS. That will help the user a better idea of what has gone wrong.
From @dougbu on August 2, 2015 19:33
@JamesNK do any released versions of Json.NET contain the bug? For example are "Newtonsoft.Json": "6.0.6"
through ... "6.0.8"
fine while ... "7.0.1"
has this problem and ... "7.0.2"
will be fine again?
From @JamesNK on August 2, 2015 20:52
No, it was from wip dev changes. Testing Json.NET is slightly difficult if the change I make during dev breaks the test application :smile:
From @rynowak on August 3, 2015 15:49
@JamesNK - do you happen to know offhand what the callstack was that throwing?
@vijayrkn My guess is that we'll want to do something dumber to serialize error messages.
@rynowak we do have our super simple JavaScript serializer/deserializer... or actually I don't know if we have a serializer; we may have only put in a deserializer...
From @rynowak on August 3, 2015 16:26
yeah, that would be good as well, we should remove json.net as a dependency as possible so you can use whatever version you want
If we want to do this I'd have to go dig up my old source codez to find the missing features.
From @vijayrkn on August 3, 2015 17:4
Created a bug in tooling repo (https://github.com/aspnet/Tooling/issues/129) to track 'removing the json.net dependency' from VS unit test adapter.
From @JamesNK on August 3, 2015 23:21
@JamesNK - do you happen to know offhand what the callstack was that throwing?
That's all the error message says. You should include more detail to VS when an error is thrown: https://github.com/aspnet/Testing/blob/979ada31b945e4597abf84a82fec3612825b4632/src/Microsoft.Dnx.TestHost/ReportingChannel.cs#L87
From @JamesNK on August 3, 2015 23:26
yeah, that would be good as well, we should remove json.net as a dependency as possible so you can use whatever version you want
You already can. The test tools were using my dev version which had a bug which is why I got that error.
TFS:149246
The GitHub Issue Tracker for the aspnet/Tooling repo is being deprecated in favor of Visual Studio's Report a Problem tool.
If this issue is still a problem with the RTW release of Visual Studio 2017, please report a new issue using the Report a Problem tool. While you can still use .NET Core and ASP.NET Preview tools with Visual Studio 2015, Visual Studio 2017 is now the officially supported tool for developing .NET Core and ASP.NET Core projects.
By using the Report a Problem tool (available in both VS 2017 and VS 2015), you can collect detailed information about the problem, and send it to Microsoft with just a few button clicks. See Visual Studio's Talk to Us page for more details.
Please use the discussion topic here for feedback and questions on the deprecation of this issue tracker. Thanks!
From @JamesNK on August 2, 2015 5:42
Error in VS2015 Output window. Using beta 6.
No tests are shown in Test Explorer window.
Copied from original issue: aspnet/Testing#134