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

Commit 8ac06fbf authored by Benjamin Schwartz's avatar Benjamin Schwartz Committed by Automerger Merge Worker
Browse files

Merge "power/stats: Split channel name validation test" am: 43374307

Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/1668785

Change-Id: Id99d784fdd3ba2c2422475cf98683086e61f2ba9
parents 5c3f73cd 43374307
Loading
Loading
Loading
Loading
+11 −1
Original line number Diff line number Diff line
@@ -239,12 +239,22 @@ TEST_P(PowerStatsAidl, TestGetEnergyMeterInfo) {
    ASSERT_OK(powerstats->getEnergyMeterInfo(&info));
}

// Each channel must have a valid name and subsystem
// Each channel must have a valid name
TEST_P(PowerStatsAidl, ValidateChannelNames) {
    std::vector<Channel> channels;
    ASSERT_OK(powerstats->getEnergyMeterInfo(&channels));

    for (auto channel : channels) {
        testNameValid(channel.name);
    }
}

// Each channel must have a valid subsystem
TEST_P(PowerStatsAidl, ValidateSubsystemNames) {
    std::vector<Channel> channels;
    ASSERT_OK(powerstats->getEnergyMeterInfo(&channels));

    for (auto channel : channels) {
        testNameValid(channel.subsystem);
    }
}