From 3afff6399e4497972adacdc59abaee6e3208aa8d Mon Sep 17 00:00:00 2001 From: Philipp Schrader Date: Thu, 21 Jan 2021 07:31:14 -0800 Subject: [PATCH] Clarify test_suite behaviour in the Platforms docs The current `test_suite` behaviour in regards to incompatible target skipping is subtle and not necessarily obvious. This patch aims to clear up some of the confusion. --- site/docs/platforms.md | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/site/docs/platforms.md b/site/docs/platforms.md index 55969efd8d3220..fe4476742f3261 100644 --- a/site/docs/platforms.md +++ b/site/docs/platforms.md @@ -156,6 +156,13 @@ $ bazel build --platforms=//:myplatform //...` $ bazel build --platforms=//:myplatform //:all` ``` +Incompatible tests in a [`test_suite`](be/general.html#test_suite) are +similarly skipped if the `test_suite` is specified on the command line with +[`--expand_test_suites`](command-line-reference.html#flag--expand_test_suites). +In other words, `test_suite` targets on the command line behave like `:all` and +`...`. Using `--noexpand_test_suites` prevents expansion and causes +`test_suite` targets with incompatible tests to also be incompatible. + Explicitly specifying an incompatible target on the command line results in an error message and a failed build.