[email/IMAP] Ensure lastSyncedAt is not updated in the failure case

RESOLVED WONTFIX

Status

RESOLVED WONTFIX
5 years ago
11 months ago

People

(Reporter: asuth, Assigned: asuth)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Assignee)

Description

5 years ago
During investigation of bug 1018828 code reading suggests that we probably call markSyncRange in the IMAP case regardless of whether there is an error or not.  We definitely don't have a unit test that verifies lastSyncedAt does not change in the failure case.

I'll add a unit test and assuming that proves we are broken, I'll fix the bug.  If the test says we're fine, I'll add some comments so subsequent code reading does not get misled too.

POP3 may or may not be affected by something similar; I'm filing a bug for that one, and it may or may not be able to rely on new test coverage from this bug.

Comment 1

11 months ago
Firefox OS is not being worked on
Status: ASSIGNED → RESOLVED
Last Resolved: 11 months ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.