Re: [PATCHv2 0/8] Android tester GATT test cases

[Szymon Janc <szymon.janc@tieto.com>]

Hi Marcin,

On Friday 29 of August 2014 10:26:01 Marcin Kraglak wrote:
> This patch set improve handling of ssp requests in few test cases
> and adds GATT test cases for searching primary services and
> characteristics.
> 
> This set contains rebased version of pending commits.
> 
> Marcin Kraglak (8):
>   android/tester: Simplify ACTION_FAIl and ACTION_SUCCESS defines
>   android/tester: Add function to set default ssp handler
>   android/tester: Add support for search services callbacks
>   android/tester: Add GATT Client Search Service test case
>   android/tester: Add GATT Search Service 2 test case
>   android/tester: Add negative GATT Search Service test
>   android/tester: Add support for get_characteristic_cb()
>   android/tester: Add GATT get_characteristic test
> 
>  android/tester-a2dp.c |   3 +
>  android/tester-gatt.c | 269 +++++++++++++++++++++++++++++++++++++++++++++++++-
>  android/tester-main.c | 122 ++++++++++++++++++++++-
>  android/tester-main.h |  44 ++++++---
>  android/tester-pan.c  |   3 +
>  5 files changed, 425 insertions(+), 16 deletions(-)
> 

There seems to be a bug  that results in command being send after daemon
shutdown resulting in:

bluetoothd[22413]: src/sdpd-service.c:remove_record_from_server() Removing record with handle 0x10004
bluetoothd[22413]: src/sdpd-service.c:remove_record_from_server() Removing record with handle 0x10003
bluetoothd[22413]: Stopping SDP server
bluetoothd[22413]: ExitBlueZ E: Notification socket closed
Gatt Client - Scan - test passed

Gatt Client - Scan - teardown
BlueZ D: android/hal-gatt.c:cleanup() 
BlueZ E: Sending command failed:Broken pipe


This needs to be fixed before patches can go in.

-- 
Best regards, 
Szymon Janc