Setting \Seen flag doesnt seem to work

[aarsilv]

I have a dummy gmail address I created to write a unit test for a component that reads and processes email. Before the test I want to mark all emails read, and then specific emails unread. However unfortunately addFlags() does not seem to be working for me.

I may also be misunderstanding the documentation; if so an example would be very helpful!

Simplified example of what I'm trying:

async.auto({
    openInbox: [function(next) {
        imap.openBox('[Gmail]/All Mail', next);
    }],
    searchUnread: ['openInbox', function(next) {
        imap.search(['UNSEEN'], next);
    }],
    markAllRead: ['searchUnread', function(next, results) {
        var unreadUids = results.searchUnread;
        imap.addFlags(unreadUids, ['\\Seen'], next);
        /* also tried '\\SEEN' and 'SEEN' */
    }],
    markTestEmailsUnread: ['markAllRead', function(next) {
        imap.delFlags(testEmailUids, ['\\Seen'], next);
        /* note: testEmailUids is an array like [3,7] -- the UIDs of specific emails */
    }],
    closeInbox: ['markTestEmailsUnread', function(next) {
        imap.closeBox(next);
    }]
}, function(err) {
    eachNext(err);
});

The code executes fine without throwing or calling back any errors. However the emails are never marked as read in gmail.

[mscdex]

What does the debug output look like if you set debug: console.log in the constructor options object?

[aarsilv]

[connection] Connected to host
<= '* OK Gimap ready for requests from 166.147.81.141 dw16if41967272pad.189'
=> 'A0 CAPABILITY'
<= '* CAPABILITY IMAP4rev1 UNSELECT IDLE NAMESPACE QUOTA ID XLIST CHILDREN X-GM-EXT-1 XYZZY SASL-IR AUTH=XOAUTH AUTH=XOAUTH2 AUTH=PLAIN AUTH=PLAIN-CLIENTTOKEN'
<= 'A0 OK Thats all she wrote! dw16if41967272pad.189'
=> 'A1 LOGIN "<emailaddress>@gmail.com" "<password>"'
<= '* CAPABILITY IMAP4rev1 UNSELECT IDLE NAMESPACE QUOTA ID XLIST CHILDREN X-GM-EXT-1 UIDPLUS COMPRESS=DEFLATE ENABLE MOVE CONDSTORE ESEARCH'
<= 'A1 OK <emailaddress>@gmail.com <First> <Last> authenticated (Success)'
=> 'A2 NAMESPACE'
<= '* NAMESPACE (("" "/")) NIL NIL'
<= 'A2 OK Success'
=> 'A3 LIST "" ""'
<= '* LIST (\\Noselect) "/" "/"'
<= 'A3 OK Success'
=> 'A4 SELECT "[Gmail]/All Mail" (CONDSTORE)'
<= '* FLAGS (\\Answered \\Flagged \\Draft \\Deleted \\Seen)'
<= '* OK [PERMANENTFLAGS (\\Answered \\Flagged \\Draft \\Deleted \\Seen \\*)] Flags permitted.'
<= '* OK [UIDVALIDITY 11] UIDs valid.'
<= '* 6 EXISTS'
<= '* 0 RECENT'
<= '* OK [UIDNEXT 9] Predicted next UID.'
<= '* OK [HIGHESTMODSEQ 2217]'
<= 'A4 OK [READ-WRITE] [Gmail]/All Mail selected. (Success)'
=> 'IDLE IDLE'
=> DONE
<= '+ idling'
<= 'IDLE OK IDLE terminated (Success)'
=> 'A5 UID SEARCH UNSEEN'
<= '* SEARCH'
<= 'A5 OK SEARCH completed (Success)'
=> 'IDLE IDLE'
=> DONE
<= '+ idling'
<= 'IDLE OK IDLE terminated (Success)'
=> 'A6 UID STORE  +FLAGS.SILENT (\\Seen)'
<= 'A6 BAD Could not parse command'

[mscdex]

So it looks like your uid list (unreadUids) is empty. I've recently pushed changes to check for such cases so that the shown bad request is not sent.

[aarsilv]

You are right adding an if statement fixed it. It seems adding flags to the empty UID list was also interfering with removing the seen flags. Adding the if statement now results in the target emails being marked as unread.

    markAllRead: ['searchUnread', function(next, results) {
      var unreadUids = results.searchUnread;
      if (unreadUids.length) {
          imap.addFlags(unreadUids, ['\\Seen'], next);
      } else {
         next();
      }
  }],