search

TestRoots / watchdog

IntelliJ & Eclipse plugin for monitoring how Java applications are developed and tested
http://www.testroots.org
Other
19 stars 10 forks source link

Make Test naming consistent #296

Closed Inventitech closed 6 years ago

Inventitech commented 6 years ago

Transfer all test names into new naming schema introduced by @TimvdLippe.

TimvdLippe commented 6 years ago

This should already be addressed in #292 where I normalized all test names

TimvdLippe commented 6 years ago

https://github.com/TestRoots/watchdog/pull/292/commits/845b589759d4be8ae6e323bba5b9c3e5f61acd60 to be precise

Inventitech commented 6 years ago

:+1: Did you do this manually? I don't understand why some function names still have "test" in them.

TimvdLippe commented 6 years ago

I did do this manually yes. It only checks for snake_case, not specifically for the existence or absence of the word "test"

TimvdLippe commented 6 years ago

Per Mattermost, @inventitech wants to remove the test suffixes in the method names. Have to look at how the regex can enforce that, else it is a manual effort.