From ba21089dc9ebda077500d07f5481b0e1c991e7d0 Mon Sep 17 00:00:00 2001 From: Yike Date: Fri, 30 Sep 2022 10:34:57 +0800 Subject: [PATCH] Add `compatibility-tradefed` for `libincident_test` All of a test's dependencies should be explicitly declared in the Soong build module to enable infrastructure features that ensure correctness and optimize runtime. The `FilePusher` class which is defined in `compatibility-tradefed` is required because the corresponding TF test config has a target preparer that uses the class and its specific option, `append-bitness`. Note that we would not normally need to add `compatibility-tradefed` to a test that's part of a test suite (e.g. CTS, etc) since the dependency is implicit. It is required in this case, however, because the test is not part of any suite. Bug: 220991354 Test: atest libincident_test Change-Id: I41feeab36744e25c740f0c0cbf74973e6ff97607 --- libs/incident/Android.bp | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libs/incident/Android.bp b/libs/incident/Android.bp index 547d7192090a6..ff1714da20082 100644 --- a/libs/incident/Android.bp +++ b/libs/incident/Android.bp @@ -133,4 +133,6 @@ cc_test { static_libs: [ "libgmock", ], + + host_required: ["compatibility-tradefed"], }