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

Commit 70975ac8 authored by satayev's avatar satayev Committed by Gerrit Code Review
Browse files

Merge "Rename generate proto config file to match classpath type."

parents 8a7bbb5b 227e745e
Loading
Loading
Loading
Loading
+2 −2
Original line number Diff line number Diff line
@@ -382,7 +382,7 @@ func TestBootclasspathFragmentInArtApex(t *testing.T) {
		).RunTest(t)

		ensureExactContents(t, result.TestContext, "com.android.art", "android_common_com.android.art_image", []string{
			"etc/classpaths/mybootclasspathfragment.pb",
			"etc/classpaths/bootclasspath.pb",
			"javalib/arm/boot.art",
			"javalib/arm/boot.oat",
			"javalib/arm/boot.vdex",
@@ -594,7 +594,7 @@ func TestBootclasspathFragmentContentsNoName(t *testing.T) {
	ensureExactContents(t, result.TestContext, "myapex", "android_common_myapex_image", []string{
		// This does not include art, oat or vdex files as they are only included for the art boot
		// image.
		"etc/classpaths/mybootclasspathfragment.pb",
		"etc/classpaths/bootclasspath.pb",
		"javalib/bar.jar",
		"javalib/foo.jar",
	})
+1 −1
Original line number Diff line number Diff line
@@ -67,7 +67,7 @@ func TestSystemserverclasspathFragmentContents(t *testing.T) {
	`)

	ensureExactContents(t, result.TestContext, "myapex", "android_common_myapex_image", []string{
		"etc/classpaths/mysystemserverclasspathfragment.pb",
		"etc/classpaths/systemserverclasspath.pb",
		"javalib/foo.jar",
	})

+1 −1
Original line number Diff line number Diff line
@@ -105,7 +105,7 @@ func configuredJarListToClasspathJars(ctx android.ModuleContext, configuredJars
}

func (c *ClasspathFragmentBase) generateClasspathProtoBuildActions(ctx android.ModuleContext, jars []classpathJar) {
	outputFilename := ctx.ModuleName() + ".pb"
	outputFilename := strings.ToLower(c.classpathType.String()) + ".pb"
	c.outputFilepath = android.PathForModuleOut(ctx, outputFilename).OutputPath
	c.installDirPath = android.PathForModuleInstall(ctx, "etc", "classpaths")

+2 −2
Original line number Diff line number Diff line
@@ -287,7 +287,7 @@ func TestPlatformBootclasspath_ClasspathFragmentPaths(t *testing.T) {
	).RunTest(t)

	p := result.Module("platform-bootclasspath", "android_common").(*platformBootclasspathModule)
	android.AssertStringEquals(t, "output filepath", p.Name()+".pb", p.ClasspathFragmentBase.outputFilepath.Base())
	android.AssertStringEquals(t, "output filepath", "bootclasspath.pb", p.ClasspathFragmentBase.outputFilepath.Base())
	android.AssertPathRelativeToTopEquals(t, "install filepath", "out/soong/target/product/test_device/system/etc/classpaths", p.ClasspathFragmentBase.installDirPath)
}

@@ -327,7 +327,7 @@ func TestPlatformBootclasspathModule_AndroidMkEntries(t *testing.T) {
		want := map[string][]string{
			"LOCAL_MODULE":                {"platform-bootclasspath"},
			"LOCAL_MODULE_CLASS":          {"ETC"},
			"LOCAL_INSTALLED_MODULE_STEM": {"platform-bootclasspath.pb"},
			"LOCAL_INSTALLED_MODULE_STEM": {"bootclasspath.pb"},
			// Output and Install paths are tested separately in TestPlatformBootclasspath_ClasspathFragmentPaths
		}

+7 −7
Original line number Diff line number Diff line
@@ -24,7 +24,7 @@ var prepareForTestWithSystemServerClasspath = android.GroupFixturePreparers(
	PrepareForTestWithJavaDefaultModules,
)

func TestPlatformSystemserverClasspathVariant(t *testing.T) {
func TestPlatformSystemServerClasspathVariant(t *testing.T) {
	result := android.GroupFixturePreparers(
		prepareForTestWithSystemServerClasspath,
		android.FixtureWithRootAndroidBp(`
@@ -38,7 +38,7 @@ func TestPlatformSystemserverClasspathVariant(t *testing.T) {
	android.AssertIntEquals(t, "expect 1 variant", 1, len(variants))
}

func TestPlatformSystemserverClasspath_ClasspathFragmentPaths(t *testing.T) {
func TestPlatformSystemServerClasspath_ClasspathFragmentPaths(t *testing.T) {
	result := android.GroupFixturePreparers(
		prepareForTestWithSystemServerClasspath,
		android.FixtureWithRootAndroidBp(`
@@ -49,11 +49,11 @@ func TestPlatformSystemserverClasspath_ClasspathFragmentPaths(t *testing.T) {
	).RunTest(t)

	p := result.Module("platform-systemserverclasspath", "android_common").(*platformSystemServerClasspathModule)
	android.AssertStringEquals(t, "output filepath", p.Name()+".pb", p.ClasspathFragmentBase.outputFilepath.Base())
	android.AssertStringEquals(t, "output filepath", "systemserverclasspath.pb", p.ClasspathFragmentBase.outputFilepath.Base())
	android.AssertPathRelativeToTopEquals(t, "install filepath", "out/soong/target/product/test_device/system/etc/classpaths", p.ClasspathFragmentBase.installDirPath)
}

func TestPlatformSystemserverClasspathModule_AndroidMkEntries(t *testing.T) {
func TestPlatformSystemServerClasspathModule_AndroidMkEntries(t *testing.T) {
	preparer := android.GroupFixturePreparers(
		prepareForTestWithSystemServerClasspath,
		android.FixtureWithRootAndroidBp(`
@@ -78,8 +78,8 @@ func TestPlatformSystemserverClasspathModule_AndroidMkEntries(t *testing.T) {
		want := map[string][]string{
			"LOCAL_MODULE":                {"platform-systemserverclasspath"},
			"LOCAL_MODULE_CLASS":          {"ETC"},
			"LOCAL_INSTALLED_MODULE_STEM": {"platform-systemserverclasspath.pb"},
			// Output and Install paths are tested separately in TestSystemserverClasspath_ClasspathFragmentPaths
			"LOCAL_INSTALLED_MODULE_STEM": {"systemserverclasspath.pb"},
			// Output and Install paths are tested separately in TestPlatformSystemServerClasspath_ClasspathFragmentPaths
		}

		p := result.Module("platform-systemserverclasspath", "android_common").(*platformSystemServerClasspathModule)
@@ -96,7 +96,7 @@ func TestPlatformSystemserverClasspathModule_AndroidMkEntries(t *testing.T) {
	})
}

func TestSystemserverclasspathFragmentWithoutContents(t *testing.T) {
func TestSystemServerClasspathFragmentWithoutContents(t *testing.T) {
	prepareForTestWithSystemServerClasspath.
		ExtendWithErrorHandler(android.FixtureExpectsAtLeastOneErrorMatchingPattern(
			`\Qempty contents are not allowed\E`)).