Use a tear_down function in build_event_stream_test.sh to shutdow…

…n the Bazel server process and delete the `bep.txt` file.

This makes the test cases independent.

PiperOrigin-RevId: 429349428

src/test/shell/integration/build_event_stream_test.sh

@@ -311,6 +311,11 @@ my_rule(name = "my_lib", outs=[])
 EOF
 }
 
+function tear_down() {
+  bazel shutdown
+  rm -f bep.txt
+}
+
 #### TESTS #############################################################
 
 function test_basic() {
@@ -882,16 +887,13 @@ function test_build_only() {
 function test_command_whitelisting() {
   # We expect the "help" command to not generate a build-event stream,
   # but the "build" command to do.
-  bazel shutdown
-  rm -f bep.txt
   bazel help --build_event_text_file=bep.txt || fail "bazel help failed"
   ( [ -f bep.txt ] && fail "bazel help generated a build-event file" ) || :
   bazel version --build_event_text_file=bep.txt || fail "bazel help failed"
   ( [ -f bep.txt ] && fail "bazel version generated a build-event file" ) || :
   bazel build --build_event_text_file=bep.txt //pkg:true \
       || fail "bazel build failed"
   [ -f bep.txt ] || fail "build did not generate requested build-event file"
-  bazel shutdown
 }
 
 function test_multiple_transports() {
@@ -991,7 +993,6 @@ function test_loading_failure() {
 }
 
 function test_visibility_failure() {
-  bazel shutdown
   (bazel build --experimental_bep_target_summary \
          --build_event_text_file=$TEST_log \
          //visibility:cannotsee && fail "build failure expected") || true
@@ -1015,7 +1016,6 @@ function test_visibility_failure() {
 function test_visibility_indirect() {
   # verify that an indirect visibility error is reported, including the
   # target that violates visibility constraints.
-  bazel shutdown
   (bazel build --build_event_text_file=$TEST_log \
          //visibility:indirect && fail "build failure expected") || true
   expect_log 'reason: ANALYSIS_FAILURE'
@@ -1028,7 +1028,6 @@ function test_visibility_indirect() {
 }
 
 function test_independent_visibility_failures() {
-  bazel shutdown
   (bazel build -k --build_event_text_file=$TEST_log \
          //visibility:indirect //visibility:indirect2 \
        && fail "build failure expected") || true
@@ -1287,15 +1286,13 @@ function test_server_pid() {
     || fail "Build failed but should have succeeded"
   cat bep.txt | grep server_pid >> "$TEST_log"
   expect_log_once "server_pid:.*$(bazel info server_pid)$"
-  rm bep.txt
 }
 
 function test_bep_report_only_important_artifacts() {
   bazel build --test_output=all --build_event_text_file=bep.txt \
     //pkg:true || fail "Build failed but should have succeeded"
   cat bep.txt >> "$TEST_log"
   expect_not_log "_hidden_top_level_INTERNAL_"
-  rm bep.txt
 }
 
 function test_starlark_flags() {
@@ -1318,7 +1315,6 @@ EOF
     --//:my_int_setting=666 \
     //pkg:true || fail "Build failed but should have succeeded"
   cat bep.txt >> "$TEST_log"
-  rm bep.txt
 
   expect_log 'option_name: "//:my_int_setting"'
   expect_log 'option_value: "666"'