blob: 6d94160fa266593229aebb4f785dbed708a56887 [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",
Baligh Uddin5b16dfb2020-02-11 17:27:19 -08001184 logging_parent: "bah",
Jaewoong Jung525443a2019-02-28 15:35:54 -08001185 }
1186
1187 android_app_certificate {
1188 name: "new_certificate",
1189 certificate: "cert/new_cert",
1190 }
Jaewoong Jung6f373f62019-03-13 10:13:24 -07001191
1192 override_android_app {
1193 name: "baz",
1194 base: "foo",
1195 package_name: "org.dandroid.bp",
1196 }
Jaewoong Jung525443a2019-02-28 15:35:54 -08001197 `)
1198
1199 expectedVariants := []struct {
Baligh Uddin5b16dfb2020-02-11 17:27:19 -08001200 moduleName string
1201 variantName string
1202 apkName string
1203 apkPath string
1204 signFlag string
1205 overrides []string
1206 aaptFlag string
1207 logging_parent string
Jaewoong Jung525443a2019-02-28 15:35:54 -08001208 }{
1209 {
Baligh Uddin5b16dfb2020-02-11 17:27:19 -08001210 moduleName: "foo",
1211 variantName: "android_common",
1212 apkPath: "/target/product/test_device/system/app/foo/foo.apk",
1213 signFlag: "build/make/target/product/security/expiredkey.x509.pem build/make/target/product/security/expiredkey.pk8",
1214 overrides: []string{"qux"},
1215 aaptFlag: "",
1216 logging_parent: "",
Jaewoong Jung525443a2019-02-28 15:35:54 -08001217 },
1218 {
Baligh Uddin5b16dfb2020-02-11 17:27:19 -08001219 moduleName: "bar",
1220 variantName: "android_common_bar",
1221 apkPath: "/target/product/test_device/system/app/bar/bar.apk",
1222 signFlag: "cert/new_cert.x509.pem cert/new_cert.pk8",
1223 overrides: []string{"qux", "foo"},
1224 aaptFlag: "",
1225 logging_parent: "bah",
Jaewoong Jung6f373f62019-03-13 10:13:24 -07001226 },
1227 {
Baligh Uddin5b16dfb2020-02-11 17:27:19 -08001228 moduleName: "baz",
1229 variantName: "android_common_baz",
1230 apkPath: "/target/product/test_device/system/app/baz/baz.apk",
1231 signFlag: "build/make/target/product/security/expiredkey.x509.pem build/make/target/product/security/expiredkey.pk8",
1232 overrides: []string{"qux", "foo"},
1233 aaptFlag: "--rename-manifest-package org.dandroid.bp",
1234 logging_parent: "",
Jaewoong Jung525443a2019-02-28 15:35:54 -08001235 },
1236 }
1237 for _, expected := range expectedVariants {
1238 variant := ctx.ModuleForTests("foo", expected.variantName)
1239
1240 // Check the final apk name
1241 outputs := variant.AllOutputs()
1242 expectedApkPath := buildDir + expected.apkPath
1243 found := false
1244 for _, o := range outputs {
1245 if o == expectedApkPath {
1246 found = true
1247 break
1248 }
1249 }
1250 if !found {
1251 t.Errorf("Can't find %q in output files.\nAll outputs:%v", expectedApkPath, outputs)
1252 }
1253
1254 // Check the certificate paths
Jaewoong Jung5a498812019-11-07 14:14:38 -08001255 signapk := variant.Output(expected.moduleName + ".apk")
Jaewoong Jung525443a2019-02-28 15:35:54 -08001256 signFlag := signapk.Args["certificates"]
1257 if expected.signFlag != signFlag {
1258 t.Errorf("Incorrect signing flags, expected: %q, got: %q", expected.signFlag, signFlag)
1259 }
1260
Jaewoong Jung6f373f62019-03-13 10:13:24 -07001261 // Check if the overrides field values are correctly aggregated.
Jaewoong Jung525443a2019-02-28 15:35:54 -08001262 mod := variant.Module().(*AndroidApp)
1263 if !reflect.DeepEqual(expected.overrides, mod.appProperties.Overrides) {
1264 t.Errorf("Incorrect overrides property value, expected: %q, got: %q",
1265 expected.overrides, mod.appProperties.Overrides)
1266 }
Jaewoong Jung6f373f62019-03-13 10:13:24 -07001267
Baligh Uddin5b16dfb2020-02-11 17:27:19 -08001268 // Test Overridable property: Logging_parent
1269 logging_parent := mod.aapt.LoggingParent
1270 if expected.logging_parent != logging_parent {
1271 t.Errorf("Incorrect overrides property value for logging parent, expected: %q, got: %q",
1272 expected.logging_parent, logging_parent)
1273 }
1274
Jaewoong Jung6f373f62019-03-13 10:13:24 -07001275 // Check the package renaming flag, if exists.
1276 res := variant.Output("package-res.apk")
1277 aapt2Flags := res.Args["flags"]
1278 if !strings.Contains(aapt2Flags, expected.aaptFlag) {
1279 t.Errorf("package renaming flag, %q is missing in aapt2 link flags, %q", expected.aaptFlag, aapt2Flags)
1280 }
Jaewoong Jung525443a2019-02-28 15:35:54 -08001281 }
1282}
Jaewoong Jungccbb3932019-04-15 09:48:31 -07001283
Jaewoong Jungb639a6a2019-05-10 15:16:29 -07001284func TestOverrideAndroidAppDependency(t *testing.T) {
Jaewoong Jungf9a04432019-07-17 11:15:09 -07001285 ctx, _ := testJava(t, `
Jaewoong Jungb639a6a2019-05-10 15:16:29 -07001286 android_app {
1287 name: "foo",
1288 srcs: ["a.java"],
Jeongik Cha538c0d02019-07-11 15:54:27 +09001289 sdk_version: "current",
Jaewoong Jungb639a6a2019-05-10 15:16:29 -07001290 }
1291
1292 override_android_app {
1293 name: "bar",
1294 base: "foo",
1295 package_name: "org.dandroid.bp",
1296 }
1297
1298 android_test {
1299 name: "baz",
1300 srcs: ["b.java"],
1301 instrumentation_for: "foo",
1302 }
1303
1304 android_test {
1305 name: "qux",
1306 srcs: ["b.java"],
1307 instrumentation_for: "bar",
1308 }
1309 `)
1310
1311 // Verify baz, which depends on the overridden module foo, has the correct classpath javac arg.
1312 javac := ctx.ModuleForTests("baz", "android_common").Rule("javac")
1313 fooTurbine := filepath.Join(buildDir, ".intermediates", "foo", "android_common", "turbine-combined", "foo.jar")
1314 if !strings.Contains(javac.Args["classpath"], fooTurbine) {
1315 t.Errorf("baz classpath %v does not contain %q", javac.Args["classpath"], fooTurbine)
1316 }
1317
1318 // Verify qux, which depends on the overriding module bar, has the correct classpath javac arg.
1319 javac = ctx.ModuleForTests("qux", "android_common").Rule("javac")
1320 barTurbine := filepath.Join(buildDir, ".intermediates", "foo", "android_common_bar", "turbine-combined", "foo.jar")
1321 if !strings.Contains(javac.Args["classpath"], barTurbine) {
1322 t.Errorf("qux classpath %v does not contain %q", javac.Args["classpath"], barTurbine)
1323 }
1324}
1325
Jaewoong Jung26dedd32019-06-06 08:45:58 -07001326func TestOverrideAndroidTest(t *testing.T) {
1327 ctx, _ := testJava(t, `
1328 android_app {
1329 name: "foo",
1330 srcs: ["a.java"],
1331 package_name: "com.android.foo",
1332 sdk_version: "current",
1333 }
1334
1335 override_android_app {
1336 name: "bar",
1337 base: "foo",
1338 package_name: "com.android.bar",
1339 }
1340
1341 android_test {
1342 name: "foo_test",
1343 srcs: ["b.java"],
1344 instrumentation_for: "foo",
1345 }
1346
1347 override_android_test {
1348 name: "bar_test",
1349 base: "foo_test",
1350 package_name: "com.android.bar.test",
1351 instrumentation_for: "bar",
1352 instrumentation_target_package: "com.android.bar",
1353 }
1354 `)
1355
1356 expectedVariants := []struct {
1357 moduleName string
1358 variantName string
1359 apkPath string
1360 overrides []string
1361 targetVariant string
1362 packageFlag string
1363 targetPackageFlag string
1364 }{
1365 {
1366 variantName: "android_common",
Jaewoong Jung326a9412019-11-21 10:41:00 -08001367 apkPath: "/target/product/test_device/testcases/foo_test/arm64/foo_test.apk",
Jaewoong Jung26dedd32019-06-06 08:45:58 -07001368 overrides: nil,
1369 targetVariant: "android_common",
1370 packageFlag: "",
1371 targetPackageFlag: "",
1372 },
1373 {
1374 variantName: "android_common_bar_test",
Jaewoong Jung326a9412019-11-21 10:41:00 -08001375 apkPath: "/target/product/test_device/testcases/bar_test/arm64/bar_test.apk",
Jaewoong Jung26dedd32019-06-06 08:45:58 -07001376 overrides: []string{"foo_test"},
1377 targetVariant: "android_common_bar",
1378 packageFlag: "com.android.bar.test",
1379 targetPackageFlag: "com.android.bar",
1380 },
1381 }
1382 for _, expected := range expectedVariants {
1383 variant := ctx.ModuleForTests("foo_test", expected.variantName)
1384
1385 // Check the final apk name
1386 outputs := variant.AllOutputs()
1387 expectedApkPath := buildDir + expected.apkPath
1388 found := false
1389 for _, o := range outputs {
1390 if o == expectedApkPath {
1391 found = true
1392 break
1393 }
1394 }
1395 if !found {
1396 t.Errorf("Can't find %q in output files.\nAll outputs:%v", expectedApkPath, outputs)
1397 }
1398
1399 // Check if the overrides field values are correctly aggregated.
1400 mod := variant.Module().(*AndroidTest)
1401 if !reflect.DeepEqual(expected.overrides, mod.appProperties.Overrides) {
1402 t.Errorf("Incorrect overrides property value, expected: %q, got: %q",
1403 expected.overrides, mod.appProperties.Overrides)
1404 }
1405
1406 // Check if javac classpath has the correct jar file path. This checks instrumentation_for overrides.
1407 javac := variant.Rule("javac")
1408 turbine := filepath.Join(buildDir, ".intermediates", "foo", expected.targetVariant, "turbine-combined", "foo.jar")
1409 if !strings.Contains(javac.Args["classpath"], turbine) {
1410 t.Errorf("classpath %q does not contain %q", javac.Args["classpath"], turbine)
1411 }
1412
1413 // Check aapt2 flags.
1414 res := variant.Output("package-res.apk")
1415 aapt2Flags := res.Args["flags"]
1416 checkAapt2LinkFlag(t, aapt2Flags, "rename-manifest-package", expected.packageFlag)
1417 checkAapt2LinkFlag(t, aapt2Flags, "rename-instrumentation-target-package", expected.targetPackageFlag)
1418 }
1419}
1420
Jaewoong Jung39982342020-01-14 10:27:18 -08001421func TestAndroidTest_FixTestConfig(t *testing.T) {
1422 ctx, _ := testJava(t, `
1423 android_app {
1424 name: "foo",
1425 srcs: ["a.java"],
1426 package_name: "com.android.foo",
1427 sdk_version: "current",
1428 }
1429
1430 android_test {
1431 name: "foo_test",
1432 srcs: ["b.java"],
1433 instrumentation_for: "foo",
1434 }
1435
1436 android_test {
1437 name: "bar_test",
1438 srcs: ["b.java"],
1439 package_name: "com.android.bar.test",
1440 instrumentation_for: "foo",
1441 }
1442
1443 override_android_test {
1444 name: "baz_test",
1445 base: "foo_test",
1446 package_name: "com.android.baz.test",
1447 }
1448 `)
1449
1450 testCases := []struct {
1451 moduleName string
1452 variantName string
1453 expectedFlags []string
1454 }{
1455 {
1456 moduleName: "foo_test",
1457 variantName: "android_common",
1458 },
1459 {
1460 moduleName: "bar_test",
1461 variantName: "android_common",
1462 expectedFlags: []string{
1463 "--manifest " + buildDir + "/.intermediates/bar_test/android_common/manifest_fixer/AndroidManifest.xml",
1464 "--package-name com.android.bar.test",
1465 },
1466 },
1467 {
1468 moduleName: "foo_test",
1469 variantName: "android_common_baz_test",
1470 expectedFlags: []string{
1471 "--manifest " + buildDir +
1472 "/.intermediates/foo_test/android_common_baz_test/manifest_fixer/AndroidManifest.xml",
1473 "--package-name com.android.baz.test",
1474 "--test-file-name baz_test.apk",
1475 },
1476 },
1477 }
1478
1479 for _, test := range testCases {
1480 variant := ctx.ModuleForTests(test.moduleName, test.variantName)
1481 params := variant.MaybeOutput("test_config_fixer/AndroidTest.xml")
1482
1483 if len(test.expectedFlags) > 0 {
1484 if params.Rule == nil {
1485 t.Errorf("test_config_fixer was expected to run, but didn't")
1486 } else {
1487 for _, flag := range test.expectedFlags {
1488 if !strings.Contains(params.RuleParams.Command, flag) {
1489 t.Errorf("Flag %q was not found in command: %q", flag, params.RuleParams.Command)
1490 }
1491 }
1492 }
1493 } else {
1494 if params.Rule != nil {
1495 t.Errorf("test_config_fixer was not expected to run, but did: %q", params.RuleParams.Command)
1496 }
1497 }
1498
1499 }
1500}
1501
Jaewoong Jungccbb3932019-04-15 09:48:31 -07001502func TestAndroidAppImport(t *testing.T) {
Jaewoong Jungf9a04432019-07-17 11:15:09 -07001503 ctx, _ := testJava(t, `
Jaewoong Jungccbb3932019-04-15 09:48:31 -07001504 android_app_import {
1505 name: "foo",
1506 apk: "prebuilts/apk/app.apk",
1507 certificate: "platform",
1508 dex_preopt: {
1509 enabled: true,
1510 },
1511 }
1512 `)
1513
1514 variant := ctx.ModuleForTests("foo", "android_common")
1515
1516 // Check dexpreopt outputs.
1517 if variant.MaybeOutput("dexpreopt/oat/arm64/package.vdex").Rule == nil ||
1518 variant.MaybeOutput("dexpreopt/oat/arm64/package.odex").Rule == nil {
1519 t.Errorf("can't find dexpreopt outputs")
1520 }
1521
1522 // Check cert signing flag.
1523 signedApk := variant.Output("signed/foo.apk")
1524 signingFlag := signedApk.Args["certificates"]
1525 expected := "build/make/target/product/security/platform.x509.pem build/make/target/product/security/platform.pk8"
1526 if expected != signingFlag {
1527 t.Errorf("Incorrect signing flags, expected: %q, got: %q", expected, signingFlag)
1528 }
1529}
1530
1531func TestAndroidAppImport_NoDexPreopt(t *testing.T) {
Jaewoong Jungf9a04432019-07-17 11:15:09 -07001532 ctx, _ := testJava(t, `
Jaewoong Jungccbb3932019-04-15 09:48:31 -07001533 android_app_import {
1534 name: "foo",
1535 apk: "prebuilts/apk/app.apk",
1536 certificate: "platform",
1537 dex_preopt: {
1538 enabled: false,
1539 },
1540 }
1541 `)
1542
1543 variant := ctx.ModuleForTests("foo", "android_common")
1544
1545 // Check dexpreopt outputs. They shouldn't exist.
1546 if variant.MaybeOutput("dexpreopt/oat/arm64/package.vdex").Rule != nil ||
1547 variant.MaybeOutput("dexpreopt/oat/arm64/package.odex").Rule != nil {
1548 t.Errorf("dexpreopt shouldn't have run.")
1549 }
1550}
1551
1552func TestAndroidAppImport_Presigned(t *testing.T) {
Jaewoong Jungf9a04432019-07-17 11:15:09 -07001553 ctx, _ := testJava(t, `
Jaewoong Jungccbb3932019-04-15 09:48:31 -07001554 android_app_import {
1555 name: "foo",
1556 apk: "prebuilts/apk/app.apk",
1557 presigned: true,
1558 dex_preopt: {
1559 enabled: true,
1560 },
1561 }
1562 `)
1563
1564 variant := ctx.ModuleForTests("foo", "android_common")
1565
1566 // Check dexpreopt outputs.
1567 if variant.MaybeOutput("dexpreopt/oat/arm64/package.vdex").Rule == nil ||
1568 variant.MaybeOutput("dexpreopt/oat/arm64/package.odex").Rule == nil {
1569 t.Errorf("can't find dexpreopt outputs")
1570 }
Nicolas Geoffrayc1bf7242019-10-18 14:51:38 +01001571 // Make sure signing was skipped and aligning was done.
Jaewoong Jungccbb3932019-04-15 09:48:31 -07001572 if variant.MaybeOutput("signed/foo.apk").Rule != nil {
1573 t.Errorf("signing rule shouldn't be included.")
1574 }
1575 if variant.MaybeOutput("zip-aligned/foo.apk").Rule == nil {
1576 t.Errorf("can't find aligning rule")
1577 }
1578}
Jaewoong Junga5e5abc2019-04-26 14:31:50 -07001579
Jaewoong Jung961d4fd2019-08-22 14:25:58 -07001580func TestAndroidAppImport_DefaultDevCert(t *testing.T) {
1581 ctx, _ := testJava(t, `
1582 android_app_import {
1583 name: "foo",
1584 apk: "prebuilts/apk/app.apk",
1585 default_dev_cert: true,
1586 dex_preopt: {
1587 enabled: true,
1588 },
1589 }
1590 `)
1591
1592 variant := ctx.ModuleForTests("foo", "android_common")
1593
1594 // Check dexpreopt outputs.
1595 if variant.MaybeOutput("dexpreopt/oat/arm64/package.vdex").Rule == nil ||
1596 variant.MaybeOutput("dexpreopt/oat/arm64/package.odex").Rule == nil {
1597 t.Errorf("can't find dexpreopt outputs")
1598 }
1599
1600 // Check cert signing flag.
1601 signedApk := variant.Output("signed/foo.apk")
1602 signingFlag := signedApk.Args["certificates"]
1603 expected := "build/make/target/product/security/testkey.x509.pem build/make/target/product/security/testkey.pk8"
1604 if expected != signingFlag {
1605 t.Errorf("Incorrect signing flags, expected: %q, got: %q", expected, signingFlag)
1606 }
1607}
1608
Jaewoong Junga5e5abc2019-04-26 14:31:50 -07001609func TestAndroidAppImport_DpiVariants(t *testing.T) {
1610 bp := `
1611 android_app_import {
1612 name: "foo",
1613 apk: "prebuilts/apk/app.apk",
1614 dpi_variants: {
1615 xhdpi: {
1616 apk: "prebuilts/apk/app_xhdpi.apk",
1617 },
1618 xxhdpi: {
1619 apk: "prebuilts/apk/app_xxhdpi.apk",
1620 },
1621 },
Jaewoong Jung961d4fd2019-08-22 14:25:58 -07001622 presigned: true,
Jaewoong Junga5e5abc2019-04-26 14:31:50 -07001623 dex_preopt: {
1624 enabled: true,
1625 },
1626 }
1627 `
1628 testCases := []struct {
1629 name string
1630 aaptPreferredConfig *string
1631 aaptPrebuiltDPI []string
1632 expected string
1633 }{
1634 {
1635 name: "no preferred",
1636 aaptPreferredConfig: nil,
1637 aaptPrebuiltDPI: []string{},
1638 expected: "prebuilts/apk/app.apk",
1639 },
1640 {
1641 name: "AAPTPreferredConfig matches",
1642 aaptPreferredConfig: proptools.StringPtr("xhdpi"),
Jaewoong Jung3e18b192019-06-11 12:25:34 -07001643 aaptPrebuiltDPI: []string{"xxhdpi", "ldpi"},
Jaewoong Junga5e5abc2019-04-26 14:31:50 -07001644 expected: "prebuilts/apk/app_xhdpi.apk",
1645 },
1646 {
1647 name: "AAPTPrebuiltDPI matches",
1648 aaptPreferredConfig: proptools.StringPtr("mdpi"),
1649 aaptPrebuiltDPI: []string{"xxhdpi", "xhdpi"},
1650 expected: "prebuilts/apk/app_xxhdpi.apk",
1651 },
1652 {
1653 name: "non-first AAPTPrebuiltDPI matches",
1654 aaptPreferredConfig: proptools.StringPtr("mdpi"),
1655 aaptPrebuiltDPI: []string{"ldpi", "xhdpi"},
1656 expected: "prebuilts/apk/app_xhdpi.apk",
1657 },
1658 {
1659 name: "no matches",
1660 aaptPreferredConfig: proptools.StringPtr("mdpi"),
1661 aaptPrebuiltDPI: []string{"ldpi", "xxxhdpi"},
1662 expected: "prebuilts/apk/app.apk",
1663 },
1664 }
1665
1666 jniRuleRe := regexp.MustCompile("^if \\(zipinfo (\\S+)")
1667 for _, test := range testCases {
Colin Cross98be1bb2019-12-13 20:41:13 -08001668 config := testAppConfig(nil, bp, nil)
Jaewoong Junga5e5abc2019-04-26 14:31:50 -07001669 config.TestProductVariables.AAPTPreferredConfig = test.aaptPreferredConfig
1670 config.TestProductVariables.AAPTPrebuiltDPI = test.aaptPrebuiltDPI
Colin Cross98be1bb2019-12-13 20:41:13 -08001671 ctx := testContext()
Jaewoong Junga5e5abc2019-04-26 14:31:50 -07001672
1673 run(t, ctx, config)
1674
1675 variant := ctx.ModuleForTests("foo", "android_common")
1676 jniRuleCommand := variant.Output("jnis-uncompressed/foo.apk").RuleParams.Command
1677 matches := jniRuleRe.FindStringSubmatch(jniRuleCommand)
1678 if len(matches) != 2 {
1679 t.Errorf("failed to extract the src apk path from %q", jniRuleCommand)
1680 }
1681 if test.expected != matches[1] {
1682 t.Errorf("wrong src apk, expected: %q got: %q", test.expected, matches[1])
1683 }
1684 }
1685}
Jaewoong Jungbc625cd2019-05-06 15:48:44 -07001686
Jaewoong Jung8aae22e2019-07-17 10:21:49 -07001687func TestAndroidAppImport_Filename(t *testing.T) {
1688 ctx, config := testJava(t, `
1689 android_app_import {
1690 name: "foo",
1691 apk: "prebuilts/apk/app.apk",
1692 presigned: true,
1693 }
1694
1695 android_app_import {
1696 name: "bar",
1697 apk: "prebuilts/apk/app.apk",
1698 presigned: true,
1699 filename: "bar_sample.apk"
1700 }
1701 `)
1702
1703 testCases := []struct {
1704 name string
1705 expected string
1706 }{
1707 {
1708 name: "foo",
1709 expected: "foo.apk",
1710 },
1711 {
1712 name: "bar",
1713 expected: "bar_sample.apk",
1714 },
1715 }
1716
1717 for _, test := range testCases {
1718 variant := ctx.ModuleForTests(test.name, "android_common")
1719 if variant.MaybeOutput(test.expected).Rule == nil {
1720 t.Errorf("can't find output named %q - all outputs: %v", test.expected, variant.AllOutputs())
1721 }
1722
1723 a := variant.Module().(*AndroidAppImport)
1724 expectedValues := []string{test.expected}
1725 actualValues := android.AndroidMkEntriesForTest(
Jiyong Park0b0e1b92019-12-03 13:24:29 +09001726 t, config, "", a)[0].EntryMap["LOCAL_INSTALLED_MODULE_STEM"]
Jaewoong Jung8aae22e2019-07-17 10:21:49 -07001727 if !reflect.DeepEqual(actualValues, expectedValues) {
1728 t.Errorf("Incorrect LOCAL_INSTALLED_MODULE_STEM value '%s', expected '%s'",
1729 actualValues, expectedValues)
1730 }
1731 }
1732}
1733
Jaewoong Jung1ce9ac62019-08-13 14:11:33 -07001734func TestAndroidAppImport_ArchVariants(t *testing.T) {
1735 // The test config's target arch is ARM64.
1736 testCases := []struct {
1737 name string
1738 bp string
1739 expected string
1740 }{
1741 {
1742 name: "matching arch",
1743 bp: `
1744 android_app_import {
1745 name: "foo",
1746 apk: "prebuilts/apk/app.apk",
1747 arch: {
1748 arm64: {
1749 apk: "prebuilts/apk/app_arm64.apk",
1750 },
1751 },
Jaewoong Jung961d4fd2019-08-22 14:25:58 -07001752 presigned: true,
Jaewoong Jung1ce9ac62019-08-13 14:11:33 -07001753 dex_preopt: {
1754 enabled: true,
1755 },
1756 }
1757 `,
1758 expected: "prebuilts/apk/app_arm64.apk",
1759 },
1760 {
1761 name: "no matching arch",
1762 bp: `
1763 android_app_import {
1764 name: "foo",
1765 apk: "prebuilts/apk/app.apk",
1766 arch: {
1767 arm: {
1768 apk: "prebuilts/apk/app_arm.apk",
1769 },
1770 },
Jaewoong Jung961d4fd2019-08-22 14:25:58 -07001771 presigned: true,
Jaewoong Jung1ce9ac62019-08-13 14:11:33 -07001772 dex_preopt: {
1773 enabled: true,
1774 },
1775 }
1776 `,
1777 expected: "prebuilts/apk/app.apk",
1778 },
1779 }
1780
1781 jniRuleRe := regexp.MustCompile("^if \\(zipinfo (\\S+)")
1782 for _, test := range testCases {
1783 ctx, _ := testJava(t, test.bp)
1784
1785 variant := ctx.ModuleForTests("foo", "android_common")
1786 jniRuleCommand := variant.Output("jnis-uncompressed/foo.apk").RuleParams.Command
1787 matches := jniRuleRe.FindStringSubmatch(jniRuleCommand)
1788 if len(matches) != 2 {
1789 t.Errorf("failed to extract the src apk path from %q", jniRuleCommand)
1790 }
1791 if test.expected != matches[1] {
1792 t.Errorf("wrong src apk, expected: %q got: %q", test.expected, matches[1])
1793 }
1794 }
1795}
1796
Jaewoong Jungb28eb5f2019-08-27 15:01:50 -07001797func TestAndroidTestImport(t *testing.T) {
1798 ctx, config := testJava(t, `
1799 android_test_import {
1800 name: "foo",
1801 apk: "prebuilts/apk/app.apk",
1802 presigned: true,
1803 data: [
1804 "testdata/data",
1805 ],
1806 }
1807 `)
1808
1809 test := ctx.ModuleForTests("foo", "android_common").Module().(*AndroidTestImport)
1810
1811 // Check android mks.
Jiyong Park0b0e1b92019-12-03 13:24:29 +09001812 entries := android.AndroidMkEntriesForTest(t, config, "", test)[0]
Jaewoong Jungb28eb5f2019-08-27 15:01:50 -07001813 expected := []string{"tests"}
1814 actual := entries.EntryMap["LOCAL_MODULE_TAGS"]
1815 if !reflect.DeepEqual(expected, actual) {
1816 t.Errorf("Unexpected module tags - expected: %q, actual: %q", expected, actual)
1817 }
1818 expected = []string{"testdata/data:testdata/data"}
1819 actual = entries.EntryMap["LOCAL_COMPATIBILITY_SUPPORT_FILES"]
1820 if !reflect.DeepEqual(expected, actual) {
1821 t.Errorf("Unexpected test data - expected: %q, actual: %q", expected, actual)
1822 }
1823}
1824
Jaewoong Jung7c5bd832020-01-13 09:55:39 -08001825func TestAndroidTestImport_NoJinUncompressForPresigned(t *testing.T) {
1826 ctx, _ := testJava(t, `
1827 android_test_import {
1828 name: "foo",
1829 apk: "prebuilts/apk/app.apk",
1830 certificate: "cert/new_cert",
1831 data: [
1832 "testdata/data",
1833 ],
1834 }
1835
1836 android_test_import {
1837 name: "foo_presigned",
1838 apk: "prebuilts/apk/app.apk",
1839 presigned: true,
1840 data: [
1841 "testdata/data",
1842 ],
1843 }
1844 `)
1845
1846 variant := ctx.ModuleForTests("foo", "android_common")
1847 jniRule := variant.Output("jnis-uncompressed/foo.apk").RuleParams.Command
1848 if !strings.HasPrefix(jniRule, "if (zipinfo") {
1849 t.Errorf("Unexpected JNI uncompress rule command: " + jniRule)
1850 }
1851
1852 variant = ctx.ModuleForTests("foo_presigned", "android_common")
1853 jniRule = variant.Output("jnis-uncompressed/foo_presigned.apk").BuildParams.Rule.String()
1854 if jniRule != android.Cp.String() {
1855 t.Errorf("Unexpected JNI uncompress rule: " + jniRule)
1856 }
1857}
1858
Jaewoong Jungbc625cd2019-05-06 15:48:44 -07001859func TestStl(t *testing.T) {
Jaewoong Jungf9a04432019-07-17 11:15:09 -07001860 ctx, _ := testJava(t, cc.GatherRequiredDepsForTest(android.Android)+`
Jaewoong Jungbc625cd2019-05-06 15:48:44 -07001861 cc_library {
Peter Collingbournead84f972019-12-17 16:46:18 -08001862 name: "ndk_libunwind",
1863 sdk_version: "current",
1864 stl: "none",
1865 system_shared_libs: [],
1866 }
1867
1868 cc_library {
1869 name: "libc.ndk.current",
1870 sdk_version: "current",
1871 stl: "none",
1872 system_shared_libs: [],
1873 }
1874
1875 cc_library {
1876 name: "libm.ndk.current",
1877 sdk_version: "current",
1878 stl: "none",
1879 system_shared_libs: [],
1880 }
1881
1882 cc_library {
1883 name: "libdl.ndk.current",
1884 sdk_version: "current",
1885 stl: "none",
1886 system_shared_libs: [],
1887 }
1888
1889 cc_object {
1890 name: "ndk_crtbegin_so.27",
1891 }
1892
1893 cc_object {
1894 name: "ndk_crtend_so.27",
1895 }
1896
1897 cc_library {
Jaewoong Jungbc625cd2019-05-06 15:48:44 -07001898 name: "libjni",
Peter Collingbournead84f972019-12-17 16:46:18 -08001899 sdk_version: "current",
1900 stl: "c++_shared",
Jaewoong Jungbc625cd2019-05-06 15:48:44 -07001901 }
1902
1903 android_test {
1904 name: "stl",
1905 jni_libs: ["libjni"],
1906 compile_multilib: "both",
1907 sdk_version: "current",
1908 stl: "c++_shared",
1909 }
1910
1911 android_test {
1912 name: "system",
1913 jni_libs: ["libjni"],
1914 compile_multilib: "both",
1915 sdk_version: "current",
1916 }
Jaewoong Jung710756a2019-06-04 11:53:47 -07001917
1918 ndk_prebuilt_shared_stl {
1919 name: "ndk_libc++_shared",
1920 }
Jaewoong Jungbc625cd2019-05-06 15:48:44 -07001921 `)
1922
1923 testCases := []struct {
1924 name string
1925 jnis []string
1926 }{
1927 {"stl",
1928 []string{
1929 "libjni.so",
Jaewoong Jung710756a2019-06-04 11:53:47 -07001930 "libc++_shared.so",
Jaewoong Jungbc625cd2019-05-06 15:48:44 -07001931 },
1932 },
1933 {"system",
1934 []string{
1935 "libjni.so",
1936 },
1937 },
1938 }
1939
1940 for _, test := range testCases {
1941 t.Run(test.name, func(t *testing.T) {
1942 app := ctx.ModuleForTests(test.name, "android_common")
1943 jniLibZip := app.Output("jnilibs.zip")
1944 var jnis []string
1945 args := strings.Fields(jniLibZip.Args["jarArgs"])
1946 for i := 0; i < len(args); i++ {
1947 if args[i] == "-f" {
1948 jnis = append(jnis, args[i+1])
1949 i += 1
1950 }
1951 }
1952 jnisJoined := strings.Join(jnis, " ")
1953 for _, jni := range test.jnis {
1954 if !strings.Contains(jnisJoined, jni) {
1955 t.Errorf("missing jni %q in %q", jni, jnis)
1956 }
1957 }
1958 })
1959 }
1960}
Colin Cross50ddcc42019-05-16 12:28:22 -07001961
1962func TestUsesLibraries(t *testing.T) {
1963 bp := `
1964 java_sdk_library {
1965 name: "foo",
1966 srcs: ["a.java"],
1967 api_packages: ["foo"],
Jeongik Cha538c0d02019-07-11 15:54:27 +09001968 sdk_version: "current",
Colin Cross50ddcc42019-05-16 12:28:22 -07001969 }
1970
1971 java_sdk_library {
1972 name: "bar",
1973 srcs: ["a.java"],
1974 api_packages: ["bar"],
Jeongik Cha538c0d02019-07-11 15:54:27 +09001975 sdk_version: "current",
Colin Cross50ddcc42019-05-16 12:28:22 -07001976 }
1977
1978 android_app {
1979 name: "app",
1980 srcs: ["a.java"],
1981 uses_libs: ["foo"],
Jeongik Cha538c0d02019-07-11 15:54:27 +09001982 sdk_version: "current",
Colin Cross50ddcc42019-05-16 12:28:22 -07001983 optional_uses_libs: [
1984 "bar",
1985 "baz",
1986 ],
1987 }
1988
1989 android_app_import {
1990 name: "prebuilt",
1991 apk: "prebuilts/apk/app.apk",
1992 certificate: "platform",
1993 uses_libs: ["foo"],
1994 optional_uses_libs: [
1995 "bar",
1996 "baz",
1997 ],
1998 }
1999 `
2000
Colin Cross98be1bb2019-12-13 20:41:13 -08002001 config := testAppConfig(nil, bp, nil)
Colin Cross50ddcc42019-05-16 12:28:22 -07002002 config.TestProductVariables.MissingUsesLibraries = []string{"baz"}
2003
Colin Cross98be1bb2019-12-13 20:41:13 -08002004 ctx := testContext()
Colin Cross50ddcc42019-05-16 12:28:22 -07002005
2006 run(t, ctx, config)
2007
2008 app := ctx.ModuleForTests("app", "android_common")
2009 prebuilt := ctx.ModuleForTests("prebuilt", "android_common")
2010
2011 // Test that all libraries are verified
2012 cmd := app.Rule("verify_uses_libraries").RuleParams.Command
2013 if w := "--uses-library foo"; !strings.Contains(cmd, w) {
2014 t.Errorf("wanted %q in %q", w, cmd)
2015 }
2016
2017 if w := "--optional-uses-library bar --optional-uses-library baz"; !strings.Contains(cmd, w) {
2018 t.Errorf("wanted %q in %q", w, cmd)
2019 }
2020
2021 cmd = prebuilt.Rule("verify_uses_libraries").RuleParams.Command
2022
2023 if w := `uses_library_names="foo"`; !strings.Contains(cmd, w) {
2024 t.Errorf("wanted %q in %q", w, cmd)
2025 }
2026
2027 if w := `optional_uses_library_names="bar baz"`; !strings.Contains(cmd, w) {
2028 t.Errorf("wanted %q in %q", w, cmd)
2029 }
2030
2031 // Test that only present libraries are preopted
2032 cmd = app.Rule("dexpreopt").RuleParams.Command
2033
2034 if w := `dex_preopt_target_libraries="/system/framework/foo.jar /system/framework/bar.jar"`; !strings.Contains(cmd, w) {
2035 t.Errorf("wanted %q in %q", w, cmd)
2036 }
2037
2038 cmd = prebuilt.Rule("dexpreopt").RuleParams.Command
2039
2040 if w := `dex_preopt_target_libraries="/system/framework/foo.jar /system/framework/bar.jar"`; !strings.Contains(cmd, w) {
2041 t.Errorf("wanted %q in %q", w, cmd)
2042 }
2043}
Jaewoong Jungc27ab662019-05-30 15:51:14 -07002044
2045func TestCodelessApp(t *testing.T) {
2046 testCases := []struct {
2047 name string
2048 bp string
2049 noCode bool
2050 }{
2051 {
2052 name: "normal",
2053 bp: `
2054 android_app {
2055 name: "foo",
2056 srcs: ["a.java"],
Jeongik Cha538c0d02019-07-11 15:54:27 +09002057 sdk_version: "current",
Jaewoong Jungc27ab662019-05-30 15:51:14 -07002058 }
2059 `,
2060 noCode: false,
2061 },
2062 {
2063 name: "app without sources",
2064 bp: `
2065 android_app {
2066 name: "foo",
Jeongik Cha538c0d02019-07-11 15:54:27 +09002067 sdk_version: "current",
Jaewoong Jungc27ab662019-05-30 15:51:14 -07002068 }
2069 `,
2070 noCode: true,
2071 },
2072 {
2073 name: "app with libraries",
2074 bp: `
2075 android_app {
2076 name: "foo",
2077 static_libs: ["lib"],
Jeongik Cha538c0d02019-07-11 15:54:27 +09002078 sdk_version: "current",
Jaewoong Jungc27ab662019-05-30 15:51:14 -07002079 }
2080
2081 java_library {
2082 name: "lib",
2083 srcs: ["a.java"],
Jeongik Cha538c0d02019-07-11 15:54:27 +09002084 sdk_version: "current",
Jaewoong Jungc27ab662019-05-30 15:51:14 -07002085 }
2086 `,
2087 noCode: false,
2088 },
2089 {
2090 name: "app with sourceless libraries",
2091 bp: `
2092 android_app {
2093 name: "foo",
2094 static_libs: ["lib"],
Jeongik Cha538c0d02019-07-11 15:54:27 +09002095 sdk_version: "current",
Jaewoong Jungc27ab662019-05-30 15:51:14 -07002096 }
2097
2098 java_library {
2099 name: "lib",
Jeongik Cha538c0d02019-07-11 15:54:27 +09002100 sdk_version: "current",
Jaewoong Jungc27ab662019-05-30 15:51:14 -07002101 }
2102 `,
2103 // TODO(jungjw): this should probably be true
2104 noCode: false,
2105 },
2106 }
2107
2108 for _, test := range testCases {
2109 t.Run(test.name, func(t *testing.T) {
2110 ctx := testApp(t, test.bp)
2111
2112 foo := ctx.ModuleForTests("foo", "android_common")
2113 manifestFixerArgs := foo.Output("manifest_fixer/AndroidManifest.xml").Args["args"]
2114 if strings.Contains(manifestFixerArgs, "--has-no-code") != test.noCode {
2115 t.Errorf("unexpected manifest_fixer args: %q", manifestFixerArgs)
2116 }
2117 })
2118 }
2119}
Jaewoong Jung5b425e22019-06-17 17:40:56 -07002120
2121func TestEmbedNotice(t *testing.T) {
Jaewoong Jungf9a04432019-07-17 11:15:09 -07002122 ctx, _ := testJava(t, cc.GatherRequiredDepsForTest(android.Android)+`
Jaewoong Jung5b425e22019-06-17 17:40:56 -07002123 android_app {
2124 name: "foo",
2125 srcs: ["a.java"],
2126 static_libs: ["javalib"],
2127 jni_libs: ["libjni"],
2128 notice: "APP_NOTICE",
2129 embed_notices: true,
Jeongik Cha538c0d02019-07-11 15:54:27 +09002130 sdk_version: "current",
Jaewoong Jung5b425e22019-06-17 17:40:56 -07002131 }
2132
2133 // No embed_notice flag
2134 android_app {
2135 name: "bar",
2136 srcs: ["a.java"],
2137 jni_libs: ["libjni"],
2138 notice: "APP_NOTICE",
Jeongik Cha538c0d02019-07-11 15:54:27 +09002139 sdk_version: "current",
Jaewoong Jung5b425e22019-06-17 17:40:56 -07002140 }
2141
2142 // No NOTICE files
2143 android_app {
2144 name: "baz",
2145 srcs: ["a.java"],
2146 embed_notices: true,
Jeongik Cha538c0d02019-07-11 15:54:27 +09002147 sdk_version: "current",
Jaewoong Jung5b425e22019-06-17 17:40:56 -07002148 }
2149
2150 cc_library {
2151 name: "libjni",
2152 system_shared_libs: [],
2153 stl: "none",
2154 notice: "LIB_NOTICE",
2155 }
2156
2157 java_library {
2158 name: "javalib",
2159 srcs: [
2160 ":gen",
2161 ],
Jeongik Cha538c0d02019-07-11 15:54:27 +09002162 sdk_version: "current",
Jaewoong Jung5b425e22019-06-17 17:40:56 -07002163 }
2164
2165 genrule {
2166 name: "gen",
2167 tools: ["gentool"],
2168 out: ["gen.java"],
2169 notice: "GENRULE_NOTICE",
2170 }
2171
2172 java_binary_host {
2173 name: "gentool",
2174 srcs: ["b.java"],
2175 notice: "TOOL_NOTICE",
2176 }
2177 `)
2178
2179 // foo has NOTICE files to process, and embed_notices is true.
2180 foo := ctx.ModuleForTests("foo", "android_common")
2181 // verify merge notices rule.
2182 mergeNotices := foo.Rule("mergeNoticesRule")
2183 noticeInputs := mergeNotices.Inputs.Strings()
2184 // TOOL_NOTICE should be excluded as it's a host module.
2185 if len(mergeNotices.Inputs) != 3 {
2186 t.Errorf("number of input notice files: expected = 3, actual = %q", noticeInputs)
2187 }
2188 if !inList("APP_NOTICE", noticeInputs) {
2189 t.Errorf("APP_NOTICE is missing from notice files, %q", noticeInputs)
2190 }
2191 if !inList("LIB_NOTICE", noticeInputs) {
2192 t.Errorf("LIB_NOTICE is missing from notice files, %q", noticeInputs)
2193 }
2194 if !inList("GENRULE_NOTICE", noticeInputs) {
2195 t.Errorf("GENRULE_NOTICE is missing from notice files, %q", noticeInputs)
2196 }
2197 // aapt2 flags should include -A <NOTICE dir> so that its contents are put in the APK's /assets.
2198 res := foo.Output("package-res.apk")
2199 aapt2Flags := res.Args["flags"]
2200 e := "-A " + buildDir + "/.intermediates/foo/android_common/NOTICE"
2201 if !strings.Contains(aapt2Flags, e) {
2202 t.Errorf("asset dir flag for NOTICE, %q is missing in aapt2 link flags, %q", e, aapt2Flags)
2203 }
2204
2205 // bar has NOTICE files to process, but embed_notices is not set.
2206 bar := ctx.ModuleForTests("bar", "android_common")
Jaewoong Jung98772792019-07-01 17:15:13 -07002207 res = bar.Output("package-res.apk")
2208 aapt2Flags = res.Args["flags"]
2209 e = "-A " + buildDir + "/.intermediates/bar/android_common/NOTICE"
2210 if strings.Contains(aapt2Flags, e) {
2211 t.Errorf("bar shouldn't have the asset dir flag for NOTICE: %q", e)
Jaewoong Jung5b425e22019-06-17 17:40:56 -07002212 }
2213
2214 // baz's embed_notice is true, but it doesn't have any NOTICE files.
2215 baz := ctx.ModuleForTests("baz", "android_common")
Jaewoong Jung98772792019-07-01 17:15:13 -07002216 res = baz.Output("package-res.apk")
2217 aapt2Flags = res.Args["flags"]
2218 e = "-A " + buildDir + "/.intermediates/baz/android_common/NOTICE"
2219 if strings.Contains(aapt2Flags, e) {
2220 t.Errorf("baz shouldn't have the asset dir flag for NOTICE: %q", e)
Jaewoong Jung5b425e22019-06-17 17:40:56 -07002221 }
2222}
Colin Cross53a87f52019-06-25 13:35:30 -07002223
2224func TestUncompressDex(t *testing.T) {
2225 testCases := []struct {
2226 name string
2227 bp string
2228
2229 uncompressedPlatform bool
2230 uncompressedUnbundled bool
2231 }{
2232 {
2233 name: "normal",
2234 bp: `
2235 android_app {
2236 name: "foo",
2237 srcs: ["a.java"],
Jeongik Cha538c0d02019-07-11 15:54:27 +09002238 sdk_version: "current",
Colin Cross53a87f52019-06-25 13:35:30 -07002239 }
2240 `,
2241 uncompressedPlatform: true,
2242 uncompressedUnbundled: false,
2243 },
2244 {
2245 name: "use_embedded_dex",
2246 bp: `
2247 android_app {
2248 name: "foo",
2249 use_embedded_dex: true,
2250 srcs: ["a.java"],
Jeongik Cha538c0d02019-07-11 15:54:27 +09002251 sdk_version: "current",
Colin Cross53a87f52019-06-25 13:35:30 -07002252 }
2253 `,
2254 uncompressedPlatform: true,
2255 uncompressedUnbundled: true,
2256 },
2257 {
2258 name: "privileged",
2259 bp: `
2260 android_app {
2261 name: "foo",
2262 privileged: true,
2263 srcs: ["a.java"],
Jeongik Cha538c0d02019-07-11 15:54:27 +09002264 sdk_version: "current",
Colin Cross53a87f52019-06-25 13:35:30 -07002265 }
2266 `,
2267 uncompressedPlatform: true,
2268 uncompressedUnbundled: true,
2269 },
2270 }
2271
2272 test := func(t *testing.T, bp string, want bool, unbundled bool) {
2273 t.Helper()
2274
Colin Cross98be1bb2019-12-13 20:41:13 -08002275 config := testAppConfig(nil, bp, nil)
Colin Cross53a87f52019-06-25 13:35:30 -07002276 if unbundled {
2277 config.TestProductVariables.Unbundled_build = proptools.BoolPtr(true)
2278 }
2279
Colin Cross98be1bb2019-12-13 20:41:13 -08002280 ctx := testContext()
Colin Cross53a87f52019-06-25 13:35:30 -07002281
2282 run(t, ctx, config)
2283
2284 foo := ctx.ModuleForTests("foo", "android_common")
2285 dex := foo.Rule("r8")
2286 uncompressedInDexJar := strings.Contains(dex.Args["zipFlags"], "-L 0")
2287 aligned := foo.MaybeRule("zipalign").Rule != nil
2288
2289 if uncompressedInDexJar != want {
2290 t.Errorf("want uncompressed in dex %v, got %v", want, uncompressedInDexJar)
2291 }
2292
2293 if aligned != want {
2294 t.Errorf("want aligned %v, got %v", want, aligned)
2295 }
2296 }
2297
2298 for _, tt := range testCases {
2299 t.Run(tt.name, func(t *testing.T) {
2300 t.Run("platform", func(t *testing.T) {
2301 test(t, tt.bp, tt.uncompressedPlatform, false)
2302 })
2303 t.Run("unbundled", func(t *testing.T) {
2304 test(t, tt.bp, tt.uncompressedUnbundled, true)
2305 })
2306 })
2307 }
2308}
Jaewoong Jung26dedd32019-06-06 08:45:58 -07002309
2310func checkAapt2LinkFlag(t *testing.T, aapt2Flags, flagName, expectedValue string) {
2311 if expectedValue != "" {
2312 expectedFlag := "--" + flagName + " " + expectedValue
2313 if !strings.Contains(aapt2Flags, expectedFlag) {
2314 t.Errorf("%q is missing in aapt2 link flags, %q", expectedFlag, aapt2Flags)
2315 }
2316 } else {
2317 unexpectedFlag := "--" + flagName
2318 if strings.Contains(aapt2Flags, unexpectedFlag) {
2319 t.Errorf("unexpected flag, %q is found in aapt2 link flags, %q", unexpectedFlag, aapt2Flags)
2320 }
2321 }
2322}
Jaewoong Jung9befb0c2020-01-18 10:33:43 -08002323
2324func TestRuntimeResourceOverlay(t *testing.T) {
2325 ctx, config := testJava(t, `
2326 runtime_resource_overlay {
2327 name: "foo",
2328 certificate: "platform",
2329 product_specific: true,
Jaewoong Jungf0f747c2020-01-24 10:30:02 -08002330 aaptflags: ["--keep-raw-values"],
Jaewoong Jung9befb0c2020-01-18 10:33:43 -08002331 }
2332
2333 runtime_resource_overlay {
2334 name: "foo_themed",
2335 certificate: "platform",
2336 product_specific: true,
2337 theme: "faza",
2338 }
2339 `)
2340
2341 m := ctx.ModuleForTests("foo", "android_common")
2342
Jaewoong Jungf0f747c2020-01-24 10:30:02 -08002343 // Check AAPT2 link flags.
2344 aapt2Flags := m.Output("package-res.apk").Args["flags"]
2345 expectedFlags := []string{"--keep-raw-values", "--no-resource-deduping", "--no-resource-removal"}
2346 absentFlags := android.RemoveListFromList(expectedFlags, strings.Split(aapt2Flags, " "))
2347 if len(absentFlags) > 0 {
2348 t.Errorf("expected values, %q are missing in aapt2 link flags, %q", absentFlags, aapt2Flags)
2349 }
2350
Jaewoong Jung9befb0c2020-01-18 10:33:43 -08002351 // Check cert signing flag.
2352 signedApk := m.Output("signed/foo.apk")
2353 signingFlag := signedApk.Args["certificates"]
2354 expected := "build/make/target/product/security/platform.x509.pem build/make/target/product/security/platform.pk8"
2355 if expected != signingFlag {
2356 t.Errorf("Incorrect signing flags, expected: %q, got: %q", expected, signingFlag)
2357 }
Jaewoong Jung78ec5d82020-01-31 10:11:47 -08002358 path := android.AndroidMkEntriesForTest(t, config, "", m.Module())[0].EntryMap["LOCAL_CERTIFICATE"]
2359 expectedPath := []string{"build/make/target/product/security/platform.x509.pem"}
2360 if !reflect.DeepEqual(path, expectedPath) {
2361 t.Errorf("Unexpected LOCAL_CERTIFICATE value: %v, expected: %v", path, expectedPath)
2362 }
Jaewoong Jung9befb0c2020-01-18 10:33:43 -08002363
2364 // Check device location.
Jaewoong Jung78ec5d82020-01-31 10:11:47 -08002365 path = android.AndroidMkEntriesForTest(t, config, "", m.Module())[0].EntryMap["LOCAL_MODULE_PATH"]
2366 expectedPath = []string{"/tmp/target/product/test_device/product/overlay"}
Jaewoong Jung9befb0c2020-01-18 10:33:43 -08002367 if !reflect.DeepEqual(path, expectedPath) {
2368 t.Errorf("Unexpected LOCAL_MODULE_PATH value: %v, expected: %v", path, expectedPath)
2369 }
2370
2371 // A themed module has a different device location
2372 m = ctx.ModuleForTests("foo_themed", "android_common")
2373 path = android.AndroidMkEntriesForTest(t, config, "", m.Module())[0].EntryMap["LOCAL_MODULE_PATH"]
2374 expectedPath = []string{"/tmp/target/product/test_device/product/overlay/faza"}
2375 if !reflect.DeepEqual(path, expectedPath) {
2376 t.Errorf("Unexpected LOCAL_MODULE_PATH value: %v, expected: %v", path, expectedPath)
2377 }
2378}