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

LAST BUILD BRANCH: automation/release/2.34.0.dev1
DEFAULT BRANCH: main
Ran
Files 324
Run time 404min
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: 69.599%. Remained the same
CI_FLAGS="-fkmsrcnet 'Unit tests for pants and pants-plugins'"

push

travis-ci

areitz 
Improve the error message when a tool fails to bootstrap.

Suppressing the error message when failling to bootstrap a tool makes it really hard to debug the problem.

Testing Done:
Apply this diff:

```
diff --git a/BUILD.tools b/BUILD.tools
index 81fc8d8..88dd916 100644
--- a/BUILD.tools
+++ b/BUILD.tools
@@ -117,7 +117,9 @@ jar_library(name = 'scalastyle',
 jar_library(name = 'checkstyle',
             jars = [
               jar(org = 'com.puppycrawl.tools', name = 'checkstyle', rev = '5.6'),
-            ])
+            ],
+            dependencies = [":bogus-dep"],
+            )

 jar_library(name = 'junit',
             jars = [
```

And run this command: `PANTS_DEV=1 ./pants compile examples/src/java/org/pantsbuild/example/hello::`

Before:
=======
```
16:54:13 00:01     [checkstyle]
                   Invalidated 3 targets.
                   Failed to resolve target for tool: //:checkstyle.
                   This target was obtained from option checkstyle in scope compile.checkstyle.
                   You probably need to add this target to your tools BUILD file(s), usually located in the workspace root.

FAILURE
```

After:
======
```
16:52:51 00:03     [checkstyle]
                   Invalidated 3 targets.
                   Failed to resolve target for tool: //:checkstyle.
                   This target was obtained from option checkstyle in scope compile.checkstyle.
                   You probably need to add this target to your tools BUILD file(s), usually located in the workspace root.

                   Exception: bogus-dep was not found in BUILD file /Users/areitz/workspace/pants/BUILD. Perhaps you meant one of:
                     BUILD:readme
                     BUILD.tools:antlr-3.4
                     BUILD.tools:antlr-4
                     BUILD.tools:benchmark-caliper-0.5
                     BUILD.tools:benchmark-java-allocation-instrumenter-2.1
                     BUILD.tools:checkstyle
                     BUILD.tools:cobertura-instrument
                     BUILD.tools:cobertura-report
                     BUILD.tools:cobertura-run
                     BUILD.tools:emma
                     BUILD.tools:jar-tool
                     BUILD.tools:jarjar
                     BUILD.tools:java-compiler
                     BUILD.tools:jmake
                     BUILD.tools:junit
                     BUILD.tools:nailgun-server
                     BUILD.tools:scala-compiler
                     BUILD.tools:scala-library
                     BUILD.tools:scala-repl
                     BUILD.tools:scala-specs
                     BUILD.tools:scalastyle
                     BUILD.tools:scrooge-gen
                     BUILD.tools:scrooge-linter
                     BUILD.tools:spindle-codegen
                     BUILD.tools:wire-compiler
                     BUILD.tools:wire-runtime
                     BUILD.tools:xalan
                     BUILD.tools:zinc
                     referenced from :checkstyle

FAILURE
```

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

13116 of 18845 relevant lines covered (69.6%)

0.7 hits per line

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