From bca36e52fd6585f81db2b2d6e9d2e69f1df96022 Mon Sep 17 00:00:00 2001 From: Yu-Han Yang Date: Fri, 27 Aug 2021 13:49:23 -0700 Subject: [PATCH] Allow getPsdsExtension to return nullptr Bug: 197962243 Test: Cuttlefish Change-Id: Id80fc453c3bf89f06f8078ba0e1aca67ec8fb488 --- gnss/aidl/vts/gnss_hal_test_cases.cpp | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/gnss/aidl/vts/gnss_hal_test_cases.cpp b/gnss/aidl/vts/gnss_hal_test_cases.cpp index 5964f815a8..91999f6368 100644 --- a/gnss/aidl/vts/gnss_hal_test_cases.cpp +++ b/gnss/aidl/vts/gnss_hal_test_cases.cpp @@ -53,17 +53,16 @@ TEST_P(GnssHalTest, SetupTeardownCreateCleanup) {} /* * TestPsdsExtension: - * 1. Gets the PsdsExtension and verifies that it returns a non-null extension. + * 1. Gets the PsdsExtension * 2. Injects empty PSDS data and verifies that it returns an error. */ TEST_P(GnssHalTest, TestPsdsExtension) { sp iGnssPsds; auto status = aidl_gnss_hal_->getExtensionPsds(&iGnssPsds); - ASSERT_TRUE(status.isOk()); - ASSERT_TRUE(iGnssPsds != nullptr); - - status = iGnssPsds->injectPsdsData(PsdsType::LONG_TERM, std::vector()); - ASSERT_FALSE(status.isOk()); + if (status.isOk() && iGnssPsds != nullptr) { + status = iGnssPsds->injectPsdsData(PsdsType::LONG_TERM, std::vector()); + ASSERT_FALSE(status.isOk()); + } } void CheckSatellitePvt(const SatellitePvt& satellitePvt) {