Merge "Add REQUEST_NOT_SUPPORTED and INVALID_ARGUMENTS to be allowed." am: 86bc150058
am: bc00ee0af0
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2471285 Change-Id: Ia3723dd46fd71cf42d8dcc7b577b2bd2e4f6def7 Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
commit
52e70ec981
2 changed files with 19 additions and 7 deletions
|
@ -595,12 +595,17 @@ TEST_P(RadioHidlTest_v1_5, startNetworkScan) {
|
|||
if (cardStatus.base.base.base.cardState == CardState::ABSENT) {
|
||||
ASSERT_TRUE(CheckAnyOfErrors(radioRsp_v1_5->rspInfo.error, {RadioError::SIM_ABSENT}));
|
||||
} else if (cardStatus.base.base.base.cardState == CardState::PRESENT) {
|
||||
// OPERATION_NOT_ALLOWED should not be allowed; however, some vendors do
|
||||
// not support the required manual GSM search functionality. This is
|
||||
// tracked in b/112206766. Modems have "GSM" rat scan need to
|
||||
// Modems have "GSM" rat scan need to
|
||||
// support scanning requests combined with some parameters.
|
||||
ASSERT_TRUE(CheckAnyOfErrors(radioRsp_v1_5->rspInfo.error,
|
||||
{RadioError::NONE, RadioError::OPERATION_NOT_ALLOWED}));
|
||||
if (deviceSupportsFeature(FEATURE_TELEPHONY_GSM)) {
|
||||
ASSERT_TRUE(CheckAnyOfErrors(radioRsp_v1_5->rspInfo.error,
|
||||
{RadioError::NONE, RadioError::OPERATION_NOT_ALLOWED}));
|
||||
} else {
|
||||
ASSERT_TRUE(CheckAnyOfErrors(
|
||||
radioRsp_v1_5->rspInfo.error,
|
||||
{RadioError::NONE, RadioError::OPERATION_NOT_ALLOWED,
|
||||
RadioError::REQUEST_NOT_SUPPORTED, RadioError::INVALID_ARGUMENTS}));
|
||||
}
|
||||
}
|
||||
|
||||
if (radioRsp_v1_5->rspInfo.error == RadioError::NONE) {
|
||||
|
|
|
@ -862,8 +862,15 @@ TEST_P(RadioNetworkTest, startNetworkScan) {
|
|||
// not support the required manual GSM search functionality. This is
|
||||
// tracked in b/112206766. Modems have "GSM" rat scan need to
|
||||
// support scanning requests combined with some parameters.
|
||||
ASSERT_TRUE(CheckAnyOfErrors(radioRsp_network->rspInfo.error,
|
||||
{RadioError::NONE, RadioError::OPERATION_NOT_ALLOWED}));
|
||||
if (deviceSupportsFeature(FEATURE_TELEPHONY_GSM)) {
|
||||
ASSERT_TRUE(CheckAnyOfErrors(radioRsp_network->rspInfo.error,
|
||||
{RadioError::NONE, RadioError::OPERATION_NOT_ALLOWED}));
|
||||
} else {
|
||||
ASSERT_TRUE(CheckAnyOfErrors(
|
||||
radioRsp_network->rspInfo.error,
|
||||
{RadioError::NONE, RadioError::OPERATION_NOT_ALLOWED, RadioError::NONE,
|
||||
RadioError::INVALID_ARGUMENTS, RadioError::REQUEST_NOT_SUPPORTED}));
|
||||
}
|
||||
}
|
||||
|
||||
if (radioRsp_network->rspInfo.error == RadioError::NONE) {
|
||||
|
|
Loading…
Reference in a new issue