Colin Cross | 2207f87 | 2021-03-24 12:39:08 -0700 | [diff] [blame] | 1 | // Copyright 2021 Google Inc. All rights reserved. |
| 2 | // |
| 3 | // Licensed under the Apache License, Version 2.0 (the "License"); |
| 4 | // you may not use this file except in compliance with the License. |
| 5 | // You may obtain a copy of the License at |
| 6 | // |
| 7 | // http://www.apache.org/licenses/LICENSE-2.0 |
| 8 | // |
| 9 | // Unless required by applicable law or agreed to in writing, software |
| 10 | // distributed under the License is distributed on an "AS IS" BASIS, |
| 11 | // WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. |
| 12 | // See the License for the specific language governing permissions and |
| 13 | // limitations under the License. |
| 14 | |
| 15 | package java |
| 16 | |
| 17 | import ( |
Pedro Loureiro | cc20350 | 2021-10-04 17:24:00 +0000 | [diff] [blame] | 18 | "fmt" |
Colin Cross | 6aa5c40 | 2021-03-24 12:28:50 -0700 | [diff] [blame] | 19 | "reflect" |
satayev | 783195c | 2021-06-23 21:49:57 +0100 | [diff] [blame] | 20 | "regexp" |
Colin Cross | 2207f87 | 2021-03-24 12:39:08 -0700 | [diff] [blame] | 21 | "strings" |
| 22 | "testing" |
| 23 | |
| 24 | "android/soong/android" |
Jihoon Kang | 3534946 | 2024-02-22 19:52:46 +0000 | [diff] [blame] | 25 | |
| 26 | "github.com/google/blueprint/proptools" |
Colin Cross | 2207f87 | 2021-03-24 12:39:08 -0700 | [diff] [blame] | 27 | ) |
| 28 | |
| 29 | func TestDroidstubs(t *testing.T) { |
Colin Cross | 844cb6a | 2025-01-29 15:53:21 -0800 | [diff] [blame^] | 30 | t.Parallel() |
Colin Cross | 2207f87 | 2021-03-24 12:39:08 -0700 | [diff] [blame] | 31 | ctx, _ := testJavaWithFS(t, ` |
| 32 | droiddoc_exported_dir { |
| 33 | name: "droiddoc-templates-sdk", |
| 34 | path: ".", |
| 35 | } |
| 36 | |
| 37 | droidstubs { |
| 38 | name: "bar-stubs", |
| 39 | srcs: ["bar-doc/a.java"], |
| 40 | api_levels_annotations_dirs: ["droiddoc-templates-sdk"], |
| 41 | api_levels_annotations_enabled: true, |
| 42 | } |
| 43 | |
| 44 | droidstubs { |
| 45 | name: "bar-stubs-other", |
| 46 | srcs: ["bar-doc/a.java"], |
| 47 | high_mem: true, |
| 48 | api_levels_annotations_dirs: ["droiddoc-templates-sdk"], |
| 49 | api_levels_annotations_enabled: true, |
| 50 | api_levels_jar_filename: "android.other.jar", |
| 51 | } |
Anton Hansson | c04a16e | 2022-05-09 09:30:26 +0000 | [diff] [blame] | 52 | |
| 53 | droidstubs { |
| 54 | name: "stubs-applying-api-versions", |
| 55 | srcs: ["bar-doc/a.java"], |
| 56 | api_levels_module: "bar-stubs-other", |
| 57 | } |
Colin Cross | 2207f87 | 2021-03-24 12:39:08 -0700 | [diff] [blame] | 58 | `, |
| 59 | map[string][]byte{ |
| 60 | "bar-doc/a.java": nil, |
| 61 | }) |
| 62 | testcases := []struct { |
| 63 | moduleName string |
| 64 | expectedJarFilename string |
Anton Hansson | c04a16e | 2022-05-09 09:30:26 +0000 | [diff] [blame] | 65 | generate_xml bool |
Colin Cross | 2207f87 | 2021-03-24 12:39:08 -0700 | [diff] [blame] | 66 | high_mem bool |
| 67 | }{ |
| 68 | { |
| 69 | moduleName: "bar-stubs", |
Anton Hansson | c04a16e | 2022-05-09 09:30:26 +0000 | [diff] [blame] | 70 | generate_xml: true, |
Colin Cross | 2207f87 | 2021-03-24 12:39:08 -0700 | [diff] [blame] | 71 | expectedJarFilename: "android.jar", |
| 72 | high_mem: false, |
| 73 | }, |
| 74 | { |
| 75 | moduleName: "bar-stubs-other", |
Anton Hansson | c04a16e | 2022-05-09 09:30:26 +0000 | [diff] [blame] | 76 | generate_xml: true, |
Colin Cross | 2207f87 | 2021-03-24 12:39:08 -0700 | [diff] [blame] | 77 | expectedJarFilename: "android.other.jar", |
| 78 | high_mem: true, |
| 79 | }, |
Anton Hansson | c04a16e | 2022-05-09 09:30:26 +0000 | [diff] [blame] | 80 | { |
| 81 | moduleName: "stubs-applying-api-versions", |
| 82 | generate_xml: false, |
| 83 | }, |
Colin Cross | 2207f87 | 2021-03-24 12:39:08 -0700 | [diff] [blame] | 84 | } |
| 85 | for _, c := range testcases { |
| 86 | m := ctx.ModuleForTests(c.moduleName, "android_common") |
Colin Cross | 8095c29 | 2021-03-30 16:40:48 -0700 | [diff] [blame] | 87 | manifest := m.Output("metalava.sbox.textproto") |
Colin Cross | f61d03d | 2023-11-02 16:56:39 -0700 | [diff] [blame] | 88 | sboxProto := android.RuleBuilderSboxProtoForTests(t, ctx, manifest) |
Anton Hansson | c04a16e | 2022-05-09 09:30:26 +0000 | [diff] [blame] | 89 | cmdline := String(sboxProto.Commands[0].Command) |
| 90 | android.AssertStringContainsEquals(t, "api-versions generation flag", cmdline, "--generate-api-levels", c.generate_xml) |
| 91 | if c.expectedJarFilename != "" { |
Paul Duffin | 9820c1a | 2025-01-20 10:35:45 +0000 | [diff] [blame] | 92 | expected := "--android-jar-pattern ./{version:major.minor?}/public/" + c.expectedJarFilename |
Anton Hansson | c04a16e | 2022-05-09 09:30:26 +0000 | [diff] [blame] | 93 | if !strings.Contains(cmdline, expected) { |
| 94 | t.Errorf("For %q, expected metalava argument %q, but was not found %q", c.moduleName, expected, cmdline) |
| 95 | } |
Colin Cross | 2207f87 | 2021-03-24 12:39:08 -0700 | [diff] [blame] | 96 | } |
| 97 | |
Colin Cross | 8095c29 | 2021-03-30 16:40:48 -0700 | [diff] [blame] | 98 | metalava := m.Rule("metalava") |
| 99 | rp := metalava.RuleParams |
Colin Cross | 2207f87 | 2021-03-24 12:39:08 -0700 | [diff] [blame] | 100 | if actual := rp.Pool != nil && strings.Contains(rp.Pool.String(), "highmem"); actual != c.high_mem { |
| 101 | t.Errorf("Expected %q high_mem to be %v, was %v", c.moduleName, c.high_mem, actual) |
| 102 | } |
| 103 | } |
| 104 | } |
| 105 | |
Pedro Loureiro | cc20350 | 2021-10-04 17:24:00 +0000 | [diff] [blame] | 106 | // runs a test for droidstubs with a customizable sdkType argument and returns |
| 107 | // the list of jar patterns that is passed as `--android-jar-pattern` |
| 108 | func getAndroidJarPatternsForDroidstubs(t *testing.T, sdkType string) []string { |
| 109 | ctx, _ := testJavaWithFS(t, fmt.Sprintf(` |
satayev | 783195c | 2021-06-23 21:49:57 +0100 | [diff] [blame] | 110 | droiddoc_exported_dir { |
| 111 | name: "some-exported-dir", |
| 112 | path: "somedir", |
| 113 | } |
| 114 | |
| 115 | droiddoc_exported_dir { |
| 116 | name: "some-other-exported-dir", |
| 117 | path: "someotherdir", |
| 118 | } |
| 119 | |
| 120 | droidstubs { |
| 121 | name: "foo-stubs", |
| 122 | srcs: ["foo-doc/a.java"], |
| 123 | api_levels_annotations_dirs: [ |
| 124 | "some-exported-dir", |
| 125 | "some-other-exported-dir", |
| 126 | ], |
| 127 | api_levels_annotations_enabled: true, |
Cole Faust | 051fa91 | 2022-10-05 12:45:42 -0700 | [diff] [blame] | 128 | api_levels_sdk_type: "%s", |
satayev | 783195c | 2021-06-23 21:49:57 +0100 | [diff] [blame] | 129 | } |
Pedro Loureiro | cc20350 | 2021-10-04 17:24:00 +0000 | [diff] [blame] | 130 | `, sdkType), |
satayev | 783195c | 2021-06-23 21:49:57 +0100 | [diff] [blame] | 131 | map[string][]byte{ |
| 132 | "foo-doc/a.java": nil, |
| 133 | }) |
| 134 | |
| 135 | m := ctx.ModuleForTests("foo-stubs", "android_common") |
| 136 | manifest := m.Output("metalava.sbox.textproto") |
Colin Cross | f61d03d | 2023-11-02 16:56:39 -0700 | [diff] [blame] | 137 | cmd := String(android.RuleBuilderSboxProtoForTests(t, ctx, manifest).Commands[0].Command) |
satayev | 783195c | 2021-06-23 21:49:57 +0100 | [diff] [blame] | 138 | r := regexp.MustCompile(`--android-jar-pattern [^ ]+/android.jar`) |
Pedro Loureiro | cc20350 | 2021-10-04 17:24:00 +0000 | [diff] [blame] | 139 | return r.FindAllString(cmd, -1) |
| 140 | } |
| 141 | |
| 142 | func TestPublicDroidstubs(t *testing.T) { |
Colin Cross | 844cb6a | 2025-01-29 15:53:21 -0800 | [diff] [blame^] | 143 | t.Parallel() |
Pedro Loureiro | cc20350 | 2021-10-04 17:24:00 +0000 | [diff] [blame] | 144 | patterns := getAndroidJarPatternsForDroidstubs(t, "public") |
| 145 | |
| 146 | android.AssertArrayString(t, "order of patterns", []string{ |
Paul Duffin | 9820c1a | 2025-01-20 10:35:45 +0000 | [diff] [blame] | 147 | "--android-jar-pattern somedir/{version:major.minor?}/public/android.jar", |
| 148 | "--android-jar-pattern someotherdir/{version:major.minor?}/public/android.jar", |
Pedro Loureiro | cc20350 | 2021-10-04 17:24:00 +0000 | [diff] [blame] | 149 | }, patterns) |
| 150 | } |
| 151 | |
| 152 | func TestSystemDroidstubs(t *testing.T) { |
Colin Cross | 844cb6a | 2025-01-29 15:53:21 -0800 | [diff] [blame^] | 153 | t.Parallel() |
Pedro Loureiro | cc20350 | 2021-10-04 17:24:00 +0000 | [diff] [blame] | 154 | patterns := getAndroidJarPatternsForDroidstubs(t, "system") |
| 155 | |
satayev | 783195c | 2021-06-23 21:49:57 +0100 | [diff] [blame] | 156 | android.AssertArrayString(t, "order of patterns", []string{ |
Paul Duffin | 9820c1a | 2025-01-20 10:35:45 +0000 | [diff] [blame] | 157 | "--android-jar-pattern somedir/{version:major.minor?}/system/android.jar", |
| 158 | "--android-jar-pattern someotherdir/{version:major.minor?}/system/android.jar", |
| 159 | "--android-jar-pattern somedir/{version:major.minor?}/public/android.jar", |
| 160 | "--android-jar-pattern someotherdir/{version:major.minor?}/public/android.jar", |
Pedro Loureiro | cc20350 | 2021-10-04 17:24:00 +0000 | [diff] [blame] | 161 | }, patterns) |
| 162 | } |
| 163 | |
| 164 | func TestModuleLibDroidstubs(t *testing.T) { |
Colin Cross | 844cb6a | 2025-01-29 15:53:21 -0800 | [diff] [blame^] | 165 | t.Parallel() |
Pedro Loureiro | cc20350 | 2021-10-04 17:24:00 +0000 | [diff] [blame] | 166 | patterns := getAndroidJarPatternsForDroidstubs(t, "module-lib") |
| 167 | |
| 168 | android.AssertArrayString(t, "order of patterns", []string{ |
Paul Duffin | 9820c1a | 2025-01-20 10:35:45 +0000 | [diff] [blame] | 169 | "--android-jar-pattern somedir/{version:major.minor?}/module-lib/android.jar", |
| 170 | "--android-jar-pattern someotherdir/{version:major.minor?}/module-lib/android.jar", |
| 171 | "--android-jar-pattern somedir/{version:major.minor?}/system/android.jar", |
| 172 | "--android-jar-pattern someotherdir/{version:major.minor?}/system/android.jar", |
| 173 | "--android-jar-pattern somedir/{version:major.minor?}/public/android.jar", |
| 174 | "--android-jar-pattern someotherdir/{version:major.minor?}/public/android.jar", |
Pedro Loureiro | cc20350 | 2021-10-04 17:24:00 +0000 | [diff] [blame] | 175 | }, patterns) |
satayev | 783195c | 2021-06-23 21:49:57 +0100 | [diff] [blame] | 176 | } |
| 177 | |
Cole Faust | 051fa91 | 2022-10-05 12:45:42 -0700 | [diff] [blame] | 178 | func TestSystemServerDroidstubs(t *testing.T) { |
| 179 | patterns := getAndroidJarPatternsForDroidstubs(t, "system-server") |
| 180 | |
| 181 | android.AssertArrayString(t, "order of patterns", []string{ |
Paul Duffin | 9820c1a | 2025-01-20 10:35:45 +0000 | [diff] [blame] | 182 | "--android-jar-pattern somedir/{version:major.minor?}/system-server/android.jar", |
| 183 | "--android-jar-pattern someotherdir/{version:major.minor?}/system-server/android.jar", |
| 184 | "--android-jar-pattern somedir/{version:major.minor?}/module-lib/android.jar", |
| 185 | "--android-jar-pattern someotherdir/{version:major.minor?}/module-lib/android.jar", |
| 186 | "--android-jar-pattern somedir/{version:major.minor?}/system/android.jar", |
| 187 | "--android-jar-pattern someotherdir/{version:major.minor?}/system/android.jar", |
| 188 | "--android-jar-pattern somedir/{version:major.minor?}/public/android.jar", |
| 189 | "--android-jar-pattern someotherdir/{version:major.minor?}/public/android.jar", |
Cole Faust | 051fa91 | 2022-10-05 12:45:42 -0700 | [diff] [blame] | 190 | }, patterns) |
| 191 | } |
| 192 | |
Colin Cross | 6aa5c40 | 2021-03-24 12:28:50 -0700 | [diff] [blame] | 193 | func TestDroidstubsSandbox(t *testing.T) { |
| 194 | ctx, _ := testJavaWithFS(t, ` |
Colin Cross | bc13992 | 2021-03-25 18:33:16 -0700 | [diff] [blame] | 195 | genrule { |
| 196 | name: "foo", |
| 197 | out: ["foo.txt"], |
| 198 | cmd: "touch $(out)", |
| 199 | } |
| 200 | |
Colin Cross | 6aa5c40 | 2021-03-24 12:28:50 -0700 | [diff] [blame] | 201 | droidstubs { |
| 202 | name: "bar-stubs", |
| 203 | srcs: ["bar-doc/a.java"], |
Colin Cross | bc13992 | 2021-03-25 18:33:16 -0700 | [diff] [blame] | 204 | |
| 205 | args: "--reference $(location :foo)", |
| 206 | arg_files: [":foo"], |
Colin Cross | 6aa5c40 | 2021-03-24 12:28:50 -0700 | [diff] [blame] | 207 | } |
| 208 | `, |
| 209 | map[string][]byte{ |
| 210 | "bar-doc/a.java": nil, |
| 211 | }) |
| 212 | |
| 213 | m := ctx.ModuleForTests("bar-stubs", "android_common") |
| 214 | metalava := m.Rule("metalava") |
| 215 | if g, w := metalava.Inputs.Strings(), []string{"bar-doc/a.java"}; !reflect.DeepEqual(w, g) { |
| 216 | t.Errorf("Expected inputs %q, got %q", w, g) |
| 217 | } |
Colin Cross | bc13992 | 2021-03-25 18:33:16 -0700 | [diff] [blame] | 218 | |
Colin Cross | f61d03d | 2023-11-02 16:56:39 -0700 | [diff] [blame] | 219 | manifest := android.RuleBuilderSboxProtoForTests(t, ctx, m.Output("metalava.sbox.textproto")) |
Cole Faust | e8561c6 | 2023-11-30 17:26:37 -0800 | [diff] [blame] | 220 | if g, w := manifest.Commands[0].GetCommand(), "reference __SBOX_SANDBOX_DIR__/out/soong/.intermediates/foo/gen/foo.txt"; !strings.Contains(g, w) { |
Colin Cross | bc13992 | 2021-03-25 18:33:16 -0700 | [diff] [blame] | 221 | t.Errorf("Expected command to contain %q, got %q", w, g) |
| 222 | } |
Colin Cross | 6aa5c40 | 2021-03-24 12:28:50 -0700 | [diff] [blame] | 223 | } |
| 224 | |
Colin Cross | 2207f87 | 2021-03-24 12:39:08 -0700 | [diff] [blame] | 225 | func TestDroidstubsWithSystemModules(t *testing.T) { |
| 226 | ctx, _ := testJava(t, ` |
| 227 | droidstubs { |
| 228 | name: "stubs-source-system-modules", |
| 229 | srcs: [ |
| 230 | "bar-doc/a.java", |
| 231 | ], |
| 232 | sdk_version: "none", |
| 233 | system_modules: "source-system-modules", |
| 234 | } |
| 235 | |
| 236 | java_library { |
| 237 | name: "source-jar", |
| 238 | srcs: [ |
| 239 | "a.java", |
| 240 | ], |
| 241 | } |
| 242 | |
| 243 | java_system_modules { |
| 244 | name: "source-system-modules", |
| 245 | libs: ["source-jar"], |
| 246 | } |
| 247 | |
| 248 | droidstubs { |
| 249 | name: "stubs-prebuilt-system-modules", |
| 250 | srcs: [ |
| 251 | "bar-doc/a.java", |
| 252 | ], |
| 253 | sdk_version: "none", |
| 254 | system_modules: "prebuilt-system-modules", |
| 255 | } |
| 256 | |
| 257 | java_import { |
| 258 | name: "prebuilt-jar", |
| 259 | jars: ["a.jar"], |
| 260 | } |
| 261 | |
| 262 | java_system_modules_import { |
| 263 | name: "prebuilt-system-modules", |
| 264 | libs: ["prebuilt-jar"], |
| 265 | } |
| 266 | `) |
| 267 | |
| 268 | checkSystemModulesUseByDroidstubs(t, ctx, "stubs-source-system-modules", "source-jar.jar") |
| 269 | |
| 270 | checkSystemModulesUseByDroidstubs(t, ctx, "stubs-prebuilt-system-modules", "prebuilt-jar.jar") |
| 271 | } |
| 272 | |
Colin Cross | 2207f87 | 2021-03-24 12:39:08 -0700 | [diff] [blame] | 273 | func checkSystemModulesUseByDroidstubs(t *testing.T, ctx *android.TestContext, moduleName string, systemJar string) { |
| 274 | metalavaRule := ctx.ModuleForTests(moduleName, "android_common").Rule("metalava") |
| 275 | var systemJars []string |
| 276 | for _, i := range metalavaRule.Implicits { |
| 277 | systemJars = append(systemJars, i.Base()) |
| 278 | } |
| 279 | if len(systemJars) < 1 || systemJars[0] != systemJar { |
| 280 | t.Errorf("inputs of %q must be []string{%q}, but was %#v.", moduleName, systemJar, systemJars) |
| 281 | } |
| 282 | } |
MÃ¥rten Kongstad | 802ae0f | 2022-07-27 13:47:32 +0200 | [diff] [blame] | 283 | |
| 284 | func TestDroidstubsWithSdkExtensions(t *testing.T) { |
| 285 | ctx, _ := testJavaWithFS(t, ` |
| 286 | droiddoc_exported_dir { |
| 287 | name: "sdk-dir", |
| 288 | path: "sdk", |
| 289 | } |
| 290 | |
| 291 | droidstubs { |
| 292 | name: "baz-stubs", |
| 293 | api_levels_annotations_dirs: ["sdk-dir"], |
| 294 | api_levels_annotations_enabled: true, |
| 295 | extensions_info_file: ":info-file", |
| 296 | } |
| 297 | |
| 298 | filegroup { |
| 299 | name: "info-file", |
| 300 | srcs: ["sdk/extensions/info.txt"], |
| 301 | } |
| 302 | `, |
| 303 | map[string][]byte{ |
| 304 | "sdk/extensions/1/public/some-mainline-module-stubs.jar": nil, |
| 305 | "sdk/extensions/info.txt": nil, |
| 306 | }) |
| 307 | m := ctx.ModuleForTests("baz-stubs", "android_common") |
| 308 | manifest := m.Output("metalava.sbox.textproto") |
Colin Cross | f61d03d | 2023-11-02 16:56:39 -0700 | [diff] [blame] | 309 | cmdline := String(android.RuleBuilderSboxProtoForTests(t, ctx, manifest).Commands[0].Command) |
Paul Duffin | 4f70729 | 2025-01-16 14:12:31 +0000 | [diff] [blame] | 310 | android.AssertStringDoesContain(t, "android-jar-pattern present", cmdline, "--android-jar-pattern sdk/extensions/{version:extension}/public/{module}.jar") |
MÃ¥rten Kongstad | 802ae0f | 2022-07-27 13:47:32 +0200 | [diff] [blame] | 311 | android.AssertStringDoesContain(t, "sdk-extensions-info present", cmdline, "--sdk-extensions-info sdk/extensions/info.txt") |
| 312 | } |
Spandan Das | 0b555e3 | 2022-11-28 18:48:51 +0000 | [diff] [blame] | 313 | |
Jihoon Kang | 3198f3c | 2023-01-26 08:08:52 +0000 | [diff] [blame] | 314 | func TestDroidStubsApiContributionGeneration(t *testing.T) { |
| 315 | ctx, _ := testJavaWithFS(t, ` |
| 316 | droidstubs { |
| 317 | name: "foo", |
| 318 | srcs: ["A/a.java"], |
| 319 | api_surface: "public", |
| 320 | check_api: { |
| 321 | current: { |
| 322 | api_file: "A/current.txt", |
| 323 | removed_api_file: "A/removed.txt", |
| 324 | } |
| 325 | } |
| 326 | } |
| 327 | `, |
| 328 | map[string][]byte{ |
| 329 | "A/a.java": nil, |
| 330 | "A/current.txt": nil, |
| 331 | "A/removed.txt": nil, |
| 332 | }, |
| 333 | ) |
| 334 | |
| 335 | ctx.ModuleForTests("foo.api.contribution", "") |
| 336 | } |
Jihoon Kang | 42b589c | 2023-02-03 22:56:13 +0000 | [diff] [blame] | 337 | |
| 338 | func TestGeneratedApiContributionVisibilityTest(t *testing.T) { |
| 339 | library_bp := ` |
| 340 | java_api_library { |
| 341 | name: "bar", |
| 342 | api_surface: "public", |
| 343 | api_contributions: ["foo.api.contribution"], |
Jihoon Kang | 5d70127 | 2024-02-15 21:53:49 +0000 | [diff] [blame] | 344 | stubs_type: "everything", |
Jihoon Kang | 42b589c | 2023-02-03 22:56:13 +0000 | [diff] [blame] | 345 | } |
| 346 | ` |
| 347 | ctx, _ := testJavaWithFS(t, ` |
| 348 | droidstubs { |
| 349 | name: "foo", |
| 350 | srcs: ["A/a.java"], |
| 351 | api_surface: "public", |
| 352 | check_api: { |
| 353 | current: { |
| 354 | api_file: "A/current.txt", |
| 355 | removed_api_file: "A/removed.txt", |
| 356 | } |
| 357 | }, |
Jihoon Kang | 063ec00 | 2023-06-28 01:16:23 +0000 | [diff] [blame] | 358 | visibility: ["//a", "//b"], |
Jihoon Kang | 42b589c | 2023-02-03 22:56:13 +0000 | [diff] [blame] | 359 | } |
| 360 | `, |
| 361 | map[string][]byte{ |
| 362 | "a/a.java": nil, |
| 363 | "a/current.txt": nil, |
| 364 | "a/removed.txt": nil, |
| 365 | "b/Android.bp": []byte(library_bp), |
| 366 | }, |
| 367 | ) |
| 368 | |
| 369 | ctx.ModuleForTests("bar", "android_common") |
| 370 | } |
Jihoon Kang | c831389 | 2023-09-20 00:54:47 +0000 | [diff] [blame] | 371 | |
Jihoon Kang | 6592e87 | 2023-12-19 01:13:16 +0000 | [diff] [blame] | 372 | func TestAconfigDeclarations(t *testing.T) { |
| 373 | result := android.GroupFixturePreparers( |
| 374 | prepareForJavaTest, |
| 375 | android.FixtureModifyProductVariables(func(variables android.FixtureProductVariables) { |
| 376 | }), |
| 377 | android.FixtureMergeMockFs(map[string][]byte{ |
| 378 | "a/A.java": nil, |
| 379 | "a/current.txt": nil, |
| 380 | "a/removed.txt": nil, |
| 381 | }), |
| 382 | ).RunTestWithBp(t, ` |
| 383 | aconfig_declarations { |
| 384 | name: "bar", |
| 385 | package: "com.example.package", |
Yu Liu | 315a53c | 2024-04-24 16:41:57 +0000 | [diff] [blame] | 386 | container: "com.android.foo", |
Jihoon Kang | 6592e87 | 2023-12-19 01:13:16 +0000 | [diff] [blame] | 387 | srcs: [ |
| 388 | "bar.aconfig", |
| 389 | ], |
| 390 | } |
| 391 | droidstubs { |
| 392 | name: "foo", |
| 393 | srcs: ["a/A.java"], |
| 394 | api_surface: "public", |
| 395 | check_api: { |
| 396 | current: { |
| 397 | api_file: "a/current.txt", |
| 398 | removed_api_file: "a/removed.txt", |
| 399 | } |
| 400 | }, |
| 401 | aconfig_declarations: [ |
| 402 | "bar", |
| 403 | ], |
| 404 | } |
| 405 | `) |
| 406 | |
| 407 | // Check that droidstubs depend on aconfig_declarations |
| 408 | android.AssertBoolEquals(t, "foo expected to depend on bar", |
| 409 | CheckModuleHasDependency(t, result.TestContext, "foo", "android_common", "bar"), true) |
| 410 | |
Jihoon Kang | d40c591 | 2024-03-05 16:12:20 +0000 | [diff] [blame] | 411 | m := result.ModuleForTests("foo", "android_common") |
Jihoon Kang | 6592e87 | 2023-12-19 01:13:16 +0000 | [diff] [blame] | 412 | android.AssertStringDoesContain(t, "foo generates revert annotations file", |
Jihoon Kang | d40c591 | 2024-03-05 16:12:20 +0000 | [diff] [blame] | 413 | strings.Join(m.AllOutputs(), ""), "revert-annotations-exportable.txt") |
Jihoon Kang | 3c89f04 | 2023-12-19 02:40:22 +0000 | [diff] [blame] | 414 | |
| 415 | // revert-annotations.txt passed to exportable stubs generation metalava command |
Jihoon Kang | d40c591 | 2024-03-05 16:12:20 +0000 | [diff] [blame] | 416 | manifest := m.Output("metalava_exportable.sbox.textproto") |
| 417 | cmdline := String(android.RuleBuilderSboxProtoForTests(t, result.TestContext, manifest).Commands[0].Command) |
| 418 | android.AssertStringDoesContain(t, "flagged api hide command not included", cmdline, "revert-annotations-exportable.txt") |
Jihoon Kang | 3c89f04 | 2023-12-19 02:40:22 +0000 | [diff] [blame] | 419 | |
| 420 | android.AssertStringDoesContain(t, "foo generates exportable stubs jar", |
Jihoon Kang | d40c591 | 2024-03-05 16:12:20 +0000 | [diff] [blame] | 421 | strings.Join(m.AllOutputs(), ""), "exportable/foo-stubs.srcjar") |
Jihoon Kang | 6592e87 | 2023-12-19 01:13:16 +0000 | [diff] [blame] | 422 | } |
Jihoon Kang | 5919815 | 2024-02-06 22:43:18 +0000 | [diff] [blame] | 423 | |
| 424 | func TestReleaseExportRuntimeApis(t *testing.T) { |
| 425 | result := android.GroupFixturePreparers( |
| 426 | prepareForJavaTest, |
| 427 | android.FixtureModifyProductVariables(func(variables android.FixtureProductVariables) { |
Jihoon Kang | 3534946 | 2024-02-22 19:52:46 +0000 | [diff] [blame] | 428 | variables.ExportRuntimeApis = proptools.BoolPtr(true) |
Jihoon Kang | 5919815 | 2024-02-06 22:43:18 +0000 | [diff] [blame] | 429 | }), |
Colin Cross | a66b463 | 2024-08-08 15:50:47 -0700 | [diff] [blame] | 430 | android.PrepareForTestWithBuildFlag("RELEASE_HIDDEN_API_EXPORTABLE_STUBS", "true"), |
Jihoon Kang | 5919815 | 2024-02-06 22:43:18 +0000 | [diff] [blame] | 431 | android.FixtureMergeMockFs(map[string][]byte{ |
| 432 | "a/A.java": nil, |
| 433 | "a/current.txt": nil, |
| 434 | "a/removed.txt": nil, |
| 435 | }), |
| 436 | ).RunTestWithBp(t, ` |
| 437 | aconfig_declarations { |
| 438 | name: "bar", |
| 439 | package: "com.example.package", |
Yu Liu | 315a53c | 2024-04-24 16:41:57 +0000 | [diff] [blame] | 440 | container: "com.android.foo", |
Jihoon Kang | 5919815 | 2024-02-06 22:43:18 +0000 | [diff] [blame] | 441 | srcs: [ |
| 442 | "bar.aconfig", |
| 443 | ], |
| 444 | } |
| 445 | droidstubs { |
| 446 | name: "foo", |
| 447 | srcs: ["a/A.java"], |
| 448 | api_surface: "public", |
| 449 | check_api: { |
| 450 | current: { |
| 451 | api_file: "a/current.txt", |
| 452 | removed_api_file: "a/removed.txt", |
| 453 | } |
| 454 | }, |
| 455 | aconfig_declarations: [ |
| 456 | "bar", |
| 457 | ], |
| 458 | } |
| 459 | `) |
| 460 | |
| 461 | m := result.ModuleForTests("foo", "android_common") |
| 462 | |
| 463 | rule := m.Output("released-flagged-apis-exportable.txt") |
| 464 | exposeWritableApisFilter := "--filter='state:ENABLED+permission:READ_ONLY' --filter='permission:READ_WRITE'" |
| 465 | android.AssertStringEquals(t, "Filter argument expected to contain READ_WRITE permissions", exposeWritableApisFilter, rule.Args["filter_args"]) |
| 466 | } |