Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit a0aa5e03 authored by Spandan Das's avatar Spandan Das Committed by Automerger Merge Worker
Browse files

Ignore prebuilt apex_contributions in coverage builds am: bffd7fba 

Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2976971



Change-Id: Ieba81fa75cec4a38159c16befa1d84c0df12aabc
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 0091bf0d bffd7fba

android/apex_contributions.go

+11 −0
Original line number Diff line number Diff line
@@ -15,6 +15,8 @@ 
package android



import (

	"strings"



	"github.com/google/blueprint"

	"github.com/google/blueprint/proptools"

)
@@ -102,6 +104,15 @@ func (a *allApexContributions) SetPrebuiltSelectionInfoProvider(ctx BaseModuleCo 
			if InList(content, ctx.Config().BuildIgnoreApexContributionContents()) {

				continue

			}

			// Coverage builds for TARGET_RELEASE=foo should always build from source,

			// even if TARGET_RELEASE=foo uses prebuilt mainline modules.

			// This is necessary because the checked-in prebuilts were generated with

			// instrumentation turned off.

			//

			// Skip any prebuilt contents in coverage builds

			if strings.HasPrefix(content, "prebuilt_") && (ctx.Config().JavaCoverageEnabled() || ctx.DeviceConfig().NativeCoverageEnabled()) {

				continue

			}

			if !ctx.OtherModuleExists(content) && !ctx.Config().AllowMissingDependencies() {

				ctx.ModuleErrorf("%s listed in apex_contributions %s does not exist\n", content, m.Name())

			}

android/prebuilt_test.go

+42 −0
Original line number Diff line number Diff line
@@ -762,3 +762,45 @@ func TestPrebuiltErrorCannotListBothSourceAndPrebuiltInContributions(t *testing. 
		}

		`, selectMainlineModuleContritbutions)

}



// Test that apex_contributions of prebuilt modules are ignored in coverage builds

func TestSourceIsSelectedInCoverageBuilds(t *testing.T) {

	prebuiltMainlineContributions := GroupFixturePreparers(

		FixtureModifyProductVariables(func(variables FixtureProductVariables) {

			variables.BuildFlags = map[string]string{

				"RELEASE_APEX_CONTRIBUTIONS_ADSERVICES": "my_prebuilt_apex_contributions",

			}

		}),

		FixtureMergeEnv(map[string]string{

			"EMMA_INSTRUMENT_FRAMEWORK": "true",

		}),

	)

	bp := `

		source {

			name: "foo",

		}

		prebuilt {

			name: "foo",

			srcs: ["prebuilt_file"],

		}

		apex_contributions {

			name: "my_prebuilt_apex_contributions",

			api_domain: "my_mainline_module",

			contents: [

			  "prebuilt_foo",

			],

		}

		all_apex_contributions {

			name: "all_apex_contributions",

		}

		`

	ctx := GroupFixturePreparers(

		PrepareForTestWithArchMutator,

		PrepareForTestWithPrebuilts,

		FixtureRegisterWithContext(registerTestPrebuiltModules),

		prebuiltMainlineContributions).RunTestWithBp(t, bp)

	source := ctx.ModuleForTests("foo", "android_common").Module()

	AssertBoolEquals(t, "Source should be preferred in coverage builds", true, !source.IsHideFromMake())

	prebuilt := ctx.ModuleForTests("prebuilt_foo", "android_common").Module()

	AssertBoolEquals(t, "Prebuilt should not be preferred in coverage builds", false, !prebuilt.IsHideFromMake())

}