Sisyphus - failure to install or initialize Spider should cause run to fail

RESOLVED FIXED

Status

Testing
Sisyphus
RESOLVED FIXED
11 years ago
10 years ago

People

(Reporter: bc, Assigned: bc)

Tracking

Trunk
x86
Linux
Points:
---
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Assignee)

Description

11 years ago
Currently if Spider fails to install, the install-extensions-*.html time out, or if the Spider check page fails with a time out the frame work will not terminate and will attempt to run even though Spider is not available. This will cause each test page to be loaded into a normal browser window which will not close on its own thus causing every test to time out and the test run to take _forever_. If Spider installation or initialization fails, then the test run should fail immediately.
Flags: in-testsuite-
Flags: in-litmus-
(Assignee)

Comment 1

11 years ago
Created attachment 292408 [details] [diff] [review]
JavaScript Tests patch
Attachment #292408 - Flags: review?(rcampbell)
(Assignee)

Comment 2

11 years ago
Created attachment 292409 [details] [diff] [review]
Sisyphus patch
Attachment #292409 - Flags: review?(rcampbell)
Attachment #292408 - Flags: review?(rcampbell) → review+
Comment on attachment 292409 [details] [diff] [review]
Sisyphus patch

these look good to me
Attachment #292409 - Flags: review?(rcampbell) → review+
(Assignee)

Comment 4

11 years ago
Checking in bin/install-extensions.sh;
/cvsroot/mozilla/testing/sisyphus/bin/install-extensions.sh,v  <--  install-extensions.sh
new revision: 1.4; previous revision: 1.3
done
Checking in bin/smoke-build.sh;
/cvsroot/mozilla/testing/sisyphus/bin/smoke-build.sh,v  <--  smoke-build.sh
new revision: 1.4; previous revision: 1.3
done
Checking in bin/test-setup.sh;
/cvsroot/mozilla/testing/sisyphus/bin/test-setup.sh,v  <--  test-setup.sh
new revision: 1.6; previous revision: 1.5
done
Checking in bin/tester.sh;
/cvsroot/mozilla/testing/sisyphus/bin/tester.sh,v  <--  tester.sh
new revision: 1.7; previous revision: 1.6
done
Checking in runtests.sh;
/cvsroot/mozilla/js/tests/runtests.sh,v  <--  runtests.sh
new revision: 1.8; previous revision: 1.7
done
Status: ASSIGNED → RESOLVED
Last Resolved: 11 years ago
Resolution: --- → FIXED
Mass move of Sisyphus bugs to Testing : Sisyphus. Filter on SisyphusMassMove to ignore.
Component: Testing → Sisyphus
Flags: in-litmus-
Product: Core → Testing
QA Contact: testing → sisyphus
You need to log in before you can comment on or make changes to this bug.