| commit | 8ac06fbf1f1a5a6666621cb4b4abeafdc534b9d7 | [log] [tgz] |
|---|---|---|
| author | Benjamin Schwartz <bsschwar@google.com> | Thu Apr 08 04:45:03 2021 +0000 |
| committer | Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> | Thu Apr 08 04:45:03 2021 +0000 |
| tree | 0c14ad731334df9e5cc0f124b522b17cdc04f8eb | |
| parent | 5c3f73cdaf1b38304a0b5f064017b43a0dfad650 [diff] | |
| parent | 43374307a3adc7e51ab9e775bb6bb85d57eb390b [diff] |
Merge "power/stats: Split channel name validation test" am: 43374307a3 Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/1668785 Change-Id: Id99d784fdd3ba2c2422475cf98683086e61f2ba9
diff --git a/power/stats/aidl/vts/VtsHalPowerStatsTargetTest.cpp b/power/stats/aidl/vts/VtsHalPowerStatsTargetTest.cpp index db2cfd8..c7ba96c 100644 --- a/power/stats/aidl/vts/VtsHalPowerStatsTargetTest.cpp +++ b/power/stats/aidl/vts/VtsHalPowerStatsTargetTest.cpp
@@ -239,12 +239,22 @@ 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); } }