Closed Bug 948858 Opened 7 years ago Closed 7 years ago

[UITest][HW] modify radio test for multi-SIM card device

Categories

(Firefox OS Graveyard :: Gaia, defect)

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: tjao, Assigned: tjao)

References

Details

Attachments

(1 file)

Since navigator.mozIccManager & navigator.mozMobileConnections are changed for multi-SIM support. The radio test in UITest app should also be fixed. Now it shows 'mozMobileConnection'(lack of 's') not support.
Assignee: nobody → tjao
Blocks: eng-mode
It seems that MSISDN in mozIccManager disappear and the document on MDN is not updated. So I decide to remove it in this bug.
(In reply to Tom Jao [:mbrsl] from comment #1)
> It seems that MSISDN in mozIccManager disappear and the document on MDN is
> not updated. So I decide to remove it in this bug.
Nope, most of the stuff moved to mozIcc object. It's actually the same.
Attachment #8358292 - Flags: review?(gasolin)
There may be some bugs in this PR since I can't test with multi-SIM phone.
I'll test it on real device once I got one...
Comment on attachment 8358292 [details] [review]
https://github.com/mozilla-b2g/gaia/pull/15191

works on device
Attachment #8358292 - Flags: review?(gasolin) → review+
merged to gaia-master https://github.com/mozilla-b2g/gaia/commit/0bc133e32567bcf6aea7a7bff3b5fb7cde08fdfd
thanks!
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.