| commit | 87905ec84c7918aa2c05ad9ca10c065073f60253 | [log] [tgz] |
|---|---|---|
| author | Benjamin Schwartz <bsschwar@google.com> | Thu Apr 08 05:07:41 2021 +0000 |
| committer | Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> | Thu Apr 08 05:07:41 2021 +0000 |
| tree | 25c44a09480a3bd8464b8f39277a48075dfabde8 | |
| parent | f33e183381ea1cb518019ea5eaa62258a97b99f6 [diff] | |
| parent | 8ac06fbf1f1a5a6666621cb4b4abeafdc534b9d7 [diff] |
Merge "power/stats: Split channel name validation test" am: 43374307a3 am: 8ac06fbf1f Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/1668785 Change-Id: Icf347c3517742e1c0bfb75a605f512cad6f59219
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); } }