Closed
Bug 761304
Opened 12 years ago
Closed 3 years ago
mailnews/addrbook/test/unit/test_nsIAbCard.js has weak tests due to divergence of nsIAbCard between Thunderbird and SeaMonkey
Categories
(Thunderbird :: Address Book, defect)
Thunderbird
Address Book
Tracking
(thunderbird15 fixed)
RESOLVED
WORKSFORME
Thunderbird 16.0
Tracking | Status | |
---|---|---|
thunderbird15 | --- | fixed |
People
(Reporter: florian, Assigned: mconley)
References
Details
Attachments
(1 file, 1 obsolete file)
16.90 KB,
patch
|
florian
:
review+
iannbugzilla
:
review+
standard8
:
approval-comm-aurora+
|
Details | Diff | Splinter Review |
The test at mailnews/addrbook/test/unit/test_nsIAbCard.js is broken since I landed bug 759238. The cause is obviously at http://mxr.mozilla.org/comm-central/source/mailnews/addrbook/test/unit/test_nsIAbCard.js#45 There's an hard coded XML result of the conversion of an address book card containing an AIM screen name.
Reporter | ||
Comment 2•12 years ago
|
||
This make the test pass for Thunderbird, but would break the same test for SeaMonkey, as the C++ code contains ifdefs.
Assignee | ||
Comment 3•12 years ago
|
||
This patch wallpaper over the failure by ensuring that the XML output for translateTo("xml") and translateTo("base64xml") match our expectations for Thunderbird OR SeaMonkey. We're doing it this way because it's hard in XPCShell tests to determine which app we're in, and we're not able to do any preprocessing. Hopefully we'll come up with a more graceful solution that doesn't weaken our tests after this lands, but the priority is to have a non-busted tree. Ian: I don't have a SeaMonkey build handy. Do the nsIAbCard tests pass for you with this patch?
Attachment #629910 -
Attachment is obsolete: true
Attachment #629929 -
Flags: review?(iann_bugzilla)
Attachment #629929 -
Flags: review?(florian)
Reporter | ||
Comment 4•12 years ago
|
||
Comment on attachment 629929 [details] [diff] [review] Wallpaper patch v1 This is OK with me. Thanks for taking care of this!
Attachment #629929 -
Flags: review?(florian) → review+
Comment on attachment 629929 [details] [diff] [review] Wallpaper patch v1 Test works both with and without the patch for SM (which is what is expected), so r=me
Attachment #629929 -
Flags: review?(iann_bugzilla) → review+
Assignee | ||
Comment 6•12 years ago
|
||
Thanks for the reviews - committed to comm-central as https://hg.mozilla.org/comm-central/rev/51f29333aafd. Waiting for comm-aurora to open before I push there.
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 16.0
Comment 7•12 years ago
|
||
Comment on attachment 629929 [details] [diff] [review] Wallpaper patch v1 [Triage Comment] a=me for bustage fix.
Attachment #629929 -
Flags: approval-comm-aurora+
Comment 8•12 years ago
|
||
Checked in: https://hg.mozilla.org/releases/comm-aurora/rev/9dcedc936a8e
status-thunderbird15:
--- → fixed
Assignee | ||
Comment 9•12 years ago
|
||
Not sure why I marked this RESOLVED FIXED, since the comments in my patch mention that a more thorough fix will come from this bug.
Status: RESOLVED → VERIFIED
Summary: mailnews/addrbook/test/unit/test_nsIAbCard.js fails because of the changes to the way AIM screen names are printed → mailnews/addrbook/test/unit/test_nsIAbCard.js has weak tests due to divergence of nsIAbCard between Thunderbird and SeaMonkey
Assignee | ||
Updated•12 years ago
|
Status: VERIFIED → REOPENED
Resolution: FIXED → ---
Comment 11•3 years ago
|
||
I think the point still stands but I'm not that bothered by it. Ironically this test has just started failing in this exact spot.
Status: REOPENED → RESOLVED
Closed: 12 years ago → 3 years ago
Flags: needinfo?(geoff)
Resolution: --- → FIXED
Updated•3 years ago
|
Resolution: FIXED → WORKSFORME
You need to log in
before you can comment on or make changes to this bug.
Description
•