summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorKarl Shaffer <karlshaffer@google.com>2021-05-26 01:29:01 +0000
committerAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>2021-05-26 01:29:01 +0000
commit1343ed540e388168cae943d246769ef16b377764 (patch)
tree3cd71b152262675a51e135e8783cdeefd7f2d2f1
parent6d5488f48432fc212c2a5b52a27afba7fe7fc7c0 (diff)
parentf535899959b27da51779a6142699d7104681b9bc (diff)
downloadpesto-1343ed540e388168cae943d246769ef16b377764.tar.gz
Minor bugfixes to address changes by ATest team am: f535899959
Original change: https://android-review.googlesource.com/c/platform/build/pesto/+/1701046 Change-Id: I2c16fcd39b9fb32fd5447140931a3838b32d0269
-rw-r--r--experiments/prepare_bazel_test_env/bazelenv.py2
-rw-r--r--experiments/prepare_bazel_test_env/data/templates/build/bazel/rules/android_test.bzl.template2
-rw-r--r--experiments/prepare_bazel_test_env/data/templates/build/bazel/rules/cc_test.bzl.template2
-rw-r--r--experiments/prepare_bazel_test_env/data/templates/build/bazel/rules/java_test.bzl.template2
-rw-r--r--experiments/prepare_bazel_test_env/data/templates/tools/tradefederation/core/BUILD.bazel.template17
5 files changed, 17 insertions, 8 deletions
diff --git a/experiments/prepare_bazel_test_env/bazelenv.py b/experiments/prepare_bazel_test_env/bazelenv.py
index e42e767..c64242c 100644
--- a/experiments/prepare_bazel_test_env/bazelenv.py
+++ b/experiments/prepare_bazel_test_env/bazelenv.py
@@ -153,7 +153,7 @@ class StaticResource(Resource):
super().sync()
def clean(self):
- super().sync()
+ super().clean()
def __repr__(self):
return (f"StaticResource(stage_path={self.stage_path}, "
diff --git a/experiments/prepare_bazel_test_env/data/templates/build/bazel/rules/android_test.bzl.template b/experiments/prepare_bazel_test_env/data/templates/build/bazel/rules/android_test.bzl.template
index 458f886..bc529d4 100644
--- a/experiments/prepare_bazel_test_env/data/templates/build/bazel/rules/android_test.bzl.template
+++ b/experiments/prepare_bazel_test_env/data/templates/build/bazel/rules/android_test.bzl.template
@@ -84,7 +84,7 @@ android_test = rule(
),
"_tradefed_script_help":
attr.label(
- default=Label("//tools/tradefederation/core:script_help"), ),
+ default=Label("//tools/tradefederation/core:atest_script_help"), ),
"_tradefed_jars":
attr.label(
default=Label("//tools/tradefederation/core:tradefed_lib"), ),
diff --git a/experiments/prepare_bazel_test_env/data/templates/build/bazel/rules/cc_test.bzl.template b/experiments/prepare_bazel_test_env/data/templates/build/bazel/rules/cc_test.bzl.template
index c6f5c46..a62aa8f 100644
--- a/experiments/prepare_bazel_test_env/data/templates/build/bazel/rules/cc_test.bzl.template
+++ b/experiments/prepare_bazel_test_env/data/templates/build/bazel/rules/cc_test.bzl.template
@@ -65,7 +65,7 @@ cc_test = rule(
allow_single_file = True,
),
"_tradefed_script_help": attr.label(
- default = Label("//tools/tradefederation/core:script_help"),
+ default = Label("//tools/tradefederation/core:atest_script_help"),
),
"_tradefed_jars": attr.label(
default = Label("//tools/tradefederation/core:tradefed_lib"),
diff --git a/experiments/prepare_bazel_test_env/data/templates/build/bazel/rules/java_test.bzl.template b/experiments/prepare_bazel_test_env/data/templates/build/bazel/rules/java_test.bzl.template
index dbfb2e3..0f0f386 100644
--- a/experiments/prepare_bazel_test_env/data/templates/build/bazel/rules/java_test.bzl.template
+++ b/experiments/prepare_bazel_test_env/data/templates/build/bazel/rules/java_test.bzl.template
@@ -74,7 +74,7 @@ java_test = rule(
),
"_tradefed_script_help":
attr.label(
- default=Label("//tools/tradefederation/core:script_help"), ),
+ default=Label("//tools/tradefederation/core:atest_script_help"), ),
"_tradefed_jars":
attr.label(
default=Label("//tools/tradefederation/core:tradefed_lib"), ),
diff --git a/experiments/prepare_bazel_test_env/data/templates/tools/tradefederation/core/BUILD.bazel.template b/experiments/prepare_bazel_test_env/data/templates/tools/tradefederation/core/BUILD.bazel.template
index 3493ba6..2158fd4 100644
--- a/experiments/prepare_bazel_test_env/data/templates/tools/tradefederation/core/BUILD.bazel.template
+++ b/experiments/prepare_bazel_test_env/data/templates/tools/tradefederation/core/BUILD.bazel.template
@@ -21,13 +21,15 @@
package(default_visibility=["//visibility:public"])
_TF_JARS = glob(["{prebuilts_dir_name}/host/tradefed/*.jar"])
+_ATEST_TF_JAR_SRC = ["{prebuilts_dir_name}/host/framework/atest-tradefed.jar"]
+_ATEST_TF_JAR_DEST = ["atest-tradefed.jar"]
_TF_JARNAMES = [
f.replace("{prebuilts_dir_name}/host/tradefed/", "") for f in _TF_JARS
]
genrule(name="tradefed_lib_soong_import",
- srcs=_TF_JARS,
- outs=_TF_JARNAMES,
+ srcs=_TF_JARS + _ATEST_TF_JAR_SRC,
+ outs=_TF_JARNAMES + _ATEST_TF_JAR_DEST,
cmd="cp -t $(RULEDIR) $(SRCS)")
java_import(name="tradefed_lib", jars=[":tradefed_lib_soong_import"])
@@ -39,6 +41,13 @@ genrule(name="script_help",
outs=["script_help.sh"],
cmd="cp -t $(RULEDIR) $<")
+genrule(name="atest_script_help",
+ srcs=[
+ "{prebuilts_dir_name}/host/bin/atest_script_help.sh"
+ ],
+ outs=["atest_script_help.sh"],
+ cmd="cp -t $(RULEDIR) $<")
+
genrule(
name="tradefed",
srcs=[
@@ -55,8 +64,8 @@ genrule(
genrule(
name="atest_tradefed",
srcs=[
- ":script_help", ":tradefed_lib",
- "{prebuilts_dir_name}/host/bin/atest_tradefed.sh"
+ ":tradefed_lib", ":atest_script_help",
+ "{prebuilts_dir_name}/host/bin/atest_tradefed.sh",
],
outs=["atest_tradefed.sh"],
cmd=