-
Notifications
You must be signed in to change notification settings - Fork 152
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
Eliminate DomainUsage setting #860
Comments
Absolutely agree with this idea - the various domain options add high complexity and complexity for a low use feature. I presume we would also remove the |
I'm now remembering that VS has some way to tell the adapter to run without creating an AppDomain. So this is another one that should involve @OsirisTerje . |
I can't remember why it was originally added, but I know I've regretted it for a long time. 😈 I think it has to do with some actions not being permitted in a secondary AppDomain. So some software can't run except in the primary AppDomain. IMO however that's what NUnitLite is for. |
@ChrisMaddock Are we ready to move this to a feature and work on it? |
I think so. 👍 |
This was merged to dev-4.0 and closed. I'm reopening it so as to replicate it in main. |
In a separate discussion, @ChrisMaddock expressed the desire to eliminate user control of how the runner and engine use AppDomains. I've been through the codebase to look at the impact and I can see it would enormously simplify a lot of code, particularly the test code. This issue is for the purpose of opening discussion on making the change. I'm flagging it as an idea for discussion by the @nunit/engine-team.
Some notes...
--domain
option would be dropped.DomainUsage
enumeration would be dropped..nunit
files, but this would have to be tested.FWIW, I favor doing this as part of a 4.0 release that resolves all the breaking issues we currently have, without necessarily adding a lot of new functionality. The new stuff can come in 4.1, 4.2, etc.
@ChrisMaddock Hope you don't mind that I added a new "breaking" label. 😄
The text was updated successfully, but these errors were encountered: