Closed Bug 1035576 Opened 10 years ago Closed 6 years ago

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

Categories

(Firefox OS Graveyard :: Gaia::E-Mail, defect)

Product:
Firefox OS Graveyard
Component:
Gaia::E-Mail
Platform:
x86_64
Linux
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED WONTFIX

People

(Reporter: asuth, Assigned: asuth)

References

Details

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.
Firefox OS is not being worked on
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.