Skip to content

Commit

Permalink
Remove bazel build from test_utils.sh.
Browse files Browse the repository at this point in the history
`bazel test` already builds all the targets. See https://bazel.build/docs/user-manual#running-tests.

"[bazel test] performs simultaneous build and test activity, building all specified targets (including any non-test targets specified on the command line)"

PiperOrigin-RevId: 513380981
  • Loading branch information
aaronnash authored and Copybara-Service committed Mar 2, 2023
1 parent dc5b8bb commit 48e47d6
Showing 1 changed file with 0 additions and 5 deletions.
5 changes: 0 additions & 5 deletions bazel/test_utils.sh
Original file line number Diff line number Diff line change
Expand Up @@ -28,12 +28,7 @@ function run_bazel_in_venv {
pip install setuptools"${SETUPTOOLS_VERSION}"
pip install wheel

# Run bazel test command. We run bazel test //... first so that the tests
# can start before all targets have been built. In particular, it takes
# around 2 minutes to build resources proto package for java - and it would
# block all cc and py tests if we run bazel build //... first.
bazel test --test_output=errors //...
bazel build //...

# Deactivate venv
deactivate
Expand Down

0 comments on commit 48e47d6

Please sign in to comment.