From fb39edbd83ff4277270158f48e987f9ecfc840c5 Mon Sep 17 00:00:00 2001 From: Zhanghao Wen Date: Tue, 22 Nov 2022 12:26:38 -0800 Subject: [PATCH] Fix a bug that did not check the status after it is assigned Test: on Cuttlefish Bug: 260002331 Change-Id: I7db69b19c3f29824970a5ae04aeae26e8bfd8447 --- gnss/aidl/vts/gnss_hal_test_cases.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/gnss/aidl/vts/gnss_hal_test_cases.cpp b/gnss/aidl/vts/gnss_hal_test_cases.cpp index c1a177a250..48027b62e0 100644 --- a/gnss/aidl/vts/gnss_hal_test_cases.cpp +++ b/gnss/aidl/vts/gnss_hal_test_cases.cpp @@ -1603,14 +1603,14 @@ TEST_P(GnssHalTest, TestAccumulatedDeltaRange) { auto callback = sp::make(); sp iGnssMeasurement; auto status = aidl_gnss_hal_->getExtensionGnssMeasurement(&iGnssMeasurement); + ASSERT_TRUE(status.isOk()); + ASSERT_TRUE(iGnssMeasurement != nullptr); IGnssMeasurementInterface::Options options; options.intervalMs = 1000; options.enableFullTracking = true; status = iGnssMeasurement->setCallbackWithOptions(callback, options); - ASSERT_TRUE(status.isOk()); - ASSERT_TRUE(iGnssMeasurement != nullptr); bool accumulatedDeltaRangeFound = false; const int kNumMeasurementEvents = 15;