3
0%
main: 93%

Ran 20 Apr 2015 08:09PM UTC

Files 322

Run time 20s

Badge

Embed ▾

Committed 20 Apr 2015 07:55PM UTC coverage: 69.169% (+0.003%) from 69.166%

Job # CI_FLAGS="-fkmsrcnet 'Unit tests for pants and pants-plugins'"

Build Type

push

travis-ci

Committed by Benjy

Commit Message

Changes to zinc analysis split/merge test data generation:

- Fix various issues with the anonymizer.
- Add a mode to the test that causes it to generate canonical test data.
- Add copious comments on how to use all of the above.
- Replace the old test data tarball with loose test files that have
  non-ASCII content. Python tarfile doesn't play nicely with non-ASCII
  names (even with the pax format), so loose test files are easier.
  Git should compress them fine anyway.
- Made the test a little more comprehensive - it now verifies the merged
  analysis against an expected file.
- Made sure that split and merge don't leave dicts with empty values,
  as those create spurious diffs when comparing analyses that are
  semantically identical.

This change establishes that our split/merge logic is sound in the presence
of non-ascii file/class names in zinc analysis files.

TODO: Test that rebasing is similarly robust to non-ascii content.

Testing Done:
CI passes: https://travis-ci.org/pantsbuild/pants/builds/58981541

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

Coverage Stats

12981 of 18767 relevant lines covered (69.17%)

0.69 hits per line

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

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 4533.3 (CI_FLAGS="-fkmsrcnet 'Unit tests for pants and pants-plugins'")

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

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 4533.3 (CI_FLAGS="-fkmsrcnet 'Unit tests for pants and pants-plugins'")

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

README BADGES
x