Fix bugs with macOS test runner (#2754)
Fixes two major issues with the `macos_unit_test` runner
- Tests were always passing because:
```
xcodebuild <...> 2>&1 | tee -i "$testlog" \
    || test_exit_code=$?
```
Without `pipefail` means the exit code of the `xcodebuild` command is
dropped. This meant as of
#2649 all
`macos_unit_test` were always passing even when tests failed 🙈
- Warning when running `xcodebuild`:
```
--- xcodebuild: WARNING: Using the first of multiple matching destinations:
{ platform:macOS, arch:arm64e, id:00006031-001851843684001C, name:My Mac }
{ platform:macOS, arch:arm64, id:00006031-001851843684001C, name:My Mac }
{ platform:macOS, arch:x86_64, id:00006031-001851843684001C, name:My Mac }
{ platform:macOS, arch:arm64e, variant:Mac Catalyst, id:00006031-001851843684001C, name:My Mac }
{ platform:macOS, arch:arm64, variant:Mac Catalyst, id:00006031-001851843684001C, name:My Mac }
{ platform:macOS, arch:x86_64, variant:Mac Catalyst, id:00006031-001851843684001C, name:My Mac }
{ platform:macOS, arch:arm64e, variant:DriverKit, id:00006031-001851843684001C, name:My Mac }
{ platform:macOS, arch:arm64, variant:DriverKit, id:00006031-001851843684001C, name:My Mac }
{ platform:macOS, arch:arm64e, variant:Designed for [iPad,iPhone], id:00006031-001851843684001C, name:My Mac }
{ platform:macOS, arch:arm64, variant:Designed for [iPad,iPhone], id:00006031-001851843684001C, name:My Mac }
```
The fix here is to specify `variant=macos` and `arch` so that xcodebuild
always picks the right destination.

[doc] update docs to reflect bazel upstream changes (#2752)
bazelbuild/bazel@02df95c has removed
repo_mapping from stardoc's data source, so it's no longer emitted to
generated docs. This PR fixes CI to reflect. Note that running stardoc
without using `last_green` Bazel produces different output, because of
this change.

Fix `TEST_PREMATURE_EXIT_FILE` not being removed in `macos_test_runne…
…r` (#2735)
Signed-off-by: Brentley Jones <github@brentleyjones.com>

Support latest rules_swift (#2729)
Changes:
- Remove use of `objc_infos` (no longer provided by **rules_swift**)
- Remove use of `emit_swiftinterface` (no longer provided by
**rules_swift**)
- **NOTE**: this means `.swiftinterface` files are no longer provided by
default unless the underlying `swift_library` uses `features =
["swift.emit_swiftinterface"]` or `library_evolution = True`
- Update to use new free functions instead of `swift_common` when
available in rules_swift 3+ (e.g. `swift_common.create_swift_info` vs.
`SwiftInfo()`)
- Update tests
Tasks:
- [x] Tag a rules_swift 3.x release and update this PR to use that as an
override.

Tests: Allow post_action to determine exit status (#2713)
Add new attr `post_action_determines_exit_code` to let the test runner
post action determine the exit status. Our use case is allow soft
failing of certain test targets.
```starlark
ios_xctestrun_runner(
    name = "ios_sim_runner_with_soft_fail",
    device_type = "iPhone Xs",
    post_action = ":post_action_soft_fail",
    post_action_determines_exit_code = True,
)
```

Relax rsync path for permissions workaround in macOS 15.4+ (#2710)
This fixes `rsync` permission issues in macOS 15.4+ if users instead use
a working `rsync` binary (e.g. one from Homebrew).

Add `collect_code_coverage` attribute to tests (#2705)
Code coverage is only collected when `True` (the default). This allows
individual targets to opt out of code coverage collection, which can be
useful in some cases, like UI tests.
Signed-off-by: Brentley Jones <github@brentleyjones.com>

Fix non-`$should_use_xcodebuild` path of test runner (#2666)
Broke in fa8ce4c.
* Revert accidental inclusion of `KEEP_XCRESULT_ON_SUCCESS`
* Move `xcrun llvm-profdata merge` back to where it belongs
Signed-off-by: Brentley Jones <github@brentleyjones.com>

Add swift-testing support to the xctest runner (#2554)
This adds support for swift-testing tests by checking for both the swift
testing output as well as the normal xctest output.

Add `manual` tag to test bundle rule (#2655)
Fixes an issue related to toolchain selection when doing `//...`
multi-platform builds.
Signed-off-by: Brentley Jones <github@brentleyjones.com>