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

Commit f1adfb38 authored by Artur Satayev's avatar Artur Satayev Committed by Android (Google) Code Review
Browse files

Merge "Rename generate proto config file to match classpath type." into sc-dev

parents bbe2d3db 53d0678b
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`)).