Evolution-x/hardware_interfaces
1
0
Fork 0
mirror of https://github.com/Evolution-X/hardware_interfaces synced 2026-02-02 20:24:19 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge "Ensure non-empty SvInfo is received" into sc-dev am: 5fb5afe821

Browse Source 
Original change: https://googleplex-android-review.googlesource.com/c/platform/hardware/interfaces/+/15300801

Change-Id: I872c2db30932672cd0038d4cf05cfb3adfba6f6e
This commit is contained in:
TreeHugger Robot
2021-07-21 00:35:25 +00:00
committed by Automerger Merge Worker
parent 2c0c4343b3 5fb5afe821
commit fd49bb779d
1 changed files with 18 additions and 13 deletions
Download Patch File Download Diff File Expand all files Collapse all files

31
gnss/2.1/vts/functional/gnss_hal_test_cases.cpp
View File

@@ -249,35 +249,40 @@ TEST_P(GnssHalTest, TestGnssAntennaInfo) {
/* /*
* TestGnssSvInfoFields: * TestGnssSvInfoFields:
* Gets 1 location and a GnssSvInfo, and verifies * Gets 1 location and a (non-empty) GnssSvInfo, and verifies basebandCN0DbHz is valid.
* 1. basebandCN0DbHz is valid.
*/ */
TEST_P(GnssHalTest, TestGnssSvInfoFields) { TEST_P(GnssHalTest, TestGnssSvInfoFields) {
gnss_cb_->location_cbq_.reset(); gnss_cb_->location_cbq_.reset();
gnss_cb_->sv_info_list_cbq_.reset();
StartAndCheckFirstLocation(/* min_interval_msec= */ 1000, /* low_power_mode= */ false); StartAndCheckFirstLocation(/* min_interval_msec= */ 1000, /* low_power_mode= */ false);
int location_called_count = gnss_cb_->location_cbq_.calledCount(); int location_called_count = gnss_cb_->location_cbq_.calledCount();
// Tolerate 1 less sv status to handle edge cases in reporting.
int sv_info_list_cbq_size = gnss_cb_->sv_info_list_cbq_.size();
EXPECT_GE(sv_info_list_cbq_size, 0);
ALOGD("Observed %d GnssSvStatus, while awaiting one location (%d received)", ALOGD("Observed %d GnssSvStatus, while awaiting one location (%d received)",
sv_info_list_cbq_size, location_called_count); gnss_cb_->sv_info_list_cbq_.size(), location_called_count);
// Get the last sv_info_list // Wait for up to kNumSvInfoLists events for kTimeoutSeconds for each event.
std::list<hidl_vec<IGnssCallback_2_1::GnssSvInfo>> sv_info_vec_list; int kTimeoutSeconds = 2;
gnss_cb_->sv_info_list_cbq_.retrieve(sv_info_vec_list, sv_info_list_cbq_size, 1); int kNumSvInfoLists = 4;
hidl_vec<IGnssCallback_2_1::GnssSvInfo> last_sv_info_list = sv_info_vec_list.back(); std::list<hidl_vec<IGnssCallback_2_1::GnssSvInfo>> sv_info_lists;
hidl_vec<IGnssCallback_2_1::GnssSvInfo> last_sv_info_list;
do {
EXPECT_GT(gnss_cb_->sv_info_list_cbq_.retrieve(sv_info_lists, kNumSvInfoLists,
kTimeoutSeconds),
0);
last_sv_info_list = sv_info_lists.back();
} while (last_sv_info_list.size() == 0);
ALOGD("last_sv_info size = %d", (int)last_sv_info_list.size());
bool nonZeroCn0Found = false; bool nonZeroCn0Found = false;
for (auto sv_info : last_sv_info_list) { for (auto sv_info : last_sv_info_list) {
ASSERT_TRUE(sv_info.basebandCN0DbHz >= 0.0 && sv_info.basebandCN0DbHz <= 65.0); EXPECT_TRUE(sv_info.basebandCN0DbHz >= 0.0 && sv_info.basebandCN0DbHz <= 65.0);
if (sv_info.basebandCN0DbHz > 0.0) { if (sv_info.basebandCN0DbHz > 0.0) {
nonZeroCn0Found = true; nonZeroCn0Found = true;
} }
} }
// Assert at least one value is non-zero. Zero is ok in status as it's possibly // Assert at least one value is non-zero. Zero is ok in status as it's possibly
// reporting a searched but not found satellite. // reporting a searched but not found satellite.
ASSERT_TRUE(nonZeroCn0Found); EXPECT_TRUE(nonZeroCn0Found);
StopAndClearLocations(); StopAndClearLocations();
} }