Propagate 'testonly' in 'dart_action' et al.

Change-Id: I1fad726fe9a809d5917b7d9a92414be59f6157be
Reviewed-on: https://dart-review.googlesource.com/59181
Reviewed-by: Zach Anderson <zra@google.com>
Commit-Queue: Ryan Macnak <rmacnak@google.com>
This commit is contained in:
Ryan Macnak 2018-06-07 20:47:02 +00:00 committed by commit-bot@chromium.org
parent 23fff58503
commit af599046e4

View file

@ -52,6 +52,10 @@ template("_compiled_action") {
visibility = invoker.visibility visibility = invoker.visibility
} }
if (defined(invoker.testonly)) {
testonly = invoker.testonly
}
script = "$_dart_root/build/gn_run_binary.py" script = "$_dart_root/build/gn_run_binary.py"
if (defined(invoker.inputs)) { if (defined(invoker.inputs)) {
@ -111,10 +115,11 @@ template("_prebuilt_tool_action") {
not_needed(invoker, ["target"]) not_needed(invoker, ["target"])
action(target_name) { action(target_name) {
forward_variables_from(invoker, [ forward_variables_from(invoker, [
"outputs",
"deps",
"visibility",
"depfile", "depfile",
"deps",
"outputs",
"testonly",
"visibility",
]) ])
script = "$_dart_root/build/gn_run_binary.py" script = "$_dart_root/build/gn_run_binary.py"
@ -148,11 +153,12 @@ template("_prebuilt_tool_action") {
not_needed(invoker, ["binary"]) not_needed(invoker, ["binary"])
_compiled_action(target_name) { _compiled_action(target_name) {
forward_variables_from(invoker, [ forward_variables_from(invoker, [
"depfile",
"deps",
"inputs", "inputs",
"outputs", "outputs",
"deps", "testonly",
"visibility", "visibility",
"depfile",
]) ])
if (defined(invoker.script)) { if (defined(invoker.script)) {
@ -200,6 +206,7 @@ template("_prebuilt_tool_action") {
# deps # deps
# inputs # inputs
# outputs # outputs
# testonly
# visibility # visibility
template("prebuilt_dart_action") { template("prebuilt_dart_action") {
assert(defined(invoker.script), "script must be defined for $target_name") assert(defined(invoker.script), "script must be defined for $target_name")
@ -253,6 +260,7 @@ template("_prebuilt_gen_snapshot_action") {
# deps # deps
# inputs # inputs
# outputs # outputs
# testonly
# visibility # visibility
template("_built_tool_action") { template("_built_tool_action") {
assert(defined(invoker.tool), "tool must be defined for $target_name") assert(defined(invoker.tool), "tool must be defined for $target_name")
@ -273,6 +281,7 @@ template("_built_tool_action") {
"inputs", "inputs",
"outputs", "outputs",
"tool", "tool",
"testonly",
"visibility", "visibility",
]) ])
@ -317,6 +326,7 @@ template("_built_tool_action") {
# deps # deps
# inputs # inputs
# outputs # outputs
# testonly
# visibility # visibility
template("dart_action") { template("dart_action") {
assert(defined(invoker.script), "script must be defined for $target_name") assert(defined(invoker.script), "script must be defined for $target_name")
@ -331,6 +341,7 @@ template("dart_action") {
"outputs", "outputs",
"packages", "packages",
"script", "script",
"testonly",
"tool", "tool",
"visibility", "visibility",
"vm_args", "vm_args",
@ -363,6 +374,7 @@ template("dart_action") {
# deps # deps
# inputs # inputs
# outputs # outputs
# testonly
# visibility # visibility
template("dart_bootstrap_action") { template("dart_bootstrap_action") {
assert(defined(invoker.script), "script must be defined for $target_name") assert(defined(invoker.script), "script must be defined for $target_name")
@ -377,6 +389,7 @@ template("dart_bootstrap_action") {
"outputs", "outputs",
"packages", "packages",
"script", "script",
"testonly",
"tool", "tool",
"visibility", "visibility",
"vm_args", "vm_args",
@ -408,6 +421,7 @@ template("dart_bootstrap_action") {
# deps # deps
# inputs # inputs
# outputs # outputs
# testonly
# visibility # visibility
template("gen_snapshot_action") { template("gen_snapshot_action") {
assert(!defined(invoker.script), assert(!defined(invoker.script),
@ -422,6 +436,7 @@ template("gen_snapshot_action") {
"inputs", "inputs",
"outputs", "outputs",
"packages", "packages",
"testonly",
"tool", "tool",
"visibility", "visibility",
"vm_args", "vm_args",