Zms 90

[NickOvt]

1. Create, Update, Delete functions in mailboxes-handler.js updated to have better and more informative error output
2. Tests updated accordingly
3. mailboxes.js API file also updated and unnecessary parts are removed

[NickOvt]

IMAP Protocol integration tests
       CREATE
         should create new mailbox:
         
        Uncaught AssertionError: expected false to be true
        + expected - actual

        -false
        +true

This test seems to fail for some reason now, will investigate.
My idea is that probably because I changed the status codes (the wording) in the mailboxes-handler.js this test fails.

[andris9]

IMAP Protocol integration tests
       CREATE
         should create new mailbox:
         
        Uncaught AssertionError: expected false to be true
        + expected - actual

        -false
        +true

This test seems to fail for some reason now, will investigate. My idea is that probably because I changed the status codes (the wording) in the mailboxes-handler.js this test fails.

Uppercase error codes are special IMAP responses, and these must stay the same. Not sure what to do with these. Maybe create some kind of wrapper that normalizes IMAP error codes for more readable codes for the API responses 🤔

[NickOvt]

in file imap-core/lib/commands/create.js the function create always returns TEMPFAIL instead of the fail returned by the server, thus the given test fails. Any ideas on fix? If TEMPFAIL error response needs to stay then we can change the test appropriately OR make it that create returns the err.code

[NickOvt]

Also fixed imap-core create function so it returns correct error instead of hardcoded TEMPFAIL value