refactor: remove unnecessary echo output from tests

Remove all echo statements to STDOUT from test files to keep test
output clean. Tests should only output on failure.

Changes:
- Remove DEBUG echo statements from integration tests
- Remove TEST/PASS progress messages from all tests
- Remove summary "All tests passed!" messages
- Keep echo statements redirected to stderr or files

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <[email protected]>

Author: cgrindel

tools/generate_excluded_gems/generate_excluded_gems_integration_test.sh

@@ -51,7 +51,6 @@ trap cleanup_temp_dirs EXIT
 
 # Test: Buildozer updates MODULE.bazel correctly
 test_buildozer_updates() {
-  echo "TEST: Buildozer updates MODULE.bazel correctly"
 
   local temp_dir
   temp_dir="$(mktemp -d)"
@@ -105,12 +104,10 @@ EOF
   assert_match 'gemfile = "//:Gemfile"' "${module_content}" \
     "MODULE.bazel should preserve existing gemfile"
 
-  echo "PASS: Buildozer updates MODULE.bazel correctly"
 }
 
 # Test: Buildozer updates only target bundle by name
 test_buildozer_name_filtering() {
-  echo "TEST: Buildozer updates only target bundle by name"
 
   local temp_dir
   temp_dir="$(mktemp -d)"
@@ -170,12 +167,9 @@ EOF
     fail "First bundle_fetch should still exist"
   fi
 
-  echo "PASS: Buildozer updates only target bundle by name"
 }
 
 # Run all tests
 test_buildozer_updates
 test_buildozer_name_filtering
 
-echo ""
-echo "All integration tests passed!"

tools/generate_excluded_gems/generate_excluded_gems_test.sh

@@ -53,7 +53,6 @@ trap cleanup_temp_dirs EXIT
 
 # Test 1: Basic dry-run with .ruby-version
 test_basic_dry_run() {
-  echo "TEST: Basic dry-run with .ruby-version"
 
   local temp_dir
   temp_dir="$(mktemp -d)"
@@ -79,12 +78,10 @@ test_basic_dry_run() {
   assert_equal "${expected}" "${output}" \
     "Output should match expected excluded gems"
 
-  echo "PASS: Basic dry-run with .ruby-version"
 }
 
 # Test 2: Explicit Ruby version
 test_explicit_ruby_version() {
-  echo "TEST: Explicit Ruby version"
 
   local temp_dir
   temp_dir="$(mktemp -d)"
@@ -109,12 +106,10 @@ test_explicit_ruby_version() {
     fail "Output should not contain csv (non-native gem)"
   fi
 
-  echo "PASS: Explicit Ruby version"
 }
 
 # Test 3: Missing .ruby-version without --ruby-version
 test_missing_ruby_version() {
-  echo "TEST: Missing .ruby-version without --ruby-version"
 
   local temp_dir
   temp_dir="$(mktemp -d)"
@@ -133,12 +128,10 @@ test_missing_ruby_version() {
     fail "Should have failed when .ruby-version is missing"
   fi
 
-  echo "PASS: Missing .ruby-version without --ruby-version"
 }
 
 # Test 4: Ruby version not in stdgems data
 test_unsupported_ruby_version() {
-  echo "TEST: Ruby version not in stdgems data"
 
   local temp_dir
   temp_dir="$(mktemp -d)"
@@ -158,12 +151,10 @@ test_unsupported_ruby_version() {
     fail "Should have failed for unsupported Ruby version"
   fi
 
-  echo "PASS: Ruby version not in stdgems data"
 }
 
 # Test 5: Ruby 3.3 version
 test_ruby_33_version() {
-  echo "TEST: Ruby 3.3 version"
 
   local temp_dir
   temp_dir="$(mktemp -d)"
@@ -183,7 +174,6 @@ test_ruby_33_version() {
   assert_match "psych" "${output}" "Output should contain psych"
   assert_match "json" "${output}" "Output should contain json"
 
-  echo "PASS: Ruby 3.3 version"
 }
 
 # Run all tests
@@ -193,5 +183,3 @@ test_missing_ruby_version
 test_unsupported_ruby_version
 test_ruby_33_version
 
-echo ""
-echo "All tests passed!"

tools/generate_rv_checksums/generate_rv_checksums_integration_test.sh

@@ -24,15 +24,11 @@ set -e
 
 # MARK - Locate Deps
 
-echo "DEBUG: Starting to locate dependencies"
-
 assertions_sh_location=cgrindel_bazel_starlib/shlib/lib/assertions.sh
 assertions_sh="$(rlocation "${assertions_sh_location}")" \
   || (echo >&2 "Failed to locate ${assertions_sh_location}" && exit 1)
-echo "DEBUG: Found assertions_sh at ${assertions_sh}"
 # shellcheck disable=SC1090
 source "${assertions_sh}"
-echo "DEBUG: Sourced assertions_sh"
 
 generate_rv_checksums_location=rules_ruby/tools/generate_rv_checksums/generate_rv_checksums.sh
 generate_rv_checksums="$(rlocation "${generate_rv_checksums_location}")"
@@ -55,8 +51,6 @@ trap cleanup_temp_dirs EXIT
 
 # Test: Buildozer updates MODULE.bazel correctly
 test_buildozer_updates() {
-  echo "TEST: Buildozer updates MODULE.bazel correctly"
-
   local temp_dir
   temp_dir="$(mktemp -d)"
   temp_dirs+=("${temp_dir}")
@@ -120,14 +114,10 @@ EOF
   # Verify ruby_version was NOT changed (we didn't update it)
   assert_match 'ruby_version = "3.3.0"' "${module_content}" \
     "MODULE.bazel should preserve existing ruby_version"
-
-  echo "PASS: Buildozer updates MODULE.bazel correctly"
 }
 
 # Test: Buildozer updates only target toolchain by name
 test_buildozer_name_filtering() {
-  echo "TEST: Buildozer updates only target toolchain by name"
-
   local temp_dir
   temp_dir="$(mktemp -d)"
   temp_dirs+=("${temp_dir}")
@@ -181,13 +171,8 @@ EOF
   if ! grep -B5 'name = "ruby_alt"' MODULE.bazel | grep -q 'name = "ruby"'; then
     fail "First toolchain should still exist"
   fi
-
-  echo "PASS: Buildozer updates only target toolchain by name"
 }
 
 # Run all tests
 test_buildozer_updates
 test_buildozer_name_filtering
-
-echo ""
-echo "All integration tests passed!"

tools/generate_rv_checksums/generate_rv_checksums_test.sh

@@ -53,7 +53,6 @@ trap cleanup_temp_dirs EXIT
 
 # Test 1: Basic dry-run with .ruby-version
 test_basic_dry_run() {
-  echo "TEST: Basic dry-run with .ruby-version"
 
   local temp_dir
   temp_dir="$(mktemp -d)"
@@ -78,12 +77,10 @@ test_basic_dry_run() {
 
   assert_equal "${expected}" "${output}" "Output should match expected checksums"
 
-  echo "PASS: Basic dry-run with .ruby-version"
 }
 
 # Test 2: Explicit Ruby version
 test_explicit_ruby_version() {
-  echo "TEST: Explicit Ruby version"
 
   local temp_dir
   temp_dir="$(mktemp -d)"
@@ -104,12 +101,10 @@ test_explicit_ruby_version() {
   assert_match "0c08c35a99f10817643d548f98012268c5433ae25a737ab4d6751336108a941d" "${output}" \
     "Output should contain correct checksum"
 
-  echo "PASS: Explicit Ruby version"
 }
 
 # Test 3: Missing .ruby-version without --ruby-version
 test_missing_ruby_version() {
-  echo "TEST: Missing .ruby-version without --ruby-version"
 
   local temp_dir
   temp_dir="$(mktemp -d)"
@@ -128,12 +123,10 @@ test_missing_ruby_version() {
     fail "Should have failed when .ruby-version is missing"
   fi
 
-  echo "PASS: Missing .ruby-version without --ruby-version"
 }
 
 # Test 4: Invalid rv_version
 test_invalid_rv_version() {
-  echo "TEST: Invalid rv_version"
 
   local temp_dir
   temp_dir="$(mktemp -d)"
@@ -150,7 +143,6 @@ test_invalid_rv_version() {
     fail "Should have failed for invalid rv_version"
   fi
 
-  echo "PASS: Invalid rv_version"
 }
 
 # Run all tests
@@ -159,5 +151,3 @@ test_explicit_ruby_version
 test_missing_ruby_version
 test_invalid_rv_version
 
-echo ""
-echo "All tests passed!"