-
Notifications
You must be signed in to change notification settings - Fork 106
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
AddTestAttachment does not work anymore in VS2022 #914
Comments
@bernd5 You mention "file-scheme". Can you elaborate on what you found there? |
It looks like the file scheme is not supported anymore for attached test files in VS2022. But this is just a guess because of the strange highlighting in test explorer. |
* Fix issue #914 * Adding Linux safety belt * SA error fixed
Thank you very much for fixing this issue 👍 |
Working on getting it out asap. If all go well, by the end of this week. |
@OsirisTerje A while ago I opened an issue related to this in VS feedback. They never fixed it in VS2019 but they claimed it was fixed in VS2022. Now I've got the opportunity to test in in VS2022 and found the issue remains, and it's related to the "file://" prefix. I've reopened the VS issue since I believe it needs to be fixed on both sides for maximum compatibility. |
@vpenades Have you checked the latest version if it works there? (From https://www.myget.org/feed/nunit/package/nuget/NUnit3TestAdapter) . It does seem the file prefix is actually still there, just not being shown in the test explorer, so would be interesting to see if it works or not elsewhere. |
@OsirisTerje not sure if I will be able to do that because it would require us to push the nuget into our production pipeline so it can reach our CI environment. I'll give it a look, though. |
@vpenades You should be able to run it locally just using the same commandline. That would be enough to verify it. Also, alternatively, you don't need to push the package up, you can just refer to the myget source feed. That feed only contains the latest dev packages, and not any releases, they will anyway be picked off nuget.org, so it should not crash with anything else. |
@bernd5 @vpenades Version 4.2.0-beta.1 released now on nuget.org https://www.nuget.org/packages/NUnit3TestAdapter/4.2.0-beta.1 . It also contains an engine upgrade, which is the reason for the beta. Would appreciate confirmation that it still works for you. |
@OsirisTerje Sorry for the late reply, I am in crunch time right now, but I'll give the beta a try as soon as I can. Thanks for the effort! |
For me the new beta works perfectly 👍. |
@OsirisTerje I was able to test the beta1 both within VS2022 test explorer and within TeamCity tests system, and both are able to display the test attachments correctly. I still believe that Visual Studio should be more forgiving regarding paths, since a "file:///c:xxx" should be valid, but that falls into MS side. |
Description
If the following code is executed:
The attached file is not clickable (it works with MSTest). I assume the file-scheme is causing the issue.
See the following image:
Versions
NUnit: 3.13.2
NUnit-Test-Adapter: 4.1.0
.net platform: 5 / 6
Visual Studio edition: VS2022 Enterprise Version 17.1.0 Preview 1.0
The text was updated successfully, but these errors were encountered: