Add a helper function that creates a test fixture preparer that sets a build flag, and use it everywhere that was setting build flags manually. Test: all soong tests Flag: EXEMPT refactor Change-Id: I68d50d68787a30d091f0827e8caa51f5c5a762ef
215 lines
6.4 KiB
Go
215 lines
6.4 KiB
Go
// Copyright 2017 Google Inc. All rights reserved.
|
|
//
|
|
// Licensed under the Apache License, Version 2.0 (the "License");
|
|
// you may not use this file except in compliance with the License.
|
|
// You may obtain a copy of the License at
|
|
//
|
|
// http://www.apache.org/licenses/LICENSE-2.0
|
|
//
|
|
// Unless required by applicable law or agreed to in writing, software
|
|
// distributed under the License is distributed on an "AS IS" BASIS,
|
|
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
// See the License for the specific language governing permissions and
|
|
// limitations under the License.
|
|
|
|
package android
|
|
|
|
import (
|
|
"fmt"
|
|
"path/filepath"
|
|
"reflect"
|
|
"strings"
|
|
"testing"
|
|
)
|
|
|
|
func validateConfigAnnotations(configurable jsonConfigurable) (err error) {
|
|
reflectType := reflect.TypeOf(configurable)
|
|
reflectType = reflectType.Elem()
|
|
for i := 0; i < reflectType.NumField(); i++ {
|
|
field := reflectType.Field(i)
|
|
jsonTag := field.Tag.Get("json")
|
|
// Check for mistakes in the json tag
|
|
if jsonTag != "" && !strings.HasPrefix(jsonTag, ",") {
|
|
if !strings.Contains(jsonTag, ",") {
|
|
// Probably an accidental rename, most likely "omitempty" instead of ",omitempty"
|
|
return fmt.Errorf("Field %s.%s has tag %s which specifies to change its json field name to %q.\n"+
|
|
"Did you mean to use an annotation of %q?\n"+
|
|
"(Alternatively, to change the json name of the field, rename the field in source instead.)",
|
|
reflectType.Name(), field.Name, field.Tag, jsonTag, ","+jsonTag)
|
|
} else {
|
|
// Although this rename was probably intentional,
|
|
// a json annotation is still more confusing than renaming the source variable
|
|
requestedName := strings.Split(jsonTag, ",")[0]
|
|
return fmt.Errorf("Field %s.%s has tag %s which specifies to change its json field name to %q.\n"+
|
|
"To change the json name of the field, rename the field in source instead.",
|
|
reflectType.Name(), field.Name, field.Tag, requestedName)
|
|
|
|
}
|
|
}
|
|
}
|
|
return nil
|
|
}
|
|
|
|
type configType struct {
|
|
PopulateMe *bool `json:"omitempty"`
|
|
}
|
|
|
|
func (c *configType) SetDefaultConfig() {
|
|
}
|
|
|
|
// tests that ValidateConfigAnnotation works
|
|
func TestValidateConfigAnnotations(t *testing.T) {
|
|
config := configType{}
|
|
err := validateConfigAnnotations(&config)
|
|
expectedError := `Field configType.PopulateMe has tag json:"omitempty" which specifies to change its json field name to "omitempty".
|
|
Did you mean to use an annotation of ",omitempty"?
|
|
(Alternatively, to change the json name of the field, rename the field in source instead.)`
|
|
if err.Error() != expectedError {
|
|
t.Errorf("Incorrect error; expected:\n"+
|
|
"%s\n"+
|
|
"got:\n"+
|
|
"%s",
|
|
expectedError, err.Error())
|
|
}
|
|
}
|
|
|
|
// run validateConfigAnnotations against each type that might have json annotations
|
|
func TestProductConfigAnnotations(t *testing.T) {
|
|
err := validateConfigAnnotations(&ProductVariables{})
|
|
if err != nil {
|
|
t.Errorf(err.Error())
|
|
}
|
|
}
|
|
|
|
func TestMissingVendorConfig(t *testing.T) {
|
|
c := &config{}
|
|
if c.VendorConfig("test").Bool("not_set") {
|
|
t.Errorf("Expected false")
|
|
}
|
|
}
|
|
|
|
func verifyProductVariableMarshaling(t *testing.T, v ProductVariables) {
|
|
dir := t.TempDir()
|
|
path := filepath.Join(dir, "test.variables")
|
|
err := saveToConfigFile(&v, path)
|
|
if err != nil {
|
|
t.Errorf("Couldn't save default product config: %q", err)
|
|
}
|
|
|
|
var v2 ProductVariables
|
|
err = loadFromConfigFile(&v2, path)
|
|
if err != nil {
|
|
t.Errorf("Couldn't load default product config: %q", err)
|
|
}
|
|
}
|
|
func TestDefaultProductVariableMarshaling(t *testing.T) {
|
|
v := ProductVariables{}
|
|
v.SetDefaultConfig()
|
|
verifyProductVariableMarshaling(t, v)
|
|
}
|
|
|
|
func TestBootJarsMarshaling(t *testing.T) {
|
|
v := ProductVariables{}
|
|
v.SetDefaultConfig()
|
|
v.BootJars = ConfiguredJarList{
|
|
apexes: []string{"apex"},
|
|
jars: []string{"jar"},
|
|
}
|
|
|
|
verifyProductVariableMarshaling(t, v)
|
|
}
|
|
|
|
func assertStringEquals(t *testing.T, expected, actual string) {
|
|
if actual != expected {
|
|
t.Errorf("expected %q found %q", expected, actual)
|
|
}
|
|
}
|
|
|
|
func TestReleaseAconfigExtraReleaseConfigs(t *testing.T) {
|
|
testCases := []struct {
|
|
name string
|
|
flag string
|
|
expected []string
|
|
}{
|
|
{
|
|
name: "empty",
|
|
flag: "",
|
|
expected: []string{},
|
|
},
|
|
{
|
|
name: "specified",
|
|
flag: "bar foo",
|
|
expected: []string{"bar", "foo"},
|
|
},
|
|
{
|
|
name: "duplicates",
|
|
flag: "foo bar foo",
|
|
expected: []string{"foo", "bar"},
|
|
},
|
|
}
|
|
|
|
for _, tc := range testCases {
|
|
fixture := GroupFixturePreparers(
|
|
PrepareForTestWithBuildFlag("RELEASE_ACONFIG_EXTRA_RELEASE_CONFIGS", tc.flag),
|
|
)
|
|
actual := fixture.RunTest(t).Config.ReleaseAconfigExtraReleaseConfigs()
|
|
AssertArrayString(t, tc.name, tc.expected, actual)
|
|
}
|
|
}
|
|
|
|
func TestConfiguredJarList(t *testing.T) {
|
|
list1 := CreateTestConfiguredJarList([]string{"apex1:jarA"})
|
|
|
|
t.Run("create", func(t *testing.T) {
|
|
assertStringEquals(t, "apex1:jarA", list1.String())
|
|
})
|
|
|
|
t.Run("create invalid - missing apex", func(t *testing.T) {
|
|
defer func() {
|
|
err := recover().(error)
|
|
assertStringEquals(t, "malformed (apex, jar) pair: 'jarA', expected format: <apex>:<jar>", err.Error())
|
|
}()
|
|
CreateTestConfiguredJarList([]string{"jarA"})
|
|
})
|
|
|
|
t.Run("create invalid - empty apex", func(t *testing.T) {
|
|
defer func() {
|
|
err := recover().(error)
|
|
assertStringEquals(t, "invalid apex '' in <apex>:<jar> pair ':jarA', expected format: <apex>:<jar>", err.Error())
|
|
}()
|
|
CreateTestConfiguredJarList([]string{":jarA"})
|
|
})
|
|
|
|
list2 := list1.Append("apex2", "jarB")
|
|
t.Run("append", func(t *testing.T) {
|
|
assertStringEquals(t, "apex1:jarA,apex2:jarB", list2.String())
|
|
})
|
|
|
|
t.Run("append does not modify", func(t *testing.T) {
|
|
assertStringEquals(t, "apex1:jarA", list1.String())
|
|
})
|
|
|
|
// Make sure that two lists created by appending to the same list do not share storage.
|
|
list3 := list1.Append("apex3", "jarC")
|
|
t.Run("append does not share", func(t *testing.T) {
|
|
assertStringEquals(t, "apex1:jarA,apex2:jarB", list2.String())
|
|
assertStringEquals(t, "apex1:jarA,apex3:jarC", list3.String())
|
|
})
|
|
|
|
list4 := list3.RemoveList(list1)
|
|
t.Run("remove", func(t *testing.T) {
|
|
assertStringEquals(t, "apex3:jarC", list4.String())
|
|
})
|
|
|
|
t.Run("remove does not modify", func(t *testing.T) {
|
|
assertStringEquals(t, "apex1:jarA,apex3:jarC", list3.String())
|
|
})
|
|
|
|
// Make sure that two lists created by removing from the same list do not share storage.
|
|
list5 := list3.RemoveList(CreateTestConfiguredJarList([]string{"apex3:jarC"}))
|
|
t.Run("remove", func(t *testing.T) {
|
|
assertStringEquals(t, "apex3:jarC", list4.String())
|
|
assertStringEquals(t, "apex1:jarA", list5.String())
|
|
})
|
|
}
|