1102111 - [MCTS] error on test "webapi_tests.bluetooth.test_bluetooth_discovery.TestBluetoothDiscovery.test_discovering"

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Certification Suite, defect)

Description

[pkj1020]

User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/39.0.2171.65 Safari/537.36

Steps to reproduce:

I have conducted MCTS using the FxOS device (NOT Flame).


Actual results:

BluetoothAdapter.stopDiscovery returned error.

===================================================
Traceback (most recent call last):
File "/root/Desktop/fxos-MCTS-2.0/certsuite_venv/local/lib/python2.7/site-packages/fxos_certsuite-2.0.1-py2.7.egg/webapi_tests/bluetooth/test_bluetooth_discovery.py", line 91, in test_discovering
self.stop_bt_discovery()
File "/root/Desktop/fxos-MCTS-2.0/certsuite_venv/local/lib/python2.7/site-packages/fxos_certsuite-2.0.1-py2.7.egg/webapi_tests/bluetooth/bluetooth_test.py", line 332, in stop_bt_discovery
self.fail("BluetoothAdapter.stopDiscovery returned error")
AssertionError: BluetoothAdapter.stopDiscovery returned error
===================================================
The stopDiscovery has to be called when Discovery is working. The stopDiscovery is called after 30 seconds in this test. But certain devices are ended the discovery about 15sec~20sec. The test case might be modified. 



Expected results:

Always "PASS"

Comment 1

[Youngwoo Jo]

I think this test case should check if the discovery is running or not, before calling stopDiscovery.
For this purpose, we can use BluetoothAdapter.discovering which indicates if the device is in the process of discovering (true) or not (false).
 - https://developer.mozilla.org/en-US/docs/Web/API/BluetoothAdapter.discovering

Comment 2

[Rachelle Yang [:ryang][[e-mail]]]

Hi Jonathan, would you please kindly check this and provide your comments on this ? 

Thank you very much!

Comment 3

[Rachelle Yang [:ryang][[e-mail]]]

Hi Jongsoo, thanks for your kindly support.

If possible, could you please kindly provide any related logs for our further investigation ?

Thank you very much!

Comment 4

[Jonathan Griffin (:jgriffin)]

(In reply to Youngwoo Jo from comment #1)
> I think this test case should check if the discovery is running or not,
> before calling stopDiscovery.
> For this purpose, we can use BluetoothAdapter.discovering which indicates if
> the device is in the process of discovering (true) or not (false).
>  -
> https://developer.mozilla.org/en-US/docs/Web/API/BluetoothAdapter.discovering

Agreed, this is the correct solution.

Comment 5

[jongsoo.oh]

comment 4, the solution might be confirmed.

Comment 6

[oo[oouyang]]

Attachment: check discovering mode before stop discovering

Please help me to check it.

Comment 7

[Jonathan Griffin (:jgriffin)]

Comment on attachment 8541587 [details] [review]
check discovering mode before stop discovering

This looks good.   Please land in the main fxos-certsuite repo as well.

Comment 8

[Al Tsai [:atsai]]

Comment on attachment 8541587 [details] [review]
check discovering mode before stop discovering

r+.

Comment 9

[Al Tsai [:atsai]]

merged