Open
Bug 455678
Opened 16 years ago
Updated 4 years ago
do correct cleanup after idcheck test
Categories
(SeaMonkey :: MailNews: General, defect)
SeaMonkey
MailNews: General
Tracking
(Not tracked)
NEW
People
(Reporter: kairo, Assigned: mnyromyr)
References
(Depends on 1 open bug)
Details
In http://hg.mozilla.org/comm-central/rev/5e42e6537169, I commented out the ExitTest() function of the idcheck test, because it throws away all prefs but this makes all following test that need to open windows fail as the popup blocker stops them from opening their windows.
We should either look very carefully into this and decide we never need to clean up those mail prefs, or correctly go and only clean up those, maybe with adding a function to mailTestUtils.js that again uses the account manager to delete the account we created.
Reporter | ||
Updated•16 years ago
|
Assignee: nobody → mnyromyr
Assignee | ||
Comment 1•16 years ago
|
||
I'm not that convinced that the problem is actually test_idcheck.xul.
Ideally, the testing environment would be created as default pref values, so that every test can easily throw away everything it doesn't know about, eg. *before* it runs.
Currently, running (chrome) tests successfully depends upon the order the tests are run, because you can't be sure about the state of the system a specific tests gets! Luckily(!), this didn't break up to now (to our knowledge at least)...
Assignee | ||
Comment 2•16 years ago
|
||
That said, we should, of course, clean up as far as possible here as well...
You need to log in
before you can comment on or make changes to this bug.
Description
•