| Colin Cross | ce75d2c | 2016-10-06 16:12:58 -0700 | [diff] [blame] | 1 | // Copyright 2016 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 android | 
|  | 16 |  | 
|  | 17 | import ( | 
|  | 18 | "testing" | 
|  | 19 |  | 
|  | 20 | "github.com/google/blueprint" | 
|  | 21 | ) | 
|  | 22 |  | 
| Colin Cross | 0c66bc6 | 2021-07-20 09:47:41 -0700 | [diff] [blame] | 23 | func TestPrebuilts(t *testing.T) { | 
|  | 24 | buildOS := TestArchConfig(t.TempDir(), nil, "", nil).BuildOS | 
| Martin Stjernholm | 009a9dc | 2020-03-05 17:34:13 +0000 | [diff] [blame] | 25 |  | 
| Colin Cross | 0c66bc6 | 2021-07-20 09:47:41 -0700 | [diff] [blame] | 26 | var prebuiltsTests = []struct { | 
|  | 27 | name      string | 
|  | 28 | replaceBp bool // modules is added to default bp boilerplate if false. | 
|  | 29 | modules   string | 
|  | 30 | prebuilt  []OsType | 
|  | 31 | preparer  FixturePreparer | 
|  | 32 | }{ | 
|  | 33 | { | 
|  | 34 | name: "no prebuilt", | 
|  | 35 | modules: ` | 
|  | 36 | source { | 
|  | 37 | name: "bar", | 
|  | 38 | }`, | 
|  | 39 | prebuilt: nil, | 
|  | 40 | }, | 
|  | 41 | { | 
|  | 42 | name: "no source prebuilt not preferred", | 
|  | 43 | modules: ` | 
|  | 44 | prebuilt { | 
|  | 45 | name: "bar", | 
|  | 46 | prefer: false, | 
|  | 47 | srcs: ["prebuilt_file"], | 
|  | 48 | }`, | 
|  | 49 | prebuilt: []OsType{Android, buildOS}, | 
|  | 50 | }, | 
|  | 51 | { | 
|  | 52 | name: "no source prebuilt preferred", | 
|  | 53 | modules: ` | 
|  | 54 | prebuilt { | 
|  | 55 | name: "bar", | 
|  | 56 | prefer: true, | 
|  | 57 | srcs: ["prebuilt_file"], | 
|  | 58 | }`, | 
|  | 59 | prebuilt: []OsType{Android, buildOS}, | 
|  | 60 | }, | 
|  | 61 | { | 
|  | 62 | name: "prebuilt not preferred", | 
|  | 63 | modules: ` | 
|  | 64 | source { | 
|  | 65 | name: "bar", | 
|  | 66 | } | 
| Martin Stjernholm | 009a9dc | 2020-03-05 17:34:13 +0000 | [diff] [blame] | 67 |  | 
| Colin Cross | 0c66bc6 | 2021-07-20 09:47:41 -0700 | [diff] [blame] | 68 | prebuilt { | 
|  | 69 | name: "bar", | 
|  | 70 | prefer: false, | 
|  | 71 | srcs: ["prebuilt_file"], | 
|  | 72 | }`, | 
|  | 73 | prebuilt: nil, | 
|  | 74 | }, | 
|  | 75 | { | 
|  | 76 | name: "prebuilt preferred", | 
|  | 77 | modules: ` | 
|  | 78 | source { | 
|  | 79 | name: "bar", | 
|  | 80 | } | 
| Martin Stjernholm | 009a9dc | 2020-03-05 17:34:13 +0000 | [diff] [blame] | 81 |  | 
| Colin Cross | 0c66bc6 | 2021-07-20 09:47:41 -0700 | [diff] [blame] | 82 | prebuilt { | 
|  | 83 | name: "bar", | 
|  | 84 | prefer: true, | 
|  | 85 | srcs: ["prebuilt_file"], | 
|  | 86 | }`, | 
|  | 87 | prebuilt: []OsType{Android, buildOS}, | 
|  | 88 | }, | 
|  | 89 | { | 
|  | 90 | name: "prebuilt no file not preferred", | 
|  | 91 | modules: ` | 
|  | 92 | source { | 
|  | 93 | name: "bar", | 
|  | 94 | } | 
| Martin Stjernholm | 009a9dc | 2020-03-05 17:34:13 +0000 | [diff] [blame] | 95 |  | 
| Colin Cross | 0c66bc6 | 2021-07-20 09:47:41 -0700 | [diff] [blame] | 96 | prebuilt { | 
|  | 97 | name: "bar", | 
|  | 98 | prefer: false, | 
|  | 99 | }`, | 
|  | 100 | prebuilt: nil, | 
|  | 101 | }, | 
|  | 102 | { | 
|  | 103 | name: "prebuilt no file preferred", | 
|  | 104 | modules: ` | 
|  | 105 | source { | 
|  | 106 | name: "bar", | 
|  | 107 | } | 
| Martin Stjernholm | 009a9dc | 2020-03-05 17:34:13 +0000 | [diff] [blame] | 108 |  | 
| Colin Cross | 0c66bc6 | 2021-07-20 09:47:41 -0700 | [diff] [blame] | 109 | prebuilt { | 
|  | 110 | name: "bar", | 
|  | 111 | prefer: true, | 
|  | 112 | }`, | 
|  | 113 | prebuilt: nil, | 
|  | 114 | }, | 
|  | 115 | { | 
|  | 116 | name: "prebuilt file from filegroup preferred", | 
|  | 117 | modules: ` | 
|  | 118 | filegroup { | 
|  | 119 | name: "fg", | 
|  | 120 | srcs: ["prebuilt_file"], | 
|  | 121 | } | 
|  | 122 | prebuilt { | 
|  | 123 | name: "bar", | 
|  | 124 | prefer: true, | 
|  | 125 | srcs: [":fg"], | 
|  | 126 | }`, | 
|  | 127 | prebuilt: []OsType{Android, buildOS}, | 
|  | 128 | }, | 
|  | 129 | { | 
|  | 130 | name: "prebuilt module for device only", | 
|  | 131 | modules: ` | 
|  | 132 | source { | 
|  | 133 | name: "bar", | 
|  | 134 | } | 
| Martin Stjernholm | 009a9dc | 2020-03-05 17:34:13 +0000 | [diff] [blame] | 135 |  | 
| Colin Cross | 0c66bc6 | 2021-07-20 09:47:41 -0700 | [diff] [blame] | 136 | prebuilt { | 
|  | 137 | name: "bar", | 
|  | 138 | host_supported: false, | 
|  | 139 | prefer: true, | 
|  | 140 | srcs: ["prebuilt_file"], | 
|  | 141 | }`, | 
|  | 142 | prebuilt: []OsType{Android}, | 
|  | 143 | }, | 
|  | 144 | { | 
|  | 145 | name: "prebuilt file for host only", | 
|  | 146 | modules: ` | 
|  | 147 | source { | 
|  | 148 | name: "bar", | 
|  | 149 | } | 
| Jaewoong Jung | fb25a64 | 2020-06-11 10:37:51 -0700 | [diff] [blame] | 150 |  | 
| Colin Cross | 0c66bc6 | 2021-07-20 09:47:41 -0700 | [diff] [blame] | 151 | prebuilt { | 
|  | 152 | name: "bar", | 
|  | 153 | prefer: true, | 
|  | 154 | target: { | 
|  | 155 | host: { | 
|  | 156 | srcs: ["prebuilt_file"], | 
| Liz Kammer | 69d6413 | 2021-06-24 13:03:06 -0400 | [diff] [blame] | 157 | }, | 
|  | 158 | }, | 
| Colin Cross | 0c66bc6 | 2021-07-20 09:47:41 -0700 | [diff] [blame] | 159 | }`, | 
|  | 160 | prebuilt: []OsType{buildOS}, | 
|  | 161 | }, | 
|  | 162 | { | 
|  | 163 | name: "prebuilt override not preferred", | 
|  | 164 | modules: ` | 
|  | 165 | source { | 
|  | 166 | name: "baz", | 
|  | 167 | } | 
| Paul Duffin | 0c52c7b | 2021-07-06 17:15:25 +0100 | [diff] [blame] | 168 |  | 
| Colin Cross | 0c66bc6 | 2021-07-20 09:47:41 -0700 | [diff] [blame] | 169 | override_source { | 
|  | 170 | name: "bar", | 
|  | 171 | base: "baz", | 
|  | 172 | } | 
| Paul Duffin | 0c52c7b | 2021-07-06 17:15:25 +0100 | [diff] [blame] | 173 |  | 
| Colin Cross | 0c66bc6 | 2021-07-20 09:47:41 -0700 | [diff] [blame] | 174 | prebuilt { | 
|  | 175 | name: "bar", | 
|  | 176 | prefer: false, | 
|  | 177 | srcs: ["prebuilt_file"], | 
|  | 178 | }`, | 
|  | 179 | prebuilt: nil, | 
|  | 180 | }, | 
|  | 181 | { | 
|  | 182 | name: "prebuilt override preferred", | 
|  | 183 | modules: ` | 
|  | 184 | source { | 
|  | 185 | name: "baz", | 
|  | 186 | } | 
| Paul Duffin | 0c52c7b | 2021-07-06 17:15:25 +0100 | [diff] [blame] | 187 |  | 
| Colin Cross | 0c66bc6 | 2021-07-20 09:47:41 -0700 | [diff] [blame] | 188 | override_source { | 
|  | 189 | name: "bar", | 
|  | 190 | base: "baz", | 
|  | 191 | } | 
| Colin Cross | ce75d2c | 2016-10-06 16:12:58 -0700 | [diff] [blame] | 192 |  | 
| Colin Cross | 0c66bc6 | 2021-07-20 09:47:41 -0700 | [diff] [blame] | 193 | prebuilt { | 
|  | 194 | name: "bar", | 
|  | 195 | prefer: true, | 
|  | 196 | srcs: ["prebuilt_file"], | 
|  | 197 | }`, | 
|  | 198 | prebuilt: []OsType{Android, buildOS}, | 
|  | 199 | }, | 
|  | 200 | { | 
|  | 201 | name:      "prebuilt including default-disabled OS", | 
|  | 202 | replaceBp: true, | 
|  | 203 | modules: ` | 
|  | 204 | source { | 
|  | 205 | name: "foo", | 
|  | 206 | deps: [":bar"], | 
|  | 207 | target: { | 
|  | 208 | windows: { | 
|  | 209 | enabled: true, | 
|  | 210 | }, | 
|  | 211 | }, | 
|  | 212 | } | 
|  | 213 |  | 
|  | 214 | source { | 
|  | 215 | name: "bar", | 
|  | 216 | target: { | 
|  | 217 | windows: { | 
|  | 218 | enabled: true, | 
|  | 219 | }, | 
|  | 220 | }, | 
|  | 221 | } | 
|  | 222 |  | 
|  | 223 | prebuilt { | 
|  | 224 | name: "bar", | 
|  | 225 | prefer: true, | 
|  | 226 | srcs: ["prebuilt_file"], | 
|  | 227 | target: { | 
|  | 228 | windows: { | 
|  | 229 | enabled: true, | 
|  | 230 | }, | 
|  | 231 | }, | 
|  | 232 | }`, | 
|  | 233 | prebuilt: []OsType{Android, buildOS, Windows}, | 
|  | 234 | }, | 
|  | 235 | { | 
|  | 236 | name:      "fall back to source for default-disabled OS", | 
|  | 237 | replaceBp: true, | 
|  | 238 | modules: ` | 
|  | 239 | source { | 
|  | 240 | name: "foo", | 
|  | 241 | deps: [":bar"], | 
|  | 242 | target: { | 
|  | 243 | windows: { | 
|  | 244 | enabled: true, | 
|  | 245 | }, | 
|  | 246 | }, | 
|  | 247 | } | 
|  | 248 |  | 
|  | 249 | source { | 
|  | 250 | name: "bar", | 
|  | 251 | target: { | 
|  | 252 | windows: { | 
|  | 253 | enabled: true, | 
|  | 254 | }, | 
|  | 255 | }, | 
|  | 256 | } | 
|  | 257 |  | 
|  | 258 | prebuilt { | 
|  | 259 | name: "bar", | 
|  | 260 | prefer: true, | 
|  | 261 | srcs: ["prebuilt_file"], | 
|  | 262 | }`, | 
|  | 263 | prebuilt: []OsType{Android, buildOS}, | 
|  | 264 | }, | 
|  | 265 | { | 
|  | 266 | name:      "prebuilt properties customizable", | 
|  | 267 | replaceBp: true, | 
|  | 268 | modules: ` | 
|  | 269 | source { | 
|  | 270 | name: "foo", | 
|  | 271 | deps: [":bar"], | 
|  | 272 | } | 
|  | 273 |  | 
|  | 274 | soong_config_module_type { | 
|  | 275 | name: "prebuilt_with_config", | 
|  | 276 | module_type: "prebuilt", | 
|  | 277 | config_namespace: "any_namespace", | 
|  | 278 | bool_variables: ["bool_var"], | 
|  | 279 | properties: ["prefer"], | 
|  | 280 | } | 
|  | 281 |  | 
|  | 282 | prebuilt_with_config { | 
|  | 283 | name: "bar", | 
|  | 284 | prefer: true, | 
|  | 285 | srcs: ["prebuilt_file"], | 
|  | 286 | soong_config_variables: { | 
|  | 287 | bool_var: { | 
|  | 288 | prefer: false, | 
|  | 289 | conditions_default: { | 
|  | 290 | prefer: true, | 
|  | 291 | }, | 
|  | 292 | }, | 
|  | 293 | }, | 
|  | 294 | }`, | 
|  | 295 | prebuilt: []OsType{Android, buildOS}, | 
|  | 296 | }, | 
| Colin Cross | 0c66bc6 | 2021-07-20 09:47:41 -0700 | [diff] [blame] | 297 | } | 
|  | 298 |  | 
| Paul Duffin | 3581612 | 2021-02-24 01:49:52 +0000 | [diff] [blame] | 299 | fs := MockFS{ | 
| Colin Cross | 98be1bb | 2019-12-13 20:41:13 -0800 | [diff] [blame] | 300 | "prebuilt_file": nil, | 
|  | 301 | "source_file":   nil, | 
|  | 302 | } | 
| Colin Cross | 0d614dd | 2016-10-14 15:38:43 -0700 | [diff] [blame] | 303 |  | 
| Colin Cross | ce75d2c | 2016-10-06 16:12:58 -0700 | [diff] [blame] | 304 | for _, test := range prebuiltsTests { | 
|  | 305 | t.Run(test.name, func(t *testing.T) { | 
| Martin Stjernholm | f467732 | 2020-07-07 02:20:40 +0100 | [diff] [blame] | 306 | bp := test.modules | 
|  | 307 | if !test.replaceBp { | 
|  | 308 | bp = bp + ` | 
|  | 309 | source { | 
|  | 310 | name: "foo", | 
|  | 311 | deps: [":bar"], | 
|  | 312 | }` | 
|  | 313 | } | 
| Colin Cross | 98be1bb | 2019-12-13 20:41:13 -0800 | [diff] [blame] | 314 |  | 
| Martin Stjernholm | f467732 | 2020-07-07 02:20:40 +0100 | [diff] [blame] | 315 | // Add windows to the target list to test the logic when a variant is | 
|  | 316 | // disabled by default. | 
|  | 317 | if !Windows.DefaultDisabled { | 
|  | 318 | t.Errorf("windows is assumed to be disabled by default") | 
|  | 319 | } | 
| Martin Stjernholm | f467732 | 2020-07-07 02:20:40 +0100 | [diff] [blame] | 320 |  | 
| Paul Duffin | 30ac3e7 | 2021-03-20 00:36:14 +0000 | [diff] [blame] | 321 | result := GroupFixturePreparers( | 
| Paul Duffin | 3581612 | 2021-02-24 01:49:52 +0000 | [diff] [blame] | 322 | PrepareForTestWithArchMutator, | 
|  | 323 | PrepareForTestWithPrebuilts, | 
|  | 324 | PrepareForTestWithOverrides, | 
|  | 325 | PrepareForTestWithFilegroup, | 
|  | 326 | // Add a Windows target to the configuration. | 
|  | 327 | FixtureModifyConfig(func(config Config) { | 
|  | 328 | config.Targets[Windows] = []Target{ | 
|  | 329 | {Windows, Arch{ArchType: X86_64}, NativeBridgeDisabled, "", "", true}, | 
|  | 330 | } | 
|  | 331 | }), | 
|  | 332 | fs.AddToFixture(), | 
|  | 333 | FixtureRegisterWithContext(registerTestPrebuiltModules), | 
| Paul Duffin | 0c52c7b | 2021-07-06 17:15:25 +0100 | [diff] [blame] | 334 | OptionalFixturePreparer(test.preparer), | 
| Paul Duffin | 3581612 | 2021-02-24 01:49:52 +0000 | [diff] [blame] | 335 | ).RunTestWithBp(t, bp) | 
| Colin Cross | ce75d2c | 2016-10-06 16:12:58 -0700 | [diff] [blame] | 336 |  | 
| Paul Duffin | 3581612 | 2021-02-24 01:49:52 +0000 | [diff] [blame] | 337 | for _, variant := range result.ModuleVariantsForTests("foo") { | 
|  | 338 | foo := result.ModuleForTests("foo", variant) | 
| Martin Stjernholm | f467732 | 2020-07-07 02:20:40 +0100 | [diff] [blame] | 339 | t.Run(foo.Module().Target().Os.String(), func(t *testing.T) { | 
| Martin Stjernholm | 009a9dc | 2020-03-05 17:34:13 +0000 | [diff] [blame] | 340 | var dependsOnSourceModule, dependsOnPrebuiltModule bool | 
| Paul Duffin | 3581612 | 2021-02-24 01:49:52 +0000 | [diff] [blame] | 341 | result.VisitDirectDeps(foo.Module(), func(m blueprint.Module) { | 
| Martin Stjernholm | 009a9dc | 2020-03-05 17:34:13 +0000 | [diff] [blame] | 342 | if _, ok := m.(*sourceModule); ok { | 
|  | 343 | dependsOnSourceModule = true | 
|  | 344 | } | 
|  | 345 | if p, ok := m.(*prebuiltModule); ok { | 
|  | 346 | dependsOnPrebuiltModule = true | 
|  | 347 | if !p.Prebuilt().properties.UsePrebuilt { | 
|  | 348 | t.Errorf("dependency on prebuilt module not marked used") | 
|  | 349 | } | 
|  | 350 | } | 
|  | 351 | }) | 
| Colin Cross | ce75d2c | 2016-10-06 16:12:58 -0700 | [diff] [blame] | 352 |  | 
| Cole Faust | a963b94 | 2024-04-11 17:43:00 -0700 | [diff] [blame] | 353 | moduleIsDisabled := !foo.Module().Enabled(PanickingConfigAndErrorContext(result.TestContext)) | 
| Martin Stjernholm | 009a9dc | 2020-03-05 17:34:13 +0000 | [diff] [blame] | 354 | deps := foo.Module().(*sourceModule).deps | 
| Martin Stjernholm | f467732 | 2020-07-07 02:20:40 +0100 | [diff] [blame] | 355 | if moduleIsDisabled { | 
|  | 356 | if len(deps) > 0 { | 
|  | 357 | t.Errorf("disabled module got deps: %v", deps) | 
|  | 358 | } | 
|  | 359 | } else { | 
|  | 360 | if len(deps) != 1 { | 
|  | 361 | t.Errorf("deps does not have single path, but is %v", deps) | 
|  | 362 | } | 
| Colin Cross | c3e7fa6 | 2017-03-17 13:14:32 -0700 | [diff] [blame] | 363 | } | 
| Martin Stjernholm | f467732 | 2020-07-07 02:20:40 +0100 | [diff] [blame] | 364 |  | 
| Martin Stjernholm | 009a9dc | 2020-03-05 17:34:13 +0000 | [diff] [blame] | 365 | var usingSourceFile, usingPrebuiltFile bool | 
| Martin Stjernholm | f467732 | 2020-07-07 02:20:40 +0100 | [diff] [blame] | 366 | if len(deps) > 0 && deps[0].String() == "source_file" { | 
| Martin Stjernholm | 009a9dc | 2020-03-05 17:34:13 +0000 | [diff] [blame] | 367 | usingSourceFile = true | 
|  | 368 | } | 
| Martin Stjernholm | f467732 | 2020-07-07 02:20:40 +0100 | [diff] [blame] | 369 | if len(deps) > 0 && deps[0].String() == "prebuilt_file" { | 
| Martin Stjernholm | 009a9dc | 2020-03-05 17:34:13 +0000 | [diff] [blame] | 370 | usingPrebuiltFile = true | 
|  | 371 | } | 
| Colin Cross | c3e7fa6 | 2017-03-17 13:14:32 -0700 | [diff] [blame] | 372 |  | 
| Martin Stjernholm | 009a9dc | 2020-03-05 17:34:13 +0000 | [diff] [blame] | 373 | prebuilt := false | 
|  | 374 | for _, os := range test.prebuilt { | 
| Martin Stjernholm | f467732 | 2020-07-07 02:20:40 +0100 | [diff] [blame] | 375 | if os == foo.Module().Target().Os { | 
| Martin Stjernholm | 009a9dc | 2020-03-05 17:34:13 +0000 | [diff] [blame] | 376 | prebuilt = true | 
|  | 377 | } | 
|  | 378 | } | 
| Jiyong Park | d467111 | 2019-04-19 17:23:04 +0900 | [diff] [blame] | 379 |  | 
| Martin Stjernholm | 009a9dc | 2020-03-05 17:34:13 +0000 | [diff] [blame] | 380 | if prebuilt { | 
| Martin Stjernholm | f467732 | 2020-07-07 02:20:40 +0100 | [diff] [blame] | 381 | if moduleIsDisabled { | 
|  | 382 | t.Errorf("dependent module for prebuilt is disabled") | 
|  | 383 | } | 
|  | 384 |  | 
| Martin Stjernholm | 009a9dc | 2020-03-05 17:34:13 +0000 | [diff] [blame] | 385 | if !dependsOnPrebuiltModule { | 
|  | 386 | t.Errorf("doesn't depend on prebuilt module") | 
|  | 387 | } | 
|  | 388 | if !usingPrebuiltFile { | 
|  | 389 | t.Errorf("doesn't use prebuilt_file") | 
|  | 390 | } | 
| Colin Cross | ce75d2c | 2016-10-06 16:12:58 -0700 | [diff] [blame] | 391 |  | 
| Martin Stjernholm | 009a9dc | 2020-03-05 17:34:13 +0000 | [diff] [blame] | 392 | if dependsOnSourceModule { | 
|  | 393 | t.Errorf("depends on source module") | 
|  | 394 | } | 
|  | 395 | if usingSourceFile { | 
|  | 396 | t.Errorf("using source_file") | 
|  | 397 | } | 
| Martin Stjernholm | f467732 | 2020-07-07 02:20:40 +0100 | [diff] [blame] | 398 | } else if !moduleIsDisabled { | 
| Martin Stjernholm | 009a9dc | 2020-03-05 17:34:13 +0000 | [diff] [blame] | 399 | if dependsOnPrebuiltModule { | 
|  | 400 | t.Errorf("depends on prebuilt module") | 
|  | 401 | } | 
|  | 402 | if usingPrebuiltFile { | 
|  | 403 | t.Errorf("using prebuilt_file") | 
|  | 404 | } | 
| Colin Cross | ce75d2c | 2016-10-06 16:12:58 -0700 | [diff] [blame] | 405 |  | 
| Martin Stjernholm | 009a9dc | 2020-03-05 17:34:13 +0000 | [diff] [blame] | 406 | if !dependsOnSourceModule { | 
|  | 407 | t.Errorf("doesn't depend on source module") | 
|  | 408 | } | 
|  | 409 | if !usingSourceFile { | 
|  | 410 | t.Errorf("doesn't use source_file") | 
|  | 411 | } | 
|  | 412 | } | 
|  | 413 | }) | 
| Colin Cross | ce75d2c | 2016-10-06 16:12:58 -0700 | [diff] [blame] | 414 | } | 
|  | 415 | }) | 
|  | 416 | } | 
| Colin Cross | ce75d2c | 2016-10-06 16:12:58 -0700 | [diff] [blame] | 417 | } | 
|  | 418 |  | 
| Spandan Das | e3fcb41 | 2023-10-26 20:48:02 +0000 | [diff] [blame] | 419 | func testPrebuiltErrorWithFixture(t *testing.T, expectedError, bp string, fixture FixturePreparer) { | 
| Jooyung Han | ebaa573 | 2023-05-02 11:43:14 +0900 | [diff] [blame] | 420 | t.Helper() | 
|  | 421 | fs := MockFS{ | 
|  | 422 | "prebuilt_file": nil, | 
|  | 423 | } | 
|  | 424 | GroupFixturePreparers( | 
|  | 425 | PrepareForTestWithArchMutator, | 
|  | 426 | PrepareForTestWithPrebuilts, | 
|  | 427 | PrepareForTestWithOverrides, | 
|  | 428 | fs.AddToFixture(), | 
|  | 429 | FixtureRegisterWithContext(registerTestPrebuiltModules), | 
| Spandan Das | e3fcb41 | 2023-10-26 20:48:02 +0000 | [diff] [blame] | 430 | OptionalFixturePreparer(fixture), | 
| Jooyung Han | ebaa573 | 2023-05-02 11:43:14 +0900 | [diff] [blame] | 431 | ). | 
|  | 432 | ExtendWithErrorHandler(FixtureExpectsAtLeastOneErrorMatchingPattern(expectedError)). | 
|  | 433 | RunTestWithBp(t, bp) | 
| Spandan Das | e3fcb41 | 2023-10-26 20:48:02 +0000 | [diff] [blame] | 434 |  | 
|  | 435 | } | 
|  | 436 |  | 
|  | 437 | func testPrebuiltError(t *testing.T, expectedError, bp string) { | 
|  | 438 | testPrebuiltErrorWithFixture(t, expectedError, bp, nil) | 
| Jooyung Han | ebaa573 | 2023-05-02 11:43:14 +0900 | [diff] [blame] | 439 | } | 
|  | 440 |  | 
|  | 441 | func TestPrebuiltShouldNotChangePartition(t *testing.T) { | 
|  | 442 | testPrebuiltError(t, `partition is different`, ` | 
|  | 443 | source { | 
|  | 444 | name: "foo", | 
|  | 445 | vendor: true, | 
|  | 446 | } | 
|  | 447 | prebuilt { | 
|  | 448 | name: "foo", | 
|  | 449 | prefer: true, | 
|  | 450 | srcs: ["prebuilt_file"], | 
|  | 451 | }`) | 
|  | 452 | } | 
|  | 453 |  | 
|  | 454 | func TestPrebuiltShouldNotChangePartition_WithOverride(t *testing.T) { | 
|  | 455 | testPrebuiltError(t, `partition is different`, ` | 
|  | 456 | source { | 
|  | 457 | name: "foo", | 
|  | 458 | vendor: true, | 
|  | 459 | } | 
|  | 460 | override_source { | 
|  | 461 | name: "bar", | 
|  | 462 | base: "foo", | 
|  | 463 | } | 
|  | 464 | prebuilt { | 
|  | 465 | name: "bar", | 
|  | 466 | prefer: true, | 
|  | 467 | srcs: ["prebuilt_file"], | 
|  | 468 | }`) | 
|  | 469 | } | 
|  | 470 |  | 
| Paul Duffin | 78ac5b9 | 2020-01-14 12:42:08 +0000 | [diff] [blame] | 471 | func registerTestPrebuiltBuildComponents(ctx RegistrationContext) { | 
| Paul Duffin | 3581612 | 2021-02-24 01:49:52 +0000 | [diff] [blame] | 472 | registerTestPrebuiltModules(ctx) | 
| Paul Duffin | 78ac5b9 | 2020-01-14 12:42:08 +0000 | [diff] [blame] | 473 |  | 
|  | 474 | RegisterPrebuiltMutators(ctx) | 
| Jaewoong Jung | fb25a64 | 2020-06-11 10:37:51 -0700 | [diff] [blame] | 475 | ctx.PostDepsMutators(RegisterOverridePostDepsMutators) | 
| Paul Duffin | 78ac5b9 | 2020-01-14 12:42:08 +0000 | [diff] [blame] | 476 | } | 
|  | 477 |  | 
| Paul Duffin | cfd3374 | 2021-02-27 11:59:02 +0000 | [diff] [blame] | 478 | var prepareForTestWithFakePrebuiltModules = FixtureRegisterWithContext(registerTestPrebuiltModules) | 
|  | 479 |  | 
| Paul Duffin | 3581612 | 2021-02-24 01:49:52 +0000 | [diff] [blame] | 480 | func registerTestPrebuiltModules(ctx RegistrationContext) { | 
|  | 481 | ctx.RegisterModuleType("prebuilt", newPrebuiltModule) | 
|  | 482 | ctx.RegisterModuleType("source", newSourceModule) | 
|  | 483 | ctx.RegisterModuleType("override_source", newOverrideSourceModule) | 
| Jingwen Chen | a47f28d | 2021-11-02 16:43:57 +0000 | [diff] [blame] | 484 | ctx.RegisterModuleType("soong_config_module_type", SoongConfigModuleTypeFactory) | 
|  | 485 | ctx.RegisterModuleType("soong_config_string_variable", SoongConfigStringVariableDummyFactory) | 
|  | 486 | ctx.RegisterModuleType("soong_config_bool_variable", SoongConfigBoolVariableDummyFactory) | 
| Spandan Das | e3fcb41 | 2023-10-26 20:48:02 +0000 | [diff] [blame] | 487 | RegisterApexContributionsBuildComponents(ctx) | 
| Paul Duffin | 3581612 | 2021-02-24 01:49:52 +0000 | [diff] [blame] | 488 | } | 
|  | 489 |  | 
| Colin Cross | ce75d2c | 2016-10-06 16:12:58 -0700 | [diff] [blame] | 490 | type prebuiltModule struct { | 
|  | 491 | ModuleBase | 
| Colin Cross | 74d73e2 | 2017-08-02 11:05:49 -0700 | [diff] [blame] | 492 | prebuilt   Prebuilt | 
|  | 493 | properties struct { | 
| Martin Stjernholm | 009a9dc | 2020-03-05 17:34:13 +0000 | [diff] [blame] | 494 | Srcs []string `android:"path,arch_variant"` | 
| Colin Cross | 74d73e2 | 2017-08-02 11:05:49 -0700 | [diff] [blame] | 495 | } | 
| Colin Cross | ce75d2c | 2016-10-06 16:12:58 -0700 | [diff] [blame] | 496 | } | 
|  | 497 |  | 
| Colin Cross | 3624285 | 2017-06-23 15:06:31 -0700 | [diff] [blame] | 498 | func newPrebuiltModule() Module { | 
| Colin Cross | ce75d2c | 2016-10-06 16:12:58 -0700 | [diff] [blame] | 499 | m := &prebuiltModule{} | 
| Colin Cross | 74d73e2 | 2017-08-02 11:05:49 -0700 | [diff] [blame] | 500 | m.AddProperties(&m.properties) | 
|  | 501 | InitPrebuiltModule(m, &m.properties.Srcs) | 
| Martin Stjernholm | 009a9dc | 2020-03-05 17:34:13 +0000 | [diff] [blame] | 502 | InitAndroidArchModule(m, HostAndDeviceDefault, MultilibCommon) | 
| Colin Cross | 3624285 | 2017-06-23 15:06:31 -0700 | [diff] [blame] | 503 | return m | 
| Colin Cross | ce75d2c | 2016-10-06 16:12:58 -0700 | [diff] [blame] | 504 | } | 
|  | 505 |  | 
|  | 506 | func (p *prebuiltModule) Name() string { | 
|  | 507 | return p.prebuilt.Name(p.ModuleBase.Name()) | 
|  | 508 | } | 
|  | 509 |  | 
| Jiyong Park | d467111 | 2019-04-19 17:23:04 +0900 | [diff] [blame] | 510 | func (p *prebuiltModule) GenerateAndroidBuildActions(ctx ModuleContext) { | 
| mrziwang | 01715ca | 2024-07-01 11:50:18 -0700 | [diff] [blame] | 511 | var src Path | 
| Jiyong Park | d467111 | 2019-04-19 17:23:04 +0900 | [diff] [blame] | 512 | if len(p.properties.Srcs) >= 1 { | 
| mrziwang | 01715ca | 2024-07-01 11:50:18 -0700 | [diff] [blame] | 513 | src = p.prebuilt.SingleSourcePath(ctx) | 
| Jiyong Park | d467111 | 2019-04-19 17:23:04 +0900 | [diff] [blame] | 514 | } | 
| mrziwang | 01715ca | 2024-07-01 11:50:18 -0700 | [diff] [blame] | 515 | ctx.SetOutputFiles(Paths{src}, "") | 
| Colin Cross | ce75d2c | 2016-10-06 16:12:58 -0700 | [diff] [blame] | 516 | } | 
|  | 517 |  | 
|  | 518 | func (p *prebuiltModule) Prebuilt() *Prebuilt { | 
|  | 519 | return &p.prebuilt | 
|  | 520 | } | 
|  | 521 |  | 
| Jaewoong Jung | fb25a64 | 2020-06-11 10:37:51 -0700 | [diff] [blame] | 522 | type sourceModuleProperties struct { | 
|  | 523 | Deps []string `android:"path,arch_variant"` | 
|  | 524 | } | 
|  | 525 |  | 
| Colin Cross | ce75d2c | 2016-10-06 16:12:58 -0700 | [diff] [blame] | 526 | type sourceModule struct { | 
|  | 527 | ModuleBase | 
| Jaewoong Jung | fb25a64 | 2020-06-11 10:37:51 -0700 | [diff] [blame] | 528 | OverridableModuleBase | 
|  | 529 |  | 
|  | 530 | properties                                     sourceModuleProperties | 
| Colin Cross | ce75d2c | 2016-10-06 16:12:58 -0700 | [diff] [blame] | 531 | dependsOnSourceModule, dependsOnPrebuiltModule bool | 
| Jiyong Park | d467111 | 2019-04-19 17:23:04 +0900 | [diff] [blame] | 532 | deps                                           Paths | 
|  | 533 | src                                            Path | 
| Colin Cross | ce75d2c | 2016-10-06 16:12:58 -0700 | [diff] [blame] | 534 | } | 
|  | 535 |  | 
| Colin Cross | 3624285 | 2017-06-23 15:06:31 -0700 | [diff] [blame] | 536 | func newSourceModule() Module { | 
| Colin Cross | ce75d2c | 2016-10-06 16:12:58 -0700 | [diff] [blame] | 537 | m := &sourceModule{} | 
| Colin Cross | 3624285 | 2017-06-23 15:06:31 -0700 | [diff] [blame] | 538 | m.AddProperties(&m.properties) | 
| Martin Stjernholm | 009a9dc | 2020-03-05 17:34:13 +0000 | [diff] [blame] | 539 | InitAndroidArchModule(m, HostAndDeviceDefault, MultilibCommon) | 
| Jaewoong Jung | fb25a64 | 2020-06-11 10:37:51 -0700 | [diff] [blame] | 540 | InitOverridableModule(m, nil) | 
| Colin Cross | 3624285 | 2017-06-23 15:06:31 -0700 | [diff] [blame] | 541 | return m | 
| Colin Cross | ce75d2c | 2016-10-06 16:12:58 -0700 | [diff] [blame] | 542 | } | 
|  | 543 |  | 
| Jaewoong Jung | fb25a64 | 2020-06-11 10:37:51 -0700 | [diff] [blame] | 544 | func (s *sourceModule) OverridablePropertiesDepsMutator(ctx BottomUpMutatorContext) { | 
| Jiyong Park | d467111 | 2019-04-19 17:23:04 +0900 | [diff] [blame] | 545 | // s.properties.Deps are annotated with android:path, so they are | 
|  | 546 | // automatically added to the dependency by pathDeps mutator | 
| Colin Cross | ce75d2c | 2016-10-06 16:12:58 -0700 | [diff] [blame] | 547 | } | 
|  | 548 |  | 
|  | 549 | func (s *sourceModule) GenerateAndroidBuildActions(ctx ModuleContext) { | 
| Jiyong Park | d467111 | 2019-04-19 17:23:04 +0900 | [diff] [blame] | 550 | s.deps = PathsForModuleSrc(ctx, s.properties.Deps) | 
|  | 551 | s.src = PathForModuleSrc(ctx, "source_file") | 
|  | 552 | } | 
|  | 553 |  | 
|  | 554 | func (s *sourceModule) Srcs() Paths { | 
|  | 555 | return Paths{s.src} | 
| Colin Cross | ce75d2c | 2016-10-06 16:12:58 -0700 | [diff] [blame] | 556 | } | 
| Jaewoong Jung | fb25a64 | 2020-06-11 10:37:51 -0700 | [diff] [blame] | 557 |  | 
|  | 558 | type overrideSourceModule struct { | 
|  | 559 | ModuleBase | 
|  | 560 | OverrideModuleBase | 
|  | 561 | } | 
|  | 562 |  | 
|  | 563 | func (o *overrideSourceModule) GenerateAndroidBuildActions(_ ModuleContext) { | 
|  | 564 | } | 
|  | 565 |  | 
|  | 566 | func newOverrideSourceModule() Module { | 
|  | 567 | m := &overrideSourceModule{} | 
|  | 568 | m.AddProperties(&sourceModuleProperties{}) | 
|  | 569 |  | 
|  | 570 | InitAndroidArchModule(m, HostAndDeviceDefault, MultilibCommon) | 
|  | 571 | InitOverrideModule(m) | 
|  | 572 | return m | 
|  | 573 | } | 
| Spandan Das | e3fcb41 | 2023-10-26 20:48:02 +0000 | [diff] [blame] | 574 |  | 
|  | 575 | func TestPrebuiltErrorCannotListBothSourceAndPrebuiltInContributions(t *testing.T) { | 
|  | 576 | selectMainlineModuleContritbutions := GroupFixturePreparers( | 
| Colin Cross | a66b463 | 2024-08-08 15:50:47 -0700 | [diff] [blame] | 577 | PrepareForTestWithBuildFlag("RELEASE_APEX_CONTRIBUTIONS_ADSERVICES", "my_apex_contributions"), | 
| Spandan Das | e3fcb41 | 2023-10-26 20:48:02 +0000 | [diff] [blame] | 578 | ) | 
| Spandan Das | 3576e76 | 2024-01-03 18:57:03 +0000 | [diff] [blame] | 579 | testPrebuiltErrorWithFixture(t, `Found duplicate variations of the same module in apex_contributions: foo and prebuilt_foo. Please remove one of these`, ` | 
| Spandan Das | e3fcb41 | 2023-10-26 20:48:02 +0000 | [diff] [blame] | 580 | source { | 
|  | 581 | name: "foo", | 
|  | 582 | } | 
|  | 583 | prebuilt { | 
|  | 584 | name: "foo", | 
|  | 585 | srcs: ["prebuilt_file"], | 
|  | 586 | } | 
|  | 587 | apex_contributions { | 
|  | 588 | name: "my_apex_contributions", | 
|  | 589 | api_domain: "my_mainline_module", | 
|  | 590 | contents: [ | 
|  | 591 | "foo", | 
|  | 592 | "prebuilt_foo", | 
|  | 593 | ], | 
|  | 594 | } | 
|  | 595 | all_apex_contributions { | 
|  | 596 | name: "all_apex_contributions", | 
|  | 597 | } | 
|  | 598 | `, selectMainlineModuleContritbutions) | 
|  | 599 | } |