tree eb26d5fa0a8f9b46ab502258a8e4914b9501cde2
parent 6e04ebacd075eae730e2275778d0af69adc2a74f
author Zhongpeng Lin <zplin@uber.com> 1729592795 -0700
committer Copybara-Service <copybara-worker@google.com> 1729592891 -0700

Reporting skipped test cases

`<skipped>` is a valid and common test case status supported by JUnit XML. Reporting skipped test cases as "passed" hides potential broken test cases. This PR reports them when `--test_summary=detailed`.

Before the fix:

```
INFO: Found 1 target and 1 test target...
INFO: Elapsed time: 103.095s, Critical Path: 43.23s
INFO: 476 processes: 36 internal, 440 darwin-sandbox.
INFO: Build completed successfully, 476 total actions
//foo/collect:go_default_test    PASSED in 0.6s
    PASSED  collect.TestCollect (0.0s)
    PASSED  collect.TestCollectBuildFailure (0.0s)
    PASSED  collect.TestFindComponentTestPaths (0.0s)
    PASSED  collect.TestFindComponentTestPaths/component_tests_dir_does_not_exist_in_directory_tree (0.0s)
    PASSED  collect.TestFindComponentTestPaths/component_tests_dir_does_not_exist_in_provided_directory (0.0s)
    PASSED  collect.TestFindComponentTestPaths/component_tests_dir_exists_in_directory_tree (0.0s)
    PASSED  collect.TestFindComponentTestPaths/component_tests_dir_exists_in_provided_directory (0.0s)
    PASSED  collect.TestFindComponentTestPaths/component_tests_dir_should_be_the_same (0.0s)
    PASSED  collect.TestFindComponentTestPaths/directory_above_service_root (0.0s)
    PASSED  collect.TestFindComponentTestPaths/in_component_tests_directory (0.0s)
    PASSED  collect.TestFindComponentTestPaths/multiple_component_tests_dirs (0.0s)
    PASSED  collect.TestFindPythonTestPath (0.0s)
    PASSED  collect.TestFindPythonTestPath/not_a_python_package (0.0s)
    PASSED  collect.TestFindPythonTestPath/python_tests_path_does_not_exist (0.0s)
    PASSED  collect.TestFindPythonTestPath/python_tests_path_exists (0.0s)
    PASSED  collect.TestRunComponentTests (0.0s)
    PASSED  collect.TestSetTest (0.0s)
    PASSED  collect.TestSetTest/component (0.0s)
    PASSED  collect.TestSetTest/fail (0.0s)
    PASSED  collect.TestSetTest/unit (0.0s)
    PASSED  collect.TestString (0.0s)
    PASSED  collect.TestString/should_be_component (0.0s)
    PASSED  collect.TestString/should_be_unit (0.0s)
Test cases: finished with 23 passing and 0 failing out of 23 test cases

Executed 1 out of 1 test: 1 test passes.
```

After the fix:

```
INFO: Found 1 target and 1 test target...
INFO: Elapsed time: 147.787s, Critical Path: 42.03s
INFO: 476 processes: 36 internal, 440 darwin-sandbox.
INFO: Build completed successfully, 476 total actions
//foo/collect:go_default_test    PASSED in 0.7s
    PASSED  collect.TestCollectBuildFailure (0.0s)
    PASSED  collect.TestFindComponentTestPaths (0.0s)
    PASSED  collect.TestFindComponentTestPaths/component_tests_dir_does_not_exist_in_directory_tree (0.0s)
    PASSED  collect.TestFindComponentTestPaths/component_tests_dir_does_not_exist_in_provided_directory (0.0s)
    PASSED  collect.TestFindComponentTestPaths/component_tests_dir_exists_in_directory_tree (0.0s)
    PASSED  collect.TestFindComponentTestPaths/component_tests_dir_exists_in_provided_directory (0.0s)
    PASSED  collect.TestFindComponentTestPaths/component_tests_dir_should_be_the_same (0.0s)
    PASSED  collect.TestFindComponentTestPaths/directory_above_service_root (0.0s)
    PASSED  collect.TestFindComponentTestPaths/in_component_tests_directory (0.0s)
    PASSED  collect.TestFindComponentTestPaths/multiple_component_tests_dirs (0.0s)
    PASSED  collect.TestFindPythonTestPath (0.0s)
    PASSED  collect.TestFindPythonTestPath/not_a_python_package (0.0s)
    PASSED  collect.TestFindPythonTestPath/python_tests_path_does_not_exist (0.0s)
    PASSED  collect.TestFindPythonTestPath/python_tests_path_exists (0.0s)
    PASSED  collect.TestRunComponentTests (0.0s)
    PASSED  collect.TestSetTest (0.0s)
    PASSED  collect.TestSetTest/component (0.0s)
    PASSED  collect.TestSetTest/fail (0.0s)
    PASSED  collect.TestSetTest/unit (0.0s)
    PASSED  collect.TestString (0.0s)
    PASSED  collect.TestString/should_be_component (0.0s)
    PASSED  collect.TestString/should_be_unit (0.0s)
    SKIPPED collect.TestCollect (0.0s)
Test cases: finished with 22 passing, 1 skipped and 0 failing out of 23 test cases

Executed 1 out of 1 test: 1 test passes.
```

Closes #23766.

PiperOrigin-RevId: 688477216
Change-Id: I4353d1dd4801a19ec56aae40d632d9f1061cfdbe