1322072 - Port glue-dependent mailnews tests to gtest

Status: RESOLVED FIXED

(MailNews Core :: Testing Infrastructure, defect)

Description

[aleth [:aleth]]

Something similar to Bug 1313752 should work here too.

Comment 1

[aleth [:aleth]]

Attachment: Port TestMailCookie to gtest

This should work... in principle. However, cppunittests aren't running in automation for TB, and neither are gtests. If that's the case, and nobody's been running this test for ages, maybe we should just stop building it?

Comment 2

[aleth [:aleth]]

Attachment: Remove Thunderbird XPCOM glue exceptions as they are no longer required

Comment 3

[aleth [:aleth]]

https://treeherder.mozilla.org/#/jobs?repo=try-comm-central&revision=6ee87a6d22d41f5472b4ae35580bdbfb6aa8c5a7

Comment 4

[aleth [:aleth]]

Comment on attachment 8816823 [details] [diff] [review]
Port TestMailCookie to gtest

Review of attachment 8816823 [details] [diff] [review]:
-----------------------------------------------------------------

Let's take this for now and we can reenable the test at a later stage.

Comment 5

[aleth [:aleth]]

Attachment: Remove Thunderbird XPCOM glue exceptions as they are no longer required

Comment 6

[Jorg K (CEST = GMT+2)]

https://hg.mozilla.org/comm-central/rev/89cfdf12b27260e9e6a8ed14b4c41066c1a1cc0c
Sorry, I had to rs this to fix immediate bustage.

Comment 7

[Jorg K (CEST = GMT+2)]

Aleth, can you please do what's left to do here, like get the M-C part landed.

Comment 8

[aleth [:aleth]]

(In reply to Jorg K (GMT+1) from comment #7)
> Aleth, can you please do what's left to do here, like get the M-C part
> landed.

The m-c part would have been required if this had landed before the XPCOM glue removal. Now it's gone, there's no exceptions any more...