3
0%
main: 93%

Ran 14 Aug 2015 10:02PM UTC

Files 351

Run time 9s

Badge

Embed ▾

Committed 14 Aug 2015 09:27PM UTC coverage: 69.591% (+0.8%) from 68.814%

Job # CI_FLAGS="-fkmsrcn -u 1/2 'Unit tests for pants and pants-plugins - shard 2'"

Build Type

push

travis-ci

Committed by

baroquebobcat

Commit Message

Re-Add debug option to benchmark run task, complain on no jvm targets, add test

The jvm subsystem options change broke the bench task. This adds the debug option back in as a task option since it means something slightly different in the bench task.

It also causes the task to fail if there are no jvm targets reachable from the target roots.

I found it a little tricky to write a test that fully exercise the task in a unit test. I think an integration test would be a good idea for this, but I wanted to fix the regression first.

Testing Done:
Wrote a short test, saw it fail with the missing debug option field. Added a registration for it and saw it pass. CI in https://github.com/pantsbuild/pants/pull/1974

Bugs closed: 1931, 1974

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

Coverage Stats

14736 of 21175 relevant lines covered (69.59%)

0.7 hits per line

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

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

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

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

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

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

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

README BADGES
x