Use Target.subsystems to expose UnknownArguments.

Review Request #2725 - Created Aug. 29, 2015 and submitted

Information
John Sirois
pants
jsirois/ignored/tighten
2089
1673685...
Reviewers
pants-reviews
gmalmquist, zundel
Previously the UnknownArguments subsystem was exposed somewhat indirectly
by the core backend, but target types themselves are scanned for subsystems
in the pants boot-up stage.

This also modernizes and expands the Target unit test and cleans up
TestUnknownArgumentsIntegration.

 src/python/pants/backend/core/register.py                             |  5 ---
 src/python/pants/base/target.py                                       |  4 +++
 tests/python/pants_test/base/test_target.py                           | 82 +++++++++++++-------------------------------------
 tests/python/pants_test/targets/test_unknown_arguments_integration.py |  3 +-
 4 files changed, 26 insertions(+), 68 deletions(-)

Ran the 2 modified tests green locally:

./pants test \
  tests/python/pants_test/base:target \
  tests/python/pants_test/targets:unknown_arguments_integration

CI went green here:
https://travis-ci.org/pantsbuild/pants/builds/77868743

John Sirois
John Sirois
Eric Ayers
John Sirois
John Sirois
Review request changed

Status: Closed (submitted)

Loading...