aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAditya Choudhary <caditya@google.com>2024-02-02 13:55:51 +0000
committerRonish Kalia <ronish@google.com>2024-04-22 15:15:56 +0000
commit1d87c9723885e19b83b5f0985f3a0e20e1662267 (patch)
tree68912c9ab89f215187057258c2647287cab015ee
parent7806e425e18a48a0f9e580e381cdc86ed5478a4c (diff)
downloadadb-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.bp54
-rw-r--r--crypto/tests/Android.bp6
-rw-r--r--fastdeploy/Android.bp3
-rw-r--r--pairing_auth/tests/Android.bp6
-rw-r--r--pairing_connection/tests/Android.bp6
-rw-r--r--tls/tests/Android.bp6
6 files changed, 60 insertions, 21 deletions
diff --git a/Android.bp b/Android.bp
index 2a0a6a8e..834089de 100644
--- a/Android.bp
+++ b/Android.bp
@@ -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",
+ ],
}