Merge lp:~mfenniak/nunitv2/AppDomainUnloadFailure into lp:nunitv2
Proposed by
Mathieu Fenniak
Status: | Merged |
---|---|
Approved by: | Charlie Poole |
Approved revision: | 3190 |
Merge reported by: | Charlie Poole |
Merged at revision: | not available |
Proposed branch: | lp:~mfenniak/nunitv2/AppDomainUnloadFailure |
Merge into: | lp:nunitv2 |
Diff against target: |
50 lines (+22/-2) 2 files modified
src/ClientUtilities/tests/DomainManagerTests.cs (+10/-0) src/ClientUtilities/util/Services/DomainManager.cs (+12/-2) |
To merge this branch: | bzr merge lp:~mfenniak/nunitv2/AppDomainUnloadFailure |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Charlie Poole | Pending | ||
Review via email: mp+32099@code.launchpad.net |
Description of the change
Fixes an occasional unhandled exception from NUnit, when attempting to unload an AppDomain that has already been unloaded.
Unhandled Exception:
System.
at System.
at NUnit.Util.
at NUnit.Util.
at NUnit.ConsoleRu
at NUnit.ConsoleRu
The fix is to abort without error when the target AppDomain is already unloaded.
To post a comment you must log in.
Not sure why this hasn't been merged before, but I'll do it now.