-
Notifications
You must be signed in to change notification settings - Fork 43
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
Make a Visual Mark on Test Results #103
Comments
Does the explorer window show the correct file location for the test method? If not, there may be something we can fix. But if it does, that means we are passing the correct info to VS and it's basically up to them to do something with it. :-( |
Yes, I think it is showing the right file location...can you make a simple test like mine and see if this isn't something isolated? |
I have verified this behavior using the NUnit 3 adapter rather than NUnit 2, which seems to indicate it's just how it works with VS. The behavior makes a certain amount of sense. If a method is parameterized and has twenty cases, potentially having differing results, then what would be shown? I'd like to close this as notabug if nobody objects. |
Well, I think if it has 20 cases, and all of them pass, so there should be a green check mark...if one fails, even if the others pass, there should be a red warning....Just like it is in the Test Explorer. |
Perhaps, but definitely not in our control. |
I see....do you have a channel of communication with Microsoft to raise this issue with them? I think this will be a nice addition. |
@OsirisTerje can you try? They don't talk to me for sure. |
I'll talk with them again, but I've done so earlier. The indicator is not part of the test explorer itself, but is a CodeLens "extension" they have. Not sure how it is related, but I'll ping them again. |
Thanks @OsirisTerje If this turns out to be something they don't expect to change anytime soon, I think we should just close it as not our bug. |
The native Visual Studio Tests have a check mark on everything related to that test, for example:
I don't know if it's possible to mimic this functionality, but it will be a nice addition.
The text was updated successfully, but these errors were encountered: