Closed
Bug 1090047
Opened 11 years ago
Closed 11 years ago
[MCTS] Geolocation: AssertError: Expected position.speed to be 'None'
Categories
(Firefox OS Graveyard :: Certification Suite, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: atsai, Assigned: oouyang)
References
Details
Attachments
(2 files)
I encountered failure while executing MCTS w/ v2.0 Flame.
1:55.90 TEST_END: MainThread FAIL, expected PASS
Traceback (most recent call last):
File "/Volumes/CaseSensitive/workspace/src/fxos-certsuite/lib/python2.7/site-packages/fxos_certsuite-2.0.1-py2.7.egg/webapi_tests/geolocation/test_geolocation_basic.py", line 45, in test_get_current_position
self.assertIsNone(position['coords']['speed'], "Expected position.speed to be 'None'")
AssertionError: Expected position.speed to be 'None'
| Reporter | ||
Comment 1•11 years ago
|
||
Comment 2•11 years ago
|
||
I wasn't able to reproduce this, but suspect it could happen if you were moving the phone while it was getting the geolocation reading. Can you try to reproduce?
It's possible we should disable the speed check for this test.
| Assignee | ||
Comment 3•11 years ago
|
||
I can reproduce it in my flame 2.0 device.
I found event I just place my flame on the desk without moving during testing, it gets the same result.
After disable 'speed' check, I also found that 'heading' check also cause the similar issue.
I suggest also disable heading checking.
Any other comment?
| Assignee | ||
Comment 4•11 years ago
|
||
| Assignee | ||
Comment 5•11 years ago
|
||
Please have a look.
| Assignee | ||
Updated•11 years ago
|
Assignee: nobody → oouyang
| Assignee | ||
Comment 6•11 years ago
|
||
Comment on attachment 8540056 [details] [review]
Disable speed and heading
Disable speed and heading checking to let the case pass
Attachment #8540056 -
Flags: review?(jgriffin)
Attachment #8540056 -
Flags: review?(atsai)
Comment 7•11 years ago
|
||
Comment on attachment 8540056 [details] [review]
Disable speed and heading
Note we should land this in the main fxos-certsuite repo.
Attachment #8540056 -
Flags: review?(jgriffin) → review+
| Reporter | ||
Comment 8•11 years ago
|
||
Comment on attachment 8540056 [details] [review]
Disable speed and heading
r+ and merged.
Attachment #8540056 -
Flags: review?(atsai) → review+
| Reporter | ||
Updated•11 years ago
|
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•