tequilaOS/platform_hardware_interfaces
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Merge "Effect: verify all AIDL IEffect versions are same as IFactory" into main am: 047484de60

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

Change-Id: I4ca7e560dbe9a60dfb8b9420493edf5628e8560d
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
Shunkai Yao 2024-02-02 20:39:08 +00:00 committed by Automerger Merge Worker
commit 4221f8783d
1 changed files with 19 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

19
audio/aidl/vts/VtsHalAudioEffectFactoryTargetTest.cpp
View file

@ -296,6 +296,25 @@ TEST_P(EffectFactoryTest, QueryProcess) {
[&](const auto& proc) { return processingSet.find(proc) != processingSet.end(); }));
}
// Make sure all effect instances have same HAL version number as IFactory.
TEST_P(EffectFactoryTest, VersionNumberForAllEffectsEqualsToIFactory) {
std::vector<Descriptor> descs;
EXPECT_IS_OK(mEffectFactory->queryEffects(std::nullopt, std::nullopt, std::nullopt, &descs));
EXPECT_NE(descs.size(), 0UL);
std::vector<std::shared_ptr<IEffect>> effects = createWithDescs(descs);
int factoryVersion = 0;
EXPECT_IS_OK(mEffectFactory->getInterfaceVersion(&factoryVersion));
for (const auto& effect : effects) {
int effectVersion = 0;
EXPECT_NE(nullptr, effect);
EXPECT_IS_OK(effect->getInterfaceVersion(&effectVersion));
EXPECT_EQ(factoryVersion, effectVersion);
}
ASSERT_NO_FATAL_FAILURE(destroyEffects(effects));
}
INSTANTIATE_TEST_SUITE_P(EffectFactoryTest, EffectFactoryTest,
testing::ValuesIn(android::getAidlHalInstanceNames(IFactory::descriptor)),
android::PrintInstanceNameToString);