blob: ef638ad4e82b6a5e4464b04d14876b5fb8ba918d [file] [log] [blame]
Colin Cross3bc7ffa2017-11-22 16:19:37 -08001// Copyright 2017 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
15package java
16
17import (
Colin Crossd09b0b62018-04-18 11:06:47 -070018 "fmt"
Colin Crossa4f08812018-10-02 22:03:40 -070019 "path/filepath"
Colin Cross3bc7ffa2017-11-22 16:19:37 -080020 "reflect"
Colin Crossb69301e2017-12-01 10:48:26 -080021 "sort"
Colin Crossd09b0b62018-04-18 11:06:47 -070022 "strings"
Colin Cross3bc7ffa2017-11-22 16:19:37 -080023 "testing"
Jaewoong Junga5e5abc2019-04-26 14:31:50 -070024
25 "github.com/google/blueprint/proptools"
26
27 "android/soong/android"
28 "android/soong/cc"
Ulya Trafimovich9023b022021-03-22 16:02:28 +000029 "android/soong/dexpreopt"
Paul Duffinfb0fe9f2021-03-22 17:31:52 +000030 "android/soong/genrule"
Colin Cross3bc7ffa2017-11-22 16:19:37 -080031)
32
Paul Duffin71ae5942021-03-22 15:36:52 +000033// testApp runs tests using the prepareForJavaTest
Paul Duffin0ed42d32021-03-13 02:19:32 +000034//
35// See testJava for an explanation as to how to stop using this deprecated method.
36//
37// deprecated
38func testApp(t *testing.T, bp string) *android.TestContext {
39 t.Helper()
Paul Duffin71ae5942021-03-22 15:36:52 +000040 result := prepareForJavaTest.RunTestWithBp(t, bp)
Paul Duffin0ed42d32021-03-13 02:19:32 +000041 return result.TestContext
42}
43
44func TestApp(t *testing.T) {
45 resourceFiles := []string{
Colin Cross3bc7ffa2017-11-22 16:19:37 -080046 "res/layout/layout.xml",
47 "res/values/strings.xml",
48 "res/values-en-rUS/strings.xml",
49 }
50
Paul Duffin0ed42d32021-03-13 02:19:32 +000051 compiledResourceFiles := []string{
Colin Cross3bc7ffa2017-11-22 16:19:37 -080052 "aapt2/res/layout_layout.xml.flat",
53 "aapt2/res/values_strings.arsc.flat",
54 "aapt2/res/values-en-rUS_strings.arsc.flat",
55 }
Colin Cross3bc7ffa2017-11-22 16:19:37 -080056
Colin Crossa97c5d32018-03-28 14:58:31 -070057 for _, moduleType := range []string{"android_app", "android_library"} {
58 t.Run(moduleType, func(t *testing.T) {
Paul Duffin71ae5942021-03-22 15:36:52 +000059 result := android.GroupFixturePreparers(
60 prepareForJavaTest,
Paul Duffin0ed42d32021-03-13 02:19:32 +000061 android.FixtureModifyMockFS(func(fs android.MockFS) {
62 for _, file := range resourceFiles {
63 fs[file] = nil
64 }
65 }),
66 ).RunTestWithBp(t, moduleType+` {
Colin Crossa97c5d32018-03-28 14:58:31 -070067 name: "foo",
68 srcs: ["a.java"],
Jeongik Cha538c0d02019-07-11 15:54:27 +090069 sdk_version: "current"
Colin Crossa97c5d32018-03-28 14:58:31 -070070 }
71 `)
Colin Cross3bc7ffa2017-11-22 16:19:37 -080072
Paul Duffin0ed42d32021-03-13 02:19:32 +000073 foo := result.ModuleForTests("foo", "android_common")
Colin Cross3bc7ffa2017-11-22 16:19:37 -080074
Colin Cross31656952018-05-24 16:11:20 -070075 var expectedLinkImplicits []string
76
77 manifestFixer := foo.Output("manifest_fixer/AndroidManifest.xml")
78 expectedLinkImplicits = append(expectedLinkImplicits, manifestFixer.Output.String())
Colin Cross3bc7ffa2017-11-22 16:19:37 -080079
Paul Duffin0ed42d32021-03-13 02:19:32 +000080 frameworkRes := result.ModuleForTests("framework-res", "android_common")
Colin Crossa97c5d32018-03-28 14:58:31 -070081 expectedLinkImplicits = append(expectedLinkImplicits,
82 frameworkRes.Output("package-res.apk").Output.String())
Colin Cross3bc7ffa2017-11-22 16:19:37 -080083
Colin Crossa97c5d32018-03-28 14:58:31 -070084 // Test the mapping from input files to compiled output file names
85 compile := foo.Output(compiledResourceFiles[0])
Paul Duffin0ed42d32021-03-13 02:19:32 +000086 android.AssertDeepEquals(t, "aapt2 compile inputs", resourceFiles, compile.Inputs.Strings())
Colin Crossb69301e2017-12-01 10:48:26 -080087
Colin Crossa97c5d32018-03-28 14:58:31 -070088 compiledResourceOutputs := compile.Outputs.Strings()
89 sort.Strings(compiledResourceOutputs)
Colin Crossb69301e2017-12-01 10:48:26 -080090
Colin Crossa97c5d32018-03-28 14:58:31 -070091 expectedLinkImplicits = append(expectedLinkImplicits, compiledResourceOutputs...)
Colin Cross3bc7ffa2017-11-22 16:19:37 -080092
Colin Crossa97c5d32018-03-28 14:58:31 -070093 list := foo.Output("aapt2/res.list")
94 expectedLinkImplicits = append(expectedLinkImplicits, list.Output.String())
Colin Cross3bc7ffa2017-11-22 16:19:37 -080095
Colin Crossa97c5d32018-03-28 14:58:31 -070096 // Check that the link rule uses
Paul Duffin0ed42d32021-03-13 02:19:32 +000097 res := result.ModuleForTests("foo", "android_common").Output("package-res.apk")
98 android.AssertDeepEquals(t, "aapt2 link implicits", expectedLinkImplicits, res.Implicits.Strings())
Colin Crossa97c5d32018-03-28 14:58:31 -070099 })
Colin Cross3bc7ffa2017-11-22 16:19:37 -0800100 }
101}
Colin Cross890ff552017-11-30 20:13:19 -0800102
Colin Crosse560c4a2019-03-19 16:03:11 -0700103func TestAppSplits(t *testing.T) {
104 ctx := testApp(t, `
105 android_app {
106 name: "foo",
107 srcs: ["a.java"],
108 package_splits: ["v4", "v7,hdpi"],
Jeongik Cha538c0d02019-07-11 15:54:27 +0900109 sdk_version: "current"
Colin Crosse560c4a2019-03-19 16:03:11 -0700110 }`)
111
112 foo := ctx.ModuleForTests("foo", "android_common")
113
114 expectedOutputs := []string{
Paul Duffinfb0fe9f2021-03-22 17:31:52 +0000115 "out/soong/.intermediates/foo/android_common/foo.apk",
116 "out/soong/.intermediates/foo/android_common/foo_v4.apk",
117 "out/soong/.intermediates/foo/android_common/foo_v7_hdpi.apk",
Colin Crosse560c4a2019-03-19 16:03:11 -0700118 }
119 for _, expectedOutput := range expectedOutputs {
120 foo.Output(expectedOutput)
121 }
122
Colin Cross41955e82019-05-29 14:40:35 -0700123 outputFiles, err := foo.Module().(*AndroidApp).OutputFiles("")
124 if err != nil {
125 t.Fatal(err)
126 }
Paul Duffinfb0fe9f2021-03-22 17:31:52 +0000127 android.AssertPathsRelativeToTopEquals(t, `OutputFiles("")`, expectedOutputs, outputFiles)
Colin Crosse560c4a2019-03-19 16:03:11 -0700128}
129
Jeongik Cha538c0d02019-07-11 15:54:27 +0900130func TestPlatformAPIs(t *testing.T) {
131 testJava(t, `
132 android_app {
133 name: "foo",
134 srcs: ["a.java"],
135 platform_apis: true,
136 }
137 `)
138
139 testJava(t, `
140 android_app {
141 name: "foo",
142 srcs: ["a.java"],
143 sdk_version: "current",
144 }
145 `)
146
Spandan Das60999342021-11-16 04:15:33 +0000147 testJavaError(t, "This module has conflicting settings. sdk_version is empty, which means that this module is build against platform APIs. However platform_apis is not set to true", `
Jeongik Cha538c0d02019-07-11 15:54:27 +0900148 android_app {
149 name: "bar",
150 srcs: ["b.java"],
151 }
152 `)
153
Spandan Das60999342021-11-16 04:15:33 +0000154 testJavaError(t, "This module has conflicting settings. sdk_version is not empty, which means this module cannot use platform APIs. However platform_apis is set to true.", `
Jeongik Cha538c0d02019-07-11 15:54:27 +0900155 android_app {
156 name: "bar",
157 srcs: ["b.java"],
158 sdk_version: "system_current",
159 platform_apis: true,
160 }
161 `)
162}
163
Jeongik Chae403e9e2019-12-07 00:16:24 +0900164func TestAndroidAppLinkType(t *testing.T) {
165 testJava(t, `
166 android_app {
167 name: "foo",
168 srcs: ["a.java"],
169 libs: ["bar"],
170 static_libs: ["baz"],
171 platform_apis: true,
172 }
173
174 java_library {
175 name: "bar",
176 sdk_version: "current",
177 srcs: ["b.java"],
178 }
179
180 android_library {
181 name: "baz",
182 sdk_version: "system_current",
183 srcs: ["c.java"],
184 }
185 `)
186
Steven Moreland00298982020-11-17 21:44:36 +0000187 testJavaError(t, "consider adjusting sdk_version: OR platform_apis:", `
Jeongik Chae403e9e2019-12-07 00:16:24 +0900188 android_app {
189 name: "foo",
190 srcs: ["a.java"],
191 libs: ["bar"],
192 sdk_version: "current",
193 static_libs: ["baz"],
194 }
195
196 java_library {
197 name: "bar",
198 sdk_version: "current",
199 srcs: ["b.java"],
200 }
201
202 android_library {
203 name: "baz",
204 sdk_version: "system_current",
205 srcs: ["c.java"],
206 }
207 `)
208
209 testJava(t, `
210 android_app {
211 name: "foo",
212 srcs: ["a.java"],
213 libs: ["bar"],
214 sdk_version: "system_current",
215 static_libs: ["baz"],
216 }
217
218 java_library {
219 name: "bar",
220 sdk_version: "current",
221 srcs: ["b.java"],
222 }
223
224 android_library {
225 name: "baz",
226 sdk_version: "system_current",
227 srcs: ["c.java"],
228 }
229 `)
230
Steven Moreland00298982020-11-17 21:44:36 +0000231 testJavaError(t, "consider adjusting sdk_version: OR platform_apis:", `
Jeongik Chae403e9e2019-12-07 00:16:24 +0900232 android_app {
233 name: "foo",
234 srcs: ["a.java"],
235 libs: ["bar"],
236 sdk_version: "system_current",
237 static_libs: ["baz"],
238 }
239
240 java_library {
241 name: "bar",
242 sdk_version: "current",
243 srcs: ["b.java"],
244 }
245
246 android_library {
247 name: "baz",
248 srcs: ["c.java"],
249 }
250 `)
251}
252
Artur Satayev2db1c3f2020-04-08 19:09:30 +0100253func TestUpdatableApps(t *testing.T) {
254 testCases := []struct {
255 name string
256 bp string
257 expectedError string
258 }{
259 {
260 name: "Stable public SDK",
261 bp: `android_app {
262 name: "foo",
263 srcs: ["a.java"],
264 sdk_version: "29",
Artur Satayevf40fc852020-04-16 13:43:02 +0100265 min_sdk_version: "29",
Artur Satayev2db1c3f2020-04-08 19:09:30 +0100266 updatable: true,
267 }`,
268 },
269 {
270 name: "Stable system SDK",
271 bp: `android_app {
272 name: "foo",
273 srcs: ["a.java"],
274 sdk_version: "system_29",
Artur Satayevf40fc852020-04-16 13:43:02 +0100275 min_sdk_version: "29",
Artur Satayev2db1c3f2020-04-08 19:09:30 +0100276 updatable: true,
277 }`,
278 },
279 {
280 name: "Current public SDK",
281 bp: `android_app {
282 name: "foo",
283 srcs: ["a.java"],
284 sdk_version: "current",
Artur Satayevf40fc852020-04-16 13:43:02 +0100285 min_sdk_version: "29",
Artur Satayev2db1c3f2020-04-08 19:09:30 +0100286 updatable: true,
287 }`,
288 },
289 {
290 name: "Current system SDK",
291 bp: `android_app {
292 name: "foo",
293 srcs: ["a.java"],
294 sdk_version: "system_current",
Artur Satayevf40fc852020-04-16 13:43:02 +0100295 min_sdk_version: "29",
Artur Satayev2db1c3f2020-04-08 19:09:30 +0100296 updatable: true,
297 }`,
298 },
299 {
300 name: "Current module SDK",
301 bp: `android_app {
302 name: "foo",
303 srcs: ["a.java"],
304 sdk_version: "module_current",
Artur Satayevf40fc852020-04-16 13:43:02 +0100305 min_sdk_version: "29",
Artur Satayev2db1c3f2020-04-08 19:09:30 +0100306 updatable: true,
307 }`,
308 },
309 {
310 name: "Current core SDK",
311 bp: `android_app {
312 name: "foo",
313 srcs: ["a.java"],
314 sdk_version: "core_current",
Artur Satayevf40fc852020-04-16 13:43:02 +0100315 min_sdk_version: "29",
Artur Satayev2db1c3f2020-04-08 19:09:30 +0100316 updatable: true,
317 }`,
318 },
319 {
320 name: "No Platform APIs",
321 bp: `android_app {
322 name: "foo",
323 srcs: ["a.java"],
324 platform_apis: true,
Artur Satayevf40fc852020-04-16 13:43:02 +0100325 min_sdk_version: "29",
Artur Satayev2db1c3f2020-04-08 19:09:30 +0100326 updatable: true,
327 }`,
328 expectedError: "Updatable apps must use stable SDKs",
329 },
330 {
331 name: "No Core Platform APIs",
332 bp: `android_app {
333 name: "foo",
334 srcs: ["a.java"],
335 sdk_version: "core_platform",
Artur Satayevf40fc852020-04-16 13:43:02 +0100336 min_sdk_version: "29",
Artur Satayev2db1c3f2020-04-08 19:09:30 +0100337 updatable: true,
338 }`,
339 expectedError: "Updatable apps must use stable SDKs",
340 },
341 {
342 name: "No unspecified APIs",
343 bp: `android_app {
344 name: "foo",
345 srcs: ["a.java"],
346 updatable: true,
Artur Satayevf40fc852020-04-16 13:43:02 +0100347 min_sdk_version: "29",
Artur Satayev2db1c3f2020-04-08 19:09:30 +0100348 }`,
349 expectedError: "Updatable apps must use stable SDK",
350 },
Artur Satayevf40fc852020-04-16 13:43:02 +0100351 {
352 name: "Must specify min_sdk_version",
353 bp: `android_app {
354 name: "app_without_min_sdk_version",
355 srcs: ["a.java"],
356 sdk_version: "29",
357 updatable: true,
358 }`,
359 expectedError: "updatable apps must set min_sdk_version.",
360 },
Artur Satayev2db1c3f2020-04-08 19:09:30 +0100361 }
362
363 for _, test := range testCases {
364 t.Run(test.name, func(t *testing.T) {
Paul Duffincdb88a92021-03-14 00:36:50 +0000365 errorHandler := android.FixtureExpectsNoErrors
366 if test.expectedError != "" {
367 errorHandler = android.FixtureExpectsAtLeastOneErrorMatchingPattern(test.expectedError)
Artur Satayev2db1c3f2020-04-08 19:09:30 +0100368 }
Paul Duffin71ae5942021-03-22 15:36:52 +0000369 android.GroupFixturePreparers(
370 prepareForJavaTest, FixtureWithPrebuiltApis(map[string][]string{
Paul Duffin2645a292021-03-13 02:36:00 +0000371 "29": {"foo"},
372 })).
Paul Duffincdb88a92021-03-14 00:36:50 +0000373 ExtendWithErrorHandler(errorHandler).RunTestWithBp(t, test.bp)
Artur Satayev2db1c3f2020-04-08 19:09:30 +0100374 })
375 }
376}
377
Jooyung Han749dc692020-04-15 11:03:39 +0900378func TestUpdatableApps_TransitiveDepsShouldSetMinSdkVersion(t *testing.T) {
379 testJavaError(t, `module "bar".*: should support min_sdk_version\(29\)`, cc.GatherRequiredDepsForTest(android.Android)+`
380 android_app {
381 name: "foo",
382 srcs: ["a.java"],
383 updatable: true,
384 sdk_version: "current",
385 min_sdk_version: "29",
386 static_libs: ["bar"],
387 }
388
389 java_library {
390 name: "bar",
391 sdk_version: "current",
392 }
393 `)
394}
395
Jooyung Hanbbc3fb72020-04-29 14:01:06 +0900396func TestUpdatableApps_JniLibsShouldShouldSupportMinSdkVersion(t *testing.T) {
397 testJava(t, cc.GatherRequiredDepsForTest(android.Android)+`
398 android_app {
399 name: "foo",
400 srcs: ["a.java"],
401 updatable: true,
402 sdk_version: "current",
403 min_sdk_version: "current",
404 jni_libs: ["libjni"],
405 }
406
407 cc_library {
408 name: "libjni",
409 stl: "none",
410 system_shared_libs: [],
411 sdk_version: "current",
412 }
413 `)
414}
415
416func TestUpdatableApps_JniLibShouldBeBuiltAgainstMinSdkVersion(t *testing.T) {
417 bp := cc.GatherRequiredDepsForTest(android.Android) + `
418 android_app {
419 name: "foo",
420 srcs: ["a.java"],
421 updatable: true,
422 sdk_version: "current",
423 min_sdk_version: "29",
424 jni_libs: ["libjni"],
425 }
426
427 cc_library {
428 name: "libjni",
429 stl: "none",
430 system_shared_libs: [],
431 sdk_version: "29",
432 }
Jooyung Hanbbc3fb72020-04-29 14:01:06 +0900433 `
434 fs := map[string][]byte{
435 "prebuilts/ndk/current/platforms/android-29/arch-arm64/usr/lib/crtbegin_so.o": nil,
436 "prebuilts/ndk/current/platforms/android-29/arch-arm64/usr/lib/crtend_so.o": nil,
437 "prebuilts/ndk/current/platforms/android-29/arch-arm/usr/lib/crtbegin_so.o": nil,
438 "prebuilts/ndk/current/platforms/android-29/arch-arm/usr/lib/crtend_so.o": nil,
439 }
440
Paul Duffinfb0fe9f2021-03-22 17:31:52 +0000441 ctx, _ := testJavaWithFS(t, bp, fs)
Jooyung Hanbbc3fb72020-04-29 14:01:06 +0900442
443 inputs := ctx.ModuleForTests("libjni", "android_arm64_armv8-a_sdk_shared").Description("link").Implicits
444 var crtbeginFound, crtendFound bool
Dan Albert92fe7402020-07-15 13:33:30 -0700445 expectedCrtBegin := ctx.ModuleForTests("crtbegin_so",
446 "android_arm64_armv8-a_sdk_29").Rule("partialLd").Output
447 expectedCrtEnd := ctx.ModuleForTests("crtend_so",
448 "android_arm64_armv8-a_sdk_29").Rule("partialLd").Output
449 implicits := []string{}
Jooyung Hanbbc3fb72020-04-29 14:01:06 +0900450 for _, input := range inputs {
Dan Albert92fe7402020-07-15 13:33:30 -0700451 implicits = append(implicits, input.String())
452 if strings.HasSuffix(input.String(), expectedCrtBegin.String()) {
Jooyung Hanbbc3fb72020-04-29 14:01:06 +0900453 crtbeginFound = true
Dan Albert92fe7402020-07-15 13:33:30 -0700454 } else if strings.HasSuffix(input.String(), expectedCrtEnd.String()) {
Jooyung Hanbbc3fb72020-04-29 14:01:06 +0900455 crtendFound = true
456 }
457 }
Dan Albert92fe7402020-07-15 13:33:30 -0700458 if !crtbeginFound {
459 t.Error(fmt.Sprintf(
460 "expected implicit with suffix %q, have the following implicits:\n%s",
461 expectedCrtBegin, strings.Join(implicits, "\n")))
462 }
463 if !crtendFound {
464 t.Error(fmt.Sprintf(
465 "expected implicit with suffix %q, have the following implicits:\n%s",
466 expectedCrtEnd, strings.Join(implicits, "\n")))
Jooyung Hanbbc3fb72020-04-29 14:01:06 +0900467 }
468}
469
470func TestUpdatableApps_ErrorIfJniLibDoesntSupportMinSdkVersion(t *testing.T) {
471 bp := cc.GatherRequiredDepsForTest(android.Android) + `
472 android_app {
473 name: "foo",
474 srcs: ["a.java"],
475 updatable: true,
476 sdk_version: "current",
477 min_sdk_version: "29", // this APK should support 29
478 jni_libs: ["libjni"],
479 }
480
481 cc_library {
482 name: "libjni",
483 stl: "none",
484 sdk_version: "current",
485 }
486 `
487 testJavaError(t, `"libjni" .*: sdk_version\(current\) is higher than min_sdk_version\(29\)`, bp)
488}
489
490func TestUpdatableApps_ErrorIfDepSdkVersionIsHigher(t *testing.T) {
491 bp := cc.GatherRequiredDepsForTest(android.Android) + `
492 android_app {
493 name: "foo",
494 srcs: ["a.java"],
495 updatable: true,
496 sdk_version: "current",
497 min_sdk_version: "29", // this APK should support 29
498 jni_libs: ["libjni"],
499 }
500
501 cc_library {
502 name: "libjni",
503 stl: "none",
504 shared_libs: ["libbar"],
505 system_shared_libs: [],
506 sdk_version: "27",
507 }
508
509 cc_library {
510 name: "libbar",
511 stl: "none",
512 system_shared_libs: [],
513 sdk_version: "current",
514 }
515 `
516 testJavaError(t, `"libjni" .*: links "libbar" built against newer API version "current"`, bp)
517}
518
Colin Cross0ddae7f2019-02-07 15:30:01 -0800519func TestResourceDirs(t *testing.T) {
520 testCases := []struct {
521 name string
522 prop string
523 resources []string
524 }{
525 {
526 name: "no resource_dirs",
527 prop: "",
528 resources: []string{"res/res/values/strings.xml"},
529 },
530 {
531 name: "resource_dirs",
532 prop: `resource_dirs: ["res"]`,
533 resources: []string{"res/res/values/strings.xml"},
534 },
535 {
536 name: "empty resource_dirs",
537 prop: `resource_dirs: []`,
538 resources: nil,
539 },
540 }
541
Paul Duffinfb0fe9f2021-03-22 17:31:52 +0000542 fs := android.MockFS{
Colin Cross0ddae7f2019-02-07 15:30:01 -0800543 "res/res/values/strings.xml": nil,
544 }
545
546 bp := `
547 android_app {
548 name: "foo",
Jeongik Cha538c0d02019-07-11 15:54:27 +0900549 sdk_version: "current",
Colin Cross0ddae7f2019-02-07 15:30:01 -0800550 %s
551 }
552 `
553
554 for _, testCase := range testCases {
555 t.Run(testCase.name, func(t *testing.T) {
Paul Duffinfb0fe9f2021-03-22 17:31:52 +0000556 result := android.GroupFixturePreparers(
557 PrepareForTestWithJavaDefaultModules,
558 PrepareForTestWithOverlayBuildComponents,
559 fs.AddToFixture(),
560 ).RunTestWithBp(t, fmt.Sprintf(bp, testCase.prop))
Colin Cross0ddae7f2019-02-07 15:30:01 -0800561
Paul Duffinfb0fe9f2021-03-22 17:31:52 +0000562 module := result.ModuleForTests("foo", "android_common")
Colin Cross0ddae7f2019-02-07 15:30:01 -0800563 resourceList := module.MaybeOutput("aapt2/res.list")
564
565 var resources []string
566 if resourceList.Rule != nil {
567 for _, compiledResource := range resourceList.Inputs.Strings() {
568 resources = append(resources, module.Output(compiledResource).Inputs.Strings()...)
569 }
570 }
571
Paul Duffinfb0fe9f2021-03-22 17:31:52 +0000572 android.AssertDeepEquals(t, "resource files", testCase.resources, resources)
Colin Cross0ddae7f2019-02-07 15:30:01 -0800573 })
574 }
575}
576
Jaewoong Jung6431ca72020-01-15 14:15:10 -0800577func TestLibraryAssets(t *testing.T) {
578 bp := `
579 android_app {
580 name: "foo",
581 sdk_version: "current",
582 static_libs: ["lib1", "lib2", "lib3"],
583 }
584
585 android_library {
586 name: "lib1",
587 sdk_version: "current",
588 asset_dirs: ["assets_a"],
589 }
590
591 android_library {
592 name: "lib2",
593 sdk_version: "current",
594 }
595
596 android_library {
597 name: "lib3",
598 sdk_version: "current",
599 static_libs: ["lib4"],
600 }
601
602 android_library {
603 name: "lib4",
604 sdk_version: "current",
605 asset_dirs: ["assets_b"],
606 }
607 `
608
609 testCases := []struct {
610 name string
611 assetFlag string
612 assetPackages []string
613 }{
614 {
615 name: "foo",
616 // lib1 has its own asset. lib3 doesn't have any, but provides lib4's transitively.
617 assetPackages: []string{
Paul Duffinfb0fe9f2021-03-22 17:31:52 +0000618 "out/soong/.intermediates/foo/android_common/aapt2/package-res.apk",
619 "out/soong/.intermediates/lib1/android_common/assets.zip",
620 "out/soong/.intermediates/lib3/android_common/assets.zip",
Jaewoong Jung6431ca72020-01-15 14:15:10 -0800621 },
622 },
623 {
624 name: "lib1",
625 assetFlag: "-A assets_a",
626 },
627 {
628 name: "lib2",
629 },
630 {
631 name: "lib3",
632 assetPackages: []string{
Paul Duffinfb0fe9f2021-03-22 17:31:52 +0000633 "out/soong/.intermediates/lib3/android_common/aapt2/package-res.apk",
634 "out/soong/.intermediates/lib4/android_common/assets.zip",
Jaewoong Jung6431ca72020-01-15 14:15:10 -0800635 },
636 },
637 {
638 name: "lib4",
639 assetFlag: "-A assets_b",
640 },
641 }
642 ctx := testApp(t, bp)
643
644 for _, test := range testCases {
645 t.Run(test.name, func(t *testing.T) {
646 m := ctx.ModuleForTests(test.name, "android_common")
647
648 // Check asset flag in aapt2 link flags
649 var aapt2link android.TestingBuildParams
650 if len(test.assetPackages) > 0 {
651 aapt2link = m.Output("aapt2/package-res.apk")
652 } else {
653 aapt2link = m.Output("package-res.apk")
654 }
Paul Duffina71a67a2021-03-29 00:42:57 +0100655 aapt2link = aapt2link
Jaewoong Jung6431ca72020-01-15 14:15:10 -0800656 aapt2Flags := aapt2link.Args["flags"]
657 if test.assetFlag != "" {
Paul Duffinfb0fe9f2021-03-22 17:31:52 +0000658 android.AssertStringDoesContain(t, "asset flag", aapt2Flags, test.assetFlag)
Jaewoong Jung6431ca72020-01-15 14:15:10 -0800659 } else {
Paul Duffinfb0fe9f2021-03-22 17:31:52 +0000660 android.AssertStringDoesNotContain(t, "aapt2 link flags", aapt2Flags, " -A ")
Jaewoong Jung6431ca72020-01-15 14:15:10 -0800661 }
662
663 // Check asset merge rule.
664 if len(test.assetPackages) > 0 {
665 mergeAssets := m.Output("package-res.apk")
Paul Duffinfb0fe9f2021-03-22 17:31:52 +0000666 android.AssertPathsRelativeToTopEquals(t, "mergeAssets inputs", test.assetPackages, mergeAssets.Inputs)
Jaewoong Jung6431ca72020-01-15 14:15:10 -0800667 }
668 })
669 }
670}
671
Colin Crossb014f072021-02-26 14:54:36 -0800672func TestAppJavaResources(t *testing.T) {
673 bp := `
674 android_app {
675 name: "foo",
676 sdk_version: "current",
677 java_resources: ["resources/a"],
678 srcs: ["a.java"],
679 }
680
681 android_app {
682 name: "bar",
683 sdk_version: "current",
684 java_resources: ["resources/a"],
685 }
686 `
687
688 ctx := testApp(t, bp)
689
690 foo := ctx.ModuleForTests("foo", "android_common")
691 fooResources := foo.Output("res/foo.jar")
692 fooDexJar := foo.Output("dex-withres/foo.jar")
693 fooDexJarAligned := foo.Output("dex-withres-aligned/foo.jar")
694 fooApk := foo.Rule("combineApk")
695
696 if g, w := fooDexJar.Inputs.Strings(), fooResources.Output.String(); !android.InList(w, g) {
697 t.Errorf("expected resource jar %q in foo dex jar inputs %q", w, g)
698 }
699
700 if g, w := fooDexJarAligned.Input.String(), fooDexJar.Output.String(); g != w {
701 t.Errorf("expected dex jar %q in foo aligned dex jar inputs %q", w, g)
702 }
703
704 if g, w := fooApk.Inputs.Strings(), fooDexJarAligned.Output.String(); !android.InList(w, g) {
705 t.Errorf("expected aligned dex jar %q in foo apk inputs %q", w, g)
706 }
707
708 bar := ctx.ModuleForTests("bar", "android_common")
709 barResources := bar.Output("res/bar.jar")
710 barApk := bar.Rule("combineApk")
711
712 if g, w := barApk.Inputs.Strings(), barResources.Output.String(); !android.InList(w, g) {
713 t.Errorf("expected resources jar %q in bar apk inputs %q", w, g)
714 }
715}
716
Colin Crossbec85302019-02-13 13:15:46 -0800717func TestAndroidResources(t *testing.T) {
Colin Cross5c4791c2019-02-01 11:44:44 -0800718 testCases := []struct {
719 name string
720 enforceRROTargets []string
721 enforceRROExcludedOverlays []string
Colin Crossbec85302019-02-13 13:15:46 -0800722 resourceFiles map[string][]string
Colin Cross5c4791c2019-02-01 11:44:44 -0800723 overlayFiles map[string][]string
724 rroDirs map[string][]string
725 }{
726 {
727 name: "no RRO",
728 enforceRROTargets: nil,
729 enforceRROExcludedOverlays: nil,
Colin Crossbec85302019-02-13 13:15:46 -0800730 resourceFiles: map[string][]string{
731 "foo": nil,
732 "bar": {"bar/res/res/values/strings.xml"},
733 "lib": nil,
734 "lib2": {"lib2/res/res/values/strings.xml"},
735 },
Colin Cross5c4791c2019-02-01 11:44:44 -0800736 overlayFiles: map[string][]string{
Colin Crossbec85302019-02-13 13:15:46 -0800737 "foo": {
Paul Duffinfb0fe9f2021-03-22 17:31:52 +0000738 "out/soong/.intermediates/lib2/android_common/package-res.apk",
739 "out/soong/.intermediates/lib/android_common/package-res.apk",
740 "out/soong/.intermediates/lib3/android_common/package-res.apk",
Colin Cross6ed7dea2019-01-31 14:44:30 -0800741 "foo/res/res/values/strings.xml",
Colin Cross5c4791c2019-02-01 11:44:44 -0800742 "device/vendor/blah/static_overlay/foo/res/values/strings.xml",
743 "device/vendor/blah/overlay/foo/res/values/strings.xml",
Anton Hansson53c88442019-03-18 15:53:16 +0000744 "product/vendor/blah/overlay/foo/res/values/strings.xml",
Colin Cross5c4791c2019-02-01 11:44:44 -0800745 },
Colin Crossbec85302019-02-13 13:15:46 -0800746 "bar": {
Colin Cross5c4791c2019-02-01 11:44:44 -0800747 "device/vendor/blah/static_overlay/bar/res/values/strings.xml",
748 "device/vendor/blah/overlay/bar/res/values/strings.xml",
749 },
Colin Crossbec85302019-02-13 13:15:46 -0800750 "lib": {
Paul Duffinfb0fe9f2021-03-22 17:31:52 +0000751 "out/soong/.intermediates/lib2/android_common/package-res.apk",
Colin Crossbec85302019-02-13 13:15:46 -0800752 "lib/res/res/values/strings.xml",
753 "device/vendor/blah/overlay/lib/res/values/strings.xml",
754 },
Colin Cross5c4791c2019-02-01 11:44:44 -0800755 },
756 rroDirs: map[string][]string{
757 "foo": nil,
758 "bar": nil,
759 },
760 },
761 {
762 name: "enforce RRO on foo",
763 enforceRROTargets: []string{"foo"},
764 enforceRROExcludedOverlays: []string{"device/vendor/blah/static_overlay"},
Colin Crossbec85302019-02-13 13:15:46 -0800765 resourceFiles: map[string][]string{
766 "foo": nil,
767 "bar": {"bar/res/res/values/strings.xml"},
768 "lib": nil,
769 "lib2": {"lib2/res/res/values/strings.xml"},
770 },
Colin Cross5c4791c2019-02-01 11:44:44 -0800771 overlayFiles: map[string][]string{
Colin Crossbec85302019-02-13 13:15:46 -0800772 "foo": {
Paul Duffinfb0fe9f2021-03-22 17:31:52 +0000773 "out/soong/.intermediates/lib2/android_common/package-res.apk",
774 "out/soong/.intermediates/lib/android_common/package-res.apk",
775 "out/soong/.intermediates/lib3/android_common/package-res.apk",
Colin Cross6ed7dea2019-01-31 14:44:30 -0800776 "foo/res/res/values/strings.xml",
777 "device/vendor/blah/static_overlay/foo/res/values/strings.xml",
778 },
Colin Crossbec85302019-02-13 13:15:46 -0800779 "bar": {
Colin Cross5c4791c2019-02-01 11:44:44 -0800780 "device/vendor/blah/static_overlay/bar/res/values/strings.xml",
781 "device/vendor/blah/overlay/bar/res/values/strings.xml",
782 },
Colin Crossbec85302019-02-13 13:15:46 -0800783 "lib": {
Paul Duffinfb0fe9f2021-03-22 17:31:52 +0000784 "out/soong/.intermediates/lib2/android_common/package-res.apk",
Colin Crossbec85302019-02-13 13:15:46 -0800785 "lib/res/res/values/strings.xml",
Colin Crossbec85302019-02-13 13:15:46 -0800786 },
Colin Cross5c4791c2019-02-01 11:44:44 -0800787 },
Colin Crossc1c37552019-01-31 11:42:41 -0800788
Colin Cross5c4791c2019-02-01 11:44:44 -0800789 rroDirs: map[string][]string{
Colin Crossbec85302019-02-13 13:15:46 -0800790 "foo": {
Anton Hansson53c88442019-03-18 15:53:16 +0000791 "device:device/vendor/blah/overlay/foo/res",
Anton Hansson53c88442019-03-18 15:53:16 +0000792 "product:product/vendor/blah/overlay/foo/res",
Jaewoong Jungc779cd42020-10-06 18:56:10 -0700793 "device:device/vendor/blah/overlay/lib/res",
Colin Crossc1c37552019-01-31 11:42:41 -0800794 },
Colin Cross5c4791c2019-02-01 11:44:44 -0800795 "bar": nil,
Jaewoong Jungc779cd42020-10-06 18:56:10 -0700796 "lib": {"device:device/vendor/blah/overlay/lib/res"},
Colin Cross5c4791c2019-02-01 11:44:44 -0800797 },
798 },
799 {
800 name: "enforce RRO on all",
801 enforceRROTargets: []string{"*"},
802 enforceRROExcludedOverlays: []string{
803 // Excluding specific apps/res directories also allowed.
804 "device/vendor/blah/static_overlay/foo",
805 "device/vendor/blah/static_overlay/bar/res",
806 },
Colin Crossbec85302019-02-13 13:15:46 -0800807 resourceFiles: map[string][]string{
808 "foo": nil,
809 "bar": {"bar/res/res/values/strings.xml"},
810 "lib": nil,
811 "lib2": {"lib2/res/res/values/strings.xml"},
812 },
Colin Cross5c4791c2019-02-01 11:44:44 -0800813 overlayFiles: map[string][]string{
Colin Crossbec85302019-02-13 13:15:46 -0800814 "foo": {
Paul Duffinfb0fe9f2021-03-22 17:31:52 +0000815 "out/soong/.intermediates/lib2/android_common/package-res.apk",
816 "out/soong/.intermediates/lib/android_common/package-res.apk",
817 "out/soong/.intermediates/lib3/android_common/package-res.apk",
Colin Cross6ed7dea2019-01-31 14:44:30 -0800818 "foo/res/res/values/strings.xml",
819 "device/vendor/blah/static_overlay/foo/res/values/strings.xml",
820 },
Colin Crossbec85302019-02-13 13:15:46 -0800821 "bar": {"device/vendor/blah/static_overlay/bar/res/values/strings.xml"},
822 "lib": {
Paul Duffinfb0fe9f2021-03-22 17:31:52 +0000823 "out/soong/.intermediates/lib2/android_common/package-res.apk",
Colin Crossbec85302019-02-13 13:15:46 -0800824 "lib/res/res/values/strings.xml",
825 },
Colin Cross5c4791c2019-02-01 11:44:44 -0800826 },
827 rroDirs: map[string][]string{
Colin Crossbec85302019-02-13 13:15:46 -0800828 "foo": {
Anton Hansson53c88442019-03-18 15:53:16 +0000829 "device:device/vendor/blah/overlay/foo/res",
830 "product:product/vendor/blah/overlay/foo/res",
831 // Lib dep comes after the direct deps
832 "device:device/vendor/blah/overlay/lib/res",
Colin Crossc1c37552019-01-31 11:42:41 -0800833 },
Anton Hansson53c88442019-03-18 15:53:16 +0000834 "bar": {"device:device/vendor/blah/overlay/bar/res"},
835 "lib": {"device:device/vendor/blah/overlay/lib/res"},
Colin Cross5c4791c2019-02-01 11:44:44 -0800836 },
837 },
838 }
839
Anton Hansson53c88442019-03-18 15:53:16 +0000840 deviceResourceOverlays := []string{
Colin Cross890ff552017-11-30 20:13:19 -0800841 "device/vendor/blah/overlay",
842 "device/vendor/blah/overlay2",
843 "device/vendor/blah/static_overlay",
844 }
845
Anton Hansson53c88442019-03-18 15:53:16 +0000846 productResourceOverlays := []string{
847 "product/vendor/blah/overlay",
848 }
849
Paul Duffinfb0fe9f2021-03-22 17:31:52 +0000850 fs := android.MockFS{
Colin Cross890ff552017-11-30 20:13:19 -0800851 "foo/res/res/values/strings.xml": nil,
852 "bar/res/res/values/strings.xml": nil,
Colin Cross6ed7dea2019-01-31 14:44:30 -0800853 "lib/res/res/values/strings.xml": nil,
Colin Crossbec85302019-02-13 13:15:46 -0800854 "lib2/res/res/values/strings.xml": nil,
Colin Cross890ff552017-11-30 20:13:19 -0800855 "device/vendor/blah/overlay/foo/res/values/strings.xml": nil,
856 "device/vendor/blah/overlay/bar/res/values/strings.xml": nil,
Colin Cross6ed7dea2019-01-31 14:44:30 -0800857 "device/vendor/blah/overlay/lib/res/values/strings.xml": nil,
Colin Cross890ff552017-11-30 20:13:19 -0800858 "device/vendor/blah/static_overlay/foo/res/values/strings.xml": nil,
859 "device/vendor/blah/static_overlay/bar/res/values/strings.xml": nil,
860 "device/vendor/blah/overlay2/res/values/strings.xml": nil,
Anton Hansson53c88442019-03-18 15:53:16 +0000861 "product/vendor/blah/overlay/foo/res/values/strings.xml": nil,
Colin Cross890ff552017-11-30 20:13:19 -0800862 }
863
864 bp := `
865 android_app {
866 name: "foo",
Jeongik Cha538c0d02019-07-11 15:54:27 +0900867 sdk_version: "current",
Colin Cross890ff552017-11-30 20:13:19 -0800868 resource_dirs: ["foo/res"],
Anton Hansson53c88442019-03-18 15:53:16 +0000869 static_libs: ["lib", "lib3"],
Colin Cross890ff552017-11-30 20:13:19 -0800870 }
871
872 android_app {
873 name: "bar",
Jeongik Cha538c0d02019-07-11 15:54:27 +0900874 sdk_version: "current",
Colin Cross890ff552017-11-30 20:13:19 -0800875 resource_dirs: ["bar/res"],
876 }
Colin Cross6ed7dea2019-01-31 14:44:30 -0800877
878 android_library {
879 name: "lib",
Jeongik Cha75b83b02019-11-01 15:28:00 +0900880 sdk_version: "current",
Colin Cross6ed7dea2019-01-31 14:44:30 -0800881 resource_dirs: ["lib/res"],
Colin Crossbec85302019-02-13 13:15:46 -0800882 static_libs: ["lib2"],
883 }
884
885 android_library {
886 name: "lib2",
Jeongik Cha75b83b02019-11-01 15:28:00 +0900887 sdk_version: "current",
Colin Crossbec85302019-02-13 13:15:46 -0800888 resource_dirs: ["lib2/res"],
Colin Cross6ed7dea2019-01-31 14:44:30 -0800889 }
Anton Hansson53c88442019-03-18 15:53:16 +0000890
891 // This library has the same resources as lib (should not lead to dupe RROs)
892 android_library {
893 name: "lib3",
Jeongik Cha75b83b02019-11-01 15:28:00 +0900894 sdk_version: "current",
Anton Hansson53c88442019-03-18 15:53:16 +0000895 resource_dirs: ["lib/res"]
896 }
Colin Cross890ff552017-11-30 20:13:19 -0800897 `
898
Colin Cross5c4791c2019-02-01 11:44:44 -0800899 for _, testCase := range testCases {
Colin Cross890ff552017-11-30 20:13:19 -0800900 t.Run(testCase.name, func(t *testing.T) {
Paul Duffinfb0fe9f2021-03-22 17:31:52 +0000901 result := android.GroupFixturePreparers(
902 PrepareForTestWithJavaDefaultModules,
903 PrepareForTestWithOverlayBuildComponents,
904 fs.AddToFixture(),
905 android.FixtureModifyProductVariables(func(variables android.FixtureProductVariables) {
906 variables.DeviceResourceOverlays = deviceResourceOverlays
907 variables.ProductResourceOverlays = productResourceOverlays
908 if testCase.enforceRROTargets != nil {
909 variables.EnforceRROTargets = testCase.enforceRROTargets
910 }
911 if testCase.enforceRROExcludedOverlays != nil {
912 variables.EnforceRROExcludedOverlays = testCase.enforceRROExcludedOverlays
913 }
914 }),
915 ).RunTestWithBp(t, bp)
Colin Cross890ff552017-11-30 20:13:19 -0800916
Colin Crossbec85302019-02-13 13:15:46 -0800917 resourceListToFiles := func(module android.TestingModule, list []string) (files []string) {
918 for _, o := range list {
919 res := module.MaybeOutput(o)
920 if res.Rule != nil {
921 // If the overlay is compiled as part of this module (i.e. a .arsc.flat file),
922 // verify the inputs to the .arsc.flat rule.
923 files = append(files, res.Inputs.Strings()...)
924 } else {
925 // Otherwise, verify the full path to the output of the other module
926 files = append(files, o)
Anton Hansson94c93f32019-01-30 16:03:37 +0000927 }
Colin Cross890ff552017-11-30 20:13:19 -0800928 }
Colin Crossbec85302019-02-13 13:15:46 -0800929 return files
Colin Cross890ff552017-11-30 20:13:19 -0800930 }
931
Colin Crossbec85302019-02-13 13:15:46 -0800932 getResources := func(moduleName string) (resourceFiles, overlayFiles, rroDirs []string) {
Paul Duffinfb0fe9f2021-03-22 17:31:52 +0000933 module := result.ModuleForTests(moduleName, "android_common")
Colin Crossbec85302019-02-13 13:15:46 -0800934 resourceList := module.MaybeOutput("aapt2/res.list")
935 if resourceList.Rule != nil {
Paul Duffinfb0fe9f2021-03-22 17:31:52 +0000936 resourceFiles = resourceListToFiles(module, android.PathsRelativeToTop(resourceList.Inputs))
Anton Hansson0375a4f2019-01-24 14:39:19 +0000937 }
Colin Crossbec85302019-02-13 13:15:46 -0800938 overlayList := module.MaybeOutput("aapt2/overlay.list")
939 if overlayList.Rule != nil {
Paul Duffinfb0fe9f2021-03-22 17:31:52 +0000940 overlayFiles = resourceListToFiles(module, android.PathsRelativeToTop(overlayList.Inputs))
Colin Crossbec85302019-02-13 13:15:46 -0800941 }
942
Anton Hansson53c88442019-03-18 15:53:16 +0000943 for _, d := range module.Module().(AndroidLibraryDependency).ExportedRRODirs() {
944 var prefix string
945 if d.overlayType == device {
946 prefix = "device:"
947 } else if d.overlayType == product {
948 prefix = "product:"
949 } else {
950 t.Fatalf("Unexpected overlayType %d", d.overlayType)
951 }
Paul Duffinfb0fe9f2021-03-22 17:31:52 +0000952 rroDirs = append(rroDirs, prefix+android.PathRelativeToTop(d.path))
Anton Hansson53c88442019-03-18 15:53:16 +0000953 }
Colin Crossbec85302019-02-13 13:15:46 -0800954
955 return resourceFiles, overlayFiles, rroDirs
956 }
957
958 modules := []string{"foo", "bar", "lib", "lib2"}
959 for _, module := range modules {
960 resourceFiles, overlayFiles, rroDirs := getResources(module)
961
962 if !reflect.DeepEqual(resourceFiles, testCase.resourceFiles[module]) {
963 t.Errorf("expected %s resource files:\n %#v\n got:\n %#v",
964 module, testCase.resourceFiles[module], resourceFiles)
965 }
966 if !reflect.DeepEqual(overlayFiles, testCase.overlayFiles[module]) {
967 t.Errorf("expected %s overlay files:\n %#v\n got:\n %#v",
968 module, testCase.overlayFiles[module], overlayFiles)
969 }
970 if !reflect.DeepEqual(rroDirs, testCase.rroDirs[module]) {
Anton Hansson0375a4f2019-01-24 14:39:19 +0000971 t.Errorf("expected %s rroDirs: %#v\n got:\n %#v",
Colin Crossbec85302019-02-13 13:15:46 -0800972 module, testCase.rroDirs[module], rroDirs)
Anton Hansson0375a4f2019-01-24 14:39:19 +0000973 }
Colin Cross890ff552017-11-30 20:13:19 -0800974 }
Colin Cross890ff552017-11-30 20:13:19 -0800975 })
976 }
977}
Colin Crossd09b0b62018-04-18 11:06:47 -0700978
Paul Duffincdb88a92021-03-14 00:36:50 +0000979func checkSdkVersion(t *testing.T, result *android.TestResult, expectedSdkVersion string) {
980 foo := result.ModuleForTests("foo", "android_common")
Jeongik Cha219141c2020-08-06 23:00:37 +0900981 link := foo.Output("package-res.apk")
982 linkFlags := strings.Split(link.Args["flags"], " ")
983 min := android.IndexList("--min-sdk-version", linkFlags)
984 target := android.IndexList("--target-sdk-version", linkFlags)
985
986 if min == -1 || target == -1 || min == len(linkFlags)-1 || target == len(linkFlags)-1 {
987 t.Fatalf("missing --min-sdk-version or --target-sdk-version in link flags: %q", linkFlags)
988 }
989
990 gotMinSdkVersion := linkFlags[min+1]
991 gotTargetSdkVersion := linkFlags[target+1]
992
Paul Duffincdb88a92021-03-14 00:36:50 +0000993 android.AssertStringEquals(t, "incorrect --min-sdk-version", expectedSdkVersion, gotMinSdkVersion)
Jeongik Cha219141c2020-08-06 23:00:37 +0900994
Paul Duffincdb88a92021-03-14 00:36:50 +0000995 android.AssertStringEquals(t, "incorrect --target-sdk-version", expectedSdkVersion, gotTargetSdkVersion)
Jeongik Cha219141c2020-08-06 23:00:37 +0900996}
997
Colin Crossd09b0b62018-04-18 11:06:47 -0700998func TestAppSdkVersion(t *testing.T) {
999 testCases := []struct {
1000 name string
1001 sdkVersion string
1002 platformSdkInt int
1003 platformSdkCodename string
1004 platformSdkFinal bool
1005 expectedMinSdkVersion string
Jeongik Cha538c0d02019-07-11 15:54:27 +09001006 platformApis bool
Dan Albert4f378d72020-07-23 17:32:15 -07001007 activeCodenames []string
Colin Crossd09b0b62018-04-18 11:06:47 -07001008 }{
1009 {
1010 name: "current final SDK",
1011 sdkVersion: "current",
1012 platformSdkInt: 27,
1013 platformSdkCodename: "REL",
1014 platformSdkFinal: true,
1015 expectedMinSdkVersion: "27",
1016 },
1017 {
1018 name: "current non-final SDK",
1019 sdkVersion: "current",
1020 platformSdkInt: 27,
1021 platformSdkCodename: "OMR1",
1022 platformSdkFinal: false,
1023 expectedMinSdkVersion: "OMR1",
Dan Albert4f378d72020-07-23 17:32:15 -07001024 activeCodenames: []string{"OMR1"},
Colin Crossd09b0b62018-04-18 11:06:47 -07001025 },
1026 {
1027 name: "default final SDK",
1028 sdkVersion: "",
Jeongik Cha538c0d02019-07-11 15:54:27 +09001029 platformApis: true,
Colin Crossd09b0b62018-04-18 11:06:47 -07001030 platformSdkInt: 27,
1031 platformSdkCodename: "REL",
1032 platformSdkFinal: true,
1033 expectedMinSdkVersion: "27",
1034 },
1035 {
1036 name: "default non-final SDK",
1037 sdkVersion: "",
Jeongik Cha538c0d02019-07-11 15:54:27 +09001038 platformApis: true,
Colin Crossd09b0b62018-04-18 11:06:47 -07001039 platformSdkInt: 27,
1040 platformSdkCodename: "OMR1",
1041 platformSdkFinal: false,
1042 expectedMinSdkVersion: "OMR1",
Dan Albert4f378d72020-07-23 17:32:15 -07001043 activeCodenames: []string{"OMR1"},
Colin Crossd09b0b62018-04-18 11:06:47 -07001044 },
1045 {
1046 name: "14",
1047 sdkVersion: "14",
1048 expectedMinSdkVersion: "14",
Dan Albert4f378d72020-07-23 17:32:15 -07001049 platformSdkCodename: "S",
1050 activeCodenames: []string{"S"},
Colin Crossd09b0b62018-04-18 11:06:47 -07001051 },
1052 }
1053
1054 for _, moduleType := range []string{"android_app", "android_library"} {
1055 for _, test := range testCases {
1056 t.Run(moduleType+" "+test.name, func(t *testing.T) {
Jeongik Cha538c0d02019-07-11 15:54:27 +09001057 platformApiProp := ""
1058 if test.platformApis {
1059 platformApiProp = "platform_apis: true,"
1060 }
Colin Crossd09b0b62018-04-18 11:06:47 -07001061 bp := fmt.Sprintf(`%s {
1062 name: "foo",
1063 srcs: ["a.java"],
1064 sdk_version: "%s",
Jeongik Cha538c0d02019-07-11 15:54:27 +09001065 %s
1066 }`, moduleType, test.sdkVersion, platformApiProp)
Colin Crossd09b0b62018-04-18 11:06:47 -07001067
Paul Duffin71ae5942021-03-22 15:36:52 +00001068 result := android.GroupFixturePreparers(
1069 prepareForJavaTest,
Paul Duffincdb88a92021-03-14 00:36:50 +00001070 android.FixtureModifyProductVariables(func(variables android.FixtureProductVariables) {
1071 variables.Platform_sdk_version = &test.platformSdkInt
1072 variables.Platform_sdk_codename = &test.platformSdkCodename
1073 variables.Platform_version_active_codenames = test.activeCodenames
1074 variables.Platform_sdk_final = &test.platformSdkFinal
1075 }),
Paul Duffin2645a292021-03-13 02:36:00 +00001076 FixtureWithPrebuiltApis(map[string][]string{
1077 "14": {"foo"},
1078 }),
Paul Duffincdb88a92021-03-14 00:36:50 +00001079 ).RunTestWithBp(t, bp)
Colin Crossd09b0b62018-04-18 11:06:47 -07001080
Paul Duffincdb88a92021-03-14 00:36:50 +00001081 checkSdkVersion(t, result, test.expectedMinSdkVersion)
Colin Crossd09b0b62018-04-18 11:06:47 -07001082 })
1083 }
1084 }
1085}
Colin Crossa4f08812018-10-02 22:03:40 -07001086
Jeongik Cha219141c2020-08-06 23:00:37 +09001087func TestVendorAppSdkVersion(t *testing.T) {
1088 testCases := []struct {
1089 name string
1090 sdkVersion string
1091 platformSdkInt int
1092 platformSdkCodename string
1093 platformSdkFinal bool
1094 deviceCurrentApiLevelForVendorModules string
1095 expectedMinSdkVersion string
1096 }{
1097 {
1098 name: "current final SDK",
1099 sdkVersion: "current",
1100 platformSdkInt: 29,
1101 platformSdkCodename: "REL",
1102 platformSdkFinal: true,
1103 deviceCurrentApiLevelForVendorModules: "29",
1104 expectedMinSdkVersion: "29",
1105 },
1106 {
1107 name: "current final SDK",
1108 sdkVersion: "current",
1109 platformSdkInt: 29,
1110 platformSdkCodename: "REL",
1111 platformSdkFinal: true,
1112 deviceCurrentApiLevelForVendorModules: "28",
1113 expectedMinSdkVersion: "28",
1114 },
1115 {
1116 name: "current final SDK",
1117 sdkVersion: "current",
1118 platformSdkInt: 29,
1119 platformSdkCodename: "Q",
1120 platformSdkFinal: false,
Jeongik Cha219141c2020-08-06 23:00:37 +09001121 deviceCurrentApiLevelForVendorModules: "28",
1122 expectedMinSdkVersion: "28",
1123 },
1124 }
1125
1126 for _, moduleType := range []string{"android_app", "android_library"} {
1127 for _, sdkKind := range []string{"", "system_"} {
1128 for _, test := range testCases {
1129 t.Run(moduleType+" "+test.name, func(t *testing.T) {
1130 bp := fmt.Sprintf(`%s {
1131 name: "foo",
1132 srcs: ["a.java"],
1133 sdk_version: "%s%s",
1134 vendor: true,
1135 }`, moduleType, sdkKind, test.sdkVersion)
1136
Paul Duffin71ae5942021-03-22 15:36:52 +00001137 result := android.GroupFixturePreparers(
1138 prepareForJavaTest,
Paul Duffincdb88a92021-03-14 00:36:50 +00001139 android.FixtureModifyProductVariables(func(variables android.FixtureProductVariables) {
1140 variables.Platform_sdk_version = &test.platformSdkInt
1141 variables.Platform_sdk_codename = &test.platformSdkCodename
1142 variables.Platform_sdk_final = &test.platformSdkFinal
1143 variables.DeviceCurrentApiLevelForVendorModules = &test.deviceCurrentApiLevelForVendorModules
1144 variables.DeviceSystemSdkVersions = []string{"28", "29"}
1145 }),
Paul Duffin2645a292021-03-13 02:36:00 +00001146 FixtureWithPrebuiltApis(map[string][]string{
1147 "28": {"foo"},
1148 "29": {"foo"},
1149 "current": {"foo"},
1150 }),
Paul Duffincdb88a92021-03-14 00:36:50 +00001151 ).RunTestWithBp(t, bp)
1152
1153 checkSdkVersion(t, result, test.expectedMinSdkVersion)
Jeongik Cha219141c2020-08-06 23:00:37 +09001154 })
1155 }
1156 }
1157 }
1158}
1159
Paul Duffin50c217c2019-06-12 13:25:22 +01001160func TestJNIABI(t *testing.T) {
Jaewoong Jungf9a04432019-07-17 11:15:09 -07001161 ctx, _ := testJava(t, cc.GatherRequiredDepsForTest(android.Android)+`
Paul Duffin50c217c2019-06-12 13:25:22 +01001162 cc_library {
1163 name: "libjni",
1164 system_shared_libs: [],
Colin Crossc511bc52020-04-07 16:50:32 +00001165 sdk_version: "current",
Paul Duffin50c217c2019-06-12 13:25:22 +01001166 stl: "none",
1167 }
1168
1169 android_test {
1170 name: "test",
1171 sdk_version: "core_platform",
1172 jni_libs: ["libjni"],
1173 }
1174
1175 android_test {
1176 name: "test_first",
1177 sdk_version: "core_platform",
1178 compile_multilib: "first",
1179 jni_libs: ["libjni"],
1180 }
1181
1182 android_test {
1183 name: "test_both",
1184 sdk_version: "core_platform",
1185 compile_multilib: "both",
1186 jni_libs: ["libjni"],
1187 }
1188
1189 android_test {
1190 name: "test_32",
1191 sdk_version: "core_platform",
1192 compile_multilib: "32",
1193 jni_libs: ["libjni"],
1194 }
1195
1196 android_test {
1197 name: "test_64",
1198 sdk_version: "core_platform",
1199 compile_multilib: "64",
1200 jni_libs: ["libjni"],
1201 }
1202 `)
1203
1204 testCases := []struct {
1205 name string
1206 abis []string
1207 }{
1208 {"test", []string{"arm64-v8a"}},
1209 {"test_first", []string{"arm64-v8a"}},
1210 {"test_both", []string{"arm64-v8a", "armeabi-v7a"}},
1211 {"test_32", []string{"armeabi-v7a"}},
1212 {"test_64", []string{"arm64-v8a"}},
1213 }
1214
1215 for _, test := range testCases {
1216 t.Run(test.name, func(t *testing.T) {
1217 app := ctx.ModuleForTests(test.name, "android_common")
1218 jniLibZip := app.Output("jnilibs.zip")
1219 var abis []string
1220 args := strings.Fields(jniLibZip.Args["jarArgs"])
1221 for i := 0; i < len(args); i++ {
1222 if args[i] == "-P" {
1223 abis = append(abis, filepath.Base(args[i+1]))
1224 i++
1225 }
1226 }
1227 if !reflect.DeepEqual(abis, test.abis) {
1228 t.Errorf("want abis %v, got %v", test.abis, abis)
1229 }
1230 })
1231 }
1232}
1233
Jeongik Cha2cc570d2019-10-29 15:44:45 +09001234func TestAppSdkVersionByPartition(t *testing.T) {
1235 testJavaError(t, "sdk_version must have a value when the module is located at vendor or product", `
1236 android_app {
1237 name: "foo",
1238 srcs: ["a.java"],
1239 vendor: true,
1240 platform_apis: true,
1241 }
1242 `)
1243
1244 testJava(t, `
1245 android_app {
1246 name: "bar",
1247 srcs: ["b.java"],
1248 platform_apis: true,
1249 }
1250 `)
1251
1252 for _, enforce := range []bool{true, false} {
Jeongik Cha2cc570d2019-10-29 15:44:45 +09001253 bp := `
1254 android_app {
1255 name: "foo",
1256 srcs: ["a.java"],
1257 product_specific: true,
1258 platform_apis: true,
1259 }
1260 `
Colin Cross98be1bb2019-12-13 20:41:13 -08001261
Paul Duffinfb0fe9f2021-03-22 17:31:52 +00001262 errorHandler := android.FixtureExpectsNoErrors
Jeongik Cha2cc570d2019-10-29 15:44:45 +09001263 if enforce {
Paul Duffinfb0fe9f2021-03-22 17:31:52 +00001264 errorHandler = android.FixtureExpectsAtLeastOneErrorMatchingPattern("sdk_version must have a value when the module is located at vendor or product")
Jeongik Cha2cc570d2019-10-29 15:44:45 +09001265 }
Paul Duffinfb0fe9f2021-03-22 17:31:52 +00001266
1267 android.GroupFixturePreparers(
1268 PrepareForTestWithJavaDefaultModules,
1269 android.FixtureModifyProductVariables(func(variables android.FixtureProductVariables) {
1270 variables.EnforceProductPartitionInterface = proptools.BoolPtr(enforce)
1271 }),
1272 ).
1273 ExtendWithErrorHandler(errorHandler).
1274 RunTestWithBp(t, bp)
Jeongik Cha2cc570d2019-10-29 15:44:45 +09001275 }
1276}
1277
Paul Duffin50c217c2019-06-12 13:25:22 +01001278func TestJNIPackaging(t *testing.T) {
Jaewoong Jungf9a04432019-07-17 11:15:09 -07001279 ctx, _ := testJava(t, cc.GatherRequiredDepsForTest(android.Android)+`
Paul Duffin50c217c2019-06-12 13:25:22 +01001280 cc_library {
1281 name: "libjni",
1282 system_shared_libs: [],
1283 stl: "none",
Colin Cross094cde42020-02-15 10:38:00 -08001284 sdk_version: "current",
Paul Duffin50c217c2019-06-12 13:25:22 +01001285 }
1286
1287 android_app {
1288 name: "app",
1289 jni_libs: ["libjni"],
Jeongik Cha538c0d02019-07-11 15:54:27 +09001290 sdk_version: "current",
Paul Duffin50c217c2019-06-12 13:25:22 +01001291 }
1292
1293 android_app {
1294 name: "app_noembed",
1295 jni_libs: ["libjni"],
1296 use_embedded_native_libs: false,
Jeongik Cha538c0d02019-07-11 15:54:27 +09001297 sdk_version: "current",
Paul Duffin50c217c2019-06-12 13:25:22 +01001298 }
1299
1300 android_app {
1301 name: "app_embed",
1302 jni_libs: ["libjni"],
1303 use_embedded_native_libs: true,
Jeongik Cha538c0d02019-07-11 15:54:27 +09001304 sdk_version: "current",
Paul Duffin50c217c2019-06-12 13:25:22 +01001305 }
1306
1307 android_test {
1308 name: "test",
Colin Crossc511bc52020-04-07 16:50:32 +00001309 sdk_version: "current",
Paul Duffin50c217c2019-06-12 13:25:22 +01001310 jni_libs: ["libjni"],
1311 }
1312
1313 android_test {
1314 name: "test_noembed",
Colin Crossc511bc52020-04-07 16:50:32 +00001315 sdk_version: "current",
Paul Duffin50c217c2019-06-12 13:25:22 +01001316 jni_libs: ["libjni"],
1317 use_embedded_native_libs: false,
1318 }
1319
1320 android_test_helper_app {
1321 name: "test_helper",
Colin Crossc511bc52020-04-07 16:50:32 +00001322 sdk_version: "current",
Paul Duffin50c217c2019-06-12 13:25:22 +01001323 jni_libs: ["libjni"],
1324 }
1325
1326 android_test_helper_app {
1327 name: "test_helper_noembed",
Colin Crossc511bc52020-04-07 16:50:32 +00001328 sdk_version: "current",
Paul Duffin50c217c2019-06-12 13:25:22 +01001329 jni_libs: ["libjni"],
1330 use_embedded_native_libs: false,
1331 }
1332 `)
1333
1334 testCases := []struct {
1335 name string
1336 packaged bool
1337 compressed bool
1338 }{
1339 {"app", false, false},
1340 {"app_noembed", false, false},
1341 {"app_embed", true, false},
1342 {"test", true, false},
1343 {"test_noembed", true, true},
1344 {"test_helper", true, false},
1345 {"test_helper_noembed", true, true},
1346 }
1347
1348 for _, test := range testCases {
1349 t.Run(test.name, func(t *testing.T) {
1350 app := ctx.ModuleForTests(test.name, "android_common")
1351 jniLibZip := app.MaybeOutput("jnilibs.zip")
1352 if g, w := (jniLibZip.Rule != nil), test.packaged; g != w {
1353 t.Errorf("expected jni packaged %v, got %v", w, g)
1354 }
1355
1356 if jniLibZip.Rule != nil {
1357 if g, w := !strings.Contains(jniLibZip.Args["jarArgs"], "-L 0"), test.compressed; g != w {
1358 t.Errorf("expected jni compressed %v, got %v", w, g)
1359 }
Colin Crossc511bc52020-04-07 16:50:32 +00001360
1361 if !strings.Contains(jniLibZip.Implicits[0].String(), "_sdk_") {
1362 t.Errorf("expected input %q to use sdk variant", jniLibZip.Implicits[0].String())
1363 }
Paul Duffin50c217c2019-06-12 13:25:22 +01001364 }
1365 })
1366 }
Colin Cross47fa9d32019-03-26 10:51:39 -07001367}
1368
Colin Cross3c007702020-05-08 11:20:24 -07001369func TestJNISDK(t *testing.T) {
1370 ctx, _ := testJava(t, cc.GatherRequiredDepsForTest(android.Android)+`
1371 cc_library {
1372 name: "libjni",
1373 system_shared_libs: [],
1374 stl: "none",
1375 sdk_version: "current",
1376 }
1377
1378 android_test {
1379 name: "app_platform",
1380 jni_libs: ["libjni"],
1381 platform_apis: true,
1382 }
1383
1384 android_test {
1385 name: "app_sdk",
1386 jni_libs: ["libjni"],
1387 sdk_version: "current",
1388 }
1389
1390 android_test {
1391 name: "app_force_platform",
1392 jni_libs: ["libjni"],
1393 sdk_version: "current",
1394 jni_uses_platform_apis: true,
1395 }
1396
1397 android_test {
1398 name: "app_force_sdk",
1399 jni_libs: ["libjni"],
1400 platform_apis: true,
1401 jni_uses_sdk_apis: true,
1402 }
Colin Crossc2d24052020-05-13 11:05:02 -07001403
1404 cc_library {
1405 name: "libvendorjni",
1406 system_shared_libs: [],
1407 stl: "none",
1408 vendor: true,
1409 }
1410
1411 android_test {
1412 name: "app_vendor",
1413 jni_libs: ["libvendorjni"],
1414 sdk_version: "current",
1415 vendor: true,
1416 }
Colin Cross3c007702020-05-08 11:20:24 -07001417 `)
1418
1419 testCases := []struct {
Colin Crossc2d24052020-05-13 11:05:02 -07001420 name string
1421 sdkJNI bool
1422 vendorJNI bool
Colin Cross3c007702020-05-08 11:20:24 -07001423 }{
Colin Crossc2d24052020-05-13 11:05:02 -07001424 {name: "app_platform"},
1425 {name: "app_sdk", sdkJNI: true},
1426 {name: "app_force_platform"},
1427 {name: "app_force_sdk", sdkJNI: true},
1428 {name: "app_vendor", vendorJNI: true},
Colin Cross3c007702020-05-08 11:20:24 -07001429 }
1430
Colin Crossc2d24052020-05-13 11:05:02 -07001431 platformJNI := ctx.ModuleForTests("libjni", "android_arm64_armv8-a_shared").
1432 Output("libjni.so").Output.String()
1433 sdkJNI := ctx.ModuleForTests("libjni", "android_arm64_armv8-a_sdk_shared").
1434 Output("libjni.so").Output.String()
1435 vendorJNI := ctx.ModuleForTests("libvendorjni", "android_arm64_armv8-a_shared").
1436 Output("libvendorjni.so").Output.String()
1437
Colin Cross3c007702020-05-08 11:20:24 -07001438 for _, test := range testCases {
1439 t.Run(test.name, func(t *testing.T) {
1440 app := ctx.ModuleForTests(test.name, "android_common")
Colin Cross3c007702020-05-08 11:20:24 -07001441
1442 jniLibZip := app.MaybeOutput("jnilibs.zip")
1443 if len(jniLibZip.Implicits) != 1 {
1444 t.Fatalf("expected exactly one jni library, got %q", jniLibZip.Implicits.Strings())
1445 }
1446 gotJNI := jniLibZip.Implicits[0].String()
1447
1448 if test.sdkJNI {
1449 if gotJNI != sdkJNI {
1450 t.Errorf("expected SDK JNI library %q, got %q", sdkJNI, gotJNI)
1451 }
Colin Crossc2d24052020-05-13 11:05:02 -07001452 } else if test.vendorJNI {
1453 if gotJNI != vendorJNI {
1454 t.Errorf("expected platform JNI library %q, got %q", vendorJNI, gotJNI)
1455 }
Colin Cross3c007702020-05-08 11:20:24 -07001456 } else {
1457 if gotJNI != platformJNI {
1458 t.Errorf("expected platform JNI library %q, got %q", platformJNI, gotJNI)
1459 }
1460 }
1461 })
1462 }
1463
1464 t.Run("jni_uses_platform_apis_error", func(t *testing.T) {
1465 testJavaError(t, `jni_uses_platform_apis: can only be set for modules that set sdk_version`, `
1466 android_test {
1467 name: "app_platform",
1468 platform_apis: true,
1469 jni_uses_platform_apis: true,
1470 }
1471 `)
1472 })
1473
1474 t.Run("jni_uses_sdk_apis_error", func(t *testing.T) {
1475 testJavaError(t, `jni_uses_sdk_apis: can only be set for modules that do not set sdk_version`, `
1476 android_test {
1477 name: "app_sdk",
1478 sdk_version: "current",
1479 jni_uses_sdk_apis: true,
1480 }
1481 `)
1482 })
1483
1484}
1485
Jaewoong Jung2ad817c2019-01-18 14:27:16 -08001486func TestCertificates(t *testing.T) {
1487 testCases := []struct {
1488 name string
1489 bp string
1490 certificateOverride string
Liz Kammere2b27f42020-05-07 13:24:05 -07001491 expectedLineage string
1492 expectedCertificate string
Jaewoong Jung2ad817c2019-01-18 14:27:16 -08001493 }{
1494 {
1495 name: "default",
1496 bp: `
1497 android_app {
1498 name: "foo",
1499 srcs: ["a.java"],
Jeongik Cha538c0d02019-07-11 15:54:27 +09001500 sdk_version: "current",
Jaewoong Jung2ad817c2019-01-18 14:27:16 -08001501 }
1502 `,
1503 certificateOverride: "",
Liz Kammere2b27f42020-05-07 13:24:05 -07001504 expectedLineage: "",
1505 expectedCertificate: "build/make/target/product/security/testkey.x509.pem build/make/target/product/security/testkey.pk8",
Jaewoong Jung2ad817c2019-01-18 14:27:16 -08001506 },
1507 {
1508 name: "module certificate property",
1509 bp: `
1510 android_app {
1511 name: "foo",
1512 srcs: ["a.java"],
Jeongik Cha538c0d02019-07-11 15:54:27 +09001513 certificate: ":new_certificate",
1514 sdk_version: "current",
Jaewoong Jung2ad817c2019-01-18 14:27:16 -08001515 }
1516
1517 android_app_certificate {
1518 name: "new_certificate",
Colin Cross3c007702020-05-08 11:20:24 -07001519 certificate: "cert/new_cert",
Jaewoong Jung2ad817c2019-01-18 14:27:16 -08001520 }
1521 `,
1522 certificateOverride: "",
Liz Kammere2b27f42020-05-07 13:24:05 -07001523 expectedLineage: "",
1524 expectedCertificate: "cert/new_cert.x509.pem cert/new_cert.pk8",
Jaewoong Jung2ad817c2019-01-18 14:27:16 -08001525 },
1526 {
1527 name: "path certificate property",
1528 bp: `
1529 android_app {
1530 name: "foo",
1531 srcs: ["a.java"],
Jeongik Cha538c0d02019-07-11 15:54:27 +09001532 certificate: "expiredkey",
1533 sdk_version: "current",
Jaewoong Jung2ad817c2019-01-18 14:27:16 -08001534 }
1535 `,
1536 certificateOverride: "",
Liz Kammere2b27f42020-05-07 13:24:05 -07001537 expectedLineage: "",
1538 expectedCertificate: "build/make/target/product/security/expiredkey.x509.pem build/make/target/product/security/expiredkey.pk8",
Jaewoong Jung2ad817c2019-01-18 14:27:16 -08001539 },
1540 {
1541 name: "certificate overrides",
1542 bp: `
1543 android_app {
1544 name: "foo",
1545 srcs: ["a.java"],
Jeongik Cha538c0d02019-07-11 15:54:27 +09001546 certificate: "expiredkey",
1547 sdk_version: "current",
Jaewoong Jung2ad817c2019-01-18 14:27:16 -08001548 }
1549
1550 android_app_certificate {
1551 name: "new_certificate",
Colin Cross3c007702020-05-08 11:20:24 -07001552 certificate: "cert/new_cert",
Jaewoong Jung2ad817c2019-01-18 14:27:16 -08001553 }
1554 `,
1555 certificateOverride: "foo:new_certificate",
Liz Kammere2b27f42020-05-07 13:24:05 -07001556 expectedLineage: "",
1557 expectedCertificate: "cert/new_cert.x509.pem cert/new_cert.pk8",
1558 },
1559 {
1560 name: "certificate lineage",
1561 bp: `
1562 android_app {
1563 name: "foo",
1564 srcs: ["a.java"],
1565 certificate: ":new_certificate",
1566 lineage: "lineage.bin",
1567 sdk_version: "current",
1568 }
1569
1570 android_app_certificate {
1571 name: "new_certificate",
1572 certificate: "cert/new_cert",
1573 }
1574 `,
1575 certificateOverride: "",
1576 expectedLineage: "--lineage lineage.bin",
1577 expectedCertificate: "cert/new_cert.x509.pem cert/new_cert.pk8",
Jaewoong Jung2ad817c2019-01-18 14:27:16 -08001578 },
Jaewoong Jung1c1b6e62021-03-09 15:02:31 -08001579 {
1580 name: "lineage from filegroup",
1581 bp: `
1582 android_app {
1583 name: "foo",
1584 srcs: ["a.java"],
1585 certificate: ":new_certificate",
1586 lineage: ":lineage_bin",
1587 sdk_version: "current",
1588 }
1589
1590 android_app_certificate {
1591 name: "new_certificate",
1592 certificate: "cert/new_cert",
1593 }
1594
1595 filegroup {
1596 name: "lineage_bin",
1597 srcs: ["lineage.bin"],
1598 }
1599 `,
1600 certificateOverride: "",
1601 expectedLineage: "--lineage lineage.bin",
1602 expectedCertificate: "cert/new_cert.x509.pem cert/new_cert.pk8",
1603 },
Jaewoong Jung2ad817c2019-01-18 14:27:16 -08001604 }
1605
1606 for _, test := range testCases {
1607 t.Run(test.name, func(t *testing.T) {
Paul Duffinfb0fe9f2021-03-22 17:31:52 +00001608 result := android.GroupFixturePreparers(
1609 PrepareForTestWithJavaDefaultModules,
1610 android.FixtureModifyProductVariables(func(variables android.FixtureProductVariables) {
1611 if test.certificateOverride != "" {
1612 variables.CertificateOverrides = []string{test.certificateOverride}
1613 }
1614 }),
1615 ).RunTestWithBp(t, test.bp)
Jaewoong Jung2ad817c2019-01-18 14:27:16 -08001616
Paul Duffinfb0fe9f2021-03-22 17:31:52 +00001617 foo := result.ModuleForTests("foo", "android_common")
Jaewoong Jung2ad817c2019-01-18 14:27:16 -08001618
1619 signapk := foo.Output("foo.apk")
Liz Kammere2b27f42020-05-07 13:24:05 -07001620 signCertificateFlags := signapk.Args["certificates"]
Paul Duffinfb0fe9f2021-03-22 17:31:52 +00001621 android.AssertStringEquals(t, "certificates flags", test.expectedCertificate, signCertificateFlags)
Liz Kammere2b27f42020-05-07 13:24:05 -07001622
1623 signFlags := signapk.Args["flags"]
Paul Duffinfb0fe9f2021-03-22 17:31:52 +00001624 android.AssertStringEquals(t, "signing flags", test.expectedLineage, signFlags)
Jaewoong Jung2ad817c2019-01-18 14:27:16 -08001625 })
1626 }
1627}
Jaewoong Jung9d22a912019-01-23 16:27:47 -08001628
Songchun Fan688de9a2020-03-24 20:32:24 -07001629func TestRequestV4SigningFlag(t *testing.T) {
1630 testCases := []struct {
1631 name string
1632 bp string
1633 expected string
1634 }{
1635 {
1636 name: "default",
1637 bp: `
1638 android_app {
1639 name: "foo",
1640 srcs: ["a.java"],
1641 sdk_version: "current",
1642 }
1643 `,
1644 expected: "",
1645 },
1646 {
1647 name: "default",
1648 bp: `
1649 android_app {
1650 name: "foo",
1651 srcs: ["a.java"],
1652 sdk_version: "current",
1653 v4_signature: false,
1654 }
1655 `,
1656 expected: "",
1657 },
1658 {
1659 name: "module certificate property",
1660 bp: `
1661 android_app {
1662 name: "foo",
1663 srcs: ["a.java"],
1664 sdk_version: "current",
1665 v4_signature: true,
1666 }
1667 `,
1668 expected: "--enable-v4",
1669 },
1670 }
1671
1672 for _, test := range testCases {
1673 t.Run(test.name, func(t *testing.T) {
Paul Duffinfb0fe9f2021-03-22 17:31:52 +00001674 result := android.GroupFixturePreparers(
1675 PrepareForTestWithJavaDefaultModules,
1676 ).RunTestWithBp(t, test.bp)
Songchun Fan688de9a2020-03-24 20:32:24 -07001677
Paul Duffinfb0fe9f2021-03-22 17:31:52 +00001678 foo := result.ModuleForTests("foo", "android_common")
Songchun Fan688de9a2020-03-24 20:32:24 -07001679
1680 signapk := foo.Output("foo.apk")
1681 signFlags := signapk.Args["flags"]
Paul Duffinfb0fe9f2021-03-22 17:31:52 +00001682 android.AssertStringEquals(t, "signing flags", test.expected, signFlags)
Songchun Fan688de9a2020-03-24 20:32:24 -07001683 })
1684 }
1685}
1686
Jaewoong Jung9d22a912019-01-23 16:27:47 -08001687func TestPackageNameOverride(t *testing.T) {
1688 testCases := []struct {
1689 name string
1690 bp string
1691 packageNameOverride string
1692 expected []string
1693 }{
1694 {
1695 name: "default",
1696 bp: `
1697 android_app {
1698 name: "foo",
1699 srcs: ["a.java"],
Jeongik Cha538c0d02019-07-11 15:54:27 +09001700 sdk_version: "current",
Jaewoong Jung9d22a912019-01-23 16:27:47 -08001701 }
1702 `,
1703 packageNameOverride: "",
1704 expected: []string{
Paul Duffinfb0fe9f2021-03-22 17:31:52 +00001705 "out/soong/.intermediates/foo/android_common/foo.apk",
1706 "out/soong/target/product/test_device/system/app/foo/foo.apk",
Jaewoong Jung9d22a912019-01-23 16:27:47 -08001707 },
1708 },
1709 {
Jooyung Han29e2f6d2022-01-08 12:13:59 +09001710 name: "overridden via PRODUCT_PACKAGE_NAME_OVERRIDES",
Jaewoong Jung9d22a912019-01-23 16:27:47 -08001711 bp: `
1712 android_app {
1713 name: "foo",
1714 srcs: ["a.java"],
Jeongik Cha538c0d02019-07-11 15:54:27 +09001715 sdk_version: "current",
Jaewoong Jung9d22a912019-01-23 16:27:47 -08001716 }
1717 `,
1718 packageNameOverride: "foo:bar",
1719 expected: []string{
1720 // The package apk should be still be the original name for test dependencies.
Paul Duffinfb0fe9f2021-03-22 17:31:52 +00001721 "out/soong/.intermediates/foo/android_common/bar.apk",
1722 "out/soong/target/product/test_device/system/app/bar/bar.apk",
Jaewoong Jung9d22a912019-01-23 16:27:47 -08001723 },
1724 },
Jooyung Han29e2f6d2022-01-08 12:13:59 +09001725 {
1726 name: "overridden via stem",
1727 bp: `
1728 android_app {
1729 name: "foo",
1730 srcs: ["a.java"],
1731 sdk_version: "current",
1732 stem: "bar",
1733 }
1734 `,
1735 packageNameOverride: "",
1736 expected: []string{
1737 "out/soong/.intermediates/foo/android_common/bar.apk",
1738 "out/soong/target/product/test_device/system/app/bar/bar.apk",
1739 },
1740 },
Jaewoong Jung9d22a912019-01-23 16:27:47 -08001741 }
1742
1743 for _, test := range testCases {
1744 t.Run(test.name, func(t *testing.T) {
Paul Duffinfb0fe9f2021-03-22 17:31:52 +00001745 result := android.GroupFixturePreparers(
1746 PrepareForTestWithJavaDefaultModules,
1747 android.FixtureModifyProductVariables(func(variables android.FixtureProductVariables) {
1748 if test.packageNameOverride != "" {
1749 variables.PackageNameOverrides = []string{test.packageNameOverride}
1750 }
1751 }),
1752 ).RunTestWithBp(t, test.bp)
Jaewoong Jung9d22a912019-01-23 16:27:47 -08001753
Paul Duffinfb0fe9f2021-03-22 17:31:52 +00001754 foo := result.ModuleForTests("foo", "android_common")
1755
Lukacs T. Berki9f6c24a2021-08-26 15:07:24 +02001756 outSoongDir := result.Config.SoongOutDir()
Jaewoong Jung9d22a912019-01-23 16:27:47 -08001757
1758 outputs := foo.AllOutputs()
1759 outputMap := make(map[string]bool)
1760 for _, o := range outputs {
Paul Duffinfb0fe9f2021-03-22 17:31:52 +00001761 outputMap[android.StringPathRelativeToTop(outSoongDir, o)] = true
Jaewoong Jung9d22a912019-01-23 16:27:47 -08001762 }
1763 for _, e := range test.expected {
1764 if _, exist := outputMap[e]; !exist {
1765 t.Errorf("Can't find %q in output files.\nAll outputs:%v", e, outputs)
1766 }
1767 }
1768 })
1769 }
1770}
Jaewoong Jung4102e5d2019-02-27 16:26:28 -08001771
1772func TestInstrumentationTargetOverridden(t *testing.T) {
1773 bp := `
1774 android_app {
1775 name: "foo",
1776 srcs: ["a.java"],
Jeongik Cha538c0d02019-07-11 15:54:27 +09001777 sdk_version: "current",
Jaewoong Jung4102e5d2019-02-27 16:26:28 -08001778 }
1779
1780 android_test {
1781 name: "bar",
1782 instrumentation_for: "foo",
Jeongik Cha538c0d02019-07-11 15:54:27 +09001783 sdk_version: "current",
Jaewoong Jung4102e5d2019-02-27 16:26:28 -08001784 }
1785 `
Jaewoong Jung4102e5d2019-02-27 16:26:28 -08001786
Paul Duffinfb0fe9f2021-03-22 17:31:52 +00001787 result := android.GroupFixturePreparers(
1788 PrepareForTestWithJavaDefaultModules,
1789 android.FixtureModifyProductVariables(func(variables android.FixtureProductVariables) {
1790 variables.ManifestPackageNameOverrides = []string{"foo:org.dandroid.bp"}
1791 }),
1792 ).RunTestWithBp(t, bp)
Jaewoong Jung4102e5d2019-02-27 16:26:28 -08001793
Paul Duffinfb0fe9f2021-03-22 17:31:52 +00001794 bar := result.ModuleForTests("bar", "android_common")
Jaewoong Jung4102e5d2019-02-27 16:26:28 -08001795 res := bar.Output("package-res.apk")
1796 aapt2Flags := res.Args["flags"]
1797 e := "--rename-instrumentation-target-package org.dandroid.bp"
1798 if !strings.Contains(aapt2Flags, e) {
1799 t.Errorf("target package renaming flag, %q is missing in aapt2 link flags, %q", e, aapt2Flags)
1800 }
1801}
Jaewoong Jung525443a2019-02-28 15:35:54 -08001802
1803func TestOverrideAndroidApp(t *testing.T) {
Paul Duffinfb0fe9f2021-03-22 17:31:52 +00001804 result := PrepareForTestWithJavaDefaultModules.RunTestWithBp(
1805 t, `
Jaewoong Jung525443a2019-02-28 15:35:54 -08001806 android_app {
1807 name: "foo",
1808 srcs: ["a.java"],
Jaewoong Junga641ee92019-03-27 11:17:14 -07001809 certificate: "expiredkey",
Jaewoong Jungb639a6a2019-05-10 15:16:29 -07001810 overrides: ["qux"],
Jeongik Cha538c0d02019-07-11 15:54:27 +09001811 sdk_version: "current",
Jaewoong Jung525443a2019-02-28 15:35:54 -08001812 }
1813
1814 override_android_app {
1815 name: "bar",
1816 base: "foo",
1817 certificate: ":new_certificate",
Liz Kammere2b27f42020-05-07 13:24:05 -07001818 lineage: "lineage.bin",
Baligh Uddin5b16dfb2020-02-11 17:27:19 -08001819 logging_parent: "bah",
Jaewoong Jung525443a2019-02-28 15:35:54 -08001820 }
1821
1822 android_app_certificate {
1823 name: "new_certificate",
1824 certificate: "cert/new_cert",
1825 }
Jaewoong Jung6f373f62019-03-13 10:13:24 -07001826
1827 override_android_app {
1828 name: "baz",
1829 base: "foo",
1830 package_name: "org.dandroid.bp",
1831 }
Liz Kammer9f9fd022020-06-18 19:44:06 +00001832
1833 override_android_app {
1834 name: "baz_no_rename_resources",
1835 base: "foo",
1836 package_name: "org.dandroid.bp",
1837 rename_resources_package: false,
1838 }
1839
1840 android_app {
1841 name: "foo_no_rename_resources",
1842 srcs: ["a.java"],
1843 certificate: "expiredkey",
1844 overrides: ["qux"],
1845 rename_resources_package: false,
1846 sdk_version: "current",
1847 }
1848
1849 override_android_app {
1850 name: "baz_base_no_rename_resources",
1851 base: "foo_no_rename_resources",
1852 package_name: "org.dandroid.bp",
1853 }
1854
1855 override_android_app {
1856 name: "baz_override_base_rename_resources",
1857 base: "foo_no_rename_resources",
1858 package_name: "org.dandroid.bp",
1859 rename_resources_package: true,
1860 }
Jaewoong Jung525443a2019-02-28 15:35:54 -08001861 `)
1862
1863 expectedVariants := []struct {
Liz Kammer9f9fd022020-06-18 19:44:06 +00001864 name string
1865 moduleName string
1866 variantName string
1867 apkName string
1868 apkPath string
1869 certFlag string
1870 lineageFlag string
1871 overrides []string
1872 packageFlag string
1873 renameResources bool
1874 logging_parent string
Jaewoong Jung525443a2019-02-28 15:35:54 -08001875 }{
1876 {
Liz Kammer9f9fd022020-06-18 19:44:06 +00001877 name: "foo",
1878 moduleName: "foo",
1879 variantName: "android_common",
Paul Duffinfb0fe9f2021-03-22 17:31:52 +00001880 apkPath: "out/soong/target/product/test_device/system/app/foo/foo.apk",
Liz Kammer9f9fd022020-06-18 19:44:06 +00001881 certFlag: "build/make/target/product/security/expiredkey.x509.pem build/make/target/product/security/expiredkey.pk8",
1882 lineageFlag: "",
1883 overrides: []string{"qux"},
1884 packageFlag: "",
1885 renameResources: false,
1886 logging_parent: "",
Jaewoong Jung525443a2019-02-28 15:35:54 -08001887 },
1888 {
Liz Kammer9f9fd022020-06-18 19:44:06 +00001889 name: "foo",
1890 moduleName: "bar",
1891 variantName: "android_common_bar",
Paul Duffinfb0fe9f2021-03-22 17:31:52 +00001892 apkPath: "out/soong/target/product/test_device/system/app/bar/bar.apk",
Liz Kammer9f9fd022020-06-18 19:44:06 +00001893 certFlag: "cert/new_cert.x509.pem cert/new_cert.pk8",
1894 lineageFlag: "--lineage lineage.bin",
1895 overrides: []string{"qux", "foo"},
1896 packageFlag: "",
1897 renameResources: false,
1898 logging_parent: "bah",
Jaewoong Jung6f373f62019-03-13 10:13:24 -07001899 },
1900 {
Liz Kammer9f9fd022020-06-18 19:44:06 +00001901 name: "foo",
1902 moduleName: "baz",
1903 variantName: "android_common_baz",
Paul Duffinfb0fe9f2021-03-22 17:31:52 +00001904 apkPath: "out/soong/target/product/test_device/system/app/baz/baz.apk",
Liz Kammer9f9fd022020-06-18 19:44:06 +00001905 certFlag: "build/make/target/product/security/expiredkey.x509.pem build/make/target/product/security/expiredkey.pk8",
1906 lineageFlag: "",
1907 overrides: []string{"qux", "foo"},
1908 packageFlag: "org.dandroid.bp",
1909 renameResources: true,
1910 logging_parent: "",
1911 },
1912 {
1913 name: "foo",
1914 moduleName: "baz_no_rename_resources",
1915 variantName: "android_common_baz_no_rename_resources",
Paul Duffinfb0fe9f2021-03-22 17:31:52 +00001916 apkPath: "out/soong/target/product/test_device/system/app/baz_no_rename_resources/baz_no_rename_resources.apk",
Liz Kammer9f9fd022020-06-18 19:44:06 +00001917 certFlag: "build/make/target/product/security/expiredkey.x509.pem build/make/target/product/security/expiredkey.pk8",
1918 lineageFlag: "",
1919 overrides: []string{"qux", "foo"},
1920 packageFlag: "org.dandroid.bp",
1921 renameResources: false,
1922 logging_parent: "",
1923 },
1924 {
1925 name: "foo_no_rename_resources",
1926 moduleName: "baz_base_no_rename_resources",
1927 variantName: "android_common_baz_base_no_rename_resources",
Paul Duffinfb0fe9f2021-03-22 17:31:52 +00001928 apkPath: "out/soong/target/product/test_device/system/app/baz_base_no_rename_resources/baz_base_no_rename_resources.apk",
Liz Kammer9f9fd022020-06-18 19:44:06 +00001929 certFlag: "build/make/target/product/security/expiredkey.x509.pem build/make/target/product/security/expiredkey.pk8",
1930 lineageFlag: "",
1931 overrides: []string{"qux", "foo_no_rename_resources"},
1932 packageFlag: "org.dandroid.bp",
1933 renameResources: false,
1934 logging_parent: "",
1935 },
1936 {
1937 name: "foo_no_rename_resources",
1938 moduleName: "baz_override_base_rename_resources",
1939 variantName: "android_common_baz_override_base_rename_resources",
Paul Duffinfb0fe9f2021-03-22 17:31:52 +00001940 apkPath: "out/soong/target/product/test_device/system/app/baz_override_base_rename_resources/baz_override_base_rename_resources.apk",
Liz Kammer9f9fd022020-06-18 19:44:06 +00001941 certFlag: "build/make/target/product/security/expiredkey.x509.pem build/make/target/product/security/expiredkey.pk8",
1942 lineageFlag: "",
1943 overrides: []string{"qux", "foo_no_rename_resources"},
1944 packageFlag: "org.dandroid.bp",
1945 renameResources: true,
1946 logging_parent: "",
Jaewoong Jung525443a2019-02-28 15:35:54 -08001947 },
1948 }
1949 for _, expected := range expectedVariants {
Paul Duffinfb0fe9f2021-03-22 17:31:52 +00001950 variant := result.ModuleForTests(expected.name, expected.variantName)
Jaewoong Jung525443a2019-02-28 15:35:54 -08001951
1952 // Check the final apk name
Paul Duffinfb0fe9f2021-03-22 17:31:52 +00001953 variant.Output(expected.apkPath)
Jaewoong Jung525443a2019-02-28 15:35:54 -08001954
1955 // Check the certificate paths
Jaewoong Jung5a498812019-11-07 14:14:38 -08001956 signapk := variant.Output(expected.moduleName + ".apk")
Liz Kammere2b27f42020-05-07 13:24:05 -07001957 certFlag := signapk.Args["certificates"]
Paul Duffinfb0fe9f2021-03-22 17:31:52 +00001958 android.AssertStringEquals(t, "certificates flags", expected.certFlag, certFlag)
Liz Kammere2b27f42020-05-07 13:24:05 -07001959
1960 // Check the lineage flags
1961 lineageFlag := signapk.Args["flags"]
Paul Duffinfb0fe9f2021-03-22 17:31:52 +00001962 android.AssertStringEquals(t, "signing flags", expected.lineageFlag, lineageFlag)
Jaewoong Jung525443a2019-02-28 15:35:54 -08001963
Jaewoong Jung6f373f62019-03-13 10:13:24 -07001964 // Check if the overrides field values are correctly aggregated.
Jaewoong Jung525443a2019-02-28 15:35:54 -08001965 mod := variant.Module().(*AndroidApp)
Paul Duffinfb0fe9f2021-03-22 17:31:52 +00001966 android.AssertDeepEquals(t, "overrides property", expected.overrides, mod.appProperties.Overrides)
Jaewoong Jung6f373f62019-03-13 10:13:24 -07001967
Baligh Uddin5b16dfb2020-02-11 17:27:19 -08001968 // Test Overridable property: Logging_parent
1969 logging_parent := mod.aapt.LoggingParent
Paul Duffinfb0fe9f2021-03-22 17:31:52 +00001970 android.AssertStringEquals(t, "overrides property value for logging parent", expected.logging_parent, logging_parent)
Baligh Uddin5b16dfb2020-02-11 17:27:19 -08001971
Liz Kammer1d5983b2020-05-19 19:15:37 +00001972 // Check the package renaming flag, if exists.
Jaewoong Jung6f373f62019-03-13 10:13:24 -07001973 res := variant.Output("package-res.apk")
1974 aapt2Flags := res.Args["flags"]
Liz Kammer9f9fd022020-06-18 19:44:06 +00001975 checkAapt2LinkFlag(t, aapt2Flags, "rename-manifest-package", expected.packageFlag)
1976 expectedPackage := expected.packageFlag
1977 if !expected.renameResources {
1978 expectedPackage = ""
Liz Kammer1d5983b2020-05-19 19:15:37 +00001979 }
Liz Kammer9f9fd022020-06-18 19:44:06 +00001980 checkAapt2LinkFlag(t, aapt2Flags, "rename-resources-package", expectedPackage)
Jaewoong Jung525443a2019-02-28 15:35:54 -08001981 }
1982}
Jaewoong Jungccbb3932019-04-15 09:48:31 -07001983
Jaewoong Jungb639a6a2019-05-10 15:16:29 -07001984func TestOverrideAndroidAppDependency(t *testing.T) {
Jaewoong Jungf9a04432019-07-17 11:15:09 -07001985 ctx, _ := testJava(t, `
Jaewoong Jungb639a6a2019-05-10 15:16:29 -07001986 android_app {
1987 name: "foo",
1988 srcs: ["a.java"],
Jeongik Cha538c0d02019-07-11 15:54:27 +09001989 sdk_version: "current",
Jaewoong Jungb639a6a2019-05-10 15:16:29 -07001990 }
1991
1992 override_android_app {
1993 name: "bar",
1994 base: "foo",
1995 package_name: "org.dandroid.bp",
1996 }
1997
1998 android_test {
1999 name: "baz",
2000 srcs: ["b.java"],
2001 instrumentation_for: "foo",
2002 }
2003
2004 android_test {
2005 name: "qux",
2006 srcs: ["b.java"],
2007 instrumentation_for: "bar",
2008 }
2009 `)
2010
2011 // Verify baz, which depends on the overridden module foo, has the correct classpath javac arg.
Paul Duffina71a67a2021-03-29 00:42:57 +01002012 javac := ctx.ModuleForTests("baz", "android_common").Rule("javac")
Paul Duffinfb0fe9f2021-03-22 17:31:52 +00002013 fooTurbine := "out/soong/.intermediates/foo/android_common/turbine-combined/foo.jar"
Jaewoong Jungb639a6a2019-05-10 15:16:29 -07002014 if !strings.Contains(javac.Args["classpath"], fooTurbine) {
2015 t.Errorf("baz classpath %v does not contain %q", javac.Args["classpath"], fooTurbine)
2016 }
2017
2018 // Verify qux, which depends on the overriding module bar, has the correct classpath javac arg.
Paul Duffina71a67a2021-03-29 00:42:57 +01002019 javac = ctx.ModuleForTests("qux", "android_common").Rule("javac")
Paul Duffinfb0fe9f2021-03-22 17:31:52 +00002020 barTurbine := "out/soong/.intermediates/foo/android_common_bar/turbine-combined/foo.jar"
Jaewoong Jungb639a6a2019-05-10 15:16:29 -07002021 if !strings.Contains(javac.Args["classpath"], barTurbine) {
2022 t.Errorf("qux classpath %v does not contain %q", javac.Args["classpath"], barTurbine)
2023 }
2024}
2025
Jaewoong Jung26dedd32019-06-06 08:45:58 -07002026func TestOverrideAndroidTest(t *testing.T) {
2027 ctx, _ := testJava(t, `
2028 android_app {
2029 name: "foo",
2030 srcs: ["a.java"],
2031 package_name: "com.android.foo",
2032 sdk_version: "current",
2033 }
2034
2035 override_android_app {
2036 name: "bar",
2037 base: "foo",
2038 package_name: "com.android.bar",
2039 }
2040
2041 android_test {
2042 name: "foo_test",
2043 srcs: ["b.java"],
2044 instrumentation_for: "foo",
2045 }
2046
2047 override_android_test {
2048 name: "bar_test",
2049 base: "foo_test",
2050 package_name: "com.android.bar.test",
2051 instrumentation_for: "bar",
2052 instrumentation_target_package: "com.android.bar",
2053 }
2054 `)
2055
2056 expectedVariants := []struct {
2057 moduleName string
2058 variantName string
2059 apkPath string
2060 overrides []string
2061 targetVariant string
2062 packageFlag string
2063 targetPackageFlag string
2064 }{
2065 {
2066 variantName: "android_common",
Jaewoong Jung326a9412019-11-21 10:41:00 -08002067 apkPath: "/target/product/test_device/testcases/foo_test/arm64/foo_test.apk",
Jaewoong Jung26dedd32019-06-06 08:45:58 -07002068 overrides: nil,
2069 targetVariant: "android_common",
2070 packageFlag: "",
2071 targetPackageFlag: "",
2072 },
2073 {
2074 variantName: "android_common_bar_test",
Jaewoong Jung326a9412019-11-21 10:41:00 -08002075 apkPath: "/target/product/test_device/testcases/bar_test/arm64/bar_test.apk",
Jaewoong Jung26dedd32019-06-06 08:45:58 -07002076 overrides: []string{"foo_test"},
2077 targetVariant: "android_common_bar",
2078 packageFlag: "com.android.bar.test",
2079 targetPackageFlag: "com.android.bar",
2080 },
2081 }
2082 for _, expected := range expectedVariants {
2083 variant := ctx.ModuleForTests("foo_test", expected.variantName)
2084
2085 // Check the final apk name
Paul Duffinfb0fe9f2021-03-22 17:31:52 +00002086 variant.Output("out/soong" + expected.apkPath)
Jaewoong Jung26dedd32019-06-06 08:45:58 -07002087
2088 // Check if the overrides field values are correctly aggregated.
2089 mod := variant.Module().(*AndroidTest)
2090 if !reflect.DeepEqual(expected.overrides, mod.appProperties.Overrides) {
2091 t.Errorf("Incorrect overrides property value, expected: %q, got: %q",
2092 expected.overrides, mod.appProperties.Overrides)
2093 }
2094
2095 // Check if javac classpath has the correct jar file path. This checks instrumentation_for overrides.
Paul Duffina71a67a2021-03-29 00:42:57 +01002096 javac := variant.Rule("javac")
Paul Duffinfb0fe9f2021-03-22 17:31:52 +00002097 turbine := filepath.Join("out", "soong", ".intermediates", "foo", expected.targetVariant, "turbine-combined", "foo.jar")
Jaewoong Jung26dedd32019-06-06 08:45:58 -07002098 if !strings.Contains(javac.Args["classpath"], turbine) {
2099 t.Errorf("classpath %q does not contain %q", javac.Args["classpath"], turbine)
2100 }
2101
2102 // Check aapt2 flags.
2103 res := variant.Output("package-res.apk")
2104 aapt2Flags := res.Args["flags"]
2105 checkAapt2LinkFlag(t, aapt2Flags, "rename-manifest-package", expected.packageFlag)
Liz Kammer9f9fd022020-06-18 19:44:06 +00002106 checkAapt2LinkFlag(t, aapt2Flags, "rename-resources-package", expected.packageFlag)
Jaewoong Jung26dedd32019-06-06 08:45:58 -07002107 checkAapt2LinkFlag(t, aapt2Flags, "rename-instrumentation-target-package", expected.targetPackageFlag)
2108 }
2109}
2110
Jaewoong Jung39982342020-01-14 10:27:18 -08002111func TestAndroidTest_FixTestConfig(t *testing.T) {
2112 ctx, _ := testJava(t, `
2113 android_app {
2114 name: "foo",
2115 srcs: ["a.java"],
2116 package_name: "com.android.foo",
2117 sdk_version: "current",
2118 }
2119
2120 android_test {
2121 name: "foo_test",
2122 srcs: ["b.java"],
2123 instrumentation_for: "foo",
2124 }
2125
2126 android_test {
2127 name: "bar_test",
2128 srcs: ["b.java"],
2129 package_name: "com.android.bar.test",
2130 instrumentation_for: "foo",
2131 }
2132
2133 override_android_test {
2134 name: "baz_test",
2135 base: "foo_test",
2136 package_name: "com.android.baz.test",
2137 }
2138 `)
2139
2140 testCases := []struct {
2141 moduleName string
2142 variantName string
2143 expectedFlags []string
2144 }{
2145 {
2146 moduleName: "foo_test",
2147 variantName: "android_common",
2148 },
2149 {
2150 moduleName: "bar_test",
2151 variantName: "android_common",
2152 expectedFlags: []string{
Paul Duffinfb0fe9f2021-03-22 17:31:52 +00002153 "--manifest out/soong/.intermediates/bar_test/android_common/manifest_fixer/AndroidManifest.xml",
Jaewoong Jung39982342020-01-14 10:27:18 -08002154 "--package-name com.android.bar.test",
2155 },
2156 },
2157 {
2158 moduleName: "foo_test",
2159 variantName: "android_common_baz_test",
2160 expectedFlags: []string{
Paul Duffinfb0fe9f2021-03-22 17:31:52 +00002161 "--manifest out/soong/.intermediates/foo_test/android_common_baz_test/manifest_fixer/AndroidManifest.xml",
Jaewoong Jung39982342020-01-14 10:27:18 -08002162 "--package-name com.android.baz.test",
2163 "--test-file-name baz_test.apk",
2164 },
2165 },
2166 }
2167
2168 for _, test := range testCases {
2169 variant := ctx.ModuleForTests(test.moduleName, test.variantName)
Paul Duffina71a67a2021-03-29 00:42:57 +01002170 params := variant.MaybeOutput("test_config_fixer/AndroidTest.xml")
Jaewoong Jung39982342020-01-14 10:27:18 -08002171
2172 if len(test.expectedFlags) > 0 {
2173 if params.Rule == nil {
2174 t.Errorf("test_config_fixer was expected to run, but didn't")
2175 } else {
2176 for _, flag := range test.expectedFlags {
2177 if !strings.Contains(params.RuleParams.Command, flag) {
2178 t.Errorf("Flag %q was not found in command: %q", flag, params.RuleParams.Command)
2179 }
2180 }
2181 }
2182 } else {
2183 if params.Rule != nil {
2184 t.Errorf("test_config_fixer was not expected to run, but did: %q", params.RuleParams.Command)
2185 }
2186 }
2187
2188 }
2189}
2190
Jaewoong Jungbc625cd2019-05-06 15:48:44 -07002191func TestStl(t *testing.T) {
Jaewoong Jungf9a04432019-07-17 11:15:09 -07002192 ctx, _ := testJava(t, cc.GatherRequiredDepsForTest(android.Android)+`
Jaewoong Jungbc625cd2019-05-06 15:48:44 -07002193 cc_library {
2194 name: "libjni",
Peter Collingbournead84f972019-12-17 16:46:18 -08002195 sdk_version: "current",
2196 stl: "c++_shared",
Jaewoong Jungbc625cd2019-05-06 15:48:44 -07002197 }
2198
2199 android_test {
2200 name: "stl",
2201 jni_libs: ["libjni"],
2202 compile_multilib: "both",
2203 sdk_version: "current",
2204 stl: "c++_shared",
2205 }
2206
2207 android_test {
2208 name: "system",
2209 jni_libs: ["libjni"],
2210 compile_multilib: "both",
2211 sdk_version: "current",
2212 }
2213 `)
2214
2215 testCases := []struct {
2216 name string
2217 jnis []string
2218 }{
2219 {"stl",
2220 []string{
2221 "libjni.so",
Jaewoong Jung710756a2019-06-04 11:53:47 -07002222 "libc++_shared.so",
Jaewoong Jungbc625cd2019-05-06 15:48:44 -07002223 },
2224 },
2225 {"system",
2226 []string{
2227 "libjni.so",
2228 },
2229 },
2230 }
2231
2232 for _, test := range testCases {
2233 t.Run(test.name, func(t *testing.T) {
2234 app := ctx.ModuleForTests(test.name, "android_common")
2235 jniLibZip := app.Output("jnilibs.zip")
2236 var jnis []string
2237 args := strings.Fields(jniLibZip.Args["jarArgs"])
2238 for i := 0; i < len(args); i++ {
2239 if args[i] == "-f" {
2240 jnis = append(jnis, args[i+1])
2241 i += 1
2242 }
2243 }
2244 jnisJoined := strings.Join(jnis, " ")
2245 for _, jni := range test.jnis {
2246 if !strings.Contains(jnisJoined, jni) {
2247 t.Errorf("missing jni %q in %q", jni, jnis)
2248 }
2249 }
2250 })
2251 }
2252}
Colin Cross50ddcc42019-05-16 12:28:22 -07002253
2254func TestUsesLibraries(t *testing.T) {
2255 bp := `
2256 java_sdk_library {
2257 name: "foo",
2258 srcs: ["a.java"],
2259 api_packages: ["foo"],
Jeongik Cha538c0d02019-07-11 15:54:27 +09002260 sdk_version: "current",
Colin Cross50ddcc42019-05-16 12:28:22 -07002261 }
2262
2263 java_sdk_library {
Paul Duffin859fe962020-05-15 10:20:31 +01002264 name: "qux",
2265 srcs: ["a.java"],
2266 api_packages: ["qux"],
2267 sdk_version: "current",
2268 }
2269
2270 java_sdk_library {
2271 name: "quuz",
2272 srcs: ["a.java"],
2273 api_packages: ["quuz"],
2274 sdk_version: "current",
2275 }
2276
2277 java_sdk_library {
Ulya Trafimovich65b03192020-12-03 16:50:22 +00002278 name: "fred",
2279 srcs: ["a.java"],
2280 api_packages: ["fred"],
2281 sdk_version: "current",
2282 }
2283
2284 java_sdk_library {
Colin Cross50ddcc42019-05-16 12:28:22 -07002285 name: "bar",
2286 srcs: ["a.java"],
2287 api_packages: ["bar"],
Jeongik Cha538c0d02019-07-11 15:54:27 +09002288 sdk_version: "current",
Colin Cross50ddcc42019-05-16 12:28:22 -07002289 }
2290
Ulya Trafimovich4b6d4c12020-08-19 14:58:01 +01002291 java_sdk_library {
2292 name: "runtime-library",
2293 srcs: ["a.java"],
2294 sdk_version: "current",
2295 }
2296
2297 java_library {
2298 name: "static-runtime-helper",
2299 srcs: ["a.java"],
2300 libs: ["runtime-library"],
2301 sdk_version: "current",
2302 }
2303
Ulya Trafimoviche4432872021-08-18 16:57:11 +01002304 java_library {
2305 name: "runtime-required-x",
2306 srcs: ["a.java"],
2307 installable: true,
2308 sdk_version: "current",
2309 }
2310
2311 java_library {
2312 name: "runtime-optional-x",
2313 srcs: ["a.java"],
2314 installable: true,
2315 sdk_version: "current",
2316 }
2317
2318 android_library {
2319 name: "static-x",
2320 uses_libs: ["runtime-required-x"],
2321 optional_uses_libs: ["runtime-optional-x"],
2322 sdk_version: "current",
2323 }
2324
2325 java_library {
2326 name: "runtime-required-y",
2327 srcs: ["a.java"],
2328 installable: true,
2329 sdk_version: "current",
2330 }
2331
2332 java_library {
2333 name: "runtime-optional-y",
2334 srcs: ["a.java"],
2335 installable: true,
2336 sdk_version: "current",
2337 }
2338
2339 java_library {
2340 name: "static-y",
2341 srcs: ["a.java"],
2342 uses_libs: ["runtime-required-y"],
2343 optional_uses_libs: ["runtime-optional-y"],
2344 sdk_version: "current",
2345 }
2346
Ulya Trafimovich861a8962021-02-26 14:49:07 +00002347 // A library that has to use "provides_uses_lib", because:
2348 // - it is not an SDK library
2349 // - its library name is different from its module name
2350 java_library {
2351 name: "non-sdk-lib",
2352 provides_uses_lib: "com.non.sdk.lib",
2353 installable: true,
2354 srcs: ["a.java"],
2355 }
2356
Colin Cross50ddcc42019-05-16 12:28:22 -07002357 android_app {
2358 name: "app",
2359 srcs: ["a.java"],
Ulya Trafimovich861a8962021-02-26 14:49:07 +00002360 libs: [
2361 "qux",
2362 "quuz.stubs"
2363 ],
Ulya Trafimovich65b03192020-12-03 16:50:22 +00002364 static_libs: [
2365 "static-runtime-helper",
2366 // statically linked component libraries should not pull their SDK libraries,
2367 // so "fred" should not be added to class loader context
2368 "fred.stubs",
Ulya Trafimoviche4432872021-08-18 16:57:11 +01002369 "static-x",
2370 "static-y",
Ulya Trafimovich65b03192020-12-03 16:50:22 +00002371 ],
Ulya Trafimovich861a8962021-02-26 14:49:07 +00002372 uses_libs: [
2373 "foo",
2374 "non-sdk-lib"
2375 ],
Jeongik Cha538c0d02019-07-11 15:54:27 +09002376 sdk_version: "current",
Colin Cross50ddcc42019-05-16 12:28:22 -07002377 optional_uses_libs: [
2378 "bar",
2379 "baz",
2380 ],
2381 }
2382
2383 android_app_import {
2384 name: "prebuilt",
2385 apk: "prebuilts/apk/app.apk",
2386 certificate: "platform",
Ulya Trafimovich861a8962021-02-26 14:49:07 +00002387 uses_libs: [
2388 "foo",
2389 "non-sdk-lib",
2390 "android.test.runner"
2391 ],
Colin Cross50ddcc42019-05-16 12:28:22 -07002392 optional_uses_libs: [
2393 "bar",
2394 "baz",
2395 ],
2396 }
2397 `
2398
Paul Duffin71ae5942021-03-22 15:36:52 +00002399 result := android.GroupFixturePreparers(
2400 prepareForJavaTest,
Paul Duffin2645a292021-03-13 02:36:00 +00002401 PrepareForTestWithJavaSdkLibraryFiles,
2402 FixtureWithLastReleaseApis("runtime-library", "foo", "quuz", "qux", "bar", "fred"),
Paul Duffind234b412021-03-12 23:04:46 +00002403 android.FixtureModifyProductVariables(func(variables android.FixtureProductVariables) {
2404 variables.MissingUsesLibraries = []string{"baz"}
2405 }),
2406 ).RunTestWithBp(t, bp)
Colin Cross50ddcc42019-05-16 12:28:22 -07002407
Paul Duffind234b412021-03-12 23:04:46 +00002408 app := result.ModuleForTests("app", "android_common")
2409 prebuilt := result.ModuleForTests("prebuilt", "android_common")
Colin Cross50ddcc42019-05-16 12:28:22 -07002410
Paul Duffin859fe962020-05-15 10:20:31 +01002411 // Test that implicit dependencies on java_sdk_library instances are passed to the manifest.
Ulya Trafimovich2eaf5c52021-02-26 12:05:11 +00002412 // This should not include explicit `uses_libs`/`optional_uses_libs` entries.
2413 actualManifestFixerArgs := app.Output("manifest_fixer/AndroidManifest.xml").Args["args"]
2414 expectManifestFixerArgs := `--extract-native-libs=true ` +
2415 `--uses-library qux ` +
2416 `--uses-library quuz ` +
Ulya Trafimovich0b1c70e2021-08-20 15:39:12 +01002417 `--uses-library runtime-library`
Paul Duffind234b412021-03-12 23:04:46 +00002418 android.AssertStringEquals(t, "manifest_fixer args", expectManifestFixerArgs, actualManifestFixerArgs)
Paul Duffin859fe962020-05-15 10:20:31 +01002419
Ulya Trafimovich2eaf5c52021-02-26 12:05:11 +00002420 // Test that all libraries are verified (library order matters).
2421 verifyCmd := app.Rule("verify_uses_libraries").RuleParams.Command
2422 verifyArgs := `--uses-library foo ` +
Ulya Trafimovicheea486a2021-02-26 11:38:21 +00002423 `--uses-library com.non.sdk.lib ` +
Ulya Trafimovich2eaf5c52021-02-26 12:05:11 +00002424 `--uses-library qux ` +
2425 `--uses-library quuz ` +
2426 `--uses-library runtime-library ` +
Ulya Trafimoviche4432872021-08-18 16:57:11 +01002427 `--uses-library runtime-required-x ` +
2428 `--uses-library runtime-required-y ` +
Ulya Trafimovich2eaf5c52021-02-26 12:05:11 +00002429 `--optional-uses-library bar ` +
Ulya Trafimoviche4432872021-08-18 16:57:11 +01002430 `--optional-uses-library baz ` +
2431 `--optional-uses-library runtime-optional-x ` +
2432 `--optional-uses-library runtime-optional-y `
Paul Duffind234b412021-03-12 23:04:46 +00002433 android.AssertStringDoesContain(t, "verify cmd args", verifyCmd, verifyArgs)
Colin Cross50ddcc42019-05-16 12:28:22 -07002434
Ulya Trafimovich2eaf5c52021-02-26 12:05:11 +00002435 // Test that all libraries are verified for an APK (library order matters).
2436 verifyApkCmd := prebuilt.Rule("verify_uses_libraries").RuleParams.Command
Ulya Trafimovich0aba2522021-03-03 16:38:37 +00002437 verifyApkArgs := `--uses-library foo ` +
2438 `--uses-library com.non.sdk.lib ` +
2439 `--uses-library android.test.runner ` +
2440 `--optional-uses-library bar ` +
2441 `--optional-uses-library baz `
Paul Duffind234b412021-03-12 23:04:46 +00002442 android.AssertStringDoesContain(t, "verify apk cmd args", verifyApkCmd, verifyApkArgs)
Colin Cross50ddcc42019-05-16 12:28:22 -07002443
Ulya Trafimovich8130c482020-10-07 15:17:13 +01002444 // Test that all present libraries are preopted, including implicit SDK dependencies, possibly stubs
Ulya Trafimovich2eaf5c52021-02-26 12:05:11 +00002445 cmd := app.Rule("dexpreopt").RuleParams.Command
Ulya Trafimovich8130c482020-10-07 15:17:13 +01002446 w := `--target-context-for-sdk any ` +
Ulya Trafimovich8130c482020-10-07 15:17:13 +01002447 `PCL[/system/framework/qux.jar]#` +
Ulya Trafimovich8cbc5d22020-11-03 15:15:46 +00002448 `PCL[/system/framework/quuz.jar]#` +
2449 `PCL[/system/framework/foo.jar]#` +
Ulya Trafimovich861a8962021-02-26 14:49:07 +00002450 `PCL[/system/framework/non-sdk-lib.jar]#` +
Ulya Trafimovich8cbc5d22020-11-03 15:15:46 +00002451 `PCL[/system/framework/bar.jar]#` +
Ulya Trafimoviche4432872021-08-18 16:57:11 +01002452 `PCL[/system/framework/runtime-library.jar]#` +
2453 `PCL[/system/framework/runtime-required-x.jar]#` +
2454 `PCL[/system/framework/runtime-optional-x.jar]#` +
2455 `PCL[/system/framework/runtime-required-y.jar]#` +
2456 `PCL[/system/framework/runtime-optional-y.jar] `
Paul Duffind234b412021-03-12 23:04:46 +00002457 android.AssertStringDoesContain(t, "dexpreopt app cmd args", cmd, w)
Colin Cross50ddcc42019-05-16 12:28:22 -07002458
Ulya Trafimovich24813e12020-10-07 15:05:21 +01002459 // Test conditional context for target SDK version 28.
Paul Duffind234b412021-03-12 23:04:46 +00002460 android.AssertStringDoesContain(t, "dexpreopt app cmd 28", cmd,
2461 `--target-context-for-sdk 28`+
2462 ` PCL[/system/framework/org.apache.http.legacy.jar] `)
Ulya Trafimovich24813e12020-10-07 15:05:21 +01002463
2464 // Test conditional context for target SDK version 29.
Paul Duffind234b412021-03-12 23:04:46 +00002465 android.AssertStringDoesContain(t, "dexpreopt app cmd 29", cmd,
2466 `--target-context-for-sdk 29`+
2467 ` PCL[/system/framework/android.hidl.manager-V1.0-java.jar]`+
2468 `#PCL[/system/framework/android.hidl.base-V1.0-java.jar] `)
Ulya Trafimovich24813e12020-10-07 15:05:21 +01002469
2470 // Test conditional context for target SDK version 30.
Ulya Trafimovich46b3d5b2020-10-21 13:20:55 +01002471 // "android.test.mock" is absent because "android.test.runner" is not used.
Paul Duffind234b412021-03-12 23:04:46 +00002472 android.AssertStringDoesContain(t, "dexpreopt app cmd 30", cmd,
2473 `--target-context-for-sdk 30`+
2474 ` PCL[/system/framework/android.test.base.jar] `)
Ulya Trafimovich24813e12020-10-07 15:05:21 +01002475
Ulya Trafimovichfc24ad32020-08-19 16:32:54 +01002476 cmd = prebuilt.Rule("dexpreopt").RuleParams.Command
Paul Duffind234b412021-03-12 23:04:46 +00002477 android.AssertStringDoesContain(t, "dexpreopt prebuilt cmd", cmd,
2478 `--target-context-for-sdk any`+
2479 ` PCL[/system/framework/foo.jar]`+
2480 `#PCL[/system/framework/non-sdk-lib.jar]`+
2481 `#PCL[/system/framework/android.test.runner.jar]`+
2482 `#PCL[/system/framework/bar.jar] `)
Ulya Trafimovich24813e12020-10-07 15:05:21 +01002483
2484 // Test conditional context for target SDK version 30.
Ulya Trafimovich46b3d5b2020-10-21 13:20:55 +01002485 // "android.test.mock" is present because "android.test.runner" is used.
Paul Duffind234b412021-03-12 23:04:46 +00002486 android.AssertStringDoesContain(t, "dexpreopt prebuilt cmd 30", cmd,
2487 `--target-context-for-sdk 30`+
2488 ` PCL[/system/framework/android.test.base.jar]`+
2489 `#PCL[/system/framework/android.test.mock.jar] `)
Colin Cross50ddcc42019-05-16 12:28:22 -07002490}
Jaewoong Jungc27ab662019-05-30 15:51:14 -07002491
Ulya Trafimovich9023b022021-03-22 16:02:28 +00002492func TestDexpreoptBcp(t *testing.T) {
2493 bp := `
2494 java_sdk_library {
2495 name: "foo",
2496 srcs: ["a.java"],
2497 api_packages: ["foo"],
2498 sdk_version: "current",
2499 }
2500
2501 java_sdk_library {
2502 name: "bar",
2503 srcs: ["a.java"],
2504 api_packages: ["bar"],
2505 permitted_packages: ["bar"],
2506 sdk_version: "current",
2507 }
2508
2509 android_app {
2510 name: "app",
2511 srcs: ["a.java"],
2512 sdk_version: "current",
2513 }
2514 `
2515
2516 testCases := []struct {
2517 name string
2518 with bool
2519 expect string
2520 }{
2521 {
2522 name: "with updatable bcp",
2523 with: true,
2524 expect: "/system/framework/foo.jar:/system/framework/bar.jar",
2525 },
2526 {
2527 name: "without updatable bcp",
2528 with: false,
2529 expect: "/system/framework/foo.jar",
2530 },
2531 }
2532
2533 for _, test := range testCases {
2534 t.Run(test.name, func(t *testing.T) {
2535 result := android.GroupFixturePreparers(
2536 prepareForJavaTest,
2537 PrepareForTestWithJavaSdkLibraryFiles,
2538 FixtureWithLastReleaseApis("runtime-library", "foo", "bar"),
2539 dexpreopt.FixtureSetBootJars("platform:foo"),
satayevd604b212021-07-21 14:23:52 +01002540 dexpreopt.FixtureSetApexBootJars("platform:bar"),
Ulya Trafimovich9023b022021-03-22 16:02:28 +00002541 dexpreopt.FixtureSetPreoptWithUpdatableBcp(test.with),
2542 ).RunTestWithBp(t, bp)
2543
2544 app := result.ModuleForTests("app", "android_common")
2545 cmd := app.Rule("dexpreopt").RuleParams.Command
2546 bcp := " -Xbootclasspath-locations:" + test.expect + " " // space at the end matters
2547 android.AssertStringDoesContain(t, "dexpreopt app bcp", cmd, bcp)
2548 })
2549 }
2550}
2551
Jaewoong Jungc27ab662019-05-30 15:51:14 -07002552func TestCodelessApp(t *testing.T) {
2553 testCases := []struct {
2554 name string
2555 bp string
2556 noCode bool
2557 }{
2558 {
2559 name: "normal",
2560 bp: `
2561 android_app {
2562 name: "foo",
2563 srcs: ["a.java"],
Jeongik Cha538c0d02019-07-11 15:54:27 +09002564 sdk_version: "current",
Jaewoong Jungc27ab662019-05-30 15:51:14 -07002565 }
2566 `,
2567 noCode: false,
2568 },
2569 {
2570 name: "app without sources",
2571 bp: `
2572 android_app {
2573 name: "foo",
Jeongik Cha538c0d02019-07-11 15:54:27 +09002574 sdk_version: "current",
Jaewoong Jungc27ab662019-05-30 15:51:14 -07002575 }
2576 `,
2577 noCode: true,
2578 },
2579 {
2580 name: "app with libraries",
2581 bp: `
2582 android_app {
2583 name: "foo",
2584 static_libs: ["lib"],
Jeongik Cha538c0d02019-07-11 15:54:27 +09002585 sdk_version: "current",
Jaewoong Jungc27ab662019-05-30 15:51:14 -07002586 }
2587
2588 java_library {
2589 name: "lib",
2590 srcs: ["a.java"],
Jeongik Cha538c0d02019-07-11 15:54:27 +09002591 sdk_version: "current",
Jaewoong Jungc27ab662019-05-30 15:51:14 -07002592 }
2593 `,
2594 noCode: false,
2595 },
2596 {
2597 name: "app with sourceless libraries",
2598 bp: `
2599 android_app {
2600 name: "foo",
2601 static_libs: ["lib"],
Jeongik Cha538c0d02019-07-11 15:54:27 +09002602 sdk_version: "current",
Jaewoong Jungc27ab662019-05-30 15:51:14 -07002603 }
2604
2605 java_library {
2606 name: "lib",
Jeongik Cha538c0d02019-07-11 15:54:27 +09002607 sdk_version: "current",
Jaewoong Jungc27ab662019-05-30 15:51:14 -07002608 }
2609 `,
2610 // TODO(jungjw): this should probably be true
2611 noCode: false,
2612 },
2613 }
2614
2615 for _, test := range testCases {
2616 t.Run(test.name, func(t *testing.T) {
2617 ctx := testApp(t, test.bp)
2618
2619 foo := ctx.ModuleForTests("foo", "android_common")
2620 manifestFixerArgs := foo.Output("manifest_fixer/AndroidManifest.xml").Args["args"]
2621 if strings.Contains(manifestFixerArgs, "--has-no-code") != test.noCode {
2622 t.Errorf("unexpected manifest_fixer args: %q", manifestFixerArgs)
2623 }
2624 })
2625 }
2626}
Jaewoong Jung5b425e22019-06-17 17:40:56 -07002627
2628func TestEmbedNotice(t *testing.T) {
Paul Duffinfb0fe9f2021-03-22 17:31:52 +00002629 result := android.GroupFixturePreparers(
2630 PrepareForTestWithJavaDefaultModules,
2631 cc.PrepareForTestWithCcDefaultModules,
2632 genrule.PrepareForTestWithGenRuleBuildComponents,
2633 android.MockFS{
2634 "APP_NOTICE": nil,
2635 "GENRULE_NOTICE": nil,
2636 "LIB_NOTICE": nil,
2637 "TOOL_NOTICE": nil,
2638 }.AddToFixture(),
2639 ).RunTestWithBp(t, `
Jaewoong Jung5b425e22019-06-17 17:40:56 -07002640 android_app {
2641 name: "foo",
2642 srcs: ["a.java"],
2643 static_libs: ["javalib"],
2644 jni_libs: ["libjni"],
2645 notice: "APP_NOTICE",
2646 embed_notices: true,
Jeongik Cha538c0d02019-07-11 15:54:27 +09002647 sdk_version: "current",
Jaewoong Jung5b425e22019-06-17 17:40:56 -07002648 }
2649
2650 // No embed_notice flag
2651 android_app {
2652 name: "bar",
2653 srcs: ["a.java"],
2654 jni_libs: ["libjni"],
2655 notice: "APP_NOTICE",
Jeongik Cha538c0d02019-07-11 15:54:27 +09002656 sdk_version: "current",
Jaewoong Jung5b425e22019-06-17 17:40:56 -07002657 }
2658
2659 // No NOTICE files
2660 android_app {
2661 name: "baz",
2662 srcs: ["a.java"],
2663 embed_notices: true,
Jeongik Cha538c0d02019-07-11 15:54:27 +09002664 sdk_version: "current",
Jaewoong Jung5b425e22019-06-17 17:40:56 -07002665 }
2666
2667 cc_library {
2668 name: "libjni",
2669 system_shared_libs: [],
2670 stl: "none",
2671 notice: "LIB_NOTICE",
Colin Cross094cde42020-02-15 10:38:00 -08002672 sdk_version: "current",
Jaewoong Jung5b425e22019-06-17 17:40:56 -07002673 }
2674
2675 java_library {
2676 name: "javalib",
2677 srcs: [
2678 ":gen",
2679 ],
Jeongik Cha538c0d02019-07-11 15:54:27 +09002680 sdk_version: "current",
Jaewoong Jung5b425e22019-06-17 17:40:56 -07002681 }
2682
2683 genrule {
2684 name: "gen",
2685 tools: ["gentool"],
2686 out: ["gen.java"],
2687 notice: "GENRULE_NOTICE",
2688 }
2689
2690 java_binary_host {
2691 name: "gentool",
2692 srcs: ["b.java"],
2693 notice: "TOOL_NOTICE",
2694 }
Paul Duffinfb0fe9f2021-03-22 17:31:52 +00002695 `)
Jaewoong Jung5b425e22019-06-17 17:40:56 -07002696
2697 // foo has NOTICE files to process, and embed_notices is true.
Paul Duffinfb0fe9f2021-03-22 17:31:52 +00002698 foo := result.ModuleForTests("foo", "android_common")
Jaewoong Jung5b425e22019-06-17 17:40:56 -07002699 // verify merge notices rule.
2700 mergeNotices := foo.Rule("mergeNoticesRule")
2701 noticeInputs := mergeNotices.Inputs.Strings()
2702 // TOOL_NOTICE should be excluded as it's a host module.
2703 if len(mergeNotices.Inputs) != 3 {
2704 t.Errorf("number of input notice files: expected = 3, actual = %q", noticeInputs)
2705 }
2706 if !inList("APP_NOTICE", noticeInputs) {
2707 t.Errorf("APP_NOTICE is missing from notice files, %q", noticeInputs)
2708 }
2709 if !inList("LIB_NOTICE", noticeInputs) {
2710 t.Errorf("LIB_NOTICE is missing from notice files, %q", noticeInputs)
2711 }
2712 if !inList("GENRULE_NOTICE", noticeInputs) {
2713 t.Errorf("GENRULE_NOTICE is missing from notice files, %q", noticeInputs)
2714 }
2715 // aapt2 flags should include -A <NOTICE dir> so that its contents are put in the APK's /assets.
Paul Duffina71a67a2021-03-29 00:42:57 +01002716 res := foo.Output("package-res.apk")
Jaewoong Jung5b425e22019-06-17 17:40:56 -07002717 aapt2Flags := res.Args["flags"]
Paul Duffinfb0fe9f2021-03-22 17:31:52 +00002718 e := "-A out/soong/.intermediates/foo/android_common/NOTICE"
2719 android.AssertStringDoesContain(t, "expected.apkPath", aapt2Flags, e)
Jaewoong Jung5b425e22019-06-17 17:40:56 -07002720
2721 // bar has NOTICE files to process, but embed_notices is not set.
Paul Duffinfb0fe9f2021-03-22 17:31:52 +00002722 bar := result.ModuleForTests("bar", "android_common")
Paul Duffina71a67a2021-03-29 00:42:57 +01002723 res = bar.Output("package-res.apk")
Jaewoong Jung98772792019-07-01 17:15:13 -07002724 aapt2Flags = res.Args["flags"]
Paul Duffinfb0fe9f2021-03-22 17:31:52 +00002725 e = "-A out/soong/.intermediates/bar/android_common/NOTICE"
2726 android.AssertStringDoesNotContain(t, "bar shouldn't have the asset dir flag for NOTICE", aapt2Flags, e)
Jaewoong Jung5b425e22019-06-17 17:40:56 -07002727
2728 // baz's embed_notice is true, but it doesn't have any NOTICE files.
Paul Duffinfb0fe9f2021-03-22 17:31:52 +00002729 baz := result.ModuleForTests("baz", "android_common")
Jaewoong Jung98772792019-07-01 17:15:13 -07002730 res = baz.Output("package-res.apk")
2731 aapt2Flags = res.Args["flags"]
Paul Duffinfb0fe9f2021-03-22 17:31:52 +00002732 e = "-A out/soong/.intermediates/baz/android_common/NOTICE"
Jaewoong Jung98772792019-07-01 17:15:13 -07002733 if strings.Contains(aapt2Flags, e) {
2734 t.Errorf("baz shouldn't have the asset dir flag for NOTICE: %q", e)
Jaewoong Jung5b425e22019-06-17 17:40:56 -07002735 }
2736}
Colin Cross53a87f52019-06-25 13:35:30 -07002737
2738func TestUncompressDex(t *testing.T) {
2739 testCases := []struct {
2740 name string
2741 bp string
2742
2743 uncompressedPlatform bool
2744 uncompressedUnbundled bool
2745 }{
2746 {
2747 name: "normal",
2748 bp: `
2749 android_app {
2750 name: "foo",
2751 srcs: ["a.java"],
Jeongik Cha538c0d02019-07-11 15:54:27 +09002752 sdk_version: "current",
Colin Cross53a87f52019-06-25 13:35:30 -07002753 }
2754 `,
2755 uncompressedPlatform: true,
2756 uncompressedUnbundled: false,
2757 },
2758 {
2759 name: "use_embedded_dex",
2760 bp: `
2761 android_app {
2762 name: "foo",
2763 use_embedded_dex: true,
2764 srcs: ["a.java"],
Jeongik Cha538c0d02019-07-11 15:54:27 +09002765 sdk_version: "current",
Colin Cross53a87f52019-06-25 13:35:30 -07002766 }
2767 `,
2768 uncompressedPlatform: true,
2769 uncompressedUnbundled: true,
2770 },
2771 {
2772 name: "privileged",
2773 bp: `
2774 android_app {
2775 name: "foo",
2776 privileged: true,
2777 srcs: ["a.java"],
Jeongik Cha538c0d02019-07-11 15:54:27 +09002778 sdk_version: "current",
Colin Cross53a87f52019-06-25 13:35:30 -07002779 }
2780 `,
2781 uncompressedPlatform: true,
2782 uncompressedUnbundled: true,
2783 },
David Srbeckye033cba2020-05-20 22:20:28 +01002784 {
2785 name: "normal_uncompress_dex_true",
2786 bp: `
2787 android_app {
2788 name: "foo",
2789 srcs: ["a.java"],
2790 sdk_version: "current",
2791 uncompress_dex: true,
2792 }
2793 `,
2794 uncompressedPlatform: true,
2795 uncompressedUnbundled: true,
2796 },
2797 {
2798 name: "normal_uncompress_dex_false",
2799 bp: `
2800 android_app {
2801 name: "foo",
2802 srcs: ["a.java"],
2803 sdk_version: "current",
2804 uncompress_dex: false,
2805 }
2806 `,
2807 uncompressedPlatform: false,
2808 uncompressedUnbundled: false,
2809 },
Colin Cross53a87f52019-06-25 13:35:30 -07002810 }
2811
2812 test := func(t *testing.T, bp string, want bool, unbundled bool) {
2813 t.Helper()
2814
Paul Duffin71ae5942021-03-22 15:36:52 +00002815 result := android.GroupFixturePreparers(
2816 prepareForJavaTest,
Paul Duffin2645a292021-03-13 02:36:00 +00002817 PrepareForTestWithPrebuiltsOfCurrentApi,
Paul Duffincdb88a92021-03-14 00:36:50 +00002818 android.FixtureModifyProductVariables(func(variables android.FixtureProductVariables) {
2819 if unbundled {
2820 variables.Unbundled_build = proptools.BoolPtr(true)
2821 variables.Always_use_prebuilt_sdks = proptools.BoolPtr(true)
2822 }
2823 }),
2824 ).RunTestWithBp(t, bp)
Colin Cross53a87f52019-06-25 13:35:30 -07002825
Paul Duffincdb88a92021-03-14 00:36:50 +00002826 foo := result.ModuleForTests("foo", "android_common")
Colin Cross53a87f52019-06-25 13:35:30 -07002827 dex := foo.Rule("r8")
2828 uncompressedInDexJar := strings.Contains(dex.Args["zipFlags"], "-L 0")
2829 aligned := foo.MaybeRule("zipalign").Rule != nil
2830
Paul Duffincdb88a92021-03-14 00:36:50 +00002831 android.AssertBoolEquals(t, "uncompressed in dex", want, uncompressedInDexJar)
Colin Cross53a87f52019-06-25 13:35:30 -07002832
Paul Duffincdb88a92021-03-14 00:36:50 +00002833 android.AssertBoolEquals(t, "aligne", want, aligned)
Colin Cross53a87f52019-06-25 13:35:30 -07002834 }
2835
2836 for _, tt := range testCases {
2837 t.Run(tt.name, func(t *testing.T) {
2838 t.Run("platform", func(t *testing.T) {
2839 test(t, tt.bp, tt.uncompressedPlatform, false)
2840 })
2841 t.Run("unbundled", func(t *testing.T) {
2842 test(t, tt.bp, tt.uncompressedUnbundled, true)
2843 })
2844 })
2845 }
2846}
Jaewoong Jung26dedd32019-06-06 08:45:58 -07002847
2848func checkAapt2LinkFlag(t *testing.T, aapt2Flags, flagName, expectedValue string) {
2849 if expectedValue != "" {
2850 expectedFlag := "--" + flagName + " " + expectedValue
2851 if !strings.Contains(aapt2Flags, expectedFlag) {
2852 t.Errorf("%q is missing in aapt2 link flags, %q", expectedFlag, aapt2Flags)
2853 }
2854 } else {
2855 unexpectedFlag := "--" + flagName
2856 if strings.Contains(aapt2Flags, unexpectedFlag) {
2857 t.Errorf("unexpected flag, %q is found in aapt2 link flags, %q", unexpectedFlag, aapt2Flags)
2858 }
2859 }
2860}
Jaewoong Jung9befb0c2020-01-18 10:33:43 -08002861
Cole Faust9a631312020-10-22 21:05:24 +00002862func TestExportedProguardFlagFiles(t *testing.T) {
2863 ctx, _ := testJava(t, `
2864 android_app {
2865 name: "foo",
2866 sdk_version: "current",
2867 static_libs: ["lib1"],
2868 }
2869
2870 android_library {
2871 name: "lib1",
2872 sdk_version: "current",
2873 optimize: {
2874 proguard_flags_files: ["lib1proguard.cfg"],
2875 }
2876 }
2877 `)
2878
2879 m := ctx.ModuleForTests("foo", "android_common")
2880 hasLib1Proguard := false
2881 for _, s := range m.Rule("java.r8").Implicits.Strings() {
2882 if s == "lib1proguard.cfg" {
2883 hasLib1Proguard = true
2884 break
2885 }
2886 }
2887
2888 if !hasLib1Proguard {
2889 t.Errorf("App does not use library proguard config")
2890 }
2891}
Spandan Dasc8cb0c32021-12-08 17:00:38 +00002892
2893func TestTargetSdkVersionManifestFixer(t *testing.T) {
2894 platform_sdk_codename := "Tiramisu"
2895 testCases := []struct {
2896 name string
2897 targetSdkVersionInBp string
2898 targetSdkVersionExpected string
2899 unbundledBuild bool
2900 }{
2901 {
2902 name: "Non-Unbundled build: Android.bp has targetSdkVersion",
2903 targetSdkVersionInBp: "30",
2904 targetSdkVersionExpected: "30",
2905 unbundledBuild: false,
2906 },
2907 {
2908 name: "Unbundled build: Android.bp has targetSdkVersion",
2909 targetSdkVersionInBp: "30",
2910 targetSdkVersionExpected: "30",
2911 unbundledBuild: true,
2912 },
2913 {
2914 name: "Non-Unbundled build: Android.bp has targetSdkVersion equal to platform_sdk_codename",
2915 targetSdkVersionInBp: platform_sdk_codename,
2916 targetSdkVersionExpected: platform_sdk_codename,
2917 unbundledBuild: false,
2918 },
2919 {
2920 name: "Unbundled build: Android.bp has targetSdkVersion equal to platform_sdk_codename",
2921 targetSdkVersionInBp: platform_sdk_codename,
2922 targetSdkVersionExpected: "10000",
2923 unbundledBuild: true,
2924 },
2925
2926 {
2927 name: "Non-Unbundled build: Android.bp has no targetSdkVersion",
2928 targetSdkVersionExpected: platform_sdk_codename,
2929 unbundledBuild: false,
2930 },
2931 {
2932 name: "Unbundled build: Android.bp has no targetSdkVersion",
2933 targetSdkVersionExpected: "10000",
2934 unbundledBuild: true,
2935 },
2936 }
2937 for _, testCase := range testCases {
2938 bp := fmt.Sprintf(`
2939 android_app {
2940 name: "foo",
2941 sdk_version: "current",
2942 target_sdk_version: "%v",
2943 }
2944 `, testCase.targetSdkVersionInBp)
2945 fixture := android.GroupFixturePreparers(
2946 prepareForJavaTest,
2947 android.FixtureModifyProductVariables(func(variables android.FixtureProductVariables) {
2948 // explicitly set platform_sdk_codename to make the test deterministic
2949 variables.Platform_sdk_codename = &platform_sdk_codename
2950 variables.Platform_version_active_codenames = []string{platform_sdk_codename}
2951 // create a non-empty list if unbundledBuild==true
2952 if testCase.unbundledBuild {
2953 variables.Unbundled_build_apps = []string{"apex_a", "apex_b"}
2954 }
2955 }),
2956 )
2957
2958 result := fixture.RunTestWithBp(t, bp)
2959 foo := result.ModuleForTests("foo", "android_common")
2960
2961 manifestFixerArgs := foo.Output("manifest_fixer/AndroidManifest.xml").Args
2962 android.AssertStringEquals(t, testCase.name, testCase.targetSdkVersionExpected, manifestFixerArgs["targetSdkVersion"])
2963 }
2964}