theoretically, that test is supposed to test cycle detection in

BuildGraph

tho. we should probably get some

Graph

specific cycle detection tests in place, I suppose - but maybe that would obviate the need for the same tests in

BuildGraph