Closed Bug 1346916 Opened 8 years ago Closed 8 years ago

TEST-UNEXPECTED-FAIL | xpcshell-unpack.ini:toolkit/mozapps/extensions/test/xpcshell/test_reload.js

Categories

(Thunderbird :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 55.0

People

(Reporter: jorgk-bmo, Assigned: jorgk-bmo)

References

(Blocks 1 open bug)

Details

(Whiteboard: [Thunderbird-testfailure: X Linux/Mac])

Attachments

(1 file, 2 obsolete files)

First seen Mon Mar 13, 2017, 15:55:58: https://treeherder.mozilla.org/#/jobs?repo=comm-central&revision=cbf7af49eabc6fa153cf7d0da1bc3fd74d689b1b TEST-UNEXPECTED-FAIL | xpcshell-unpack.ini:toolkit/mozapps/extensions/test/xpcshell/test_reload.js | xpcshell return code: 0 TEST-UNEXPECTED-FAIL | xpcshell-unpack.ini:toolkit/mozapps/extensions/test/xpcshell/test_reload.js | test_reloading_a_temp_addon - [test_reloading_a_temp_addon : 224] A promise chain failed to handle a rejection: A mutation operation was attempted on a database that did not allow mutations. - rejection date: Mon Mar 13 2017 10:25:20 GMT-0700 (PDT) - stack: _do_main@/builds/slave/test/build/tests/xpcshell/head.js:211:5 TEST-UNEXPECTED-FAIL | xpcshell.ini:toolkit/mozapps/extensions/test/xpcshell/test_reload.js | xpcshell return code: 0 TEST-UNEXPECTED-FAIL | xpcshell.ini:toolkit/mozapps/extensions/test/xpcshell/test_reload.js | test_reloading_a_temp_addon - [test_reloading_a_temp_addon : 224] A promise chain failed to handle a rejection: A mutation operation was attempted on a database that did not allow mutations. - rejection date: Mon Mar 13 2017 10:25:22 GMT-0700 (PDT) - stack: _do_main@/builds/slave/test/build/tests/xpcshell/head.js:211:5 Looks like Bug 1344590: Part 1 - Handle extension reloads, upgrades, and manager restarts in xpcshell or Bug 1344590: Part 4 - Store parsed and normalized extension data in indexedDB https://hg.mozilla.org/mozilla-central/rev/d580a74eae4e Kris, another (web?) extension related test that fails in TB, so far strangely only on Linux and Mac. Could you please advise us here. Should we just disable the test?
Flags: needinfo?(kmaglione+bmo)
Blocks: TBWebExtFail
Only fails on Linux and Mac. Also adding NI for Andrew.
Flags: needinfo?(aswan)
Whiteboard: [Thunderbird-testfailure: X all] → [Thunderbird-testfailure: X Linux/Mac]
To state the obvious, this test is passing on mozilla-central so something is different in Thunderbird. I'm not familiar enough with Thunderbird to know what's different that could be responsible. This particular test uses webextensions though, if Thunderbird doesn't support webextensions, disabling that test might be your best bet.
Flags: needinfo?(aswan)
Attached patch 1346916-disable-test-tb.patch (obsolete) (deleted) — Splinter Review
Thanks for your reply. "Best bet" approach ;-)
Assignee: nobody → jorgk
Status: NEW → ASSIGNED
Flags: needinfo?(kmaglione+bmo)
Attachment #8848397 - Flags: review?(aswan)
Comment on attachment 8848397 [details] [diff] [review] 1346916-disable-test-tb.patch Review of attachment 8848397 [details] [diff] [review]: ----------------------------------------------------------------- This is fine if its what you want to do but this file contains other tests that you may not want to disable, I was thinking you would just disable the webextensions specific test cases inside this file.
Attachment #8848397 - Flags: review?(aswan) → review+
Attached patch 1346916-skip-test.patch (obsolete) (deleted) — Splinter Review
For the record: I prepared this patch, but for some reason my try run hasn't gone anywhere yet :-( https://treeherder.mozilla.org/#/jobs?repo=try-comm-central&revision=30f49b89af21210e84d7ccf745c566c5b5c220b3
Attached patch 1346916-skip-test.patch (deleted) — Splinter Review
Oops, wrong patch.
Attachment #8849578 - Attachment is obsolete: true
Comment on attachment 8849579 [details] [diff] [review] 1346916-skip-test.patch Thanks for your suggestion, Andrew. I finally managed to get a try run through: https://treeherder.mozilla.org/#/jobs?repo=try-comm-central&revision=f883d29f1585772b9c13acd8c27bf81fbc0f5e50 You can't see the changeset since that was pushed earlier on a job that never completed due to some infra problems today. Anyway, this just skips the part of the test that doesn't work.
Attachment #8849579 - Flags: review?(aswan)
Attachment #8849579 - Flags: review?(aswan) → review+
Attachment #8848397 - Attachment is obsolete: true
Andrew, thanks for the quick review. === Hi Tomcat, this needs to be landed on M-C/M-I. I didn't want to move the bug to an M-C product/component, so I'm setting NI so you can pick it up. Thank you in advance.
Flags: needinfo?(cbook)
Keywords: checkin-needed
Pushed by cbook@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/eb0321aaeca4 Skip test_reload.js:test_reloading_a_temp_addon() for Thunderbird. a=aswan
Keywords: checkin-needed
pushed :)
Flags: needinfo?(cbook)
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 55.0
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: