Closed Bug 861520 Opened 12 years ago Closed 11 years ago

Test failure "Timeout exceeded for waitForElement ID: useSSL3" in testPreferences/testDefaultSecurityPrefs.js

Categories

(Mozilla QA Graveyard :: Mozmill Tests, defect, P2)

defect

Tracking

(firefox22 unaffected, firefox23 fixed)

RESOLVED FIXED
Tracking Status
firefox22 --- unaffected
firefox23 --- fixed

People

(Reporter: AndreeaMatei, Assigned: AndreeaMatei)

References

()

Details

(Keywords: regression, Whiteboard: [mozmill-test-failure][sprint2013-31])

Attachments

(3 files)

This started today and it has another one, related to the same id, for Windows: TimeoutError: Timeout exceeded for waitForElement ID: useSSL3 - got 'false' Happens only on nightly, all platforms/locales.
Whiteboard: [mozmill-test-failure]
The UI has been changed for that tab in Prefs -> Advanced -> Certificates, in bug 733632. Here you can see the change before/after: https://bug733632.bugzilla.mozilla.org/attachment.cgi?id=603591 By default now is checked 'Ask me every time' radio button. We should adapt our test. I'll prepare a patch to stop the errors.
Attached patch skip test (deleted) — Splinter Review
Adding skip patch for now because I can't test my fix on all platforms, I have only Ubuntu at home (and at best Windows).
Assignee: nobody → andreea.matei
Status: NEW → ASSIGNED
Attachment #737138 - Flags: review?(hskupin)
Attachment #737138 - Flags: review?(dave.hunt)
Attachment #737138 - Flags: review?(hskupin)
Attachment #737138 - Flags: review?(dave.hunt)
Attachment #737138 - Flags: review+
Blocks: 733632
Keywords: regression
Priority: -- → P2
Whiteboard: [mozmill-test-failure] → [mozmill-test-failure][mozmill-test-skipped]
Whiteboard: [mozmill-test-failure][mozmill-test-skipped] → [mozmill-test-failure][mozmill-test-skipped][sprint2013-31]
Attached patch patch v1 (deleted) — Splinter Review
The fix I mentioned above would have tested the other 2 radio buttons: 'Ask me every time' and 'Select one automatically'. But giving that the prefs have been removed and the test was only about testing those, the patch is removing it. If we want to consider testing those radio buttons, I'll file another bug for it.
Attachment #747383 - Flags: review?(hskupin)
Comment on attachment 747383 [details] [diff] [review] patch v1 Review of attachment 747383 [details] [diff] [review]: ----------------------------------------------------------------- Sounds good to me. Please land it.
Attachment #747383 - Flags: review?(hskupin)
Attachment #747383 - Flags: review+
Attachment #747383 - Flags: checkin?(andreea.matei)
Comment on attachment 747383 [details] [diff] [review] patch v1 Review of attachment 747383 [details] [diff] [review]: ----------------------------------------------------------------- Landed: http://hg.mozilla.org/qa/mozmill-tests/rev/9281cf5d8b53 (default)
Attachment #747383 - Flags: checkin?(andreea.matei) → checkin+
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Whiteboard: [mozmill-test-failure][mozmill-test-skipped][sprint2013-31] → [mozmill-test-failure][sprint2013-31]
Due to some merge issues regarding this test, in bug 899629, I'm reopening this to add a release patch and fix it.
Blocks: 899629
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Attached patch [release] removeTest.patch (deleted) — Splinter Review
The manifest already had the test removed, so I only deleted the test. Giving that Henrik and Dave are in PTO and can't review this, I'm going to land it, it's just removing the test. http://hg.mozilla.org/qa/mozmill-tests/rev/c7b4352add39 (release)
Attachment #783612 - Flags: checkin+
Status: REOPENED → RESOLVED
Closed: 12 years ago11 years ago
Resolution: --- → FIXED
This was still happening in spite of it being checked in. I had to force Mozmill-CI to pull an updated copy of mozilla-release. It seems to have resolved this issue now. Again, sorry for the churn.
Product: Mozilla QA → Mozilla QA Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: