From 5341e17199c4de012454ab51bdad9754fcfdfb15 Mon Sep 17 00:00:00 2001 From: "jun.wang" Date: Tue, 3 Sep 2019 14:33:13 +0800 Subject: [PATCH] Fix RadioHidlTest_v1_2.startNetworkScan maxSearchTime Add the initialization of the incrementalResultsPeriodicity and maxSearchTime in the networkScan request. The incrementalResultsPeriodicity and maxSearchTime are not initialized in the networkScan request. So the default value is 0. But the incrementalResultsPeriodicity and maxSearchTime are required not to be 0 in subsequent tests. CL ported from pag/1433565 Bug:140451434 Test: make vts && vts-tradefed run commandAndExit vts -m \ VtsHalRadioV1_2Target -t RadioHidlTest_v1_2#startNetworkScan Change-Id: I1b4e512a37d3d85ebc905a64987a40af801bce53 (cherry picked from commit f7894eb4d8c395add7c70a20520336057ae2656f) --- radio/1.2/vts/functional/radio_hidl_hal_api.cpp | 5 ++++- radio/1.4/vts/functional/radio_hidl_hal_api.cpp | 7 ++++++- 2 files changed, 10 insertions(+), 2 deletions(-) diff --git a/radio/1.2/vts/functional/radio_hidl_hal_api.cpp b/radio/1.2/vts/functional/radio_hidl_hal_api.cpp index 5184ef962c..a98f22aacf 100644 --- a/radio/1.2/vts/functional/radio_hidl_hal_api.cpp +++ b/radio/1.2/vts/functional/radio_hidl_hal_api.cpp @@ -46,7 +46,10 @@ TEST_F(RadioHidlTest_v1_2, startNetworkScan) { ::android::hardware::radio::V1_2::NetworkScanRequest request = { .type = ScanType::ONE_SHOT, .interval = 60, - .specifiers = {::GERAN_SPECIFIER_P900, ::GERAN_SPECIFIER_850}}; + .specifiers = {::GERAN_SPECIFIER_P900, ::GERAN_SPECIFIER_850}, + .maxSearchTime = 60, + .incrementalResults = false, + .incrementalResultsPeriodicity = 1}; Return res = radio_v1_2->startNetworkScan_1_2(serial, request); ASSERT_OK(res); diff --git a/radio/1.4/vts/functional/radio_hidl_hal_api.cpp b/radio/1.4/vts/functional/radio_hidl_hal_api.cpp index b2d19a2b0c..d3012bbd71 100644 --- a/radio/1.4/vts/functional/radio_hidl_hal_api.cpp +++ b/radio/1.4/vts/functional/radio_hidl_hal_api.cpp @@ -178,7 +178,12 @@ TEST_F(RadioHidlTest_v1_4, startNetworkScan) { .channels = {1, 2}}; ::android::hardware::radio::V1_2::NetworkScanRequest request = { - .type = ScanType::ONE_SHOT, .interval = 60, .specifiers = {specifier}}; + .type = ScanType::ONE_SHOT, + .interval = 60, + .specifiers = {specifier}, + .maxSearchTime = 60, + .incrementalResults = false, + .incrementalResultsPeriodicity = 1}; Return res = radio_v1_4->startNetworkScan_1_4(serial, request); ASSERT_OK(res);