Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit f5344f6c authored by Michael Groover's avatar Michael Groover Committed by Paul Duffin
Browse files

Update PMService tests to use SDK version 10000 

A recent update to the lint checks now enforces SDK versions be
specified as a supported value (currently released SDK versions
or 10000 for a development release). The PackageManagerService
tests use a minExtensionVersion set to 1000 which causes the
lint check to fail due to 1000 not being a supported SDK version.
This commit updates these values to 10000 to be a supported SDK
version recognized by the lint checks.

Bug: 273389716
Bug: 280595738
Test: atest PackageParserLegacyCoreTest
Merged-In: I05081ef96c44dbf0cbedf585a2e13897712d080c
Change-Id: I05081ef96c44dbf0cbedf585a2e13897712d080c
(cherry picked from commit 714d1f1b)
parent 1ee97e0e

services/tests/PackageManagerServiceTests/apks/install_uses_sdk/Android.bp

+4 −4
Original line number Original line Diff line number Diff line
@@ -32,9 +32,9 @@ android_test_helper_app { 
}

}





android_test_helper_app {

android_test_helper_app {

    name: "FrameworksServicesTests_install_uses_sdk_r1000",

    name: "FrameworksServicesTests_install_uses_sdk_r10000",

    defaults: ["FrameworksServicesTests_apks_defaults"],

    defaults: ["FrameworksServicesTests_apks_defaults"],

    manifest: "AndroidManifest-r1000.xml",

    manifest: "AndroidManifest-r10000.xml",

}

}





android_test_helper_app {

android_test_helper_app {
@@ -44,9 +44,9 @@ android_test_helper_app { 
}

}





android_test_helper_app {

android_test_helper_app {

    name: "FrameworksServicesTests_install_uses_sdk_r0_s1000",

    name: "FrameworksServicesTests_install_uses_sdk_r0_s10000",

    defaults: ["FrameworksServicesTests_apks_defaults"],

    defaults: ["FrameworksServicesTests_apks_defaults"],

    manifest: "AndroidManifest-r0-s1000.xml",

    manifest: "AndroidManifest-r0-s10000.xml",

}

}





android_test_helper_app {

android_test_helper_app {

services/tests/PackageManagerServiceTests/apks/install_uses_sdk/AndroidManifest-r0-s1000.xml→services/tests/PackageManagerServiceTests/apks/install_uses_sdk/AndroidManifest-r0-s10000.xml

+1 −1
Original line number Original line Diff line number Diff line
@@ -19,7 +19,7 @@ 
    <uses-sdk android:minSdkVersion="4" android:targetSdkVersion="29">

    <uses-sdk android:minSdkVersion="4" android:targetSdkVersion="29">

        <!-- This fails because 31 is not version 5 -->

        <!-- This fails because 31 is not version 5 -->

        <extension-sdk android:sdkVersion="30" android:minExtensionVersion="0" />

        <extension-sdk android:sdkVersion="30" android:minExtensionVersion="0" />

        <extension-sdk android:sdkVersion="31" android:minExtensionVersion="1000" />

        <extension-sdk android:sdkVersion="31" android:minExtensionVersion="10000" />

    </uses-sdk>

    </uses-sdk>





    <application>

    <application>

services/tests/PackageManagerServiceTests/apks/install_uses_sdk/AndroidManifest-r1000.xml→services/tests/PackageManagerServiceTests/apks/install_uses_sdk/AndroidManifest-r10000.xml

+1 −1
Original line number Original line Diff line number Diff line
@@ -18,7 +18,7 @@ 




    <uses-sdk android:minSdkVersion="4" android:targetSdkVersion="29">

    <uses-sdk android:minSdkVersion="4" android:targetSdkVersion="29">

        <!-- This will fail to install, because minExtensionVersion is not met -->

        <!-- This will fail to install, because minExtensionVersion is not met -->

        <extension-sdk android:sdkVersion="30" android:minExtensionVersion="1000" />

        <extension-sdk android:sdkVersion="30" android:minExtensionVersion="10000" />

    </uses-sdk>

    </uses-sdk>





    <application>

    <application>

services/tests/PackageManagerServiceTests/server/Android.bp

+2 −2
Original line number Original line Diff line number Diff line
@@ -127,10 +127,10 @@ java_genrule { 
        ":FrameworksServicesTests_install_uses_sdk_q0",

        ":FrameworksServicesTests_install_uses_sdk_q0",

        ":FrameworksServicesTests_install_uses_sdk_q0_r0",

        ":FrameworksServicesTests_install_uses_sdk_q0_r0",

        ":FrameworksServicesTests_install_uses_sdk_r0",

        ":FrameworksServicesTests_install_uses_sdk_r0",

        ":FrameworksServicesTests_install_uses_sdk_r1000",

        ":FrameworksServicesTests_install_uses_sdk_r10000",

        ":FrameworksServicesTests_install_uses_sdk_r_none",

        ":FrameworksServicesTests_install_uses_sdk_r_none",

        ":FrameworksServicesTests_install_uses_sdk_r0_s0",

        ":FrameworksServicesTests_install_uses_sdk_r0_s0",

        ":FrameworksServicesTests_install_uses_sdk_r0_s1000",

        ":FrameworksServicesTests_install_uses_sdk_r0_s10000",

        ":FrameworksServicesTests_keyset_permdef_sa_unone",

        ":FrameworksServicesTests_keyset_permdef_sa_unone",

        ":FrameworksServicesTests_keyset_permuse_sa_ua_ub",

        ":FrameworksServicesTests_keyset_permuse_sa_ua_ub",

        ":FrameworksServicesTests_keyset_permuse_sb_ua_ub",

        ":FrameworksServicesTests_keyset_permuse_sb_ua_ub",

services/tests/PackageManagerServiceTests/server/src/com/android/server/parsing/PackageParserLegacyCoreTest.java

+2 −2
Original line number Original line Diff line number Diff line
@@ -630,10 +630,10 @@ public class PackageParserLegacyCoreTest { 
        assertEquals(0, minExtVers.get(31, -1));

        assertEquals(0, minExtVers.get(31, -1));





        Map<Pair<String, Integer>, Integer> appToError = new HashMap<>();

        Map<Pair<String, Integer>, Integer> appToError = new HashMap<>();

        appToError.put(Pair.create("install_uses_sdk.apk_r1000", R.raw.install_uses_sdk_r1000),

        appToError.put(Pair.create("install_uses_sdk.apk_r10000", R.raw.install_uses_sdk_r10000),

                       PackageManager.INSTALL_FAILED_OLDER_SDK);

                       PackageManager.INSTALL_FAILED_OLDER_SDK);

        appToError.put(

        appToError.put(

                Pair.create("install_uses_sdk.apk_r0_s1000", R.raw.install_uses_sdk_r0_s1000),

                Pair.create("install_uses_sdk.apk_r0_s10000", R.raw.install_uses_sdk_r0_s10000),

                PackageManager.INSTALL_FAILED_OLDER_SDK);

                PackageManager.INSTALL_FAILED_OLDER_SDK);





        appToError.put(Pair.create("install_uses_sdk.apk_q0", R.raw.install_uses_sdk_q0),

        appToError.put(Pair.create("install_uses_sdk.apk_q0", R.raw.install_uses_sdk_q0),