Note: There are a few cases of duplicates in user autocompletion which are being worked on.

Unhide some skipped xpcshell-tests

RESOLVED FIXED in Thunderbird 17.0

Status

MailNews Core
Testing Infrastructure
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: standard8, Assigned: standard8)

Tracking

Thunderbird 17.0
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

Created attachment 651523 [details] [diff] [review]
The fix

We have various xpcshell tests that are "skipped" by doing something like function run_test() { return; ... }

We should convert those to use skip-if as a) it exposes the fact they are skipped to the unit test logs, and b) we get those tests run ever so slightly faster as we don't have to load xpcshell, the python script can just dump the skip notification.

Attached is a patch of the ones I found, I went through all the tests, so I should have all of them.
Attachment #651523 - Flags: review?(Pidgeot18)
Comment on attachment 651523 [details] [diff] [review]
The fix

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

::: mailnews/imap/test/unit/xpcshell.ini
@@ +45,5 @@
>  [test_mailboxes.js]
>  [test_nsIMsgFolderListenerIMAP.js]
>  [test_offlinePlayback.js]
>  [test_offlineStoreLocking.js]
> +skip-if = os == "win"

Is there a bug filed to fix this test on windows? If not, please file one and note it in a comment here.

::: mailnews/news/test/unit/xpcshell.ini
@@ +11,5 @@
>  [test_getNewsMessage.js]
>  [test_internalUris.js]
>  [test_nntpContentLength.js]
> +# The server doesn't support returning sizes!
> +skip-if = true

Ditto here.
Attachment #651523 - Flags: review?(Pidgeot18) → review+
Created attachment 651746 [details] [diff] [review]
The fix v2

Updated patch with comments and for the fact that bug 772712 just landed, removing the need to skip that test now.
Attachment #651523 - Attachment is obsolete: true
Checked in: https://hg.mozilla.org/comm-central/rev/f0e8916a9ad1
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 17.0
You need to log in before you can comment on or make changes to this bug.