Get back the test for Windows Live Mail settings importer

RESOLVED FIXED in Thunderbird 17.0

Status

MailNews Core
Import
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: hiro, Assigned: hiro)

Tracking

Trunk
Thunderbird 17.0
x86_64
Linux
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

5 years ago
mailnews/import/test/unit/test_winmail.js is now disabled but we need it.
(Assignee)

Comment 1

5 years ago
Created attachment 641346 [details] [diff] [review]
Proposed fix

This patch needs both the fixes for bug 773127 and 773124.
Attachment #641346 - Flags: review?(mconley)
(Assignee)

Updated

5 years ago
Depends on: 773124, 773127
(Assignee)

Comment 2

5 years ago
Unfortunately the test fails on the current trunk because the return value of GetValueForTag is not checked.

For example:

http://mxr.mozilla.org/comm-central/source/mailnews/import/winlivemail/nsWMSettings.cpp#500

500       GetValueForTag(xmlDoc, NS_LITERAL_STRING("Leave_Mail_On_Server"), value);
501       if (!value.IsEmpty())
502         pop3Server->SetLeaveMessagesOnServer((bool)value.ToInteger(&errorCode, 16));
503       GetValueForTag(xmlDoc, NS_LITERAL_STRING("Remove_When_Deleted"), value);
504       if (!value.IsEmpty())
505         pop3Server->SetDeleteMailLeftOnServer((bool)value.ToInteger(&errorCode, 16));

the value of "Remove_When_Deleted" is the previous one, i.e. "Leave_Mail_On_Server", if "Remove_When_Deleted" does not exist in registry data.

I will open a new bug for this issue.
(Assignee)

Comment 3

5 years ago
Created attachment 641350 [details] [diff] [review]
Fix

I forgot including the diff of xpcshell.ini.
Attachment #641346 - Attachment is obsolete: true
Attachment #641346 - Flags: review?(mconley)
Attachment #641350 - Flags: review?(mconley)
(Assignee)

Updated

5 years ago
Depends on: 773145
I'll wait for bug 773145 to land before reviewing this.
Comment on attachment 641350 [details] [diff] [review]
Fix

Sorry it took me so long to get to this!

This looks good. Thanks Hiro!
Attachment #641350 - Flags: review?(mconley) → review+
Keywords: checkin-needed
https://hg.mozilla.org/comm-central/rev/286425ebf6a5
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Flags: in-testsuite+
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 17.0
You need to log in before you can comment on or make changes to this bug.