Alex Márquez Pérez Muñíz Díaz Púras Thaureaux | f5a3eac | 2021-08-23 17:05:17 +0000 | [diff] [blame] | 1 | // Copyright 2021 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 | |
Liz Kammer | 2dd9ca4 | 2020-11-25 16:06:39 -0800 | [diff] [blame] | 15 | package bp2build |
| 16 | |
Alex Márquez Pérez Muñíz Díaz Púras Thaureaux | 0da7ce6 | 2021-08-23 17:04:20 +0000 | [diff] [blame] | 17 | /* |
| 18 | For shareable/common bp2build testing functionality and dumping ground for |
| 19 | specific-but-shared functionality among tests in package |
| 20 | */ |
| 21 | |
Liz Kammer | 2dd9ca4 | 2020-11-25 16:06:39 -0800 | [diff] [blame] | 22 | import ( |
Alex Márquez Pérez Muñíz Díaz Púras Thaureaux | 1c92aef | 2021-08-23 16:10:00 +0000 | [diff] [blame] | 23 | "strings" |
Rupert Shuttleworth | 06559d0 | 2021-05-19 09:14:26 -0400 | [diff] [blame] | 24 | "testing" |
| 25 | |
Liz Kammer | 2dd9ca4 | 2020-11-25 16:06:39 -0800 | [diff] [blame] | 26 | "android/soong/android" |
Jingwen Chen | 7385067 | 2020-12-14 08:25:34 -0500 | [diff] [blame] | 27 | "android/soong/bazel" |
Liz Kammer | 2dd9ca4 | 2020-11-25 16:06:39 -0800 | [diff] [blame] | 28 | ) |
| 29 | |
Jingwen Chen | 91220d7 | 2021-03-24 02:18:33 -0400 | [diff] [blame] | 30 | var ( |
| 31 | // A default configuration for tests to not have to specify bp2build_available on top level targets. |
| 32 | bp2buildConfig = android.Bp2BuildConfig{ |
| 33 | android.BP2BUILD_TOPLEVEL: android.Bp2BuildDefaultTrueRecursively, |
| 34 | } |
Rupert Shuttleworth | 06559d0 | 2021-05-19 09:14:26 -0400 | [diff] [blame] | 35 | |
| 36 | buildDir string |
Jingwen Chen | 91220d7 | 2021-03-24 02:18:33 -0400 | [diff] [blame] | 37 | ) |
| 38 | |
Jingwen Chen | 5146ac0 | 2021-09-02 11:44:42 +0000 | [diff] [blame] | 39 | func checkError(t *testing.T, errs []error, expectedErr error) bool { |
Alex Márquez Pérez Muñíz Díaz Púras Thaureaux | 1c92aef | 2021-08-23 16:10:00 +0000 | [diff] [blame] | 40 | t.Helper() |
Jingwen Chen | 5146ac0 | 2021-09-02 11:44:42 +0000 | [diff] [blame] | 41 | |
Jingwen Chen | 5146ac0 | 2021-09-02 11:44:42 +0000 | [diff] [blame] | 42 | if len(errs) != 1 { |
Liz Kammer | 6eff323 | 2021-08-26 08:37:59 -0400 | [diff] [blame^] | 43 | return false |
Jingwen Chen | 5146ac0 | 2021-09-02 11:44:42 +0000 | [diff] [blame] | 44 | } |
| 45 | if errs[0].Error() == expectedErr.Error() { |
| 46 | return true |
| 47 | } |
| 48 | |
| 49 | return false |
| 50 | } |
| 51 | |
| 52 | func errored(t *testing.T, tc bp2buildTestCase, errs []error) bool { |
| 53 | t.Helper() |
| 54 | if tc.expectedErr != nil { |
| 55 | // Rely on checkErrors, as this test case is expected to have an error. |
| 56 | return false |
| 57 | } |
| 58 | |
Alex Márquez Pérez Muñíz Díaz Púras Thaureaux | 1c92aef | 2021-08-23 16:10:00 +0000 | [diff] [blame] | 59 | if len(errs) > 0 { |
| 60 | for _, err := range errs { |
Jingwen Chen | 5146ac0 | 2021-09-02 11:44:42 +0000 | [diff] [blame] | 61 | t.Errorf("%s: %s", tc.description, err) |
Alex Márquez Pérez Muñíz Díaz Púras Thaureaux | 1c92aef | 2021-08-23 16:10:00 +0000 | [diff] [blame] | 62 | } |
| 63 | return true |
| 64 | } |
Jingwen Chen | 5146ac0 | 2021-09-02 11:44:42 +0000 | [diff] [blame] | 65 | |
| 66 | // All good, continue execution. |
Alex Márquez Pérez Muñíz Díaz Púras Thaureaux | 1c92aef | 2021-08-23 16:10:00 +0000 | [diff] [blame] | 67 | return false |
| 68 | } |
| 69 | |
Alex Márquez Pérez Muñíz Díaz Púras Thaureaux | ce0a07e | 2021-08-23 16:17:32 +0000 | [diff] [blame] | 70 | func runBp2BuildTestCaseSimple(t *testing.T, tc bp2buildTestCase) { |
Alex Márquez Pérez Muñíz Díaz Púras Thaureaux | 1c92aef | 2021-08-23 16:10:00 +0000 | [diff] [blame] | 71 | t.Helper() |
| 72 | runBp2BuildTestCase(t, func(ctx android.RegistrationContext) {}, tc) |
| 73 | } |
| 74 | |
| 75 | type bp2buildTestCase struct { |
| 76 | description string |
| 77 | moduleTypeUnderTest string |
| 78 | moduleTypeUnderTestFactory android.ModuleFactory |
| 79 | moduleTypeUnderTestBp2BuildMutator func(android.TopDownMutatorContext) |
| 80 | blueprint string |
| 81 | expectedBazelTargets []string |
| 82 | filesystem map[string]string |
| 83 | dir string |
Jingwen Chen | 5146ac0 | 2021-09-02 11:44:42 +0000 | [diff] [blame] | 84 | expectedErr error |
Liz Kammer | 6eff323 | 2021-08-26 08:37:59 -0400 | [diff] [blame^] | 85 | unconvertedDepsMode unconvertedDepsMode |
Alex Márquez Pérez Muñíz Díaz Púras Thaureaux | 1c92aef | 2021-08-23 16:10:00 +0000 | [diff] [blame] | 86 | } |
| 87 | |
| 88 | func runBp2BuildTestCase(t *testing.T, registerModuleTypes func(ctx android.RegistrationContext), tc bp2buildTestCase) { |
| 89 | t.Helper() |
| 90 | dir := "." |
| 91 | filesystem := make(map[string][]byte) |
| 92 | toParse := []string{ |
| 93 | "Android.bp", |
| 94 | } |
| 95 | for f, content := range tc.filesystem { |
| 96 | if strings.HasSuffix(f, "Android.bp") { |
| 97 | toParse = append(toParse, f) |
| 98 | } |
| 99 | filesystem[f] = []byte(content) |
| 100 | } |
| 101 | config := android.TestConfig(buildDir, nil, tc.blueprint, filesystem) |
| 102 | ctx := android.NewTestContext(config) |
| 103 | |
| 104 | registerModuleTypes(ctx) |
| 105 | ctx.RegisterModuleType(tc.moduleTypeUnderTest, tc.moduleTypeUnderTestFactory) |
| 106 | ctx.RegisterBp2BuildConfig(bp2buildConfig) |
| 107 | ctx.RegisterBp2BuildMutator(tc.moduleTypeUnderTest, tc.moduleTypeUnderTestBp2BuildMutator) |
| 108 | ctx.RegisterForBazelConversion() |
| 109 | |
Jingwen Chen | 5146ac0 | 2021-09-02 11:44:42 +0000 | [diff] [blame] | 110 | _, parseErrs := ctx.ParseFileList(dir, toParse) |
| 111 | if errored(t, tc, parseErrs) { |
Alex Márquez Pérez Muñíz Díaz Púras Thaureaux | 1c92aef | 2021-08-23 16:10:00 +0000 | [diff] [blame] | 112 | return |
| 113 | } |
Jingwen Chen | 5146ac0 | 2021-09-02 11:44:42 +0000 | [diff] [blame] | 114 | _, resolveDepsErrs := ctx.ResolveDependencies(config) |
| 115 | if errored(t, tc, resolveDepsErrs) { |
| 116 | return |
| 117 | } |
| 118 | |
| 119 | errs := append(parseErrs, resolveDepsErrs...) |
| 120 | if tc.expectedErr != nil && checkError(t, errs, tc.expectedErr) { |
Alex Márquez Pérez Muñíz Díaz Púras Thaureaux | 1c92aef | 2021-08-23 16:10:00 +0000 | [diff] [blame] | 121 | return |
| 122 | } |
| 123 | |
| 124 | checkDir := dir |
| 125 | if tc.dir != "" { |
| 126 | checkDir = tc.dir |
| 127 | } |
| 128 | codegenCtx := NewCodegenContext(config, *ctx.Context, Bp2Build) |
Liz Kammer | 6eff323 | 2021-08-26 08:37:59 -0400 | [diff] [blame^] | 129 | codegenCtx.unconvertedDepMode = tc.unconvertedDepsMode |
| 130 | bazelTargets, errs := generateBazelTargetsForDir(codegenCtx, checkDir) |
| 131 | if tc.expectedErr != nil && checkError(t, errs, tc.expectedErr) { |
| 132 | return |
| 133 | } else { |
| 134 | android.FailIfErrored(t, errs) |
| 135 | } |
Alex Márquez Pérez Muñíz Díaz Púras Thaureaux | 1c92aef | 2021-08-23 16:10:00 +0000 | [diff] [blame] | 136 | if actualCount, expectedCount := len(bazelTargets), len(tc.expectedBazelTargets); actualCount != expectedCount { |
Alex Márquez Pérez Muñíz Díaz Púras Thaureaux | 560cb66 | 2021-08-26 20:13:29 +0000 | [diff] [blame] | 137 | t.Errorf("%s: Expected %d bazel target, got %d; %v", |
| 138 | tc.description, expectedCount, actualCount, bazelTargets) |
Alex Márquez Pérez Muñíz Díaz Púras Thaureaux | 1c92aef | 2021-08-23 16:10:00 +0000 | [diff] [blame] | 139 | } else { |
| 140 | for i, target := range bazelTargets { |
| 141 | if w, g := tc.expectedBazelTargets[i], target.content; w != g { |
| 142 | t.Errorf( |
| 143 | "%s: Expected generated Bazel target to be '%s', got '%s'", |
| 144 | tc.description, |
| 145 | w, |
| 146 | g, |
| 147 | ) |
| 148 | } |
| 149 | } |
| 150 | } |
| 151 | } |
| 152 | |
Liz Kammer | 2dd9ca4 | 2020-11-25 16:06:39 -0800 | [diff] [blame] | 153 | type nestedProps struct { |
| 154 | Nested_prop string |
| 155 | } |
| 156 | |
| 157 | type customProps struct { |
| 158 | Bool_prop bool |
| 159 | Bool_ptr_prop *bool |
| 160 | // Ensure that properties tagged `blueprint:mutated` are omitted |
| 161 | Int_prop int `blueprint:"mutated"` |
| 162 | Int64_ptr_prop *int64 |
| 163 | String_prop string |
| 164 | String_ptr_prop *string |
| 165 | String_list_prop []string |
| 166 | |
| 167 | Nested_props nestedProps |
| 168 | Nested_props_ptr *nestedProps |
Liz Kammer | 4562a3b | 2021-04-21 18:15:34 -0400 | [diff] [blame] | 169 | |
Liz Kammer | 32b77cf | 2021-08-04 15:17:02 -0400 | [diff] [blame] | 170 | Arch_paths []string `android:"path,arch_variant"` |
| 171 | Arch_paths_exclude []string `android:"path,arch_variant"` |
Liz Kammer | 2dd9ca4 | 2020-11-25 16:06:39 -0800 | [diff] [blame] | 172 | } |
| 173 | |
| 174 | type customModule struct { |
| 175 | android.ModuleBase |
Liz Kammer | ea6666f | 2021-02-17 10:17:28 -0500 | [diff] [blame] | 176 | android.BazelModuleBase |
Liz Kammer | 2dd9ca4 | 2020-11-25 16:06:39 -0800 | [diff] [blame] | 177 | |
| 178 | props customProps |
| 179 | } |
| 180 | |
| 181 | // OutputFiles is needed because some instances of this module use dist with a |
| 182 | // tag property which requires the module implements OutputFileProducer. |
| 183 | func (m *customModule) OutputFiles(tag string) (android.Paths, error) { |
| 184 | return android.PathsForTesting("path" + tag), nil |
| 185 | } |
| 186 | |
| 187 | func (m *customModule) GenerateAndroidBuildActions(ctx android.ModuleContext) { |
| 188 | // nothing for now. |
| 189 | } |
| 190 | |
| 191 | func customModuleFactoryBase() android.Module { |
| 192 | module := &customModule{} |
| 193 | module.AddProperties(&module.props) |
Liz Kammer | ea6666f | 2021-02-17 10:17:28 -0500 | [diff] [blame] | 194 | android.InitBazelModule(module) |
Liz Kammer | 2dd9ca4 | 2020-11-25 16:06:39 -0800 | [diff] [blame] | 195 | return module |
| 196 | } |
| 197 | |
| 198 | func customModuleFactory() android.Module { |
| 199 | m := customModuleFactoryBase() |
Liz Kammer | 4562a3b | 2021-04-21 18:15:34 -0400 | [diff] [blame] | 200 | android.InitAndroidArchModule(m, android.HostAndDeviceSupported, android.MultilibBoth) |
Liz Kammer | 2dd9ca4 | 2020-11-25 16:06:39 -0800 | [diff] [blame] | 201 | return m |
| 202 | } |
| 203 | |
| 204 | type testProps struct { |
| 205 | Test_prop struct { |
| 206 | Test_string_prop string |
| 207 | } |
| 208 | } |
| 209 | |
| 210 | type customTestModule struct { |
| 211 | android.ModuleBase |
| 212 | |
| 213 | props customProps |
| 214 | test_props testProps |
| 215 | } |
| 216 | |
| 217 | func (m *customTestModule) GenerateAndroidBuildActions(ctx android.ModuleContext) { |
| 218 | // nothing for now. |
| 219 | } |
| 220 | |
| 221 | func customTestModuleFactoryBase() android.Module { |
| 222 | m := &customTestModule{} |
| 223 | m.AddProperties(&m.props) |
| 224 | m.AddProperties(&m.test_props) |
| 225 | return m |
| 226 | } |
| 227 | |
| 228 | func customTestModuleFactory() android.Module { |
| 229 | m := customTestModuleFactoryBase() |
| 230 | android.InitAndroidModule(m) |
| 231 | return m |
| 232 | } |
| 233 | |
| 234 | type customDefaultsModule struct { |
| 235 | android.ModuleBase |
| 236 | android.DefaultsModuleBase |
| 237 | } |
| 238 | |
| 239 | func customDefaultsModuleFactoryBase() android.DefaultsModule { |
| 240 | module := &customDefaultsModule{} |
| 241 | module.AddProperties(&customProps{}) |
| 242 | return module |
| 243 | } |
| 244 | |
| 245 | func customDefaultsModuleFactoryBasic() android.Module { |
| 246 | return customDefaultsModuleFactoryBase() |
| 247 | } |
| 248 | |
| 249 | func customDefaultsModuleFactory() android.Module { |
| 250 | m := customDefaultsModuleFactoryBase() |
| 251 | android.InitDefaultsModule(m) |
| 252 | return m |
| 253 | } |
Jingwen Chen | 7385067 | 2020-12-14 08:25:34 -0500 | [diff] [blame] | 254 | |
| 255 | type customBazelModuleAttributes struct { |
Jingwen Chen | 7385067 | 2020-12-14 08:25:34 -0500 | [diff] [blame] | 256 | String_prop string |
| 257 | String_list_prop []string |
Liz Kammer | 4562a3b | 2021-04-21 18:15:34 -0400 | [diff] [blame] | 258 | Arch_paths bazel.LabelListAttribute |
Jingwen Chen | 7385067 | 2020-12-14 08:25:34 -0500 | [diff] [blame] | 259 | } |
| 260 | |
Jingwen Chen | 7385067 | 2020-12-14 08:25:34 -0500 | [diff] [blame] | 261 | func customBp2BuildMutator(ctx android.TopDownMutatorContext) { |
| 262 | if m, ok := ctx.Module().(*customModule); ok { |
Jingwen Chen | 12b4c27 | 2021-03-10 02:05:59 -0500 | [diff] [blame] | 263 | if !m.ConvertWithBp2build(ctx) { |
Jingwen Chen | 77e8b7b | 2021-02-05 03:03:24 -0500 | [diff] [blame] | 264 | return |
| 265 | } |
| 266 | |
Liz Kammer | 32b77cf | 2021-08-04 15:17:02 -0400 | [diff] [blame] | 267 | paths := bazel.MakeLabelListAttribute(android.BazelLabelForModuleSrcExcludes(ctx, m.props.Arch_paths, m.props.Arch_paths_exclude)) |
Liz Kammer | 4562a3b | 2021-04-21 18:15:34 -0400 | [diff] [blame] | 268 | |
Liz Kammer | 9abd62d | 2021-05-21 08:37:59 -0400 | [diff] [blame] | 269 | for axis, configToProps := range m.GetArchVariantProperties(ctx, &customProps{}) { |
| 270 | for config, props := range configToProps { |
| 271 | if archProps, ok := props.(*customProps); ok && archProps.Arch_paths != nil { |
Liz Kammer | 32b77cf | 2021-08-04 15:17:02 -0400 | [diff] [blame] | 272 | paths.SetSelectValue(axis, config, android.BazelLabelForModuleSrcExcludes(ctx, archProps.Arch_paths, archProps.Arch_paths_exclude)) |
Liz Kammer | 9abd62d | 2021-05-21 08:37:59 -0400 | [diff] [blame] | 273 | } |
Liz Kammer | 4562a3b | 2021-04-21 18:15:34 -0400 | [diff] [blame] | 274 | } |
| 275 | } |
| 276 | |
Liz Kammer | 32b77cf | 2021-08-04 15:17:02 -0400 | [diff] [blame] | 277 | paths.ResolveExcludes() |
| 278 | |
Jingwen Chen | 1fd1469 | 2021-02-05 03:01:50 -0500 | [diff] [blame] | 279 | attrs := &customBazelModuleAttributes{ |
Jingwen Chen | 7385067 | 2020-12-14 08:25:34 -0500 | [diff] [blame] | 280 | String_prop: m.props.String_prop, |
| 281 | String_list_prop: m.props.String_list_prop, |
Liz Kammer | 4562a3b | 2021-04-21 18:15:34 -0400 | [diff] [blame] | 282 | Arch_paths: paths, |
Jingwen Chen | 1fd1469 | 2021-02-05 03:01:50 -0500 | [diff] [blame] | 283 | } |
| 284 | |
Liz Kammer | fc46bc1 | 2021-02-19 11:06:17 -0500 | [diff] [blame] | 285 | props := bazel.BazelTargetModuleProperties{ |
| 286 | Rule_class: "custom", |
| 287 | } |
Jingwen Chen | 1fd1469 | 2021-02-05 03:01:50 -0500 | [diff] [blame] | 288 | |
Liz Kammer | 2ada09a | 2021-08-11 00:17:36 -0400 | [diff] [blame] | 289 | ctx.CreateBazelTargetModule(m.Name(), props, attrs) |
Jingwen Chen | 7385067 | 2020-12-14 08:25:34 -0500 | [diff] [blame] | 290 | } |
| 291 | } |
Jingwen Chen | 40067de | 2021-01-26 21:58:43 -0500 | [diff] [blame] | 292 | |
| 293 | // A bp2build mutator that uses load statements and creates a 1:M mapping from |
| 294 | // module to target. |
| 295 | func customBp2BuildMutatorFromStarlark(ctx android.TopDownMutatorContext) { |
| 296 | if m, ok := ctx.Module().(*customModule); ok { |
Jingwen Chen | 12b4c27 | 2021-03-10 02:05:59 -0500 | [diff] [blame] | 297 | if !m.ConvertWithBp2build(ctx) { |
Jingwen Chen | 77e8b7b | 2021-02-05 03:03:24 -0500 | [diff] [blame] | 298 | return |
| 299 | } |
| 300 | |
Jingwen Chen | 1fd1469 | 2021-02-05 03:01:50 -0500 | [diff] [blame] | 301 | baseName := m.Name() |
| 302 | attrs := &customBazelModuleAttributes{} |
| 303 | |
Liz Kammer | fc46bc1 | 2021-02-19 11:06:17 -0500 | [diff] [blame] | 304 | myLibraryProps := bazel.BazelTargetModuleProperties{ |
| 305 | Rule_class: "my_library", |
| 306 | Bzl_load_location: "//build/bazel/rules:rules.bzl", |
| 307 | } |
Liz Kammer | 2ada09a | 2021-08-11 00:17:36 -0400 | [diff] [blame] | 308 | ctx.CreateBazelTargetModule(baseName, myLibraryProps, attrs) |
Jingwen Chen | 1fd1469 | 2021-02-05 03:01:50 -0500 | [diff] [blame] | 309 | |
Liz Kammer | fc46bc1 | 2021-02-19 11:06:17 -0500 | [diff] [blame] | 310 | protoLibraryProps := bazel.BazelTargetModuleProperties{ |
| 311 | Rule_class: "proto_library", |
| 312 | Bzl_load_location: "//build/bazel/rules:proto.bzl", |
| 313 | } |
Liz Kammer | 2ada09a | 2021-08-11 00:17:36 -0400 | [diff] [blame] | 314 | ctx.CreateBazelTargetModule(baseName+"_proto_library_deps", protoLibraryProps, attrs) |
Jingwen Chen | 1fd1469 | 2021-02-05 03:01:50 -0500 | [diff] [blame] | 315 | |
Liz Kammer | fc46bc1 | 2021-02-19 11:06:17 -0500 | [diff] [blame] | 316 | myProtoLibraryProps := bazel.BazelTargetModuleProperties{ |
| 317 | Rule_class: "my_proto_library", |
| 318 | Bzl_load_location: "//build/bazel/rules:proto.bzl", |
| 319 | } |
Liz Kammer | 2ada09a | 2021-08-11 00:17:36 -0400 | [diff] [blame] | 320 | ctx.CreateBazelTargetModule(baseName+"_my_proto_library_deps", myProtoLibraryProps, attrs) |
Jingwen Chen | 40067de | 2021-01-26 21:58:43 -0500 | [diff] [blame] | 321 | } |
| 322 | } |
Jingwen Chen | ba369ad | 2021-02-22 10:19:34 -0500 | [diff] [blame] | 323 | |
| 324 | // Helper method for tests to easily access the targets in a dir. |
Liz Kammer | 6eff323 | 2021-08-26 08:37:59 -0400 | [diff] [blame^] | 325 | func generateBazelTargetsForDir(codegenCtx *CodegenContext, dir string) (BazelTargets, []error) { |
Rupert Shuttleworth | 2a4fc3e | 2021-04-21 07:10:09 -0400 | [diff] [blame] | 326 | // TODO: Set generateFilegroups to true and/or remove the generateFilegroups argument completely |
Liz Kammer | 6eff323 | 2021-08-26 08:37:59 -0400 | [diff] [blame^] | 327 | res, err := GenerateBazelTargets(codegenCtx, false) |
| 328 | return res.buildFileToTargets[dir], err |
Jingwen Chen | ba369ad | 2021-02-22 10:19:34 -0500 | [diff] [blame] | 329 | } |
Liz Kammer | 32b77cf | 2021-08-04 15:17:02 -0400 | [diff] [blame] | 330 | |
| 331 | func registerCustomModuleForBp2buildConversion(ctx *android.TestContext) { |
| 332 | ctx.RegisterModuleType("custom", customModuleFactory) |
| 333 | ctx.RegisterBp2BuildMutator("custom", customBp2BuildMutator) |
| 334 | ctx.RegisterForBazelConversion() |
| 335 | } |