diff options
author | Aditya Choudhary <caditya@google.com> | 2024-02-02 13:55:51 +0000 |
---|---|---|
committer | Ronish Kalia <ronish@google.com> | 2024-04-22 15:15:56 +0000 |
commit | 1d87c9723885e19b83b5f0985f3a0e20e1662267 (patch) | |
tree | 68912c9ab89f215187057258c2647287cab015ee | |
parent | 7806e425e18a48a0f9e580e381cdc86ed5478a4c (diff) | |
download | adb-1d87c9723885e19b83b5f0985f3a0e20e1662267.tar.gz |
Migrate Test Targets to New Android Ownership Model
This CL is created as a best effort to migrate test targets to the new Android ownership model.
It is based on historical data from repository history and insights from git blame.
Given the nature of this effort, there may be instances of incorrect attribution. If you find incorrect or unnecessary
attribution in this CL, please create a new CL to fix that.
For detailed guidelines and further information on the migration please refer to the link below,
go/new-android-ownership-model
Bug: 304529413
Test: N/A
Change-Id: Id064843b942cb1b7f7d77c99327374561e642e3b
-rw-r--r-- | Android.bp | 54 | ||||
-rw-r--r-- | crypto/tests/Android.bp | 6 | ||||
-rw-r--r-- | fastdeploy/Android.bp | 3 | ||||
-rw-r--r-- | pairing_auth/tests/Android.bp | 6 | ||||
-rw-r--r-- | pairing_connection/tests/Android.bp | 6 | ||||
-rw-r--r-- | tls/tests/Android.bp | 6 |
6 files changed, 60 insertions, 21 deletions
@@ -13,6 +13,7 @@ // limitations under the License. package { + default_team: "trendy_team_native_tools_libraries", default_applicable_licenses: ["packages_modules_adb_license"], } @@ -48,7 +49,7 @@ cc_defaults { "-Wno-missing-field-initializers", "-Wthread-safety", "-Wvla", - "-DADB_HOST=1", // overridden by adbd_defaults + "-DADB_HOST=1", // overridden by adbd_defaults "-DANDROID_BASE_UNIQUE_FD_DISABLE_IMPLICIT_CONVERSION=1", ], cpp_std: "experimental", @@ -113,7 +114,10 @@ cc_defaults { name: "adbd_defaults", defaults: ["adb_defaults"], - cflags: ["-UADB_HOST", "-DADB_HOST=0"], + cflags: [ + "-UADB_HOST", + "-DADB_HOST=0", + ], } cc_defaults { @@ -171,7 +175,7 @@ library_linking_strategy_cc_defaults { "libselinux", ], - soong_config_variables:{ + soong_config_variables: { library_linking_strategy: { prefer_static: { static_libs: [ @@ -402,7 +406,7 @@ cc_test_host { test_options: { // TODO(b/247985207) remove "no-remote" tag when b/247985207 is fixed. tags: ["no-remote"], - } + }, } cc_binary_host { @@ -430,7 +434,7 @@ cc_binary_host { generated_headers: [ "bin2c_fastdeployagent", - "bin2c_fastdeployagentscript" + "bin2c_fastdeployagentscript", ], static_libs: [ @@ -502,7 +506,10 @@ cc_binary_host { // libadbd_core contains the common sources to build libadbd and libadbd_services. cc_library_static { name: "libadbd_core", - defaults: ["adbd_defaults", "host_adbd_supported"], + defaults: [ + "adbd_defaults", + "host_adbd_supported", + ], recovery_available: true, // libminadbd wants both, as it's used to build native tests. @@ -551,7 +558,7 @@ cc_library_static { "daemon/usb.cpp", "daemon/usb_ffs.cpp", "daemon/watchdog.cpp", - ] + ], }, recovery: { exclude_shared_libs: [ @@ -559,7 +566,7 @@ cc_library_static { "libadb_pairing_connection", "libapp_processes_protos_lite", ], - } + }, }, min_sdk_version: "30", @@ -575,7 +582,10 @@ cc_library_static { cc_library { name: "libadbd_services", - defaults: ["adbd_defaults", "host_adbd_supported"], + defaults: [ + "adbd_defaults", + "host_adbd_supported", + ], recovery_available: true, compile_multilib: "both", @@ -659,7 +669,10 @@ cc_library { cc_library { name: "libadbd", - defaults: ["adbd_defaults", "host_adbd_supported"], + defaults: [ + "adbd_defaults", + "host_adbd_supported", + ], recovery_available: true, min_sdk_version: "30", apex_available: ["com.android.adbd"], @@ -706,10 +719,9 @@ cc_library { "libadb_pairing_auth", "libadb_pairing_connection", ], - } + }, }, - visibility: [ "//bootable/recovery/minadbd", "//packages/modules/adb", @@ -718,7 +730,11 @@ cc_library { cc_binary { name: "adbd", - defaults: ["adbd_defaults", "host_adbd_supported", "libadbd_binary_dependencies"], + defaults: [ + "adbd_defaults", + "host_adbd_supported", + "libadbd_binary_dependencies", + ], recovery_available: true, min_sdk_version: "30", apex_available: ["com.android.adbd"], @@ -757,7 +773,7 @@ cc_binary { "libadb_pairing_auth", "libadb_pairing_connection", ], - } + }, }, } @@ -833,7 +849,10 @@ ADBD_TEST_LIBS = [ cc_test { name: "adbd_test", - defaults: ["adbd_defaults", "libadbd_binary_dependencies"], + defaults: [ + "adbd_defaults", + "libadbd_binary_dependencies", + ], recovery_available: false, srcs: libadb_test_srcs + [ @@ -871,7 +890,10 @@ cc_test { // adbd_test uses the static version of these libraries above, so exclude them here. exclude_shared_libs: ADBD_TEST_LIBS, - test_suites: ["general-tests", "mts-adbd"], + test_suites: [ + "general-tests", + "mts-adbd", + ], require_root: true, } diff --git a/crypto/tests/Android.bp b/crypto/tests/Android.bp index d45b05d2..41e805eb 100644 --- a/crypto/tests/Android.bp +++ b/crypto/tests/Android.bp @@ -15,6 +15,7 @@ // package { + default_team: "trendy_team_native_tools_libraries", // See: http://go/android-license-faq // A large-scale-change added 'default_applicable_licenses' to import // all of the 'license_kinds' from "packages_modules_adb_license" @@ -47,5 +48,8 @@ cc_test { "libadb_sysdeps", ], - test_suites: ["general-tests", "mts-adbd"], + test_suites: [ + "general-tests", + "mts-adbd", + ], } diff --git a/fastdeploy/Android.bp b/fastdeploy/Android.bp index 84e62c62..309c6d1a 100644 --- a/fastdeploy/Android.bp +++ b/fastdeploy/Android.bp @@ -14,6 +14,7 @@ // limitations under the License. // package { + default_team: "trendy_team_framework_android_packages", // See: http://go/android-license-faq // A large-scale-change added 'default_applicable_licenses' to import // all of the 'license_kinds' from "packages_modules_adb_license" @@ -43,7 +44,7 @@ java_binary { ], dex_preopt: { enabled: false, - } + }, } android_test { diff --git a/pairing_auth/tests/Android.bp b/pairing_auth/tests/Android.bp index 9105aa5a..ed515942 100644 --- a/pairing_auth/tests/Android.bp +++ b/pairing_auth/tests/Android.bp @@ -15,6 +15,7 @@ // package { + default_team: "trendy_team_native_tools_libraries", // See: http://go/android-license-faq // A large-scale-change added 'default_applicable_licenses' to import // all of the 'license_kinds' from "packages_modules_adb_license" @@ -43,5 +44,8 @@ cc_test { "libadb_pairing_auth_static", ], - test_suites: ["general-tests", "mts-adbd"], + test_suites: [ + "general-tests", + "mts-adbd", + ], } diff --git a/pairing_connection/tests/Android.bp b/pairing_connection/tests/Android.bp index 4199cabc..2d8c1e74 100644 --- a/pairing_connection/tests/Android.bp +++ b/pairing_connection/tests/Android.bp @@ -15,6 +15,7 @@ // package { + default_team: "trendy_team_android_developer_tools", // See: http://go/android-license-faq // A large-scale-change added 'default_applicable_licenses' to import // all of the 'license_kinds' from "packages_modules_adb_license" @@ -44,5 +45,8 @@ cc_test { "libadb_pairing_server", ], - test_suites: ["general-tests", "mts-adbd"], + test_suites: [ + "general-tests", + "mts-adbd", + ], } diff --git a/tls/tests/Android.bp b/tls/tests/Android.bp index cbb3a6a4..5041d6f7 100644 --- a/tls/tests/Android.bp +++ b/tls/tests/Android.bp @@ -15,6 +15,7 @@ // package { + default_team: "trendy_team_native_tools_libraries", // See: http://go/android-license-faq // A large-scale-change added 'default_applicable_licenses' to import // all of the 'license_kinds' from "packages_modules_adb_license" @@ -47,5 +48,8 @@ cc_test { "libadb_tls_connection_static", ], - test_suites: ["general-tests", "mts-adbd"], + test_suites: [ + "general-tests", + "mts-adbd", + ], } |