pantsbuild / pants / 7389 / 3
0%
main: 93%

LAST BUILD BRANCH: cherry-pick-23400-to-2.32.x
DEFAULT BRANCH: main
Ran
Files 33
Run time 1s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 89.583%. First build
CI_FLAGS="-fkmsrcn -u 1/2 'Unit tests for pants and pants-plugins - shard 2'"

push

travis-ci

jsirois 
Fix mixed classpaths where internal-only is needed.

DetectDuplicates, DxCompile and JarTask all make distinctions in their
processing between internal classpath elements and external ones.
Although DxCompile was handled in 2d515b55, DetectDuplicates and
JarTask were not.  This change fixes the latter two and makes the
handling of the former uniform via a new `ClasspathUtil` helper method
that can return the `internal_classpath`.

The `DuplicateDetectorTest` is re-vamped to cover duplicates between
jars, between internal targets and across both types of targets to
exercise the case that triggered
https://github.com/pantsbuild/pants/issues/2358.

The `BinaryCreateIntegrationTest.test_deploy_excludes` is unmarked fail
as a result of the `JarTask` fix and now successfully tests the case
that triggered https://github.com/pantsbuild/pants/issues/2359.

In the course of adding `ClasspathUtil.internal_classpath`,
`ClasspathUtil.compute_classpath` is removed since it was only called
by `ClasspathUtilTest`.  That test is fixed to exercise the production
codepath used by `JvmCompile`,
`ClasspathUtil.compute_classpath_for_target`.  The unused
`target_closure` is also removed from that method as well as from its
call chain through `JvmCompile`.

Testing Done:
CI went green here:
  https://travis-ci.org/pantsbuild/pants/builds/84961315

Bugs closed: 2358, 2359, 2363

Reviewed at https://rbcommons.com/s/twitter/r/2964/

1290 of 1440 relevant lines covered (89.58%)

0.9 hits per line

Source Files on job 7389.3 (CI_FLAGS="-fkmsrcn -u 1/2 'Unit tests for pants and pants-plugins - shard 2'")