2021-06-01 22:18:08 +02:00
// Copyright 2021 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 java
import (
"fmt"
"path/filepath"
"regexp"
"testing"
2021-09-28 17:40:23 +02:00
"android/soong/android"
2021-06-01 22:18:08 +02:00
"github.com/google/blueprint/proptools"
)
func TestJavaSdkLibrary ( t * testing . T ) {
result := android . GroupFixturePreparers (
prepareForJavaTest ,
PrepareForTestWithJavaSdkLibraryFiles ,
FixtureWithPrebuiltApis ( map [ string ] [ ] string {
"28" : { "foo" } ,
"29" : { "foo" } ,
"30" : { "bar" , "barney" , "baz" , "betty" , "foo" , "fred" , "quuz" , "wilma" } ,
} ) ,
) . RunTestWithBp ( t , `
droiddoc_exported_dir {
name : "droiddoc-templates-sdk" ,
path : "." ,
}
java_sdk_library {
name : "foo" ,
srcs : [ "a.java" , "b.java" ] ,
api_packages : [ "foo" ] ,
}
java_sdk_library {
name : "bar" ,
srcs : [ "a.java" , "b.java" ] ,
api_packages : [ "bar" ] ,
}
java_library {
name : "baz" ,
srcs : [ "c.java" ] ,
libs : [ "foo" , "bar.stubs" ] ,
sdk_version : "system_current" ,
}
java_sdk_library {
name : "barney" ,
srcs : [ "c.java" ] ,
api_only : true ,
}
java_sdk_library {
name : "betty" ,
srcs : [ "c.java" ] ,
shared_library : false ,
}
java_sdk_library_import {
name : "quuz" ,
public : {
jars : [ "c.jar" ] ,
} ,
}
java_sdk_library_import {
name : "fred" ,
public : {
jars : [ "b.jar" ] ,
} ,
}
java_sdk_library_import {
name : "wilma" ,
public : {
jars : [ "b.jar" ] ,
} ,
shared_library : false ,
}
java_library {
name : "qux" ,
srcs : [ "c.java" ] ,
libs : [ "baz" , "fred" , "quuz.stubs" , "wilma" , "barney" , "betty" ] ,
sdk_version : "system_current" ,
}
java_library {
name : "baz-test" ,
srcs : [ "c.java" ] ,
libs : [ "foo" ] ,
sdk_version : "test_current" ,
}
java_library {
name : "baz-29" ,
srcs : [ "c.java" ] ,
libs : [ "foo" ] ,
sdk_version : "system_29" ,
}
java_library {
name : "baz-module-30" ,
srcs : [ "c.java" ] ,
libs : [ "foo" ] ,
sdk_version : "module_30" ,
}
2021-09-07 19:21:59 +02:00
` )
2021-06-01 22:18:08 +02:00
// check the existence of the internal modules
Remove duplicate component from sdk snapshot
Previously, an sdk snapshot could contain the following:
* A java_sdk_library_import module, e.g. "foo" which creates component
modules "foo.stubs", etc.
* A corresponding versioned module, e.g. "sdk_foo@current" which
created component modules "sdk_foo@current.stubs", etc.
* An internal (to the sdk snapshot) java_import for one of "foo"'s
components, e.g. "sdk_foo.stubs"
* A corresponding versioned module, e.g. "sdk_foo.stubs@current".
That causes a few problems:
1. The "foo.stubs" is duplicated.
2. The names of the components created by the versioned
java_sdk_library_import are invalid, as they append the component's
suffix to the version and not the name before the version.
The latter causes problems when building against prebuilts and fixing
that causes the generated snapshot to be invalid because it contains
duplicate definitions of the "sdk_foo.stubs@current" module. One
explicitly in the Android.bp file and one created by the
"sdk_foo@current" module.
Removing the duplicates from the snapshot causes errors as the name
generated by the snapshot for the component module, i.e.
"sdk_foo.stubs@current" does not match the name generated by the
"sdk_foo@current", i.e. "sdk_foo@current.stubs".
This change fixes them together.
Bug: 179354495
Test: m nothing
Merged-In: I515f235fe21755b5275af12366e96c24c94c0273
Change-Id: I515f235fe21755b5275af12366e96c24c94c0273
(cherry picked from commit a1aa7387f74a49c8c974ba2198def0e081488624)
2021-04-29 22:50:40 +02:00
foo := result . ModuleForTests ( "foo" , "android_common" )
2021-06-01 22:18:08 +02:00
result . ModuleForTests ( apiScopePublic . stubsLibraryModuleName ( "foo" ) , "android_common" )
result . ModuleForTests ( apiScopeSystem . stubsLibraryModuleName ( "foo" ) , "android_common" )
result . ModuleForTests ( apiScopeTest . stubsLibraryModuleName ( "foo" ) , "android_common" )
result . ModuleForTests ( apiScopePublic . stubsSourceModuleName ( "foo" ) , "android_common" )
result . ModuleForTests ( apiScopeSystem . stubsSourceModuleName ( "foo" ) , "android_common" )
result . ModuleForTests ( apiScopeTest . stubsSourceModuleName ( "foo" ) , "android_common" )
result . ModuleForTests ( "foo" + sdkXmlFileSuffix , "android_common" )
result . ModuleForTests ( "foo.api.public.28" , "" )
result . ModuleForTests ( "foo.api.system.28" , "" )
result . ModuleForTests ( "foo.api.test.28" , "" )
2021-06-27 21:42:04 +02:00
exportedComponentsInfo := result . ModuleProvider ( foo . Module ( ) , android . ExportedComponentsInfoProvider ) . ( android . ExportedComponentsInfo )
Remove duplicate component from sdk snapshot
Previously, an sdk snapshot could contain the following:
* A java_sdk_library_import module, e.g. "foo" which creates component
modules "foo.stubs", etc.
* A corresponding versioned module, e.g. "sdk_foo@current" which
created component modules "sdk_foo@current.stubs", etc.
* An internal (to the sdk snapshot) java_import for one of "foo"'s
components, e.g. "sdk_foo.stubs"
* A corresponding versioned module, e.g. "sdk_foo.stubs@current".
That causes a few problems:
1. The "foo.stubs" is duplicated.
2. The names of the components created by the versioned
java_sdk_library_import are invalid, as they append the component's
suffix to the version and not the name before the version.
The latter causes problems when building against prebuilts and fixing
that causes the generated snapshot to be invalid because it contains
duplicate definitions of the "sdk_foo.stubs@current" module. One
explicitly in the Android.bp file and one created by the
"sdk_foo@current" module.
Removing the duplicates from the snapshot causes errors as the name
generated by the snapshot for the component module, i.e.
"sdk_foo.stubs@current" does not match the name generated by the
"sdk_foo@current", i.e. "sdk_foo@current.stubs".
This change fixes them together.
Bug: 179354495
Test: m nothing
Merged-In: I515f235fe21755b5275af12366e96c24c94c0273
Change-Id: I515f235fe21755b5275af12366e96c24c94c0273
(cherry picked from commit a1aa7387f74a49c8c974ba2198def0e081488624)
2021-04-29 22:50:40 +02:00
expectedFooExportedComponents := [ ] string {
"foo.stubs" ,
"foo.stubs.source" ,
"foo.stubs.source.system" ,
"foo.stubs.source.test" ,
"foo.stubs.system" ,
"foo.stubs.test" ,
}
android . AssertArrayString ( t , "foo exported components" , expectedFooExportedComponents , exportedComponentsInfo . Components )
2021-06-01 22:18:08 +02:00
bazJavac := result . ModuleForTests ( "baz" , "android_common" ) . Rule ( "javac" )
// tests if baz is actually linked to the stubs lib
android . AssertStringDoesContain ( t , "baz javac classpath" , bazJavac . Args [ "classpath" ] , "foo.stubs.system.jar" )
// ... and not to the impl lib
android . AssertStringDoesNotContain ( t , "baz javac classpath" , bazJavac . Args [ "classpath" ] , "foo.jar" )
// test if baz is not linked to the system variant of foo
android . AssertStringDoesNotContain ( t , "baz javac classpath" , bazJavac . Args [ "classpath" ] , "foo.stubs.jar" )
bazTestJavac := result . ModuleForTests ( "baz-test" , "android_common" ) . Rule ( "javac" )
// tests if baz-test is actually linked to the test stubs lib
android . AssertStringDoesContain ( t , "baz-test javac classpath" , bazTestJavac . Args [ "classpath" ] , "foo.stubs.test.jar" )
baz29Javac := result . ModuleForTests ( "baz-29" , "android_common" ) . Rule ( "javac" )
// tests if baz-29 is actually linked to the system 29 stubs lib
android . AssertStringDoesContain ( t , "baz-29 javac classpath" , baz29Javac . Args [ "classpath" ] , "prebuilts/sdk/29/system/foo.jar" )
bazModule30Javac := result . ModuleForTests ( "baz-module-30" , "android_common" ) . Rule ( "javac" )
// tests if "baz-module-30" is actually linked to the module 30 stubs lib
android . AssertStringDoesContain ( t , "baz-module-30 javac classpath" , bazModule30Javac . Args [ "classpath" ] , "prebuilts/sdk/30/module-lib/foo.jar" )
// test if baz has exported SDK lib names foo and bar to qux
qux := result . ModuleForTests ( "qux" , "android_common" )
if quxLib , ok := qux . Module ( ) . ( * Library ) ; ok {
2021-08-12 17:16:11 +02:00
requiredSdkLibs , optionalSdkLibs := quxLib . ClassLoaderContexts ( ) . UsesLibs ( )
2021-08-18 17:57:11 +02:00
android . AssertDeepEquals ( t , "qux exports (required)" , [ ] string { "fred" , "quuz" , "foo" , "bar" } , requiredSdkLibs )
2021-08-12 17:16:11 +02:00
android . AssertDeepEquals ( t , "qux exports (optional)" , [ ] string { } , optionalSdkLibs )
2021-06-01 22:18:08 +02:00
}
}
2021-09-07 19:21:59 +02:00
func TestJavaSdkLibrary_UpdatableLibrary ( t * testing . T ) {
result := android . GroupFixturePreparers (
prepareForJavaTest ,
PrepareForTestWithJavaSdkLibraryFiles ,
FixtureWithPrebuiltApis ( map [ string ] [ ] string {
"28" : { "foo" } ,
"29" : { "foo" } ,
"30" : { "foo" , "fooUpdatable" , "fooUpdatableErr" } ,
} ) ,
2021-09-28 17:40:23 +02:00
android . FixtureModifyProductVariables ( func ( variables android . FixtureProductVariables ) {
variables . Platform_version_active_codenames = [ ] string { "Tiramisu" , "U" , "V" , "W" }
} ) ,
2021-09-07 19:21:59 +02:00
) . RunTestWithBp ( t ,
`
java_sdk_library {
name : "fooUpdatable" ,
srcs : [ "a.java" , "b.java" ] ,
api_packages : [ "foo" ] ,
2021-09-28 17:40:23 +02:00
on_bootclasspath_since : "U" ,
on_bootclasspath_before : "V" ,
min_device_sdk : "W" ,
max_device_sdk : "current" ,
min_sdk_version : "S" ,
2021-09-07 19:21:59 +02:00
}
java_sdk_library {
name : "foo" ,
srcs : [ "a.java" , "b.java" ] ,
api_packages : [ "foo" ] ,
}
` )
// test that updatability attributes are passed on correctly
fooUpdatable := result . ModuleForTests ( "fooUpdatable.xml" , "android_common" ) . Rule ( "java_sdk_xml" )
2021-10-29 11:32:32 +02:00
android . AssertStringDoesContain ( t , "fooUpdatable.xml java_sdk_xml command" , fooUpdatable . RuleParams . Command , ` on-bootclasspath-since=\"9001\" ` )
android . AssertStringDoesContain ( t , "fooUpdatable.xml java_sdk_xml command" , fooUpdatable . RuleParams . Command , ` on-bootclasspath-before=\"9002\" ` )
android . AssertStringDoesContain ( t , "fooUpdatable.xml java_sdk_xml command" , fooUpdatable . RuleParams . Command , ` min-device-sdk=\"9003\" ` )
android . AssertStringDoesContain ( t , "fooUpdatable.xml java_sdk_xml command" , fooUpdatable . RuleParams . Command , ` max-device-sdk=\"10000\" ` )
2021-09-07 19:21:59 +02:00
// double check that updatability attributes are not written if they don't exist in the bp file
// the permissions file for the foo library defined above
fooPermissions := result . ModuleForTests ( "foo.xml" , "android_common" ) . Rule ( "java_sdk_xml" )
2021-10-29 11:32:32 +02:00
android . AssertStringDoesNotContain ( t , "foo.xml java_sdk_xml command" , fooPermissions . RuleParams . Command , ` on-bootclasspath-since ` )
android . AssertStringDoesNotContain ( t , "foo.xml java_sdk_xml command" , fooPermissions . RuleParams . Command , ` on-bootclasspath-before ` )
android . AssertStringDoesNotContain ( t , "foo.xml java_sdk_xml command" , fooPermissions . RuleParams . Command , ` min-device-sdk ` )
android . AssertStringDoesNotContain ( t , "foo.xml java_sdk_xml command" , fooPermissions . RuleParams . Command , ` max-device-sdk ` )
2021-09-07 19:21:59 +02:00
}
2021-09-28 17:40:23 +02:00
func TestJavaSdkLibrary_UpdatableLibrary_Validation_ValidVersion ( t * testing . T ) {
2021-09-07 19:21:59 +02:00
android . GroupFixturePreparers (
prepareForJavaTest ,
PrepareForTestWithJavaSdkLibraryFiles ,
FixtureWithPrebuiltApis ( map [ string ] [ ] string {
"30" : { "fooUpdatable" , "fooUpdatableErr" } ,
} ) ,
) . ExtendWithErrorHandler ( android . FixtureExpectsAllErrorsToMatchAPattern (
[ ] string {
` on_bootclasspath_since: "aaa" could not be parsed as an integer and is not a recognized codename ` ,
` on_bootclasspath_before: "bbc" could not be parsed as an integer and is not a recognized codename ` ,
` min_device_sdk: "ccc" could not be parsed as an integer and is not a recognized codename ` ,
` max_device_sdk: "ddd" could not be parsed as an integer and is not a recognized codename ` ,
} ) ) . RunTestWithBp ( t ,
`
java_sdk_library {
name : "fooUpdatableErr" ,
srcs : [ "a.java" , "b.java" ] ,
api_packages : [ "foo" ] ,
2021-09-28 17:40:23 +02:00
on_bootclasspath_since : "aaa" ,
on_bootclasspath_before : "bbc" ,
min_device_sdk : "ccc" ,
max_device_sdk : "ddd" ,
}
` )
}
func TestJavaSdkLibrary_UpdatableLibrary_Validation_AtLeastTAttributes ( t * testing . T ) {
android . GroupFixturePreparers (
prepareForJavaTest ,
PrepareForTestWithJavaSdkLibraryFiles ,
FixtureWithPrebuiltApis ( map [ string ] [ ] string {
"28" : { "foo" } ,
} ) ,
) . ExtendWithErrorHandler ( android . FixtureExpectsAllErrorsToMatchAPattern (
[ ] string {
"on_bootclasspath_since: Attribute value needs to be at least T" ,
"on_bootclasspath_before: Attribute value needs to be at least T" ,
"min_device_sdk: Attribute value needs to be at least T" ,
"max_device_sdk: Attribute value needs to be at least T" ,
} ,
) ) . RunTestWithBp ( t ,
`
java_sdk_library {
name : "foo" ,
srcs : [ "a.java" , "b.java" ] ,
api_packages : [ "foo" ] ,
on_bootclasspath_since : "S" ,
on_bootclasspath_before : "S" ,
min_device_sdk : "S" ,
max_device_sdk : "S" ,
min_sdk_version : "S" ,
}
` )
}
func TestJavaSdkLibrary_UpdatableLibrary_Validation_MinAndMaxDeviceSdk ( t * testing . T ) {
android . GroupFixturePreparers (
prepareForJavaTest ,
PrepareForTestWithJavaSdkLibraryFiles ,
FixtureWithPrebuiltApis ( map [ string ] [ ] string {
"28" : { "foo" } ,
} ) ,
android . FixtureModifyProductVariables ( func ( variables android . FixtureProductVariables ) {
variables . Platform_version_active_codenames = [ ] string { "Tiramisu" , "U" , "V" }
} ) ,
) . ExtendWithErrorHandler ( android . FixtureExpectsAllErrorsToMatchAPattern (
[ ] string {
"min_device_sdk can't be greater than max_device_sdk" ,
} ,
) ) . RunTestWithBp ( t ,
`
java_sdk_library {
name : "foo" ,
srcs : [ "a.java" , "b.java" ] ,
api_packages : [ "foo" ] ,
min_device_sdk : "V" ,
max_device_sdk : "U" ,
min_sdk_version : "S" ,
}
` )
}
func TestJavaSdkLibrary_UpdatableLibrary_Validation_MinAndMaxDeviceSdkAndModuleMinSdk ( t * testing . T ) {
android . GroupFixturePreparers (
prepareForJavaTest ,
PrepareForTestWithJavaSdkLibraryFiles ,
FixtureWithPrebuiltApis ( map [ string ] [ ] string {
"28" : { "foo" } ,
} ) ,
android . FixtureModifyProductVariables ( func ( variables android . FixtureProductVariables ) {
variables . Platform_version_active_codenames = [ ] string { "Tiramisu" , "U" , "V" }
} ) ,
) . ExtendWithErrorHandler ( android . FixtureExpectsAllErrorsToMatchAPattern (
[ ] string {
regexp . QuoteMeta ( "min_device_sdk: Can't be less than module's min sdk (V)" ) ,
regexp . QuoteMeta ( "max_device_sdk: Can't be less than module's min sdk (V)" ) ,
} ,
) ) . RunTestWithBp ( t ,
`
java_sdk_library {
name : "foo" ,
srcs : [ "a.java" , "b.java" ] ,
api_packages : [ "foo" ] ,
min_device_sdk : "U" ,
max_device_sdk : "U" ,
min_sdk_version : "V" ,
}
` )
}
func TestJavaSdkLibrary_UpdatableLibrary_usesNewTag ( t * testing . T ) {
result := android . GroupFixturePreparers (
prepareForJavaTest ,
PrepareForTestWithJavaSdkLibraryFiles ,
FixtureWithPrebuiltApis ( map [ string ] [ ] string {
"30" : { "foo" } ,
} ) ,
) . RunTestWithBp ( t ,
`
java_sdk_library {
name : "foo" ,
srcs : [ "a.java" , "b.java" ] ,
min_device_sdk : "Tiramisu" ,
min_sdk_version : "S" ,
2021-09-07 19:21:59 +02:00
}
` )
2021-09-28 17:40:23 +02:00
// test that updatability attributes are passed on correctly
fooUpdatable := result . ModuleForTests ( "foo.xml" , "android_common" ) . Rule ( "java_sdk_xml" )
android . AssertStringDoesContain ( t , "foo.xml java_sdk_xml command" , fooUpdatable . RuleParams . Command , ` <updatable-library ` )
android . AssertStringDoesNotContain ( t , "foo.xml java_sdk_xml command" , fooUpdatable . RuleParams . Command , ` <library ` )
2021-09-07 19:21:59 +02:00
}
2021-06-01 22:18:08 +02:00
func TestJavaSdkLibrary_StubOrImplOnlyLibs ( t * testing . T ) {
result := android . GroupFixturePreparers (
prepareForJavaTest ,
PrepareForTestWithJavaSdkLibraryFiles ,
FixtureWithLastReleaseApis ( "sdklib" ) ,
) . RunTestWithBp ( t , `
java_sdk_library {
name : "sdklib" ,
srcs : [ "a.java" ] ,
libs : [ "lib" ] ,
static_libs : [ "static-lib" ] ,
impl_only_libs : [ "impl-only-lib" ] ,
stub_only_libs : [ "stub-only-lib" ] ,
stub_only_static_libs : [ "stub-only-static-lib" ] ,
}
java_defaults {
name : "defaults" ,
srcs : [ "a.java" ] ,
sdk_version : "current" ,
}
java_library { name : "lib" , defaults : [ "defaults" ] }
java_library { name : "static-lib" , defaults : [ "defaults" ] }
java_library { name : "impl-only-lib" , defaults : [ "defaults" ] }
java_library { name : "stub-only-lib" , defaults : [ "defaults" ] }
java_library { name : "stub-only-static-lib" , defaults : [ "defaults" ] }
` )
var expectations = [ ] struct {
lib string
on_impl_classpath bool
on_stub_classpath bool
in_impl_combined bool
in_stub_combined bool
} {
{ lib : "lib" , on_impl_classpath : true } ,
{ lib : "static-lib" , in_impl_combined : true } ,
{ lib : "impl-only-lib" , on_impl_classpath : true } ,
{ lib : "stub-only-lib" , on_stub_classpath : true } ,
{ lib : "stub-only-static-lib" , in_stub_combined : true } ,
}
verify := func ( sdklib , dep string , cp , combined bool ) {
sdklibCp := result . ModuleForTests ( sdklib , "android_common" ) . Rule ( "javac" ) . Args [ "classpath" ]
expected := cp || combined // Every combined jar is also on the classpath.
android . AssertStringContainsEquals ( t , "bad classpath for " + sdklib , sdklibCp , "/" + dep + ".jar" , expected )
combineJarInputs := result . ModuleForTests ( sdklib , "android_common" ) . Rule ( "combineJar" ) . Inputs . Strings ( )
depPath := filepath . Join ( "out" , "soong" , ".intermediates" , dep , "android_common" , "turbine-combined" , dep + ".jar" )
android . AssertStringListContainsEquals ( t , "bad combined inputs for " + sdklib , combineJarInputs , depPath , combined )
}
for _ , expectation := range expectations {
verify ( "sdklib" , expectation . lib , expectation . on_impl_classpath , expectation . in_impl_combined )
verify ( "sdklib.impl" , expectation . lib , expectation . on_impl_classpath , expectation . in_impl_combined )
stubName := apiScopePublic . stubsLibraryModuleName ( "sdklib" )
verify ( stubName , expectation . lib , expectation . on_stub_classpath , expectation . in_stub_combined )
}
}
func TestJavaSdkLibrary_DoNotAccessImplWhenItIsNotBuilt ( t * testing . T ) {
result := android . GroupFixturePreparers (
prepareForJavaTest ,
PrepareForTestWithJavaSdkLibraryFiles ,
FixtureWithLastReleaseApis ( "foo" ) ,
) . RunTestWithBp ( t , `
java_sdk_library {
name : "foo" ,
srcs : [ "a.java" ] ,
api_only : true ,
public : {
enabled : true ,
} ,
}
java_library {
name : "bar" ,
srcs : [ "b.java" ] ,
libs : [ "foo" ] ,
}
` )
// The bar library should depend on the stubs jar.
barLibrary := result . ModuleForTests ( "bar" , "android_common" ) . Rule ( "javac" )
if expected , actual := ` ^-classpath .*:out/soong/[^:]*/turbine-combined/foo\.stubs\.jar$ ` , barLibrary . Args [ "classpath" ] ; ! regexp . MustCompile ( expected ) . MatchString ( actual ) {
t . Errorf ( "expected %q, found %#q" , expected , actual )
}
}
2021-09-21 16:25:12 +02:00
func TestJavaSdkLibrary_AccessOutputFiles ( t * testing . T ) {
2021-06-01 22:18:08 +02:00
android . GroupFixturePreparers (
prepareForJavaTest ,
PrepareForTestWithJavaSdkLibraryFiles ,
FixtureWithLastReleaseApis ( "foo" ) ,
) . RunTestWithBp ( t , `
2021-09-21 16:25:12 +02:00
java_sdk_library {
name : "foo" ,
srcs : [ "a.java" ] ,
api_packages : [ "foo" ] ,
annotations_enabled : true ,
public : {
enabled : true ,
} ,
}
java_library {
name : "bar" ,
srcs : [ "b.java" , ":foo{.public.stubs.source}" ] ,
java_resources : [ ":foo{.public.annotations.zip}" ] ,
}
` )
}
func TestJavaSdkLibrary_AccessOutputFiles_NoAnnotations ( t * testing . T ) {
android . GroupFixturePreparers (
prepareForJavaTest ,
PrepareForTestWithJavaSdkLibraryFiles ,
FixtureWithLastReleaseApis ( "foo" ) ,
) .
ExtendWithErrorHandler ( android . FixtureExpectsAtLeastOneErrorMatchingPattern ( ` module "bar" variant "android_common": path dependency ":foo { .public.annotations.zip}": annotations.zip not available for api scope public ` ) ) .
RunTestWithBp ( t , `
2021-06-01 22:18:08 +02:00
java_sdk_library {
name : "foo" ,
srcs : [ "a.java" ] ,
api_packages : [ "foo" ] ,
public : {
enabled : true ,
} ,
}
java_library {
name : "bar" ,
srcs : [ "b.java" , ":foo{.public.stubs.source}" ] ,
2021-09-21 16:25:12 +02:00
java_resources : [ ":foo{.public.annotations.zip}" ] ,
2021-06-01 22:18:08 +02:00
}
` )
}
func TestJavaSdkLibrary_AccessOutputFiles_MissingScope ( t * testing . T ) {
android . GroupFixturePreparers (
prepareForJavaTest ,
PrepareForTestWithJavaSdkLibraryFiles ,
FixtureWithLastReleaseApis ( "foo" ) ,
) .
ExtendWithErrorHandler ( android . FixtureExpectsAtLeastOneErrorMatchingPattern ( ` "foo" does not provide api scope system ` ) ) .
RunTestWithBp ( t , `
java_sdk_library {
name : "foo" ,
srcs : [ "a.java" ] ,
api_packages : [ "foo" ] ,
public : {
enabled : true ,
} ,
}
java_library {
name : "bar" ,
srcs : [ "b.java" , ":foo{.system.stubs.source}" ] ,
}
` )
}
func TestJavaSdkLibrary_Deps ( t * testing . T ) {
result := android . GroupFixturePreparers (
prepareForJavaTest ,
PrepareForTestWithJavaSdkLibraryFiles ,
FixtureWithLastReleaseApis ( "sdklib" ) ,
) . RunTestWithBp ( t , `
java_sdk_library {
name : "sdklib" ,
srcs : [ "a.java" ] ,
sdk_version : "none" ,
system_modules : "none" ,
public : {
enabled : true ,
} ,
}
` )
CheckModuleDependencies ( t , result . TestContext , "sdklib" , "android_common" , [ ] string {
` dex2oatd ` ,
` sdklib.impl ` ,
` sdklib.stubs ` ,
` sdklib.stubs.source ` ,
` sdklib.xml ` ,
} )
}
func TestJavaSdkLibraryImport_AccessOutputFiles ( t * testing . T ) {
prepareForJavaTest . RunTestWithBp ( t , `
java_sdk_library_import {
name : "foo" ,
public : {
jars : [ "a.jar" ] ,
stub_srcs : [ "a.java" ] ,
current_api : "api/current.txt" ,
removed_api : "api/removed.txt" ,
2021-09-21 16:25:12 +02:00
annotations : "x/annotations.zip" ,
2021-06-01 22:18:08 +02:00
} ,
}
java_library {
name : "bar" ,
srcs : [ ":foo{.public.stubs.source}" ] ,
java_resources : [
":foo{.public.api.txt}" ,
":foo{.public.removed-api.txt}" ,
2021-09-21 16:25:12 +02:00
":foo{.public.annotations.zip}" ,
2021-06-01 22:18:08 +02:00
] ,
}
` )
}
func TestJavaSdkLibraryImport_AccessOutputFiles_Invalid ( t * testing . T ) {
bp := `
java_sdk_library_import {
name : "foo" ,
public : {
jars : [ "a.jar" ] ,
} ,
}
`
t . Run ( "stubs.source" , func ( t * testing . T ) {
prepareForJavaTest .
ExtendWithErrorHandler ( android . FixtureExpectsAtLeastOneErrorMatchingPattern ( ` stubs.source not available for api scope public ` ) ) .
RunTestWithBp ( t , bp + `
java_library {
name : "bar" ,
srcs : [ ":foo{.public.stubs.source}" ] ,
java_resources : [
":foo{.public.api.txt}" ,
":foo{.public.removed-api.txt}" ,
] ,
}
` )
} )
t . Run ( "api.txt" , func ( t * testing . T ) {
prepareForJavaTest .
ExtendWithErrorHandler ( android . FixtureExpectsAtLeastOneErrorMatchingPattern ( ` api.txt not available for api scope public ` ) ) .
RunTestWithBp ( t , bp + `
java_library {
name : "bar" ,
srcs : [ "a.java" ] ,
java_resources : [
":foo{.public.api.txt}" ,
] ,
}
` )
} )
t . Run ( "removed-api.txt" , func ( t * testing . T ) {
prepareForJavaTest .
ExtendWithErrorHandler ( android . FixtureExpectsAtLeastOneErrorMatchingPattern ( ` removed-api.txt not available for api scope public ` ) ) .
RunTestWithBp ( t , bp + `
java_library {
name : "bar" ,
srcs : [ "a.java" ] ,
java_resources : [
":foo{.public.removed-api.txt}" ,
] ,
}
` )
} )
}
func TestJavaSdkLibrary_InvalidScopes ( t * testing . T ) {
prepareForJavaTest .
ExtendWithErrorHandler ( android . FixtureExpectsAtLeastOneErrorMatchingPattern ( ` module "foo": enabled api scope "system" depends on disabled scope "public" ` ) ) .
RunTestWithBp ( t , `
java_sdk_library {
name : "foo" ,
srcs : [ "a.java" , "b.java" ] ,
api_packages : [ "foo" ] ,
// Explicitly disable public to test the check that ensures the set of enabled
// scopes is consistent.
public : {
enabled : false ,
} ,
system : {
enabled : true ,
} ,
}
` )
}
func TestJavaSdkLibrary_SdkVersion_ForScope ( t * testing . T ) {
android . GroupFixturePreparers (
prepareForJavaTest ,
PrepareForTestWithJavaSdkLibraryFiles ,
FixtureWithLastReleaseApis ( "foo" ) ,
) . RunTestWithBp ( t , `
java_sdk_library {
name : "foo" ,
srcs : [ "a.java" , "b.java" ] ,
api_packages : [ "foo" ] ,
system : {
enabled : true ,
sdk_version : "module_current" ,
} ,
}
` )
}
func TestJavaSdkLibrary_ModuleLib ( t * testing . T ) {
android . GroupFixturePreparers (
prepareForJavaTest ,
PrepareForTestWithJavaSdkLibraryFiles ,
FixtureWithLastReleaseApis ( "foo" ) ,
) . RunTestWithBp ( t , `
java_sdk_library {
name : "foo" ,
srcs : [ "a.java" , "b.java" ] ,
api_packages : [ "foo" ] ,
system : {
enabled : true ,
} ,
module_lib : {
enabled : true ,
} ,
}
` )
}
func TestJavaSdkLibrary_SystemServer ( t * testing . T ) {
android . GroupFixturePreparers (
prepareForJavaTest ,
PrepareForTestWithJavaSdkLibraryFiles ,
FixtureWithLastReleaseApis ( "foo" ) ,
) . RunTestWithBp ( t , `
java_sdk_library {
name : "foo" ,
srcs : [ "a.java" , "b.java" ] ,
api_packages : [ "foo" ] ,
system : {
enabled : true ,
} ,
system_server : {
enabled : true ,
} ,
}
` )
}
func TestJavaSdkLibrary_MissingScope ( t * testing . T ) {
prepareForJavaTest .
ExtendWithErrorHandler ( android . FixtureExpectsAtLeastOneErrorMatchingPattern ( ` requires api scope module-lib from foo but it only has \[\] available ` ) ) .
RunTestWithBp ( t , `
java_sdk_library {
name : "foo" ,
srcs : [ "a.java" ] ,
public : {
enabled : false ,
} ,
}
java_library {
name : "baz" ,
srcs : [ "a.java" ] ,
libs : [ "foo" ] ,
sdk_version : "module_current" ,
}
` )
}
func TestJavaSdkLibrary_FallbackScope ( t * testing . T ) {
android . GroupFixturePreparers (
prepareForJavaTest ,
PrepareForTestWithJavaSdkLibraryFiles ,
FixtureWithLastReleaseApis ( "foo" ) ,
) . RunTestWithBp ( t , `
java_sdk_library {
name : "foo" ,
srcs : [ "a.java" ] ,
system : {
enabled : true ,
} ,
}
java_library {
name : "baz" ,
srcs : [ "a.java" ] ,
libs : [ "foo" ] ,
// foo does not have module-lib scope so it should fallback to system
sdk_version : "module_current" ,
}
` )
}
func TestJavaSdkLibrary_DefaultToStubs ( t * testing . T ) {
result := android . GroupFixturePreparers (
prepareForJavaTest ,
PrepareForTestWithJavaSdkLibraryFiles ,
FixtureWithLastReleaseApis ( "foo" ) ,
) . RunTestWithBp ( t , `
java_sdk_library {
name : "foo" ,
srcs : [ "a.java" ] ,
system : {
enabled : true ,
} ,
default_to_stubs : true ,
}
java_library {
name : "baz" ,
srcs : [ "a.java" ] ,
libs : [ "foo" ] ,
// does not have sdk_version set, should fallback to module,
// which will then fallback to system because the module scope
// is not enabled.
}
` )
// The baz library should depend on the system stubs jar.
bazLibrary := result . ModuleForTests ( "baz" , "android_common" ) . Rule ( "javac" )
if expected , actual := ` ^-classpath .*:out/soong/[^:]*/turbine-combined/foo\.stubs.system\.jar$ ` , bazLibrary . Args [ "classpath" ] ; ! regexp . MustCompile ( expected ) . MatchString ( actual ) {
t . Errorf ( "expected %q, found %#q" , expected , actual )
}
}
func TestJavaSdkLibraryImport ( t * testing . T ) {
result := prepareForJavaTest . RunTestWithBp ( t , `
java_library {
name : "foo" ,
srcs : [ "a.java" ] ,
libs : [ "sdklib" ] ,
sdk_version : "current" ,
}
java_library {
name : "foo.system" ,
srcs : [ "a.java" ] ,
libs : [ "sdklib" ] ,
sdk_version : "system_current" ,
}
java_library {
name : "foo.test" ,
srcs : [ "a.java" ] ,
libs : [ "sdklib" ] ,
sdk_version : "test_current" ,
}
java_sdk_library_import {
name : "sdklib" ,
public : {
jars : [ "a.jar" ] ,
} ,
system : {
jars : [ "b.jar" ] ,
} ,
test : {
jars : [ "c.jar" ] ,
stub_srcs : [ "c.java" ] ,
} ,
}
` )
for _ , scope := range [ ] string { "" , ".system" , ".test" } {
fooModule := result . ModuleForTests ( "foo" + scope , "android_common" )
javac := fooModule . Rule ( "javac" )
sdklibStubsJar := result . ModuleForTests ( "sdklib.stubs" + scope , "android_common" ) . Rule ( "combineJar" ) . Output
android . AssertStringDoesContain ( t , "foo classpath" , javac . Args [ "classpath" ] , sdklibStubsJar . String ( ) )
}
CheckModuleDependencies ( t , result . TestContext , "sdklib" , "android_common" , [ ] string {
2021-09-09 10:12:46 +02:00
` dex2oatd ` ,
2021-06-01 22:18:08 +02:00
` prebuilt_sdklib.stubs ` ,
` prebuilt_sdklib.stubs.source.test ` ,
` prebuilt_sdklib.stubs.system ` ,
` prebuilt_sdklib.stubs.test ` ,
} )
}
func TestJavaSdkLibraryImport_WithSource ( t * testing . T ) {
result := android . GroupFixturePreparers (
prepareForJavaTest ,
PrepareForTestWithJavaSdkLibraryFiles ,
FixtureWithLastReleaseApis ( "sdklib" ) ,
) . RunTestWithBp ( t , `
java_sdk_library {
name : "sdklib" ,
srcs : [ "a.java" ] ,
sdk_version : "none" ,
system_modules : "none" ,
public : {
enabled : true ,
} ,
}
java_sdk_library_import {
name : "sdklib" ,
public : {
jars : [ "a.jar" ] ,
} ,
}
` )
CheckModuleDependencies ( t , result . TestContext , "sdklib" , "android_common" , [ ] string {
` dex2oatd ` ,
` prebuilt_sdklib ` ,
` sdklib.impl ` ,
` sdklib.stubs ` ,
` sdklib.stubs.source ` ,
` sdklib.xml ` ,
} )
CheckModuleDependencies ( t , result . TestContext , "prebuilt_sdklib" , "android_common" , [ ] string {
` prebuilt_sdklib.stubs ` ,
` sdklib.impl ` ,
// This should be prebuilt_sdklib.stubs but is set to sdklib.stubs because the
// dependency is added after prebuilts may have been renamed and so has to use
// the renamed name.
` sdklib.xml ` ,
} )
}
func TestJavaSdkLibraryImport_Preferred ( t * testing . T ) {
result := android . GroupFixturePreparers (
prepareForJavaTest ,
PrepareForTestWithJavaSdkLibraryFiles ,
FixtureWithLastReleaseApis ( "sdklib" ) ,
) . RunTestWithBp ( t , `
java_sdk_library {
name : "sdklib" ,
srcs : [ "a.java" ] ,
sdk_version : "none" ,
system_modules : "none" ,
public : {
enabled : true ,
} ,
}
java_sdk_library_import {
name : "sdklib" ,
prefer : true ,
public : {
jars : [ "a.jar" ] ,
} ,
}
` )
CheckModuleDependencies ( t , result . TestContext , "sdklib" , "android_common" , [ ] string {
` prebuilt_sdklib ` ,
` sdklib.impl ` ,
` sdklib.stubs ` ,
` sdklib.stubs.source ` ,
` sdklib.xml ` ,
} )
CheckModuleDependencies ( t , result . TestContext , "prebuilt_sdklib" , "android_common" , [ ] string {
2021-09-09 10:12:46 +02:00
` dex2oatd ` ,
2021-06-01 22:18:08 +02:00
` prebuilt_sdklib.stubs ` ,
` sdklib.impl ` ,
` sdklib.xml ` ,
} )
}
func TestJavaSdkLibraryEnforce ( t * testing . T ) {
partitionToBpOption := func ( partition string ) string {
switch partition {
case "system" :
return ""
case "vendor" :
return "soc_specific: true,"
case "product" :
return "product_specific: true,"
default :
panic ( "Invalid partition group name: " + partition )
}
}
type testConfigInfo struct {
libraryType string
fromPartition string
toPartition string
enforceVendorInterface bool
enforceProductInterface bool
enforceJavaSdkLibraryCheck bool
allowList [ ] string
}
createPreparer := func ( info testConfigInfo ) android . FixturePreparer {
bpFileTemplate := `
java_library {
name : "foo" ,
srcs : [ "foo.java" ] ,
libs : [ "bar" ] ,
sdk_version : "current" ,
% s
}
% s {
name : "bar" ,
srcs : [ "bar.java" ] ,
sdk_version : "current" ,
% s
}
`
bpFile := fmt . Sprintf ( bpFileTemplate ,
partitionToBpOption ( info . fromPartition ) ,
info . libraryType ,
partitionToBpOption ( info . toPartition ) )
return android . GroupFixturePreparers (
PrepareForTestWithJavaSdkLibraryFiles ,
FixtureWithLastReleaseApis ( "bar" ) ,
android . FixtureWithRootAndroidBp ( bpFile ) ,
android . FixtureModifyProductVariables ( func ( variables android . FixtureProductVariables ) {
variables . EnforceProductPartitionInterface = proptools . BoolPtr ( info . enforceProductInterface )
if info . enforceVendorInterface {
variables . DeviceVndkVersion = proptools . StringPtr ( "current" )
}
variables . EnforceInterPartitionJavaSdkLibrary = proptools . BoolPtr ( info . enforceJavaSdkLibraryCheck )
variables . InterPartitionJavaLibraryAllowList = info . allowList
} ) ,
)
}
runTest := func ( t * testing . T , info testConfigInfo , expectedErrorPattern string ) {
t . Run ( fmt . Sprintf ( "%v" , info ) , func ( t * testing . T ) {
errorHandler := android . FixtureExpectsNoErrors
if expectedErrorPattern != "" {
errorHandler = android . FixtureExpectsAtLeastOneErrorMatchingPattern ( expectedErrorPattern )
}
android . GroupFixturePreparers (
prepareForJavaTest ,
createPreparer ( info ) ,
) .
ExtendWithErrorHandler ( errorHandler ) .
RunTest ( t )
} )
}
errorMessage := "is not allowed across the partitions"
runTest ( t , testConfigInfo {
libraryType : "java_library" ,
fromPartition : "product" ,
toPartition : "system" ,
enforceVendorInterface : true ,
enforceProductInterface : true ,
enforceJavaSdkLibraryCheck : false ,
} , "" )
runTest ( t , testConfigInfo {
libraryType : "java_library" ,
fromPartition : "product" ,
toPartition : "system" ,
enforceVendorInterface : true ,
enforceProductInterface : false ,
enforceJavaSdkLibraryCheck : true ,
} , "" )
runTest ( t , testConfigInfo {
libraryType : "java_library" ,
fromPartition : "product" ,
toPartition : "system" ,
enforceVendorInterface : true ,
enforceProductInterface : true ,
enforceJavaSdkLibraryCheck : true ,
} , errorMessage )
runTest ( t , testConfigInfo {
libraryType : "java_library" ,
fromPartition : "vendor" ,
toPartition : "system" ,
enforceVendorInterface : true ,
enforceProductInterface : true ,
enforceJavaSdkLibraryCheck : true ,
} , errorMessage )
runTest ( t , testConfigInfo {
libraryType : "java_library" ,
fromPartition : "vendor" ,
toPartition : "system" ,
enforceVendorInterface : true ,
enforceProductInterface : true ,
enforceJavaSdkLibraryCheck : true ,
allowList : [ ] string { "bar" } ,
} , "" )
runTest ( t , testConfigInfo {
libraryType : "java_library" ,
fromPartition : "vendor" ,
toPartition : "product" ,
enforceVendorInterface : true ,
enforceProductInterface : true ,
enforceJavaSdkLibraryCheck : true ,
} , errorMessage )
runTest ( t , testConfigInfo {
libraryType : "java_sdk_library" ,
fromPartition : "product" ,
toPartition : "system" ,
enforceVendorInterface : true ,
enforceProductInterface : true ,
enforceJavaSdkLibraryCheck : true ,
} , "" )
runTest ( t , testConfigInfo {
libraryType : "java_sdk_library" ,
fromPartition : "vendor" ,
toPartition : "system" ,
enforceVendorInterface : true ,
enforceProductInterface : true ,
enforceJavaSdkLibraryCheck : true ,
} , "" )
runTest ( t , testConfigInfo {
libraryType : "java_sdk_library" ,
fromPartition : "vendor" ,
toPartition : "product" ,
enforceVendorInterface : true ,
enforceProductInterface : true ,
enforceJavaSdkLibraryCheck : true ,
} , "" )
}
2021-06-01 22:39:09 +02:00
func TestJavaSdkLibraryDist ( t * testing . T ) {
result := android . GroupFixturePreparers (
PrepareForTestWithJavaBuildComponents ,
PrepareForTestWithJavaDefaultModules ,
PrepareForTestWithJavaSdkLibraryFiles ,
2021-06-02 22:02:51 +02:00
FixtureWithLastReleaseApis (
"sdklib_no_group" ,
"sdklib_group_foo" ,
"sdklib_owner_foo" ,
"foo" ) ,
2021-06-01 22:39:09 +02:00
) . RunTestWithBp ( t , `
java_sdk_library {
2021-06-01 23:07:56 +02:00
name : "sdklib_no_group" ,
2021-06-01 22:39:09 +02:00
srcs : [ "foo.java" ] ,
}
2021-06-01 22:13:40 +02:00
java_sdk_library {
name : "sdklib_group_foo" ,
srcs : [ "foo.java" ] ,
dist_group : "foo" ,
}
2021-06-01 22:39:09 +02:00
java_sdk_library {
name : "sdklib_owner_foo" ,
srcs : [ "foo.java" ] ,
owner : "foo" ,
}
java_sdk_library {
name : "sdklib_stem_foo" ,
srcs : [ "foo.java" ] ,
dist_stem : "foo" ,
}
` )
type testCase struct {
module string
distDir string
distStem string
}
testCases := [ ] testCase {
{
2021-06-01 23:07:56 +02:00
module : "sdklib_no_group" ,
2021-06-01 23:05:09 +02:00
distDir : "apistubs/unknown/public" ,
2021-06-01 23:07:56 +02:00
distStem : "sdklib_no_group.jar" ,
2021-06-01 22:39:09 +02:00
} ,
2021-06-01 22:13:40 +02:00
{
module : "sdklib_group_foo" ,
distDir : "apistubs/foo/public" ,
distStem : "sdklib_group_foo.jar" ,
} ,
2021-06-01 22:39:09 +02:00
{
2021-06-01 23:07:56 +02:00
// Owner doesn't affect distDir after b/186723288.
2021-06-01 22:39:09 +02:00
module : "sdklib_owner_foo" ,
2021-06-01 23:07:56 +02:00
distDir : "apistubs/unknown/public" ,
2021-06-01 22:39:09 +02:00
distStem : "sdklib_owner_foo.jar" ,
} ,
{
module : "sdklib_stem_foo" ,
2021-06-01 23:05:09 +02:00
distDir : "apistubs/unknown/public" ,
2021-06-01 22:39:09 +02:00
distStem : "foo.jar" ,
} ,
}
for _ , tt := range testCases {
t . Run ( tt . module , func ( t * testing . T ) {
m := result . ModuleForTests ( tt . module + ".stubs" , "android_common" ) . Module ( ) . ( * Library )
dists := m . Dists ( )
if len ( dists ) != 1 {
t . Fatalf ( "expected exactly 1 dist entry, got %d" , len ( dists ) )
}
if g , w := String ( dists [ 0 ] . Dir ) , tt . distDir ; g != w {
t . Errorf ( "expected dist dir %q, got %q" , w , g )
}
if g , w := String ( dists [ 0 ] . Dest ) , tt . distStem ; g != w {
t . Errorf ( "expected dist stem %q, got %q" , w , g )
}
} )
}
}