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

Commit 0e6d63fe authored by Remi NGUYEN VAN's avatar Remi NGUYEN VAN
Browse files

Add NetworkStackTests / NetworkStackNextTests to mainline-presubmit 

The tests are already in presubmit, but the general presubmit target is
not used in mainline-prod because there is no usable device image there.
Add them to mainline-presubmit so mainline-prod is also covered. The
test targets are unit tests that run in the order of 5min.

This involves moving the unit tests from device-tests to general-tests,
which is fine as they do not use device-specific signing keys.

Change-Id: I034a82b3c58859ee2dc364509549f31fb5c670b7
Test: only treehugger can validate the config change
parent e0ddb33c

TEST_MAPPING

+11 −0
Original line number Diff line number Diff line
@@ -14,5 +14,16 @@ 
    {

      "name": "NetworkStackIntegrationTests"

    }

  ],

  "mainline-presubmit": [

    // These are unit tests only, so they don't actually require any modules to be installed.

    // We must specify at least one module here or the tests won't run. Use the same set as CTS

    // so in theory the infra would not need to reinstall/reboot devices to run both.

    {

      "name": "NetworkStackTests[CaptivePortalLoginGoogle.apk+NetworkStackGoogle.apk+com.google.android.resolv.apex+com.google.android.tethering.apex]"

    },

    {

      "name": "NetworkStackNextTests[CaptivePortalLoginGoogle.apk+NetworkStackGoogle.apk+com.google.android.resolv.apex+com.google.android.tethering.apex]"

    }

  ]

}

tests/unit/Android.bp

+5 −2
Original line number Diff line number Diff line
@@ -47,9 +47,11 @@ java_defaults { 
android_test {

    name: "NetworkStackNextTests",

    srcs: [], // TODO: tests that only apply to the current, non-stable API can be added here

    test_suites: ["device-tests"],

    test_suites: ["general-tests"],

    test_mainline_modules: ["CaptivePortalLoginGoogle.apk+NetworkStackGoogle.apk+com.google.android.resolv.apex+com.google.android.tethering.apex"],

    defaults: ["NetworkStackTestsDefaults"],

    static_libs: ["NetworkStackApiCurrentLib"],

    compile_multilib: "both", // Workaround for b/147785146 for mainline-presubmit

}



// Library containing the unit tests. This is used by the coverage test target to pull in the
@@ -70,7 +72,8 @@ android_library { 
android_test {

    name: "NetworkStackTests",

    min_sdk_version: "29",

    test_suites: ["device-tests", "mts"],

    test_suites: ["general-tests", "mts"],

    test_mainline_modules: ["CaptivePortalLoginGoogle.apk+NetworkStackGoogle.apk+com.google.android.resolv.apex+com.google.android.tethering.apex"],

    defaults: ["NetworkStackTestsDefaults"],

    static_libs: ["NetworkStackApiStableLib"],

    compile_multilib: "both",