-
Notifications
You must be signed in to change notification settings - Fork 325
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
.Net Core tests take a long time to report a missing SDK. #1692
Comments
@pvlakshm Please triage. |
>> This isn't very important but would give a better experience. |
I also raised this feedback ticket for the same issue. And IMHO this feels like a broken scenario. I installed the latest VS 15.8 and opened an existing solution and tried to run tests. If Test explorer window has hard dependency on some sdk version then shouldn't that be included as part of installer or Test explorer should take dependency on latest sdk which is part of the installer itself...? either way we shouldn't expect customers to install those specific sdk versions separately for us to be working properly... |
@jainaashish : Test Explorer or the platform does not take a hard dependency on an sdk. It picks up whatever the test app specifies. The VS installer AFAIK downloads only the latest stable sdk. you might want to look at additional components to install 1.* sdks or pull the one the app requires from the downloads page. |
@AbhitejJohn Is this still an issue? I thought this was fixed a long time ago. |
@singhsarab : Tried this out on a simple MSTest .NET Core project and I did not see the delay. So I guess this is fixed - Don't really know what fixed this though. |
Thanks for the confirmation. I think I remember we rolled our a fix for this. |
Description
When I run a .Net Core test where the sdk being targeted is not installed on the box we do nothing for about a minute after which we show the following message in the output pane
This is precise and helpful messaging. Is there a way we could report this earlier then having to wait for the connection timeout? This isn't very important but would give a better experience.
The text was updated successfully, but these errors were encountered: