commit 40b07e71084efb63a2d29803405bd3f09e7f077e
author Treehugger Robot <treehugger-gerrit@google.com> Mon Mar 02 23:02:19 2020 +0000
committer Gerrit Code Review <noreply-gerritcodereview@google.com> Mon Mar 02 23:02:19 2020 +0000
tree d941bb5c85c342e75d6d9a25e64d6e9ea8cb5ca4
parent 8cdbb7c8ceab9e99f62ad266f8215521af026b17
parent dd8769b44795a03a67c4065d69814beae43b39cf

Merge "Make GetLocationLowPower test warning instead of failing" into android10-tests-dev

radio/1.0/vts/functional/radio_hidl_hal_cell_broadcast.cpp

diff --git a/radio/1.0/vts/functional/radio_hidl_hal_cell_broadcast.cpp b/radio/1.0/vts/functional/radio_hidl_hal_cell_broadcast.cpp
index 2c1eb60..605b569 100644
--- a/radio/1.0/vts/functional/radio_hidl_hal_cell_broadcast.cpp
+++ b/radio/1.0/vts/functional/radio_hidl_hal_cell_broadcast.cpp
@@ -21,6 +21,9 @@
 /*
  * Test IRadio.setGsmBroadcastConfig() for the response returned.
  */
+/*
+ * TODO(b/145710242) this is commented out because it breaks on TSTS devices on Q. The test
+ * can be safely enabled for future devices).
 TEST_F(RadioHidlTest, setGsmBroadcastConfig) {
     serial = GetRandomSerialNumber();
 
@@ -80,6 +83,7 @@
                                      CHECK_GENERAL_ERROR));
     }
 }
+*/
 
 /*
  * Test IRadio.getGsmBroadcastConfig() for the response returned.

radio/1.0/vts/functional/radio_hidl_hal_ims.cpp

diff --git a/radio/1.0/vts/functional/radio_hidl_hal_ims.cpp b/radio/1.0/vts/functional/radio_hidl_hal_ims.cpp
index 4331c06..15dc6de 100644
--- a/radio/1.0/vts/functional/radio_hidl_hal_ims.cpp
+++ b/radio/1.0/vts/functional/radio_hidl_hal_ims.cpp
@@ -80,6 +80,9 @@
 /*
  * Test IRadio.setFacilityLockForApp() for the response returned.
  */
+/*
+ * TODO(b/145710242) this is commented out because it breaks on TSTS devices on Q. The test
+ * can be safely enabled for future devices).
 TEST_F(RadioHidlTest, setFacilityLockForApp) {
     serial = GetRandomSerialNumber();
     std::string facility = "";
@@ -100,6 +103,7 @@
                                      CHECK_GENERAL_ERROR));
     }
 }
+*/
 
 /*
  * Test IRadio.setBarringPassword() for the response returned.
@@ -127,6 +131,9 @@
 /*
  * Test IRadio.getClip() for the response returned.
  */
+/*
+ * TODO(b/145710242) this is commented out because it breaks on TSTS devices on Q. The test
+ * can be safely enabled for future devices).
 TEST_F(RadioHidlTest, getClip) {
     serial = GetRandomSerialNumber();
 
@@ -141,6 +148,7 @@
                                      CHECK_GENERAL_ERROR));
     }
 }
+*/
 
 /*
  * Test IRadio.setSuppServiceNotifications() for the response returned.

radio/1.1/vts/functional/radio_hidl_hal_api.cpp

diff --git a/radio/1.1/vts/functional/radio_hidl_hal_api.cpp b/radio/1.1/vts/functional/radio_hidl_hal_api.cpp
index 33347c5..42f6916 100644
--- a/radio/1.1/vts/functional/radio_hidl_hal_api.cpp
+++ b/radio/1.1/vts/functional/radio_hidl_hal_api.cpp
@@ -24,6 +24,9 @@
     /* Record the sim card state for the testing environment */
     CardState cardStateForTest = cardStatus.cardState;
 
+#if 0
+    /* This test has to be removed for Japan Model.
+     * After "setSimCardPower power down", Japan model can not "setSimCardPower power up" */
     /* Test setSimCardPower power down */
     serial = GetRandomSerialNumber();
     radio_v1_1->setSimCardPower_1_1(serial, CardPowerState::POWER_DOWN);
@@ -56,6 +59,7 @@
     ASSERT_TRUE(CheckAnyOfErrors(radioRsp_v1_1->rspInfo.error,
                                  {RadioError::NONE, RadioError::REQUEST_NOT_SUPPORTED,
                                   RadioError::INVALID_ARGUMENTS, RadioError::RADIO_NOT_AVAILABLE}));
+#endif
 
     /**
      * If the sim card status for the testing environment is PRESENT,