diff options
author | 2024-10-07 15:05:23 -0700 | |
---|---|---|
committer | 2024-10-07 20:12:40 -0700 | |
commit | b2388e3e3d67ab20292ada184bfbeb32d65d7d4c (patch) | |
tree | 63cdf7c1a2605ba9dba49ef7a6eb111905cdfc80 /android/mutator_test.go | |
parent | d224fcab8b898f2cd68b10360a1e3aa17d46ba09 (diff) |
Remove unused TopDownMutatorContext methods
Remove methods on TopDownMutatorContext that are no longer used.
This deletes everything unique to BaseMutatorContext, so delete that
too and replace it with BaseModuleContext.
Bug: 367784740
Test: builds
Flag: EXEMPT refactor
Change-Id: Id37c7b7e3d46d75afaf7752862b08f766217f08b
Diffstat (limited to 'android/mutator_test.go')
-rw-r--r-- | android/mutator_test.go | 38 |
1 files changed, 15 insertions, 23 deletions
diff --git a/android/mutator_test.go b/android/mutator_test.go index 5d4074a54..88bf1d2ea 100644 --- a/android/mutator_test.go +++ b/android/mutator_test.go @@ -134,10 +134,6 @@ func TestModuleString(t *testing.T) { return []string{"a", "b"} }, }) - ctx.TopDown("rename_top_down", func(ctx TopDownMutatorContext) { - moduleStrings = append(moduleStrings, ctx.Module().String()) - ctx.Rename(ctx.Module().base().Name() + "_renamed1") - }) }) ctx.PreDepsMutators(func(ctx RegisterMutatorsContext) { @@ -161,7 +157,7 @@ func TestModuleString(t *testing.T) { }) ctx.BottomUp("rename_bottom_up", func(ctx BottomUpMutatorContext) { moduleStrings = append(moduleStrings, ctx.Module().String()) - ctx.Rename(ctx.Module().base().Name() + "_renamed2") + ctx.Rename(ctx.Module().base().Name() + "_renamed1") }) ctx.BottomUp("final", func(ctx BottomUpMutatorContext) { moduleStrings = append(moduleStrings, ctx.Module().String()) @@ -181,17 +177,23 @@ func TestModuleString(t *testing.T) { "foo{pre_arch:b}", "foo{pre_arch:a}", - // After rename_top_down (reversed because pre_deps TransitionMutator.Split is TopDown). - "foo_renamed1{pre_arch:b}", - "foo_renamed1{pre_arch:a}", - // After pre_deps (reversed because post_deps TransitionMutator.Split is TopDown). - "foo_renamed1{pre_arch:b,pre_deps:d}", - "foo_renamed1{pre_arch:b,pre_deps:c}", - "foo_renamed1{pre_arch:a,pre_deps:d}", - "foo_renamed1{pre_arch:a,pre_deps:c}", + "foo{pre_arch:b,pre_deps:d}", + "foo{pre_arch:b,pre_deps:c}", + "foo{pre_arch:a,pre_deps:d}", + "foo{pre_arch:a,pre_deps:c}", // After post_deps. + "foo{pre_arch:a,pre_deps:c,post_deps:e}", + "foo{pre_arch:a,pre_deps:c,post_deps:f}", + "foo{pre_arch:a,pre_deps:d,post_deps:e}", + "foo{pre_arch:a,pre_deps:d,post_deps:f}", + "foo{pre_arch:b,pre_deps:c,post_deps:e}", + "foo{pre_arch:b,pre_deps:c,post_deps:f}", + "foo{pre_arch:b,pre_deps:d,post_deps:e}", + "foo{pre_arch:b,pre_deps:d,post_deps:f}", + + // After rename_bottom_up. "foo_renamed1{pre_arch:a,pre_deps:c,post_deps:e}", "foo_renamed1{pre_arch:a,pre_deps:c,post_deps:f}", "foo_renamed1{pre_arch:a,pre_deps:d,post_deps:e}", @@ -200,16 +202,6 @@ func TestModuleString(t *testing.T) { "foo_renamed1{pre_arch:b,pre_deps:c,post_deps:f}", "foo_renamed1{pre_arch:b,pre_deps:d,post_deps:e}", "foo_renamed1{pre_arch:b,pre_deps:d,post_deps:f}", - - // After rename_bottom_up. - "foo_renamed2{pre_arch:a,pre_deps:c,post_deps:e}", - "foo_renamed2{pre_arch:a,pre_deps:c,post_deps:f}", - "foo_renamed2{pre_arch:a,pre_deps:d,post_deps:e}", - "foo_renamed2{pre_arch:a,pre_deps:d,post_deps:f}", - "foo_renamed2{pre_arch:b,pre_deps:c,post_deps:e}", - "foo_renamed2{pre_arch:b,pre_deps:c,post_deps:f}", - "foo_renamed2{pre_arch:b,pre_deps:d,post_deps:e}", - "foo_renamed2{pre_arch:b,pre_deps:d,post_deps:f}", } AssertDeepEquals(t, "module String() values", want, moduleStrings) |