hpi-swa-teaching / IMAPClient

This project provides a library and tool set to access email through IMAP in the Squeak/Smalltalk environment. (SWT22-13)
MIT License
22 stars 3 forks source link

Access to email account in offline mode #413

Closed elenagensch closed 3 years ago

elenagensch commented 3 years ago

What does this change?

[Feature] Adds the possibility to access offline mode of IMAP Client without being logged in.

Splits Login Process from ICFolderDialog initialization. Emails and Folders are loaded from File Storage (structure from last synchronization). Uses the UIManager for password request. Layered structure for calls from UI to interface with returns implemented. Bugs for login fixed (fatal error extracted from login process). Closes #409 and closes #410

Co-authored-by: Elena Gensch elena.gensch@student.hpi.de Co-authored-by: Jan Groeneveld jan.niklas.groeneveld@gmail.com

Decisions / Choices I made

We removed the Password Dialog, as it oppposes the layered structure of handling requests and there's a lot of logic (passing arround FolderDialog instance, defining button) just for retrieving the password. Uses UIManager instead.

Checklist

coveralls commented 3 years ago

Pull Request Test Coverage Report for Build 1008403859


Changes Missing Coverage Covered Lines Changed/Added Lines %
packages/IMAPClient-Core.package/ICConnectionHandler.class/instance/select..st 0 1 0.0%
packages/IMAPClient-Core.package/ICEndPoint.class/instance/connectAndLogin.st 0 3 0.0%
packages/IMAPClient-Core.package/ICEndPoint.class/instance/readEmailsFromDisk.st 0 4 0.0%
<!-- Total: 16 24 66.67% -->
Files with Coverage Reduction New Missed Lines %
packages/IMAPClient-Core.package/ICConnectionHandler.class/instance/select..st 3 0%
packages/IMAPClient-Core.package/ICConnectionHandler.class/instance/throwErrorIfAccountInvalid.st 7 0%
<!-- Total: 10 -->
Totals Coverage Status
Change from base Build 993052216: -0.5%
Covered Lines: 1132
Relevant Lines: 1452

💛 - Coveralls
elenagensch commented 3 years ago

We are still working on that, this adresses only #409

elenagensch commented 3 years ago

No it's not, we still need to find a solution to embed login in the update process

sm1lla commented 3 years ago

Is it correct that there are several accounts named 'testAccountName' under the account I'm logged in with?

laugengebaeck commented 3 years ago

Is it correct that there are several accounts named 'testAccountName' under the account I'm logged in with?

That's because I didn't tear down one of the new tests correctly (which is now fixed) :sweat_smile: So it shouldn't be related to this branch and you can just delete these accounts.