Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit e6b1f319 authored by Lei Ju's avatar Lei Ju
Browse files

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
parent 3af3d5bd

contexthub/aidl/vts/VtsAidlHalContextHubTargetTest.cpp

+0 −7
Original line number Diff line number Diff line
@@ -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) {