[JAMES-2960] /server tests should rely on MailImpl, not FakeMail

• Type: Improvement
• Status: Open
• Priority: Major
• Resolution: Unresolved
• Component/s: Mailet Contributions, tests
• Assignee: Unassigned
• Reporter: Benoit Tellier
• Created: 04/Nov/19 02:14
• Updated: 04/Nov/19 02:14

Description

While working on JAMES-2927 we tracked down some null values in Cassandra prepared statements. We found some possible null values present only within our tests but that production code completly prevents.

This was due to the use of FakeMail in tests and of MailImpl in production code.

Such diverging behaviours actually decreases the value of our test.

We should perform /server tests with MailImpl in order to minimize the gap between production and tests.