B2G Bluetooth: add test cases for navigator.BluetoothManager.enabled

RESOLVED FIXED in 1.3 C1/1.4 S1(20dec)

Status

RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: vicamo, Assigned: vicamo)

Tracking

unspecified
1.3 C1/1.4 S1(20dec)
x86_64
Linux
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

Comment hidden (empty)
(Assignee)

Comment 1

5 years ago
Created attachment 833358 [details] [diff] [review]
patch

@echou: Somehow I find Bluetooth emulation is broken again but this test case still runs.  I'll find some time to fix it.

@jgriffin, this is the first WeBluetooth Marionette test case, anything else I should have noticed?

Will push to tryserver when it's back from maintenance.
Assignee: nobody → vyang
Attachment #833358 - Flags: review?(echou)
Attachment #833358 - Flags: feedback?(jgriffin)
(Assignee)

Comment 3

5 years ago
(In reply to Vicamo Yang [:vicamo][:vyang] from comment #1)
> Will push to tryserver when it's back from maintenance.

Confirmed it was executed and passed.
(Assignee)

Updated

5 years ago
Blocks: 939637
(Assignee)

Comment 4

5 years ago
Created attachment 8333637 [details] [diff] [review]
patch : v2

s/startTestCommon/startBluetoothTestBase/.  Most Bluetooth tests can't live without a valid adapter and this make the test script here a special case.  Will add another |function startBluetoothTest| that serves all others.  See bug 939637.
Attachment #833358 - Attachment is obsolete: true
Attachment #833358 - Flags: review?(echou)
Attachment #833358 - Flags: feedback?(jgriffin)
Attachment #8333637 - Flags: review?(echou)
Attachment #8333637 - Flags: feedback?(jgriffin)
Comment on attachment 8333637 [details] [diff] [review]
patch : v2

Review of attachment 8333637 [details] [diff] [review]:
-----------------------------------------------------------------

Everything looks good to me.
Attachment #8333637 - Flags: feedback?(jgriffin) → feedback+
Comment on attachment 8333637 [details] [diff] [review]
patch : v2

Review of attachment 8333637 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good to me. Thanks.
Attachment #8333637 - Flags: review?(echou) → review+
https://hg.mozilla.org/mozilla-central/rev/cc5b56a25d87
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → 1.3 C1/1.4 S1(20dec)
You need to log in before you can comment on or make changes to this bug.