diff options
author | 2021-03-16 19:31:17 +0000 | |
---|---|---|
committer | 2021-03-17 16:18:17 +0000 | |
commit | 7c166b452958c6c4d8d2f14f459bf1affd455bf4 (patch) | |
tree | 8b1865494b4bc0cfff5bdb7be887ca141afde8cb | |
parent | 3d11961d43073f7ee5a2aac60cab20aa3ffbe9a2 (diff) |
Convert android/defaults_test.go to test fixtures
Bug: 182885307
Test: m nothing
Change-Id: Iaf74bb9bece4eed717efa813b9b8ba51ecce5135
-rw-r--r-- | android/defaults_test.go | 70 |
1 files changed, 23 insertions, 47 deletions
diff --git a/android/defaults_test.go b/android/defaults_test.go index 2689d864e..b33cb527c 100644 --- a/android/defaults_test.go +++ b/android/defaults_test.go @@ -15,10 +15,7 @@ package android import ( - "reflect" "testing" - - "github.com/google/blueprint/proptools" ) type defaultsTestProperties struct { @@ -58,6 +55,14 @@ func defaultsTestDefaultsFactory() Module { return defaults } +var prepareForDefaultsTest = GroupFixturePreparers( + PrepareForTestWithDefaults, + FixtureRegisterWithContext(func(ctx RegistrationContext) { + ctx.RegisterModuleType("test", defaultsTestModuleFactory) + ctx.RegisterModuleType("defaults", defaultsTestDefaultsFactory) + }), +) + func TestDefaults(t *testing.T) { bp := ` defaults { @@ -78,27 +83,14 @@ func TestDefaults(t *testing.T) { } ` - config := TestConfig(buildDir, nil, bp, nil) - - ctx := NewTestContext(config) - - ctx.RegisterModuleType("test", defaultsTestModuleFactory) - ctx.RegisterModuleType("defaults", defaultsTestDefaultsFactory) - - ctx.PreArchMutators(RegisterDefaultsPreArchMutators) - - ctx.Register() + result := emptyTestFixtureFactory.RunTest(t, + prepareForDefaultsTest, + FixtureWithRootAndroidBp(bp), + ) - _, errs := ctx.ParseFileList(".", []string{"Android.bp"}) - FailIfErrored(t, errs) - _, errs = ctx.PrepareBuildActions(config) - FailIfErrored(t, errs) + foo := result.Module("foo", "").(*defaultsTestModule) - foo := ctx.ModuleForTests("foo", "").Module().(*defaultsTestModule) - - if g, w := foo.properties.Foo, []string{"transitive", "defaults", "module"}; !reflect.DeepEqual(g, w) { - t.Errorf("expected foo %q, got %q", w, g) - } + AssertDeepEquals(t, "foo", []string{"transitive", "defaults", "module"}, foo.properties.Foo) } func TestDefaultsAllowMissingDependencies(t *testing.T) { @@ -122,34 +114,18 @@ func TestDefaultsAllowMissingDependencies(t *testing.T) { } ` - config := TestConfig(buildDir, nil, bp, nil) - config.TestProductVariables.Allow_missing_dependencies = proptools.BoolPtr(true) - - ctx := NewTestContext(config) - ctx.SetAllowMissingDependencies(true) - - ctx.RegisterModuleType("test", defaultsTestModuleFactory) - ctx.RegisterModuleType("defaults", defaultsTestDefaultsFactory) - - ctx.PreArchMutators(RegisterDefaultsPreArchMutators) - - ctx.Register() - - _, errs := ctx.ParseFileList(".", []string{"Android.bp"}) - FailIfErrored(t, errs) - _, errs = ctx.PrepareBuildActions(config) - FailIfErrored(t, errs) + result := emptyTestFixtureFactory.RunTest(t, + prepareForDefaultsTest, + PrepareForTestWithAllowMissingDependencies, + FixtureWithRootAndroidBp(bp), + ) - missingDefaults := ctx.ModuleForTests("missing_defaults", "").Output("out") - missingTransitiveDefaults := ctx.ModuleForTests("missing_transitive_defaults", "").Output("out") + missingDefaults := result.ModuleForTests("missing_defaults", "").Output("out") + missingTransitiveDefaults := result.ModuleForTests("missing_transitive_defaults", "").Output("out") - if missingDefaults.Rule != ErrorRule { - t.Errorf("expected missing_defaults rule to be ErrorRule, got %#v", missingDefaults.Rule) - } + AssertSame(t, "missing_defaults rule", ErrorRule, missingDefaults.Rule) - if g, w := missingDefaults.Args["error"], "module missing_defaults missing dependencies: missing\n"; g != w { - t.Errorf("want error %q, got %q", w, g) - } + AssertStringEquals(t, "missing_defaults", "module missing_defaults missing dependencies: missing\n", missingDefaults.Args["error"]) // TODO: missing transitive defaults is currently not handled _ = missingTransitiveDefaults |