Test that CycleException happens for cycles (instead of a stack overflow). This broke recently, and I wanted to memorialize that it has been fixed.

Review Request #1686 — Created Jan. 29, 2015 and submitted — Latest diff uploaded

dturner-tw
pants
b0f0dab...
pants-reviews
benjyw, jsirois, patricklaw
Test that CycleException happens for cycles (instead of a stack overflow).  This broke recently, and I wanted to memorialize that it has been fixed.

https://travis-ci.org/pantsbuild/pants/builds/48798226

Also tested that the new test fails on 147bc99dd089488c47f8751992d6a37ed4b91d1c (it's fixed by the next commit)

Loading...