I say unit tests are code like any other code. As such, they deserve to be refactored and kept clean.
I learned today that, by default, SonarLint for Eclipse disagrees. If you select Window->Preferences->Sonar Lint, you will see that "Test file regular expressions" contains "**/*Test.*,**/test/**/*":
For SonarLint to behave the way I want, which is to treat any file whose name ends in ".java" as code, I have to clear the regex.