Skip to content

Commit

Permalink
Remove unused androidx.test.annotation.
Browse files Browse the repository at this point in the history
androidx.test.annotation library only contained a single internal class: ExperimentalTestApi
which is now unused. Since this is now unused there is no need to reference or publish the
androidx.test:annotation library.

This change removes build references to androidx.test.annotation from within androidx.test
and remove maven publishing build logic.

A future change will remove the source entirely.

PiperOrigin-RevId: 633624350
  • Loading branch information
brettchabot authored and copybara-androidxtest committed May 14, 2024
1 parent b33f54c commit d0379fc
Show file tree
Hide file tree
Showing 30 changed files with 16 additions and 55 deletions.
1 change: 0 additions & 1 deletion BUILD
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,6 @@ maven_repository(
name = "axt_m2repository",
testonly = 1,
srcs = [
"//annotation/java/androidx/test/annotation:annotation_maven_artifact",
"//core/java/androidx/test/core:core_maven_artifact",
"//espresso/accessibility/java/androidx/test/espresso/accessibility:accessibility_checks_maven_artifact",
"//espresso/contrib/java/androidx/test/espresso/contrib:espresso_contrib_maven_artifact",
Expand Down
31 changes: 0 additions & 31 deletions annotation/java/androidx/test/annotation/BUILD
Original file line number Diff line number Diff line change
@@ -1,10 +1,6 @@
# Description: Build rules for building androidx.test.annotation from source

load("@build_bazel_rules_android//android:rules.bzl", "android_library")
load("//build_extensions:api_checks.bzl", "api_checks")
load("//build_extensions:dackka_test.bzl", "dackka_test")
load("//build_extensions/maven:axt_android_aar.bzl", "axt_android_aar")
load("//build_extensions/maven:maven_artifact.bzl", "maven_artifact")

# all users should reference the equivalent targets in //annotation
package(
Expand All @@ -26,30 +22,3 @@ android_library(
"@maven//:androidx_annotation_annotation_experimental",
],
)

# Generate rules for the release artifacts.
axt_android_aar(
name = "annotation_aar",
expected_class_prefixes = [
"androidx.test.annotation",
],
included_dep = ":annotation",
)

maven_artifact(
name = "annotation_maven_artifact",
last_updated = "20210901000000",
target = ":annotation_aar",
)

dackka_test(
name = "annotation_doc",
runtime_dep = ":annotation",
src_jar = ":annotation_aar-src.jar",
)

api_checks(
name = "annotation_api",
runtime_dep = ":annotation",
src_jar = ":annotation_aar-src.jar",
)
1 change: 0 additions & 1 deletion build_extensions/axt_released_versions.bzl
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,5 @@ CORE_VERSION = "1.6.0-alpha06"
ESPRESSO_DEVICE_VERSION = "1.0.0-alpha09"
ANDROIDX_JUNIT_VERSION = "1.2.0-alpha04"
ANDROIDX_TRUTH_VERSION = "1.6.0-alpha04"
ANNOTATION_VERSION = "1.1.0-alpha04"
ORCHESTRATOR_VERSION = "1.5.0-alpha04"
SERVICES_VERSION = "1.5.0-alpha04"
1 change: 0 additions & 1 deletion build_extensions/axt_versions.bzl
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,6 @@ CORE_VERSION = "1.6.0-alpha07"
ESPRESSO_DEVICE_VERSION = "1.0.0-alpha10"
ANDROIDX_JUNIT_VERSION = "1.2.0-alpha05"
ANDROIDX_TRUTH_VERSION = "1.6.0-alpha05"
ANNOTATION_VERSION = "1.1.0-alpha05"
ORCHESTRATOR_VERSION = "1.5.0-alpha05"
SERVICES_VERSION = "1.5.0-alpha05"

Expand Down
2 changes: 0 additions & 2 deletions build_extensions/maven/maven_registry.bzl
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@ load(
"//build_extensions:axt_versions.bzl",
"ANDROIDX_JUNIT_VERSION",
"ANDROIDX_TRUTH_VERSION",
"ANNOTATION_VERSION",
"CORE_VERSION",
"ESPRESSO_DEVICE_VERSION",
"ESPRESSO_VERSION",
Expand All @@ -20,7 +19,6 @@ load(
# under a given directory. Or in other words, that code in this repo is organized according to which
# maven artifact it belongs to.
_TARGET_TO_MAVEN_ARTIFACT = {
"//annotation/java/": "androidx.test:annotation:%s" % ANNOTATION_VERSION,
"//runner/android_junit_runner/java/": "androidx.test:runner:%s" % RUNNER_VERSION,
"//runner/rules/java/": "androidx.test:rules:%s" % RULES_VERSION,
"//runner/rules:rules": "androidx.test:rules:%s" % RULES_VERSION,
Expand Down
2 changes: 2 additions & 0 deletions core/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,8 @@

**Bug Fixes**

* Remove unused androidx.test.annotation dependency

**New Features**

**Breaking Changes**
Expand Down
1 change: 0 additions & 1 deletion core/java/androidx/test/core/BUILD
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,6 @@ kt_android_library(
visibility = ["//visibility:private"],
deps = [
":manifest",
"//annotation",
"//opensource/androidx:annotation",
"//runner/monitor",
"@maven//:androidx_concurrent_concurrent_futures_ktx",
Expand Down
2 changes: 2 additions & 0 deletions espresso/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,8 @@ The following artifacts were released:

**Bug Fixes**

* Remove unused androidx.test.annotation dependency

**New Features**

**Breaking Changes**
Expand Down
1 change: 0 additions & 1 deletion espresso/core/java/androidx/test/espresso/BUILD
Original file line number Diff line number Diff line change
Expand Up @@ -182,7 +182,6 @@ android_library(
srcs = BASE_LAYER_INTERFACES,
manifest = "AndroidManifest.xml",
deps = [
"//annotation",
"//espresso/core/java/androidx/test/espresso/util",
"//opensource/androidx:annotation",
"//runner/monitor",
Expand Down
1 change: 0 additions & 1 deletion espresso/core/java/androidx/test/espresso/action/BUILD
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,6 @@ kt_android_library(
javacopts = COMMON_JAVACOPTS,
deps = [
":adapter_view_protocol",
"//annotation/java/androidx/test/annotation",
"//core/java/androidx/test/core:core_not_test_only",
"//espresso/core/java/androidx/test/espresso:framework",
"//espresso/core/java/androidx/test/espresso:interface",
Expand Down
1 change: 0 additions & 1 deletion espresso/core/java/androidx/test/espresso/matcher/BUILD
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,6 @@ android_library(
"@maven//:org_hamcrest_hamcrest_library",
],
deps = [
"//annotation",
"//espresso/core/java/androidx/test/espresso:interface",
"//espresso/core/java/androidx/test/espresso/remote:interface",
"//espresso/core/java/androidx/test/espresso/remote/annotation:remote_msg_annotations",
Expand Down
2 changes: 2 additions & 0 deletions espresso/device/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,8 @@

**Bug Fixes**

* Remove unused androidx.test.annotation dependency

**New Features**

**Breaking Changes**
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,6 @@ kt_android_library(
testonly = 1,
srcs = glob(["*.kt"]),
deps = [
"//annotation/java/androidx/test/annotation",
"//espresso/device/java/androidx/test/espresso/device/common",
"//espresso/device/java/androidx/test/espresso/device/controller",
"//espresso/device/java/androidx/test/espresso/device/sizeclass",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,6 @@ kt_android_library(
"DeviceMode.kt",
],
deps = [
"//annotation/java/androidx/test/annotation",
"//espresso/core/java/androidx/test/espresso:interface",
"//opensource/androidx:annotation",
],
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,6 @@ kt_android_library(
testonly = 1,
srcs = glob(["*.kt"]),
deps = [
"//annotation/java/androidx/test/annotation",
"//espresso/device/java/androidx/test/espresso/device/common",
"//espresso/device/java/androidx/test/espresso/device/controller",
"//espresso/device/java/androidx/test/espresso/device/sizeclass",
Expand All @@ -35,7 +34,6 @@ kt_android_library(
"RequiresDisplayFilter.kt",
],
deps = [
"//annotation/java/androidx/test/annotation",
"//espresso/device/java/androidx/test/espresso/device/sizeclass",
"//runner/android_junit_runner",
"//runner/monitor",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,6 @@ kt_android_library(
testonly = 1,
srcs = glob(["*.kt"]),
deps = [
"//annotation/java/androidx/test/annotation",
"//espresso/device/java/androidx/test/espresso/device",
"//espresso/device/java/androidx/test/espresso/device/action",
"//espresso/device/java/androidx/test/espresso/device/common",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,5 @@ kt_android_library(
testonly = 1,
srcs = glob(["*.kt"]),
deps = [
"//annotation",
],
)
1 change: 0 additions & 1 deletion espresso/intents/java/androidx/test/espresso/intent/BUILD
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,6 @@ android_library(
deps = [
":precondition_checks",
":resolved_intent_interface",
"//annotation",
"//espresso/core/java/androidx/test/espresso",
"//espresso/core/java/androidx/test/espresso:framework",
"//espresso/core/java/androidx/test/espresso:interface",
Expand Down
2 changes: 2 additions & 0 deletions ext/truth/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,8 @@

**Bug Fixes**

* Remove unused androidx.test.annotation dependency

**New Features**

**Breaking Changes**
Expand Down
1 change: 0 additions & 1 deletion ext/truth/java/androidx/test/ext/truth/BUILD
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,6 @@ android_library(
),
manifest = "AndroidManifest.xml",
deps = [
"//annotation",
"//core",
"//opensource/androidx:annotation",
"@maven//:com_google_guava_guava",
Expand Down
2 changes: 0 additions & 2 deletions gradle-tests/BUILD.bazel
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@ load(
"//build_extensions:axt_versions.bzl",
"ANDROIDX_JUNIT_VERSION",
"ANDROIDX_TRUTH_VERSION",
"ANNOTATION_VERSION",
"CORE_VERSION",
"ESPRESSO_VERSION",
"MONITOR_VERSION",
Expand All @@ -26,7 +25,6 @@ expand_template(
substitutions = {
"{ANDROIDX_JUNIT_VERSION}": ANDROIDX_JUNIT_VERSION,
"{ANDROIDX_TRUTH_VERSION}": ANDROIDX_TRUTH_VERSION,
"{ANNOTATION_VERSION}": ANNOTATION_VERSION,
"{CORE_VERSION}": CORE_VERSION,
"{ESPRESSO_VERSION}": ESPRESSO_VERSION,
"{MONITOR_VERSION}": MONITOR_VERSION,
Expand Down
2 changes: 2 additions & 0 deletions runner/android_junit_runner/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,8 @@

**Bug Fixes**

* Remove unused androidx.test.annotation dependency

**New Features**

**Breaking Changes**
Expand Down
1 change: 0 additions & 1 deletion runner/android_junit_runner/java/androidx/test/BUILD
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,6 @@ android_library(
"//runner/monitor",
],
deps = [
"//annotation",
"//opensource/androidx:annotation",
"//runner/monitor",
"//services/events/java/androidx/test/services/events",
Expand Down
2 changes: 2 additions & 0 deletions runner/monitor/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,8 @@

**Bug Fixes**

* Remove unused androidx.test.annotation dependency

**New Features**

**Breaking Changes**
Expand Down
1 change: 0 additions & 1 deletion runner/monitor/java/androidx/test/BUILD
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,6 @@ kt_android_library(
":compiletime_hidden_apis",
":errorprone_neverlink",
":runtime_hidden_apis",
"//annotation",
"//opensource/androidx:annotation",
"@maven//:androidx_tracing_tracing",
],
Expand Down
2 changes: 1 addition & 1 deletion runner/monitor/java/androidx/test/annotation/Beta.java
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@
* annotation implies nothing about the quality or performance of the API in question, only the fact
* that it is not "API-frozen."
*
* @deprecated Use {@link ExperimentalTestApi} instead.
* @deprecated unused.
*/
@Retention(RetentionPolicy.CLASS)
@Target({
Expand Down
2 changes: 2 additions & 0 deletions runner/rules/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,8 @@

**Bug Fixes**

* Remove unused androidx.test.annotation dependency

**New Features**

**Breaking Changes**
Expand Down
1 change: 0 additions & 1 deletion runner/rules/java/androidx/test/BUILD
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,6 @@ android_library(
"proguard.cfg",
],
deps = [
"//annotation",
"//opensource/androidx:annotation",
"//runner/android_junit_runner",
"@androidsdk//:legacy_test-34",
Expand Down
1 change: 1 addition & 0 deletions services/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@
**Bug Fixes**

* Reduce HostedFile log spam
* Remove unused androidx.test.annotation dependency

**New Features**

Expand Down
1 change: 0 additions & 1 deletion services/storage/java/androidx/test/services/storage/BUILD
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,6 @@ android_library(
],
deps = [
":test_storage_constants",
"//annotation",
"//opensource/androidx:annotation",
"//runner/monitor",
"//services/storage/java/androidx/test/services/storage/file",
Expand Down

0 comments on commit d0379fc

Please sign in to comment.