Checkbox

replicated effort in bluetooth.txt

Bug #671136 reported by Jeff Lane 
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Checkbox
Fix Released
High
Daniel Manrique
Checkbox 0.13.1 "second_p_release"

Bug Description

in bluetooth.txt the two test cases, bluetooth-detect and bluetooth-detect-output are identical.

bluetooth-detect-output is run via shell where bluetooth-detect is run manually simply to get some bluetooth data into the logs.

It would be better to replace both with a single test that also logs the data discovered.

Tags: test

Related branches

lp://staging/~roadmr/checkbox/671136
Marc Tardif (community): Approve
Diff: 107 lines (+14/-15) (has conflicts)
3 files modified
data/whitelists/default.whitelist (+0/-1)
debian/changelog (+8/-0)
jobs/bluetooth.txt.in (+6/-14)
Jeff Lane  (bladernr)
Changed in checkbox-compatibility:
importance: Undecided → Wishlist
Revision history for this message
Ara Pulido (ara) wrote :

Moving it to checkbox

affects: checkbox-compatibility → checkbox
tags: added: test
Ara Pulido (ara)
Changed in checkbox:
status: New → Triaged
importance: Wishlist → Low
visibility: private → public
Ara Pulido (ara)
Changed in checkbox:
importance: Low → High
milestone: none → 0.13
Revision history for this message
Brendan Donegan (brendan-donegan) wrote :

Is the decision here then to make this test fully automated? I think it's confusing anyway for the user to be asking them about Bluetooth addresses so I think it's better as just a shell test, and the testing of Bluetooth should be done in a more functional test (sending a file for example)

Revision history for this message
Daniel Manrique (roadmr) wrote :

As is neither of those tests ask the user anymore, maybe one of them could be removed? They seem almost the same to me, except that bluetooth/detect will fail if there's no bluetooth address, while bluetooth/detect-output just outputs the same information to a text file.

Just remember that we need to let the OEM QA folks know about test removals, in case it might impact them.

Changed in checkbox:
milestone: 0.13 → 0.13.1
Daniel Manrique (roadmr)
Changed in checkbox:
status: Triaged → In Progress
assignee: nobody → Daniel Manrique (roadmr)
Marc Tardif (cr3)
Changed in checkbox:
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.