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

Issue 617746 link

Starred by 1 user

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Feature

Blocking:
issue 707630



Sign in to add a comment

Provide framework to write BlueZ tests.

Project Member Reported by r...@chromium.org, Jun 6 2016

Issue description

Currently we have close to no unittests for the code that interacts with BlueZ over DBus. We've started with one test bluetooth_gatt_application_service_provider_unittest.cc which verifies the output of the DBus call against a fixed string. Even though this isn't a great way of testing, it is much better than nothing.

We need to add the ability to write this kind of a test easily for other DBus components that talk to BlueZ. Ideally we should have a small framework that allows a developer to specify the DBus object provider and an expected DBus output.

Without these tests this code is very brittle. We've had bugs happen often due to unexpected changes that were never caught due to a complete lack of testing in this area.

 
Project Member

Comment 1 by sheriffbot@chromium.org, Jul 6 2016

Labels: -M-53 M-54 MovedFrom-53
Moving this nonessential bug to the next milestone.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 2 by r...@chromium.org, Aug 5 2016

Labels: -M-54 -MovedFrom-53
This is still important, just removing milestone since it doesn't need to land by any particular branch date.

Labels: -Type-Bug Type-Feature

Comment 4 by scheib@chromium.org, Aug 12 2016

Blocking: 580406

Comment 5 by st...@chromium.org, Aug 24 2016

Owner: st...@chromium.org

Comment 6 by st...@chromium.org, Mar 3 2017

Owner: r...@chromium.org
Blocking: -580406
Blocking: 707630
Cc: -scheib@chromium.org
Owner: jhawkins@chromium.org
James now owns connectivity in Chrome OS UI, over to him.

Sign in to add a comment