From fd18c42b37b800b31deb598a01ae5ce5f9591f7a Mon Sep 17 00:00:00 2001
From: Bill Yang <yangbill@google.com>
Date: Thu, 25 Oct 2018 05:36:13 +0000
Subject: [PATCH] Revert "ATest: GTest auto gen config support run_test_as in
 Android.bp ."

This reverts commit d30b940dffa10b1a2e703178af386d4b5a8616ae.

Reason for revert: sed -i requires an argument on mac, break build on mac.

Change-Id: Iba2db061ae888d265e02b750e50959019cf0f168
---
 cc/test.go          | 19 +------------------
 tradefed/autogen.go | 44 ++++++++++++++------------------------------
 2 files changed, 15 insertions(+), 48 deletions(-)

diff --git a/cc/test.go b/cc/test.go
index e6251d33c..96049db8d 100644
--- a/cc/test.go
+++ b/cc/test.go
@@ -31,12 +31,6 @@ type TestProperties struct {
 	Isolated *bool
 }
 
-// Test option struct.
-type TestOptions struct {
-	// the UID that you want to run in device.
-	Run_test_as string `android:"arch_variant"`
-}
-
 type TestBinaryProperties struct {
 	// Create a separate binary for each source file.  Useful when there is
 	// global state that can not be torn down and reset between each test suite.
@@ -62,9 +56,6 @@ type TestBinaryProperties struct {
 	// the name of the test configuration template (for example "AndroidTestTemplate.xml") that
 	// should be installed with the module.
 	Test_config_template *string `android:"arch_variant"`
-
-	// Test options.
-	Test_options *TestOptions
 }
 
 func init() {
@@ -253,16 +244,8 @@ func (test *testBinary) linkerFlags(ctx ModuleContext, flags Flags) Flags {
 
 func (test *testBinary) install(ctx ModuleContext, file android.Path) {
 	test.data = ctx.ExpandSources(test.Properties.Data, nil)
-
-	// Append new line in template like below
-	// <option name="run-test-as" value="1234" />
-	optionsMap := map[string]string{}
-	if test.Properties.Test_options != nil {
-		optionsMap["run-test-as"] = string(test.Properties.Test_options.Run_test_as)
-	}
-
 	test.testConfig = tradefed.AutoGenNativeTestConfig(ctx, test.Properties.Test_config,
-		test.Properties.Test_config_template, optionsMap)
+		test.Properties.Test_config_template)
 
 	test.binaryDecorator.baseInstaller.dir = "nativetest"
 	test.binaryDecorator.baseInstaller.dir64 = "nativetest64"
diff --git a/tradefed/autogen.go b/tradefed/autogen.go
index 5fb85c3ca..131fdc44c 100644
--- a/tradefed/autogen.go
+++ b/tradefed/autogen.go
@@ -15,7 +15,6 @@
 package tradefed
 
 import (
-	"fmt"
 	"strings"
 
 	"github.com/google/blueprint"
@@ -37,10 +36,9 @@ func getTestConfig(ctx android.ModuleContext, prop *string) android.Path {
 }
 
 var autogenTestConfig = pctx.StaticRule("autogenTestConfig", blueprint.RuleParams{
-	Command: "sed 's&{MODULE}&${name}&g' $template > $out &&" +
-		"${optionCmd} $out",
+	Command:     "sed 's&{MODULE}&${name}&g' $template > $out",
 	CommandDeps: []string{"$template"},
-}, "name", "template", "optionCmd")
+}, "name", "template")
 
 func testConfigPath(ctx android.ModuleContext, prop *string) (path android.Path, autogenPath android.WritablePath) {
 	if p := getTestConfig(ctx, prop); p != nil {
@@ -56,44 +54,30 @@ func testConfigPath(ctx android.ModuleContext, prop *string) (path android.Path,
 	}
 }
 
-func autogenTemplate(ctx android.ModuleContext, output android.WritablePath, template string, optionsMap map[string]string) {
-	// If no test option found, delete {UID_OPTION} line.
-	// If found, replace it with corresponding options format.
-	optionCmd := "sed -i '/{UID_OPTION}/d'"
-	if optionsMap != nil {
-		//Append options
-		var options []string
-		for optionName, value := range optionsMap {
-			if value != "" {
-				options = append(options, fmt.Sprintf("<option name=\"%s\" value=\"%s\" />", optionName, value))
-			}
-		}
-		optionCmd = fmt.Sprintf("sed -i 's&{UID_OPTION}&%s&g'", strings.Join(options, "\\n        "))
-	}
+func autogenTemplate(ctx android.ModuleContext, output android.WritablePath, template string) {
 	ctx.Build(pctx, android.BuildParams{
 		Rule:        autogenTestConfig,
 		Description: "test config",
 		Output:      output,
 		Args: map[string]string{
-			"name":      ctx.ModuleName(),
-			"template":  template,
-			"optionCmd": optionCmd,
+			"name":     ctx.ModuleName(),
+			"template": template,
 		},
 	})
 }
 
 func AutoGenNativeTestConfig(ctx android.ModuleContext, testConfigProp *string,
-	testConfigTemplateProp *string, optionsMap map[string]string) android.Path {
+	testConfigTemplateProp *string) android.Path {
 	path, autogenPath := testConfigPath(ctx, testConfigProp)
 	if autogenPath != nil {
 		templatePath := getTestConfigTemplate(ctx, testConfigTemplateProp)
 		if templatePath.Valid() {
-			autogenTemplate(ctx, autogenPath, templatePath.String(), optionsMap)
+			autogenTemplate(ctx, autogenPath, templatePath.String())
 		} else {
 			if ctx.Device() {
-				autogenTemplate(ctx, autogenPath, "${NativeTestConfigTemplate}", optionsMap)
+				autogenTemplate(ctx, autogenPath, "${NativeTestConfigTemplate}")
 			} else {
-				autogenTemplate(ctx, autogenPath, "${NativeHostTestConfigTemplate}", optionsMap)
+				autogenTemplate(ctx, autogenPath, "${NativeHostTestConfigTemplate}")
 			}
 		}
 		return autogenPath
@@ -107,9 +91,9 @@ func AutoGenNativeBenchmarkTestConfig(ctx android.ModuleContext, testConfigProp
 	if autogenPath != nil {
 		templatePath := getTestConfigTemplate(ctx, testConfigTemplateProp)
 		if templatePath.Valid() {
-			autogenTemplate(ctx, autogenPath, templatePath.String(), nil)
+			autogenTemplate(ctx, autogenPath, templatePath.String())
 		} else {
-			autogenTemplate(ctx, autogenPath, "${NativeBenchmarkTestConfigTemplate}", nil)
+			autogenTemplate(ctx, autogenPath, "${NativeBenchmarkTestConfigTemplate}")
 		}
 		return autogenPath
 	}
@@ -121,12 +105,12 @@ func AutoGenJavaTestConfig(ctx android.ModuleContext, testConfigProp *string, te
 	if autogenPath != nil {
 		templatePath := getTestConfigTemplate(ctx, testConfigTemplateProp)
 		if templatePath.Valid() {
-			autogenTemplate(ctx, autogenPath, templatePath.String(), nil)
+			autogenTemplate(ctx, autogenPath, templatePath.String())
 		} else {
 			if ctx.Device() {
-				autogenTemplate(ctx, autogenPath, "${JavaTestConfigTemplate}", nil)
+				autogenTemplate(ctx, autogenPath, "${JavaTestConfigTemplate}")
 			} else {
-				autogenTemplate(ctx, autogenPath, "${JavaHostTestConfigTemplate}", nil)
+				autogenTemplate(ctx, autogenPath, "${JavaHostTestConfigTemplate}")
 			}
 		}
 		return autogenPath