New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 680378 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Jan 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug



Sign in to add a comment

Automated test for bluetooth classic mouse events

Project Member Reported by josephsih@chromium.org, Jan 12 2017

Issue description

It is desired to have an autotest in which an RN-42 kit is emulated as a bluetooth classic mouse and sends various mouse events to a chromebook. In the chromebook, verify that all the events are received completely and in the same sequence. 
 
Status: Started (was: Untriaged)
Project Member

Comment 3 by bugdroid1@chromium.org, Jan 17 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/9dfa960af3f7b56ef52291d3c7e21f52770ee381

commit 9dfa960af3f7b56ef52291d3c7e21f52770ee381
Author: Joseph Hwang <josephsih@chromium.org>
Date: Mon Sep 05 06:15:06 2016

client/bin/input: add an input_event_recorder module

Add an input_event_recorder module to record input events received on a DUT.

BUG= chromium:680378 
TEST=Record mouse events on a Samus
$ python input_event_recorder.py
Then move a wired mouse around, click buttons, and scroll wheels.
Observe the mouse events are recorded correctly.

Change-Id: I97ca22831de5ae1d80a5d2a15dda042e172afece
Reviewed-on: https://chromium-review.googlesource.com/381153
Commit-Ready: Shyh-In Hwang <josephsih@chromium.org>
Tested-by: Shyh-In Hwang <josephsih@chromium.org>
Reviewed-by: Wai-Hong Tam <waihong@google.com>

[add] https://crrev.com/9dfa960af3f7b56ef52291d3c7e21f52770ee381/client/bin/input/input_event_recorder.py

Project Member

Comment 4 by bugdroid1@chromium.org, Jan 17 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/a66e59232a33379adeb5a7028ad8689d4fff9da8

commit a66e59232a33379adeb5a7028ad8689d4fff9da8
Author: Joseph Hwang <josephsih@chromium.org>
Date: Mon Sep 05 06:22:00 2016

bluetooth_AdapterHIDReports: mouse events test

Verify that the bluetooth adapter of the DUT could correctly receive
HID reports sent from a connected bluetooth device.

Specifically, the following subtests are executed in this autotest.
    - test_reset_on_adapter
    - test_pairable
    - test_discover_device
    - test_pairing
    - test_connection_by_adapter
    - test_mouse_left_click
    - test_mouse_right_click
    - test_mouse_move_in_x
    - test_mouse_move_in_y
    - test_mouse_move_in_xy
    - test_mouse_scroll_down
    - test_mouse_scroll_up
    - test_mouse_click_and_drag
    - test_disconnection_by_adapter
    - test_remove_pairing

BUG= chromium:680378 
TEST=Use a DUT and a chameleon board with an RN-42 kit
(cr) $ test_that --board=$BOARD --args "chameleon_host=$CHAMELEON_IP"
     $DUT_IP bluetooth_AdapterHIDReports.mouse

Change-Id: I949da4b1fbca689bc5960b39dcba5a0369e31454
Reviewed-on: https://chromium-review.googlesource.com/381155
Commit-Ready: Shyh-In Hwang <josephsih@chromium.org>
Tested-by: Shyh-In Hwang <josephsih@chromium.org>
Reviewed-by: Cheng-Yi Chiang <cychiang@chromium.org>

[modify] https://crrev.com/a66e59232a33379adeb5a7028ad8689d4fff9da8/server/cros/bluetooth/bluetooth_adpater_tests.py
[add] https://crrev.com/a66e59232a33379adeb5a7028ad8689d4fff9da8/server/site_tests/bluetooth_AdapterHIDReports/control.mouse
[add] https://crrev.com/a66e59232a33379adeb5a7028ad8689d4fff9da8/server/site_tests/bluetooth_AdapterHIDReports/bluetooth_AdapterHIDReports.py

Project Member

Comment 5 by bugdroid1@chromium.org, Jan 17 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/e832e7f7f364296c8ae05676a55763558f10ecc5

commit e832e7f7f364296c8ae05676a55763558f10ecc5
Author: Joseph Hwang <josephsih@chromium.org>
Date: Mon Sep 05 10:22:42 2016

autotest-server-tests: add bluetooth_AdapterHIDReports test

Add a server-side bluetooth_AdapterHIDReports test.

BUG= chromium:680378 
TEST=None
CQ-DEPEND=381155

Change-Id: Ibe15aaf3aad1f9abece4a4fe97e988577c35958b
Reviewed-on: https://chromium-review.googlesource.com/381095
Commit-Ready: Shyh-In Hwang <josephsih@chromium.org>
Tested-by: Shyh-In Hwang <josephsih@chromium.org>
Reviewed-by: Wai-Hong Tam <waihong@google.com>

[modify] https://crrev.com/e832e7f7f364296c8ae05676a55763558f10ecc5/chromeos-base/autotest-server-tests/autotest-server-tests-9999.ebuild

Status: Fixed (was: Started)

Comment 7 by dchan@google.com, Apr 17 2017

Labels: VerifyIn-59
I don't think this test is currently running in the two devices that have the "chameleon:bt_hid" label. Is a test_bluetooth label needed as well? 
Hi Ruchi, thank you for catching the issue!

We need

ATTRIBUTES = 'suite:bluetooth, suite:bluetooth_sanity'

instead of

ATTRIBUTES = 'suite:bluetooth'

in the control file. Then this test would be running in bluetooth_sanity suite. Could you help send a patch to fix it? Thanks!
Cc: rjahagir@chromium.org
Sure thing!
https://chromium-review.googlesource.com/c/489045/
Project Member

Comment 11 by bugdroid1@chromium.org, Apr 28 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/90ca97177f61ce7a1174d2b0e57c20ad2ad217d8

commit 90ca97177f61ce7a1174d2b0e57c20ad2ad217d8
Author: Ruchi Jahagirdar <rjahagir@google.com>
Date: Fri Apr 28 09:58:16 2017

[Bluetooth] Add bluetooth_sanity suite to control

Add suite:bluetooth_sanity suite to control file to run
bluetooth_AdapterHIDReports test.

BUG= chromium:680378 
TEST=None

Change-Id: I25a21cb947f92060967e86ded1d37263b7387ce1
Reviewed-on: https://chromium-review.googlesource.com/489045
Commit-Ready: Ruchi Jahagirdar <rjahagir@chromium.org>
Tested-by: Ruchi Jahagirdar <rjahagir@chromium.org>
Reviewed-by: Shyh-In Hwang <josephsih@chromium.org>

[modify] https://crrev.com/90ca97177f61ce7a1174d2b0e57c20ad2ad217d8/server/site_tests/bluetooth_AdapterHIDReports/control.mouse

Status: Verified (was: Fixed)

Sign in to add a comment