2
0%
main: 93%

Ran 28 May 2015 06:01PM UTC

Files 324

Run time 15s

Badge

Embed ▾

Committed 28 May 2015 05:13PM UTC coverage: 65.724% (-0.005%) from 65.729%

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

Build Type

push

travis-ci

Committed by

fkorotkov

Commit Message

Moved creation of per test data to testStarted method.

During an upgrade of Pants at Twitter I found out that some of our tests are using org.powermock.modules.junit4.PowerMockRunner and it doesn't call fireTestRunStarted at all. We were getting NPEs because of that.

According to the docs testRunStarted and testStarted have pretty much the same meaning except testStarted is invoked just before an execution in the same thread.

Testing Done:
Build a jar and used it with out tests and they passed.

Bugs closed: 1234, 1588

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

Coverage Stats

12828 of 19518 relevant lines covered (65.72%)

0.66 hits per line

pantsbuild / pants / 5057 / 2
0%
main: 93%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

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

pantsbuild / pants / 5057 / 2 0% main: 93%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

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

pantsbuild / pants / 5057 / 2
0%
main: 93%

README BADGES
x