Closed
Bug 1147849
Opened 9 years ago
Closed 9 years ago
Disable test_edit_contact.py on Treeherder
Categories
(Firefox OS Graveyard :: Gaia::UI Tests, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: jlorenzo, Assigned: jlorenzo)
References
()
Details
Attachments
(1 file)
This test overlaps with the existing https://github.com/mozilla-b2g/gaia/blob/master/apps/communications/contacts/test/marionette/form_test.js#L82 . It's safe enough to disable it.
Comment 1•9 years ago
|
||
test_edit_contact.py has a link to the Moztrap test, form_test.js hasn't. Also, I assume it's not going to be disabled for device yet, right?
Assignee | ||
Updated•9 years ago
|
Summary: Disable test_edit_contact.py → Disable test_edit_contact.py on Treeherder
Assignee | ||
Comment 2•9 years ago
|
||
Like discussed during the roundtable, integration tests are not testing like a user would do. This is the purpose of acceptance tests. Having a link to moztrap is a Gij test is not worth it, then.
Comment 3•9 years ago
|
||
Yes, although gaia developers currently use the integration tests partly as UI tests and partly as unit tests, more or less as the bridge between UI tests and unit tests, from what I understood from the roundtable.
Comment 4•9 years ago
|
||
Updated•9 years ago
|
Attachment #8587936 -
Flags: review?(jlorenzo)
Assignee | ||
Comment 5•9 years ago
|
||
Comment on attachment 8587936 [details] [review] [gaia] bebef1987:Bug1147849 > mozilla-b2g:master Test disabled. Treeherder green.
Attachment #8587936 -
Flags: review?(jlorenzo) → review+
Assignee | ||
Updated•9 years ago
|
Keywords: checkin-needed
Updated•9 years ago
|
Keywords: checkin-needed
Comment 6•9 years ago
|
||
Since this was checked in, I guess this was fixed.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Updated•9 years ago
|
Assignee: nobody → jlorenzo
You need to log in
before you can comment on or make changes to this bug.
Description
•