Ronald Braunstein | fce4316 | 2024-02-02 12:37:20 -0800 | [diff] [blame] | 1 | package tradefed_modules |
| 2 | |
| 3 | import ( |
| 4 | "android/soong/android" |
| 5 | "android/soong/tradefed" |
| 6 | "encoding/json" |
| 7 | "fmt" |
| 8 | |
| 9 | "github.com/google/blueprint" |
| 10 | "github.com/google/blueprint/proptools" |
| 11 | ) |
| 12 | |
| 13 | func init() { |
| 14 | RegisterTestModuleConfigBuildComponents(android.InitRegistrationContext) |
| 15 | } |
| 16 | |
| 17 | // Register the license_kind module type. |
| 18 | func RegisterTestModuleConfigBuildComponents(ctx android.RegistrationContext) { |
| 19 | ctx.RegisterModuleType("test_module_config", TestModuleConfigFactory) |
Ronald Braunstein | 1a6e7c0 | 2024-03-14 21:14:39 +0000 | [diff] [blame] | 20 | ctx.RegisterModuleType("test_module_config_host", TestModuleConfigHostFactory) |
Ronald Braunstein | fce4316 | 2024-02-02 12:37:20 -0800 | [diff] [blame] | 21 | } |
| 22 | |
| 23 | type testModuleConfigModule struct { |
| 24 | android.ModuleBase |
| 25 | android.DefaultableModuleBase |
| 26 | base android.Module |
| 27 | |
| 28 | tradefedProperties |
| 29 | |
| 30 | // Our updated testConfig. |
| 31 | testConfig android.OutputPath |
| 32 | manifest android.InstallPath |
| 33 | provider tradefed.BaseTestProviderData |
| 34 | } |
| 35 | |
Ronald Braunstein | 1a6e7c0 | 2024-03-14 21:14:39 +0000 | [diff] [blame] | 36 | // Host is mostly the same as non-host, just some diffs for AddDependency and |
| 37 | // AndroidMkEntries, but the properties are the same. |
| 38 | type testModuleConfigHostModule struct { |
| 39 | testModuleConfigModule |
| 40 | } |
| 41 | |
Ronald Braunstein | fce4316 | 2024-02-02 12:37:20 -0800 | [diff] [blame] | 42 | // Properties to list in Android.bp for this module. |
| 43 | type tradefedProperties struct { |
| 44 | // Module name of the base test that we will run. |
| 45 | Base *string `android:"path,arch_variant"` |
| 46 | |
| 47 | // Tradefed Options to add to tradefed xml when not one of the include or exclude filter or property. |
| 48 | // Sample: [{name: "TestRunnerOptionName", value: "OptionValue" }] |
| 49 | Options []tradefed.Option |
| 50 | |
| 51 | // List of tradefed include annotations to add to tradefed xml, like "android.platform.test.annotations.Presubmit". |
| 52 | // Tests will be restricted to those matching an include_annotation or include_filter. |
| 53 | Include_annotations []string |
| 54 | |
| 55 | // List of tradefed include annotations to add to tradefed xml, like "android.support.test.filters.FlakyTest". |
| 56 | // Tests matching an exclude annotation or filter will be skipped. |
| 57 | Exclude_annotations []string |
| 58 | |
| 59 | // List of tradefed include filters to add to tradefed xml, like "fully.qualified.class#method". |
| 60 | // Tests will be restricted to those matching an include_annotation or include_filter. |
| 61 | Include_filters []string |
| 62 | |
| 63 | // List of tradefed exclude filters to add to tradefed xml, like "fully.qualified.class#method". |
| 64 | // Tests matching an exclude annotation or filter will be skipped. |
| 65 | Exclude_filters []string |
| 66 | |
| 67 | // List of compatibility suites (for example "cts", "vts") that the module should be |
| 68 | // installed into. |
| 69 | Test_suites []string |
| 70 | } |
| 71 | |
| 72 | type dependencyTag struct { |
| 73 | blueprint.BaseDependencyTag |
| 74 | name string |
| 75 | } |
| 76 | |
| 77 | var ( |
Ronald Braunstein | 1a6e7c0 | 2024-03-14 21:14:39 +0000 | [diff] [blame] | 78 | testModuleConfigTag = dependencyTag{name: "TestModuleConfigBase"} |
| 79 | testModuleConfigHostTag = dependencyTag{name: "TestModuleConfigHostBase"} |
| 80 | pctx = android.NewPackageContext("android/soong/tradefed_modules") |
Ronald Braunstein | fce4316 | 2024-02-02 12:37:20 -0800 | [diff] [blame] | 81 | ) |
| 82 | |
| 83 | func (m *testModuleConfigModule) InstallInTestcases() bool { |
| 84 | return true |
| 85 | } |
| 86 | |
| 87 | func (m *testModuleConfigModule) DepsMutator(ctx android.BottomUpMutatorContext) { |
Ronald Braunstein | 1a6e7c0 | 2024-03-14 21:14:39 +0000 | [diff] [blame] | 88 | if m.Base == nil { |
| 89 | ctx.ModuleErrorf("'base' field must be set to a 'android_test' module.") |
| 90 | return |
| 91 | } |
Ronald Braunstein | fce4316 | 2024-02-02 12:37:20 -0800 | [diff] [blame] | 92 | ctx.AddDependency(ctx.Module(), testModuleConfigTag, *m.Base) |
| 93 | } |
| 94 | |
| 95 | // Takes base's Tradefed Config xml file and generates a new one with the test properties |
| 96 | // appeneded from this module. |
| 97 | // Rewrite the name of the apk in "test-file-name" to be our module's name, rather than the original one. |
| 98 | func (m *testModuleConfigModule) fixTestConfig(ctx android.ModuleContext, baseTestConfig android.Path) android.OutputPath { |
| 99 | // Test safe to do when no test_runner_options, but check for that earlier? |
| 100 | fixedConfig := android.PathForModuleOut(ctx, "test_config_fixer", ctx.ModuleName()+".config") |
| 101 | rule := android.NewRuleBuilder(pctx, ctx) |
| 102 | command := rule.Command().BuiltTool("test_config_fixer").Input(baseTestConfig).Output(fixedConfig) |
| 103 | options := m.composeOptions() |
| 104 | if len(options) == 0 { |
| 105 | ctx.ModuleErrorf("Test options must be given when using test_module_config. Set include/exclude filter or annotation.") |
| 106 | } |
| 107 | xmlTestModuleConfigSnippet, _ := json.Marshal(options) |
| 108 | escaped := proptools.NinjaAndShellEscape(string(xmlTestModuleConfigSnippet)) |
| 109 | command.FlagWithArg("--test-file-name=", ctx.ModuleName()+".apk"). |
| 110 | FlagWithArg("--orig-test-file-name=", *m.tradefedProperties.Base+".apk"). |
| 111 | FlagWithArg("--test-runner-options=", escaped) |
| 112 | rule.Build("fix_test_config", "fix test config") |
| 113 | return fixedConfig.OutputPath |
| 114 | } |
| 115 | |
| 116 | // Convert --exclude_filters: ["filter1", "filter2"] -> |
| 117 | // [ Option{Name: "exclude-filters", Value: "filter1"}, Option{Name: "exclude-filters", Value: "filter2"}, |
| 118 | // ... + include + annotations ] |
| 119 | func (m *testModuleConfigModule) composeOptions() []tradefed.Option { |
| 120 | options := m.Options |
| 121 | for _, e := range m.Exclude_filters { |
| 122 | options = append(options, tradefed.Option{Name: "exclude-filter", Value: e}) |
| 123 | } |
| 124 | for _, i := range m.Include_filters { |
| 125 | options = append(options, tradefed.Option{Name: "include-filter", Value: i}) |
| 126 | } |
| 127 | for _, e := range m.Exclude_annotations { |
| 128 | options = append(options, tradefed.Option{Name: "exclude-annotation", Value: e}) |
| 129 | } |
| 130 | for _, i := range m.Include_annotations { |
| 131 | options = append(options, tradefed.Option{Name: "include-annotation", Value: i}) |
| 132 | } |
| 133 | return options |
| 134 | } |
| 135 | |
| 136 | // Files to write and where they come from: |
| 137 | // 1) test_module_config.manifest |
| 138 | // - Leave a trail of where we got files from in case other tools need it. |
| 139 | // |
| 140 | // 2) $Module.config |
| 141 | // - comes from base's module.config (AndroidTest.xml), and then we add our test_options. |
| 142 | // provider.TestConfig |
| 143 | // [rules via soong_app_prebuilt] |
| 144 | // |
| 145 | // 3) $ARCH/$Module.apk |
| 146 | // - comes from base |
| 147 | // provider.OutputFile |
| 148 | // [rules via soong_app_prebuilt] |
| 149 | // |
| 150 | // 4) [bases data] |
| 151 | // - We copy all of bases data (like helper apks) to our install directory too. |
| 152 | // Since we call AndroidMkEntries on base, it will write out LOCAL_COMPATIBILITY_SUPPORT_FILES |
| 153 | // with this data and app_prebuilt.mk will generate the rules to copy it from base. |
| 154 | // We have no direct rules here to add to ninja. |
| 155 | // |
| 156 | // If we change to symlinks, this all needs to change. |
| 157 | func (m *testModuleConfigModule) GenerateAndroidBuildActions(ctx android.ModuleContext) { |
Ronald Braunstein | 1a6e7c0 | 2024-03-14 21:14:39 +0000 | [diff] [blame] | 158 | m.validateBase(ctx, &testModuleConfigTag, "android_test", false) |
| 159 | m.generateManifestAndConfig(ctx) |
Ronald Braunstein | fce4316 | 2024-02-02 12:37:20 -0800 | [diff] [blame] | 160 | |
Ronald Braunstein | 1a6e7c0 | 2024-03-14 21:14:39 +0000 | [diff] [blame] | 161 | } |
| 162 | |
Ronald Braunstein | b4a4ef9 | 2024-04-30 14:02:59 +0000 | [diff] [blame] | 163 | // Ensure at least one test_suite is listed. Ideally it should be general-tests |
| 164 | // or device-tests, whichever is listed in base and prefer general-tests if both are listed. |
| 165 | // However this is not enforced yet. |
Ronald Braunstein | 1a6e7c0 | 2024-03-14 21:14:39 +0000 | [diff] [blame] | 166 | // |
Ronald Braunstein | b4a4ef9 | 2024-04-30 14:02:59 +0000 | [diff] [blame] | 167 | // Returns true if okay and reports errors via ModuleErrorf. |
Ronald Braunstein | 1a6e7c0 | 2024-03-14 21:14:39 +0000 | [diff] [blame] | 168 | func (m *testModuleConfigModule) validateTestSuites(ctx android.ModuleContext) bool { |
| 169 | if len(m.tradefedProperties.Test_suites) == 0 { |
Ronald Braunstein | b4a4ef9 | 2024-04-30 14:02:59 +0000 | [diff] [blame] | 170 | ctx.ModuleErrorf("At least one test-suite must be set or this won't run. Use \"general-tests\" or \"device-tests\"") |
Ronald Braunstein | 1a6e7c0 | 2024-03-14 21:14:39 +0000 | [diff] [blame] | 171 | return false |
| 172 | } |
| 173 | |
Ronald Braunstein | 1a6e7c0 | 2024-03-14 21:14:39 +0000 | [diff] [blame] | 174 | return true |
Ronald Braunstein | fce4316 | 2024-02-02 12:37:20 -0800 | [diff] [blame] | 175 | } |
| 176 | |
| 177 | func TestModuleConfigFactory() android.Module { |
| 178 | module := &testModuleConfigModule{} |
| 179 | |
| 180 | module.AddProperties(&module.tradefedProperties) |
| 181 | android.InitAndroidArchModule(module, android.DeviceSupported, android.MultilibCommon) |
| 182 | android.InitDefaultableModule(module) |
| 183 | |
| 184 | return module |
| 185 | } |
| 186 | |
Ronald Braunstein | 1a6e7c0 | 2024-03-14 21:14:39 +0000 | [diff] [blame] | 187 | func TestModuleConfigHostFactory() android.Module { |
| 188 | module := &testModuleConfigHostModule{} |
| 189 | |
| 190 | module.AddProperties(&module.tradefedProperties) |
| 191 | android.InitAndroidMultiTargetsArchModule(module, android.HostSupported, android.MultilibCommon) |
| 192 | android.InitDefaultableModule(module) |
| 193 | |
| 194 | return module |
| 195 | } |
| 196 | |
Ronald Braunstein | fce4316 | 2024-02-02 12:37:20 -0800 | [diff] [blame] | 197 | // Implements android.AndroidMkEntriesProvider |
| 198 | var _ android.AndroidMkEntriesProvider = (*testModuleConfigModule)(nil) |
| 199 | |
| 200 | func (m *testModuleConfigModule) AndroidMkEntries() []android.AndroidMkEntries { |
| 201 | // We rely on base writing LOCAL_COMPATIBILITY_SUPPORT_FILES for its data files |
| 202 | entriesList := m.base.(android.AndroidMkEntriesProvider).AndroidMkEntries() |
| 203 | entries := &entriesList[0] |
| 204 | entries.OutputFile = android.OptionalPathForPath(m.provider.OutputFile) |
| 205 | entries.ExtraEntries = append(entries.ExtraEntries, func(ctx android.AndroidMkExtraEntriesContext, entries *android.AndroidMkEntries) { |
| 206 | entries.SetString("LOCAL_MODULE", m.Name()) // out module name, not base's |
| 207 | |
| 208 | // Out update config file with extra options. |
| 209 | entries.SetPath("LOCAL_FULL_TEST_CONFIG", m.testConfig) |
| 210 | entries.SetString("LOCAL_MODULE_TAGS", "tests") |
Ronald Braunstein | fce4316 | 2024-02-02 12:37:20 -0800 | [diff] [blame] | 211 | |
| 212 | // Don't append to base's test-suites, only use the ones we define, so clear it before |
| 213 | // appending to it. |
| 214 | entries.SetString("LOCAL_COMPATIBILITY_SUITE", "") |
Ronald Braunstein | 1a6e7c0 | 2024-03-14 21:14:39 +0000 | [diff] [blame] | 215 | entries.AddCompatibilityTestSuites(m.tradefedProperties.Test_suites...) |
| 216 | |
| 217 | if len(m.provider.HostRequiredModuleNames) > 0 { |
| 218 | entries.AddStrings("LOCAL_HOST_REQUIRED_MODULES", m.provider.HostRequiredModuleNames...) |
| 219 | } |
| 220 | if len(m.provider.RequiredModuleNames) > 0 { |
| 221 | entries.AddStrings("LOCAL_REQUIRED_MODULES", m.provider.RequiredModuleNames...) |
| 222 | } |
| 223 | |
| 224 | if m.provider.IsHost == false { |
| 225 | // Not needed for jar_host_test |
| 226 | // |
| 227 | // Clear the JNI symbols because they belong to base not us. Either transform the names in the string |
| 228 | // or clear the variable because we don't need it, we are copying bases libraries not generating |
| 229 | // new ones. |
| 230 | entries.SetString("LOCAL_SOONG_JNI_LIBS_SYMBOLS", "") |
Ronald Braunstein | fce4316 | 2024-02-02 12:37:20 -0800 | [diff] [blame] | 231 | } |
| 232 | }) |
| 233 | return entriesList |
| 234 | } |
Ronald Braunstein | 1a6e7c0 | 2024-03-14 21:14:39 +0000 | [diff] [blame] | 235 | |
| 236 | func (m *testModuleConfigHostModule) DepsMutator(ctx android.BottomUpMutatorContext) { |
| 237 | if m.Base == nil { |
| 238 | ctx.ModuleErrorf("'base' field must be set to a 'java_test_host' module") |
| 239 | return |
| 240 | } |
| 241 | ctx.AddVariationDependencies(ctx.Config().BuildOSCommonTarget.Variations(), testModuleConfigHostTag, *m.Base) |
| 242 | } |
| 243 | |
| 244 | // File to write: |
| 245 | // 1) out/host/linux-x86/testcases/derived-module/test_module_config.manifest # contains base's name. |
| 246 | // 2) out/host/linux-x86/testcases/derived-module/derived-module.config # Update AnroidTest.xml |
| 247 | // 3) out/host/linux-x86/testcases/derived-module/base.jar |
| 248 | // - written via soong_java_prebuilt.mk |
| 249 | // |
| 250 | // 4) out/host/linux-x86/testcases/derived-module/* # data dependencies from base. |
| 251 | // - written via soong_java_prebuilt.mk |
| 252 | func (m *testModuleConfigHostModule) GenerateAndroidBuildActions(ctx android.ModuleContext) { |
| 253 | m.validateBase(ctx, &testModuleConfigHostTag, "java_test_host", true) |
| 254 | m.generateManifestAndConfig(ctx) |
| 255 | } |
| 256 | |
| 257 | // Ensure the base listed is the right type by checking that we get the expected provider data. |
| 258 | // Returns false on errors and the context is updated with an error indicating the baseType expected. |
| 259 | func (m *testModuleConfigModule) validateBase(ctx android.ModuleContext, depTag *dependencyTag, baseType string, baseShouldBeHost bool) { |
| 260 | ctx.VisitDirectDepsWithTag(*depTag, func(dep android.Module) { |
| 261 | if provider, ok := android.OtherModuleProvider(ctx, dep, tradefed.BaseTestProviderKey); ok { |
| 262 | if baseShouldBeHost == provider.IsHost { |
| 263 | m.base = dep |
| 264 | m.provider = provider |
| 265 | } else { |
| 266 | if baseShouldBeHost { |
| 267 | ctx.ModuleErrorf("'android_test' module used as base, but 'java_test_host' expected.") |
| 268 | } else { |
| 269 | ctx.ModuleErrorf("'java_test_host' module used as base, but 'android_test' expected.") |
| 270 | } |
| 271 | } |
| 272 | } else { |
| 273 | ctx.ModuleErrorf("'%s' module used as base but it is not a '%s' module.", *m.Base, baseType) |
| 274 | } |
| 275 | }) |
| 276 | } |
| 277 | |
| 278 | // Actions to write: |
| 279 | // 1. manifest file to testcases dir |
| 280 | // 2. New Module.config / AndroidTest.xml file with our options. |
| 281 | func (m *testModuleConfigModule) generateManifestAndConfig(ctx android.ModuleContext) { |
Ronald Braunstein | d245346 | 2024-04-18 09:18:29 -0700 | [diff] [blame] | 282 | // Keep before early returns. |
| 283 | android.SetProvider(ctx, android.TestOnlyProviderKey, android.TestModuleInformation{ |
| 284 | TestOnly: true, |
| 285 | TopLevelTarget: true, |
| 286 | }) |
| 287 | |
Ronald Braunstein | 1a6e7c0 | 2024-03-14 21:14:39 +0000 | [diff] [blame] | 288 | if !m.validateTestSuites(ctx) { |
| 289 | return |
| 290 | } |
Ronald Braunstein | d245346 | 2024-04-18 09:18:29 -0700 | [diff] [blame] | 291 | // Ensure the base provider is accurate |
Ronald Braunstein | 1a6e7c0 | 2024-03-14 21:14:39 +0000 | [diff] [blame] | 292 | if m.provider.TestConfig == nil { |
| 293 | return |
| 294 | } |
| 295 | |
| 296 | // 1) A manifest file listing the base, write text to a tiny file. |
| 297 | installDir := android.PathForModuleInstall(ctx, ctx.ModuleName()) |
| 298 | manifest := android.PathForModuleOut(ctx, "test_module_config.manifest") |
| 299 | android.WriteFileRule(ctx, manifest, fmt.Sprintf("{%q: %q}", "base", *m.tradefedProperties.Base)) |
| 300 | // build/soong/android/androidmk.go has this comment: |
| 301 | // Assume the primary install file is last |
| 302 | // so we need to Install our file last. |
| 303 | ctx.InstallFile(installDir, manifest.Base(), manifest) |
| 304 | |
| 305 | // 2) Module.config / AndroidTest.xml |
| 306 | m.testConfig = m.fixTestConfig(ctx, m.provider.TestConfig) |
| 307 | } |
| 308 | |
| 309 | var _ android.AndroidMkEntriesProvider = (*testModuleConfigHostModule)(nil) |