ReferenceError: gCurrStatus is not defined in various mailnews xpcshell-tests

RESOLVED FIXED

Status

RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: standard8, Assigned: standard8)

Tracking

Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(1 attachment)

I think this has been highlighted by recent changes to do_timeout (bug 586692) - meaning that errors thrown within do_timeout callbacks are actually caught and handled properly as test failures.

This has shown that there's a set of mailnews tests that are using a big timeout in case the test fails, and try to report "gCurrStatus" but that isn't actually defined for those tests...
Assignee: nobody → bugzilla
Created attachment 466282 [details] [diff] [review]
The fix

There's been too much copy and paste here ;-) All of these don't use the file that gives them gCurrStatus.
Attachment #466282 - Flags: review?(bienvenu)

Updated

8 years ago
Attachment #466282 - Flags: review?(bienvenu) → review+
Checked in: http://hg.mozilla.org/comm-central/rev/6468ea1719e6
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.