summaryrefslogtreecommitdiff
path: root/android/androidmk_test.go
diff options
context:
space:
mode:
author Paul Duffin <paulduffin@google.com> 2020-11-26 17:36:46 +0000
committer Paul Duffin <paulduffin@google.com> 2020-11-27 01:24:02 +0000
commit103aaae715243b0f1675690657dd22d75c294057 (patch)
treedac45aa5ce26c3fc4ce07bf7db402deefd1bf937 /android/androidmk_test.go
parent9c4c534d25e158f743a01f0a34db2e0c92d2e6bb (diff)
Extract method to process bp file for androidmk_test.go
Test: m nothing Bug: 174226317 Change-Id: Ifc9ee5088fe766d971718fc112d1e95719a2b825
Diffstat (limited to 'android/androidmk_test.go')
-rw-r--r--android/androidmk_test.go45
1 files changed, 20 insertions, 25 deletions
diff --git a/android/androidmk_test.go b/android/androidmk_test.go
index 80df6a7cb..bcaf5d7fe 100644
--- a/android/androidmk_test.go
+++ b/android/androidmk_test.go
@@ -67,16 +67,10 @@ func customModuleFactory() Module {
return module
}
-func TestAndroidMkSingleton_PassesUpdatedAndroidMkDataToCustomCallback(t *testing.T) {
- bp := `
- custom {
- name: "foo",
- required: ["bar"],
- host_required: ["baz"],
- target_required: ["qux"],
- }
- `
-
+// buildConfigAndCustomModuleFoo creates a config object, processes the supplied
+// bp module and then returns the config and the custom module called "foo".
+func buildConfigAndCustomModuleFoo(t *testing.T, bp string) (Config, *customModule) {
+ t.Helper()
config := TestConfig(buildDir, nil, bp, nil)
config.katiEnabled = true // Enable androidmk Singleton
@@ -90,7 +84,21 @@ func TestAndroidMkSingleton_PassesUpdatedAndroidMkDataToCustomCallback(t *testin
_, errs = ctx.PrepareBuildActions(config)
FailIfErrored(t, errs)
- m := ctx.ModuleForTests("foo", "").Module().(*customModule)
+ module := ctx.ModuleForTests("foo", "").Module().(*customModule)
+ return config, module
+}
+
+func TestAndroidMkSingleton_PassesUpdatedAndroidMkDataToCustomCallback(t *testing.T) {
+ bp := `
+ custom {
+ name: "foo",
+ required: ["bar"],
+ host_required: ["baz"],
+ target_required: ["qux"],
+ }
+ `
+
+ _, m := buildConfigAndCustomModuleFoo(t, bp)
assertEqual := func(expected interface{}, actual interface{}) {
if !reflect.DeepEqual(expected, actual) {
@@ -257,20 +265,7 @@ func TestGetDistForGoals(t *testing.T) {
for _, testCase := range testCases {
t.Run(testCase.name, func(t *testing.T) {
- config := TestConfig(buildDir, nil, testCase.bp, nil)
- config.katiEnabled = true // Enable androidmk Singleton
-
- ctx := NewTestContext(config)
- ctx.RegisterSingletonType("androidmk", AndroidMkSingleton)
- ctx.RegisterModuleType("custom", customModuleFactory)
- ctx.Register()
-
- _, errs := ctx.ParseFileList(".", []string{"Android.bp"})
- FailIfErrored(t, errs)
- _, errs = ctx.PrepareBuildActions(config)
- FailIfErrored(t, errs)
-
- module := ctx.ModuleForTests("foo", "").Module().(*customModule)
+ config, module := buildConfigAndCustomModuleFoo(t, testCase.bp)
entries := AndroidMkEntriesForTest(t, config, "", module)
if len(entries) != 1 {
t.Errorf("Expected a single AndroidMk entry, got %d", len(entries))