Comment 2 for bug 928421

Revision history for this message
Charlie Poole (charlie.poole) wrote :

There are a couple of problems with this bug and the proposed patch...

1. DomainManager and ProjectService are two internal services used by
NUnit. Services never instantiate other services, since that would
require them to know about the existence of other services. All
services must be created before any of them are initialized. NUnit
does this in each of it's main executables. It's fine for a single
service like DomainManager to be instantiated and executed on its own
if you need it. That's how we test it, in fact. But if you need
multiple services, you should create and initialize them all in one
place, following a pattern similar to that used in the console runner.

2. But more significantly, it's not clear why the NAnt task needs this
service at all. It uses a TestPackage to describe the tests to be run.
A TestPackage never contains an NUnit project but has a list of
assemblies. Can you explain how it is that the ProjectService ends up
being required?

Charlie