[GitHub] [james-project] woj-tek commented on pull request #1476: JAMES-3895 Automatically provision default (IMAP) mailboxes

["woj-tek (via GitHub)" <gi...@apache.org>]

woj-tek commented on PR #1476:
URL: https://github.com/apache/james-project/pull/1476#issuecomment-1467161360

   There is a problem with lots of tests failing (which I mentioned in the in-line comment) -  I checked the sources and if I get them right, it would seem that in order to fix those tests virtually all tests in `org.apache.james.mpt.imapmailbox` would have to be updated as they all rely on logging in. Alternatively I could set `imapConfiguration` in `ImapConfiguration` to `null` and do the null check in `org.apache.james.imap.processor.AbstractAuthProcessor#provisionInbox` but this would be just an ugly fix (workaround).
   
   Btw. is `apache-james-mpt-imapmailbox-core` package used anywhere or is it only for tests? It looks like testing code but it's in `main` instead of `test` directory.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@james.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@james.apache.org
For additional commands, e-mail: notifications-help@james.apache.org