Closed Bug 1034637 Opened 5 years ago Closed 5 years ago
.js unit tests are failing when run locally
Once patch for bug 974867 has been landed some test suites inside utils_test.js started to depend on MockSettings that is not properly set up for these suites.
Comment on attachment 8451025 [details] [review] GitHub pull request URL Hey Julien, Here is tiny patch that adds mocks initialization to the utils test suites that require it. Also it seems I found a reason why these tests are passing locally with newer version of FF - something has changed in Gecko recently so that 'navigator.mozPhoneNumberService' is no longer accessible and FixturePhones object isn't initialized that lead to partial execution of utils_test.js tests (when test env analyzes what suites to run in this file, it encounters error in 'Varied Cases' suite and all next suites are just skipped). I'm not sure why this situation isn't reported by test env as failure. I've fixed that too.
Attachment #8451025 - Flags: review?(felash)
Comment on attachment 8451025 [details] [review] GitHub pull request URL r=me thanks
Attachment #8451025 - Flags: review?(felash) → review+
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.