Hide cycle in testprojects

Review Request #3600 - Created March 23, 2016 and submitted

Information
Stu Hood
pants
3058, 3081
Reviewers
pants-reviews
gmalmquist, molsen, peiyu

Cycle detection is tested in test_sort_targets.py, but having a cycle existing directly in the repo breaks implementations of graph parsing that validate cycles early. While testprojects is explicitly for "broken" code, keeping it ./pants listable seems important for maintaining it.

  • Move cycle detection integration test into test_build_graph_integration with mangled BUILD file names.

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

Issues

  • 0
  • 2
  • 0
  • 2
Description From Last Updated
Garrett Malmquist
Stu Hood
Peiyu Wang
Peiyu Wang
Garrett Malmquist
Stu Hood
Review request changed

Status: Closed (submitted)

Change Summary:

Merged as 5503041efa4c6b9ac892c46da8f0de5715a05322

Loading...