#129 "Test source Folder" Settings very sensitive...

[baer]

Hi,

I just struggled to set the settings for "Test source folder" as I put in "src/test/java/".
But it always wanted to create my new test class in "src/main/java".
On the first glance you can't tell whats wrong here. Until I saw it: A slash to much at the end. So this means that "src/test/java/" and also "/src/test/java" and of course "/src/test/java/" will be ignored.

I don't know if it would make trouble else where, but might it be possible to take that field not too sensitive? Someone might think the whole thing doesn't work properly :/

Greetz

PS: I wanted to chose >lowest< priority, but got confused by 1 or 9, as you can interprete a 9 as high or low ... If I got that wrong it needs to be changed to lowest priority please.

[Vera Henneberger]

Thank you for your feedback.
You are right without user feedback it is error-prone.
Perhaps a warning would be of help...

[baer]

What DID help was the tooltip showing: "examples: junit, test, src/test/java" and then I realized that I had a slash too much.
But first off I googled like half an hour why more unit might not work etc. So yeah, in my opinion there needs to be a better hint like a warning or a static text instead of / plus the tooltip directly mentioning the slashes. You'll find a nice way. Nice to hear so quick from you :)

[Vera Henneberger]

I added a validator for the text field.
The preference page now shows an error when the folder ends with a slash.

[Nicolas Demengel]

Alternatively, I was thinking that maybe we could check for leading/trailing slashes and automatically remove them. That way users would not be annoyed.

[Vera Henneberger]

A check for leading slashes should be added of course.
On my opinion I would not remove the slashes automatically and doing the magic behind without the users awareness. Perhaps someone has the intention to enter an absolute path (a bad idea of course) and will be confused why the leading slash is lost.
The extra work for the user to remove leading/trailing slashes isn't to much effort I think.