Typically, the Derby community prefers to work on multiple smaller patches, rather than a single larger patch.
Each patch should address a single problem. We agree with you that this makes the SVN history easier
So, in the cases that you identify, we'd prefer to address the test case refactoring separately
from the non-English locale issues, as two patches rather than one.
Similarly, we try to avoid doing things like removing unneeded "import" statements or changing whitespace
or formatting/indentation of the code as part of a patch. The Derby code unfortunately has a significant
number of whitespace issues, as it is a old and mature codebase that has been worked on by many
programmers over the years, so we just accept those problems and try to avoid those changes in our patches.
Meanwhile, Tiago, if you get a chance, can you confirm that this test now works properly for you in
a non-English locale? Then I think you could close this job?