From e6b1f319c0807c2dfe4190ce2b3e13c44f6e9262 Mon Sep 17 00:00:00 2001 From: Lei Ju Date: Fri, 21 Apr 2023 11:32:55 -0700 Subject: [PATCH] Update CHRE HAL VTS test Remove the tests related to null callbacks and the restriction of nonempty preloaded nanoapps. Bug: 279220092 Test: VTS test and manual test Change-Id: Iaa8918f398e71ba25d47c64bd36e73746bf09b84 --- contexthub/aidl/vts/VtsAidlHalContextHubTargetTest.cpp | 7 ------- 1 file changed, 7 deletions(-) diff --git a/contexthub/aidl/vts/VtsAidlHalContextHubTargetTest.cpp b/contexthub/aidl/vts/VtsAidlHalContextHubTargetTest.cpp index f544d833d9..89a9e23a77 100644 --- a/contexthub/aidl/vts/VtsAidlHalContextHubTargetTest.cpp +++ b/contexthub/aidl/vts/VtsAidlHalContextHubTargetTest.cpp @@ -133,10 +133,6 @@ TEST_P(ContextHubAidl, TestRegisterCallback) { ASSERT_TRUE(contextHub->registerCallback(getHubId(), cb).isOk()); } -TEST_P(ContextHubAidl, TestRegisterNullCallback) { - ASSERT_TRUE(contextHub->registerCallback(getHubId(), nullptr).isOk()); -} - // Helper callback that puts the async appInfo callback data into a promise class QueryAppsCallback : public android::hardware::contexthub::BnContextHubCallback { public: @@ -195,7 +191,6 @@ TEST_P(ContextHubAidl, TestGetPreloadedNanoappIds) { GTEST_SKIP() << "Not supported -> old API; or not implemented"; } else { ASSERT_TRUE(status.isOk()); - ASSERT_FALSE(preloadedNanoappIds.empty()); } } @@ -323,8 +318,6 @@ void ContextHubAidl::testSettingChanged(Setting setting) { ASSERT_TRUE(contextHub->onSettingChanged(setting, true /* enabled */).isOk()); ASSERT_TRUE(contextHub->onSettingChanged(setting, false /* enabled */).isOk()); - - ASSERT_TRUE(contextHub->registerCallback(getHubId(), nullptr).isOk()); } TEST_P(ContextHubAidl, TestOnLocationSettingChanged) {