Closed
Bug 1123652
Opened 10 years ago
Closed 10 years ago
Disable test_dialer_set_up_conference_call.py due to Bug 1117813
Categories
(Firefox OS Graveyard :: Gaia::UI Tests, defect)
Tracking
(b2g-v2.2 fixed, b2g-master fixed)
RESOLVED
FIXED
People
(Reporter: viorela, Assigned: jlorenzo)
References
Details
Attachments
(2 files)
Test test_dialer_set_up_conference_call.py is constantly failing because of the issue described in https://bugzilla.mozilla.org/show_bug.cgi?id=1117813#c0.
We should disable it until that bug is fixed.
Assignee | ||
Updated•10 years ago
|
Assignee | ||
Comment 1•10 years ago
|
||
Attachment #8551772 -
Flags: review?(viorela.ioia)
Reporter | ||
Comment 2•10 years ago
|
||
Comment on attachment 8551772 [details] [review]
Gaia PR
Merged in https://github.com/mozilla-b2g/gaia/commit/5d8abe663c0254c71d1568823210daaa7fb310d1
Attachment #8551772 -
Flags: review?(viorela.ioia) → review+
Reporter | ||
Updated•10 years ago
|
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Comment 3•10 years ago
|
||
Hi Johan, I would like to turn off plivo cases in V2.2 as well, thank you.
Attachment #8561872 -
Flags: review?(jlorenzo)
Comment 4•10 years ago
|
||
Comment on attachment 8561872 [details]
PR 28096 - merge to mozilla-b2g:v2.2 from echangmoz:1123652
https://github.com/mozilla-b2g/gaia/pull/28096
Updated•10 years ago
|
Attachment #8561872 -
Flags: review?(viorela.ioia)
Assignee | ||
Comment 5•10 years ago
|
||
Comment on attachment 8561872 [details]
PR 28096 - merge to mozilla-b2g:v2.2 from echangmoz:1123652
Having one review is okay in this case.
Attachment #8561872 -
Flags: review?(viorela.ioia)
Attachment #8561872 -
Flags: review?(jlorenzo)
Attachment #8561872 -
Flags: review+
Assignee | ||
Updated•10 years ago
|
Keywords: checkin-needed
Assignee | ||
Comment 6•10 years ago
|
||
Comment 7•10 years ago
|
||
(In reply to Johan Lorenzo [:jlorenzo] (QA) from comment #5)
> Comment on attachment 8561872 [details]
> PR 28096 - merge to mozilla-b2g:v2.2 from echangmoz:1123652
>
> Having one review is okay in this case.
But, if we can have a adhoc test, it will be better since it may cause compatibility problem. :)
Assignee | ||
Comment 8•10 years ago
|
||
(In reply to William Hsu [:whsu] from comment #7)
In the general case, I would say yes. But here's this is just a one-line change to deactivate a test. What kind of adhoc run do you want to have?
Flags: needinfo?(whsu)
Comment 9•10 years ago
|
||
(In reply to Johan Lorenzo [:jlorenzo] (QA) from comment #8)
> (In reply to William Hsu [:whsu] from comment #7)
> In the general case, I would say yes. But here's this is just a one-line
> change to deactivate a test. What kind of adhoc run do you want to have?
Okay! I see.
Many thanks!
Flags: needinfo?(whsu)
You need to log in
before you can comment on or make changes to this bug.
Description
•