1597095 - TEST-UNEXPECTED-TIMEOUT | toolkit/components/crashes/tests/xpcshell/test_crash_service.js, TEST-UNEXPECTED-TIMEOUT | toolkit/crashreporter/test/unit_ipc/test_content_annotation.js and more (Mac only)

Status: RESOLVED FIXED

(Thunderbird :: General, defect)

Description

[Jorg K (CEST = GMT+2)]

TEST-UNEXPECTED-TIMEOUT | toolkit/components/crashes/tests/xpcshell/test_crash_service.js | Test timed out
EST-UNEXPECTED-TIMEOUT | toolkit/crashreporter/test/unit_ipc/test_content_rust_panic.js | Test timed out
TEST-UNEXPECTED-TIMEOUT | toolkit/crashreporter/test/unit_ipc/test_content_exception_time_annotation.js | Test timed out
TEST-UNEXPECTED-TIMEOUT | toolkit/crashreporter/test/unit_ipc/test_content_annotation.js | Test timed out
TEST-UNEXPECTED-TIMEOUT | toolkit/crashreporter/test/unit_ipc/test_content_rust_panic_multiline.js | Test timed out

First seen here:
https://treeherder.mozilla.org/#/jobs?repo=comm-central&revision=f5213fb4cd253d07da06753c9512137e3de8f681

https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=88db9bea4580df16dc444668f8c2cddbb3&tochange=1e1617c67238dfb685fb3d07bf1793232c

Geoff, can you see anything in that range?

Comment 1

[Geoff Lankow (:darktrojan)]

Looks like bug 1516367.

Comment 2

[Jorg K (CEST = GMT+2)]

We're still packaging it:
https://searchfox.org/comm-central/rev/496170162015a8c4a4087c18479279b4ce3b2cbf/mail/installer/package-manifest.in#498
I guess we should fix this and move it here:
https://searchfox.org/comm-central/source/mail/app/macbuild/Contents/MacOS-files.in

Looking at the file, that would also explain why gTests don't work on Mac, bug 1581762.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/c9fb9ef25bbb
Port bug 1581762: Move minidump-analyzer packaging. rs=bustage-fix

Comment 4

[Jorg K (CEST = GMT+2)]

I hope this works.

Comment 5

[Jorg K (CEST = GMT+2)]

Didn't work :-( - Gabriele, could you please take a look.

Comment 6

[Jorg K (CEST = GMT+2)]

Grrr, I missed this bit: https://hg.mozilla.org/mozilla-central/rev/5df0e903d13f#l2.23

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/a1618ce2c50c
Follow-up: Add hunk missing from rev c9fb9ef25bbb. rs=bustage-fix

Comment 8

[Jorg K (CEST = GMT+2)]

That did the trick.

Comment 9

[Gabriele Svelto [:gsvelto]]

Good to know this is fixed, sorry for the bustage.

Comment 10

[Jorg K (CEST = GMT+2)]

No problem. Usually we watch changes to package-manifest.in, I don't know how this one slipped through. And messing up the fix was my fault :-(