blob: 36121124351c23c82d539d396adc8e8d84992121 [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"
Jaewoong Junga5e5abc2019-04-26 14:31:50 -070021 "regexp"
Colin Crossb69301e2017-12-01 10:48:26 -080022 "sort"
Colin Crossd09b0b62018-04-18 11:06:47 -070023 "strings"
Colin Cross3bc7ffa2017-11-22 16:19:37 -080024 "testing"
Jaewoong Junga5e5abc2019-04-26 14:31:50 -070025
26 "github.com/google/blueprint/proptools"
27
28 "android/soong/android"
29 "android/soong/cc"
Colin Cross3bc7ffa2017-11-22 16:19:37 -080030)
31
32var (
33 resourceFiles = []string{
34 "res/layout/layout.xml",
35 "res/values/strings.xml",
36 "res/values-en-rUS/strings.xml",
37 }
38
39 compiledResourceFiles = []string{
40 "aapt2/res/layout_layout.xml.flat",
41 "aapt2/res/values_strings.arsc.flat",
42 "aapt2/res/values-en-rUS_strings.arsc.flat",
43 }
44)
45
Colin Cross98be1bb2019-12-13 20:41:13 -080046func testAppConfig(env map[string]string, bp string, fs map[string][]byte) android.Config {
Colin Cross527012a2017-11-30 22:56:16 -080047 appFS := map[string][]byte{}
48 for k, v := range fs {
49 appFS[k] = v
Colin Cross3bc7ffa2017-11-22 16:19:37 -080050 }
51
Colin Cross527012a2017-11-30 22:56:16 -080052 for _, file := range resourceFiles {
53 appFS[file] = nil
54 }
55
Colin Cross98be1bb2019-12-13 20:41:13 -080056 return testConfig(env, bp, appFS)
Colin Cross527012a2017-11-30 22:56:16 -080057}
58
59func testApp(t *testing.T, bp string) *android.TestContext {
Colin Cross98be1bb2019-12-13 20:41:13 -080060 config := testAppConfig(nil, bp, nil)
Colin Cross527012a2017-11-30 22:56:16 -080061
Colin Cross98be1bb2019-12-13 20:41:13 -080062 ctx := testContext()
Colin Cross527012a2017-11-30 22:56:16 -080063
64 run(t, ctx, config)
65
66 return ctx
Colin Cross3bc7ffa2017-11-22 16:19:37 -080067}
68
69func TestApp(t *testing.T) {
Colin Crossa97c5d32018-03-28 14:58:31 -070070 for _, moduleType := range []string{"android_app", "android_library"} {
71 t.Run(moduleType, func(t *testing.T) {
72 ctx := testApp(t, moduleType+` {
73 name: "foo",
74 srcs: ["a.java"],
Jeongik Cha538c0d02019-07-11 15:54:27 +090075 sdk_version: "current"
Colin Crossa97c5d32018-03-28 14:58:31 -070076 }
77 `)
Colin Cross3bc7ffa2017-11-22 16:19:37 -080078
Colin Crossa97c5d32018-03-28 14:58:31 -070079 foo := ctx.ModuleForTests("foo", "android_common")
Colin Cross3bc7ffa2017-11-22 16:19:37 -080080
Colin Cross31656952018-05-24 16:11:20 -070081 var expectedLinkImplicits []string
82
83 manifestFixer := foo.Output("manifest_fixer/AndroidManifest.xml")
84 expectedLinkImplicits = append(expectedLinkImplicits, manifestFixer.Output.String())
Colin Cross3bc7ffa2017-11-22 16:19:37 -080085
Colin Crossa97c5d32018-03-28 14:58:31 -070086 frameworkRes := ctx.ModuleForTests("framework-res", "android_common")
87 expectedLinkImplicits = append(expectedLinkImplicits,
88 frameworkRes.Output("package-res.apk").Output.String())
Colin Cross3bc7ffa2017-11-22 16:19:37 -080089
Colin Crossa97c5d32018-03-28 14:58:31 -070090 // Test the mapping from input files to compiled output file names
91 compile := foo.Output(compiledResourceFiles[0])
92 if !reflect.DeepEqual(resourceFiles, compile.Inputs.Strings()) {
93 t.Errorf("expected aapt2 compile inputs expected:\n %#v\n got:\n %#v",
94 resourceFiles, compile.Inputs.Strings())
95 }
Colin Crossb69301e2017-12-01 10:48:26 -080096
Colin Crossa97c5d32018-03-28 14:58:31 -070097 compiledResourceOutputs := compile.Outputs.Strings()
98 sort.Strings(compiledResourceOutputs)
Colin Crossb69301e2017-12-01 10:48:26 -080099
Colin Crossa97c5d32018-03-28 14:58:31 -0700100 expectedLinkImplicits = append(expectedLinkImplicits, compiledResourceOutputs...)
Colin Cross3bc7ffa2017-11-22 16:19:37 -0800101
Colin Crossa97c5d32018-03-28 14:58:31 -0700102 list := foo.Output("aapt2/res.list")
103 expectedLinkImplicits = append(expectedLinkImplicits, list.Output.String())
Colin Cross3bc7ffa2017-11-22 16:19:37 -0800104
Colin Crossa97c5d32018-03-28 14:58:31 -0700105 // Check that the link rule uses
106 res := ctx.ModuleForTests("foo", "android_common").Output("package-res.apk")
107 if !reflect.DeepEqual(expectedLinkImplicits, res.Implicits.Strings()) {
108 t.Errorf("expected aapt2 link implicits expected:\n %#v\n got:\n %#v",
109 expectedLinkImplicits, res.Implicits.Strings())
110 }
111 })
Colin Cross3bc7ffa2017-11-22 16:19:37 -0800112 }
113}
Colin Cross890ff552017-11-30 20:13:19 -0800114
Colin Crosse560c4a2019-03-19 16:03:11 -0700115func TestAppSplits(t *testing.T) {
116 ctx := testApp(t, `
117 android_app {
118 name: "foo",
119 srcs: ["a.java"],
120 package_splits: ["v4", "v7,hdpi"],
Jeongik Cha538c0d02019-07-11 15:54:27 +0900121 sdk_version: "current"
Colin Crosse560c4a2019-03-19 16:03:11 -0700122 }`)
123
124 foo := ctx.ModuleForTests("foo", "android_common")
125
126 expectedOutputs := []string{
127 filepath.Join(buildDir, ".intermediates/foo/android_common/foo.apk"),
128 filepath.Join(buildDir, ".intermediates/foo/android_common/foo_v4.apk"),
129 filepath.Join(buildDir, ".intermediates/foo/android_common/foo_v7_hdpi.apk"),
130 }
131 for _, expectedOutput := range expectedOutputs {
132 foo.Output(expectedOutput)
133 }
134
Colin Cross41955e82019-05-29 14:40:35 -0700135 outputFiles, err := foo.Module().(*AndroidApp).OutputFiles("")
136 if err != nil {
137 t.Fatal(err)
138 }
139 if g, w := outputFiles.Strings(), expectedOutputs; !reflect.DeepEqual(g, w) {
140 t.Errorf(`want OutputFiles("") = %q, got %q`, w, g)
Colin Crosse560c4a2019-03-19 16:03:11 -0700141 }
142}
143
Jeongik Cha538c0d02019-07-11 15:54:27 +0900144func TestPlatformAPIs(t *testing.T) {
145 testJava(t, `
146 android_app {
147 name: "foo",
148 srcs: ["a.java"],
149 platform_apis: true,
150 }
151 `)
152
153 testJava(t, `
154 android_app {
155 name: "foo",
156 srcs: ["a.java"],
157 sdk_version: "current",
158 }
159 `)
160
161 testJavaError(t, "platform_apis must be true when sdk_version is empty.", `
162 android_app {
163 name: "bar",
164 srcs: ["b.java"],
165 }
166 `)
167
168 testJavaError(t, "platform_apis must be false when sdk_version is not empty.", `
169 android_app {
170 name: "bar",
171 srcs: ["b.java"],
172 sdk_version: "system_current",
173 platform_apis: true,
174 }
175 `)
176}
177
Jeongik Chae403e9e2019-12-07 00:16:24 +0900178func TestAndroidAppLinkType(t *testing.T) {
179 testJava(t, `
180 android_app {
181 name: "foo",
182 srcs: ["a.java"],
183 libs: ["bar"],
184 static_libs: ["baz"],
185 platform_apis: true,
186 }
187
188 java_library {
189 name: "bar",
190 sdk_version: "current",
191 srcs: ["b.java"],
192 }
193
194 android_library {
195 name: "baz",
196 sdk_version: "system_current",
197 srcs: ["c.java"],
198 }
199 `)
200
201 testJavaError(t, "Adjust sdk_version: property of the source or target module so that target module is built with the same or smaller API set than the source.", `
202 android_app {
203 name: "foo",
204 srcs: ["a.java"],
205 libs: ["bar"],
206 sdk_version: "current",
207 static_libs: ["baz"],
208 }
209
210 java_library {
211 name: "bar",
212 sdk_version: "current",
213 srcs: ["b.java"],
214 }
215
216 android_library {
217 name: "baz",
218 sdk_version: "system_current",
219 srcs: ["c.java"],
220 }
221 `)
222
223 testJava(t, `
224 android_app {
225 name: "foo",
226 srcs: ["a.java"],
227 libs: ["bar"],
228 sdk_version: "system_current",
229 static_libs: ["baz"],
230 }
231
232 java_library {
233 name: "bar",
234 sdk_version: "current",
235 srcs: ["b.java"],
236 }
237
238 android_library {
239 name: "baz",
240 sdk_version: "system_current",
241 srcs: ["c.java"],
242 }
243 `)
244
245 testJavaError(t, "Adjust sdk_version: property of the source or target module so that target module is built with the same or smaller API set than the source.", `
246 android_app {
247 name: "foo",
248 srcs: ["a.java"],
249 libs: ["bar"],
250 sdk_version: "system_current",
251 static_libs: ["baz"],
252 }
253
254 java_library {
255 name: "bar",
256 sdk_version: "current",
257 srcs: ["b.java"],
258 }
259
260 android_library {
261 name: "baz",
262 srcs: ["c.java"],
263 }
264 `)
265}
266
Colin Cross0ddae7f2019-02-07 15:30:01 -0800267func TestResourceDirs(t *testing.T) {
268 testCases := []struct {
269 name string
270 prop string
271 resources []string
272 }{
273 {
274 name: "no resource_dirs",
275 prop: "",
276 resources: []string{"res/res/values/strings.xml"},
277 },
278 {
279 name: "resource_dirs",
280 prop: `resource_dirs: ["res"]`,
281 resources: []string{"res/res/values/strings.xml"},
282 },
283 {
284 name: "empty resource_dirs",
285 prop: `resource_dirs: []`,
286 resources: nil,
287 },
288 }
289
290 fs := map[string][]byte{
291 "res/res/values/strings.xml": nil,
292 }
293
294 bp := `
295 android_app {
296 name: "foo",
Jeongik Cha538c0d02019-07-11 15:54:27 +0900297 sdk_version: "current",
Colin Cross0ddae7f2019-02-07 15:30:01 -0800298 %s
299 }
300 `
301
302 for _, testCase := range testCases {
303 t.Run(testCase.name, func(t *testing.T) {
Colin Cross98be1bb2019-12-13 20:41:13 -0800304 config := testConfig(nil, fmt.Sprintf(bp, testCase.prop), fs)
305 ctx := testContext()
Colin Cross0ddae7f2019-02-07 15:30:01 -0800306 run(t, ctx, config)
307
308 module := ctx.ModuleForTests("foo", "android_common")
309 resourceList := module.MaybeOutput("aapt2/res.list")
310
311 var resources []string
312 if resourceList.Rule != nil {
313 for _, compiledResource := range resourceList.Inputs.Strings() {
314 resources = append(resources, module.Output(compiledResource).Inputs.Strings()...)
315 }
316 }
317
318 if !reflect.DeepEqual(resources, testCase.resources) {
319 t.Errorf("expected resource files %q, got %q",
320 testCase.resources, resources)
321 }
322 })
323 }
324}
325
Jaewoong Jung6431ca72020-01-15 14:15:10 -0800326func TestLibraryAssets(t *testing.T) {
327 bp := `
328 android_app {
329 name: "foo",
330 sdk_version: "current",
331 static_libs: ["lib1", "lib2", "lib3"],
332 }
333
334 android_library {
335 name: "lib1",
336 sdk_version: "current",
337 asset_dirs: ["assets_a"],
338 }
339
340 android_library {
341 name: "lib2",
342 sdk_version: "current",
343 }
344
345 android_library {
346 name: "lib3",
347 sdk_version: "current",
348 static_libs: ["lib4"],
349 }
350
351 android_library {
352 name: "lib4",
353 sdk_version: "current",
354 asset_dirs: ["assets_b"],
355 }
356 `
357
358 testCases := []struct {
359 name string
360 assetFlag string
361 assetPackages []string
362 }{
363 {
364 name: "foo",
365 // lib1 has its own asset. lib3 doesn't have any, but provides lib4's transitively.
366 assetPackages: []string{
367 buildDir + "/.intermediates/foo/android_common/aapt2/package-res.apk",
368 buildDir + "/.intermediates/lib1/android_common/assets.zip",
369 buildDir + "/.intermediates/lib3/android_common/assets.zip",
370 },
371 },
372 {
373 name: "lib1",
374 assetFlag: "-A assets_a",
375 },
376 {
377 name: "lib2",
378 },
379 {
380 name: "lib3",
381 assetPackages: []string{
382 buildDir + "/.intermediates/lib3/android_common/aapt2/package-res.apk",
383 buildDir + "/.intermediates/lib4/android_common/assets.zip",
384 },
385 },
386 {
387 name: "lib4",
388 assetFlag: "-A assets_b",
389 },
390 }
391 ctx := testApp(t, bp)
392
393 for _, test := range testCases {
394 t.Run(test.name, func(t *testing.T) {
395 m := ctx.ModuleForTests(test.name, "android_common")
396
397 // Check asset flag in aapt2 link flags
398 var aapt2link android.TestingBuildParams
399 if len(test.assetPackages) > 0 {
400 aapt2link = m.Output("aapt2/package-res.apk")
401 } else {
402 aapt2link = m.Output("package-res.apk")
403 }
404 aapt2Flags := aapt2link.Args["flags"]
405 if test.assetFlag != "" {
406 if !strings.Contains(aapt2Flags, test.assetFlag) {
407 t.Errorf("Can't find asset flag %q in aapt2 link flags %q", test.assetFlag, aapt2Flags)
408 }
409 } else {
410 if strings.Contains(aapt2Flags, " -A ") {
411 t.Errorf("aapt2 link flags %q contain unexpected asset flag", aapt2Flags)
412 }
413 }
414
415 // Check asset merge rule.
416 if len(test.assetPackages) > 0 {
417 mergeAssets := m.Output("package-res.apk")
418 if !reflect.DeepEqual(test.assetPackages, mergeAssets.Inputs.Strings()) {
419 t.Errorf("Unexpected mergeAssets inputs: %v, expected: %v",
420 mergeAssets.Inputs.Strings(), test.assetPackages)
421 }
422 }
423 })
424 }
425}
426
Colin Crossbec85302019-02-13 13:15:46 -0800427func TestAndroidResources(t *testing.T) {
Colin Cross5c4791c2019-02-01 11:44:44 -0800428 testCases := []struct {
429 name string
430 enforceRROTargets []string
431 enforceRROExcludedOverlays []string
Colin Crossbec85302019-02-13 13:15:46 -0800432 resourceFiles map[string][]string
Colin Cross5c4791c2019-02-01 11:44:44 -0800433 overlayFiles map[string][]string
434 rroDirs map[string][]string
435 }{
436 {
437 name: "no RRO",
438 enforceRROTargets: nil,
439 enforceRROExcludedOverlays: nil,
Colin Crossbec85302019-02-13 13:15:46 -0800440 resourceFiles: map[string][]string{
441 "foo": nil,
442 "bar": {"bar/res/res/values/strings.xml"},
443 "lib": nil,
444 "lib2": {"lib2/res/res/values/strings.xml"},
445 },
Colin Cross5c4791c2019-02-01 11:44:44 -0800446 overlayFiles: map[string][]string{
Colin Crossbec85302019-02-13 13:15:46 -0800447 "foo": {
448 buildDir + "/.intermediates/lib2/android_common/package-res.apk",
Colin Cross6ed7dea2019-01-31 14:44:30 -0800449 buildDir + "/.intermediates/lib/android_common/package-res.apk",
Anton Hansson53c88442019-03-18 15:53:16 +0000450 buildDir + "/.intermediates/lib3/android_common/package-res.apk",
Colin Cross6ed7dea2019-01-31 14:44:30 -0800451 "foo/res/res/values/strings.xml",
Colin Cross5c4791c2019-02-01 11:44:44 -0800452 "device/vendor/blah/static_overlay/foo/res/values/strings.xml",
453 "device/vendor/blah/overlay/foo/res/values/strings.xml",
Anton Hansson53c88442019-03-18 15:53:16 +0000454 "product/vendor/blah/overlay/foo/res/values/strings.xml",
Colin Cross5c4791c2019-02-01 11:44:44 -0800455 },
Colin Crossbec85302019-02-13 13:15:46 -0800456 "bar": {
Colin Cross5c4791c2019-02-01 11:44:44 -0800457 "device/vendor/blah/static_overlay/bar/res/values/strings.xml",
458 "device/vendor/blah/overlay/bar/res/values/strings.xml",
459 },
Colin Crossbec85302019-02-13 13:15:46 -0800460 "lib": {
461 buildDir + "/.intermediates/lib2/android_common/package-res.apk",
462 "lib/res/res/values/strings.xml",
463 "device/vendor/blah/overlay/lib/res/values/strings.xml",
464 },
Colin Cross5c4791c2019-02-01 11:44:44 -0800465 },
466 rroDirs: map[string][]string{
467 "foo": nil,
468 "bar": nil,
469 },
470 },
471 {
472 name: "enforce RRO on foo",
473 enforceRROTargets: []string{"foo"},
474 enforceRROExcludedOverlays: []string{"device/vendor/blah/static_overlay"},
Colin Crossbec85302019-02-13 13:15:46 -0800475 resourceFiles: map[string][]string{
476 "foo": nil,
477 "bar": {"bar/res/res/values/strings.xml"},
478 "lib": nil,
479 "lib2": {"lib2/res/res/values/strings.xml"},
480 },
Colin Cross5c4791c2019-02-01 11:44:44 -0800481 overlayFiles: map[string][]string{
Colin Crossbec85302019-02-13 13:15:46 -0800482 "foo": {
483 buildDir + "/.intermediates/lib2/android_common/package-res.apk",
Colin Cross6ed7dea2019-01-31 14:44:30 -0800484 buildDir + "/.intermediates/lib/android_common/package-res.apk",
Anton Hansson53c88442019-03-18 15:53:16 +0000485 buildDir + "/.intermediates/lib3/android_common/package-res.apk",
Colin Cross6ed7dea2019-01-31 14:44:30 -0800486 "foo/res/res/values/strings.xml",
487 "device/vendor/blah/static_overlay/foo/res/values/strings.xml",
488 },
Colin Crossbec85302019-02-13 13:15:46 -0800489 "bar": {
Colin Cross5c4791c2019-02-01 11:44:44 -0800490 "device/vendor/blah/static_overlay/bar/res/values/strings.xml",
491 "device/vendor/blah/overlay/bar/res/values/strings.xml",
492 },
Colin Crossbec85302019-02-13 13:15:46 -0800493 "lib": {
494 buildDir + "/.intermediates/lib2/android_common/package-res.apk",
495 "lib/res/res/values/strings.xml",
496 "device/vendor/blah/overlay/lib/res/values/strings.xml",
497 },
Colin Cross5c4791c2019-02-01 11:44:44 -0800498 },
Colin Crossc1c37552019-01-31 11:42:41 -0800499
Colin Cross5c4791c2019-02-01 11:44:44 -0800500 rroDirs: map[string][]string{
Colin Crossbec85302019-02-13 13:15:46 -0800501 "foo": {
Anton Hansson53c88442019-03-18 15:53:16 +0000502 "device:device/vendor/blah/overlay/foo/res",
Colin Crossc1c37552019-01-31 11:42:41 -0800503 // Enforce RRO on "foo" could imply RRO on static dependencies, but for now it doesn't.
504 // "device/vendor/blah/overlay/lib/res",
Anton Hansson53c88442019-03-18 15:53:16 +0000505 "product:product/vendor/blah/overlay/foo/res",
Colin Crossc1c37552019-01-31 11:42:41 -0800506 },
Colin Cross5c4791c2019-02-01 11:44:44 -0800507 "bar": nil,
Colin Crossbec85302019-02-13 13:15:46 -0800508 "lib": nil,
Colin Cross5c4791c2019-02-01 11:44:44 -0800509 },
510 },
511 {
512 name: "enforce RRO on all",
513 enforceRROTargets: []string{"*"},
514 enforceRROExcludedOverlays: []string{
515 // Excluding specific apps/res directories also allowed.
516 "device/vendor/blah/static_overlay/foo",
517 "device/vendor/blah/static_overlay/bar/res",
518 },
Colin Crossbec85302019-02-13 13:15:46 -0800519 resourceFiles: map[string][]string{
520 "foo": nil,
521 "bar": {"bar/res/res/values/strings.xml"},
522 "lib": nil,
523 "lib2": {"lib2/res/res/values/strings.xml"},
524 },
Colin Cross5c4791c2019-02-01 11:44:44 -0800525 overlayFiles: map[string][]string{
Colin Crossbec85302019-02-13 13:15:46 -0800526 "foo": {
527 buildDir + "/.intermediates/lib2/android_common/package-res.apk",
Colin Cross6ed7dea2019-01-31 14:44:30 -0800528 buildDir + "/.intermediates/lib/android_common/package-res.apk",
Anton Hansson53c88442019-03-18 15:53:16 +0000529 buildDir + "/.intermediates/lib3/android_common/package-res.apk",
Colin Cross6ed7dea2019-01-31 14:44:30 -0800530 "foo/res/res/values/strings.xml",
531 "device/vendor/blah/static_overlay/foo/res/values/strings.xml",
532 },
Colin Crossbec85302019-02-13 13:15:46 -0800533 "bar": {"device/vendor/blah/static_overlay/bar/res/values/strings.xml"},
534 "lib": {
535 buildDir + "/.intermediates/lib2/android_common/package-res.apk",
536 "lib/res/res/values/strings.xml",
537 },
Colin Cross5c4791c2019-02-01 11:44:44 -0800538 },
539 rroDirs: map[string][]string{
Colin Crossbec85302019-02-13 13:15:46 -0800540 "foo": {
Anton Hansson53c88442019-03-18 15:53:16 +0000541 "device:device/vendor/blah/overlay/foo/res",
542 "product:product/vendor/blah/overlay/foo/res",
543 // Lib dep comes after the direct deps
544 "device:device/vendor/blah/overlay/lib/res",
Colin Crossc1c37552019-01-31 11:42:41 -0800545 },
Anton Hansson53c88442019-03-18 15:53:16 +0000546 "bar": {"device:device/vendor/blah/overlay/bar/res"},
547 "lib": {"device:device/vendor/blah/overlay/lib/res"},
Colin Cross5c4791c2019-02-01 11:44:44 -0800548 },
549 },
550 }
551
Anton Hansson53c88442019-03-18 15:53:16 +0000552 deviceResourceOverlays := []string{
Colin Cross890ff552017-11-30 20:13:19 -0800553 "device/vendor/blah/overlay",
554 "device/vendor/blah/overlay2",
555 "device/vendor/blah/static_overlay",
556 }
557
Anton Hansson53c88442019-03-18 15:53:16 +0000558 productResourceOverlays := []string{
559 "product/vendor/blah/overlay",
560 }
561
Colin Cross890ff552017-11-30 20:13:19 -0800562 fs := map[string][]byte{
563 "foo/res/res/values/strings.xml": nil,
564 "bar/res/res/values/strings.xml": nil,
Colin Cross6ed7dea2019-01-31 14:44:30 -0800565 "lib/res/res/values/strings.xml": nil,
Colin Crossbec85302019-02-13 13:15:46 -0800566 "lib2/res/res/values/strings.xml": nil,
Colin Cross890ff552017-11-30 20:13:19 -0800567 "device/vendor/blah/overlay/foo/res/values/strings.xml": nil,
568 "device/vendor/blah/overlay/bar/res/values/strings.xml": nil,
Colin Cross6ed7dea2019-01-31 14:44:30 -0800569 "device/vendor/blah/overlay/lib/res/values/strings.xml": nil,
Colin Cross890ff552017-11-30 20:13:19 -0800570 "device/vendor/blah/static_overlay/foo/res/values/strings.xml": nil,
571 "device/vendor/blah/static_overlay/bar/res/values/strings.xml": nil,
572 "device/vendor/blah/overlay2/res/values/strings.xml": nil,
Anton Hansson53c88442019-03-18 15:53:16 +0000573 "product/vendor/blah/overlay/foo/res/values/strings.xml": nil,
Colin Cross890ff552017-11-30 20:13:19 -0800574 }
575
576 bp := `
577 android_app {
578 name: "foo",
Jeongik Cha538c0d02019-07-11 15:54:27 +0900579 sdk_version: "current",
Colin Cross890ff552017-11-30 20:13:19 -0800580 resource_dirs: ["foo/res"],
Anton Hansson53c88442019-03-18 15:53:16 +0000581 static_libs: ["lib", "lib3"],
Colin Cross890ff552017-11-30 20:13:19 -0800582 }
583
584 android_app {
585 name: "bar",
Jeongik Cha538c0d02019-07-11 15:54:27 +0900586 sdk_version: "current",
Colin Cross890ff552017-11-30 20:13:19 -0800587 resource_dirs: ["bar/res"],
588 }
Colin Cross6ed7dea2019-01-31 14:44:30 -0800589
590 android_library {
591 name: "lib",
Jeongik Cha75b83b02019-11-01 15:28:00 +0900592 sdk_version: "current",
Colin Cross6ed7dea2019-01-31 14:44:30 -0800593 resource_dirs: ["lib/res"],
Colin Crossbec85302019-02-13 13:15:46 -0800594 static_libs: ["lib2"],
595 }
596
597 android_library {
598 name: "lib2",
Jeongik Cha75b83b02019-11-01 15:28:00 +0900599 sdk_version: "current",
Colin Crossbec85302019-02-13 13:15:46 -0800600 resource_dirs: ["lib2/res"],
Colin Cross6ed7dea2019-01-31 14:44:30 -0800601 }
Anton Hansson53c88442019-03-18 15:53:16 +0000602
603 // This library has the same resources as lib (should not lead to dupe RROs)
604 android_library {
605 name: "lib3",
Jeongik Cha75b83b02019-11-01 15:28:00 +0900606 sdk_version: "current",
Anton Hansson53c88442019-03-18 15:53:16 +0000607 resource_dirs: ["lib/res"]
608 }
Colin Cross890ff552017-11-30 20:13:19 -0800609 `
610
Colin Cross5c4791c2019-02-01 11:44:44 -0800611 for _, testCase := range testCases {
Colin Cross890ff552017-11-30 20:13:19 -0800612 t.Run(testCase.name, func(t *testing.T) {
Colin Cross98be1bb2019-12-13 20:41:13 -0800613 config := testAppConfig(nil, bp, fs)
Anton Hansson53c88442019-03-18 15:53:16 +0000614 config.TestProductVariables.DeviceResourceOverlays = deviceResourceOverlays
615 config.TestProductVariables.ProductResourceOverlays = productResourceOverlays
Colin Cross890ff552017-11-30 20:13:19 -0800616 if testCase.enforceRROTargets != nil {
Colin Crossa74ca042019-01-31 14:31:51 -0800617 config.TestProductVariables.EnforceRROTargets = testCase.enforceRROTargets
Colin Cross890ff552017-11-30 20:13:19 -0800618 }
619 if testCase.enforceRROExcludedOverlays != nil {
Colin Crossa74ca042019-01-31 14:31:51 -0800620 config.TestProductVariables.EnforceRROExcludedOverlays = testCase.enforceRROExcludedOverlays
Colin Cross890ff552017-11-30 20:13:19 -0800621 }
622
Colin Cross98be1bb2019-12-13 20:41:13 -0800623 ctx := testContext()
Colin Cross890ff552017-11-30 20:13:19 -0800624 run(t, ctx, config)
625
Colin Crossbec85302019-02-13 13:15:46 -0800626 resourceListToFiles := func(module android.TestingModule, list []string) (files []string) {
627 for _, o := range list {
628 res := module.MaybeOutput(o)
629 if res.Rule != nil {
630 // If the overlay is compiled as part of this module (i.e. a .arsc.flat file),
631 // verify the inputs to the .arsc.flat rule.
632 files = append(files, res.Inputs.Strings()...)
633 } else {
634 // Otherwise, verify the full path to the output of the other module
635 files = append(files, o)
Anton Hansson94c93f32019-01-30 16:03:37 +0000636 }
Colin Cross890ff552017-11-30 20:13:19 -0800637 }
Colin Crossbec85302019-02-13 13:15:46 -0800638 return files
Colin Cross890ff552017-11-30 20:13:19 -0800639 }
640
Colin Crossbec85302019-02-13 13:15:46 -0800641 getResources := func(moduleName string) (resourceFiles, overlayFiles, rroDirs []string) {
642 module := ctx.ModuleForTests(moduleName, "android_common")
643 resourceList := module.MaybeOutput("aapt2/res.list")
644 if resourceList.Rule != nil {
645 resourceFiles = resourceListToFiles(module, resourceList.Inputs.Strings())
Anton Hansson0375a4f2019-01-24 14:39:19 +0000646 }
Colin Crossbec85302019-02-13 13:15:46 -0800647 overlayList := module.MaybeOutput("aapt2/overlay.list")
648 if overlayList.Rule != nil {
649 overlayFiles = resourceListToFiles(module, overlayList.Inputs.Strings())
650 }
651
Anton Hansson53c88442019-03-18 15:53:16 +0000652 for _, d := range module.Module().(AndroidLibraryDependency).ExportedRRODirs() {
653 var prefix string
654 if d.overlayType == device {
655 prefix = "device:"
656 } else if d.overlayType == product {
657 prefix = "product:"
658 } else {
659 t.Fatalf("Unexpected overlayType %d", d.overlayType)
660 }
661 rroDirs = append(rroDirs, prefix+d.path.String())
662 }
Colin Crossbec85302019-02-13 13:15:46 -0800663
664 return resourceFiles, overlayFiles, rroDirs
665 }
666
667 modules := []string{"foo", "bar", "lib", "lib2"}
668 for _, module := range modules {
669 resourceFiles, overlayFiles, rroDirs := getResources(module)
670
671 if !reflect.DeepEqual(resourceFiles, testCase.resourceFiles[module]) {
672 t.Errorf("expected %s resource files:\n %#v\n got:\n %#v",
673 module, testCase.resourceFiles[module], resourceFiles)
674 }
675 if !reflect.DeepEqual(overlayFiles, testCase.overlayFiles[module]) {
676 t.Errorf("expected %s overlay files:\n %#v\n got:\n %#v",
677 module, testCase.overlayFiles[module], overlayFiles)
678 }
679 if !reflect.DeepEqual(rroDirs, testCase.rroDirs[module]) {
Anton Hansson0375a4f2019-01-24 14:39:19 +0000680 t.Errorf("expected %s rroDirs: %#v\n got:\n %#v",
Colin Crossbec85302019-02-13 13:15:46 -0800681 module, testCase.rroDirs[module], rroDirs)
Anton Hansson0375a4f2019-01-24 14:39:19 +0000682 }
Colin Cross890ff552017-11-30 20:13:19 -0800683 }
Colin Cross890ff552017-11-30 20:13:19 -0800684 })
685 }
686}
Colin Crossd09b0b62018-04-18 11:06:47 -0700687
688func TestAppSdkVersion(t *testing.T) {
689 testCases := []struct {
690 name string
691 sdkVersion string
692 platformSdkInt int
693 platformSdkCodename string
694 platformSdkFinal bool
695 expectedMinSdkVersion string
Jeongik Cha538c0d02019-07-11 15:54:27 +0900696 platformApis bool
Colin Crossd09b0b62018-04-18 11:06:47 -0700697 }{
698 {
699 name: "current final SDK",
700 sdkVersion: "current",
701 platformSdkInt: 27,
702 platformSdkCodename: "REL",
703 platformSdkFinal: true,
704 expectedMinSdkVersion: "27",
705 },
706 {
707 name: "current non-final SDK",
708 sdkVersion: "current",
709 platformSdkInt: 27,
710 platformSdkCodename: "OMR1",
711 platformSdkFinal: false,
712 expectedMinSdkVersion: "OMR1",
713 },
714 {
715 name: "default final SDK",
716 sdkVersion: "",
Jeongik Cha538c0d02019-07-11 15:54:27 +0900717 platformApis: true,
Colin Crossd09b0b62018-04-18 11:06:47 -0700718 platformSdkInt: 27,
719 platformSdkCodename: "REL",
720 platformSdkFinal: true,
721 expectedMinSdkVersion: "27",
722 },
723 {
724 name: "default non-final SDK",
725 sdkVersion: "",
Jeongik Cha538c0d02019-07-11 15:54:27 +0900726 platformApis: true,
Colin Crossd09b0b62018-04-18 11:06:47 -0700727 platformSdkInt: 27,
728 platformSdkCodename: "OMR1",
729 platformSdkFinal: false,
730 expectedMinSdkVersion: "OMR1",
731 },
732 {
733 name: "14",
734 sdkVersion: "14",
735 expectedMinSdkVersion: "14",
736 },
737 }
738
739 for _, moduleType := range []string{"android_app", "android_library"} {
740 for _, test := range testCases {
741 t.Run(moduleType+" "+test.name, func(t *testing.T) {
Jeongik Cha538c0d02019-07-11 15:54:27 +0900742 platformApiProp := ""
743 if test.platformApis {
744 platformApiProp = "platform_apis: true,"
745 }
Colin Crossd09b0b62018-04-18 11:06:47 -0700746 bp := fmt.Sprintf(`%s {
747 name: "foo",
748 srcs: ["a.java"],
749 sdk_version: "%s",
Jeongik Cha538c0d02019-07-11 15:54:27 +0900750 %s
751 }`, moduleType, test.sdkVersion, platformApiProp)
Colin Crossd09b0b62018-04-18 11:06:47 -0700752
Colin Cross98be1bb2019-12-13 20:41:13 -0800753 config := testAppConfig(nil, bp, nil)
Colin Crossd09b0b62018-04-18 11:06:47 -0700754 config.TestProductVariables.Platform_sdk_version = &test.platformSdkInt
755 config.TestProductVariables.Platform_sdk_codename = &test.platformSdkCodename
756 config.TestProductVariables.Platform_sdk_final = &test.platformSdkFinal
757
Colin Cross98be1bb2019-12-13 20:41:13 -0800758 ctx := testContext()
Colin Crossd09b0b62018-04-18 11:06:47 -0700759
760 run(t, ctx, config)
761
762 foo := ctx.ModuleForTests("foo", "android_common")
763 link := foo.Output("package-res.apk")
764 linkFlags := strings.Split(link.Args["flags"], " ")
765 min := android.IndexList("--min-sdk-version", linkFlags)
766 target := android.IndexList("--target-sdk-version", linkFlags)
767
768 if min == -1 || target == -1 || min == len(linkFlags)-1 || target == len(linkFlags)-1 {
769 t.Fatalf("missing --min-sdk-version or --target-sdk-version in link flags: %q", linkFlags)
770 }
771
772 gotMinSdkVersion := linkFlags[min+1]
773 gotTargetSdkVersion := linkFlags[target+1]
774
775 if gotMinSdkVersion != test.expectedMinSdkVersion {
776 t.Errorf("incorrect --min-sdk-version, expected %q got %q",
777 test.expectedMinSdkVersion, gotMinSdkVersion)
778 }
779
780 if gotTargetSdkVersion != test.expectedMinSdkVersion {
781 t.Errorf("incorrect --target-sdk-version, expected %q got %q",
782 test.expectedMinSdkVersion, gotTargetSdkVersion)
783 }
784 })
785 }
786 }
787}
Colin Crossa4f08812018-10-02 22:03:40 -0700788
Paul Duffin50c217c2019-06-12 13:25:22 +0100789func TestJNIABI(t *testing.T) {
Jaewoong Jungf9a04432019-07-17 11:15:09 -0700790 ctx, _ := testJava(t, cc.GatherRequiredDepsForTest(android.Android)+`
Paul Duffin50c217c2019-06-12 13:25:22 +0100791 cc_library {
792 name: "libjni",
793 system_shared_libs: [],
794 stl: "none",
795 }
796
797 android_test {
798 name: "test",
799 sdk_version: "core_platform",
800 jni_libs: ["libjni"],
801 }
802
803 android_test {
804 name: "test_first",
805 sdk_version: "core_platform",
806 compile_multilib: "first",
807 jni_libs: ["libjni"],
808 }
809
810 android_test {
811 name: "test_both",
812 sdk_version: "core_platform",
813 compile_multilib: "both",
814 jni_libs: ["libjni"],
815 }
816
817 android_test {
818 name: "test_32",
819 sdk_version: "core_platform",
820 compile_multilib: "32",
821 jni_libs: ["libjni"],
822 }
823
824 android_test {
825 name: "test_64",
826 sdk_version: "core_platform",
827 compile_multilib: "64",
828 jni_libs: ["libjni"],
829 }
830 `)
831
832 testCases := []struct {
833 name string
834 abis []string
835 }{
836 {"test", []string{"arm64-v8a"}},
837 {"test_first", []string{"arm64-v8a"}},
838 {"test_both", []string{"arm64-v8a", "armeabi-v7a"}},
839 {"test_32", []string{"armeabi-v7a"}},
840 {"test_64", []string{"arm64-v8a"}},
841 }
842
843 for _, test := range testCases {
844 t.Run(test.name, func(t *testing.T) {
845 app := ctx.ModuleForTests(test.name, "android_common")
846 jniLibZip := app.Output("jnilibs.zip")
847 var abis []string
848 args := strings.Fields(jniLibZip.Args["jarArgs"])
849 for i := 0; i < len(args); i++ {
850 if args[i] == "-P" {
851 abis = append(abis, filepath.Base(args[i+1]))
852 i++
853 }
854 }
855 if !reflect.DeepEqual(abis, test.abis) {
856 t.Errorf("want abis %v, got %v", test.abis, abis)
857 }
858 })
859 }
860}
861
Jeongik Cha2cc570d2019-10-29 15:44:45 +0900862func TestAppSdkVersionByPartition(t *testing.T) {
863 testJavaError(t, "sdk_version must have a value when the module is located at vendor or product", `
864 android_app {
865 name: "foo",
866 srcs: ["a.java"],
867 vendor: true,
868 platform_apis: true,
869 }
870 `)
871
872 testJava(t, `
873 android_app {
874 name: "bar",
875 srcs: ["b.java"],
876 platform_apis: true,
877 }
878 `)
879
880 for _, enforce := range []bool{true, false} {
Jeongik Cha2cc570d2019-10-29 15:44:45 +0900881 bp := `
882 android_app {
883 name: "foo",
884 srcs: ["a.java"],
885 product_specific: true,
886 platform_apis: true,
887 }
888 `
Colin Cross98be1bb2019-12-13 20:41:13 -0800889
890 config := testAppConfig(nil, bp, nil)
891 config.TestProductVariables.EnforceProductPartitionInterface = proptools.BoolPtr(enforce)
Jeongik Cha2cc570d2019-10-29 15:44:45 +0900892 if enforce {
Colin Cross98be1bb2019-12-13 20:41:13 -0800893 testJavaErrorWithConfig(t, "sdk_version must have a value when the module is located at vendor or product", config)
Jeongik Cha2cc570d2019-10-29 15:44:45 +0900894 } else {
Colin Cross98be1bb2019-12-13 20:41:13 -0800895 testJavaWithConfig(t, config)
Jeongik Cha2cc570d2019-10-29 15:44:45 +0900896 }
897 }
898}
899
Paul Duffin50c217c2019-06-12 13:25:22 +0100900func TestJNIPackaging(t *testing.T) {
Jaewoong Jungf9a04432019-07-17 11:15:09 -0700901 ctx, _ := testJava(t, cc.GatherRequiredDepsForTest(android.Android)+`
Paul Duffin50c217c2019-06-12 13:25:22 +0100902 cc_library {
903 name: "libjni",
904 system_shared_libs: [],
905 stl: "none",
906 }
907
908 android_app {
909 name: "app",
910 jni_libs: ["libjni"],
Jeongik Cha538c0d02019-07-11 15:54:27 +0900911 sdk_version: "current",
Paul Duffin50c217c2019-06-12 13:25:22 +0100912 }
913
914 android_app {
915 name: "app_noembed",
916 jni_libs: ["libjni"],
917 use_embedded_native_libs: false,
Jeongik Cha538c0d02019-07-11 15:54:27 +0900918 sdk_version: "current",
Paul Duffin50c217c2019-06-12 13:25:22 +0100919 }
920
921 android_app {
922 name: "app_embed",
923 jni_libs: ["libjni"],
924 use_embedded_native_libs: true,
Jeongik Cha538c0d02019-07-11 15:54:27 +0900925 sdk_version: "current",
Paul Duffin50c217c2019-06-12 13:25:22 +0100926 }
927
928 android_test {
929 name: "test",
930 sdk_version: "core_platform",
931 jni_libs: ["libjni"],
932 }
933
934 android_test {
935 name: "test_noembed",
936 sdk_version: "core_platform",
937 jni_libs: ["libjni"],
938 use_embedded_native_libs: false,
939 }
940
941 android_test_helper_app {
942 name: "test_helper",
943 sdk_version: "core_platform",
944 jni_libs: ["libjni"],
945 }
946
947 android_test_helper_app {
948 name: "test_helper_noembed",
949 sdk_version: "core_platform",
950 jni_libs: ["libjni"],
951 use_embedded_native_libs: false,
952 }
953 `)
954
955 testCases := []struct {
956 name string
957 packaged bool
958 compressed bool
959 }{
960 {"app", false, false},
961 {"app_noembed", false, false},
962 {"app_embed", true, false},
963 {"test", true, false},
964 {"test_noembed", true, true},
965 {"test_helper", true, false},
966 {"test_helper_noembed", true, true},
967 }
968
969 for _, test := range testCases {
970 t.Run(test.name, func(t *testing.T) {
971 app := ctx.ModuleForTests(test.name, "android_common")
972 jniLibZip := app.MaybeOutput("jnilibs.zip")
973 if g, w := (jniLibZip.Rule != nil), test.packaged; g != w {
974 t.Errorf("expected jni packaged %v, got %v", w, g)
975 }
976
977 if jniLibZip.Rule != nil {
978 if g, w := !strings.Contains(jniLibZip.Args["jarArgs"], "-L 0"), test.compressed; g != w {
979 t.Errorf("expected jni compressed %v, got %v", w, g)
980 }
981 }
982 })
983 }
Colin Cross47fa9d32019-03-26 10:51:39 -0700984}
985
Jaewoong Jung2ad817c2019-01-18 14:27:16 -0800986func TestCertificates(t *testing.T) {
987 testCases := []struct {
988 name string
989 bp string
990 certificateOverride string
991 expected string
992 }{
993 {
994 name: "default",
995 bp: `
996 android_app {
997 name: "foo",
998 srcs: ["a.java"],
Jeongik Cha538c0d02019-07-11 15:54:27 +0900999 sdk_version: "current",
Jaewoong Jung2ad817c2019-01-18 14:27:16 -08001000 }
1001 `,
1002 certificateOverride: "",
Dan Willemsen412160e2019-04-09 21:36:26 -07001003 expected: "build/make/target/product/security/testkey.x509.pem build/make/target/product/security/testkey.pk8",
Jaewoong Jung2ad817c2019-01-18 14:27:16 -08001004 },
1005 {
1006 name: "module certificate property",
1007 bp: `
1008 android_app {
1009 name: "foo",
1010 srcs: ["a.java"],
Jeongik Cha538c0d02019-07-11 15:54:27 +09001011 certificate: ":new_certificate",
1012 sdk_version: "current",
Jaewoong Jung2ad817c2019-01-18 14:27:16 -08001013 }
1014
1015 android_app_certificate {
1016 name: "new_certificate",
1017 certificate: "cert/new_cert",
1018 }
1019 `,
1020 certificateOverride: "",
1021 expected: "cert/new_cert.x509.pem cert/new_cert.pk8",
1022 },
1023 {
1024 name: "path certificate property",
1025 bp: `
1026 android_app {
1027 name: "foo",
1028 srcs: ["a.java"],
Jeongik Cha538c0d02019-07-11 15:54:27 +09001029 certificate: "expiredkey",
1030 sdk_version: "current",
Jaewoong Jung2ad817c2019-01-18 14:27:16 -08001031 }
1032 `,
1033 certificateOverride: "",
Dan Willemsen412160e2019-04-09 21:36:26 -07001034 expected: "build/make/target/product/security/expiredkey.x509.pem build/make/target/product/security/expiredkey.pk8",
Jaewoong Jung2ad817c2019-01-18 14:27:16 -08001035 },
1036 {
1037 name: "certificate overrides",
1038 bp: `
1039 android_app {
1040 name: "foo",
1041 srcs: ["a.java"],
Jeongik Cha538c0d02019-07-11 15:54:27 +09001042 certificate: "expiredkey",
1043 sdk_version: "current",
Jaewoong Jung2ad817c2019-01-18 14:27:16 -08001044 }
1045
1046 android_app_certificate {
1047 name: "new_certificate",
1048 certificate: "cert/new_cert",
1049 }
1050 `,
1051 certificateOverride: "foo:new_certificate",
1052 expected: "cert/new_cert.x509.pem cert/new_cert.pk8",
1053 },
1054 }
1055
1056 for _, test := range testCases {
1057 t.Run(test.name, func(t *testing.T) {
Colin Cross98be1bb2019-12-13 20:41:13 -08001058 config := testAppConfig(nil, test.bp, nil)
Jaewoong Jung2ad817c2019-01-18 14:27:16 -08001059 if test.certificateOverride != "" {
1060 config.TestProductVariables.CertificateOverrides = []string{test.certificateOverride}
1061 }
Colin Cross98be1bb2019-12-13 20:41:13 -08001062 ctx := testContext()
Jaewoong Jung2ad817c2019-01-18 14:27:16 -08001063
1064 run(t, ctx, config)
1065 foo := ctx.ModuleForTests("foo", "android_common")
1066
1067 signapk := foo.Output("foo.apk")
1068 signFlags := signapk.Args["certificates"]
1069 if test.expected != signFlags {
1070 t.Errorf("Incorrect signing flags, expected: %q, got: %q", test.expected, signFlags)
1071 }
1072 })
1073 }
1074}
Jaewoong Jung9d22a912019-01-23 16:27:47 -08001075
1076func TestPackageNameOverride(t *testing.T) {
1077 testCases := []struct {
1078 name string
1079 bp string
1080 packageNameOverride string
1081 expected []string
1082 }{
1083 {
1084 name: "default",
1085 bp: `
1086 android_app {
1087 name: "foo",
1088 srcs: ["a.java"],
Jeongik Cha538c0d02019-07-11 15:54:27 +09001089 sdk_version: "current",
Jaewoong Jung9d22a912019-01-23 16:27:47 -08001090 }
1091 `,
1092 packageNameOverride: "",
1093 expected: []string{
1094 buildDir + "/.intermediates/foo/android_common/foo.apk",
1095 buildDir + "/target/product/test_device/system/app/foo/foo.apk",
1096 },
1097 },
1098 {
1099 name: "overridden",
1100 bp: `
1101 android_app {
1102 name: "foo",
1103 srcs: ["a.java"],
Jeongik Cha538c0d02019-07-11 15:54:27 +09001104 sdk_version: "current",
Jaewoong Jung9d22a912019-01-23 16:27:47 -08001105 }
1106 `,
1107 packageNameOverride: "foo:bar",
1108 expected: []string{
1109 // The package apk should be still be the original name for test dependencies.
Jaewoong Jung5a498812019-11-07 14:14:38 -08001110 buildDir + "/.intermediates/foo/android_common/bar.apk",
Jaewoong Jung9d22a912019-01-23 16:27:47 -08001111 buildDir + "/target/product/test_device/system/app/bar/bar.apk",
1112 },
1113 },
1114 }
1115
1116 for _, test := range testCases {
1117 t.Run(test.name, func(t *testing.T) {
Colin Cross98be1bb2019-12-13 20:41:13 -08001118 config := testAppConfig(nil, test.bp, nil)
Jaewoong Jung9d22a912019-01-23 16:27:47 -08001119 if test.packageNameOverride != "" {
1120 config.TestProductVariables.PackageNameOverrides = []string{test.packageNameOverride}
1121 }
Colin Cross98be1bb2019-12-13 20:41:13 -08001122 ctx := testContext()
Jaewoong Jung9d22a912019-01-23 16:27:47 -08001123
1124 run(t, ctx, config)
1125 foo := ctx.ModuleForTests("foo", "android_common")
1126
1127 outputs := foo.AllOutputs()
1128 outputMap := make(map[string]bool)
1129 for _, o := range outputs {
1130 outputMap[o] = true
1131 }
1132 for _, e := range test.expected {
1133 if _, exist := outputMap[e]; !exist {
1134 t.Errorf("Can't find %q in output files.\nAll outputs:%v", e, outputs)
1135 }
1136 }
1137 })
1138 }
1139}
Jaewoong Jung4102e5d2019-02-27 16:26:28 -08001140
1141func TestInstrumentationTargetOverridden(t *testing.T) {
1142 bp := `
1143 android_app {
1144 name: "foo",
1145 srcs: ["a.java"],
Jeongik Cha538c0d02019-07-11 15:54:27 +09001146 sdk_version: "current",
Jaewoong Jung4102e5d2019-02-27 16:26:28 -08001147 }
1148
1149 android_test {
1150 name: "bar",
1151 instrumentation_for: "foo",
Jeongik Cha538c0d02019-07-11 15:54:27 +09001152 sdk_version: "current",
Jaewoong Jung4102e5d2019-02-27 16:26:28 -08001153 }
1154 `
Colin Cross98be1bb2019-12-13 20:41:13 -08001155 config := testAppConfig(nil, bp, nil)
Jaewoong Jung4102e5d2019-02-27 16:26:28 -08001156 config.TestProductVariables.ManifestPackageNameOverrides = []string{"foo:org.dandroid.bp"}
Colin Cross98be1bb2019-12-13 20:41:13 -08001157 ctx := testContext()
Jaewoong Jung4102e5d2019-02-27 16:26:28 -08001158
1159 run(t, ctx, config)
1160
1161 bar := ctx.ModuleForTests("bar", "android_common")
1162 res := bar.Output("package-res.apk")
1163 aapt2Flags := res.Args["flags"]
1164 e := "--rename-instrumentation-target-package org.dandroid.bp"
1165 if !strings.Contains(aapt2Flags, e) {
1166 t.Errorf("target package renaming flag, %q is missing in aapt2 link flags, %q", e, aapt2Flags)
1167 }
1168}
Jaewoong Jung525443a2019-02-28 15:35:54 -08001169
1170func TestOverrideAndroidApp(t *testing.T) {
Jaewoong Jungf9a04432019-07-17 11:15:09 -07001171 ctx, _ := testJava(t, `
Jaewoong Jung525443a2019-02-28 15:35:54 -08001172 android_app {
1173 name: "foo",
1174 srcs: ["a.java"],
Jaewoong Junga641ee92019-03-27 11:17:14 -07001175 certificate: "expiredkey",
Jaewoong Jungb639a6a2019-05-10 15:16:29 -07001176 overrides: ["qux"],
Jeongik Cha538c0d02019-07-11 15:54:27 +09001177 sdk_version: "current",
Jaewoong Jung525443a2019-02-28 15:35:54 -08001178 }
1179
1180 override_android_app {
1181 name: "bar",
1182 base: "foo",
1183 certificate: ":new_certificate",
1184 }
1185
1186 android_app_certificate {
1187 name: "new_certificate",
1188 certificate: "cert/new_cert",
1189 }
Jaewoong Jung6f373f62019-03-13 10:13:24 -07001190
1191 override_android_app {
1192 name: "baz",
1193 base: "foo",
1194 package_name: "org.dandroid.bp",
1195 }
Jaewoong Jung525443a2019-02-28 15:35:54 -08001196 `)
1197
1198 expectedVariants := []struct {
Jaewoong Jungb639a6a2019-05-10 15:16:29 -07001199 moduleName string
Jaewoong Jung525443a2019-02-28 15:35:54 -08001200 variantName string
1201 apkName string
1202 apkPath string
1203 signFlag string
1204 overrides []string
Jaewoong Jung6f373f62019-03-13 10:13:24 -07001205 aaptFlag string
Jaewoong Jung525443a2019-02-28 15:35:54 -08001206 }{
1207 {
Jaewoong Jungb639a6a2019-05-10 15:16:29 -07001208 moduleName: "foo",
Jaewoong Jung525443a2019-02-28 15:35:54 -08001209 variantName: "android_common",
1210 apkPath: "/target/product/test_device/system/app/foo/foo.apk",
Dan Willemsen412160e2019-04-09 21:36:26 -07001211 signFlag: "build/make/target/product/security/expiredkey.x509.pem build/make/target/product/security/expiredkey.pk8",
Jaewoong Jungb639a6a2019-05-10 15:16:29 -07001212 overrides: []string{"qux"},
Jaewoong Jung6f373f62019-03-13 10:13:24 -07001213 aaptFlag: "",
Jaewoong Jung525443a2019-02-28 15:35:54 -08001214 },
1215 {
Jaewoong Jungb639a6a2019-05-10 15:16:29 -07001216 moduleName: "bar",
1217 variantName: "android_common_bar",
Jaewoong Jung525443a2019-02-28 15:35:54 -08001218 apkPath: "/target/product/test_device/system/app/bar/bar.apk",
1219 signFlag: "cert/new_cert.x509.pem cert/new_cert.pk8",
Jaewoong Jungb639a6a2019-05-10 15:16:29 -07001220 overrides: []string{"qux", "foo"},
Jaewoong Jung6f373f62019-03-13 10:13:24 -07001221 aaptFlag: "",
1222 },
1223 {
Jaewoong Jungb639a6a2019-05-10 15:16:29 -07001224 moduleName: "baz",
1225 variantName: "android_common_baz",
Jaewoong Jung6f373f62019-03-13 10:13:24 -07001226 apkPath: "/target/product/test_device/system/app/baz/baz.apk",
Dan Willemsen412160e2019-04-09 21:36:26 -07001227 signFlag: "build/make/target/product/security/expiredkey.x509.pem build/make/target/product/security/expiredkey.pk8",
Jaewoong Jungb639a6a2019-05-10 15:16:29 -07001228 overrides: []string{"qux", "foo"},
Jaewoong Jung6f373f62019-03-13 10:13:24 -07001229 aaptFlag: "--rename-manifest-package org.dandroid.bp",
Jaewoong Jung525443a2019-02-28 15:35:54 -08001230 },
1231 }
1232 for _, expected := range expectedVariants {
1233 variant := ctx.ModuleForTests("foo", expected.variantName)
1234
1235 // Check the final apk name
1236 outputs := variant.AllOutputs()
1237 expectedApkPath := buildDir + expected.apkPath
1238 found := false
1239 for _, o := range outputs {
1240 if o == expectedApkPath {
1241 found = true
1242 break
1243 }
1244 }
1245 if !found {
1246 t.Errorf("Can't find %q in output files.\nAll outputs:%v", expectedApkPath, outputs)
1247 }
1248
1249 // Check the certificate paths
Jaewoong Jung5a498812019-11-07 14:14:38 -08001250 signapk := variant.Output(expected.moduleName + ".apk")
Jaewoong Jung525443a2019-02-28 15:35:54 -08001251 signFlag := signapk.Args["certificates"]
1252 if expected.signFlag != signFlag {
1253 t.Errorf("Incorrect signing flags, expected: %q, got: %q", expected.signFlag, signFlag)
1254 }
1255
Jaewoong Jung6f373f62019-03-13 10:13:24 -07001256 // Check if the overrides field values are correctly aggregated.
Jaewoong Jung525443a2019-02-28 15:35:54 -08001257 mod := variant.Module().(*AndroidApp)
1258 if !reflect.DeepEqual(expected.overrides, mod.appProperties.Overrides) {
1259 t.Errorf("Incorrect overrides property value, expected: %q, got: %q",
1260 expected.overrides, mod.appProperties.Overrides)
1261 }
Jaewoong Jung6f373f62019-03-13 10:13:24 -07001262
1263 // Check the package renaming flag, if exists.
1264 res := variant.Output("package-res.apk")
1265 aapt2Flags := res.Args["flags"]
1266 if !strings.Contains(aapt2Flags, expected.aaptFlag) {
1267 t.Errorf("package renaming flag, %q is missing in aapt2 link flags, %q", expected.aaptFlag, aapt2Flags)
1268 }
Jaewoong Jung525443a2019-02-28 15:35:54 -08001269 }
1270}
Jaewoong Jungccbb3932019-04-15 09:48:31 -07001271
Jaewoong Jungb639a6a2019-05-10 15:16:29 -07001272func TestOverrideAndroidAppDependency(t *testing.T) {
Jaewoong Jungf9a04432019-07-17 11:15:09 -07001273 ctx, _ := testJava(t, `
Jaewoong Jungb639a6a2019-05-10 15:16:29 -07001274 android_app {
1275 name: "foo",
1276 srcs: ["a.java"],
Jeongik Cha538c0d02019-07-11 15:54:27 +09001277 sdk_version: "current",
Jaewoong Jungb639a6a2019-05-10 15:16:29 -07001278 }
1279
1280 override_android_app {
1281 name: "bar",
1282 base: "foo",
1283 package_name: "org.dandroid.bp",
1284 }
1285
1286 android_test {
1287 name: "baz",
1288 srcs: ["b.java"],
1289 instrumentation_for: "foo",
1290 }
1291
1292 android_test {
1293 name: "qux",
1294 srcs: ["b.java"],
1295 instrumentation_for: "bar",
1296 }
1297 `)
1298
1299 // Verify baz, which depends on the overridden module foo, has the correct classpath javac arg.
1300 javac := ctx.ModuleForTests("baz", "android_common").Rule("javac")
1301 fooTurbine := filepath.Join(buildDir, ".intermediates", "foo", "android_common", "turbine-combined", "foo.jar")
1302 if !strings.Contains(javac.Args["classpath"], fooTurbine) {
1303 t.Errorf("baz classpath %v does not contain %q", javac.Args["classpath"], fooTurbine)
1304 }
1305
1306 // Verify qux, which depends on the overriding module bar, has the correct classpath javac arg.
1307 javac = ctx.ModuleForTests("qux", "android_common").Rule("javac")
1308 barTurbine := filepath.Join(buildDir, ".intermediates", "foo", "android_common_bar", "turbine-combined", "foo.jar")
1309 if !strings.Contains(javac.Args["classpath"], barTurbine) {
1310 t.Errorf("qux classpath %v does not contain %q", javac.Args["classpath"], barTurbine)
1311 }
1312}
1313
Jaewoong Jung26dedd32019-06-06 08:45:58 -07001314func TestOverrideAndroidTest(t *testing.T) {
1315 ctx, _ := testJava(t, `
1316 android_app {
1317 name: "foo",
1318 srcs: ["a.java"],
1319 package_name: "com.android.foo",
1320 sdk_version: "current",
1321 }
1322
1323 override_android_app {
1324 name: "bar",
1325 base: "foo",
1326 package_name: "com.android.bar",
1327 }
1328
1329 android_test {
1330 name: "foo_test",
1331 srcs: ["b.java"],
1332 instrumentation_for: "foo",
1333 }
1334
1335 override_android_test {
1336 name: "bar_test",
1337 base: "foo_test",
1338 package_name: "com.android.bar.test",
1339 instrumentation_for: "bar",
1340 instrumentation_target_package: "com.android.bar",
1341 }
1342 `)
1343
1344 expectedVariants := []struct {
1345 moduleName string
1346 variantName string
1347 apkPath string
1348 overrides []string
1349 targetVariant string
1350 packageFlag string
1351 targetPackageFlag string
1352 }{
1353 {
1354 variantName: "android_common",
Jaewoong Jung326a9412019-11-21 10:41:00 -08001355 apkPath: "/target/product/test_device/testcases/foo_test/arm64/foo_test.apk",
Jaewoong Jung26dedd32019-06-06 08:45:58 -07001356 overrides: nil,
1357 targetVariant: "android_common",
1358 packageFlag: "",
1359 targetPackageFlag: "",
1360 },
1361 {
1362 variantName: "android_common_bar_test",
Jaewoong Jung326a9412019-11-21 10:41:00 -08001363 apkPath: "/target/product/test_device/testcases/bar_test/arm64/bar_test.apk",
Jaewoong Jung26dedd32019-06-06 08:45:58 -07001364 overrides: []string{"foo_test"},
1365 targetVariant: "android_common_bar",
1366 packageFlag: "com.android.bar.test",
1367 targetPackageFlag: "com.android.bar",
1368 },
1369 }
1370 for _, expected := range expectedVariants {
1371 variant := ctx.ModuleForTests("foo_test", expected.variantName)
1372
1373 // Check the final apk name
1374 outputs := variant.AllOutputs()
1375 expectedApkPath := buildDir + expected.apkPath
1376 found := false
1377 for _, o := range outputs {
1378 if o == expectedApkPath {
1379 found = true
1380 break
1381 }
1382 }
1383 if !found {
1384 t.Errorf("Can't find %q in output files.\nAll outputs:%v", expectedApkPath, outputs)
1385 }
1386
1387 // Check if the overrides field values are correctly aggregated.
1388 mod := variant.Module().(*AndroidTest)
1389 if !reflect.DeepEqual(expected.overrides, mod.appProperties.Overrides) {
1390 t.Errorf("Incorrect overrides property value, expected: %q, got: %q",
1391 expected.overrides, mod.appProperties.Overrides)
1392 }
1393
1394 // Check if javac classpath has the correct jar file path. This checks instrumentation_for overrides.
1395 javac := variant.Rule("javac")
1396 turbine := filepath.Join(buildDir, ".intermediates", "foo", expected.targetVariant, "turbine-combined", "foo.jar")
1397 if !strings.Contains(javac.Args["classpath"], turbine) {
1398 t.Errorf("classpath %q does not contain %q", javac.Args["classpath"], turbine)
1399 }
1400
1401 // Check aapt2 flags.
1402 res := variant.Output("package-res.apk")
1403 aapt2Flags := res.Args["flags"]
1404 checkAapt2LinkFlag(t, aapt2Flags, "rename-manifest-package", expected.packageFlag)
1405 checkAapt2LinkFlag(t, aapt2Flags, "rename-instrumentation-target-package", expected.targetPackageFlag)
1406 }
1407}
1408
Jaewoong Jungccbb3932019-04-15 09:48:31 -07001409func TestAndroidAppImport(t *testing.T) {
Jaewoong Jungf9a04432019-07-17 11:15:09 -07001410 ctx, _ := testJava(t, `
Jaewoong Jungccbb3932019-04-15 09:48:31 -07001411 android_app_import {
1412 name: "foo",
1413 apk: "prebuilts/apk/app.apk",
1414 certificate: "platform",
1415 dex_preopt: {
1416 enabled: true,
1417 },
1418 }
1419 `)
1420
1421 variant := ctx.ModuleForTests("foo", "android_common")
1422
1423 // Check dexpreopt outputs.
1424 if variant.MaybeOutput("dexpreopt/oat/arm64/package.vdex").Rule == nil ||
1425 variant.MaybeOutput("dexpreopt/oat/arm64/package.odex").Rule == nil {
1426 t.Errorf("can't find dexpreopt outputs")
1427 }
1428
1429 // Check cert signing flag.
1430 signedApk := variant.Output("signed/foo.apk")
1431 signingFlag := signedApk.Args["certificates"]
1432 expected := "build/make/target/product/security/platform.x509.pem build/make/target/product/security/platform.pk8"
1433 if expected != signingFlag {
1434 t.Errorf("Incorrect signing flags, expected: %q, got: %q", expected, signingFlag)
1435 }
1436}
1437
1438func TestAndroidAppImport_NoDexPreopt(t *testing.T) {
Jaewoong Jungf9a04432019-07-17 11:15:09 -07001439 ctx, _ := testJava(t, `
Jaewoong Jungccbb3932019-04-15 09:48:31 -07001440 android_app_import {
1441 name: "foo",
1442 apk: "prebuilts/apk/app.apk",
1443 certificate: "platform",
1444 dex_preopt: {
1445 enabled: false,
1446 },
1447 }
1448 `)
1449
1450 variant := ctx.ModuleForTests("foo", "android_common")
1451
1452 // Check dexpreopt outputs. They shouldn't exist.
1453 if variant.MaybeOutput("dexpreopt/oat/arm64/package.vdex").Rule != nil ||
1454 variant.MaybeOutput("dexpreopt/oat/arm64/package.odex").Rule != nil {
1455 t.Errorf("dexpreopt shouldn't have run.")
1456 }
1457}
1458
1459func TestAndroidAppImport_Presigned(t *testing.T) {
Jaewoong Jungf9a04432019-07-17 11:15:09 -07001460 ctx, _ := testJava(t, `
Jaewoong Jungccbb3932019-04-15 09:48:31 -07001461 android_app_import {
1462 name: "foo",
1463 apk: "prebuilts/apk/app.apk",
1464 presigned: true,
1465 dex_preopt: {
1466 enabled: true,
1467 },
1468 }
1469 `)
1470
1471 variant := ctx.ModuleForTests("foo", "android_common")
1472
1473 // Check dexpreopt outputs.
1474 if variant.MaybeOutput("dexpreopt/oat/arm64/package.vdex").Rule == nil ||
1475 variant.MaybeOutput("dexpreopt/oat/arm64/package.odex").Rule == nil {
1476 t.Errorf("can't find dexpreopt outputs")
1477 }
Nicolas Geoffrayc1bf7242019-10-18 14:51:38 +01001478 // Make sure signing was skipped and aligning was done.
Jaewoong Jungccbb3932019-04-15 09:48:31 -07001479 if variant.MaybeOutput("signed/foo.apk").Rule != nil {
1480 t.Errorf("signing rule shouldn't be included.")
1481 }
1482 if variant.MaybeOutput("zip-aligned/foo.apk").Rule == nil {
1483 t.Errorf("can't find aligning rule")
1484 }
1485}
Jaewoong Junga5e5abc2019-04-26 14:31:50 -07001486
Jaewoong Jung961d4fd2019-08-22 14:25:58 -07001487func TestAndroidAppImport_DefaultDevCert(t *testing.T) {
1488 ctx, _ := testJava(t, `
1489 android_app_import {
1490 name: "foo",
1491 apk: "prebuilts/apk/app.apk",
1492 default_dev_cert: true,
1493 dex_preopt: {
1494 enabled: true,
1495 },
1496 }
1497 `)
1498
1499 variant := ctx.ModuleForTests("foo", "android_common")
1500
1501 // Check dexpreopt outputs.
1502 if variant.MaybeOutput("dexpreopt/oat/arm64/package.vdex").Rule == nil ||
1503 variant.MaybeOutput("dexpreopt/oat/arm64/package.odex").Rule == nil {
1504 t.Errorf("can't find dexpreopt outputs")
1505 }
1506
1507 // Check cert signing flag.
1508 signedApk := variant.Output("signed/foo.apk")
1509 signingFlag := signedApk.Args["certificates"]
1510 expected := "build/make/target/product/security/testkey.x509.pem build/make/target/product/security/testkey.pk8"
1511 if expected != signingFlag {
1512 t.Errorf("Incorrect signing flags, expected: %q, got: %q", expected, signingFlag)
1513 }
1514}
1515
Jaewoong Junga5e5abc2019-04-26 14:31:50 -07001516func TestAndroidAppImport_DpiVariants(t *testing.T) {
1517 bp := `
1518 android_app_import {
1519 name: "foo",
1520 apk: "prebuilts/apk/app.apk",
1521 dpi_variants: {
1522 xhdpi: {
1523 apk: "prebuilts/apk/app_xhdpi.apk",
1524 },
1525 xxhdpi: {
1526 apk: "prebuilts/apk/app_xxhdpi.apk",
1527 },
1528 },
Jaewoong Jung961d4fd2019-08-22 14:25:58 -07001529 presigned: true,
Jaewoong Junga5e5abc2019-04-26 14:31:50 -07001530 dex_preopt: {
1531 enabled: true,
1532 },
1533 }
1534 `
1535 testCases := []struct {
1536 name string
1537 aaptPreferredConfig *string
1538 aaptPrebuiltDPI []string
1539 expected string
1540 }{
1541 {
1542 name: "no preferred",
1543 aaptPreferredConfig: nil,
1544 aaptPrebuiltDPI: []string{},
1545 expected: "prebuilts/apk/app.apk",
1546 },
1547 {
1548 name: "AAPTPreferredConfig matches",
1549 aaptPreferredConfig: proptools.StringPtr("xhdpi"),
Jaewoong Jung3e18b192019-06-11 12:25:34 -07001550 aaptPrebuiltDPI: []string{"xxhdpi", "ldpi"},
Jaewoong Junga5e5abc2019-04-26 14:31:50 -07001551 expected: "prebuilts/apk/app_xhdpi.apk",
1552 },
1553 {
1554 name: "AAPTPrebuiltDPI matches",
1555 aaptPreferredConfig: proptools.StringPtr("mdpi"),
1556 aaptPrebuiltDPI: []string{"xxhdpi", "xhdpi"},
1557 expected: "prebuilts/apk/app_xxhdpi.apk",
1558 },
1559 {
1560 name: "non-first AAPTPrebuiltDPI matches",
1561 aaptPreferredConfig: proptools.StringPtr("mdpi"),
1562 aaptPrebuiltDPI: []string{"ldpi", "xhdpi"},
1563 expected: "prebuilts/apk/app_xhdpi.apk",
1564 },
1565 {
1566 name: "no matches",
1567 aaptPreferredConfig: proptools.StringPtr("mdpi"),
1568 aaptPrebuiltDPI: []string{"ldpi", "xxxhdpi"},
1569 expected: "prebuilts/apk/app.apk",
1570 },
1571 }
1572
1573 jniRuleRe := regexp.MustCompile("^if \\(zipinfo (\\S+)")
1574 for _, test := range testCases {
Colin Cross98be1bb2019-12-13 20:41:13 -08001575 config := testAppConfig(nil, bp, nil)
Jaewoong Junga5e5abc2019-04-26 14:31:50 -07001576 config.TestProductVariables.AAPTPreferredConfig = test.aaptPreferredConfig
1577 config.TestProductVariables.AAPTPrebuiltDPI = test.aaptPrebuiltDPI
Colin Cross98be1bb2019-12-13 20:41:13 -08001578 ctx := testContext()
Jaewoong Junga5e5abc2019-04-26 14:31:50 -07001579
1580 run(t, ctx, config)
1581
1582 variant := ctx.ModuleForTests("foo", "android_common")
1583 jniRuleCommand := variant.Output("jnis-uncompressed/foo.apk").RuleParams.Command
1584 matches := jniRuleRe.FindStringSubmatch(jniRuleCommand)
1585 if len(matches) != 2 {
1586 t.Errorf("failed to extract the src apk path from %q", jniRuleCommand)
1587 }
1588 if test.expected != matches[1] {
1589 t.Errorf("wrong src apk, expected: %q got: %q", test.expected, matches[1])
1590 }
1591 }
1592}
Jaewoong Jungbc625cd2019-05-06 15:48:44 -07001593
Jaewoong Jung8aae22e2019-07-17 10:21:49 -07001594func TestAndroidAppImport_Filename(t *testing.T) {
1595 ctx, config := testJava(t, `
1596 android_app_import {
1597 name: "foo",
1598 apk: "prebuilts/apk/app.apk",
1599 presigned: true,
1600 }
1601
1602 android_app_import {
1603 name: "bar",
1604 apk: "prebuilts/apk/app.apk",
1605 presigned: true,
1606 filename: "bar_sample.apk"
1607 }
1608 `)
1609
1610 testCases := []struct {
1611 name string
1612 expected string
1613 }{
1614 {
1615 name: "foo",
1616 expected: "foo.apk",
1617 },
1618 {
1619 name: "bar",
1620 expected: "bar_sample.apk",
1621 },
1622 }
1623
1624 for _, test := range testCases {
1625 variant := ctx.ModuleForTests(test.name, "android_common")
1626 if variant.MaybeOutput(test.expected).Rule == nil {
1627 t.Errorf("can't find output named %q - all outputs: %v", test.expected, variant.AllOutputs())
1628 }
1629
1630 a := variant.Module().(*AndroidAppImport)
1631 expectedValues := []string{test.expected}
1632 actualValues := android.AndroidMkEntriesForTest(
Jiyong Park0b0e1b92019-12-03 13:24:29 +09001633 t, config, "", a)[0].EntryMap["LOCAL_INSTALLED_MODULE_STEM"]
Jaewoong Jung8aae22e2019-07-17 10:21:49 -07001634 if !reflect.DeepEqual(actualValues, expectedValues) {
1635 t.Errorf("Incorrect LOCAL_INSTALLED_MODULE_STEM value '%s', expected '%s'",
1636 actualValues, expectedValues)
1637 }
1638 }
1639}
1640
Jaewoong Jung1ce9ac62019-08-13 14:11:33 -07001641func TestAndroidAppImport_ArchVariants(t *testing.T) {
1642 // The test config's target arch is ARM64.
1643 testCases := []struct {
1644 name string
1645 bp string
1646 expected string
1647 }{
1648 {
1649 name: "matching arch",
1650 bp: `
1651 android_app_import {
1652 name: "foo",
1653 apk: "prebuilts/apk/app.apk",
1654 arch: {
1655 arm64: {
1656 apk: "prebuilts/apk/app_arm64.apk",
1657 },
1658 },
Jaewoong Jung961d4fd2019-08-22 14:25:58 -07001659 presigned: true,
Jaewoong Jung1ce9ac62019-08-13 14:11:33 -07001660 dex_preopt: {
1661 enabled: true,
1662 },
1663 }
1664 `,
1665 expected: "prebuilts/apk/app_arm64.apk",
1666 },
1667 {
1668 name: "no matching arch",
1669 bp: `
1670 android_app_import {
1671 name: "foo",
1672 apk: "prebuilts/apk/app.apk",
1673 arch: {
1674 arm: {
1675 apk: "prebuilts/apk/app_arm.apk",
1676 },
1677 },
Jaewoong Jung961d4fd2019-08-22 14:25:58 -07001678 presigned: true,
Jaewoong Jung1ce9ac62019-08-13 14:11:33 -07001679 dex_preopt: {
1680 enabled: true,
1681 },
1682 }
1683 `,
1684 expected: "prebuilts/apk/app.apk",
1685 },
1686 }
1687
1688 jniRuleRe := regexp.MustCompile("^if \\(zipinfo (\\S+)")
1689 for _, test := range testCases {
1690 ctx, _ := testJava(t, test.bp)
1691
1692 variant := ctx.ModuleForTests("foo", "android_common")
1693 jniRuleCommand := variant.Output("jnis-uncompressed/foo.apk").RuleParams.Command
1694 matches := jniRuleRe.FindStringSubmatch(jniRuleCommand)
1695 if len(matches) != 2 {
1696 t.Errorf("failed to extract the src apk path from %q", jniRuleCommand)
1697 }
1698 if test.expected != matches[1] {
1699 t.Errorf("wrong src apk, expected: %q got: %q", test.expected, matches[1])
1700 }
1701 }
1702}
1703
Jaewoong Jungb28eb5f2019-08-27 15:01:50 -07001704func TestAndroidTestImport(t *testing.T) {
1705 ctx, config := testJava(t, `
1706 android_test_import {
1707 name: "foo",
1708 apk: "prebuilts/apk/app.apk",
1709 presigned: true,
1710 data: [
1711 "testdata/data",
1712 ],
1713 }
1714 `)
1715
1716 test := ctx.ModuleForTests("foo", "android_common").Module().(*AndroidTestImport)
1717
1718 // Check android mks.
Jiyong Park0b0e1b92019-12-03 13:24:29 +09001719 entries := android.AndroidMkEntriesForTest(t, config, "", test)[0]
Jaewoong Jungb28eb5f2019-08-27 15:01:50 -07001720 expected := []string{"tests"}
1721 actual := entries.EntryMap["LOCAL_MODULE_TAGS"]
1722 if !reflect.DeepEqual(expected, actual) {
1723 t.Errorf("Unexpected module tags - expected: %q, actual: %q", expected, actual)
1724 }
1725 expected = []string{"testdata/data:testdata/data"}
1726 actual = entries.EntryMap["LOCAL_COMPATIBILITY_SUPPORT_FILES"]
1727 if !reflect.DeepEqual(expected, actual) {
1728 t.Errorf("Unexpected test data - expected: %q, actual: %q", expected, actual)
1729 }
1730}
1731
Jaewoong Jungbc625cd2019-05-06 15:48:44 -07001732func TestStl(t *testing.T) {
Jaewoong Jungf9a04432019-07-17 11:15:09 -07001733 ctx, _ := testJava(t, cc.GatherRequiredDepsForTest(android.Android)+`
Jaewoong Jungbc625cd2019-05-06 15:48:44 -07001734 cc_library {
Peter Collingbournead84f972019-12-17 16:46:18 -08001735 name: "ndk_libunwind",
1736 sdk_version: "current",
1737 stl: "none",
1738 system_shared_libs: [],
1739 }
1740
1741 cc_library {
1742 name: "libc.ndk.current",
1743 sdk_version: "current",
1744 stl: "none",
1745 system_shared_libs: [],
1746 }
1747
1748 cc_library {
1749 name: "libm.ndk.current",
1750 sdk_version: "current",
1751 stl: "none",
1752 system_shared_libs: [],
1753 }
1754
1755 cc_library {
1756 name: "libdl.ndk.current",
1757 sdk_version: "current",
1758 stl: "none",
1759 system_shared_libs: [],
1760 }
1761
1762 cc_object {
1763 name: "ndk_crtbegin_so.27",
1764 }
1765
1766 cc_object {
1767 name: "ndk_crtend_so.27",
1768 }
1769
1770 cc_library {
Jaewoong Jungbc625cd2019-05-06 15:48:44 -07001771 name: "libjni",
Peter Collingbournead84f972019-12-17 16:46:18 -08001772 sdk_version: "current",
1773 stl: "c++_shared",
Jaewoong Jungbc625cd2019-05-06 15:48:44 -07001774 }
1775
1776 android_test {
1777 name: "stl",
1778 jni_libs: ["libjni"],
1779 compile_multilib: "both",
1780 sdk_version: "current",
1781 stl: "c++_shared",
1782 }
1783
1784 android_test {
1785 name: "system",
1786 jni_libs: ["libjni"],
1787 compile_multilib: "both",
1788 sdk_version: "current",
1789 }
Jaewoong Jung710756a2019-06-04 11:53:47 -07001790
1791 ndk_prebuilt_shared_stl {
1792 name: "ndk_libc++_shared",
1793 }
Jaewoong Jungbc625cd2019-05-06 15:48:44 -07001794 `)
1795
1796 testCases := []struct {
1797 name string
1798 jnis []string
1799 }{
1800 {"stl",
1801 []string{
1802 "libjni.so",
Jaewoong Jung710756a2019-06-04 11:53:47 -07001803 "libc++_shared.so",
Jaewoong Jungbc625cd2019-05-06 15:48:44 -07001804 },
1805 },
1806 {"system",
1807 []string{
1808 "libjni.so",
1809 },
1810 },
1811 }
1812
1813 for _, test := range testCases {
1814 t.Run(test.name, func(t *testing.T) {
1815 app := ctx.ModuleForTests(test.name, "android_common")
1816 jniLibZip := app.Output("jnilibs.zip")
1817 var jnis []string
1818 args := strings.Fields(jniLibZip.Args["jarArgs"])
1819 for i := 0; i < len(args); i++ {
1820 if args[i] == "-f" {
1821 jnis = append(jnis, args[i+1])
1822 i += 1
1823 }
1824 }
1825 jnisJoined := strings.Join(jnis, " ")
1826 for _, jni := range test.jnis {
1827 if !strings.Contains(jnisJoined, jni) {
1828 t.Errorf("missing jni %q in %q", jni, jnis)
1829 }
1830 }
1831 })
1832 }
1833}
Colin Cross50ddcc42019-05-16 12:28:22 -07001834
1835func TestUsesLibraries(t *testing.T) {
1836 bp := `
1837 java_sdk_library {
1838 name: "foo",
1839 srcs: ["a.java"],
1840 api_packages: ["foo"],
Jeongik Cha538c0d02019-07-11 15:54:27 +09001841 sdk_version: "current",
Colin Cross50ddcc42019-05-16 12:28:22 -07001842 }
1843
1844 java_sdk_library {
1845 name: "bar",
1846 srcs: ["a.java"],
1847 api_packages: ["bar"],
Jeongik Cha538c0d02019-07-11 15:54:27 +09001848 sdk_version: "current",
Colin Cross50ddcc42019-05-16 12:28:22 -07001849 }
1850
1851 android_app {
1852 name: "app",
1853 srcs: ["a.java"],
1854 uses_libs: ["foo"],
Jeongik Cha538c0d02019-07-11 15:54:27 +09001855 sdk_version: "current",
Colin Cross50ddcc42019-05-16 12:28:22 -07001856 optional_uses_libs: [
1857 "bar",
1858 "baz",
1859 ],
1860 }
1861
1862 android_app_import {
1863 name: "prebuilt",
1864 apk: "prebuilts/apk/app.apk",
1865 certificate: "platform",
1866 uses_libs: ["foo"],
1867 optional_uses_libs: [
1868 "bar",
1869 "baz",
1870 ],
1871 }
1872 `
1873
Colin Cross98be1bb2019-12-13 20:41:13 -08001874 config := testAppConfig(nil, bp, nil)
Colin Cross50ddcc42019-05-16 12:28:22 -07001875 config.TestProductVariables.MissingUsesLibraries = []string{"baz"}
1876
Colin Cross98be1bb2019-12-13 20:41:13 -08001877 ctx := testContext()
Colin Cross50ddcc42019-05-16 12:28:22 -07001878
1879 run(t, ctx, config)
1880
1881 app := ctx.ModuleForTests("app", "android_common")
1882 prebuilt := ctx.ModuleForTests("prebuilt", "android_common")
1883
1884 // Test that all libraries are verified
1885 cmd := app.Rule("verify_uses_libraries").RuleParams.Command
1886 if w := "--uses-library foo"; !strings.Contains(cmd, w) {
1887 t.Errorf("wanted %q in %q", w, cmd)
1888 }
1889
1890 if w := "--optional-uses-library bar --optional-uses-library baz"; !strings.Contains(cmd, w) {
1891 t.Errorf("wanted %q in %q", w, cmd)
1892 }
1893
1894 cmd = prebuilt.Rule("verify_uses_libraries").RuleParams.Command
1895
1896 if w := `uses_library_names="foo"`; !strings.Contains(cmd, w) {
1897 t.Errorf("wanted %q in %q", w, cmd)
1898 }
1899
1900 if w := `optional_uses_library_names="bar baz"`; !strings.Contains(cmd, w) {
1901 t.Errorf("wanted %q in %q", w, cmd)
1902 }
1903
1904 // Test that only present libraries are preopted
1905 cmd = app.Rule("dexpreopt").RuleParams.Command
1906
1907 if w := `dex_preopt_target_libraries="/system/framework/foo.jar /system/framework/bar.jar"`; !strings.Contains(cmd, w) {
1908 t.Errorf("wanted %q in %q", w, cmd)
1909 }
1910
1911 cmd = prebuilt.Rule("dexpreopt").RuleParams.Command
1912
1913 if w := `dex_preopt_target_libraries="/system/framework/foo.jar /system/framework/bar.jar"`; !strings.Contains(cmd, w) {
1914 t.Errorf("wanted %q in %q", w, cmd)
1915 }
1916}
Jaewoong Jungc27ab662019-05-30 15:51:14 -07001917
1918func TestCodelessApp(t *testing.T) {
1919 testCases := []struct {
1920 name string
1921 bp string
1922 noCode bool
1923 }{
1924 {
1925 name: "normal",
1926 bp: `
1927 android_app {
1928 name: "foo",
1929 srcs: ["a.java"],
Jeongik Cha538c0d02019-07-11 15:54:27 +09001930 sdk_version: "current",
Jaewoong Jungc27ab662019-05-30 15:51:14 -07001931 }
1932 `,
1933 noCode: false,
1934 },
1935 {
1936 name: "app without sources",
1937 bp: `
1938 android_app {
1939 name: "foo",
Jeongik Cha538c0d02019-07-11 15:54:27 +09001940 sdk_version: "current",
Jaewoong Jungc27ab662019-05-30 15:51:14 -07001941 }
1942 `,
1943 noCode: true,
1944 },
1945 {
1946 name: "app with libraries",
1947 bp: `
1948 android_app {
1949 name: "foo",
1950 static_libs: ["lib"],
Jeongik Cha538c0d02019-07-11 15:54:27 +09001951 sdk_version: "current",
Jaewoong Jungc27ab662019-05-30 15:51:14 -07001952 }
1953
1954 java_library {
1955 name: "lib",
1956 srcs: ["a.java"],
Jeongik Cha538c0d02019-07-11 15:54:27 +09001957 sdk_version: "current",
Jaewoong Jungc27ab662019-05-30 15:51:14 -07001958 }
1959 `,
1960 noCode: false,
1961 },
1962 {
1963 name: "app with sourceless libraries",
1964 bp: `
1965 android_app {
1966 name: "foo",
1967 static_libs: ["lib"],
Jeongik Cha538c0d02019-07-11 15:54:27 +09001968 sdk_version: "current",
Jaewoong Jungc27ab662019-05-30 15:51:14 -07001969 }
1970
1971 java_library {
1972 name: "lib",
Jeongik Cha538c0d02019-07-11 15:54:27 +09001973 sdk_version: "current",
Jaewoong Jungc27ab662019-05-30 15:51:14 -07001974 }
1975 `,
1976 // TODO(jungjw): this should probably be true
1977 noCode: false,
1978 },
1979 }
1980
1981 for _, test := range testCases {
1982 t.Run(test.name, func(t *testing.T) {
1983 ctx := testApp(t, test.bp)
1984
1985 foo := ctx.ModuleForTests("foo", "android_common")
1986 manifestFixerArgs := foo.Output("manifest_fixer/AndroidManifest.xml").Args["args"]
1987 if strings.Contains(manifestFixerArgs, "--has-no-code") != test.noCode {
1988 t.Errorf("unexpected manifest_fixer args: %q", manifestFixerArgs)
1989 }
1990 })
1991 }
1992}
Jaewoong Jung5b425e22019-06-17 17:40:56 -07001993
1994func TestEmbedNotice(t *testing.T) {
Jaewoong Jungf9a04432019-07-17 11:15:09 -07001995 ctx, _ := testJava(t, cc.GatherRequiredDepsForTest(android.Android)+`
Jaewoong Jung5b425e22019-06-17 17:40:56 -07001996 android_app {
1997 name: "foo",
1998 srcs: ["a.java"],
1999 static_libs: ["javalib"],
2000 jni_libs: ["libjni"],
2001 notice: "APP_NOTICE",
2002 embed_notices: true,
Jeongik Cha538c0d02019-07-11 15:54:27 +09002003 sdk_version: "current",
Jaewoong Jung5b425e22019-06-17 17:40:56 -07002004 }
2005
2006 // No embed_notice flag
2007 android_app {
2008 name: "bar",
2009 srcs: ["a.java"],
2010 jni_libs: ["libjni"],
2011 notice: "APP_NOTICE",
Jeongik Cha538c0d02019-07-11 15:54:27 +09002012 sdk_version: "current",
Jaewoong Jung5b425e22019-06-17 17:40:56 -07002013 }
2014
2015 // No NOTICE files
2016 android_app {
2017 name: "baz",
2018 srcs: ["a.java"],
2019 embed_notices: true,
Jeongik Cha538c0d02019-07-11 15:54:27 +09002020 sdk_version: "current",
Jaewoong Jung5b425e22019-06-17 17:40:56 -07002021 }
2022
2023 cc_library {
2024 name: "libjni",
2025 system_shared_libs: [],
2026 stl: "none",
2027 notice: "LIB_NOTICE",
2028 }
2029
2030 java_library {
2031 name: "javalib",
2032 srcs: [
2033 ":gen",
2034 ],
Jeongik Cha538c0d02019-07-11 15:54:27 +09002035 sdk_version: "current",
Jaewoong Jung5b425e22019-06-17 17:40:56 -07002036 }
2037
2038 genrule {
2039 name: "gen",
2040 tools: ["gentool"],
2041 out: ["gen.java"],
2042 notice: "GENRULE_NOTICE",
2043 }
2044
2045 java_binary_host {
2046 name: "gentool",
2047 srcs: ["b.java"],
2048 notice: "TOOL_NOTICE",
2049 }
2050 `)
2051
2052 // foo has NOTICE files to process, and embed_notices is true.
2053 foo := ctx.ModuleForTests("foo", "android_common")
2054 // verify merge notices rule.
2055 mergeNotices := foo.Rule("mergeNoticesRule")
2056 noticeInputs := mergeNotices.Inputs.Strings()
2057 // TOOL_NOTICE should be excluded as it's a host module.
2058 if len(mergeNotices.Inputs) != 3 {
2059 t.Errorf("number of input notice files: expected = 3, actual = %q", noticeInputs)
2060 }
2061 if !inList("APP_NOTICE", noticeInputs) {
2062 t.Errorf("APP_NOTICE is missing from notice files, %q", noticeInputs)
2063 }
2064 if !inList("LIB_NOTICE", noticeInputs) {
2065 t.Errorf("LIB_NOTICE is missing from notice files, %q", noticeInputs)
2066 }
2067 if !inList("GENRULE_NOTICE", noticeInputs) {
2068 t.Errorf("GENRULE_NOTICE is missing from notice files, %q", noticeInputs)
2069 }
2070 // aapt2 flags should include -A <NOTICE dir> so that its contents are put in the APK's /assets.
2071 res := foo.Output("package-res.apk")
2072 aapt2Flags := res.Args["flags"]
2073 e := "-A " + buildDir + "/.intermediates/foo/android_common/NOTICE"
2074 if !strings.Contains(aapt2Flags, e) {
2075 t.Errorf("asset dir flag for NOTICE, %q is missing in aapt2 link flags, %q", e, aapt2Flags)
2076 }
2077
2078 // bar has NOTICE files to process, but embed_notices is not set.
2079 bar := ctx.ModuleForTests("bar", "android_common")
Jaewoong Jung98772792019-07-01 17:15:13 -07002080 res = bar.Output("package-res.apk")
2081 aapt2Flags = res.Args["flags"]
2082 e = "-A " + buildDir + "/.intermediates/bar/android_common/NOTICE"
2083 if strings.Contains(aapt2Flags, e) {
2084 t.Errorf("bar shouldn't have the asset dir flag for NOTICE: %q", e)
Jaewoong Jung5b425e22019-06-17 17:40:56 -07002085 }
2086
2087 // baz's embed_notice is true, but it doesn't have any NOTICE files.
2088 baz := ctx.ModuleForTests("baz", "android_common")
Jaewoong Jung98772792019-07-01 17:15:13 -07002089 res = baz.Output("package-res.apk")
2090 aapt2Flags = res.Args["flags"]
2091 e = "-A " + buildDir + "/.intermediates/baz/android_common/NOTICE"
2092 if strings.Contains(aapt2Flags, e) {
2093 t.Errorf("baz shouldn't have the asset dir flag for NOTICE: %q", e)
Jaewoong Jung5b425e22019-06-17 17:40:56 -07002094 }
2095}
Colin Cross53a87f52019-06-25 13:35:30 -07002096
2097func TestUncompressDex(t *testing.T) {
2098 testCases := []struct {
2099 name string
2100 bp string
2101
2102 uncompressedPlatform bool
2103 uncompressedUnbundled bool
2104 }{
2105 {
2106 name: "normal",
2107 bp: `
2108 android_app {
2109 name: "foo",
2110 srcs: ["a.java"],
Jeongik Cha538c0d02019-07-11 15:54:27 +09002111 sdk_version: "current",
Colin Cross53a87f52019-06-25 13:35:30 -07002112 }
2113 `,
2114 uncompressedPlatform: true,
2115 uncompressedUnbundled: false,
2116 },
2117 {
2118 name: "use_embedded_dex",
2119 bp: `
2120 android_app {
2121 name: "foo",
2122 use_embedded_dex: true,
2123 srcs: ["a.java"],
Jeongik Cha538c0d02019-07-11 15:54:27 +09002124 sdk_version: "current",
Colin Cross53a87f52019-06-25 13:35:30 -07002125 }
2126 `,
2127 uncompressedPlatform: true,
2128 uncompressedUnbundled: true,
2129 },
2130 {
2131 name: "privileged",
2132 bp: `
2133 android_app {
2134 name: "foo",
2135 privileged: true,
2136 srcs: ["a.java"],
Jeongik Cha538c0d02019-07-11 15:54:27 +09002137 sdk_version: "current",
Colin Cross53a87f52019-06-25 13:35:30 -07002138 }
2139 `,
2140 uncompressedPlatform: true,
2141 uncompressedUnbundled: true,
2142 },
2143 }
2144
2145 test := func(t *testing.T, bp string, want bool, unbundled bool) {
2146 t.Helper()
2147
Colin Cross98be1bb2019-12-13 20:41:13 -08002148 config := testAppConfig(nil, bp, nil)
Colin Cross53a87f52019-06-25 13:35:30 -07002149 if unbundled {
2150 config.TestProductVariables.Unbundled_build = proptools.BoolPtr(true)
2151 }
2152
Colin Cross98be1bb2019-12-13 20:41:13 -08002153 ctx := testContext()
Colin Cross53a87f52019-06-25 13:35:30 -07002154
2155 run(t, ctx, config)
2156
2157 foo := ctx.ModuleForTests("foo", "android_common")
2158 dex := foo.Rule("r8")
2159 uncompressedInDexJar := strings.Contains(dex.Args["zipFlags"], "-L 0")
2160 aligned := foo.MaybeRule("zipalign").Rule != nil
2161
2162 if uncompressedInDexJar != want {
2163 t.Errorf("want uncompressed in dex %v, got %v", want, uncompressedInDexJar)
2164 }
2165
2166 if aligned != want {
2167 t.Errorf("want aligned %v, got %v", want, aligned)
2168 }
2169 }
2170
2171 for _, tt := range testCases {
2172 t.Run(tt.name, func(t *testing.T) {
2173 t.Run("platform", func(t *testing.T) {
2174 test(t, tt.bp, tt.uncompressedPlatform, false)
2175 })
2176 t.Run("unbundled", func(t *testing.T) {
2177 test(t, tt.bp, tt.uncompressedUnbundled, true)
2178 })
2179 })
2180 }
2181}
Jaewoong Jung26dedd32019-06-06 08:45:58 -07002182
2183func checkAapt2LinkFlag(t *testing.T, aapt2Flags, flagName, expectedValue string) {
2184 if expectedValue != "" {
2185 expectedFlag := "--" + flagName + " " + expectedValue
2186 if !strings.Contains(aapt2Flags, expectedFlag) {
2187 t.Errorf("%q is missing in aapt2 link flags, %q", expectedFlag, aapt2Flags)
2188 }
2189 } else {
2190 unexpectedFlag := "--" + flagName
2191 if strings.Contains(aapt2Flags, unexpectedFlag) {
2192 t.Errorf("unexpected flag, %q is found in aapt2 link flags, %q", unexpectedFlag, aapt2Flags)
2193 }
2194 }
2195}