Commit Graph

15087 Commits

Author SHA1 Message Date
sqian
fcd55caafd Update the currentCalls before VTS notification.
It's reasonable to update the global variable "currentCalls" before
notifying the reader thread. Otherwise this can cause some issues such
as null pointer dereference in other modem.

Bug: 139264227
Test: Don't have devices that uses the corresponding modem that reports
 this issues. This is verified according to a partner CL:
https://partner-android-review.googlesource.com/c/platform/hardware/interfaces/+/1422356

Change-Id: Iee9e18149397c39b7c84ec1dc570b9bac7142eb6
2019-08-22 16:21:14 +00:00
Anil Admal
02376b9e11 Add GetLocationLowPower VTS 2.0 test
- Run test only if LOW_POWER_MODE capability is supported.
- Modify the corresponding VTS 1.1 test to run only if the GNSS
  HAL version is 1.1.

Fixes: 127434062
Test: atest VtsHalGnssV2_0Target
Change-Id: Icbd18d80468e9e5a542f2eace5bfbe45af08ca47
Merged-In: Icbd18d80468e9e5a542f2eace5bfbe45af08ca47
(cherry picked from commit 03bc98e028)
2019-08-12 17:11:31 -07:00
Anil Admal
ad9f6b17a3 Add GNSS satellite blacklist VTS 2.0 test
- Use IGnssCallback@2.0::gnssSvStatusCb_2_0() method to receive SV
  info.
- Run the test only if SATELLITE_BLACKLIST capability is supported.
- IRNSS is excluded from blacklisting as IGnssConfiguration@2.0 is
  not updated to support the blacklisting new constellation type.
- Modify the corresponding VTS 1.1 test to run only if the GNSS
  HAL version is 1.1.

Bug: 128028791
Test: atest VtsHalGnssV2_0Target
Change-Id: Idd44a5a1c9d54d7434da945ecbd1ee90007f5054
Merged-In: Idd44a5a1c9d54d7434da945ecbd1ee90007f5054
(cherry picked from commit f42602678f)
2019-08-10 07:28:27 +00:00
TreeHugger Robot
1f12b5323a Merge "Verify there are no more current calls" into qt-dev 2019-07-30 22:37:53 +00:00
Roshan Pius
4335de4cac hostapd(vts): Use the correct syntax for vendor prop
Change the prop to add "ro.vendor." prefix for the new vendor prop.

Bug: 136645414
Change-Id: I5edfdc98991068eac279975397b6a29313bf31da
2019-07-29 11:16:56 -07:00
sqian
5f68117e87 Verify there are no more current calls
After disconnect calls, we want to verify there are no more current calls
by calling IRadio.getCurrenctCalls

Test: run vts
Bug: 138413412
Change-Id: Iacca49abff11fbdfb4ae6dd060a5f150e9e4a74d
2019-07-26 18:49:02 +00:00
Youming Ye
d005bbeb53 Add REQUEST_NOT_SUPPORTED to startNetworkScan
Test: VTS
Bug: 137298570
Change-Id: I0afacd413d467c3e25585d6d5431278fd61a1562
2019-07-22 18:34:56 -07:00
Thomas Joseph Avila
b4e4ddc88e [automerger skipped] Merge "Setting callbacktype type to invalid value before invoking HIDL call" into pie-vts-dev am: 5e9cf46491
am: d7059a8549 -s ours
am skip reason: change_id I9211c3e172727f56d0cceaf91b98f8d5b3c7fc7e with SHA1 18408a93d9 is in history

Change-Id: I261c1136ef8736309d6c05a8b1e7e2384196d76b
2019-07-22 18:25:55 -07:00
Thomas Joseph Avila
d7059a8549 Merge "Setting callbacktype type to invalid value before invoking HIDL call" into pie-vts-dev
am: 5e9cf46491

Change-Id: I487650af18179f7df94843618fe5032c1476db44
2019-07-22 18:18:05 -07:00
Thomas Joseph Avila
5e9cf46491 Merge "Setting callbacktype type to invalid value before invoking HIDL call" into pie-vts-dev 2019-07-23 00:50:43 +00:00
Youming Ye
53861e16c3 Merge "Add non-dds check for startNetworkScan" into qt-dev 2019-07-23 00:08:37 +00:00
Nate Jiang
1a20113e68 Setting callbacktype type to invalid value before invoking HIDL call
Avoid race condition between event callback and wait context. Make
callbacktype to invalid before call function to change it.

Test: excute VtsHalWifiNanV1_0TargetTest VtsHalWifiNanV1_2TargetTest
Bug: 133484330
Merged-In: I9211c3e172727f56d0cceaf91b98f8d5b3c7fc7e
Change-Id: I0a7e27411cb8fbd877f8228e9d53b62cac424b17
2019-07-22 18:52:06 +00:00
Youming Ye
fc85e8fa50 Add non-dds check for startNetworkScan
Providing non-dds exemption for HAL 1.2 network scan due to some devices
only perform network scan on preferred data sim. Since logical modem
id(0/1) and physical sim slot id(0/1) are intuitively aligned. Set
first sim as dds sim, and skip the network scan on the second one
(non-dds).

Test: Vts
Bug: 135243177
Change-Id: I58b89473714dc6d3ce6567ba1809baad6cd6d799
2019-07-19 16:32:46 -07:00
Anil Admal
e16ca9abca VTS 1.1 TestGnssMeasurementCallback test must allow 1.1 or 1.0 versions
Fixes: 137839372
Test: atest VtsHalGnssV1_1Target
Change-Id: Ie3b1fa8b4738c03bff7924396b5eb5c6774841f1
(cherry picked from commit 6abdaac714)
2019-07-19 18:18:16 +00:00
Valerie Hau
818fff840a Merge changes from topic "FixSRGBSetColorMode_VTS_qt-dev" into qt-dev
* changes:
  DO NOT MERGE: Fixing SRGB for devices that do not support SRGB color mode
  VTS : fix vts fail [1/1]
2019-07-19 14:49:44 +00:00
TreeHugger Robot
896c71a7d2 Merge "hostapd(vts): Add a property for SAP interface" into qt-dev 2019-07-19 00:57:14 +00:00
Roshan Pius
9a8538b4ee hostapd(vts): Add a property for SAP interface
Changes to the VTS test to support OEM's who use a separate
dedicated SAP interface.

Note: This is a VTS test change only. The default implementation does
not still support a dedicated SAP interface (OEM's can obviously
customize the default implementation for their devices).

Bug: 136645414
Test: None (verified by partner OEM)
Change-Id: I42b87bf90cfa90c7c218309164ab2b77f3685f9e
2019-07-18 14:38:41 +00:00
TreeHugger Robot
fc480f5904 Merge "Align NFCEE ID range with NCI 1.0 in VtsNfcv1.2 test" into qt-dev 2019-07-18 07:06:16 +00:00
zhangweiwei
0a50473369 Adapt change clearkey to Lazy hal
Exempt-From-Owner-Approval

Bug:135896867
Test: vts-tradefed run commandAndExit vts-staging-default -m VtsHalDrmV1_1Target

Change-Id: Iadba7a7afdccfe8a6299e3c0e1e03f63a58c5ca8
2019-07-17 23:24:56 +00:00
Valerie Hau
2f3553e1ed DO NOT MERGE: Fixing SRGB for devices that do not support SRGB color mode
Bug: 135375302
Test: build, boot, VtsHalGraphicsComposerV2_2TargetTest
Change-Id: Ie8f53effe3e6e29a4523ebfa450c79fdf96ed585
Merged-In: Ie8f53effe3e6e29a4523ebfa450c79fdf96ed585
2019-07-16 14:24:02 -07:00
jie.yuan
70e5b556ef VTS : fix vts fail [1/1]
Problem:
VtsHalGraphicsComposerV2_1Target#GraphicsComposerHidlCommandTest.PRESENT_DISPLAY_NO_LAYER_STATE_CHANGES fail
the testcase setcolormode SRGB without check if current device support SRGB

Solution:
change setColorMode from SRGB to NATIVE

Verify:
verify in on franklin

Bug: 135375302
Test: build, boot, VtsHalGraphicsComposerV2_1TargetTest
Change-Id: I646c263d96b03f9d30e6563a45cff37835001609
Merged-In: I5eebe5f530e3b62037d669992cf2eca0849f10be
2019-07-16 14:19:05 -07:00
Anthony Stange
65945cfb15 Verify mPollThread is joinable before detaching
If HidlSetUp() bails before startPollingThread() is called (which can
happen if the HAL isn't implemented on the given device), mPollThread
will initialize with the default constructor resulting in joinable()
returning false which means calling detach() throws an exception.

Checking joinable() before detaching allows the test suite to be skipped
successfully.

Fixes: 136736906
Test: Run vts-tradefed on VtsHalSensorsV1_0Target and verify that it is
    skipped successfully on a device that doesn't support HAL 1.0

Change-Id: Ie685ae2dc314edb8df2f3cc7112141a2f5e46008
2019-07-15 19:17:25 +00:00
TreeHugger Robot
c69700441e Merge "Disable startNetworkScan tests with parameters." into qt-dev 2019-07-12 11:41:24 +00:00
Youming Ye
bc5f18aae1 Disable startNetworkScan tests with parameters.
Network scan tests with parameters are disabled because of vendors
failed to fully implement startNetworkScan in HAL 1.4, and GSM rat scan
with parameters will get REQUEST_NOT_SUPPORTED error from modem.

Test: VTS
Bug: 137298570
Change-Id: I99df8495c5f654cb1420b7da36d7f5128ce14dbc
2019-07-12 00:07:25 +00:00
Willy Hu
fb1616eb94 VTS: Change stopNetworkScan from 1_1 to 1_2
Change stopNetworkScan from 1_1 to 1_2 when testing VtsHalRadioV1_2Target.

Symptom:

When vts process slot2 "startnetworkscan", it will call "stopnetworkscan()" when startnetworkscan
success. But from current design it will set stopnetworkscan to slot1(wrong slot, expect is slot2)
as bwlow log.

CTS fail log:
hardware/interfaces/radio/1.2/vts/functional/radio_hidl_hal_test.cpp:111

//Device request NwScan on 2nd Sim
06-28 11:30:22.770 radio   873  1213 F RILQ    : RIL[1][Log.cpp: 48] [DispatcherModul(873,1213)]
d: [DispatcherModule]: Handling msg = RIL_REQUEST_START_NETWORK_SCAN

//But Device want to abort on Primary Sim
06-28 11:30:22.776 radio   923  1205 F RILQ    : RIL[0][Log.cpp: 48] [DispatcherModul(923,1205)]
d: [NasModule]: Handling msg = RIL_REQUEST_STOP_NETWORK_SCAN[Context: IRadio(1681692777)]

Bug: 135982495
Test: Build pass. Local test VTS pass.
Change-Id: Ic53c24ab2a670e806b4ac7f192e6eb81252ade84
(cherry picked from commit 5cef297cb2)
2019-07-11 18:53:58 +00:00
nxp72763
67306b3aa6 Align NFCEE ID range with NCI 1.0 in VtsNfcv1.2 test
As per spec,NFCEE_ID value dynamically assigned by NFCC
has the following range
- In NCI_2.0,  0x80 <= NFCEE ID <= 0xFE.
- In NCI_1.0,  0x01 <= NFCEE ID <= 0xFE.

Bug: 137053584
Test: VtsHalNfcV1_2TargetTest with NCI1.0 & NCI2.0 supported device.
Change-Id: I09adaca8b36900a92d737b176e09234251aae2bc
2019-07-11 15:30:28 +08:00
TreeHugger Robot
56e9dd901e Merge "Support IMapper@3.0 in OMX VTS" into qt-dev 2019-07-09 17:25:39 +00:00
Pawin Vongmasa
94a3234219 Support IMapper@3.0 in OMX VTS
Test: vts-tradefed run vts -m VtsHalMediaOmxV1_0Host

Bug: 135422812

Change-Id: I9a37b4f926813b0cd90f96e318760eecafd04e06
2019-07-08 21:31:33 -07:00
zhangweiwei
d21dd860ce Camera: Vts: Fix missing session close
Test: vendor testing, Camera VTS on Pixel
Bug:135898951
Change-Id: I61f5a90d62732a00d0bc9abad05db0a1a9cc2b63
2019-07-08 08:45:18 -07:00
Shuo Qian
33e8a3668d Merge "Hang up EmergencyDial after Trigger in VTS" into qt-dev 2019-07-02 17:11:16 +00:00
TreeHugger Robot
8913d1df2c Merge "Deactivate sensors after flush events are received" into qt-dev 2019-07-02 16:28:53 +00:00
TreeHugger Robot
d016fd7949 Merge "Camera: VTS: Don't access invalid resources" into qt-dev 2019-07-01 21:55:58 +00:00
sqian
b2e7c71f39 Hang up EmergencyDial after Trigger in VTS
The whole test case run emergencyDial.RadioHidlTest_v1_4 first and then
 come after startNetworkScan.RadioHidlTest_v1_4 test case immediately
 (within ~3 secs) the NWscan failure is device in emergency call in
 progress/connected while doing emergencyDial.RadioHidlTest_v1_4 test

Although emergencyDial.RadioHidlTest_v1_4 only care about command/response
 is successful or not and don't care call is established or not, modem do
 care about call condition is in idle/progress/connected state before it
 can do NWscan

When there's dedicated channel established, modem can not perform AP
 request NWscan in parallel as we are single HW modem restriction.

Test: run vts
Bug: 135595082
Change-Id: Iecba54b93b96d33b57bc59e00f082ff133d2e57b
Merged-In: Iecba54b93b96d33b57bc59e00f082ff133d2e57b
2019-07-01 21:48:22 +00:00
Anthony Stange
4d90400985 Deactivate sensors after flush events are received
The VTS flush test case was previously deactivating sensors before
waiting for flush events to be received causing any pending flush events
to be discarded per the HAL contract.

Bug: 136472044
Test: Run test and ensure it passes
Change-Id: I23b94e650c6dbbc33640768bee356a49565ba753
2019-07-01 21:29:01 +00:00
TreeHugger Robot
d10841a83d Merge "keymaster: Relax testing under GSI" into qt-dev 2019-06-29 18:27:43 +00:00
nagendra modadugu
f18a8328a1 keymaster: Relax testing under GSI
GSI images do not have AVB verification enabled and therefore lack
several properties the keymaster HAL test depended on.  Selectively
disable those parts of the test that would fail with AVB verification
disabled.  Also disable date format checks under GSI.  When invoked from
GSI the TEE-backed keymaster doesn't use the correct date format.

Bug: 130843899
Test: VtsHalKeymasterV4_0TargetTest
Exempt-From-Owner-Approval: change only affects VTS-on-GSI behavior
Change-Id: Idaafb7b515c41290c766a8132f35d498ca15f48a
2019-06-29 18:27:38 +00:00
TreeHugger Robot
30ecfabab5 Merge "Q matrix: use radio.config@1.2::IRadioConfig" into qt-dev 2019-06-29 05:06:32 +00:00
Yifan Hong
7d350fed1e Q matrix: use radio.config@1.2::IRadioConfig
1.1 IRadioConfig does not exist. radio.config@1.2 is a
minor version uprev that only updates the sub interfaces,
not the top level interface.

Test: builds
Test: on device with target-level 4, w/o this change,
      with radio config 1.1, see test failure on deprecated
      radio config HAL
Test: on this device, with this change, with radio config 1.1,
      deprecated radio config HAL failure goes away.

Bug: 135172251
Change-Id: Id51870a8a757af0895577a27723cff8ee3ba0fdd
2019-06-28 16:28:54 -07:00
Emilian Peev
fd4ce7e76a Camera: VTS: Don't access invalid resources
There is no guarantee that the static metadata
pointer passed in the "DeviceCb" contructor will
be valid after the call completes.  The device
callback instance is expected to be active until
the camera session is open. Clone the required
metadata in "DeviceCb" and manage the lifetime
appropriately by using the "CameraMetadata"
wrapper.

Bug: 135976837
Test: CameraHidlTest#processCaptureRequestPreview
Change-Id: Idd3c6c8c2e5a3fc44a49712e25a04009cbd471b1
2019-06-28 17:03:12 +00:00
Ilya Matyukhin
00bc962dea Update VTS tests to support the latest BiometricsFace.hal
Bug: 134967390
Test: vts-tradefed run commandAndExit vts-hal -m VtsHalBiometricsFaceV1_0Target
Change-Id: Ifbd04d127626315efd8c8ce36636e550cbb843ff
2019-06-27 17:41:36 -07:00
Jimmy Chen
33723f56b5 Merge "p2p: skip MAC randomization vts test if not supported" into qt-dev 2019-06-27 05:19:46 +00:00
TreeHugger Robot
d4686a1a06 Merge "Camera: VTS: Allow unsupported logical/physical combination" into qt-dev 2019-06-26 15:42:57 +00:00
Jimmy Chen
7af5384d2d p2p: skip MAC randomization vts test if not supported
Bug: 135247522
Test: atest VtsHalWifiSupplicantP2pV1_2TargetTest
Change-Id: I1aafcdbe88f93dbf73069d24ae90cad0d395df6e
2019-06-26 14:40:10 +08:00
TreeHugger Robot
6c4e33d079 Merge "Increase leniency of attestation record timestamps" into qt-dev 2019-06-26 06:08:42 +00:00
Steven Moreland
7a605ba87e Merge "compatibility_matrix: specify @1.2::ISap directly" into qt-dev 2019-06-25 20:15:08 +00:00
Prachi Hande
c6d2640d29 Merge "Updated START_SESSION message interpretation" into qt-dev 2019-06-25 18:23:53 +00:00
Steven Moreland
c689517d59 compatibility_matrix: specify @1.2::ISap directly
@1.4::ISap and @1.3::ISap don't exist, so we can't declare a requirement
for them.

Bug: 135627722
Test: reproduce error with vts_treble_vintf_test and then test to see
    this change fixes it.
Change-Id: I8e48fcaf56701087d5720d5133ffa9b23a4ea31e
2019-06-25 18:22:04 +00:00
Prachi Hande
af6af767e7 Updated START_SESSION message interpretation
Removed the dependency on the fact that the core's new ID is always >= existing ID.
New ID and the old ID may not have any such relation.

Bug: 135475397
Fixes: 135475397
Test: Updated unit tests in VmsUtilsTest. Ran the tests on Hawk.

Change-Id: I5a7e72f36a84f5bf2d3c7bf1ccb89d27c5c0da97
2019-06-24 17:47:56 -07:00
Garret Kelly
9c0a45795f Increase leniency of attestation record timestamps
The TEE keymaster has been seen to be almost a minute out of sync with
the host clock during attestation.  Increase the leniency window to two
minutes.

Bug: 134408892
Bug: 134408367
Test: VtsHalKeymasterV4_0TargetTest
Change-Id: Ic256a939dcd7e7b108099cfcf237cacde8dde059
2019-06-24 23:28:52 +00:00
Yifan Hong
5a58efedd8 Merge "health storage hal vts test: add more tolerance." into qt-dev 2019-06-24 21:54:27 +00:00