Closed
Bug 907101
Opened 11 years ago
Closed 11 years ago
Test failure "Certificate's country code is displayed - '(US)' should equal '(US,postalCode=94041,STREET=650 Castro St Ste 300)'" in testGreenLarry.js
Categories
(Mozilla QA Graveyard :: Mozmill Tests, defect, P1)
Mozilla QA Graveyard
Mozmill Tests
Tracking
(firefox23 fixed, firefox24 fixed, firefox25 fixed, firefox26 fixed, firefox-esr17 fixed)
People
(Reporter: AndreeaMatei, Assigned: cosmin-malutan)
References
()
Details
(Whiteboard: [mozmill-test-failure])
Attachments
(1 file, 1 obsolete file)
|
2.71 KB,
patch
|
AndreeaMatei
:
review+
|
Details | Diff | Splinter Review |
Started today to fail on Beta, all platforms, with en-US. I can reproduce with Nightly as well, Cosmin will add the fix in bug 905960 as he already was working there on the same test.
|
Reporter | |
Updated•11 years ago
|
status-firefox23:
--- → affected
status-firefox24:
--- → affected
status-firefox25:
--- → affected
status-firefox26:
--- → affected
status-firefox-esr17:
--- → affected
OS: Linux → All
Priority: -- → P1
Hardware: x86_64 → All
|
|
Reporter | |
Updated•11 years ago
|
Assignee: nobody → cosmin.malutan
Status: NEW → ASSIGNED
|
|
Reporter | |
Updated•11 years ago
|
Summary: Test failure "Certificate's country code is displayed - '(US)' should equal '(US,postalCode=94041,STREET=650 Castro St Ste 300)'" in testGreyLarry.js → Test failure "Certificate's country code is displayed - '(US)' should equal '(US,postalCode=94041,STREET=650 Castro St Ste 300)'" in testGreenLarry.js
|
||
Comment 1•11 years ago
|
||
How is this related to bug 905960? I do not see the connection between those two bugs. Can you please explain?
|
|
||
Comment 2•11 years ago
|
||
As the cert info for https://addons.mozilla.org shows me a new certificate has been issued here. So all branches will be affected. Please handle that separately from bug 905960.
No longer depends on: 905960
|
Assignee | |
Comment 3•11 years ago
|
||
This bug is produced because the certificate has been changed, so I had to update a string, so we extract the state correctly from certificate. http://mozmill-crowd.blargon7.com/#/functional/report/e503b7b0a70a3839c66c64572e8da873 http://mozmill-crowd.blargon7.com/#/functional/report/e503b7b0a70a3839c66c64572e8b6a78 http://mozmill-crowd.blargon7.com/#/functional/report/e503b7b0a70a3839c66c64572e9148a2
Attachment #792803 -
Flags: review?(andreea.matei)
|
|
Reporter | |
Comment 4•11 years ago
|
||
Comment on attachment 792803 [details] [diff] [review] patch_v1.0 Review of attachment 792803 [details] [diff] [review]: ----------------------------------------------------------------- Looks good. Please just quickly update the username and make this patch for default first. Thanks!
Attachment #792803 -
Flags: review?(andreea.matei) → review-
|
|
Assignee | |
Comment 5•11 years ago
|
||
Sorry for that I didn't changed my .hgrc file on mac.
Attachment #792803 -
Attachment is obsolete: true
Attachment #792805 -
Flags: review?(andreea.matei)
|
|
Reporter | |
Comment 6•11 years ago
|
||
Comment on attachment 792805 [details] [diff] [review] patch_v1.1 Review of attachment 792805 [details] [diff] [review]: ----------------------------------------------------------------- Landed on default: http://hg.mozilla.org/qa/mozmill-tests/rev/59171fbd5f02 (default) We need backporting, please be aware of the commit message.
Attachment #792805 -
Flags: review?(andreea.matei) → review+
|
|
Reporter | |
Updated•11 years ago
|
|
|
Assignee | |
Comment 7•11 years ago
|
||
It backports cleanly on all branches.
|
|
Reporter | |
Comment 8•11 years ago
|
||
Transplanted: http://hg.mozilla.org/qa/mozmill-tests/rev/5fedde087e2a (aurora) http://hg.mozilla.org/qa/mozmill-tests/rev/856bb01851f3 (beta) http://hg.mozilla.org/qa/mozmill-tests/rev/d24b428d66b7 (release) http://hg.mozilla.org/qa/mozmill-tests/rev/106cedaa5f7c (esr17) Thanks Cosmin!
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
|
||
Updated•5 years ago
|
Product: Mozilla QA → Mozilla QA Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•