-
Notifications
You must be signed in to change notification settings - Fork 153
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
NUnit Console Extensions are not loaded in version 3.19 #1598
Comments
Thanks for opening this issue - I'd been meaning to but didn't have time. Related discussion here: #1507 (comment) |
@CharliePoole This is the way we use it in TeamCity: server downloads *.zip from GitHub, distributes it across build agents and agents are running nunit3-console.exe without additional modifications. We can adjust the process if you guide us, maybe setting some env var will help? |
I've accepted this as a critical bug. Confirmation should be easy from the steps you provided. We define "Critical" to mean that an immediate release is needed to fix it.
Thank you! You may be surprised to know that your single sentence is the most information I have ever received from JetBrains about how the console runner is installed under TeamCity. For 3.19, I had originally dropped the zip entirely from the distiribution, but I had a vague memory that it was being used by you so I restored it. It's not clear to me how the zip file package passed it's tests before release, but maybe I skpped a step in putting it back.
Until about 2010, I had pretty regular contact with several JetBrains guys and until five years ago the teamcity extension was maintained by one of them. I no longer have any contacts. So, I'd say that the first need is for JetBrains to re-establish some kind of contact. Of course, it is possible that some of the users who have filed issues are actually from JetBrains, but nobody has identified themselves so far, except you. :-) We already have an environment variable that tells us we are running under TeamCity. The original discussion on #1507 was because we silently fail if it is set but the extension is not installed. The discussion around fixing that led me to realize that the install process might not be working. Version 3.19 had a very significant change in how extensions are located by the engine. My tests determined it to be non-breaking but of course that's only in the light of use cases I know about. Back when there was a primary JetBrains person working on the extension. I would have asked him to review the changes. So... a primary contact for the extension would be the biggest thing we need, both to receive and give guidance about how to keep this working! Beyond this issue, there are a few other things that may need to be dealt with.
Anyway, I'll check out this bug. Feel free to contact me via email if you'd like to discuss any of this privately. I don't want to be more specific about past discussions or the people involved in a public forum. |
@vmayushan I confirm this bug and I'll work on a fix. As a temporary workaround, you can install the NUnit.Console nuget package into the same directory in which you unzipped the package. The extensions installed alongside the bin directory will be discovered. As a further step, you could create your own zip package from this structure, possibly eliminating the extra copy of the console runner and the addins subdirectory. |
Ironically, the workaround explains why my package tests passed and allowed the release to go ahead. It happens that the zip package tests are run last of all the package types. The extensions installed by the nuget tests are already in place and the new algorithm used in 3.19 finds them. Running the zip package tests alone shows the problem. |
@vmayushan I'm planning to resolve this by changing the format of the zip file. I need you to verify that the specific internal changes won't affect your usage. CURRENT: Extensions are stored in an PROPOSED: Extensions will be stored in multiple directories, siblings to the RATIONALE: The new algorithm for finding "standard" extensions depends on this naming pattern. I'd prefer to allow that algorithm to work for the zip so long as we retain it. The alternative would be adding adhoc code to also look for a FYI: this investigation has uncovered a problem with the new |
@vmayushan I believe this is fixed in the 3.19.1-alpha.1 build on our myget feed. I'd appreciate your trying it before I do the full release. |
@vmayushan @Quppa I rushed this out as it seemed critical to you but there hasn't been any feedback. I don't want to release it as 3.19.1 until somebody tests it in a practical setting, running under TeamCity. |
Hi @CharliePoole ! I am sorry for delayed verification, i am confirming that 3.19.1-alpha.1 works in TeamCity Although I had an issue with using package from MyGet: https://www.myget.org/feed/nunit/package/nuget/NUnit.ConsoleRunner To verify 3.19.1 I have cloned repository with corresponding tag and built it with Not related to the task: we have tried to reach you in NUnit slack, please check it when you have time. Thank you again for the quick fix! |
This issue has been resolved in version 3.19.1 The release is available on: |
Hello!
I’ve encountered an issue with NUnit Console Runner version 3.19.0. The extensions (addins) are not being loaded as they were in version 3.18.3.
How to reproduce:
bin\net462
bin\net462>nunit3-console.exe --list-extensions
The output is
Expected output is like it was in the 3.18.3
Thank you in advance for your help!
The text was updated successfully, but these errors were encountered: