Closed Bug 431453 Opened 14 years ago Closed 14 years ago

Make browser mochitests pass on SeaMonkey

Categories

(SeaMonkey :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: kairo, Assigned: kairo)

References

Details

Attachments

(1 file)

We currently have a number of failing browser mochitests on SeaMonkey, for the following reasons:

FUEL is Firefox-only:
chrome://mochikit/content/browser/docshell/test/browser/browser_bug134911.js
chrome://mochikit/content/browser/docshell/test/browser/browser_bug92473.js

PageProxySetIcon() function is Firefox-only:
chrome://mochikit/content/browser/extensions/cookie/test/browser_test_favicon.js

history.addPageWithDetails is not defined in xpfe history:
chrome://mochikit/content/browser/suite/modules/test/browser_sanitizer.js

SeaMonkey doesn't use toolkit download manager:
chrome://mochikit/content/browser/toolkit/mozapps/downloads/tests/browser/* (14 failures on Linux, 13 on Windows)

Unknown:
FAIL - Cookie test setup successfully - chrome://mochikit/content/browser/suite/modules/test/browser_sanitizer.js


We should find out some way to make SeaMonkey not fail those tests. This might mean making some files not copied to tests on other apps than Firefox or manipulating how tests perform thing so that SeaMonkey can pass them, or even fix SeaMonkey to pass those tests.
the toolkit download manager stuff will still (likely) fail if we use our own UI for the download manager (including progress dialogs).

Solution would be for us to write our own tests mimicking the functionality tested in those toolkit ones, (and disabling those toolkit/ ones somehow of course).
This patch makes the tests that use Firefox-only features be installed for Firefox only, the download manager tests to be excluded for SeaMonkey just like the source (currently, we can figure out details when we switch to the toolkit dlmgr) and fixes the sanitizer tests.

Neil, can you r/sr the sanitizer changes?
The first change there is to make us accept 3rd-party cookies for the cookie test, so setting them works.
The second change is the only solution I could find to make the history test work with xpfe history: remove all cookies and just check the count once we have set one. When the more advanced interfaces of toolkit are found, we use them instead.

smontagu, can you please r making the docshell tests you added Firefox-only?

I'll add additional reviewers for cookie and downloads in a minute.
Assignee: general → kairo
Status: NEW → ASSIGNED
Attachment #318695 - Flags: superreview?(neil)
Attachment #318695 - Flags: review?(smontagu)
Comment on attachment 318695 [details] [diff] [review]
patch: make a few tests FF-only or non-SM, fix sanitizer tests

dwitte, can you review making the favicon cookie test Firefox-only?
Attachment #318695 - Flags: review?(dwitte)
Comment on attachment 318695 [details] [diff] [review]
patch: make a few tests FF-only or non-SM, fix sanitizer tests

gavin, can you please review excluding the dlmgr tests for SeaMonkey as well for now?
Attachment #318695 - Flags: review?(gavin.sharp)
Comment on attachment 318695 [details] [diff] [review]
patch: make a few tests FF-only or non-SM, fix sanitizer tests

r=me on the toolkit/mozapps/downloads/Makefile.in
 change.
Attachment #318695 - Flags: review?(gavin.sharp) → review+
Comment on attachment 318695 [details] [diff] [review]
patch: make a few tests FF-only or non-SM, fix sanitizer tests

>Index: mozilla/extensions/cookie/test/Makefile.in
>===================================================================
>+# see above, only defined for Fireofx

r=me provided you spell Firefox right ;)
Attachment #318695 - Flags: review?(dwitte) → review+
Comment on attachment 318695 [details] [diff] [review]
patch: make a few tests FF-only or non-SM, fix sanitizer tests

r=me on docshell/test/browser/Makefile.in
Attachment #318695 - Flags: review?(smontagu) → review+
Attachment #318695 - Flags: superreview?(neil) → superreview+
Comment on attachment 318695 [details] [diff] [review]
patch: make a few tests FF-only or non-SM, fix sanitizer tests

requesting a1.9 - all this does is exclude some tests from being installed in non-Firefox or in SeaMonkey builds (apart from the suite/ part that fixes tests but is NPOTB for FF).
Attachment #318695 - Flags: approval1.9?
Comment on attachment 318695 [details] [diff] [review]
patch: make a few tests FF-only or non-SM, fix sanitizer tests

a1.9+=damons
Attachment #318695 - Flags: approval1.9? → approval1.9+
Checked in, thanks everybody!
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.