diff options
author | Chih-Hung Hsieh <chh@google.com> | 2019-10-31 20:56:47 -0700 |
---|---|---|
committer | Chih-Hung Hsieh <chh@google.com> | 2019-11-14 15:16:26 -0800 |
commit | 41805bedbfb53ae03ac6d017402bb082ae8ea050 (patch) | |
tree | c834f9d114476292c2ebb1c45349e197516565a0 /tradefed | |
parent | ed0070d45379e8e1e0b156c774885f62518d91bf (diff) | |
download | build_soong-41805bedbfb53ae03ac6d017402bb082ae8ea050.tar.gz build_soong-41805bedbfb53ae03ac6d017402bb082ae8ea050.tar.bz2 build_soong-41805bedbfb53ae03ac6d017402bb082ae8ea050.zip |
Add TestProperties, gen test config, fix names
* Rename testBinaryDecorator to testDecorator
* Add TestProperties
* Add install function for testDecorator to install config files
* Add tradefed.AutoGenRustHostTestConfig
* Depend on new build/make/core/rust_host_test_config_template.xml
and new tradefed.testtype.rust.RustBinaryHostTest class
* Add autogenTemplateWithName in tradefed/autogen.go
to generate config files with customized(mutated) executable name.
* Make rust_test module names more robust and easy to use.
* Use crate name instead of source file name as the Stem
for single source file modules, to match original user
specified output file name in Cargo.toml.
* Do not set up test module SubName when Stem is empty
or when the module name already contains Stem suffix.
That happens when TestPerSrcMutator is disabled or when
there is only one source file with renamed output file name.
* In TEST_MAPPING, references to mutated rust_test modules should be
(1) <module_name> for single source file modules without mutation, or
(2) <module_name>_<crate_name> for single source file modules, or
(3) <module_name>_<source_file_base_name> for multi-file modules.
Bug: 140938178
Test: mm in rust projects, check output test file names
Change-Id: Ifdbfa14d5eed4f10b4fb983f82c93bbb9be3f899
Diffstat (limited to 'tradefed')
-rw-r--r-- | tradefed/autogen.go | 21 | ||||
-rw-r--r-- | tradefed/config.go | 1 | ||||
-rw-r--r-- | tradefed/makevars.go | 1 |
3 files changed, 22 insertions, 1 deletions
diff --git a/tradefed/autogen.go b/tradefed/autogen.go index 3d30cfa1..905acfac 100644 --- a/tradefed/autogen.go +++ b/tradefed/autogen.go @@ -105,6 +105,10 @@ func (ob Object) Config() string { } func autogenTemplate(ctx android.ModuleContext, output android.WritablePath, template string, configs []Config) { + autogenTemplateWithName(ctx, ctx.ModuleName(), output, template, configs) +} + +func autogenTemplateWithName(ctx android.ModuleContext, name string, output android.WritablePath, template string, configs []Config) { var configStrings []string for _, config := range configs { configStrings = append(configStrings, config.Config()) @@ -117,7 +121,7 @@ func autogenTemplate(ctx android.ModuleContext, output android.WritablePath, tem Description: "test config", Output: output, Args: map[string]string{ - "name": ctx.ModuleName(), + "name": name, "template": template, "extraConfigs": extraConfigs, }, @@ -193,6 +197,21 @@ func AutoGenPythonBinaryHostTestConfig(ctx android.ModuleContext, testConfigProp return path } +func AutoGenRustHostTestConfig(ctx android.ModuleContext, name string, testConfigProp *string, + testConfigTemplateProp *string, testSuites []string, autoGenConfig *bool) android.Path { + path, autogenPath := testConfigPath(ctx, testConfigProp, testSuites, autoGenConfig) + if autogenPath != nil { + templatePathString := "${RustHostTestConfigTemplate}" + templatePath := getTestConfigTemplate(ctx, testConfigTemplateProp) + if templatePath.Valid() { + templatePathString = templatePath.String() + } + autogenTemplateWithName(ctx, name, autogenPath, templatePathString, nil) + return autogenPath + } + return path +} + var autogenInstrumentationTest = pctx.StaticRule("autogenInstrumentationTest", blueprint.RuleParams{ Command: "${AutoGenTestConfigScript} $out $in ${EmptyTestConfig} $template", CommandDeps: []string{ diff --git a/tradefed/config.go b/tradefed/config.go index 141e0c57..8249ffe8 100644 --- a/tradefed/config.go +++ b/tradefed/config.go @@ -31,6 +31,7 @@ func init() { pctx.SourcePathVariable("NativeHostTestConfigTemplate", "build/make/core/native_host_test_config_template.xml") pctx.SourcePathVariable("NativeTestConfigTemplate", "build/make/core/native_test_config_template.xml") pctx.SourcePathVariable("PythonBinaryHostTestConfigTemplate", "build/make/core/python_binary_host_test_config_template.xml") + pctx.SourcePathVariable("RustHostTestConfigTemplate", "build/make/core/rust_host_test_config_template.xml") pctx.SourcePathVariable("EmptyTestConfig", "build/make/core/empty_test_config.xml") } diff --git a/tradefed/makevars.go b/tradefed/makevars.go index aad7273e..e6b88ea1 100644 --- a/tradefed/makevars.go +++ b/tradefed/makevars.go @@ -31,6 +31,7 @@ func makeVarsProvider(ctx android.MakeVarsContext) { ctx.Strict("NATIVE_HOST_TEST_CONFIG_TEMPLATE", "${NativeHostTestConfigTemplate}") ctx.Strict("NATIVE_TEST_CONFIG_TEMPLATE", "${NativeTestConfigTemplate}") ctx.Strict("PYTHON_BINARY_HOST_TEST_CONFIG_TEMPLATE", "${PythonBinaryHostTestConfigTemplate}") + ctx.Strict("RUST_HOST_TEST_CONFIG_TEMPLATE", "${RustHostTestConfigTemplate}") ctx.Strict("EMPTY_TEST_CONFIG", "${EmptyTestConfig}") } |