| Jaewoong Jung | f9b4465 | 2020-12-21 12:29:12 -0800 | [diff] [blame] | 1 | // Copyright 2020 Google Inc. All rights reserved. | 
|  | 2 | // | 
|  | 3 | // Licensed under the Apache License, Version 2.0 (the "License"); | 
|  | 4 | // you may not use this file except in compliance with the License. | 
|  | 5 | // You may obtain a copy of the License at | 
|  | 6 | // | 
|  | 7 | //     http://www.apache.org/licenses/LICENSE-2.0 | 
|  | 8 | // | 
|  | 9 | // Unless required by applicable law or agreed to in writing, software | 
|  | 10 | // distributed under the License is distributed on an "AS IS" BASIS, | 
|  | 11 | // WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. | 
|  | 12 | // See the License for the specific language governing permissions and | 
|  | 13 | // limitations under the License. | 
|  | 14 |  | 
|  | 15 | package java | 
|  | 16 |  | 
|  | 17 | import ( | 
|  | 18 | "reflect" | 
|  | 19 | "strings" | 
|  | 20 | "testing" | 
|  | 21 |  | 
|  | 22 | "android/soong/android" | 
| Lukacs T. Berki | 7690c09 | 2021-02-26 14:27:36 +0100 | [diff] [blame] | 23 | "android/soong/shared" | 
| Jaewoong Jung | f9b4465 | 2020-12-21 12:29:12 -0800 | [diff] [blame] | 24 | ) | 
|  | 25 |  | 
|  | 26 | func TestRuntimeResourceOverlay(t *testing.T) { | 
| Paul Duffin | 0342dc9 | 2021-03-22 17:31:52 +0000 | [diff] [blame] | 27 | fs := android.MockFS{ | 
| Jaewoong Jung | f9b4465 | 2020-12-21 12:29:12 -0800 | [diff] [blame] | 28 | "baz/res/res/values/strings.xml": nil, | 
|  | 29 | "bar/res/res/values/strings.xml": nil, | 
|  | 30 | } | 
|  | 31 | bp := ` | 
|  | 32 | runtime_resource_overlay { | 
|  | 33 | name: "foo", | 
|  | 34 | certificate: "platform", | 
|  | 35 | lineage: "lineage.bin", | 
| Rupert Shuttleworth | 8eab869 | 2021-11-03 10:39:39 -0400 | [diff] [blame] | 36 | rotationMinSdkVersion: "32", | 
| Jaewoong Jung | f9b4465 | 2020-12-21 12:29:12 -0800 | [diff] [blame] | 37 | product_specific: true, | 
|  | 38 | static_libs: ["bar"], | 
|  | 39 | resource_libs: ["baz"], | 
|  | 40 | aaptflags: ["--keep-raw-values"], | 
|  | 41 | } | 
|  | 42 |  | 
|  | 43 | runtime_resource_overlay { | 
|  | 44 | name: "foo_themed", | 
|  | 45 | certificate: "platform", | 
|  | 46 | product_specific: true, | 
|  | 47 | theme: "faza", | 
|  | 48 | overrides: ["foo"], | 
|  | 49 | } | 
|  | 50 |  | 
|  | 51 | android_library { | 
|  | 52 | name: "bar", | 
|  | 53 | resource_dirs: ["bar/res"], | 
|  | 54 | } | 
|  | 55 |  | 
|  | 56 | android_app { | 
|  | 57 | name: "baz", | 
|  | 58 | sdk_version: "current", | 
|  | 59 | resource_dirs: ["baz/res"], | 
|  | 60 | } | 
| Paul Duffin | 0342dc9 | 2021-03-22 17:31:52 +0000 | [diff] [blame] | 61 | ` | 
| Jaewoong Jung | f9b4465 | 2020-12-21 12:29:12 -0800 | [diff] [blame] | 62 |  | 
| Paul Duffin | 0342dc9 | 2021-03-22 17:31:52 +0000 | [diff] [blame] | 63 | result := android.GroupFixturePreparers( | 
|  | 64 | PrepareForTestWithJavaDefaultModules, | 
| Colin Cross | c68db4b | 2021-11-11 18:59:15 -0800 | [diff] [blame] | 65 | android.FixtureModifyConfig(android.SetKatiEnabledForTests), | 
| Paul Duffin | 0342dc9 | 2021-03-22 17:31:52 +0000 | [diff] [blame] | 66 | fs.AddToFixture(), | 
|  | 67 | ).RunTestWithBp(t, bp) | 
|  | 68 |  | 
|  | 69 | m := result.ModuleForTests("foo", "android_common") | 
| Jaewoong Jung | f9b4465 | 2020-12-21 12:29:12 -0800 | [diff] [blame] | 70 |  | 
|  | 71 | // Check AAPT2 link flags. | 
| Paul Duffin | a71a67a | 2021-03-29 00:42:57 +0100 | [diff] [blame] | 72 | aapt2Flags := m.Output("package-res.apk").Args["flags"] | 
| Jaewoong Jung | f9b4465 | 2020-12-21 12:29:12 -0800 | [diff] [blame] | 73 | expectedFlags := []string{"--keep-raw-values", "--no-resource-deduping", "--no-resource-removal"} | 
|  | 74 | absentFlags := android.RemoveListFromList(expectedFlags, strings.Split(aapt2Flags, " ")) | 
|  | 75 | if len(absentFlags) > 0 { | 
|  | 76 | t.Errorf("expected values, %q are missing in aapt2 link flags, %q", absentFlags, aapt2Flags) | 
|  | 77 | } | 
|  | 78 |  | 
|  | 79 | // Check overlay.list output for static_libs dependency. | 
| Paul Duffin | 0342dc9 | 2021-03-22 17:31:52 +0000 | [diff] [blame] | 80 | overlayList := android.PathsRelativeToTop(m.Output("aapt2/overlay.list").Inputs) | 
|  | 81 | staticLibPackage := "out/soong/.intermediates/bar/android_common/package-res.apk" | 
| Jaewoong Jung | f9b4465 | 2020-12-21 12:29:12 -0800 | [diff] [blame] | 82 | if !inList(staticLibPackage, overlayList) { | 
|  | 83 | t.Errorf("Stactic lib res package %q missing in overlay list: %q", staticLibPackage, overlayList) | 
|  | 84 | } | 
|  | 85 |  | 
|  | 86 | // Check AAPT2 link flags for resource_libs dependency. | 
| Paul Duffin | 0342dc9 | 2021-03-22 17:31:52 +0000 | [diff] [blame] | 87 | resourceLibFlag := "-I " + "out/soong/.intermediates/baz/android_common/package-res.apk" | 
| Jaewoong Jung | f9b4465 | 2020-12-21 12:29:12 -0800 | [diff] [blame] | 88 | if !strings.Contains(aapt2Flags, resourceLibFlag) { | 
|  | 89 | t.Errorf("Resource lib flag %q missing in aapt2 link flags: %q", resourceLibFlag, aapt2Flags) | 
|  | 90 | } | 
|  | 91 |  | 
| Rupert Shuttleworth | 8eab869 | 2021-11-03 10:39:39 -0400 | [diff] [blame] | 92 | // Check cert signing flags. | 
| Jaewoong Jung | f9b4465 | 2020-12-21 12:29:12 -0800 | [diff] [blame] | 93 | signedApk := m.Output("signed/foo.apk") | 
| Rupert Shuttleworth | 8eab869 | 2021-11-03 10:39:39 -0400 | [diff] [blame] | 94 | actualCertSigningFlags := signedApk.Args["flags"] | 
|  | 95 | expectedCertSigningFlags := "--lineage lineage.bin --rotation-min-sdk-version 32" | 
|  | 96 | if expectedCertSigningFlags != actualCertSigningFlags { | 
|  | 97 | t.Errorf("Incorrect cert signing flags, expected: %q, got: %q", expectedCertSigningFlags, actualCertSigningFlags) | 
| Jaewoong Jung | f9b4465 | 2020-12-21 12:29:12 -0800 | [diff] [blame] | 98 | } | 
| Rupert Shuttleworth | 8eab869 | 2021-11-03 10:39:39 -0400 | [diff] [blame] | 99 |  | 
| Jaewoong Jung | f9b4465 | 2020-12-21 12:29:12 -0800 | [diff] [blame] | 100 | signingFlag := signedApk.Args["certificates"] | 
|  | 101 | expected := "build/make/target/product/security/platform.x509.pem build/make/target/product/security/platform.pk8" | 
|  | 102 | if expected != signingFlag { | 
|  | 103 | t.Errorf("Incorrect signing flags, expected: %q, got: %q", expected, signingFlag) | 
|  | 104 | } | 
| Paul Duffin | 0342dc9 | 2021-03-22 17:31:52 +0000 | [diff] [blame] | 105 | androidMkEntries := android.AndroidMkEntriesForTest(t, result.TestContext, m.Module())[0] | 
| Jaewoong Jung | f9b4465 | 2020-12-21 12:29:12 -0800 | [diff] [blame] | 106 | path := androidMkEntries.EntryMap["LOCAL_CERTIFICATE"] | 
|  | 107 | expectedPath := []string{"build/make/target/product/security/platform.x509.pem"} | 
|  | 108 | if !reflect.DeepEqual(path, expectedPath) { | 
|  | 109 | t.Errorf("Unexpected LOCAL_CERTIFICATE value: %v, expected: %v", path, expectedPath) | 
|  | 110 | } | 
|  | 111 |  | 
|  | 112 | // Check device location. | 
|  | 113 | path = androidMkEntries.EntryMap["LOCAL_MODULE_PATH"] | 
| Paul Duffin | 0342dc9 | 2021-03-22 17:31:52 +0000 | [diff] [blame] | 114 | expectedPath = []string{shared.JoinPath("out/target/product/test_device/product/overlay")} | 
|  | 115 | android.AssertStringPathsRelativeToTopEquals(t, "LOCAL_MODULE_PATH", result.Config, expectedPath, path) | 
| Jaewoong Jung | f9b4465 | 2020-12-21 12:29:12 -0800 | [diff] [blame] | 116 |  | 
|  | 117 | // A themed module has a different device location | 
| Paul Duffin | 0342dc9 | 2021-03-22 17:31:52 +0000 | [diff] [blame] | 118 | m = result.ModuleForTests("foo_themed", "android_common") | 
|  | 119 | androidMkEntries = android.AndroidMkEntriesForTest(t, result.TestContext, m.Module())[0] | 
| Jaewoong Jung | f9b4465 | 2020-12-21 12:29:12 -0800 | [diff] [blame] | 120 | path = androidMkEntries.EntryMap["LOCAL_MODULE_PATH"] | 
| Paul Duffin | 0342dc9 | 2021-03-22 17:31:52 +0000 | [diff] [blame] | 121 | expectedPath = []string{shared.JoinPath("out/target/product/test_device/product/overlay/faza")} | 
|  | 122 | android.AssertStringPathsRelativeToTopEquals(t, "LOCAL_MODULE_PATH", result.Config, expectedPath, path) | 
| Jaewoong Jung | f9b4465 | 2020-12-21 12:29:12 -0800 | [diff] [blame] | 123 |  | 
|  | 124 | overrides := androidMkEntries.EntryMap["LOCAL_OVERRIDES_PACKAGES"] | 
|  | 125 | expectedOverrides := []string{"foo"} | 
|  | 126 | if !reflect.DeepEqual(overrides, expectedOverrides) { | 
|  | 127 | t.Errorf("Unexpected LOCAL_OVERRIDES_PACKAGES value: %v, expected: %v", overrides, expectedOverrides) | 
|  | 128 | } | 
|  | 129 | } | 
|  | 130 |  | 
|  | 131 | func TestRuntimeResourceOverlay_JavaDefaults(t *testing.T) { | 
| Colin Cross | c68db4b | 2021-11-11 18:59:15 -0800 | [diff] [blame] | 132 | result := android.GroupFixturePreparers( | 
|  | 133 | PrepareForTestWithJavaDefaultModules, | 
|  | 134 | android.FixtureModifyConfig(android.SetKatiEnabledForTests), | 
|  | 135 | ).RunTestWithBp(t, ` | 
| Jaewoong Jung | f9b4465 | 2020-12-21 12:29:12 -0800 | [diff] [blame] | 136 | java_defaults { | 
|  | 137 | name: "rro_defaults", | 
|  | 138 | theme: "default_theme", | 
|  | 139 | product_specific: true, | 
|  | 140 | aaptflags: ["--keep-raw-values"], | 
|  | 141 | } | 
|  | 142 |  | 
|  | 143 | runtime_resource_overlay { | 
|  | 144 | name: "foo_with_defaults", | 
|  | 145 | defaults: ["rro_defaults"], | 
|  | 146 | } | 
|  | 147 |  | 
|  | 148 | runtime_resource_overlay { | 
|  | 149 | name: "foo_barebones", | 
|  | 150 | } | 
|  | 151 | `) | 
|  | 152 |  | 
|  | 153 | // | 
|  | 154 | // RRO module with defaults | 
|  | 155 | // | 
| Colin Cross | c68db4b | 2021-11-11 18:59:15 -0800 | [diff] [blame] | 156 | m := result.ModuleForTests("foo_with_defaults", "android_common") | 
| Jaewoong Jung | f9b4465 | 2020-12-21 12:29:12 -0800 | [diff] [blame] | 157 |  | 
|  | 158 | // Check AAPT2 link flags. | 
|  | 159 | aapt2Flags := strings.Split(m.Output("package-res.apk").Args["flags"], " ") | 
|  | 160 | expectedFlags := []string{"--keep-raw-values", "--no-resource-deduping", "--no-resource-removal"} | 
|  | 161 | absentFlags := android.RemoveListFromList(expectedFlags, aapt2Flags) | 
|  | 162 | if len(absentFlags) > 0 { | 
|  | 163 | t.Errorf("expected values, %q are missing in aapt2 link flags, %q", absentFlags, aapt2Flags) | 
|  | 164 | } | 
|  | 165 |  | 
|  | 166 | // Check device location. | 
| Colin Cross | c68db4b | 2021-11-11 18:59:15 -0800 | [diff] [blame] | 167 | path := android.AndroidMkEntriesForTest(t, result.TestContext, m.Module())[0].EntryMap["LOCAL_MODULE_PATH"] | 
| Paul Duffin | 0342dc9 | 2021-03-22 17:31:52 +0000 | [diff] [blame] | 168 | expectedPath := []string{shared.JoinPath("out/target/product/test_device/product/overlay/default_theme")} | 
| Colin Cross | c68db4b | 2021-11-11 18:59:15 -0800 | [diff] [blame] | 169 | android.AssertStringPathsRelativeToTopEquals(t, "LOCAL_MODULE_PATH", result.Config, expectedPath, path) | 
| Jaewoong Jung | f9b4465 | 2020-12-21 12:29:12 -0800 | [diff] [blame] | 170 |  | 
|  | 171 | // | 
|  | 172 | // RRO module without defaults | 
|  | 173 | // | 
| Colin Cross | c68db4b | 2021-11-11 18:59:15 -0800 | [diff] [blame] | 174 | m = result.ModuleForTests("foo_barebones", "android_common") | 
| Jaewoong Jung | f9b4465 | 2020-12-21 12:29:12 -0800 | [diff] [blame] | 175 |  | 
|  | 176 | // Check AAPT2 link flags. | 
|  | 177 | aapt2Flags = strings.Split(m.Output("package-res.apk").Args["flags"], " ") | 
|  | 178 | unexpectedFlags := "--keep-raw-values" | 
|  | 179 | if inList(unexpectedFlags, aapt2Flags) { | 
|  | 180 | t.Errorf("unexpected value, %q is present in aapt2 link flags, %q", unexpectedFlags, aapt2Flags) | 
|  | 181 | } | 
|  | 182 |  | 
|  | 183 | // Check device location. | 
| Colin Cross | c68db4b | 2021-11-11 18:59:15 -0800 | [diff] [blame] | 184 | path = android.AndroidMkEntriesForTest(t, result.TestContext, m.Module())[0].EntryMap["LOCAL_MODULE_PATH"] | 
| Spandan Das | 5d1b929 | 2021-06-03 19:36:41 +0000 | [diff] [blame] | 185 | expectedPath = []string{shared.JoinPath("out/target/product/test_device/product/overlay")} | 
| Colin Cross | c68db4b | 2021-11-11 18:59:15 -0800 | [diff] [blame] | 186 | android.AssertStringPathsRelativeToTopEquals(t, "LOCAL_MODULE_PATH", result.Config, expectedPath, path) | 
| Jaewoong Jung | f9b4465 | 2020-12-21 12:29:12 -0800 | [diff] [blame] | 187 | } | 
|  | 188 |  | 
|  | 189 | func TestOverrideRuntimeResourceOverlay(t *testing.T) { | 
|  | 190 | ctx, _ := testJava(t, ` | 
|  | 191 | runtime_resource_overlay { | 
|  | 192 | name: "foo_overlay", | 
|  | 193 | certificate: "platform", | 
|  | 194 | product_specific: true, | 
|  | 195 | sdk_version: "current", | 
|  | 196 | } | 
|  | 197 |  | 
|  | 198 | override_runtime_resource_overlay { | 
|  | 199 | name: "bar_overlay", | 
|  | 200 | base: "foo_overlay", | 
|  | 201 | package_name: "com.android.bar.overlay", | 
|  | 202 | target_package_name: "com.android.bar", | 
| Jeremy Meyer | 7e67129 | 2022-10-07 18:21:34 +0000 | [diff] [blame] | 203 | category: "mycategory", | 
| Jaewoong Jung | f9b4465 | 2020-12-21 12:29:12 -0800 | [diff] [blame] | 204 | } | 
|  | 205 | `) | 
|  | 206 |  | 
|  | 207 | expectedVariants := []struct { | 
|  | 208 | moduleName        string | 
|  | 209 | variantName       string | 
|  | 210 | apkPath           string | 
|  | 211 | overrides         []string | 
|  | 212 | targetVariant     string | 
|  | 213 | packageFlag       string | 
|  | 214 | targetPackageFlag string | 
| Jeremy Meyer | 7e67129 | 2022-10-07 18:21:34 +0000 | [diff] [blame] | 215 | categoryFlag      string | 
| Jaewoong Jung | f9b4465 | 2020-12-21 12:29:12 -0800 | [diff] [blame] | 216 | }{ | 
|  | 217 | { | 
|  | 218 | variantName:       "android_common", | 
| Paul Duffin | 0342dc9 | 2021-03-22 17:31:52 +0000 | [diff] [blame] | 219 | apkPath:           "out/soong/target/product/test_device/product/overlay/foo_overlay.apk", | 
| Jaewoong Jung | f9b4465 | 2020-12-21 12:29:12 -0800 | [diff] [blame] | 220 | overrides:         nil, | 
|  | 221 | targetVariant:     "android_common", | 
|  | 222 | packageFlag:       "", | 
|  | 223 | targetPackageFlag: "", | 
|  | 224 | }, | 
|  | 225 | { | 
|  | 226 | variantName:       "android_common_bar_overlay", | 
| Paul Duffin | 0342dc9 | 2021-03-22 17:31:52 +0000 | [diff] [blame] | 227 | apkPath:           "out/soong/target/product/test_device/product/overlay/bar_overlay.apk", | 
| Jaewoong Jung | f9b4465 | 2020-12-21 12:29:12 -0800 | [diff] [blame] | 228 | overrides:         []string{"foo_overlay"}, | 
|  | 229 | targetVariant:     "android_common_bar", | 
|  | 230 | packageFlag:       "com.android.bar.overlay", | 
|  | 231 | targetPackageFlag: "com.android.bar", | 
| Jeremy Meyer | 7e67129 | 2022-10-07 18:21:34 +0000 | [diff] [blame] | 232 | categoryFlag:      "mycategory", | 
| Jaewoong Jung | f9b4465 | 2020-12-21 12:29:12 -0800 | [diff] [blame] | 233 | }, | 
|  | 234 | } | 
|  | 235 | for _, expected := range expectedVariants { | 
|  | 236 | variant := ctx.ModuleForTests("foo_overlay", expected.variantName) | 
|  | 237 |  | 
|  | 238 | // Check the final apk name | 
| Paul Duffin | 0342dc9 | 2021-03-22 17:31:52 +0000 | [diff] [blame] | 239 | variant.Output(expected.apkPath) | 
| Jaewoong Jung | f9b4465 | 2020-12-21 12:29:12 -0800 | [diff] [blame] | 240 |  | 
|  | 241 | // Check if the overrides field values are correctly aggregated. | 
|  | 242 | mod := variant.Module().(*RuntimeResourceOverlay) | 
|  | 243 | if !reflect.DeepEqual(expected.overrides, mod.properties.Overrides) { | 
|  | 244 | t.Errorf("Incorrect overrides property value, expected: %q, got: %q", | 
|  | 245 | expected.overrides, mod.properties.Overrides) | 
|  | 246 | } | 
|  | 247 |  | 
|  | 248 | // Check aapt2 flags. | 
|  | 249 | res := variant.Output("package-res.apk") | 
|  | 250 | aapt2Flags := res.Args["flags"] | 
|  | 251 | checkAapt2LinkFlag(t, aapt2Flags, "rename-manifest-package", expected.packageFlag) | 
|  | 252 | checkAapt2LinkFlag(t, aapt2Flags, "rename-resources-package", "") | 
|  | 253 | checkAapt2LinkFlag(t, aapt2Flags, "rename-overlay-target-package", expected.targetPackageFlag) | 
| Jeremy Meyer | 7e67129 | 2022-10-07 18:21:34 +0000 | [diff] [blame] | 254 | checkAapt2LinkFlag(t, aapt2Flags, "rename-overlay-category", expected.categoryFlag) | 
| Jaewoong Jung | f9b4465 | 2020-12-21 12:29:12 -0800 | [diff] [blame] | 255 | } | 
|  | 256 | } | 
|  | 257 |  | 
|  | 258 | func TestEnforceRRO_propagatesToDependencies(t *testing.T) { | 
|  | 259 | testCases := []struct { | 
| Jeongik Cha | cee5ba9 | 2021-02-19 12:11:51 +0900 | [diff] [blame] | 260 | name              string | 
|  | 261 | enforceRROTargets []string | 
|  | 262 | rroDirs           map[string][]string | 
| Jaewoong Jung | f9b4465 | 2020-12-21 12:29:12 -0800 | [diff] [blame] | 263 | }{ | 
|  | 264 | { | 
| Jeongik Cha | cee5ba9 | 2021-02-19 12:11:51 +0900 | [diff] [blame] | 265 | name:              "no RRO", | 
|  | 266 | enforceRROTargets: nil, | 
| Jaewoong Jung | f9b4465 | 2020-12-21 12:29:12 -0800 | [diff] [blame] | 267 | rroDirs: map[string][]string{ | 
|  | 268 | "foo": nil, | 
|  | 269 | "bar": nil, | 
|  | 270 | }, | 
|  | 271 | }, | 
|  | 272 | { | 
| Jeongik Cha | cee5ba9 | 2021-02-19 12:11:51 +0900 | [diff] [blame] | 273 | name:              "enforce RRO on all", | 
|  | 274 | enforceRROTargets: []string{"*"}, | 
| Jaewoong Jung | f9b4465 | 2020-12-21 12:29:12 -0800 | [diff] [blame] | 275 | rroDirs: map[string][]string{ | 
|  | 276 | "foo": {"product/vendor/blah/overlay/lib2/res"}, | 
|  | 277 | "bar": {"product/vendor/blah/overlay/lib2/res"}, | 
|  | 278 | }, | 
|  | 279 | }, | 
|  | 280 | { | 
| Jeongik Cha | cee5ba9 | 2021-02-19 12:11:51 +0900 | [diff] [blame] | 281 | name:              "enforce RRO on foo", | 
|  | 282 | enforceRROTargets: []string{"foo"}, | 
| Jaewoong Jung | f9b4465 | 2020-12-21 12:29:12 -0800 | [diff] [blame] | 283 | rroDirs: map[string][]string{ | 
|  | 284 | "foo": {"product/vendor/blah/overlay/lib2/res"}, | 
|  | 285 | "bar": {"product/vendor/blah/overlay/lib2/res"}, | 
|  | 286 | }, | 
|  | 287 | }, | 
| Jaewoong Jung | f9b4465 | 2020-12-21 12:29:12 -0800 | [diff] [blame] | 288 | } | 
|  | 289 |  | 
|  | 290 | productResourceOverlays := []string{ | 
|  | 291 | "product/vendor/blah/overlay", | 
|  | 292 | } | 
|  | 293 |  | 
| Paul Duffin | 0342dc9 | 2021-03-22 17:31:52 +0000 | [diff] [blame] | 294 | fs := android.MockFS{ | 
| Jaewoong Jung | f9b4465 | 2020-12-21 12:29:12 -0800 | [diff] [blame] | 295 | "lib2/res/values/strings.xml":                             nil, | 
|  | 296 | "product/vendor/blah/overlay/lib2/res/values/strings.xml": nil, | 
|  | 297 | } | 
|  | 298 |  | 
|  | 299 | bp := ` | 
|  | 300 | android_app { | 
|  | 301 | name: "foo", | 
|  | 302 | sdk_version: "current", | 
|  | 303 | resource_dirs: [], | 
|  | 304 | static_libs: ["lib"], | 
|  | 305 | } | 
|  | 306 |  | 
|  | 307 | android_app { | 
|  | 308 | name: "bar", | 
|  | 309 | sdk_version: "current", | 
|  | 310 | resource_dirs: [], | 
|  | 311 | static_libs: ["lib"], | 
|  | 312 | } | 
|  | 313 |  | 
|  | 314 | android_library { | 
|  | 315 | name: "lib", | 
|  | 316 | sdk_version: "current", | 
|  | 317 | resource_dirs: [], | 
|  | 318 | static_libs: ["lib2"], | 
|  | 319 | } | 
|  | 320 |  | 
|  | 321 | android_library { | 
|  | 322 | name: "lib2", | 
|  | 323 | sdk_version: "current", | 
|  | 324 | resource_dirs: ["lib2/res"], | 
|  | 325 | } | 
|  | 326 | ` | 
|  | 327 |  | 
|  | 328 | for _, testCase := range testCases { | 
|  | 329 | t.Run(testCase.name, func(t *testing.T) { | 
| Paul Duffin | 0342dc9 | 2021-03-22 17:31:52 +0000 | [diff] [blame] | 330 | result := android.GroupFixturePreparers( | 
|  | 331 | PrepareForTestWithJavaDefaultModules, | 
| Paul Duffin | 0342dc9 | 2021-03-22 17:31:52 +0000 | [diff] [blame] | 332 | fs.AddToFixture(), | 
|  | 333 | android.FixtureModifyProductVariables(func(variables android.FixtureProductVariables) { | 
|  | 334 | variables.ProductResourceOverlays = productResourceOverlays | 
|  | 335 | if testCase.enforceRROTargets != nil { | 
|  | 336 | variables.EnforceRROTargets = testCase.enforceRROTargets | 
|  | 337 | } | 
|  | 338 | }), | 
|  | 339 | ).RunTestWithBp(t, bp) | 
| Jaewoong Jung | f9b4465 | 2020-12-21 12:29:12 -0800 | [diff] [blame] | 340 |  | 
|  | 341 | modules := []string{"foo", "bar"} | 
|  | 342 | for _, moduleName := range modules { | 
| Paul Duffin | 0342dc9 | 2021-03-22 17:31:52 +0000 | [diff] [blame] | 343 | module := result.ModuleForTests(moduleName, "android_common") | 
|  | 344 | mkEntries := android.AndroidMkEntriesForTest(t, result.TestContext, module.Module())[0] | 
| Jaewoong Jung | f9b4465 | 2020-12-21 12:29:12 -0800 | [diff] [blame] | 345 | actualRRODirs := mkEntries.EntryMap["LOCAL_SOONG_PRODUCT_RRO_DIRS"] | 
|  | 346 | if !reflect.DeepEqual(actualRRODirs, testCase.rroDirs[moduleName]) { | 
|  | 347 | t.Errorf("exected %s LOCAL_SOONG_PRODUCT_RRO_DIRS entry: %v\ngot:%q", | 
|  | 348 | moduleName, testCase.rroDirs[moduleName], actualRRODirs) | 
|  | 349 | } | 
|  | 350 | } | 
|  | 351 | }) | 
|  | 352 | } | 
|  | 353 | } | 
| Spandan Das | 5d1b929 | 2021-06-03 19:36:41 +0000 | [diff] [blame] | 354 |  | 
|  | 355 | func TestRuntimeResourceOverlayPartition(t *testing.T) { | 
|  | 356 | bp := ` | 
|  | 357 | runtime_resource_overlay { | 
|  | 358 | name: "device_specific", | 
|  | 359 | device_specific: true, | 
|  | 360 | } | 
|  | 361 | runtime_resource_overlay { | 
|  | 362 | name: "soc_specific", | 
|  | 363 | soc_specific: true, | 
|  | 364 | } | 
|  | 365 | runtime_resource_overlay { | 
|  | 366 | name: "system_ext_specific", | 
|  | 367 | system_ext_specific: true, | 
|  | 368 | } | 
|  | 369 | runtime_resource_overlay { | 
|  | 370 | name: "product_specific", | 
|  | 371 | product_specific: true, | 
|  | 372 | } | 
|  | 373 | runtime_resource_overlay { | 
|  | 374 | name: "default" | 
|  | 375 | } | 
|  | 376 | ` | 
|  | 377 | testCases := []struct { | 
|  | 378 | name         string | 
|  | 379 | expectedPath string | 
|  | 380 | }{ | 
|  | 381 | { | 
|  | 382 | name:         "device_specific", | 
|  | 383 | expectedPath: "out/soong/target/product/test_device/odm/overlay", | 
|  | 384 | }, | 
|  | 385 | { | 
|  | 386 | name:         "soc_specific", | 
|  | 387 | expectedPath: "out/soong/target/product/test_device/vendor/overlay", | 
|  | 388 | }, | 
|  | 389 | { | 
|  | 390 | name:         "system_ext_specific", | 
|  | 391 | expectedPath: "out/soong/target/product/test_device/system_ext/overlay", | 
|  | 392 | }, | 
|  | 393 | { | 
|  | 394 | name:         "product_specific", | 
|  | 395 | expectedPath: "out/soong/target/product/test_device/product/overlay", | 
|  | 396 | }, | 
|  | 397 | { | 
|  | 398 | name:         "default", | 
|  | 399 | expectedPath: "out/soong/target/product/test_device/product/overlay", | 
|  | 400 | }, | 
|  | 401 | } | 
|  | 402 | for _, testCase := range testCases { | 
|  | 403 | ctx, _ := testJava(t, bp) | 
|  | 404 | mod := ctx.ModuleForTests(testCase.name, "android_common").Module().(*RuntimeResourceOverlay) | 
|  | 405 | android.AssertPathRelativeToTopEquals(t, "Install dir is not correct for "+testCase.name, testCase.expectedPath, mod.installDir) | 
|  | 406 | } | 
|  | 407 | } | 
| Jihoon Kang | 9f442dc | 2024-03-20 22:09:04 +0000 | [diff] [blame] | 408 |  | 
|  | 409 | func TestRuntimeResourceOverlayFlagsPackages(t *testing.T) { | 
|  | 410 | result := android.GroupFixturePreparers( | 
|  | 411 | prepareForJavaTest, | 
|  | 412 | ).RunTestWithBp(t, ` | 
|  | 413 | runtime_resource_overlay { | 
|  | 414 | name: "foo", | 
|  | 415 | sdk_version: "current", | 
|  | 416 | flags_packages: [ | 
|  | 417 | "bar", | 
|  | 418 | "baz", | 
|  | 419 | ], | 
|  | 420 | } | 
|  | 421 | aconfig_declarations { | 
|  | 422 | name: "bar", | 
|  | 423 | package: "com.example.package.bar", | 
|  | 424 | srcs: [ | 
|  | 425 | "bar.aconfig", | 
|  | 426 | ], | 
|  | 427 | } | 
|  | 428 | aconfig_declarations { | 
|  | 429 | name: "baz", | 
|  | 430 | package: "com.example.package.baz", | 
|  | 431 | srcs: [ | 
|  | 432 | "baz.aconfig", | 
|  | 433 | ], | 
|  | 434 | } | 
|  | 435 | `) | 
|  | 436 |  | 
|  | 437 | foo := result.ModuleForTests("foo", "android_common") | 
|  | 438 |  | 
|  | 439 | // runtime_resource_overlay module depends on aconfig_declarations listed in flags_packages | 
|  | 440 | android.AssertBoolEquals(t, "foo expected to depend on bar", true, | 
|  | 441 | CheckModuleHasDependency(t, result.TestContext, "foo", "android_common", "bar")) | 
|  | 442 |  | 
|  | 443 | android.AssertBoolEquals(t, "foo expected to depend on baz", true, | 
|  | 444 | CheckModuleHasDependency(t, result.TestContext, "foo", "android_common", "baz")) | 
|  | 445 |  | 
|  | 446 | aapt2LinkRule := foo.Rule("android/soong/java.aapt2Link") | 
|  | 447 | linkInFlags := aapt2LinkRule.Args["inFlags"] | 
|  | 448 | android.AssertStringDoesContain(t, | 
|  | 449 | "aapt2 link command expected to pass feature flags arguments", | 
|  | 450 | linkInFlags, | 
|  | 451 | "--feature-flags @out/soong/.intermediates/bar/intermediate.txt --feature-flags @out/soong/.intermediates/baz/intermediate.txt", | 
|  | 452 | ) | 
|  | 453 | } |