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

Commit ea154f0d authored by Harish Mahendrakar's avatar Harish Mahendrakar
Browse files

Enable unit tests for some utils in postsubmit 

Enabled unit tests for avcUtils, ID3, Mpeg2Ts, OpusHeader and
ESDS in postsubmit.

These were marked as dynamic-presubmit and those are now
changed to postsubmit as these tests now use DynamicConfig.xml and
can download the required files without any extra arguments.

After some time of running these in postsubmit, these will be changed
to presubmit.

Also, updated Android.bp and AndroidTest.xml for a couple of tests
to add to the device-tests test suite and cleanup after the test.

Bug: 304383609
Test: atest AVCUtilsUnitTest ESDSTest HEVCUtilsUnitTest ID3Test \
Mpeg2tsUnitTest OpusHeaderTest

Change-Id: I8383072f47b936d78299f4a3953229c992a88b4f
parent 9a7da20a

media/module/esds/TEST_MAPPING

+1 −5
Original line number Diff line number Diff line 
// mappings for frameworks/av/media/module/esds

{

  // tests which require dynamic content

  // invoke with: atest -- --enable-module-dynamic-download=true

  // TODO(b/148094059): unit tests not allowed to download content

  "dynamic-presubmit": [

  "postsubmit": [

    { "name": "ESDSTest" }

  ]

}

media/module/esds/tests/Android.bp

+1 −0
Original line number Diff line number Diff line
@@ -25,6 +25,7 @@ package { 
cc_test {

    name: "ESDSTest",

    gtest: true,

    test_suites: ["device-tests"],



    srcs: [

        "ESDSTest.cpp",

media/module/foundation/TEST_MAPPING

+2 −5
Original line number Diff line number Diff line 
// mappings for frameworks/av/media/libstagefright/foundation

{

  // tests which require dynamic content

  // invoke with: atest -- --enable-module-dynamic-download=true

  // TODO(b/148094059): unit tests not allowed to download content

  "dynamic-presubmit": [

  "postsubmit": [

    { "name": "AVCUtilsUnitTest" },

    { "name": "OpusHeaderTest" }

  ],

media/module/foundation/tests/AVCUtils/Android.bp

+1 −0
Original line number Diff line number Diff line
@@ -28,6 +28,7 @@ package { 
cc_test {

    name: "AVCUtilsUnitTest",

    gtest: true,

    test_suites: ["device-tests"],



    srcs: [

        "AVCUtilsUnitTest.cpp",

media/module/foundation/tests/AVCUtils/AndroidTest.xml

+1 −1
Original line number Diff line number Diff line
@@ -16,7 +16,7 @@ 
<configuration description="Test module config for AVC Utils unit tests">

    <option name="test-suite-tag" value="AVCUtilsUnitTest" />

    <target_preparer class="com.android.tradefed.targetprep.PushFilePreparer">

        <option name="cleanup" value="false" />

        <option name="cleanup" value="true" />

        <option name="push" value="AVCUtilsUnitTest->/data/local/tmp/AVCUtilsUnitTest" />

    </target_preparer>