Closed Bug 968914 Opened 7 years ago Closed 7 years ago
Follow-up: Add unit tests for SIM Dialog Retry count
+++ This bug was initially created as a clone of Bug #966571 +++ We need to add integration tests for the fix in 966571.
Francisco, This will need to be re-run on travis once bug 966571 lands, but I wanted to get the ball rolling on your review in the meantime if possible.
Comment on attachment 8371613 [details] [review] Added unit tests Hi! Thanks for the unit tests! Just left a message in github to try to reduce the number of lines of the patch, perhaps using the setup and teardown methods of the suite. Flag me again if it's not possible to implement :), a part from that the PR is looking great!
Comment on attachment 8371613 [details] [review] Added unit tests Addressed you comments.
Comment on attachment 8371613 [details] [review] Added unit tests Awesome work! Will land the patch since today is holidays in US :)
Attachment #8371613 - Flags: review?(francisco.jordano) → review+
Landed on master: https://github.com/mozilla-b2g/gaia/commit/ca9fa63953b2fc8a4fdeb9a5bc09cf0f31944f3b
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.