Fix workaround for ISoundDose in hal_implementation_test
Refactored the package of ISoundDose.aidl to match the aidl_interface
and declared it types only.
Test: VtsHalSoundDoseFactoryTargetTest, VtsHalAudioCoreTargetTest,
hal_implementation_test
Bug: 263388737
Change-Id: If34a1fc1ede8794050d28e18756859334c00b3d7
Merged-In: If34a1fc1ede8794050d28e18756859334c00b3d7
diff --git a/tests/hal/hal_implementation_test.cpp b/tests/hal/hal_implementation_test.cpp
index f8f2966..f8c66d0 100644
--- a/tests/hal/hal_implementation_test.cpp
+++ b/tests/hal/hal_implementation_test.cpp
@@ -136,6 +136,7 @@
// types-only packages, which never expect a default implementation
"android.hardware.audio.common.",
+ "android.hardware.audio.core.sounddose.",
"android.hardware.biometrics.common.",
"android.hardware.common.",
"android.hardware.common.fmq.",
@@ -441,17 +442,12 @@
}
if (!latestRegistered && !expectedVersions.rbegin()->second.knownMissing) {
- // TODO(b/263388737): avoid this exception - it's due to this
- // part of the interface being put in two aidl_interface and
- // it was merged while this test was broken.
- if (treePackage.name != "android.hardware.audio.core.sounddose") {
- ADD_FAILURE() << "The latest version ("
- << expectedVersions.rbegin()->first
- << ") of the module is not implemented: "
- << treePackage.name
- << " which declares the following types:\n "
- << base::Join(treePackage.types, "\n ");
- }
+ ADD_FAILURE() << "The latest version ("
+ << expectedVersions.rbegin()->first
+ << ") of the module is not implemented: "
+ << treePackage.name
+ << " which declares the following types:\n "
+ << base::Join(treePackage.types, "\n ");
}
for (const auto& [version, check] : expectedVersions) {