Bluetooth API is going to switch to API v2. Tinderbox shouldn't run <Gecko>/dom/bluetooth/bluetooth1/tests/marionette/manifest.ini since it's designed for API v1.

Comment on attachment 8622950 [details] [diff] [review] Switch Bluetooth marionette test set to BT API v2. Review of attachment 8622950 [details] [diff] [review]: ----------------------------------------------------------------- Hi Jamin, Please make sure we could pass all tests on try for v2 before checking in. Switch to v2 test will work on tinderbox and won't affect other tests in theory, but I want to make sure nothing will be broken. Bug 1167064 haven't landed into m-c yet, you might need to qimport those changesets when you push to try. Thanks, Jocelyn

- Add reviewer's name to commit message. Thank Jocelyn for reviewing the patch. Treeherder: https://treeherder.mozilla.org/#/jobs?repo=try&revision=3666a513e311

Marionette tests of bluetooth2 were designed to run at real devices. Disable unsupported tests and create a follow-up Bug 1175389 to support BT tests on B2G emulator-kk.

Comment on attachment 8623489 [details] [diff] [review] Switch Bluetooth marionette test set to BT API v2 and disable unsupported tests (v2) Review of attachment 8623489 [details] [diff] [review]: ----------------------------------------------------------------- LGTM, thanks.

(In reply to Jocelyn Liu [:jocelyn] [:joliu] from comment #5) > Comment on attachment 8623489 [details] [diff] [review] > Switch Bluetooth marionette test set to BT API v2 and disable unsupported > tests (v2) > > Review of attachment 8623489 [details] [diff] [review]: > ----------------------------------------------------------------- > > LGTM, thanks. Thank Jocelyn for reviewing the patch. Treeherder: https://treeherder.mozilla.org/#/jobs?repo=try&revision=c5cc6ece53e3

+++ b/dom/bluetooth/bluetooth2/tests/marionette/manifest.ini @@ -1,11 +1,17 @@ [DEFAULT] b2g = true browser = false qemu = false +disabled = Bug 1175389 [test_dom_BluetoothManager_API2.js] I thought the "disabled =" stuff needs to be added after the test that you want to have disabled, no?

(In reply to Martijn Wargers [:mwargers] (QA) from comment #9) > +++ b/dom/bluetooth/bluetooth2/tests/marionette/manifest.ini > @@ -1,11 +1,17 @@ > [DEFAULT] > b2g = true > browser = false > qemu = false > > +disabled = Bug 1175389 > [test_dom_BluetoothManager_API2.js] > > I thought the "disabled =" stuff needs to be added after the test that you > want to have disabled, no? You are right. Thank you for pointing this out. I've filed a bug 1179713 to fix this problem.