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

audio VTS: Use the right device type in PcmOnlyConfigInputStreamTest

Browse Source 
The fix in https://r.android.com/2322475 modifies the device
address, which becomes 'IN_DEFAULT' for the telephony case.
As a result, tests that verify capture position were not
skipped, and they do fail on certain platforms when there is
no active telephone call. The fix is to use the initial device
type which was passed to the test as a parameter, not the adjusted
one.

Bug: 263305254
Test: atest VtsHalAudioV7_0TargetTest
Test: atest VtsHalAudioV7_1TargetTest
Change-Id: Ied7134a1bd97e70dcc2400e0827810f9d71a6438
(cherry picked from commit 7029707e34)
This commit is contained in:
Mikhail Naganov
2022-12-21 23:14:37 +00:00
parent 017cb98ee9
commit 6de19308ab
1 changed files with 6 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
audio/core/all-versions/vts/functional/7.0/AudioPrimaryHidlHalTest.cpp
View File

@@ -687,7 +687,12 @@ class PcmOnlyConfigInputStreamTest : public InputStreamTest {
InputStreamTest::TearDown();
}
bool canQueryCapturePosition() const { return !xsd::isTelephonyDevice(address.deviceType); }
bool canQueryCapturePosition() const {
// See b/263305254 and b/259636577. Must use the device initially passed in
// as a parameter, not 'address' which gets adjusted during test setup for
// the telephony case.
return !xsd::isTelephonyDevice(getAttachedDeviceAddress().deviceType);
}
void createPatchIfNeeded() {
if (areAudioPatchesSupported()) {