Vatavuk / excel-io

Object-oriented java Excel library
MIT License
81 stars 8 forks source link

Refactor tests #28

Closed mertserezli closed 6 years ago

mertserezli commented 6 years ago

Statements like MatcherAssert.assertThat can be changed to assertThat by using import static to improve readability and reduce line width.

Vatavuk commented 6 years ago

@mertserezli If we use assertThat without accompanied class a user will lose visibility from where this method comes from. It also introduces additional complexity if we want to use a method with the same name through some other third-party library. Although is more verbose, I think using class prefix is cleaner.