Make the Android test apks depend on their run_ scripts
The dependency should actually be the other way around, but
for compatibility with GYP, it has to go this way.
BUG=599919
Review URL: https://codereview.chromium.org/1893063002
Cr-Commit-Position: refs/heads/master@{#388126}
diff --git a/testing/test.gni b/testing/test.gni
index 84c51e3..8434fc5 100644
--- a/testing/test.gni
+++ b/testing/test.gni
@@ -25,6 +25,7 @@
_output_name = invoker.output_name
}
+ _test_runner_target = "${_output_name}__test_runner_script"
_wrapper_script_vars = [
"isolate_file",
"shard_timeout",
@@ -105,6 +106,16 @@
install_script_name = "install_${invoker.output_name}"
}
deps += [ ":$_library_target" ]
+
+ # TODO(agrieve): Remove this data_dep once bots don't build the _apk
+ # target (post-GYP).
+ # It's a bit backwards for the apk to depend on the runner script, since
+ # the apk is conceptually a runtime_dep of the script. However, it is
+ # currently necessary because the bots build this _apk target directly
+ # rather than the group() below.
+ data_deps = [
+ ":$_test_runner_target",
+ ]
}
# Incremental test targets work only for .apks.
