Just some code improvements I've found while studying code in that area. It boils down to using more AssertJ features instead of hand-written code and removes public at some places (unnecessary with JUnit 5).
Comment From: snicoll
@neiser thanks for this. Going forward, please refrain from reformating the code. Running the build locally should also reveal checkstile violations. I've polished things up in baebf71
Comment From: neiser
@snicoll Thanks for reformatting. I hoped that the .editorconfig would have setup my IntelliJ correctly, but I'll run the build next time locally before pushing!
Comment From: snicoll
I hoped that the .editorconfig would have setup my IntelliJ correctly
It can't catch everything. If you look at your commit, it is obvious that import statements have moved and new lines have been removed, etc.