commit fd8bd765b484f7d98cf2d20946b7d4168d6039de
author TreeHugger Robot <treehugger-gerrit@google.com> Wed Nov 23 00:00:03 2022 +0000
committer Android (Google) Code Review <android-gerrit@google.com> Wed Nov 23 00:00:03 2022 +0000
tree 600f4ee9e520a08e4c3b924137da0bea703fcb52
parent a955a595cb8880eab9a6052a3c593c045ca616f2
parent fb39edbd83ff4277270158f48e987f9ecfc840c5

Merge "Fix a bug that did not check the status after it is assigned"

gnss/aidl/vts/gnss_hal_test_cases.cpp

diff --git a/gnss/aidl/vts/gnss_hal_test_cases.cpp b/gnss/aidl/vts/gnss_hal_test_cases.cpp
index c1a177a..48027b6 100644
--- a/gnss/aidl/vts/gnss_hal_test_cases.cpp
+++ b/gnss/aidl/vts/gnss_hal_test_cases.cpp
@@ -1603,14 +1603,14 @@
     auto callback = sp<GnssMeasurementCallbackAidl>::make();
     sp<IGnssMeasurementInterface> 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;