Inject an automatic dep on junit for all junit_tests targets.

Review Request #4228 - Created Sept. 13, 2016 and submitted

Information
Benjy Weinberger
pants
Reviewers
pants-reviews
gmalmquist, kwlzn, mateor

It's silly to require an explicit dep.

This required creating a new "JUnit" subsystem. While there,
I moved the junit runner tool to that subsystem, so the runner
and the library are in the same place.

It's still clunkier than it needs to be to inject deps onto targets,
but that's a problem for another time.

This change removes the explicit junit dep from one of our examples,
to prove that this all works. A subsequent change will remove any
other unnecessary explicit deps on junit.

CI passes: https://travis-ci.org/pantsbuild/pants/builds/159432189

Issues

  • 0
  • 1
  • 0
  • 1
Description From Last Updated
Yi Cheng
Yi Cheng
Nick Howard (Twitter)
Stu Hood
Benjy Weinberger
Yi Cheng
Mateo Rodriguez
Mateo Rodriguez
Benjy Weinberger
Benjy Weinberger
Review request changed

Status: Closed (submitted)

Change Summary:

d2244274097b6d51236d19a5f121b02e729e9881

Loading...