commit 4feb1b20e679f66b4a44e6b1761009f7d8cc3236
author Tomasz Wasilczyk <twasilczyk@google.com> Thu Jul 25 10:57:19 2024 -0700
committer Tomasz Wasilczyk <twasilczyk@google.com> Mon Aug 19 03:13:06 2024 -0700
tree 42257ff727f358d2a3e86a79bd96fb98a587d08c
parent 9415a2f3efc77a4f9db8563a99a70919471129fa

Fix VtsHalRadioTargetTest crash

Bug: 355008150
Test: atest VtsHalRadioTargetTest
Change-Id: Ibcff341223709b522e62dadfc745c42f4358df75

radio/aidl/vts/radio_sim_test.cpp

diff --git a/radio/aidl/vts/radio_sim_test.cpp b/radio/aidl/vts/radio_sim_test.cpp
index 9d1c356..6ffe2c5 100644
--- a/radio/aidl/vts/radio_sim_test.cpp
+++ b/radio/aidl/vts/radio_sim_test.cpp
@@ -531,7 +531,7 @@
         EXPECT_EQ(RadioError::NONE, radioRsp_sim->rspInfo.error);
 
         if (aidl_version <= 2) {
-            EXPECT_EQ(1, radioRsp_sim->carrierRestrictionsResp.allowedCarriers.size());
+            ASSERT_EQ(1, radioRsp_sim->carrierRestrictionsResp.allowedCarriers.size());
             EXPECT_EQ(0, radioRsp_sim->carrierRestrictionsResp.excludedCarriers.size());
 
             ASSERT_TRUE(std::string("123") ==
@@ -543,7 +543,7 @@
             ASSERT_TRUE(radioRsp_sim->carrierRestrictionsResp.allowedCarriersPrioritized);
             EXPECT_EQ(SimLockMultiSimPolicy::NO_MULTISIM_POLICY, radioRsp_sim->multiSimPolicyResp);
         } else {
-            EXPECT_EQ(1, radioRsp_sim->carrierRestrictionsResp.allowedCarrierInfoList.size());
+            ASSERT_EQ(1, radioRsp_sim->carrierRestrictionsResp.allowedCarrierInfoList.size());
             EXPECT_EQ(0, radioRsp_sim->carrierRestrictionsResp.excludedCarrierInfoList.size());
             ASSERT_TRUE(std::string("321") ==
                         radioRsp_sim->carrierRestrictionsResp.allowedCarrierInfoList[0].mcc);