commit 1854cf1ef26c961a48d764502ac02ec6ebd7df4c
author Steven Moreland <smoreland@google.com> Mon Apr 26 16:29:43 2021 +0000
committer Steven Moreland <smoreland@google.com> Mon Apr 26 16:32:34 2021 +0000
tree 67c46693fc2107563ddb66a9ab533712cfd18a19
parent 33257c127ac8f70f8f3ec5eb8f4f4470c0b162d9

hal_implementation_test: clear 'AOSP' error

Cuttlefish should only have AOSP HAL implementations. In the internal
TreeTop dashboard, the test failure is shown separately from the test
name. So, introducing this extra message to clarify the test failure
when it is by itself.

Bug: N/A
Test: build only
Change-Id: I506b02c16ca1211b4e26d3f461754d44827ce6dd

tests/hal/hal_implementation_test.cpp

diff --git a/tests/hal/hal_implementation_test.cpp b/tests/hal/hal_implementation_test.cpp
index 471aebb..3674d15 100644
--- a/tests/hal/hal_implementation_test.cpp
+++ b/tests/hal/hal_implementation_test.cpp
@@ -190,7 +190,9 @@
 
 TEST(Hal, AllHidlInterfacesAreInAosp) {
     for (const FQName& name : allHidlManifestInterfaces()) {
-        EXPECT_TRUE(isAospHidlInterface(name)) << name.string();
+      EXPECT_TRUE(isAospHidlInterface(name))
+          << "This device should only have AOSP interfaces, not: "
+          << name.string();
     }
 }
 
@@ -248,7 +250,8 @@
 
 TEST(Hal, AllAidlInterfacesAreInAosp) {
     for (const std::string& name : allAidlManifestInterfaces()) {
-        EXPECT_TRUE(isAospAidlInterface(name)) << name;
+      EXPECT_TRUE(isAospAidlInterface(name))
+          << "This device should only have AOSP interfaces, not: " << name;
     }
 }