Closed Bug 1086269 Opened 5 years ago Closed 5 years ago

Add a test to set up a conference call


(Firefox OS Graveyard :: Gaia::UI Tests, defect, P2)

Gonk (Firefox OS)


(b2g-v2.2 affected)

Tracking Status
b2g-v2.2 --- affected


(Reporter: jlorenzo, Assigned: jlorenzo)




(2 files)

Bug 1005862 and bug 1083402 are regressions on the same piece of code in the Dialer. We don't cover conference calls with automation. Let's fix that.

Then we would be able to check the regression on the notification in another bug.
Attached file Gaia PR
Asking Zac for review because I added a new helper to kill active calls in
Attachment #8508732 - Flags: review?(zcampbell)
Attachment #8508732 - Flags: review?(florin.strugariu)
(In reply to Johan Lorenzo [:jlorenzo] (QA) from comment #1)
> I added a new helper to kill active calls
I meant active conference calls. I'm using the same loop as they do in the callscreen:
Comment on attachment 8508732 [details] [review]
Gaia PR

r+ but I'm relying on the adhoc job to assert the functionality as I don't have a local Plivo account.
Attachment #8508732 - Flags: review?(zcampbell) → review+
Comment on attachment 8508732 [details] [review]
Gaia PR

r+, nice test!
Attachment #8508732 - Flags: review+
btw Johan, is this a smoketest? in that case you need to add it to the manifest files.
Comment on attachment 8508732 [details] [review]
Gaia PR

No, it's not. This test doesn't appear in this suite:

I added the test in the manifest file. My PR is ready to merge then.
Attachment #8508732 - Flags: review?(florin.strugariu)
Is it possible to merge it?
Flags: needinfo?(viorela.ioia)
Merged in:

I'll leave this bug opened as the patch needs uplift to v2.1. Johan, can you uplift this? Thanks!
Flags: needinfo?(viorela.ioia)
Flags: needinfo?(jlorenzo)
Attached file Gaia PR 2.1
Tested on 2.1 and passing.
Flags: needinfo?(jlorenzo)
Attachment #8514437 - Flags: review?(viorela.ioia)
Closed: 5 years ago
Resolution: --- → FIXED
Attachment #8514437 - Flags: review?(viorela.ioia) → review+
QA Whiteboard: [fxosqa-auto-s2]
You need to log in before you can comment on or make changes to this bug.