Closed
Bug 1218930
Opened 10 years ago
Closed 10 years ago
test_clock_run_stopwatch_then_reset.py : AssertionError: u'00:01.58' == u'00:01.58'
Categories
(Firefox OS Graveyard :: Gaia::UI Tests, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: MaxIvanov, Assigned: Silne30)
References
Details
Attachments
(1 file)
Description: Test case fails with "AssertionError"
Steps:
1. Update a flame device to 20151027030203
2. Run this test case http://lxr.mozilla.org/gaia/source/tests/python/gaia-ui-tests/gaiatest/tests/functional/clock/test_clock_run_stopwatch_then_reset.py
Actual result:
"AssertionError"
Expected result:
Test completes successfully.
Repro Rate: Reproduced via Jenkins adhoc (0/5 passing)
http://jenkins1.qa.scl3.mozilla.com/job/flame-kk.ui.adhoc.bitbar/318/HTML_Report/
Traceback (most recent call last):
File "/var/lib/jenkins/jobs/flame-kk.ui.adhoc.bitbar/workspace/.env/lib/python2.7/site-packages/marionette_client-1.0.0-py2.7.egg/marionette/marionette_test.py", line 296, in run
testMethod()
File "/var/lib/jenkins/jobs/flame-kk.ui.adhoc.bitbar/workspace/tests/python/gaia-ui-tests/gaiatest/tests/functional/clock/test_clock_run_stopwatch_then_reset.py", line 38, in test_click_run_stopwatch_then_reset
self.assertNotEqual(first_time, stopwatch_view.current_time)
AssertionError: u'00:01.58' == u'00:01.58'
Comment 1•10 years ago
|
||
This test was created as part of bug 1193978.
Comment 2•10 years ago
|
||
test_clock_create_new_alarm.py seems to have merge issues in its file, btw:
http://mxr.mozilla.org/gaia/source/tests/python/gaia-ui-tests/gaiatest/tests/functional/clock/test_clock_create_new_alarm.py#79
| Assignee | ||
Comment 3•10 years ago
|
||
Git this test working and updated the pull request.
Flags: needinfo?(jdorlus)
Comment 4•10 years ago
|
||
| Assignee | ||
Updated•10 years ago
|
Attachment #8681391 -
Flags: review?(martijn.martijn)
Comment 5•10 years ago
|
||
Comment on attachment 8681391 [details] [review]
[gaia] silne30:Bug_1193978_Stopwatch_Laps_Test > mozilla-b2g:master
I'm still getting this failure:
TEST-UNEXPECTED-ERROR | test_clock_create_new_alarm.py TestClockCreateNewAlarm.test_clock_create_new_alarm | AttributeError: 'Alarm' object has no attribute 'alarms'
Traceback (most recent call last):
File "/Users/mwargers/.virtualenvs/zapion_test2/lib/python2.7/site-packages/marionette_client-1.0.0-py2.7.egg/marionette/marionette_test.py", line 296, in run
testMethod()
File "/Users/mwargers/B2G/gaia_clean/tests/python/gaia-ui-tests/gaiatest/tests/functional/clock/test_clock_create_new_alarm.py", line 88, in test_clock_create_new_alarm
alarm_view.alarms[0].disable()
TEST-INFO took 112848ms
alarms[0] need to become alarm_items[0] there, I think.
Attachment #8681391 -
Flags: review?(martijn.martijn)
Comment 6•10 years ago
|
||
John, could you please look at this again? Especially bug 1219344 is very bad, causing other tests to not run.
Flags: needinfo?(jdorlus)
| Assignee | ||
Comment 7•10 years ago
|
||
I got a new commit in and the test is working now.
Flags: needinfo?(jdorlus) → needinfo?(martijn.martijn)
Comment 8•10 years ago
|
||
Comment on attachment 8681391 [details] [review]
[gaia] silne30:Bug_1193978_Stopwatch_Laps_Test > mozilla-b2g:master
Thanks, let's get this merged then.
Flags: needinfo?(martijn.martijn)
Attachment #8681391 -
Flags: review+
Attachment #8681391 -
Flags: feedback?(jlorenzo)
Comment 9•10 years ago
|
||
Comment on attachment 8681391 [details] [review]
[gaia] silne30:Bug_1193978_Stopwatch_Laps_Test > mozilla-b2g:master
r+ modulo the 2 time.sleep that don't look necessary. I ran both of the tests, they were all executed correctly.
Attachment #8681391 -
Flags: feedback?(jlorenzo) → review+
| Assignee | ||
Comment 10•10 years ago
|
||
Removed excess waits. Fixed.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Comment 12•10 years ago
|
||
(In reply to Johan Lorenzo [:jlorenzo] (QA) from comment #9)
> Comment on attachment 8681391 [details] [review]
> [gaia] silne30:Bug_1193978_Stopwatch_Laps_Test > mozilla-b2g:master
>
> r+ modulo the 2 time.sleep that don't look necessary. I ran both of the
> tests, they were all executed correctly.
Could you uplift this patch to v2.5 branch, please?
You need to log in
before you can comment on or make changes to this bug.
Description
•