Merge "Add test to TestJavaStableSdkVersion for legacy core platform" am: 3769a27c5e
am: 45dd37c6e3
am: 90a5e41516
am: da8ef1d18a
am: 246e0d388c
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/1655611 Change-Id: Ibf571a6f6c29de12d810989d83931f3b660971d2
This commit is contained in:
@@ -2185,6 +2185,7 @@ func TestJavaStableSdkVersion(t *testing.T) {
|
||||
name string
|
||||
expectedError string
|
||||
bp string
|
||||
preparer android.FixturePreparer
|
||||
}{
|
||||
{
|
||||
name: "Non-updatable apex with non-stable dep",
|
||||
@@ -2255,6 +2256,30 @@ func TestJavaStableSdkVersion(t *testing.T) {
|
||||
}
|
||||
`,
|
||||
},
|
||||
{
|
||||
name: "Updatable apex with non-stable legacy core platform dep",
|
||||
expectedError: `\Qcannot depend on "myjar-uses-legacy": non stable SDK core_platform_current - uses legacy core platform\E`,
|
||||
bp: `
|
||||
apex {
|
||||
name: "myapex",
|
||||
java_libs: ["myjar-uses-legacy"],
|
||||
key: "myapex.key",
|
||||
updatable: true,
|
||||
}
|
||||
apex_key {
|
||||
name: "myapex.key",
|
||||
public_key: "testkey.avbpubkey",
|
||||
private_key: "testkey.pem",
|
||||
}
|
||||
java_library {
|
||||
name: "myjar-uses-legacy",
|
||||
srcs: ["foo/bar/MyClass.java"],
|
||||
sdk_version: "core_platform",
|
||||
apex_available: ["myapex"],
|
||||
}
|
||||
`,
|
||||
preparer: java.FixtureUseLegacyCorePlatformApi("myjar-uses-legacy"),
|
||||
},
|
||||
{
|
||||
name: "Updatable apex with non-stable transitive dep",
|
||||
// This is not actually detecting that the transitive dependency is unstable, rather it is
|
||||
@@ -2291,12 +2316,22 @@ func TestJavaStableSdkVersion(t *testing.T) {
|
||||
}
|
||||
|
||||
for _, test := range testCases {
|
||||
if test.name != "Updatable apex with non-stable legacy core platform dep" {
|
||||
continue
|
||||
}
|
||||
t.Run(test.name, func(t *testing.T) {
|
||||
if test.expectedError == "" {
|
||||
testApex(t, test.bp)
|
||||
} else {
|
||||
testApexError(t, test.expectedError, test.bp)
|
||||
errorHandler := android.FixtureExpectsNoErrors
|
||||
if test.expectedError != "" {
|
||||
errorHandler = android.FixtureExpectsAtLeastOneErrorMatchingPattern(test.expectedError)
|
||||
}
|
||||
android.GroupFixturePreparers(
|
||||
java.PrepareForTestWithJavaDefaultModules,
|
||||
PrepareForTestWithApexBuildComponents,
|
||||
prepareForTestWithMyapex,
|
||||
android.OptionalFixturePreparer(test.preparer),
|
||||
).
|
||||
ExtendWithErrorHandler(errorHandler).
|
||||
RunTestWithBp(t, test.bp)
|
||||
})
|
||||
}
|
||||
}
|
||||
|
Reference in New Issue
Block a user