commit e586c5c80576706d4fe925fd6ab0d2f3b40f110f
author Steven Moreland <smoreland@google.com> Tue Mar 10 17:31:43 2020 -0700
committer Jeongik Cha <jeongik@google.com> Thu Apr 09 17:10:59 2020 +0900
tree eb8922be45a00013470ef418f130c4575b633a80
parent 3fe29a7da1d65a4f8cc4e527695a922d5c1e610d

hal_implementation_test: no filtering on mod name

Module name is arbitrary, so filtering on types instead.

Before this, we were accidentally not tracking AOSP interfaces w/
non-AOSP names, such as 'test-android.hardware.vibrator-ext'.

Fixes: 151105750
Test: N/A
Change-Id: I01eb33bdf94ae37457886db8437fa76fc16bdaf0
Merged-In: I01eb33bdf94ae37457886db8437fa76fc16bdaf0

tests/hal/hal_implementation_test.cpp

diff --git a/tests/hal/hal_implementation_test.cpp b/tests/hal/hal_implementation_test.cpp
index 6efadf0..58c1f4f 100644
--- a/tests/hal/hal_implementation_test.cpp
+++ b/tests/hal/hal_implementation_test.cpp
@@ -254,7 +254,7 @@
 
     for (const auto& iface : AidlInterfaceMetadata::all()) {
         ASSERT_FALSE(iface.types.empty()) << iface.name;  // sanity
-        if (!isAospAidlInterface(iface.name)) continue;
+        if (std::none_of(iface.types.begin(), iface.types.end(), isAospAidlInterface)) continue;
         if (iface.stability != "vintf") continue;
 
         bool hasRegistration = false;