IMAPClient icon indicating copy to clipboard operation
IMAPClient copied to clipboard

Refactor tests related to ICEndpointCommandTest

Open JuliaJoch opened this issue 4 years ago • 1 comments

The tests of ICEndpointCommandTest and several other older Test Classes have so many test smells that we are overwhelmed by all the changes that would be necessary in order to make this readable.

For example there is code duplication for setting up the email objects that will be tested which could be refactored into the setup method. Also there are tests with a lot of assertions in one method.

JuliaJoch avatar Jul 31 '20 12:07 JuliaJoch

In addition the tearDown in ICEndpointCommandTest does not delete the folders which it builds

MrHenrik avatar Jul 31 '20 19:07 MrHenrik