blob: 5cf0ba78d12abf03a3cbf06930adb220013b4ac3 [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
Jaewoong Jungf9a04432019-07-17 11:15:09 -070046func testAppContext(bp string, fs map[string][]byte) *android.TestContext {
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
Jaewoong Jungf9a04432019-07-17 11:15:09 -070056 return testContext(bp, appFS)
Colin Cross527012a2017-11-30 22:56:16 -080057}
58
59func testApp(t *testing.T, bp string) *android.TestContext {
60 config := testConfig(nil)
61
Jaewoong Jungf9a04432019-07-17 11:15:09 -070062 ctx := testAppContext(bp, nil)
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) {
304 config := testConfig(nil)
Jaewoong Jungf9a04432019-07-17 11:15:09 -0700305 ctx := testContext(fmt.Sprintf(bp, testCase.prop), fs)
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
Colin Crossbec85302019-02-13 13:15:46 -0800326func TestAndroidResources(t *testing.T) {
Colin Cross5c4791c2019-02-01 11:44:44 -0800327 testCases := []struct {
328 name string
329 enforceRROTargets []string
330 enforceRROExcludedOverlays []string
Colin Crossbec85302019-02-13 13:15:46 -0800331 resourceFiles map[string][]string
Colin Cross5c4791c2019-02-01 11:44:44 -0800332 overlayFiles map[string][]string
333 rroDirs map[string][]string
334 }{
335 {
336 name: "no RRO",
337 enforceRROTargets: nil,
338 enforceRROExcludedOverlays: nil,
Colin Crossbec85302019-02-13 13:15:46 -0800339 resourceFiles: map[string][]string{
340 "foo": nil,
341 "bar": {"bar/res/res/values/strings.xml"},
342 "lib": nil,
343 "lib2": {"lib2/res/res/values/strings.xml"},
344 },
Colin Cross5c4791c2019-02-01 11:44:44 -0800345 overlayFiles: map[string][]string{
Colin Crossbec85302019-02-13 13:15:46 -0800346 "foo": {
347 buildDir + "/.intermediates/lib2/android_common/package-res.apk",
Colin Cross6ed7dea2019-01-31 14:44:30 -0800348 buildDir + "/.intermediates/lib/android_common/package-res.apk",
Anton Hansson53c88442019-03-18 15:53:16 +0000349 buildDir + "/.intermediates/lib3/android_common/package-res.apk",
Colin Cross6ed7dea2019-01-31 14:44:30 -0800350 "foo/res/res/values/strings.xml",
Colin Cross5c4791c2019-02-01 11:44:44 -0800351 "device/vendor/blah/static_overlay/foo/res/values/strings.xml",
352 "device/vendor/blah/overlay/foo/res/values/strings.xml",
Anton Hansson53c88442019-03-18 15:53:16 +0000353 "product/vendor/blah/overlay/foo/res/values/strings.xml",
Colin Cross5c4791c2019-02-01 11:44:44 -0800354 },
Colin Crossbec85302019-02-13 13:15:46 -0800355 "bar": {
Colin Cross5c4791c2019-02-01 11:44:44 -0800356 "device/vendor/blah/static_overlay/bar/res/values/strings.xml",
357 "device/vendor/blah/overlay/bar/res/values/strings.xml",
358 },
Colin Crossbec85302019-02-13 13:15:46 -0800359 "lib": {
360 buildDir + "/.intermediates/lib2/android_common/package-res.apk",
361 "lib/res/res/values/strings.xml",
362 "device/vendor/blah/overlay/lib/res/values/strings.xml",
363 },
Colin Cross5c4791c2019-02-01 11:44:44 -0800364 },
365 rroDirs: map[string][]string{
366 "foo": nil,
367 "bar": nil,
368 },
369 },
370 {
371 name: "enforce RRO on foo",
372 enforceRROTargets: []string{"foo"},
373 enforceRROExcludedOverlays: []string{"device/vendor/blah/static_overlay"},
Colin Crossbec85302019-02-13 13:15:46 -0800374 resourceFiles: map[string][]string{
375 "foo": nil,
376 "bar": {"bar/res/res/values/strings.xml"},
377 "lib": nil,
378 "lib2": {"lib2/res/res/values/strings.xml"},
379 },
Colin Cross5c4791c2019-02-01 11:44:44 -0800380 overlayFiles: map[string][]string{
Colin Crossbec85302019-02-13 13:15:46 -0800381 "foo": {
382 buildDir + "/.intermediates/lib2/android_common/package-res.apk",
Colin Cross6ed7dea2019-01-31 14:44:30 -0800383 buildDir + "/.intermediates/lib/android_common/package-res.apk",
Anton Hansson53c88442019-03-18 15:53:16 +0000384 buildDir + "/.intermediates/lib3/android_common/package-res.apk",
Colin Cross6ed7dea2019-01-31 14:44:30 -0800385 "foo/res/res/values/strings.xml",
386 "device/vendor/blah/static_overlay/foo/res/values/strings.xml",
387 },
Colin Crossbec85302019-02-13 13:15:46 -0800388 "bar": {
Colin Cross5c4791c2019-02-01 11:44:44 -0800389 "device/vendor/blah/static_overlay/bar/res/values/strings.xml",
390 "device/vendor/blah/overlay/bar/res/values/strings.xml",
391 },
Colin Crossbec85302019-02-13 13:15:46 -0800392 "lib": {
393 buildDir + "/.intermediates/lib2/android_common/package-res.apk",
394 "lib/res/res/values/strings.xml",
395 "device/vendor/blah/overlay/lib/res/values/strings.xml",
396 },
Colin Cross5c4791c2019-02-01 11:44:44 -0800397 },
Colin Crossc1c37552019-01-31 11:42:41 -0800398
Colin Cross5c4791c2019-02-01 11:44:44 -0800399 rroDirs: map[string][]string{
Colin Crossbec85302019-02-13 13:15:46 -0800400 "foo": {
Anton Hansson53c88442019-03-18 15:53:16 +0000401 "device:device/vendor/blah/overlay/foo/res",
Colin Crossc1c37552019-01-31 11:42:41 -0800402 // Enforce RRO on "foo" could imply RRO on static dependencies, but for now it doesn't.
403 // "device/vendor/blah/overlay/lib/res",
Anton Hansson53c88442019-03-18 15:53:16 +0000404 "product:product/vendor/blah/overlay/foo/res",
Colin Crossc1c37552019-01-31 11:42:41 -0800405 },
Colin Cross5c4791c2019-02-01 11:44:44 -0800406 "bar": nil,
Colin Crossbec85302019-02-13 13:15:46 -0800407 "lib": nil,
Colin Cross5c4791c2019-02-01 11:44:44 -0800408 },
409 },
410 {
411 name: "enforce RRO on all",
412 enforceRROTargets: []string{"*"},
413 enforceRROExcludedOverlays: []string{
414 // Excluding specific apps/res directories also allowed.
415 "device/vendor/blah/static_overlay/foo",
416 "device/vendor/blah/static_overlay/bar/res",
417 },
Colin Crossbec85302019-02-13 13:15:46 -0800418 resourceFiles: map[string][]string{
419 "foo": nil,
420 "bar": {"bar/res/res/values/strings.xml"},
421 "lib": nil,
422 "lib2": {"lib2/res/res/values/strings.xml"},
423 },
Colin Cross5c4791c2019-02-01 11:44:44 -0800424 overlayFiles: map[string][]string{
Colin Crossbec85302019-02-13 13:15:46 -0800425 "foo": {
426 buildDir + "/.intermediates/lib2/android_common/package-res.apk",
Colin Cross6ed7dea2019-01-31 14:44:30 -0800427 buildDir + "/.intermediates/lib/android_common/package-res.apk",
Anton Hansson53c88442019-03-18 15:53:16 +0000428 buildDir + "/.intermediates/lib3/android_common/package-res.apk",
Colin Cross6ed7dea2019-01-31 14:44:30 -0800429 "foo/res/res/values/strings.xml",
430 "device/vendor/blah/static_overlay/foo/res/values/strings.xml",
431 },
Colin Crossbec85302019-02-13 13:15:46 -0800432 "bar": {"device/vendor/blah/static_overlay/bar/res/values/strings.xml"},
433 "lib": {
434 buildDir + "/.intermediates/lib2/android_common/package-res.apk",
435 "lib/res/res/values/strings.xml",
436 },
Colin Cross5c4791c2019-02-01 11:44:44 -0800437 },
438 rroDirs: map[string][]string{
Colin Crossbec85302019-02-13 13:15:46 -0800439 "foo": {
Anton Hansson53c88442019-03-18 15:53:16 +0000440 "device:device/vendor/blah/overlay/foo/res",
441 "product:product/vendor/blah/overlay/foo/res",
442 // Lib dep comes after the direct deps
443 "device:device/vendor/blah/overlay/lib/res",
Colin Crossc1c37552019-01-31 11:42:41 -0800444 },
Anton Hansson53c88442019-03-18 15:53:16 +0000445 "bar": {"device:device/vendor/blah/overlay/bar/res"},
446 "lib": {"device:device/vendor/blah/overlay/lib/res"},
Colin Cross5c4791c2019-02-01 11:44:44 -0800447 },
448 },
449 }
450
Anton Hansson53c88442019-03-18 15:53:16 +0000451 deviceResourceOverlays := []string{
Colin Cross890ff552017-11-30 20:13:19 -0800452 "device/vendor/blah/overlay",
453 "device/vendor/blah/overlay2",
454 "device/vendor/blah/static_overlay",
455 }
456
Anton Hansson53c88442019-03-18 15:53:16 +0000457 productResourceOverlays := []string{
458 "product/vendor/blah/overlay",
459 }
460
Colin Cross890ff552017-11-30 20:13:19 -0800461 fs := map[string][]byte{
462 "foo/res/res/values/strings.xml": nil,
463 "bar/res/res/values/strings.xml": nil,
Colin Cross6ed7dea2019-01-31 14:44:30 -0800464 "lib/res/res/values/strings.xml": nil,
Colin Crossbec85302019-02-13 13:15:46 -0800465 "lib2/res/res/values/strings.xml": nil,
Colin Cross890ff552017-11-30 20:13:19 -0800466 "device/vendor/blah/overlay/foo/res/values/strings.xml": nil,
467 "device/vendor/blah/overlay/bar/res/values/strings.xml": nil,
Colin Cross6ed7dea2019-01-31 14:44:30 -0800468 "device/vendor/blah/overlay/lib/res/values/strings.xml": nil,
Colin Cross890ff552017-11-30 20:13:19 -0800469 "device/vendor/blah/static_overlay/foo/res/values/strings.xml": nil,
470 "device/vendor/blah/static_overlay/bar/res/values/strings.xml": nil,
471 "device/vendor/blah/overlay2/res/values/strings.xml": nil,
Anton Hansson53c88442019-03-18 15:53:16 +0000472 "product/vendor/blah/overlay/foo/res/values/strings.xml": nil,
Colin Cross890ff552017-11-30 20:13:19 -0800473 }
474
475 bp := `
476 android_app {
477 name: "foo",
Jeongik Cha538c0d02019-07-11 15:54:27 +0900478 sdk_version: "current",
Colin Cross890ff552017-11-30 20:13:19 -0800479 resource_dirs: ["foo/res"],
Anton Hansson53c88442019-03-18 15:53:16 +0000480 static_libs: ["lib", "lib3"],
Colin Cross890ff552017-11-30 20:13:19 -0800481 }
482
483 android_app {
484 name: "bar",
Jeongik Cha538c0d02019-07-11 15:54:27 +0900485 sdk_version: "current",
Colin Cross890ff552017-11-30 20:13:19 -0800486 resource_dirs: ["bar/res"],
487 }
Colin Cross6ed7dea2019-01-31 14:44:30 -0800488
489 android_library {
490 name: "lib",
Jeongik Cha75b83b02019-11-01 15:28:00 +0900491 sdk_version: "current",
Colin Cross6ed7dea2019-01-31 14:44:30 -0800492 resource_dirs: ["lib/res"],
Colin Crossbec85302019-02-13 13:15:46 -0800493 static_libs: ["lib2"],
494 }
495
496 android_library {
497 name: "lib2",
Jeongik Cha75b83b02019-11-01 15:28:00 +0900498 sdk_version: "current",
Colin Crossbec85302019-02-13 13:15:46 -0800499 resource_dirs: ["lib2/res"],
Colin Cross6ed7dea2019-01-31 14:44:30 -0800500 }
Anton Hansson53c88442019-03-18 15:53:16 +0000501
502 // This library has the same resources as lib (should not lead to dupe RROs)
503 android_library {
504 name: "lib3",
Jeongik Cha75b83b02019-11-01 15:28:00 +0900505 sdk_version: "current",
Anton Hansson53c88442019-03-18 15:53:16 +0000506 resource_dirs: ["lib/res"]
507 }
Colin Cross890ff552017-11-30 20:13:19 -0800508 `
509
Colin Cross5c4791c2019-02-01 11:44:44 -0800510 for _, testCase := range testCases {
Colin Cross890ff552017-11-30 20:13:19 -0800511 t.Run(testCase.name, func(t *testing.T) {
512 config := testConfig(nil)
Anton Hansson53c88442019-03-18 15:53:16 +0000513 config.TestProductVariables.DeviceResourceOverlays = deviceResourceOverlays
514 config.TestProductVariables.ProductResourceOverlays = productResourceOverlays
Colin Cross890ff552017-11-30 20:13:19 -0800515 if testCase.enforceRROTargets != nil {
Colin Crossa74ca042019-01-31 14:31:51 -0800516 config.TestProductVariables.EnforceRROTargets = testCase.enforceRROTargets
Colin Cross890ff552017-11-30 20:13:19 -0800517 }
518 if testCase.enforceRROExcludedOverlays != nil {
Colin Crossa74ca042019-01-31 14:31:51 -0800519 config.TestProductVariables.EnforceRROExcludedOverlays = testCase.enforceRROExcludedOverlays
Colin Cross890ff552017-11-30 20:13:19 -0800520 }
521
Jaewoong Jungf9a04432019-07-17 11:15:09 -0700522 ctx := testAppContext(bp, fs)
Colin Cross890ff552017-11-30 20:13:19 -0800523 run(t, ctx, config)
524
Colin Crossbec85302019-02-13 13:15:46 -0800525 resourceListToFiles := func(module android.TestingModule, list []string) (files []string) {
526 for _, o := range list {
527 res := module.MaybeOutput(o)
528 if res.Rule != nil {
529 // If the overlay is compiled as part of this module (i.e. a .arsc.flat file),
530 // verify the inputs to the .arsc.flat rule.
531 files = append(files, res.Inputs.Strings()...)
532 } else {
533 // Otherwise, verify the full path to the output of the other module
534 files = append(files, o)
Anton Hansson94c93f32019-01-30 16:03:37 +0000535 }
Colin Cross890ff552017-11-30 20:13:19 -0800536 }
Colin Crossbec85302019-02-13 13:15:46 -0800537 return files
Colin Cross890ff552017-11-30 20:13:19 -0800538 }
539
Colin Crossbec85302019-02-13 13:15:46 -0800540 getResources := func(moduleName string) (resourceFiles, overlayFiles, rroDirs []string) {
541 module := ctx.ModuleForTests(moduleName, "android_common")
542 resourceList := module.MaybeOutput("aapt2/res.list")
543 if resourceList.Rule != nil {
544 resourceFiles = resourceListToFiles(module, resourceList.Inputs.Strings())
Anton Hansson0375a4f2019-01-24 14:39:19 +0000545 }
Colin Crossbec85302019-02-13 13:15:46 -0800546 overlayList := module.MaybeOutput("aapt2/overlay.list")
547 if overlayList.Rule != nil {
548 overlayFiles = resourceListToFiles(module, overlayList.Inputs.Strings())
549 }
550
Anton Hansson53c88442019-03-18 15:53:16 +0000551 for _, d := range module.Module().(AndroidLibraryDependency).ExportedRRODirs() {
552 var prefix string
553 if d.overlayType == device {
554 prefix = "device:"
555 } else if d.overlayType == product {
556 prefix = "product:"
557 } else {
558 t.Fatalf("Unexpected overlayType %d", d.overlayType)
559 }
560 rroDirs = append(rroDirs, prefix+d.path.String())
561 }
Colin Crossbec85302019-02-13 13:15:46 -0800562
563 return resourceFiles, overlayFiles, rroDirs
564 }
565
566 modules := []string{"foo", "bar", "lib", "lib2"}
567 for _, module := range modules {
568 resourceFiles, overlayFiles, rroDirs := getResources(module)
569
570 if !reflect.DeepEqual(resourceFiles, testCase.resourceFiles[module]) {
571 t.Errorf("expected %s resource files:\n %#v\n got:\n %#v",
572 module, testCase.resourceFiles[module], resourceFiles)
573 }
574 if !reflect.DeepEqual(overlayFiles, testCase.overlayFiles[module]) {
575 t.Errorf("expected %s overlay files:\n %#v\n got:\n %#v",
576 module, testCase.overlayFiles[module], overlayFiles)
577 }
578 if !reflect.DeepEqual(rroDirs, testCase.rroDirs[module]) {
Anton Hansson0375a4f2019-01-24 14:39:19 +0000579 t.Errorf("expected %s rroDirs: %#v\n got:\n %#v",
Colin Crossbec85302019-02-13 13:15:46 -0800580 module, testCase.rroDirs[module], rroDirs)
Anton Hansson0375a4f2019-01-24 14:39:19 +0000581 }
Colin Cross890ff552017-11-30 20:13:19 -0800582 }
Colin Cross890ff552017-11-30 20:13:19 -0800583 })
584 }
585}
Colin Crossd09b0b62018-04-18 11:06:47 -0700586
587func TestAppSdkVersion(t *testing.T) {
588 testCases := []struct {
589 name string
590 sdkVersion string
591 platformSdkInt int
592 platformSdkCodename string
593 platformSdkFinal bool
594 expectedMinSdkVersion string
Jeongik Cha538c0d02019-07-11 15:54:27 +0900595 platformApis bool
Colin Crossd09b0b62018-04-18 11:06:47 -0700596 }{
597 {
598 name: "current final SDK",
599 sdkVersion: "current",
600 platformSdkInt: 27,
601 platformSdkCodename: "REL",
602 platformSdkFinal: true,
603 expectedMinSdkVersion: "27",
604 },
605 {
606 name: "current non-final SDK",
607 sdkVersion: "current",
608 platformSdkInt: 27,
609 platformSdkCodename: "OMR1",
610 platformSdkFinal: false,
611 expectedMinSdkVersion: "OMR1",
612 },
613 {
614 name: "default final SDK",
615 sdkVersion: "",
Jeongik Cha538c0d02019-07-11 15:54:27 +0900616 platformApis: true,
Colin Crossd09b0b62018-04-18 11:06:47 -0700617 platformSdkInt: 27,
618 platformSdkCodename: "REL",
619 platformSdkFinal: true,
620 expectedMinSdkVersion: "27",
621 },
622 {
623 name: "default non-final SDK",
624 sdkVersion: "",
Jeongik Cha538c0d02019-07-11 15:54:27 +0900625 platformApis: true,
Colin Crossd09b0b62018-04-18 11:06:47 -0700626 platformSdkInt: 27,
627 platformSdkCodename: "OMR1",
628 platformSdkFinal: false,
629 expectedMinSdkVersion: "OMR1",
630 },
631 {
632 name: "14",
633 sdkVersion: "14",
634 expectedMinSdkVersion: "14",
635 },
636 }
637
638 for _, moduleType := range []string{"android_app", "android_library"} {
639 for _, test := range testCases {
640 t.Run(moduleType+" "+test.name, func(t *testing.T) {
Jeongik Cha538c0d02019-07-11 15:54:27 +0900641 platformApiProp := ""
642 if test.platformApis {
643 platformApiProp = "platform_apis: true,"
644 }
Colin Crossd09b0b62018-04-18 11:06:47 -0700645 bp := fmt.Sprintf(`%s {
646 name: "foo",
647 srcs: ["a.java"],
648 sdk_version: "%s",
Jeongik Cha538c0d02019-07-11 15:54:27 +0900649 %s
650 }`, moduleType, test.sdkVersion, platformApiProp)
Colin Crossd09b0b62018-04-18 11:06:47 -0700651
652 config := testConfig(nil)
653 config.TestProductVariables.Platform_sdk_version = &test.platformSdkInt
654 config.TestProductVariables.Platform_sdk_codename = &test.platformSdkCodename
655 config.TestProductVariables.Platform_sdk_final = &test.platformSdkFinal
656
Jaewoong Jungf9a04432019-07-17 11:15:09 -0700657 ctx := testAppContext(bp, nil)
Colin Crossd09b0b62018-04-18 11:06:47 -0700658
659 run(t, ctx, config)
660
661 foo := ctx.ModuleForTests("foo", "android_common")
662 link := foo.Output("package-res.apk")
663 linkFlags := strings.Split(link.Args["flags"], " ")
664 min := android.IndexList("--min-sdk-version", linkFlags)
665 target := android.IndexList("--target-sdk-version", linkFlags)
666
667 if min == -1 || target == -1 || min == len(linkFlags)-1 || target == len(linkFlags)-1 {
668 t.Fatalf("missing --min-sdk-version or --target-sdk-version in link flags: %q", linkFlags)
669 }
670
671 gotMinSdkVersion := linkFlags[min+1]
672 gotTargetSdkVersion := linkFlags[target+1]
673
674 if gotMinSdkVersion != test.expectedMinSdkVersion {
675 t.Errorf("incorrect --min-sdk-version, expected %q got %q",
676 test.expectedMinSdkVersion, gotMinSdkVersion)
677 }
678
679 if gotTargetSdkVersion != test.expectedMinSdkVersion {
680 t.Errorf("incorrect --target-sdk-version, expected %q got %q",
681 test.expectedMinSdkVersion, gotTargetSdkVersion)
682 }
683 })
684 }
685 }
686}
Colin Crossa4f08812018-10-02 22:03:40 -0700687
Paul Duffin50c217c2019-06-12 13:25:22 +0100688func TestJNIABI(t *testing.T) {
Jaewoong Jungf9a04432019-07-17 11:15:09 -0700689 ctx, _ := testJava(t, cc.GatherRequiredDepsForTest(android.Android)+`
Paul Duffin50c217c2019-06-12 13:25:22 +0100690 cc_library {
691 name: "libjni",
692 system_shared_libs: [],
693 stl: "none",
694 }
695
696 android_test {
697 name: "test",
698 sdk_version: "core_platform",
699 jni_libs: ["libjni"],
700 }
701
702 android_test {
703 name: "test_first",
704 sdk_version: "core_platform",
705 compile_multilib: "first",
706 jni_libs: ["libjni"],
707 }
708
709 android_test {
710 name: "test_both",
711 sdk_version: "core_platform",
712 compile_multilib: "both",
713 jni_libs: ["libjni"],
714 }
715
716 android_test {
717 name: "test_32",
718 sdk_version: "core_platform",
719 compile_multilib: "32",
720 jni_libs: ["libjni"],
721 }
722
723 android_test {
724 name: "test_64",
725 sdk_version: "core_platform",
726 compile_multilib: "64",
727 jni_libs: ["libjni"],
728 }
729 `)
730
731 testCases := []struct {
732 name string
733 abis []string
734 }{
735 {"test", []string{"arm64-v8a"}},
736 {"test_first", []string{"arm64-v8a"}},
737 {"test_both", []string{"arm64-v8a", "armeabi-v7a"}},
738 {"test_32", []string{"armeabi-v7a"}},
739 {"test_64", []string{"arm64-v8a"}},
740 }
741
742 for _, test := range testCases {
743 t.Run(test.name, func(t *testing.T) {
744 app := ctx.ModuleForTests(test.name, "android_common")
745 jniLibZip := app.Output("jnilibs.zip")
746 var abis []string
747 args := strings.Fields(jniLibZip.Args["jarArgs"])
748 for i := 0; i < len(args); i++ {
749 if args[i] == "-P" {
750 abis = append(abis, filepath.Base(args[i+1]))
751 i++
752 }
753 }
754 if !reflect.DeepEqual(abis, test.abis) {
755 t.Errorf("want abis %v, got %v", test.abis, abis)
756 }
757 })
758 }
759}
760
Jeongik Cha2cc570d2019-10-29 15:44:45 +0900761func TestAppSdkVersionByPartition(t *testing.T) {
762 testJavaError(t, "sdk_version must have a value when the module is located at vendor or product", `
763 android_app {
764 name: "foo",
765 srcs: ["a.java"],
766 vendor: true,
767 platform_apis: true,
768 }
769 `)
770
771 testJava(t, `
772 android_app {
773 name: "bar",
774 srcs: ["b.java"],
775 platform_apis: true,
776 }
777 `)
778
779 for _, enforce := range []bool{true, false} {
780
781 config := testConfig(nil)
782 config.TestProductVariables.EnforceProductPartitionInterface = proptools.BoolPtr(enforce)
783 bp := `
784 android_app {
785 name: "foo",
786 srcs: ["a.java"],
787 product_specific: true,
788 platform_apis: true,
789 }
790 `
791 if enforce {
792 testJavaErrorWithConfig(t, "sdk_version must have a value when the module is located at vendor or product", bp, config)
793 } else {
794 testJavaWithConfig(t, bp, config)
795 }
796 }
797}
798
Paul Duffin50c217c2019-06-12 13:25:22 +0100799func TestJNIPackaging(t *testing.T) {
Jaewoong Jungf9a04432019-07-17 11:15:09 -0700800 ctx, _ := testJava(t, cc.GatherRequiredDepsForTest(android.Android)+`
Paul Duffin50c217c2019-06-12 13:25:22 +0100801 cc_library {
802 name: "libjni",
803 system_shared_libs: [],
804 stl: "none",
805 }
806
807 android_app {
808 name: "app",
809 jni_libs: ["libjni"],
Jeongik Cha538c0d02019-07-11 15:54:27 +0900810 sdk_version: "current",
Paul Duffin50c217c2019-06-12 13:25:22 +0100811 }
812
813 android_app {
814 name: "app_noembed",
815 jni_libs: ["libjni"],
816 use_embedded_native_libs: false,
Jeongik Cha538c0d02019-07-11 15:54:27 +0900817 sdk_version: "current",
Paul Duffin50c217c2019-06-12 13:25:22 +0100818 }
819
820 android_app {
821 name: "app_embed",
822 jni_libs: ["libjni"],
823 use_embedded_native_libs: true,
Jeongik Cha538c0d02019-07-11 15:54:27 +0900824 sdk_version: "current",
Paul Duffin50c217c2019-06-12 13:25:22 +0100825 }
826
827 android_test {
828 name: "test",
829 sdk_version: "core_platform",
830 jni_libs: ["libjni"],
831 }
832
833 android_test {
834 name: "test_noembed",
835 sdk_version: "core_platform",
836 jni_libs: ["libjni"],
837 use_embedded_native_libs: false,
838 }
839
840 android_test_helper_app {
841 name: "test_helper",
842 sdk_version: "core_platform",
843 jni_libs: ["libjni"],
844 }
845
846 android_test_helper_app {
847 name: "test_helper_noembed",
848 sdk_version: "core_platform",
849 jni_libs: ["libjni"],
850 use_embedded_native_libs: false,
851 }
852 `)
853
854 testCases := []struct {
855 name string
856 packaged bool
857 compressed bool
858 }{
859 {"app", false, false},
860 {"app_noembed", false, false},
861 {"app_embed", true, false},
862 {"test", true, false},
863 {"test_noembed", true, true},
864 {"test_helper", true, false},
865 {"test_helper_noembed", true, true},
866 }
867
868 for _, test := range testCases {
869 t.Run(test.name, func(t *testing.T) {
870 app := ctx.ModuleForTests(test.name, "android_common")
871 jniLibZip := app.MaybeOutput("jnilibs.zip")
872 if g, w := (jniLibZip.Rule != nil), test.packaged; g != w {
873 t.Errorf("expected jni packaged %v, got %v", w, g)
874 }
875
876 if jniLibZip.Rule != nil {
877 if g, w := !strings.Contains(jniLibZip.Args["jarArgs"], "-L 0"), test.compressed; g != w {
878 t.Errorf("expected jni compressed %v, got %v", w, g)
879 }
880 }
881 })
882 }
Colin Cross47fa9d32019-03-26 10:51:39 -0700883}
884
Jaewoong Jung2ad817c2019-01-18 14:27:16 -0800885func TestCertificates(t *testing.T) {
886 testCases := []struct {
887 name string
888 bp string
889 certificateOverride string
890 expected string
891 }{
892 {
893 name: "default",
894 bp: `
895 android_app {
896 name: "foo",
897 srcs: ["a.java"],
Jeongik Cha538c0d02019-07-11 15:54:27 +0900898 sdk_version: "current",
Jaewoong Jung2ad817c2019-01-18 14:27:16 -0800899 }
900 `,
901 certificateOverride: "",
Dan Willemsen412160e2019-04-09 21:36:26 -0700902 expected: "build/make/target/product/security/testkey.x509.pem build/make/target/product/security/testkey.pk8",
Jaewoong Jung2ad817c2019-01-18 14:27:16 -0800903 },
904 {
905 name: "module certificate property",
906 bp: `
907 android_app {
908 name: "foo",
909 srcs: ["a.java"],
Jeongik Cha538c0d02019-07-11 15:54:27 +0900910 certificate: ":new_certificate",
911 sdk_version: "current",
Jaewoong Jung2ad817c2019-01-18 14:27:16 -0800912 }
913
914 android_app_certificate {
915 name: "new_certificate",
916 certificate: "cert/new_cert",
917 }
918 `,
919 certificateOverride: "",
920 expected: "cert/new_cert.x509.pem cert/new_cert.pk8",
921 },
922 {
923 name: "path certificate property",
924 bp: `
925 android_app {
926 name: "foo",
927 srcs: ["a.java"],
Jeongik Cha538c0d02019-07-11 15:54:27 +0900928 certificate: "expiredkey",
929 sdk_version: "current",
Jaewoong Jung2ad817c2019-01-18 14:27:16 -0800930 }
931 `,
932 certificateOverride: "",
Dan Willemsen412160e2019-04-09 21:36:26 -0700933 expected: "build/make/target/product/security/expiredkey.x509.pem build/make/target/product/security/expiredkey.pk8",
Jaewoong Jung2ad817c2019-01-18 14:27:16 -0800934 },
935 {
936 name: "certificate overrides",
937 bp: `
938 android_app {
939 name: "foo",
940 srcs: ["a.java"],
Jeongik Cha538c0d02019-07-11 15:54:27 +0900941 certificate: "expiredkey",
942 sdk_version: "current",
Jaewoong Jung2ad817c2019-01-18 14:27:16 -0800943 }
944
945 android_app_certificate {
946 name: "new_certificate",
947 certificate: "cert/new_cert",
948 }
949 `,
950 certificateOverride: "foo:new_certificate",
951 expected: "cert/new_cert.x509.pem cert/new_cert.pk8",
952 },
953 }
954
955 for _, test := range testCases {
956 t.Run(test.name, func(t *testing.T) {
957 config := testConfig(nil)
958 if test.certificateOverride != "" {
959 config.TestProductVariables.CertificateOverrides = []string{test.certificateOverride}
960 }
Jaewoong Jungf9a04432019-07-17 11:15:09 -0700961 ctx := testAppContext(test.bp, nil)
Jaewoong Jung2ad817c2019-01-18 14:27:16 -0800962
963 run(t, ctx, config)
964 foo := ctx.ModuleForTests("foo", "android_common")
965
966 signapk := foo.Output("foo.apk")
967 signFlags := signapk.Args["certificates"]
968 if test.expected != signFlags {
969 t.Errorf("Incorrect signing flags, expected: %q, got: %q", test.expected, signFlags)
970 }
971 })
972 }
973}
Jaewoong Jung9d22a912019-01-23 16:27:47 -0800974
975func TestPackageNameOverride(t *testing.T) {
976 testCases := []struct {
977 name string
978 bp string
979 packageNameOverride string
980 expected []string
981 }{
982 {
983 name: "default",
984 bp: `
985 android_app {
986 name: "foo",
987 srcs: ["a.java"],
Jeongik Cha538c0d02019-07-11 15:54:27 +0900988 sdk_version: "current",
Jaewoong Jung9d22a912019-01-23 16:27:47 -0800989 }
990 `,
991 packageNameOverride: "",
992 expected: []string{
993 buildDir + "/.intermediates/foo/android_common/foo.apk",
994 buildDir + "/target/product/test_device/system/app/foo/foo.apk",
995 },
996 },
997 {
998 name: "overridden",
999 bp: `
1000 android_app {
1001 name: "foo",
1002 srcs: ["a.java"],
Jeongik Cha538c0d02019-07-11 15:54:27 +09001003 sdk_version: "current",
Jaewoong Jung9d22a912019-01-23 16:27:47 -08001004 }
1005 `,
1006 packageNameOverride: "foo:bar",
1007 expected: []string{
1008 // The package apk should be still be the original name for test dependencies.
Jaewoong Jung5a498812019-11-07 14:14:38 -08001009 buildDir + "/.intermediates/foo/android_common/bar.apk",
Jaewoong Jung9d22a912019-01-23 16:27:47 -08001010 buildDir + "/target/product/test_device/system/app/bar/bar.apk",
1011 },
1012 },
1013 }
1014
1015 for _, test := range testCases {
1016 t.Run(test.name, func(t *testing.T) {
1017 config := testConfig(nil)
1018 if test.packageNameOverride != "" {
1019 config.TestProductVariables.PackageNameOverrides = []string{test.packageNameOverride}
1020 }
Jaewoong Jungf9a04432019-07-17 11:15:09 -07001021 ctx := testAppContext(test.bp, nil)
Jaewoong Jung9d22a912019-01-23 16:27:47 -08001022
1023 run(t, ctx, config)
1024 foo := ctx.ModuleForTests("foo", "android_common")
1025
1026 outputs := foo.AllOutputs()
1027 outputMap := make(map[string]bool)
1028 for _, o := range outputs {
1029 outputMap[o] = true
1030 }
1031 for _, e := range test.expected {
1032 if _, exist := outputMap[e]; !exist {
1033 t.Errorf("Can't find %q in output files.\nAll outputs:%v", e, outputs)
1034 }
1035 }
1036 })
1037 }
1038}
Jaewoong Jung4102e5d2019-02-27 16:26:28 -08001039
1040func TestInstrumentationTargetOverridden(t *testing.T) {
1041 bp := `
1042 android_app {
1043 name: "foo",
1044 srcs: ["a.java"],
Jeongik Cha538c0d02019-07-11 15:54:27 +09001045 sdk_version: "current",
Jaewoong Jung4102e5d2019-02-27 16:26:28 -08001046 }
1047
1048 android_test {
1049 name: "bar",
1050 instrumentation_for: "foo",
Jeongik Cha538c0d02019-07-11 15:54:27 +09001051 sdk_version: "current",
Jaewoong Jung4102e5d2019-02-27 16:26:28 -08001052 }
1053 `
1054 config := testConfig(nil)
1055 config.TestProductVariables.ManifestPackageNameOverrides = []string{"foo:org.dandroid.bp"}
Jaewoong Jungf9a04432019-07-17 11:15:09 -07001056 ctx := testAppContext(bp, nil)
Jaewoong Jung4102e5d2019-02-27 16:26:28 -08001057
1058 run(t, ctx, config)
1059
1060 bar := ctx.ModuleForTests("bar", "android_common")
1061 res := bar.Output("package-res.apk")
1062 aapt2Flags := res.Args["flags"]
1063 e := "--rename-instrumentation-target-package org.dandroid.bp"
1064 if !strings.Contains(aapt2Flags, e) {
1065 t.Errorf("target package renaming flag, %q is missing in aapt2 link flags, %q", e, aapt2Flags)
1066 }
1067}
Jaewoong Jung525443a2019-02-28 15:35:54 -08001068
1069func TestOverrideAndroidApp(t *testing.T) {
Jaewoong Jungf9a04432019-07-17 11:15:09 -07001070 ctx, _ := testJava(t, `
Jaewoong Jung525443a2019-02-28 15:35:54 -08001071 android_app {
1072 name: "foo",
1073 srcs: ["a.java"],
Jaewoong Junga641ee92019-03-27 11:17:14 -07001074 certificate: "expiredkey",
Jaewoong Jungb639a6a2019-05-10 15:16:29 -07001075 overrides: ["qux"],
Jeongik Cha538c0d02019-07-11 15:54:27 +09001076 sdk_version: "current",
Jaewoong Jung525443a2019-02-28 15:35:54 -08001077 }
1078
1079 override_android_app {
1080 name: "bar",
1081 base: "foo",
1082 certificate: ":new_certificate",
1083 }
1084
1085 android_app_certificate {
1086 name: "new_certificate",
1087 certificate: "cert/new_cert",
1088 }
Jaewoong Jung6f373f62019-03-13 10:13:24 -07001089
1090 override_android_app {
1091 name: "baz",
1092 base: "foo",
1093 package_name: "org.dandroid.bp",
1094 }
Jaewoong Jung525443a2019-02-28 15:35:54 -08001095 `)
1096
1097 expectedVariants := []struct {
Jaewoong Jungb639a6a2019-05-10 15:16:29 -07001098 moduleName string
Jaewoong Jung525443a2019-02-28 15:35:54 -08001099 variantName string
1100 apkName string
1101 apkPath string
1102 signFlag string
1103 overrides []string
Jaewoong Jung6f373f62019-03-13 10:13:24 -07001104 aaptFlag string
Jaewoong Jung525443a2019-02-28 15:35:54 -08001105 }{
1106 {
Jaewoong Jungb639a6a2019-05-10 15:16:29 -07001107 moduleName: "foo",
Jaewoong Jung525443a2019-02-28 15:35:54 -08001108 variantName: "android_common",
1109 apkPath: "/target/product/test_device/system/app/foo/foo.apk",
Dan Willemsen412160e2019-04-09 21:36:26 -07001110 signFlag: "build/make/target/product/security/expiredkey.x509.pem build/make/target/product/security/expiredkey.pk8",
Jaewoong Jungb639a6a2019-05-10 15:16:29 -07001111 overrides: []string{"qux"},
Jaewoong Jung6f373f62019-03-13 10:13:24 -07001112 aaptFlag: "",
Jaewoong Jung525443a2019-02-28 15:35:54 -08001113 },
1114 {
Jaewoong Jungb639a6a2019-05-10 15:16:29 -07001115 moduleName: "bar",
1116 variantName: "android_common_bar",
Jaewoong Jung525443a2019-02-28 15:35:54 -08001117 apkPath: "/target/product/test_device/system/app/bar/bar.apk",
1118 signFlag: "cert/new_cert.x509.pem cert/new_cert.pk8",
Jaewoong Jungb639a6a2019-05-10 15:16:29 -07001119 overrides: []string{"qux", "foo"},
Jaewoong Jung6f373f62019-03-13 10:13:24 -07001120 aaptFlag: "",
1121 },
1122 {
Jaewoong Jungb639a6a2019-05-10 15:16:29 -07001123 moduleName: "baz",
1124 variantName: "android_common_baz",
Jaewoong Jung6f373f62019-03-13 10:13:24 -07001125 apkPath: "/target/product/test_device/system/app/baz/baz.apk",
Dan Willemsen412160e2019-04-09 21:36:26 -07001126 signFlag: "build/make/target/product/security/expiredkey.x509.pem build/make/target/product/security/expiredkey.pk8",
Jaewoong Jungb639a6a2019-05-10 15:16:29 -07001127 overrides: []string{"qux", "foo"},
Jaewoong Jung6f373f62019-03-13 10:13:24 -07001128 aaptFlag: "--rename-manifest-package org.dandroid.bp",
Jaewoong Jung525443a2019-02-28 15:35:54 -08001129 },
1130 }
1131 for _, expected := range expectedVariants {
1132 variant := ctx.ModuleForTests("foo", expected.variantName)
1133
1134 // Check the final apk name
1135 outputs := variant.AllOutputs()
1136 expectedApkPath := buildDir + expected.apkPath
1137 found := false
1138 for _, o := range outputs {
1139 if o == expectedApkPath {
1140 found = true
1141 break
1142 }
1143 }
1144 if !found {
1145 t.Errorf("Can't find %q in output files.\nAll outputs:%v", expectedApkPath, outputs)
1146 }
1147
1148 // Check the certificate paths
Jaewoong Jung5a498812019-11-07 14:14:38 -08001149 signapk := variant.Output(expected.moduleName + ".apk")
Jaewoong Jung525443a2019-02-28 15:35:54 -08001150 signFlag := signapk.Args["certificates"]
1151 if expected.signFlag != signFlag {
1152 t.Errorf("Incorrect signing flags, expected: %q, got: %q", expected.signFlag, signFlag)
1153 }
1154
Jaewoong Jung6f373f62019-03-13 10:13:24 -07001155 // Check if the overrides field values are correctly aggregated.
Jaewoong Jung525443a2019-02-28 15:35:54 -08001156 mod := variant.Module().(*AndroidApp)
1157 if !reflect.DeepEqual(expected.overrides, mod.appProperties.Overrides) {
1158 t.Errorf("Incorrect overrides property value, expected: %q, got: %q",
1159 expected.overrides, mod.appProperties.Overrides)
1160 }
Jaewoong Jung6f373f62019-03-13 10:13:24 -07001161
1162 // Check the package renaming flag, if exists.
1163 res := variant.Output("package-res.apk")
1164 aapt2Flags := res.Args["flags"]
1165 if !strings.Contains(aapt2Flags, expected.aaptFlag) {
1166 t.Errorf("package renaming flag, %q is missing in aapt2 link flags, %q", expected.aaptFlag, aapt2Flags)
1167 }
Jaewoong Jung525443a2019-02-28 15:35:54 -08001168 }
1169}
Jaewoong Jungccbb3932019-04-15 09:48:31 -07001170
Jaewoong Jungb639a6a2019-05-10 15:16:29 -07001171func TestOverrideAndroidAppDependency(t *testing.T) {
Jaewoong Jungf9a04432019-07-17 11:15:09 -07001172 ctx, _ := testJava(t, `
Jaewoong Jungb639a6a2019-05-10 15:16:29 -07001173 android_app {
1174 name: "foo",
1175 srcs: ["a.java"],
Jeongik Cha538c0d02019-07-11 15:54:27 +09001176 sdk_version: "current",
Jaewoong Jungb639a6a2019-05-10 15:16:29 -07001177 }
1178
1179 override_android_app {
1180 name: "bar",
1181 base: "foo",
1182 package_name: "org.dandroid.bp",
1183 }
1184
1185 android_test {
1186 name: "baz",
1187 srcs: ["b.java"],
1188 instrumentation_for: "foo",
1189 }
1190
1191 android_test {
1192 name: "qux",
1193 srcs: ["b.java"],
1194 instrumentation_for: "bar",
1195 }
1196 `)
1197
1198 // Verify baz, which depends on the overridden module foo, has the correct classpath javac arg.
1199 javac := ctx.ModuleForTests("baz", "android_common").Rule("javac")
1200 fooTurbine := filepath.Join(buildDir, ".intermediates", "foo", "android_common", "turbine-combined", "foo.jar")
1201 if !strings.Contains(javac.Args["classpath"], fooTurbine) {
1202 t.Errorf("baz classpath %v does not contain %q", javac.Args["classpath"], fooTurbine)
1203 }
1204
1205 // Verify qux, which depends on the overriding module bar, has the correct classpath javac arg.
1206 javac = ctx.ModuleForTests("qux", "android_common").Rule("javac")
1207 barTurbine := filepath.Join(buildDir, ".intermediates", "foo", "android_common_bar", "turbine-combined", "foo.jar")
1208 if !strings.Contains(javac.Args["classpath"], barTurbine) {
1209 t.Errorf("qux classpath %v does not contain %q", javac.Args["classpath"], barTurbine)
1210 }
1211}
1212
Jaewoong Jung26dedd32019-06-06 08:45:58 -07001213func TestOverrideAndroidTest(t *testing.T) {
1214 ctx, _ := testJava(t, `
1215 android_app {
1216 name: "foo",
1217 srcs: ["a.java"],
1218 package_name: "com.android.foo",
1219 sdk_version: "current",
1220 }
1221
1222 override_android_app {
1223 name: "bar",
1224 base: "foo",
1225 package_name: "com.android.bar",
1226 }
1227
1228 android_test {
1229 name: "foo_test",
1230 srcs: ["b.java"],
1231 instrumentation_for: "foo",
1232 }
1233
1234 override_android_test {
1235 name: "bar_test",
1236 base: "foo_test",
1237 package_name: "com.android.bar.test",
1238 instrumentation_for: "bar",
1239 instrumentation_target_package: "com.android.bar",
1240 }
1241 `)
1242
1243 expectedVariants := []struct {
1244 moduleName string
1245 variantName string
1246 apkPath string
1247 overrides []string
1248 targetVariant string
1249 packageFlag string
1250 targetPackageFlag string
1251 }{
1252 {
1253 variantName: "android_common",
Jaewoong Jung326a9412019-11-21 10:41:00 -08001254 apkPath: "/target/product/test_device/testcases/foo_test/arm64/foo_test.apk",
Jaewoong Jung26dedd32019-06-06 08:45:58 -07001255 overrides: nil,
1256 targetVariant: "android_common",
1257 packageFlag: "",
1258 targetPackageFlag: "",
1259 },
1260 {
1261 variantName: "android_common_bar_test",
Jaewoong Jung326a9412019-11-21 10:41:00 -08001262 apkPath: "/target/product/test_device/testcases/bar_test/arm64/bar_test.apk",
Jaewoong Jung26dedd32019-06-06 08:45:58 -07001263 overrides: []string{"foo_test"},
1264 targetVariant: "android_common_bar",
1265 packageFlag: "com.android.bar.test",
1266 targetPackageFlag: "com.android.bar",
1267 },
1268 }
1269 for _, expected := range expectedVariants {
1270 variant := ctx.ModuleForTests("foo_test", expected.variantName)
1271
1272 // Check the final apk name
1273 outputs := variant.AllOutputs()
1274 expectedApkPath := buildDir + expected.apkPath
1275 found := false
1276 for _, o := range outputs {
1277 if o == expectedApkPath {
1278 found = true
1279 break
1280 }
1281 }
1282 if !found {
1283 t.Errorf("Can't find %q in output files.\nAll outputs:%v", expectedApkPath, outputs)
1284 }
1285
1286 // Check if the overrides field values are correctly aggregated.
1287 mod := variant.Module().(*AndroidTest)
1288 if !reflect.DeepEqual(expected.overrides, mod.appProperties.Overrides) {
1289 t.Errorf("Incorrect overrides property value, expected: %q, got: %q",
1290 expected.overrides, mod.appProperties.Overrides)
1291 }
1292
1293 // Check if javac classpath has the correct jar file path. This checks instrumentation_for overrides.
1294 javac := variant.Rule("javac")
1295 turbine := filepath.Join(buildDir, ".intermediates", "foo", expected.targetVariant, "turbine-combined", "foo.jar")
1296 if !strings.Contains(javac.Args["classpath"], turbine) {
1297 t.Errorf("classpath %q does not contain %q", javac.Args["classpath"], turbine)
1298 }
1299
1300 // Check aapt2 flags.
1301 res := variant.Output("package-res.apk")
1302 aapt2Flags := res.Args["flags"]
1303 checkAapt2LinkFlag(t, aapt2Flags, "rename-manifest-package", expected.packageFlag)
1304 checkAapt2LinkFlag(t, aapt2Flags, "rename-instrumentation-target-package", expected.targetPackageFlag)
1305 }
1306}
1307
Jaewoong Jungccbb3932019-04-15 09:48:31 -07001308func TestAndroidAppImport(t *testing.T) {
Jaewoong Jungf9a04432019-07-17 11:15:09 -07001309 ctx, _ := testJava(t, `
Jaewoong Jungccbb3932019-04-15 09:48:31 -07001310 android_app_import {
1311 name: "foo",
1312 apk: "prebuilts/apk/app.apk",
1313 certificate: "platform",
1314 dex_preopt: {
1315 enabled: true,
1316 },
1317 }
1318 `)
1319
1320 variant := ctx.ModuleForTests("foo", "android_common")
1321
1322 // Check dexpreopt outputs.
1323 if variant.MaybeOutput("dexpreopt/oat/arm64/package.vdex").Rule == nil ||
1324 variant.MaybeOutput("dexpreopt/oat/arm64/package.odex").Rule == nil {
1325 t.Errorf("can't find dexpreopt outputs")
1326 }
1327
1328 // Check cert signing flag.
1329 signedApk := variant.Output("signed/foo.apk")
1330 signingFlag := signedApk.Args["certificates"]
1331 expected := "build/make/target/product/security/platform.x509.pem build/make/target/product/security/platform.pk8"
1332 if expected != signingFlag {
1333 t.Errorf("Incorrect signing flags, expected: %q, got: %q", expected, signingFlag)
1334 }
1335}
1336
1337func TestAndroidAppImport_NoDexPreopt(t *testing.T) {
Jaewoong Jungf9a04432019-07-17 11:15:09 -07001338 ctx, _ := testJava(t, `
Jaewoong Jungccbb3932019-04-15 09:48:31 -07001339 android_app_import {
1340 name: "foo",
1341 apk: "prebuilts/apk/app.apk",
1342 certificate: "platform",
1343 dex_preopt: {
1344 enabled: false,
1345 },
1346 }
1347 `)
1348
1349 variant := ctx.ModuleForTests("foo", "android_common")
1350
1351 // Check dexpreopt outputs. They shouldn't exist.
1352 if variant.MaybeOutput("dexpreopt/oat/arm64/package.vdex").Rule != nil ||
1353 variant.MaybeOutput("dexpreopt/oat/arm64/package.odex").Rule != nil {
1354 t.Errorf("dexpreopt shouldn't have run.")
1355 }
1356}
1357
1358func TestAndroidAppImport_Presigned(t *testing.T) {
Jaewoong Jungf9a04432019-07-17 11:15:09 -07001359 ctx, _ := testJava(t, `
Jaewoong Jungccbb3932019-04-15 09:48:31 -07001360 android_app_import {
1361 name: "foo",
1362 apk: "prebuilts/apk/app.apk",
1363 presigned: true,
1364 dex_preopt: {
1365 enabled: true,
1366 },
1367 }
1368 `)
1369
1370 variant := ctx.ModuleForTests("foo", "android_common")
1371
1372 // Check dexpreopt outputs.
1373 if variant.MaybeOutput("dexpreopt/oat/arm64/package.vdex").Rule == nil ||
1374 variant.MaybeOutput("dexpreopt/oat/arm64/package.odex").Rule == nil {
1375 t.Errorf("can't find dexpreopt outputs")
1376 }
Nicolas Geoffrayc1bf7242019-10-18 14:51:38 +01001377 // Make sure signing was skipped and aligning was done.
Jaewoong Jungccbb3932019-04-15 09:48:31 -07001378 if variant.MaybeOutput("signed/foo.apk").Rule != nil {
1379 t.Errorf("signing rule shouldn't be included.")
1380 }
1381 if variant.MaybeOutput("zip-aligned/foo.apk").Rule == nil {
1382 t.Errorf("can't find aligning rule")
1383 }
1384}
Jaewoong Junga5e5abc2019-04-26 14:31:50 -07001385
Jaewoong Jung961d4fd2019-08-22 14:25:58 -07001386func TestAndroidAppImport_DefaultDevCert(t *testing.T) {
1387 ctx, _ := testJava(t, `
1388 android_app_import {
1389 name: "foo",
1390 apk: "prebuilts/apk/app.apk",
1391 default_dev_cert: true,
1392 dex_preopt: {
1393 enabled: true,
1394 },
1395 }
1396 `)
1397
1398 variant := ctx.ModuleForTests("foo", "android_common")
1399
1400 // Check dexpreopt outputs.
1401 if variant.MaybeOutput("dexpreopt/oat/arm64/package.vdex").Rule == nil ||
1402 variant.MaybeOutput("dexpreopt/oat/arm64/package.odex").Rule == nil {
1403 t.Errorf("can't find dexpreopt outputs")
1404 }
1405
1406 // Check cert signing flag.
1407 signedApk := variant.Output("signed/foo.apk")
1408 signingFlag := signedApk.Args["certificates"]
1409 expected := "build/make/target/product/security/testkey.x509.pem build/make/target/product/security/testkey.pk8"
1410 if expected != signingFlag {
1411 t.Errorf("Incorrect signing flags, expected: %q, got: %q", expected, signingFlag)
1412 }
1413}
1414
Jaewoong Junga5e5abc2019-04-26 14:31:50 -07001415func TestAndroidAppImport_DpiVariants(t *testing.T) {
1416 bp := `
1417 android_app_import {
1418 name: "foo",
1419 apk: "prebuilts/apk/app.apk",
1420 dpi_variants: {
1421 xhdpi: {
1422 apk: "prebuilts/apk/app_xhdpi.apk",
1423 },
1424 xxhdpi: {
1425 apk: "prebuilts/apk/app_xxhdpi.apk",
1426 },
1427 },
Jaewoong Jung961d4fd2019-08-22 14:25:58 -07001428 presigned: true,
Jaewoong Junga5e5abc2019-04-26 14:31:50 -07001429 dex_preopt: {
1430 enabled: true,
1431 },
1432 }
1433 `
1434 testCases := []struct {
1435 name string
1436 aaptPreferredConfig *string
1437 aaptPrebuiltDPI []string
1438 expected string
1439 }{
1440 {
1441 name: "no preferred",
1442 aaptPreferredConfig: nil,
1443 aaptPrebuiltDPI: []string{},
1444 expected: "prebuilts/apk/app.apk",
1445 },
1446 {
1447 name: "AAPTPreferredConfig matches",
1448 aaptPreferredConfig: proptools.StringPtr("xhdpi"),
Jaewoong Jung3e18b192019-06-11 12:25:34 -07001449 aaptPrebuiltDPI: []string{"xxhdpi", "ldpi"},
Jaewoong Junga5e5abc2019-04-26 14:31:50 -07001450 expected: "prebuilts/apk/app_xhdpi.apk",
1451 },
1452 {
1453 name: "AAPTPrebuiltDPI matches",
1454 aaptPreferredConfig: proptools.StringPtr("mdpi"),
1455 aaptPrebuiltDPI: []string{"xxhdpi", "xhdpi"},
1456 expected: "prebuilts/apk/app_xxhdpi.apk",
1457 },
1458 {
1459 name: "non-first AAPTPrebuiltDPI matches",
1460 aaptPreferredConfig: proptools.StringPtr("mdpi"),
1461 aaptPrebuiltDPI: []string{"ldpi", "xhdpi"},
1462 expected: "prebuilts/apk/app_xhdpi.apk",
1463 },
1464 {
1465 name: "no matches",
1466 aaptPreferredConfig: proptools.StringPtr("mdpi"),
1467 aaptPrebuiltDPI: []string{"ldpi", "xxxhdpi"},
1468 expected: "prebuilts/apk/app.apk",
1469 },
1470 }
1471
1472 jniRuleRe := regexp.MustCompile("^if \\(zipinfo (\\S+)")
1473 for _, test := range testCases {
1474 config := testConfig(nil)
1475 config.TestProductVariables.AAPTPreferredConfig = test.aaptPreferredConfig
1476 config.TestProductVariables.AAPTPrebuiltDPI = test.aaptPrebuiltDPI
Jaewoong Jungf9a04432019-07-17 11:15:09 -07001477 ctx := testAppContext(bp, nil)
Jaewoong Junga5e5abc2019-04-26 14:31:50 -07001478
1479 run(t, ctx, config)
1480
1481 variant := ctx.ModuleForTests("foo", "android_common")
1482 jniRuleCommand := variant.Output("jnis-uncompressed/foo.apk").RuleParams.Command
1483 matches := jniRuleRe.FindStringSubmatch(jniRuleCommand)
1484 if len(matches) != 2 {
1485 t.Errorf("failed to extract the src apk path from %q", jniRuleCommand)
1486 }
1487 if test.expected != matches[1] {
1488 t.Errorf("wrong src apk, expected: %q got: %q", test.expected, matches[1])
1489 }
1490 }
1491}
Jaewoong Jungbc625cd2019-05-06 15:48:44 -07001492
Jaewoong Jung8aae22e2019-07-17 10:21:49 -07001493func TestAndroidAppImport_Filename(t *testing.T) {
1494 ctx, config := testJava(t, `
1495 android_app_import {
1496 name: "foo",
1497 apk: "prebuilts/apk/app.apk",
1498 presigned: true,
1499 }
1500
1501 android_app_import {
1502 name: "bar",
1503 apk: "prebuilts/apk/app.apk",
1504 presigned: true,
1505 filename: "bar_sample.apk"
1506 }
1507 `)
1508
1509 testCases := []struct {
1510 name string
1511 expected string
1512 }{
1513 {
1514 name: "foo",
1515 expected: "foo.apk",
1516 },
1517 {
1518 name: "bar",
1519 expected: "bar_sample.apk",
1520 },
1521 }
1522
1523 for _, test := range testCases {
1524 variant := ctx.ModuleForTests(test.name, "android_common")
1525 if variant.MaybeOutput(test.expected).Rule == nil {
1526 t.Errorf("can't find output named %q - all outputs: %v", test.expected, variant.AllOutputs())
1527 }
1528
1529 a := variant.Module().(*AndroidAppImport)
1530 expectedValues := []string{test.expected}
1531 actualValues := android.AndroidMkEntriesForTest(
Jiyong Park0b0e1b92019-12-03 13:24:29 +09001532 t, config, "", a)[0].EntryMap["LOCAL_INSTALLED_MODULE_STEM"]
Jaewoong Jung8aae22e2019-07-17 10:21:49 -07001533 if !reflect.DeepEqual(actualValues, expectedValues) {
1534 t.Errorf("Incorrect LOCAL_INSTALLED_MODULE_STEM value '%s', expected '%s'",
1535 actualValues, expectedValues)
1536 }
1537 }
1538}
1539
Jaewoong Jung1ce9ac62019-08-13 14:11:33 -07001540func TestAndroidAppImport_ArchVariants(t *testing.T) {
1541 // The test config's target arch is ARM64.
1542 testCases := []struct {
1543 name string
1544 bp string
1545 expected string
1546 }{
1547 {
1548 name: "matching arch",
1549 bp: `
1550 android_app_import {
1551 name: "foo",
1552 apk: "prebuilts/apk/app.apk",
1553 arch: {
1554 arm64: {
1555 apk: "prebuilts/apk/app_arm64.apk",
1556 },
1557 },
Jaewoong Jung961d4fd2019-08-22 14:25:58 -07001558 presigned: true,
Jaewoong Jung1ce9ac62019-08-13 14:11:33 -07001559 dex_preopt: {
1560 enabled: true,
1561 },
1562 }
1563 `,
1564 expected: "prebuilts/apk/app_arm64.apk",
1565 },
1566 {
1567 name: "no matching arch",
1568 bp: `
1569 android_app_import {
1570 name: "foo",
1571 apk: "prebuilts/apk/app.apk",
1572 arch: {
1573 arm: {
1574 apk: "prebuilts/apk/app_arm.apk",
1575 },
1576 },
Jaewoong Jung961d4fd2019-08-22 14:25:58 -07001577 presigned: true,
Jaewoong Jung1ce9ac62019-08-13 14:11:33 -07001578 dex_preopt: {
1579 enabled: true,
1580 },
1581 }
1582 `,
1583 expected: "prebuilts/apk/app.apk",
1584 },
1585 }
1586
1587 jniRuleRe := regexp.MustCompile("^if \\(zipinfo (\\S+)")
1588 for _, test := range testCases {
1589 ctx, _ := testJava(t, test.bp)
1590
1591 variant := ctx.ModuleForTests("foo", "android_common")
1592 jniRuleCommand := variant.Output("jnis-uncompressed/foo.apk").RuleParams.Command
1593 matches := jniRuleRe.FindStringSubmatch(jniRuleCommand)
1594 if len(matches) != 2 {
1595 t.Errorf("failed to extract the src apk path from %q", jniRuleCommand)
1596 }
1597 if test.expected != matches[1] {
1598 t.Errorf("wrong src apk, expected: %q got: %q", test.expected, matches[1])
1599 }
1600 }
1601}
1602
Jaewoong Jungb28eb5f2019-08-27 15:01:50 -07001603func TestAndroidTestImport(t *testing.T) {
1604 ctx, config := testJava(t, `
1605 android_test_import {
1606 name: "foo",
1607 apk: "prebuilts/apk/app.apk",
1608 presigned: true,
1609 data: [
1610 "testdata/data",
1611 ],
1612 }
1613 `)
1614
1615 test := ctx.ModuleForTests("foo", "android_common").Module().(*AndroidTestImport)
1616
1617 // Check android mks.
Jiyong Park0b0e1b92019-12-03 13:24:29 +09001618 entries := android.AndroidMkEntriesForTest(t, config, "", test)[0]
Jaewoong Jungb28eb5f2019-08-27 15:01:50 -07001619 expected := []string{"tests"}
1620 actual := entries.EntryMap["LOCAL_MODULE_TAGS"]
1621 if !reflect.DeepEqual(expected, actual) {
1622 t.Errorf("Unexpected module tags - expected: %q, actual: %q", expected, actual)
1623 }
1624 expected = []string{"testdata/data:testdata/data"}
1625 actual = entries.EntryMap["LOCAL_COMPATIBILITY_SUPPORT_FILES"]
1626 if !reflect.DeepEqual(expected, actual) {
1627 t.Errorf("Unexpected test data - expected: %q, actual: %q", expected, actual)
1628 }
1629}
1630
Jaewoong Jungbc625cd2019-05-06 15:48:44 -07001631func TestStl(t *testing.T) {
Jaewoong Jungf9a04432019-07-17 11:15:09 -07001632 ctx, _ := testJava(t, cc.GatherRequiredDepsForTest(android.Android)+`
Jaewoong Jungbc625cd2019-05-06 15:48:44 -07001633 cc_library {
Peter Collingbournead84f972019-12-17 16:46:18 -08001634 name: "ndk_libunwind",
1635 sdk_version: "current",
1636 stl: "none",
1637 system_shared_libs: [],
1638 }
1639
1640 cc_library {
1641 name: "libc.ndk.current",
1642 sdk_version: "current",
1643 stl: "none",
1644 system_shared_libs: [],
1645 }
1646
1647 cc_library {
1648 name: "libm.ndk.current",
1649 sdk_version: "current",
1650 stl: "none",
1651 system_shared_libs: [],
1652 }
1653
1654 cc_library {
1655 name: "libdl.ndk.current",
1656 sdk_version: "current",
1657 stl: "none",
1658 system_shared_libs: [],
1659 }
1660
1661 cc_object {
1662 name: "ndk_crtbegin_so.27",
1663 }
1664
1665 cc_object {
1666 name: "ndk_crtend_so.27",
1667 }
1668
1669 cc_library {
Jaewoong Jungbc625cd2019-05-06 15:48:44 -07001670 name: "libjni",
Peter Collingbournead84f972019-12-17 16:46:18 -08001671 sdk_version: "current",
1672 stl: "c++_shared",
Jaewoong Jungbc625cd2019-05-06 15:48:44 -07001673 }
1674
1675 android_test {
1676 name: "stl",
1677 jni_libs: ["libjni"],
1678 compile_multilib: "both",
1679 sdk_version: "current",
1680 stl: "c++_shared",
1681 }
1682
1683 android_test {
1684 name: "system",
1685 jni_libs: ["libjni"],
1686 compile_multilib: "both",
1687 sdk_version: "current",
1688 }
Jaewoong Jung710756a2019-06-04 11:53:47 -07001689
1690 ndk_prebuilt_shared_stl {
1691 name: "ndk_libc++_shared",
1692 }
Jaewoong Jungbc625cd2019-05-06 15:48:44 -07001693 `)
1694
1695 testCases := []struct {
1696 name string
1697 jnis []string
1698 }{
1699 {"stl",
1700 []string{
1701 "libjni.so",
Jaewoong Jung710756a2019-06-04 11:53:47 -07001702 "libc++_shared.so",
Jaewoong Jungbc625cd2019-05-06 15:48:44 -07001703 },
1704 },
1705 {"system",
1706 []string{
1707 "libjni.so",
1708 },
1709 },
1710 }
1711
1712 for _, test := range testCases {
1713 t.Run(test.name, func(t *testing.T) {
1714 app := ctx.ModuleForTests(test.name, "android_common")
1715 jniLibZip := app.Output("jnilibs.zip")
1716 var jnis []string
1717 args := strings.Fields(jniLibZip.Args["jarArgs"])
1718 for i := 0; i < len(args); i++ {
1719 if args[i] == "-f" {
1720 jnis = append(jnis, args[i+1])
1721 i += 1
1722 }
1723 }
1724 jnisJoined := strings.Join(jnis, " ")
1725 for _, jni := range test.jnis {
1726 if !strings.Contains(jnisJoined, jni) {
1727 t.Errorf("missing jni %q in %q", jni, jnis)
1728 }
1729 }
1730 })
1731 }
1732}
Colin Cross50ddcc42019-05-16 12:28:22 -07001733
1734func TestUsesLibraries(t *testing.T) {
1735 bp := `
1736 java_sdk_library {
1737 name: "foo",
1738 srcs: ["a.java"],
1739 api_packages: ["foo"],
Jeongik Cha538c0d02019-07-11 15:54:27 +09001740 sdk_version: "current",
Colin Cross50ddcc42019-05-16 12:28:22 -07001741 }
1742
1743 java_sdk_library {
1744 name: "bar",
1745 srcs: ["a.java"],
1746 api_packages: ["bar"],
Jeongik Cha538c0d02019-07-11 15:54:27 +09001747 sdk_version: "current",
Colin Cross50ddcc42019-05-16 12:28:22 -07001748 }
1749
1750 android_app {
1751 name: "app",
1752 srcs: ["a.java"],
1753 uses_libs: ["foo"],
Jeongik Cha538c0d02019-07-11 15:54:27 +09001754 sdk_version: "current",
Colin Cross50ddcc42019-05-16 12:28:22 -07001755 optional_uses_libs: [
1756 "bar",
1757 "baz",
1758 ],
1759 }
1760
1761 android_app_import {
1762 name: "prebuilt",
1763 apk: "prebuilts/apk/app.apk",
1764 certificate: "platform",
1765 uses_libs: ["foo"],
1766 optional_uses_libs: [
1767 "bar",
1768 "baz",
1769 ],
1770 }
1771 `
1772
1773 config := testConfig(nil)
1774 config.TestProductVariables.MissingUsesLibraries = []string{"baz"}
1775
Jaewoong Jungf9a04432019-07-17 11:15:09 -07001776 ctx := testAppContext(bp, nil)
Colin Cross50ddcc42019-05-16 12:28:22 -07001777
1778 run(t, ctx, config)
1779
1780 app := ctx.ModuleForTests("app", "android_common")
1781 prebuilt := ctx.ModuleForTests("prebuilt", "android_common")
1782
1783 // Test that all libraries are verified
1784 cmd := app.Rule("verify_uses_libraries").RuleParams.Command
1785 if w := "--uses-library foo"; !strings.Contains(cmd, w) {
1786 t.Errorf("wanted %q in %q", w, cmd)
1787 }
1788
1789 if w := "--optional-uses-library bar --optional-uses-library baz"; !strings.Contains(cmd, w) {
1790 t.Errorf("wanted %q in %q", w, cmd)
1791 }
1792
1793 cmd = prebuilt.Rule("verify_uses_libraries").RuleParams.Command
1794
1795 if w := `uses_library_names="foo"`; !strings.Contains(cmd, w) {
1796 t.Errorf("wanted %q in %q", w, cmd)
1797 }
1798
1799 if w := `optional_uses_library_names="bar baz"`; !strings.Contains(cmd, w) {
1800 t.Errorf("wanted %q in %q", w, cmd)
1801 }
1802
1803 // Test that only present libraries are preopted
1804 cmd = app.Rule("dexpreopt").RuleParams.Command
1805
1806 if w := `dex_preopt_target_libraries="/system/framework/foo.jar /system/framework/bar.jar"`; !strings.Contains(cmd, w) {
1807 t.Errorf("wanted %q in %q", w, cmd)
1808 }
1809
1810 cmd = prebuilt.Rule("dexpreopt").RuleParams.Command
1811
1812 if w := `dex_preopt_target_libraries="/system/framework/foo.jar /system/framework/bar.jar"`; !strings.Contains(cmd, w) {
1813 t.Errorf("wanted %q in %q", w, cmd)
1814 }
1815}
Jaewoong Jungc27ab662019-05-30 15:51:14 -07001816
1817func TestCodelessApp(t *testing.T) {
1818 testCases := []struct {
1819 name string
1820 bp string
1821 noCode bool
1822 }{
1823 {
1824 name: "normal",
1825 bp: `
1826 android_app {
1827 name: "foo",
1828 srcs: ["a.java"],
Jeongik Cha538c0d02019-07-11 15:54:27 +09001829 sdk_version: "current",
Jaewoong Jungc27ab662019-05-30 15:51:14 -07001830 }
1831 `,
1832 noCode: false,
1833 },
1834 {
1835 name: "app without sources",
1836 bp: `
1837 android_app {
1838 name: "foo",
Jeongik Cha538c0d02019-07-11 15:54:27 +09001839 sdk_version: "current",
Jaewoong Jungc27ab662019-05-30 15:51:14 -07001840 }
1841 `,
1842 noCode: true,
1843 },
1844 {
1845 name: "app with libraries",
1846 bp: `
1847 android_app {
1848 name: "foo",
1849 static_libs: ["lib"],
Jeongik Cha538c0d02019-07-11 15:54:27 +09001850 sdk_version: "current",
Jaewoong Jungc27ab662019-05-30 15:51:14 -07001851 }
1852
1853 java_library {
1854 name: "lib",
1855 srcs: ["a.java"],
Jeongik Cha538c0d02019-07-11 15:54:27 +09001856 sdk_version: "current",
Jaewoong Jungc27ab662019-05-30 15:51:14 -07001857 }
1858 `,
1859 noCode: false,
1860 },
1861 {
1862 name: "app with sourceless libraries",
1863 bp: `
1864 android_app {
1865 name: "foo",
1866 static_libs: ["lib"],
Jeongik Cha538c0d02019-07-11 15:54:27 +09001867 sdk_version: "current",
Jaewoong Jungc27ab662019-05-30 15:51:14 -07001868 }
1869
1870 java_library {
1871 name: "lib",
Jeongik Cha538c0d02019-07-11 15:54:27 +09001872 sdk_version: "current",
Jaewoong Jungc27ab662019-05-30 15:51:14 -07001873 }
1874 `,
1875 // TODO(jungjw): this should probably be true
1876 noCode: false,
1877 },
1878 }
1879
1880 for _, test := range testCases {
1881 t.Run(test.name, func(t *testing.T) {
1882 ctx := testApp(t, test.bp)
1883
1884 foo := ctx.ModuleForTests("foo", "android_common")
1885 manifestFixerArgs := foo.Output("manifest_fixer/AndroidManifest.xml").Args["args"]
1886 if strings.Contains(manifestFixerArgs, "--has-no-code") != test.noCode {
1887 t.Errorf("unexpected manifest_fixer args: %q", manifestFixerArgs)
1888 }
1889 })
1890 }
1891}
Jaewoong Jung5b425e22019-06-17 17:40:56 -07001892
1893func TestEmbedNotice(t *testing.T) {
Jaewoong Jungf9a04432019-07-17 11:15:09 -07001894 ctx, _ := testJava(t, cc.GatherRequiredDepsForTest(android.Android)+`
Jaewoong Jung5b425e22019-06-17 17:40:56 -07001895 android_app {
1896 name: "foo",
1897 srcs: ["a.java"],
1898 static_libs: ["javalib"],
1899 jni_libs: ["libjni"],
1900 notice: "APP_NOTICE",
1901 embed_notices: true,
Jeongik Cha538c0d02019-07-11 15:54:27 +09001902 sdk_version: "current",
Jaewoong Jung5b425e22019-06-17 17:40:56 -07001903 }
1904
1905 // No embed_notice flag
1906 android_app {
1907 name: "bar",
1908 srcs: ["a.java"],
1909 jni_libs: ["libjni"],
1910 notice: "APP_NOTICE",
Jeongik Cha538c0d02019-07-11 15:54:27 +09001911 sdk_version: "current",
Jaewoong Jung5b425e22019-06-17 17:40:56 -07001912 }
1913
1914 // No NOTICE files
1915 android_app {
1916 name: "baz",
1917 srcs: ["a.java"],
1918 embed_notices: true,
Jeongik Cha538c0d02019-07-11 15:54:27 +09001919 sdk_version: "current",
Jaewoong Jung5b425e22019-06-17 17:40:56 -07001920 }
1921
1922 cc_library {
1923 name: "libjni",
1924 system_shared_libs: [],
1925 stl: "none",
1926 notice: "LIB_NOTICE",
1927 }
1928
1929 java_library {
1930 name: "javalib",
1931 srcs: [
1932 ":gen",
1933 ],
Jeongik Cha538c0d02019-07-11 15:54:27 +09001934 sdk_version: "current",
Jaewoong Jung5b425e22019-06-17 17:40:56 -07001935 }
1936
1937 genrule {
1938 name: "gen",
1939 tools: ["gentool"],
1940 out: ["gen.java"],
1941 notice: "GENRULE_NOTICE",
1942 }
1943
1944 java_binary_host {
1945 name: "gentool",
1946 srcs: ["b.java"],
1947 notice: "TOOL_NOTICE",
1948 }
1949 `)
1950
1951 // foo has NOTICE files to process, and embed_notices is true.
1952 foo := ctx.ModuleForTests("foo", "android_common")
1953 // verify merge notices rule.
1954 mergeNotices := foo.Rule("mergeNoticesRule")
1955 noticeInputs := mergeNotices.Inputs.Strings()
1956 // TOOL_NOTICE should be excluded as it's a host module.
1957 if len(mergeNotices.Inputs) != 3 {
1958 t.Errorf("number of input notice files: expected = 3, actual = %q", noticeInputs)
1959 }
1960 if !inList("APP_NOTICE", noticeInputs) {
1961 t.Errorf("APP_NOTICE is missing from notice files, %q", noticeInputs)
1962 }
1963 if !inList("LIB_NOTICE", noticeInputs) {
1964 t.Errorf("LIB_NOTICE is missing from notice files, %q", noticeInputs)
1965 }
1966 if !inList("GENRULE_NOTICE", noticeInputs) {
1967 t.Errorf("GENRULE_NOTICE is missing from notice files, %q", noticeInputs)
1968 }
1969 // aapt2 flags should include -A <NOTICE dir> so that its contents are put in the APK's /assets.
1970 res := foo.Output("package-res.apk")
1971 aapt2Flags := res.Args["flags"]
1972 e := "-A " + buildDir + "/.intermediates/foo/android_common/NOTICE"
1973 if !strings.Contains(aapt2Flags, e) {
1974 t.Errorf("asset dir flag for NOTICE, %q is missing in aapt2 link flags, %q", e, aapt2Flags)
1975 }
1976
1977 // bar has NOTICE files to process, but embed_notices is not set.
1978 bar := ctx.ModuleForTests("bar", "android_common")
Jaewoong Jung98772792019-07-01 17:15:13 -07001979 res = bar.Output("package-res.apk")
1980 aapt2Flags = res.Args["flags"]
1981 e = "-A " + buildDir + "/.intermediates/bar/android_common/NOTICE"
1982 if strings.Contains(aapt2Flags, e) {
1983 t.Errorf("bar shouldn't have the asset dir flag for NOTICE: %q", e)
Jaewoong Jung5b425e22019-06-17 17:40:56 -07001984 }
1985
1986 // baz's embed_notice is true, but it doesn't have any NOTICE files.
1987 baz := ctx.ModuleForTests("baz", "android_common")
Jaewoong Jung98772792019-07-01 17:15:13 -07001988 res = baz.Output("package-res.apk")
1989 aapt2Flags = res.Args["flags"]
1990 e = "-A " + buildDir + "/.intermediates/baz/android_common/NOTICE"
1991 if strings.Contains(aapt2Flags, e) {
1992 t.Errorf("baz shouldn't have the asset dir flag for NOTICE: %q", e)
Jaewoong Jung5b425e22019-06-17 17:40:56 -07001993 }
1994}
Colin Cross53a87f52019-06-25 13:35:30 -07001995
1996func TestUncompressDex(t *testing.T) {
1997 testCases := []struct {
1998 name string
1999 bp string
2000
2001 uncompressedPlatform bool
2002 uncompressedUnbundled bool
2003 }{
2004 {
2005 name: "normal",
2006 bp: `
2007 android_app {
2008 name: "foo",
2009 srcs: ["a.java"],
Jeongik Cha538c0d02019-07-11 15:54:27 +09002010 sdk_version: "current",
Colin Cross53a87f52019-06-25 13:35:30 -07002011 }
2012 `,
2013 uncompressedPlatform: true,
2014 uncompressedUnbundled: false,
2015 },
2016 {
2017 name: "use_embedded_dex",
2018 bp: `
2019 android_app {
2020 name: "foo",
2021 use_embedded_dex: true,
2022 srcs: ["a.java"],
Jeongik Cha538c0d02019-07-11 15:54:27 +09002023 sdk_version: "current",
Colin Cross53a87f52019-06-25 13:35:30 -07002024 }
2025 `,
2026 uncompressedPlatform: true,
2027 uncompressedUnbundled: true,
2028 },
2029 {
2030 name: "privileged",
2031 bp: `
2032 android_app {
2033 name: "foo",
2034 privileged: true,
2035 srcs: ["a.java"],
Jeongik Cha538c0d02019-07-11 15:54:27 +09002036 sdk_version: "current",
Colin Cross53a87f52019-06-25 13:35:30 -07002037 }
2038 `,
2039 uncompressedPlatform: true,
2040 uncompressedUnbundled: true,
2041 },
2042 }
2043
2044 test := func(t *testing.T, bp string, want bool, unbundled bool) {
2045 t.Helper()
2046
2047 config := testConfig(nil)
2048 if unbundled {
2049 config.TestProductVariables.Unbundled_build = proptools.BoolPtr(true)
2050 }
2051
Jaewoong Jungf9a04432019-07-17 11:15:09 -07002052 ctx := testAppContext(bp, nil)
Colin Cross53a87f52019-06-25 13:35:30 -07002053
2054 run(t, ctx, config)
2055
2056 foo := ctx.ModuleForTests("foo", "android_common")
2057 dex := foo.Rule("r8")
2058 uncompressedInDexJar := strings.Contains(dex.Args["zipFlags"], "-L 0")
2059 aligned := foo.MaybeRule("zipalign").Rule != nil
2060
2061 if uncompressedInDexJar != want {
2062 t.Errorf("want uncompressed in dex %v, got %v", want, uncompressedInDexJar)
2063 }
2064
2065 if aligned != want {
2066 t.Errorf("want aligned %v, got %v", want, aligned)
2067 }
2068 }
2069
2070 for _, tt := range testCases {
2071 t.Run(tt.name, func(t *testing.T) {
2072 t.Run("platform", func(t *testing.T) {
2073 test(t, tt.bp, tt.uncompressedPlatform, false)
2074 })
2075 t.Run("unbundled", func(t *testing.T) {
2076 test(t, tt.bp, tt.uncompressedUnbundled, true)
2077 })
2078 })
2079 }
2080}
Jaewoong Jung26dedd32019-06-06 08:45:58 -07002081
2082func checkAapt2LinkFlag(t *testing.T, aapt2Flags, flagName, expectedValue string) {
2083 if expectedValue != "" {
2084 expectedFlag := "--" + flagName + " " + expectedValue
2085 if !strings.Contains(aapt2Flags, expectedFlag) {
2086 t.Errorf("%q is missing in aapt2 link flags, %q", expectedFlag, aapt2Flags)
2087 }
2088 } else {
2089 unexpectedFlag := "--" + flagName
2090 if strings.Contains(aapt2Flags, unexpectedFlag) {
2091 t.Errorf("unexpected flag, %q is found in aapt2 link flags, %q", unexpectedFlag, aapt2Flags)
2092 }
2093 }
2094}