#173 Move circular dependency checking

[Glen Lewis]

Move the code for automatically checking test plans for
circular dependencies into the test planner base (or
whatever is appropriate). This is so that each test
loader can't botch the engine, forcing it into a
permanent loop.