Closed
Bug 449763
Opened 16 years ago
Closed 3 years ago
test_handlerApps.xhtml should not launch a real app
Categories
(Firefox :: File Handling, defect)
Tracking
()
RESOLVED
DUPLICATE
of bug 749872
People
(Reporter: jorendorff, Unassigned)
References
Details
uriloader/exthandler/tests/mochitest/test_handlerApps.xhtml opens a webcal: URL. On the Mac this opens iCal, which grabs the focus and prompts the user to subscribe to some bogus event.
This is kind of silly, obnoxious behavior for an automated test. It would be better to use a dummy app.
Back when we were first testing the web-application support, we used this test application: http://handler-test.mozilla.org/webcal?url=%s
It's still active. Do we want to use that instead?
Comment 2•16 years ago
|
||
The particular test Jason's mentioning is specifically to check that OS-native app launching works (as much as it's possible to check that).
Comment 4•13 years ago
|
||
Hope it doesn't matter to the test whether or not iCal is already running, since thanks to 10.7's annoying habit of reopening programs that were running at the time of a restart, I think (based on screenshots from test hangs in mochitest chunks that aren't the one with this test in it) that we are permanently running iCal on every 10.7 slave once they have run this test once.
Is there any less awkward way to kill it on our buildbot slaves than to have the http://mxr.mozilla.org/mozilla-central/source/uriloader/exthandler/tests/mochitest/handlerApps.js#115 chunk check for a pref that buildbot will set while starting mochitests?
Comment 5•13 years ago
|
||
Filed bug 743594 for ducking the problem by persuading the 10.7 slaves not to restore running apps.
Comment 6•13 years ago
|
||
(In reply to Phil Ringnalda (:philor) from comment #4)
> Is there any less awkward way to kill it on our buildbot slaves than to have
> the
> http://mxr.mozilla.org/mozilla-central/source/uriloader/exthandler/tests/
> mochitest/handlerApps.js#115 chunk check for a pref that buildbot will set
> while starting mochitests?
Ftr,
{
115 // if we ever decide that killing iCal is the right thing to do, change
116 // the if statement below from "NOTDarwin" to "Darwin"
117 if (osString == "NOTDarwin") {
...
}
Updated•8 years ago
|
Product: Core → Firefox
Version: Trunk → unspecified
Comment 7•3 years ago
|
||
I'm just removing this test in bug 749872 so duping forward to that bug.
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → DUPLICATE
Updated•3 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•