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

Commit b6285952 authored by Adam Bookatz's avatar Adam Bookatz
Browse files

TEST_MAPPING for BatteryStats 

Creates presubmit test mapping for BatteryStats-related
files.

These tests can be run manually via
atest FrameworksServicesTests:com.android.server.am.BatteryExternalStatsWorkerTest FrameworksServicesTests:com.android.server.am.MeasuredEnergySnapshotTest FrameworksServicesTests:com.android.server.am.BatteryStatsServiceTest FrameworksCoreTests:com.android.internal.os.BatteryStatsTests

Tests that currently fail are marked to be skipped;
they should be addressed by b/180015146.
This is accomplished by introducing a new annotation, @SkipPresubmit.

Test: treehugger presubmit
Bug: 180016384
Change-Id: If01beaa342468aeded660e5766a14391d3ee9bcc
parent 0088eefa

core/java/android/os/TEST_MAPPING

+17 −0
Original line number Diff line number Diff line
@@ -38,6 +38,23 @@ 
          "include-filter": "com.android.server.pm.parsing.PackageInfoUserFieldsTest"

        }

      ]

    },

    {

      "file_patterns": ["BatteryStats.java"],

      "name": "FrameworksCoreTests",

      "options": [

        { "include-filter": "com.android.internal.os.BatteryStatsTests" },

        { "exclude-annotation": "com.android.internal.os.SkipPresubmit" }

      ]

    },

    {

      "file_patterns": ["BatteryStats.java"],

      "name": "FrameworksServicesTests",

      "options": [

        { "include-filter": "com.android.server.am.BatteryStatsServiceTest" },

        { "include-filter": "com.android.server.am.MeasuredEnergySnapshotTest" },

        { "include-filter": "com.android.server.am.BatteryExternalStatsWorkerTest" }

      ]

    }

  ],

  "postsubmit": [

core/java/com/android/internal/os/TEST_MAPPING

+17 −0
Original line number Diff line number Diff line 
{

  "presubmit": [

    {

      "file_patterns": ["Battery[^/]*\\.java"],

      "name": "FrameworksCoreTests",

      "options": [

        { "include-filter": "com.android.internal.os.BatteryStatsTests" },

        { "exclude-annotation": "com.android.internal.os.SkipPresubmit" }

      ]

    },

    {

      "file_patterns": ["Battery[^/]*\\.java"],

      "name": "FrameworksServicesTests",

      "options": [

        { "include-filter": "com.android.server.am.BatteryStatsServiceTest" },

        { "include-filter": "com.android.server.am.MeasuredEnergySnapshotTest" },

        { "include-filter": "com.android.server.am.BatteryExternalStatsWorkerTest" }

      ]

    },

    {

      "name": "FrameworksCoreTests",

      "options": [

core/java/com/android/internal/power/TEST_MAPPING

0 → 100644
+19 −0
Original line number Diff line number Diff line 
{

  "presubmit": [

    {

      "name": "FrameworksCoreTests",

      "options": [

        { "include-filter": "com.android.internal.os.BatteryStatsTests" },

        { "exclude-annotation": "com.android.internal.os.SkipPresubmit" }

      ]

    },

    {

      "name": "FrameworksServicesTests",

      "options": [

        { "include-filter": "com.android.server.am.BatteryStatsServiceTest" },

        { "include-filter": "com.android.server.am.MeasuredEnergySnapshotTest" },

        { "include-filter": "com.android.server.am.BatteryExternalStatsWorkerTest" }

      ]

    }

  ]

}

core/tests/coretests/src/com/android/internal/os/BatteryStatsCpuTimesTest.java

+1 −0
Original line number Diff line number Diff line
@@ -77,6 +77,7 @@ import java.util.Arrays; 
 * bit FrameworksCoreTests:com.android.internal.os.BatteryStatsCpuTimesTest

 */

@SmallTest

@SkipPresubmit("b/180015146")

@RunWith(AndroidJUnit4.class)

public class BatteryStatsCpuTimesTest {

    @Mock

core/tests/coretests/src/com/android/internal/os/BatteryStatsImplTest.java

+2 −0
Original line number Diff line number Diff line
@@ -73,6 +73,7 @@ public class BatteryStatsImplTest { 
    }



    @Test

    @SkipPresubmit("b/180015146")

    public void testUpdateProcStateCpuTimes() {

        mBatteryStatsImpl.setOnBatteryInternal(true);

        mBatteryStatsImpl.updateTimeBasesLocked(false, Display.STATE_ON, 0, 0);
@@ -230,6 +231,7 @@ public class BatteryStatsImplTest { 
    }



    @Test

    @SkipPresubmit("b/180015146")

    public void testCopyFromAllUidsCpuTimes() {

        mBatteryStatsImpl.setOnBatteryInternal(false);

        mBatteryStatsImpl.updateTimeBasesLocked(false, Display.STATE_ON, 0, 0);