Disable test_dialer_set_up_conference_call.py due to Bug 1117813

RESOLVED FIXED

Status

RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: viorela, Assigned: jlorenzo)

Tracking

unspecified
ARM
Gonk (Firefox OS)

Firefox Tracking Flags

(b2g-v2.2 fixed, b2g-master fixed)

Details

Attachments

(2 attachments)

(Reporter)

Description

4 years ago
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

4 years ago
Assignee: nobody → jlorenzo
Blocks: 1117813
QA Whiteboard: [fxosqa-auto-s8][fxosqa-auto-points=1]
(Assignee)

Comment 1

4 years ago
Created attachment 8551772 [details] [review]
Gaia PR
Attachment #8551772 - Flags: review?(viorela.ioia)
(Reporter)

Updated

4 years ago
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Created attachment 8561872 [details]
PR 28096 - merge to mozilla-b2g:v2.2 from echangmoz:1123652

Hi Johan, I would like to turn off plivo cases in V2.2 as well, thank you.
Attachment #8561872 - Flags: review?(jlorenzo)
Comment on attachment 8561872 [details]
PR 28096 - merge to mozilla-b2g:v2.2 from echangmoz:1123652

https://github.com/mozilla-b2g/gaia/pull/28096
Attachment #8561872 - Flags: review?(viorela.ioia)
(Assignee)

Comment 5

4 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

4 years ago
Keywords: checkin-needed
(Assignee)

Comment 6

4 years ago
Merged in 2.2 at: https://github.com/mozilla-b2g/gaia/commit/be5e0605c90b2ecc3fbf9d739d728273d81d4b7e
status-b2g-v2.2: --- → fixed
status-b2g-master: --- → fixed
Keywords: checkin-needed

Comment 7

4 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

4 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

4 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.