Our test coverage for ICE contacts within the emergency-call app needs to be audited as we suspect that there are some tests missing.

PR: https://github.com/mozilla-b2g/gaia/pull/23871 The test coverage was actually pretty good, but I didn't like how the file was structured. Each test now has its own sandboxed HTML.

Comment on attachment 8486613 [details] [diff] [review] Add missing tests to ICEContacts. Review of attachment 8486613 [details] [diff] [review]: ----------------------------------------------------------------- We're missing tests for our error screens. I'm still giving r+ because I think we can do this in bug 1060451. But maybe you want to write some in this patch. Up to you.

Comment on attachment 8486613 [details] [diff] [review] Add missing tests to ICEContacts. [Approval Request Comment] [Bug caused by] (feature/regressing bug #): Not a bug, asking for uplitfing since this bug just adds more unit tests, and having this bug in 2.1 will solve merge conflicts with bug 1061421 that has been already approved for 2.1 [User impact] if declined: Will need to rewrite the patch for 2.1. Not user, but developer inpacted ;P [Testing completed]: This bug adds more unit tests and have been in 2.2 for a while [Risk to taking this patch] (and alternatives if risky): Pretty low risk, checked with Dialer guys and adds more tests, which will increase confident in 2.1 release [String changes made]:

v2.1: https://github.com/mozilla-b2g/gaia/commit/d5d27995f2ec2079466b98312dee08505e55964a