Redirecting test error output

Author: Igal Tabachnik

test_rules_scala.sh

@@ -545,19 +545,20 @@ scala_specs2_all_tests_show_in_the_xml(){
 }
 
 scala_specs2_only_failed_test_shows_in_the_xml(){
+  set +e
   bazel test \
-    --nocache_test_results \
-    --test_output=streamed \
-    '--test_filter=scalarules.test.junit.specs2.SuiteWithOneFailingTest#specs2 tests should::fail$' \
-    test_expect_failure/scala_junit_test:specs2_failing_test
+  --nocache_test_results \
+  --test_output=streamed \
+  '--test_filter=scalarules.test.junit.specs2.SuiteWithOneFailingTest#specs2 tests should::fail$' \
+  test_expect_failure/scala_junit_test:specs2_failing_test
+  echo "got results"
   matches=$(grep -c -e "testcase name='specs2 tests should::fail'" -e "testcase name='specs2 tests should::succeed'" ./bazel-testlogs/test_expect_failure/scala_junit_test/specs2_failing_test/test.xml)
   if [ $matches -eq 1 ]; then
     return 0
   else
     echo "Expecting only one result, found more than one. Please check './bazel-testlogs/test_expect_failure/scala_junit_test/specs2_failing_test/test.xml'"
-    return 1
+  return 1
   fi
-  test -e
 }
 
 scala_specs2_junit_test_test_filter_exact_match(){
@@ -997,93 +998,93 @@ dir=$( cd "$( dirname "${BASH_SOURCE[0]}" )" && pwd )
 . "${dir}"/test_runner.sh
 runner=$(get_test_runner "${1:-local}")
 
-$runner bazel build test/...
-#$runner bazel build "test/... --all_incompatible_changes"
-$runner bazel test test/...
-$runner bazel test third_party/...
-# UnusedDependencyChecker doesn't work with strict_java_deps
-$runner bazel build "--strict_java_deps=ERROR -- test/... -test:UnusedDependencyChecker"
-#$runner bazel build "--strict_java_deps=ERROR --all_incompatible_changes -- test/... -test:UnusedDependencyChecker"
-$runner bazel test "--strict_java_deps=ERROR -- test/... -test:UnusedDependencyChecker"
-$runner test_disappearing_class
-$runner find -L ./bazel-testlogs -iname "*.xml"
-$runner xmllint_test
-$runner test_transitive_deps
-$runner test_scala_library_suite
-$runner test_repl
-$runner test_benchmark_jmh
-$runner multiple_junit_suffixes
-$runner multiple_junit_prefixes
-$runner test_scala_junit_test_can_fail
-$runner junit_generates_xml_logs
-$runner scala_library_jar_without_srcs_must_fail_on_mismatching_resource_strip_prefix
-$runner multiple_junit_patterns
-$runner test_junit_test_must_have_prefix_or_suffix
-$runner test_junit_test_errors_when_no_tests_found
-$runner scala_library_jar_without_srcs_must_include_direct_file_resources
-$runner scala_library_jar_without_srcs_must_include_filegroup_resources
-$runner scala_test_test_filters
-$runner scala_junit_test_test_filter
-$runner scala_junit_test_test_filter_custom_runner
-$runner scala_specs2_junit_test_test_filter_everything
-$runner scala_specs2_junit_test_test_filter_one_test
-$runner scala_specs2_junit_test_test_filter_whole_spec
-$runner scala_specs2_junit_test_test_filter_exact_match
-$runner scala_specs2_junit_test_test_filter_exact_match_unsafe_characters
-$runner scala_specs2_junit_test_test_filter_exact_match_escaped_and_sanitized
-$runner scala_specs2_junit_test_test_filter_match_multiple_methods
-$runner scala_specs2_exception_in_initializer_without_filter
-$runner scala_specs2_exception_in_initializer_terminates_without_timeout
-$runner scala_specs2_all_tests_show_in_the_xml
-$runner scala_specs2_only_filtered_test_shows_in_the_xml
+# $runner bazel build test/...
+# #$runner bazel build "test/... --all_incompatible_changes"
+# $runner bazel test test/...
+# $runner bazel test third_party/...
+# # UnusedDependencyChecker doesn't work with strict_java_deps
+# $runner bazel build "--strict_java_deps=ERROR -- test/... -test:UnusedDependencyChecker"
+# #$runner bazel build "--strict_java_deps=ERROR --all_incompatible_changes -- test/... -test:UnusedDependencyChecker"
+# $runner bazel test "--strict_java_deps=ERROR -- test/... -test:UnusedDependencyChecker"
+# $runner test_disappearing_class
+# $runner find -L ./bazel-testlogs -iname "*.xml"
+# $runner xmllint_test
+# $runner test_transitive_deps
+# $runner test_scala_library_suite
+# $runner test_repl
+# $runner test_benchmark_jmh
+# $runner multiple_junit_suffixes
+# $runner multiple_junit_prefixes
+# $runner test_scala_junit_test_can_fail
+# $runner junit_generates_xml_logs
+# $runner scala_library_jar_without_srcs_must_fail_on_mismatching_resource_strip_prefix
+# $runner multiple_junit_patterns
+# $runner test_junit_test_must_have_prefix_or_suffix
+# $runner test_junit_test_errors_when_no_tests_found
+# $runner scala_library_jar_without_srcs_must_include_direct_file_resources
+# $runner scala_library_jar_without_srcs_must_include_filegroup_resources
+# $runner scala_test_test_filters
+# $runner scala_junit_test_test_filter
+# $runner scala_junit_test_test_filter_custom_runner
+# $runner scala_specs2_junit_test_test_filter_everything
+# $runner scala_specs2_junit_test_test_filter_one_test
+# $runner scala_specs2_junit_test_test_filter_whole_spec
+# $runner scala_specs2_junit_test_test_filter_exact_match
+# $runner scala_specs2_junit_test_test_filter_exact_match_unsafe_characters
+# $runner scala_specs2_junit_test_test_filter_exact_match_escaped_and_sanitized
+# $runner scala_specs2_junit_test_test_filter_match_multiple_methods
+# $runner scala_specs2_exception_in_initializer_without_filter
+# $runner scala_specs2_exception_in_initializer_terminates_without_timeout
+# $runner scala_specs2_all_tests_show_in_the_xml
+# $runner scala_specs2_only_filtered_test_shows_in_the_xml
 $runner scala_specs2_only_failed_test_shows_in_the_xml
-$runner scalac_jvm_flags_are_configured
-$runner javac_jvm_flags_are_configured
-$runner javac_jvm_flags_via_javacopts_are_configured
-$runner scalac_jvm_flags_are_expanded
-$runner javac_jvm_flags_are_expanded
-$runner javac_jvm_flags_via_javacopts_are_expanded
-$runner test_scala_library_expect_failure_on_missing_direct_internal_deps
-$runner test_scala_library_expect_failure_on_missing_direct_external_deps_jar
-$runner test_scala_library_expect_failure_on_missing_direct_external_deps_file_group
-$runner test_scala_library_expect_failure_on_missing_direct_deps_strict_is_disabled_by_default
-$runner test_scala_binary_expect_failure_on_missing_direct_deps
-$runner test_scala_binary_expect_failure_on_missing_direct_deps_located_in_dependency_which_is_scala_binary
-$runner test_scala_library_expect_failure_on_missing_direct_deps_warn_mode
-$runner test_scala_library_expect_failure_on_missing_direct_deps_off_mode
-$runner test_unused_dependency_checker_mode_from_scala_toolchain
-$runner test_unused_dependency_checker_mode_set_in_rule
-$runner test_unused_dependency_checker_mode_override_toolchain
-$runner test_scala_library_expect_no_recompilation_on_internal_change_of_transitive_dependency
-$runner test_multi_service_manifest
-$runner test_scala_library_expect_no_recompilation_on_internal_change_of_scala_dependency
-$runner test_scala_library_expect_no_recompilation_on_internal_change_of_java_dependency
-$runner test_scala_library_expect_no_java_recompilation_on_internal_change_of_scala_sibling
-$runner test_scala_library_expect_failure_on_missing_direct_java
-$runner test_scala_library_expect_failure_on_java_in_src_jar_when_disabled
-$runner test_scala_library_expect_failure_on_missing_direct_deps_warn_mode_java
-$runner test_scala_library_expect_better_failure_message_on_missing_transitive_dependency_labels_from_other_jvm_rules
-$runner test_scala_import_expect_failure_on_missing_direct_deps_warn_mode
-$runner bazel build "test_expect_failure/missing_direct_deps/internal_deps/... --strict_java_deps=warn"
-$runner test_scalaopts_from_scala_toolchain
-$runner test_scala_import_library_passes_labels_of_direct_deps
-$runner java_toolchain_javacopts_are_used
-$runner test_scala_classpath_resources_expect_warning_on_namespace_conflict
-$runner bazel build //test_expect_failure/proto_source_root/... --strict_proto_deps=off
-$runner scala_binary_jar_is_exposed_in_build_event_protocol
-$runner scala_test_jar_is_exposed_in_build_event_protocol
-$runner scala_junit_test_jar_is_exposed_in_build_event_protocol
-$runner test_scala_import_source_jar_should_be_fetched_when_fetch_sources_is_set_to_true
-$runner test_scala_import_source_jar_should_be_fetched_when_env_bazel_jvm_fetch_sources_is_set_to_true
-$runner test_scala_import_source_jar_should_not_be_fetched_when_env_bazel_jvm_fetch_sources_is_set_to_non_true
-$runner test_unused_dependency_checker_mode_warn
-$runner test_override_javabin
-$runner test_compilation_succeeds_with_plus_one_deps_on
-$runner test_compilation_fails_with_plus_one_deps_undefined
-$runner test_compilation_succeeds_with_plus_one_deps_on_for_external_deps
-$runner test_compilation_succeeds_with_plus_one_deps_on_also_for_exports_of_deps
-$runner test_compilation_succeeds_with_plus_one_deps_on_also_for_deps_of_exports
-$runner test_plus_one_deps_only_works_for_java_info_targets
-$runner bazel test //test/... --extra_toolchains="//test_expect_failure/plus_one_deps:plus_one_deps"
-$runner test_unused_dependency_fails_even_if_also_exists_in_plus_one_deps
-$runner test_coverage_on
+# $runner scalac_jvm_flags_are_configured
+# $runner javac_jvm_flags_are_configured
+# $runner javac_jvm_flags_via_javacopts_are_configured
+# $runner scalac_jvm_flags_are_expanded
+# $runner javac_jvm_flags_are_expanded
+# $runner javac_jvm_flags_via_javacopts_are_expanded
+# $runner test_scala_library_expect_failure_on_missing_direct_internal_deps
+# $runner test_scala_library_expect_failure_on_missing_direct_external_deps_jar
+# $runner test_scala_library_expect_failure_on_missing_direct_external_deps_file_group
+# $runner test_scala_library_expect_failure_on_missing_direct_deps_strict_is_disabled_by_default
+# $runner test_scala_binary_expect_failure_on_missing_direct_deps
+# $runner test_scala_binary_expect_failure_on_missing_direct_deps_located_in_dependency_which_is_scala_binary
+# $runner test_scala_library_expect_failure_on_missing_direct_deps_warn_mode
+# $runner test_scala_library_expect_failure_on_missing_direct_deps_off_mode
+# $runner test_unused_dependency_checker_mode_from_scala_toolchain
+# $runner test_unused_dependency_checker_mode_set_in_rule
+# $runner test_unused_dependency_checker_mode_override_toolchain
+# $runner test_scala_library_expect_no_recompilation_on_internal_change_of_transitive_dependency
+# $runner test_multi_service_manifest
+# $runner test_scala_library_expect_no_recompilation_on_internal_change_of_scala_dependency
+# $runner test_scala_library_expect_no_recompilation_on_internal_change_of_java_dependency
+# $runner test_scala_library_expect_no_java_recompilation_on_internal_change_of_scala_sibling
+# $runner test_scala_library_expect_failure_on_missing_direct_java
+# $runner test_scala_library_expect_failure_on_java_in_src_jar_when_disabled
+# $runner test_scala_library_expect_failure_on_missing_direct_deps_warn_mode_java
+# $runner test_scala_library_expect_better_failure_message_on_missing_transitive_dependency_labels_from_other_jvm_rules
+# $runner test_scala_import_expect_failure_on_missing_direct_deps_warn_mode
+# $runner bazel build "test_expect_failure/missing_direct_deps/internal_deps/... --strict_java_deps=warn"
+# $runner test_scalaopts_from_scala_toolchain
+# $runner test_scala_import_library_passes_labels_of_direct_deps
+# $runner java_toolchain_javacopts_are_used
+# $runner test_scala_classpath_resources_expect_warning_on_namespace_conflict
+# $runner bazel build //test_expect_failure/proto_source_root/... --strict_proto_deps=off
+# $runner scala_binary_jar_is_exposed_in_build_event_protocol
+# $runner scala_test_jar_is_exposed_in_build_event_protocol
+# $runner scala_junit_test_jar_is_exposed_in_build_event_protocol
+# $runner test_scala_import_source_jar_should_be_fetched_when_fetch_sources_is_set_to_true
+# $runner test_scala_import_source_jar_should_be_fetched_when_env_bazel_jvm_fetch_sources_is_set_to_true
+# $runner test_scala_import_source_jar_should_not_be_fetched_when_env_bazel_jvm_fetch_sources_is_set_to_non_true
+# $runner test_unused_dependency_checker_mode_warn
+# $runner test_override_javabin
+# $runner test_compilation_succeeds_with_plus_one_deps_on
+# $runner test_compilation_fails_with_plus_one_deps_undefined
+# $runner test_compilation_succeeds_with_plus_one_deps_on_for_external_deps
+# $runner test_compilation_succeeds_with_plus_one_deps_on_also_for_exports_of_deps
+# $runner test_compilation_succeeds_with_plus_one_deps_on_also_for_deps_of_exports
+# $runner test_plus_one_deps_only_works_for_java_info_targets
+# $runner bazel test //test/... --extra_toolchains="//test_expect_failure/plus_one_deps:plus_one_deps"
+# $runner test_unused_dependency_fails_even_if_also_exists_in_plus_one_deps
+# $runner test_coverage_on