diff --git a/contexthub/aidl/vts/VtsAidlHalContextHubTargetTest.cpp b/contexthub/aidl/vts/VtsAidlHalContextHubTargetTest.cpp index 2b91b97dad..4b0d60fe51 100644 --- a/contexthub/aidl/vts/VtsAidlHalContextHubTargetTest.cpp +++ b/contexthub/aidl/vts/VtsAidlHalContextHubTargetTest.cpp @@ -82,10 +82,28 @@ TEST_P(ContextHubAidl, TestGetHubs) { } } +class EmptyContextHubCallback : public android::hardware::contexthub::BnContextHubCallback { + public: + Status handleNanoappInfo(const std::vector& /* appInfo */) override { + return Status::ok(); + } + + Status handleContextHubMessage(const ContextHubMessage& /* msg */, + const std::vector& /* msgContentPerms */) override { + return Status::ok(); + } + + Status handleContextHubAsyncEvent(AsyncEventType /* evt */) override { return Status::ok(); } + + Status handleTransactionResult(int32_t /* transactionId */, bool /* success */) override { + return Status::ok(); + } +}; + TEST_P(ContextHubAidl, TestRegisterCallback) { bool success; - ASSERT_TRUE(contextHub->registerCallback(getHubId(), new IContextHubCallbackDefault(), &success) - .isOk()); + sp cb = sp::make(); + ASSERT_TRUE(contextHub->registerCallback(getHubId(), cb, &success).isOk()); ASSERT_TRUE(success); } @@ -263,8 +281,8 @@ void ContextHubAidl::testSettingChanged(Setting setting) { // In VTS, we only test that sending the values doesn't cause things to blow up - GTS tests // verify the expected E2E behavior in CHRE bool success; - ASSERT_TRUE(contextHub->registerCallback(getHubId(), new IContextHubCallbackDefault(), &success) - .isOk()); + sp cb = sp::make(); + ASSERT_TRUE(contextHub->registerCallback(getHubId(), cb, &success).isOk()); ASSERT_TRUE(success); ASSERT_TRUE(contextHub->onSettingChanged(setting, true /* enabled */).isOk());