945669 - [email/POP3] Make FolderStorage.sliceOpenMostRecent not trigger connection establishment for POP3 folders other than the inbox; clean up tests

Status: RESOLVED FIXED

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

Description

[Andrew Sutherland [:asuth] (he/him)]

For bug 945544 I am implementing emptying the trash for POP3.  While writing the unit test I noticed that we are attempting to establish a connection when opening the trash folder.  This occurs as a result of Pop3FolderSyncer.sync using the lazyWithConnection wrapper.  Even though inside the actual sync implementation it knows to do nothing with the server, it's too late by that point.

Trivial fixes will be insufficient because the sync method does the horrible _TEST_pendingAdds and _perofrmTestDeletions logic in there which are essential for many of the tests shoe-horned to work with POP3.

I think there may be some bad efficiency issues since lazyWithConnection for sync passes getNew=true which means we will get the connection just to discard it in another folder.

Note that this bug does not affect localdrafts because it is specially handled by FolderStorage._sliceOpenMostRecent.  Stop-gap solutions may be possible by just making that method more aware of POP3's special-ness for the purposes of the trash folder.

I'm proposing we mark this as a 1.3 blocker since having the trash folder request a connection could interfere with the correct operation of pending inbox syncs, etc.  :mcav would be most qualified to make a determination of how serious a problem this is, though.

Comment 1

[Marcus Cavanaugh [:mcav]]

I think this can be addressed without too much fuss. I'll whip up a patch and we can take a look to make sure it addresses this fully.

Comment 2

[Marcus Cavanaugh [:mcav]]

Attachment: GELAM Pull Request

This patch would force lazyWithConnection to only establish a connection if the folder is the inbox. The title of this bug, though, seems to indicate that you thought a more involved fix was necessary -- so it's possible I'm missing something.

Comment 3

[Marcus Cavanaugh [:mcav]]

Landed:

https://github.com/mozilla-b2g/gaia-email-libs-and-more/commit/511b0dfeaf9785e96a34fbcbb4f9c44ceab5aeb5
https://github.com/mozilla-b2g/gaia/commit/a20a4aab8875139cc8511e707f9068a1ef781975

Comment 4

[Andrew Sutherland [:asuth] (he/him)]

(landed in 1.3 as non-feature bugfix prior to branching, no further triage required)