tequilaOS/platform_build_soong
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Support multiple expected errors in neverallow_test.go

Browse source 
Extracted some common code used by visibility_test.go and
neverallow_test.go into a new function CheckErrorsAgainstExpectations
in testing.go.

Bug: 138428610
Test: m nothing
Change-Id: I6996fa52968c387e38b4a67ffa59cd07c2d524e8
Merged-In: Iafbadf12c6ffdc4d9128fcfe7f15792df5cfd020
This commit is contained in:
Paul Duffin 2019-08-05 15:07:57 +01:00
parent cc4b0ad706
commit 91e3819335
3 changed files with 55 additions and 39 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

53
android/neverallow_test.go
View file

@ -28,9 +28,9 @@ func init() {
}
var neverallowTests = []struct {
name string
fs map[string][]byte
expectedError string
name string
fs map[string][]byte
expectedErrors []string
}{
// Test General Functionality
@ -48,7 +48,9 @@ var neverallowTests = []struct {
static_libs: ["not_allowed_in_direct_deps"],
}`),
},
expectedError: `module "libother": violates neverallow deps:not_allowed_in_direct_deps`,
expectedErrors: []string{
`module "libother": violates neverallow deps:not_allowed_in_direct_deps`,
},
},
// Test specific rules
@ -63,7 +65,9 @@ var neverallowTests = []struct {
include_dirs: ["art/libdexfile/include"],
}`),
},
expectedError: "all usages of 'art' have been migrated",
expectedErrors: []string{
"all usages of 'art' have been migrated",
},
},
{
name: "include_dir can reference another location",
@ -88,7 +92,9 @@ var neverallowTests = []struct {
},
}`),
},
expectedError: "VNDK can never contain a library that is device dependent",
expectedErrors: []string{
"VNDK can never contain a library that is device dependent",
},
},
{
name: "no vndk.enabled under device directory",
@ -102,7 +108,9 @@ var neverallowTests = []struct {
},
}`),
},
expectedError: "VNDK can never contain a library that is device dependent",
expectedErrors: []string{
"VNDK can never contain a library that is device dependent",
},
},
{
name: "vndk-ext under vendor or device directory",
@ -124,7 +132,6 @@ var neverallowTests = []struct {
},
}`),
},
expectedError: "",
},
{
@ -140,7 +147,9 @@ var neverallowTests = []struct {
},
}`),
},
expectedError: "manifest enforcement should be independent",
expectedErrors: []string{
"manifest enforcement should be independent",
},
},
{
name: "libhidltransport enforce_vintf_manifest.cflags",
@ -155,7 +164,6 @@ var neverallowTests = []struct {
},
}`),
},
expectedError: "",
},
{
@ -171,7 +179,9 @@ var neverallowTests = []struct {
},
}`),
},
expectedError: "nothing should care if linker namespaces are enabled or not",
expectedErrors: []string{
"nothing should care if linker namespaces are enabled or not",
},
},
{
name: "libc_bionic_ndk treble_linker_namespaces.cflags",
@ -186,7 +196,6 @@ var neverallowTests = []struct {
},
}`),
},
expectedError: "",
},
{
name: "java_device_for_host",
@ -197,7 +206,9 @@ var neverallowTests = []struct {
libs: ["core-libart"],
}`),
},
expectedError: "java_device_for_host can only be used in whitelisted projects",
expectedErrors: []string{
"java_device_for_host can only be used in whitelisted projects",
},
},
// Libcore rule tests
{
@ -219,7 +230,9 @@ var neverallowTests = []struct {
sdk_version: "none",
}`),
},
expectedError: "module \"outside_core_libraries\": violates neverallow",
expectedErrors: []string{
"module \"outside_core_libraries\": violates neverallow",
},
},
{
name: "sdk_version: \"current\"",
@ -237,19 +250,15 @@ func TestNeverallow(t *testing.T) {
config := TestConfig(buildDir, nil)
for _, test := range neverallowTests {
t.Run(test.name, func(t *testing.T) {
_, errs := testNeverallow(t, config, test.fs)
if test.expectedError == "" {
FailIfErrored(t, errs)
} else {
FailIfNoMatchingErrors(t, test.expectedError, errs)
}
t.Run(test.name, func(t *testing.T) {
_, errs := testNeverallow(config, test.fs)
CheckErrorsAgainstExpectations(t, errs, test.expectedErrors)
})
}
}
func testNeverallow(t *testing.T, config Config, fs map[string][]byte) (*TestContext, []error) {
func testNeverallow(config Config, fs map[string][]byte) (*TestContext, []error) {
ctx := NewTestContext()
ctx.RegisterModuleType("cc_library", ModuleFactoryAdaptor(newMockCcLibraryModule))
ctx.RegisterModuleType("java_library", ModuleFactoryAdaptor(newMockJavaLibraryModule))

23
android/testing.go
View file

@ -372,6 +372,29 @@ func FailIfNoMatchingErrors(t *testing.T, pattern string, errs []error) {
}
}
func CheckErrorsAgainstExpectations(t *testing.T, errs []error, expectedErrorPatterns []string) {
t.Helper()
if expectedErrorPatterns == nil {
FailIfErrored(t, errs)
} else {
for _, expectedError := range expectedErrorPatterns {
FailIfNoMatchingErrors(t, expectedError, errs)
}
if len(errs) > len(expectedErrorPatterns) {
t.Errorf("additional errors found, expected %d, found %d",
len(expectedErrorPatterns), len(errs))
for i, expectedError := range expectedErrorPatterns {
t.Errorf("expectedErrors[%d] = %s", i, expectedError)
}
for i, err := range errs {
t.Errorf("errs[%d] = %s", i, err)
}
}
}
}
func AndroidMkEntriesForTest(t *testing.T, config Config, bpPath string, mod blueprint.Module) AndroidMkEntries {
var p AndroidMkEntriesProvider
var ok bool

18
android/visibility_test.go
View file

@ -860,23 +860,7 @@ func TestVisibility(t *testing.T) {
t.Run(test.name, func(t *testing.T) {
_, errs := testVisibility(buildDir, test.fs)
expectedErrors := test.expectedErrors
if expectedErrors == nil {
FailIfErrored(t, errs)
} else {
for _, expectedError := range expectedErrors {
FailIfNoMatchingErrors(t, expectedError, errs)
}
if len(errs) > len(expectedErrors) {
t.Errorf("additional errors found, expected %d, found %d", len(expectedErrors), len(errs))
for i, expectedError := range expectedErrors {
t.Errorf("expectedErrors[%d] = %s", i, expectedError)
}
for i, err := range errs {
t.Errorf("errs[%d] = %s", i, err)
}
}
}
CheckErrorsAgainstExpectations(t, errs, test.expectedErrors)
})
}
}