From c861f53234e67a4708974395d31b20ceb3b9afe0 Mon Sep 17 00:00:00 2001 From: Michael Li Date: Wed, 4 Mar 2026 17:12:10 -0500 Subject: [PATCH] test(bats): Add logging to flaky tests --- internal/tests/cli/boundary/_connect.bash | 2 +- internal/tests/cli/boundary/sessions.bats | 12 ++++++++---- internal/tests/cli/boundary/target.bats | 3 +++ 3 files changed, 12 insertions(+), 5 deletions(-) diff --git a/internal/tests/cli/boundary/_connect.bash b/internal/tests/cli/boundary/_connect.bash index 648fb3e6e0..da0ce2a1b9 100644 --- a/internal/tests/cli/boundary/_connect.bash +++ b/internal/tests/cli/boundary/_connect.bash @@ -18,4 +18,4 @@ function connect_alias_with_host_id() { local hostid=$2 # Note: When this command returns, the session immediately goes into a "canceling" state echo "foo" | boundary connect $alias -host-id $hostid -exec nc -- {{boundary.ip}} {{boundary.port}} -} \ No newline at end of file +} diff --git a/internal/tests/cli/boundary/sessions.bats b/internal/tests/cli/boundary/sessions.bats index fd4f3bfe90..e2f04c12de 100644 --- a/internal/tests/cli/boundary/sessions.bats +++ b/internal/tests/cli/boundary/sessions.bats @@ -11,12 +11,14 @@ load _helpers [ "$status" -eq 0 ] run connect_nc $DEFAULT_TARGET - echo "$output" + echo "connecting to $id: $output" + echo "status: $status" [ "$status" -eq 0 ] # Run twice so we have two values for later testing run connect_nc $DEFAULT_TARGET - echo "$output" + echo "connecting to $id: $output" + echo "status: $status" [ "$status" -eq 0 ] } @@ -26,12 +28,14 @@ load _helpers [ "$status" -eq 0 ] run connect_nc $DEFAULT_TARGET - echo "$output" + echo "connecting to $id: $output" + echo "status: $status" [ "$status" -eq 0 ] # Run twice so we have two values for later testing run connect_nc $DEFAULT_TARGET - echo "$output" + echo "connecting to $id: $output" + echo "status: $status" [ "$status" -eq 0 ] } diff --git a/internal/tests/cli/boundary/target.bats b/internal/tests/cli/boundary/target.bats index 4f8e953ae6..559194be2a 100644 --- a/internal/tests/cli/boundary/target.bats +++ b/internal/tests/cli/boundary/target.bats @@ -29,6 +29,8 @@ load _target_host_sources @test "boundary/target/connect: unpriv user can connect to default target" { run connect_nc $DEFAULT_TARGET + echo "connecting to $id: $output" + echo "status: $status" [ "$status" -eq 0 ] } @@ -87,6 +89,7 @@ load _target_host_sources local id=$(target_id_from_name $DEFAULT_P_ID $TGT_NAME) run connect_nc $id echo "connecting to $id: $output" + echo "status: $status" [ "$status" -eq 0 ] }